On Tue, Oct 12, 2004 at 04:25:09PM +0200, Adrian Bunk wrote:
> On Mon, Oct 11, 2004 at 03:25:02AM -0700, Andrew Morton wrote:
> >...
> > All 741 patches
> >...
> > bk-usb.patch
> >...
> 
> 
> This removes an #ifdef CONFIG_PROC_FS from drivers/usb/core/inode.c 
> which is required, since now compilation fails with CONFIG_PROC_FS=n:
> 
> 
> <--  snip  -->
> 
> ...
>   CC      drivers/usb/core/inode.o
> drivers/usb/core/inode.c: In function `usbfs_init':
> drivers/usb/core/inode.c:750: `proc_bus' undeclared (first use in this function)
> drivers/usb/core/inode.c:750: (Each undeclared identifier is reported only once
> drivers/usb/core/inode.c:750: for each function it appears in.)
> make[3]: *** [drivers/usb/core/inode.o] Error 1
> 
> <--  snip  -->
> 
> 
> The fix is simple:
> 
> 
> Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]>
> 
> --- linux-2.6.9-rc4-mm1-full/drivers/usb/core/inode.c.old     2004-10-12 
> 16:20:54.000000000 +0200
> +++ linux-2.6.9-rc4-mm1-full/drivers/usb/core/inode.c 2004-10-12 16:23:53.000000000 
> +0200
> @@ -746,8 +746,10 @@
>               return retval;
>       }
>  
> +#ifdef CONFIG_PROC_FS
>       /* create mount point for usbfs */
>       usbdir = proc_mkdir("usb", proc_bus);
> +#endif

No, the proper fix is to make the proc_fs.h header file give an empty
definition for proc_bus, so we don't have to have unneeded #ifdefs all
over the place.

thanks,

greg k-h


-------------------------------------------------------
This SF.net email is sponsored by: IT Product Guide on ITManagersJournal
Use IT products in your business? Tell us what you think of them. Give us
Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more
http://productguide.itmanagersjournal.com/guidepromo.tmpl
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to