Re: [PATCH v1 13/27] usb: musb: simplify musb_ep_select&MUSB_EP_OFFSET for different hardware

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux