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

["Satyam Sharma" <satyam.sharma@gmail.com>]

On 9/11/07, Adrian Bunk <bunk@kernel.org> 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@kernel.org>

Makes sense, I like the patch ...

Acked-by: Satyam Sharma <satyam@infradead.org>

> ---
>
>  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/
>