Re: [PATCH resend 1/3] AM35x: Add musb support

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

 



Hello.

Tony Lindgren wrote:

  I think Ajay has explained why it's needed. The option is
necessary in one or another form.

It's not needed for omaps, we can already build in support for
omap2, omap3 and omap4 into the same kernel binary.

   Don't see how it hinders that.

   Not with AM35x USB support merged -- at least you won't be able
to build single kernel with monolithic MUSB support.

Right. I believe musb is pretty much the only remaining driver
that won't behave with multi-omap. But let's not merge code that
would make fixing that even harder.

   I don't see how it's making fixing this harder... (though it's already hard).

If a Kconfig option is needed for optionally compiling in the support
for am35x musb, it should be called USB_MUSB_AM35X or similar that
gets selected if the boards using it are selected.

   Do you mean that we should have this option in

drivers/usb/musb/Kconfig?

Yeah, it could be set automatically with default y if
MACH_AM35X_SOME_BOARD.

Then options like this should not be mutually exclusive like they
currently are for musb, that breaks using musb with multi omap.

Choosing USB_MUSB_AM35X would anyways compile am35x.c and not omap2430.c
File; thus musb would not work on OMAP3x boards with same binary.

You should set up things so both can be compiled in, that's standard
behaviour for all Linux drivers :)

  This is much easier said than done.

I will update the patches and submit for further reviews.

Thanks,
Tony

WBR, Sergei
--
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