Re: usb_nop_xceiv_register() missing when OTG built as modules

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

 



<me@xxxxxxxxxxxxxxx> writes:

> On Fri, 05 Mar 2010 09:59:52 -0800, Kevin Hilman
>> It would be my fault if I wrote those board files.  ;)
>
> sure, sorry. Maybe I didn't express myself really well.
>
>>> I suggest you keep usb as a module, but keep nop xceiv built-in.
>> 
>> Yes, that's the workaround I'm already using, but it is not a fix.
>> 
>> This dependency breaks the ability to build a minimal kernel with
>> everything as modules (allmodconfig)
>
> now this is a really good point. Haven't thought that way...
>
>> The nop xceiv needs a way for built-in code to register itself for
>> the cases when no xceiv is built as a module.
>
> when way to do that is not to use usb_nop_xceiv_register() and
> add the platform_device to the board-files. I never really liked
> the whole usb_nop_xceiv_register() hackery. Almost all board-files
> have a list of platform_devices which get added by platform_add_devices()
> anyways, so why not using that ??
>
> to me the usb_nop_xceiv_register() thing is just a lazy way of
> defining platform_devices. Specially one that takes some 3
> lines of code only.

Completely agree.  I didn't take the time to look into the USB code to
see if a simple plaform device creation would fix this.

If so, any chance of a patch for OMAP3EVM and 4430SDP?

I'll be happy to test on OMAP3EVM, but don't have a 4430.

Kevin

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux