Barry Song <21cnbao@xxxxxxxxx> writes: > +if USB_CHIPIDEA_UDC && USB_CHIPIDEA_HOST > + > +config USB_CHIPIDEA_SIRF > + depends on ARCH_SIRF > + bool "SiRF USB controller ChipIdea driver binding" > + default y > + help > + Say Y here to enable sirf usb ChipIdea driver binding. > + > +config USB_CHIPIDEA_MSM > + depends on ARCH_MSM > + bool "MSM USB controller ChipIdea driver binding" > + default y > + help > + Say Y here to enable msm usb ChipIdea driver binding. > + > +config USB_CHIPIDEA_IMX > + depends on ARCH_MXC || ARCH_MXS > + bool "i.MX USB controller ChipIdea driver binding" > + default y > + help > + Say Y here to enable imx usb ChipIdea driver binding. > + > +endif No. This is wrong on at least 3 different levels. > + > endif > diff --git a/drivers/usb/chipidea/Makefile b/drivers/usb/chipidea/Makefile > index 4ab83e9..7004fde 100644 > --- a/drivers/usb/chipidea/Makefile > +++ b/drivers/usb/chipidea/Makefile > @@ -9,7 +9,7 @@ ci_hdrc-$(CONFIG_USB_CHIPIDEA_DEBUG) += debug.o > > # Glue/Bridge layers go here > > -obj-$(CONFIG_USB_CHIPIDEA) += ci13xxx_msm.o > +obj-$(CONFIG_USB_CHIPIDEA_MSM) += ci13xxx_msm.o No. > # PCI doesn't provide stubs, need to check > ifneq ($(CONFIG_PCI),) > @@ -17,5 +17,6 @@ ifneq ($(CONFIG_PCI),) > endif > > ifneq ($(CONFIG_OF_DEVICE),) > - obj-$(CONFIG_USB_CHIPIDEA) += ci13xxx_imx.o usbmisc_imx.o > + obj-$(CONFIG_USB_CHIPIDEA_IMX) += ci13xxx_imx.o usbmisc_imx.o > + obj-$(CONFIG_USB_CHIPIDEA_SIRF) += ci13xxx_sirf.o > endif No. Regards, -- Alex -- 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