Messages in this thread Patch in this message | | | Date | Fri, 10 Oct 2003 16:06:29 -0700 (PDT) | From | "Noah J. Misch" <> | Subject | Re: patches for PROC_FS=n (2.6.0-test7) |
| |
> drivers/char/toshiba.c and > net/atm/clip.c don't build if PROC_FS=n.
> Patches for them are available at:
> http://developer.osdl.org/rddunlap/patches/toshiba_inline_260t7.patch
Consider the patch at the bottom of this message instead. The tosh_get_info function is really internal to drivers/char/toshiba.c, so instead I changed it to static in that file and completely removed the prototype in toshiba.h.
BTW, both your patch (I think) and mine also solve a duplicate symbol problem - see the ChangeSet log for details.
> http://developer.osdl.org/rddunlap/patches/atmprocfs_260t7.patch
Cool.
> There are several other drivers/protocols that don't build > with PROC_FS=n, like arlan, siimage, ipx, llc, and bluetooth.
I put in a patch for ipx yesterday, and I have patches for siimage and llc pretty much ready to go. I noticed the others but I have not done much.
Thanks, Noah
Toshiba patch:
# This is a BitKeeper generated patch for the following project: # Project Name: Linux kernel tree # This patch format is intended for GNU patch command version 2.5 or higher. # This patch includes the following deltas: # ChangeSet 1.1344 -> 1.1345 # include/linux/toshiba.h 1.3 -> 1.4 # drivers/char/toshiba.c 1.11 -> 1.12 # # The following is the BitKeeper ChangeSet Log # -------------------------------------------- # 03/10/10 noah@caltech.edu 1.1345 # Remove the prototypes for tosh_get_info from include/linux/toshiba.h and # make the function's definition in drivers/char/toshiba.c static. This # function is specific to toshiba.c, so no other file needs the prototype. # # This allows drivers/char/toshiba.c to compile with CONFIG_PROC_FS=n and # allows one to link more than one driver that includes toshiba.h into the # kernel at the same time without a multiple declaration; for example, both # drivers/char/toshiba.c and drivers/video/neofb.c. # -------------------------------------------- # diff -Nru a/drivers/char/toshiba.c b/drivers/char/toshiba.c --- a/drivers/char/toshiba.c Fri Oct 10 18:50:17 2003 +++ b/drivers/char/toshiba.c Fri Oct 10 18:50:17 2003 @@ -292,7 +292,7 @@ * Print the information for /proc/toshiba */ #ifdef CONFIG_PROC_FS -int tosh_get_info(char *buffer, char **start, off_t fpos, int length) +static int tosh_get_info(char *buffer, char **start, off_t fpos, int length) { char *temp; int key; diff -Nru a/include/linux/toshiba.h b/include/linux/toshiba.h --- a/include/linux/toshiba.h Fri Oct 10 18:50:17 2003 +++ b/include/linux/toshiba.h Fri Oct 10 18:50:17 2003 @@ -33,13 +33,4 @@ unsigned int edi __attribute__ ((packed)); } SMMRegisters;
-#ifdef CONFIG_PROC_FS -static int tosh_get_info(char *, char **, off_t, int); -#else /* !CONFIG_PROC_FS */ -inline int tosh_get_info(char *buffer, char **start, off_t fpos, int lenght) -{ - return 0; -} -#endif /* CONFIG_PROC_FS */ - #endif - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |