Re: [PATCH] usb: phy: Fix Kconfig dependency for Phy drivers

[Alexandre Pereira da Silva]

On Mon, Jul 2, 2012 at 4:24 AM, Felipe Balbi <balbi@xxxxxx> wrote:
> Hi,
>
> On Tue, Jun 26, 2012 at 11:56:48AM -0300, Alexandre Pereira da Silva wrote:
>> USB phy layer driver are only built if usb host is selected, but they
>> are used too by USB_GADGET drivers
>>
>> Signed-off-by: Alexandre Pereira da Silva <aletes.xgr@xxxxxxxxx>
>> ---
>>  drivers/usb/Makefile    |    2 +-
>>  drivers/usb/phy/Kconfig |    4 ++--
>>  2 files changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/usb/Makefile b/drivers/usb/Makefile
>> index c691eea..f5ed3d7 100644
>> --- a/drivers/usb/Makefile
>> +++ b/drivers/usb/Makefile
>> @@ -46,7 +46,7 @@ obj-$(CONFIG_USB_MICROTEK)  += image/
>>  obj-$(CONFIG_USB_SERIAL)     += serial/
>>
>>  obj-$(CONFIG_USB)            += misc/
>> -obj-$(CONFIG_USB)            += phy/
>> +obj-$(CONFIG_USB_COMMON)     += phy/
>
> I'm not sure this is the best solution. Should we introduce
> CONFIG_USB_PHY for this ?

In my opinion, the right solution is to move the ISP1301 to the otg
dir, where it belongs and merge the duplicated code from lpc32xx_udc
and ohxi-nxp to it.

I suggest to leave this as is for now, as it's preventing the driver
to build in some configurations. Later, we can try to move this to the
right place.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/