Hi, 2010/12/23 Sergei Shtylyov <sshtylyov@xxxxxxxxxx>: > Hello. > > Ming Lei wrote: > >>>> From: Ming Lei <tom.leiming@xxxxxxxxx> >>>> In fact, for tusb, musb_ep_select and MUSB_EP_OFFSET are same with >>>> that for indexed mapping, so merge the two cases to simplify >>>> definition of the two helpers. >>>> Signed-off-by: Ming Lei <tom.leiming@xxxxxxxxx> > >>> [...] > >>>> diff --git a/drivers/usb/musb/musb_regs.h b/drivers/usb/musb/musb_regs.h >>>> index edc0cca..3321bae 100644 >>>> --- a/drivers/usb/musb/musb_regs.h >>>> +++ b/drivers/usb/musb/musb_regs.h >>>> @@ -292,9 +292,6 @@ >>>> #define MUSB_FLAT_OFFSET(_epnum, _offset) \ >>>> (0x100 + (0x10*(_epnum)) + (_offset)) >>>> -/* TUSB6010 EP0 configuration register is special */ > >>> Hm, not sure if it's worth dropping this comment... > >> I think it is worth, please see the commit log. > >> Tusb takes indexed mapping about musb_ep_select and >> MUSB_EP_OFFSET, so it should not be special, should it? > > One register, TUSB_EP0_CONF is special. "tusb6010.h" is later included > because of it. Also, see the code in musb_core.c. Yes, but the comment should not be put before MUSB_TUSB_OFFSET, so I deleted it. You should have mentioned it a little sooner, this can be added into v2, but seems too late to do it now, :-). -- Lei Ming -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html