Re: [2.6 patch] always visit drivers/usb/misc/

From: Satyam Sharma
Date: Tue Sep 11 2007 - 14:36:25 EST


On 9/11/07, Adrian Bunk <bunk@xxxxxxxxxx> wrote:
> On Tue, Sep 11, 2007 at 04:06:21PM +0200, Toralf Förster wrote:
> > Hello,
> >
> > the build with the attached .config failed, make ends with:
> > ...
> > AS arch/i386/boot/compressed/head.o
> > CC arch/i386/boot/compressed/misc.o
> > OBJCOPY arch/i386/boot/compressed/vmlinux.bin
> > GZIP arch/i386/boot/compressed/vmlinux.bin.gz
> > LD arch/i386/boot/compressed/piggy.o
> > LD arch/i386/boot/compressed/vmlinux
> > OBJCOPY arch/i386/boot/vmlinux.bin
> > HOSTCC arch/i386/boot/tools/build
> > BUILD arch/i386/boot/bzImage
> > Root device is (3, 8)
> > Setup is 10456 bytes (padded to 10752 bytes).
> > System is 2531 kB
> > Kernel: arch/i386/boot/bzImage is ready (#1)
> > Building modules, stage 2.
> > MODPOST 180 modules
> > ERROR: "phidget_class" [drivers/usb/misc/phidgetservo.ko] undefined!
> > make[1]: *** [__modpost] Error 1
> > make: *** [modules] Error 2
> >...
>
> The patch below fixes this constant source of problems.
>
> <-- snip -->
>
> Currently, every driver under drivers/usb/misc/ also has to be listed in
> drivers/usb/Makefile. This has been forgotten more than once, and this
> patch changes drivers/usb/Makefile to simply always visit
> drivers/usb/misc/ when building the USB code.
>
> Signed-off-by: Adrian Bunk <bunk@xxxxxxxxxx>

Makes sense, I like the patch ...

Acked-by: Satyam Sharma <satyam@xxxxxxxxxxxxx>

> ---
>
> drivers/usb/Makefile | 22 +---------------------
> 1 file changed, 1 insertion(+), 21 deletions(-)
>
> e03ef8eb70600ef876dda2cc30814e25b7d7fb07
> diff --git a/drivers/usb/Makefile b/drivers/usb/Makefile
> index ac49b15..516a640 100644
> --- a/drivers/usb/Makefile
> +++ b/drivers/usb/Makefile
> @@ -28,27 +28,7 @@ obj-$(CONFIG_USB_MICROTEK) += image/
>
> obj-$(CONFIG_USB_SERIAL) += serial/
>
> -obj-$(CONFIG_USB_ADUTUX) += misc/
> -obj-$(CONFIG_USB_APPLEDISPLAY) += misc/
> -obj-$(CONFIG_USB_AUERSWALD) += misc/
> -obj-$(CONFIG_USB_BERRY_CHARGE) += misc/
> -obj-$(CONFIG_USB_CYPRESS_CY7C63)+= misc/
> -obj-$(CONFIG_USB_CYTHERM) += misc/
> -obj-$(CONFIG_USB_EMI26) += misc/
> -obj-$(CONFIG_USB_EMI62) += misc/
> -obj-$(CONFIG_USB_FTDI_ELAN) += misc/
> -obj-$(CONFIG_USB_IDMOUSE) += misc/
> -obj-$(CONFIG_USB_LCD) += misc/
> -obj-$(CONFIG_USB_LD) += misc/
> -obj-$(CONFIG_USB_LED) += misc/
> -obj-$(CONFIG_USB_LEGOTOWER) += misc/
> -obj-$(CONFIG_USB_PHIDGETSERVO) += misc/
> -obj-$(CONFIG_USB_RIO500) += misc/
> -obj-$(CONFIG_USB_SISUSBVGA) += misc/
> -obj-$(CONFIG_USB_TEST) += misc/
> -obj-$(CONFIG_USB_TRANCEVIBRATOR)+= misc/
> -obj-$(CONFIG_USB_USS720) += misc/
> -obj-$(CONFIG_USB_IOWARRIOR) += misc/
> +obj-$(CONFIG_USB) += misc/
>
> obj-$(CONFIG_USB_ATM) += atm/
> obj-$(CONFIG_USB_SPEEDTOUCH) += atm/
>
-
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/