[U-Boot-Users] PXA27x USB device
Markus Klotzbücher
mk at denx.de
Fri May 4 12:20:37 CEST 2007
Hi Stefan,
Stefan Roese <sr at denx.de> writes:
> On Thursday 03 May 2007 09:49, Markus Klotzbücher wrote:
>> Rodolfo Giometti <giometti at enneenne.com> writes:
>> > I'm looking at drivers/usbdcore_mpc8xx.c for an implementation example
>> > but I don't understand why such support has been put there instead of
>> > cpu/mpc8xx directory.
>> >
>> > Can someone explain to me why? Should I put my PXA27x support also
>> > there?
>>
>> I have no idea why this is so, but it does seem wrong to me. Platform
>> specific code should go into the corresponding cpu/ directory, and so
>> should your pxa support.
>>
>> Any objections?
>
> We had a lengthy discussion about the right places for such drivers a few
> months ago. IIRC the consent was to put them into the drivers directory
> (please correct me, if my memory fails me here). But we wanted to structure
> this directly better. So I would suggest something like:
>
> drivers/usb/device/mpc8xx.c
> drivers/usb/device/pxa27x.c
>
> or even
>
> drivers/usb/gadget/mpc8xx.c
> drivers/usb/gadget/pxa27x.c
>
> to follow the Linux example a little more.
>
> Any comments?
Hmm. Seems reasonable, especially if this structure will be used for all
drivers. But is this the case? I still feel uncomfortable about placing
a cpu dependant host controller driver in a generic driver
directory. Will such a driver be maintained mainly by the USB Custodian
or the respective Architecture Custodian? I would assume the latter.
Best regards
Markus
--
DENX Software Engineering GmbH, HRB 165235 Munich, CEO: Wolfgang Denk
Office: Kirchenstr. 5, D-82194 Groebenzell, Germany
More information about the U-Boot
mailing list