[2.6 patch] drivers/net/smc-mca.c: cleanups

2005-03-27 Thread Adrian Bunk
This patch contains the following cleanups: - make a needlessly global function static - make three needlessly global structs static const Since after moving the now-static stucts to smc-mca.c the file smc-mca.h was empty except for two #define's, I've also killed the rest of smc-mca.h . Signed

[2.6 patch] drivers/net/smc-mca.c: cleanups

2005-03-12 Thread Adrian Bunk
This patch contains the following cleanups: - make a needlessly global function static - make three needlessly global structs static const Since after moving the now-static stucts to smc-mca.c the file smc-mca.h was empty except for two #define's, I've also killed the rest of smc-mca.h . Signed

[2.6 patch] drivers/net/smc-mca.c: cleanups

2005-02-21 Thread Adrian Bunk
On Sun, Feb 20, 2005 at 08:26:42PM -0500, Jeff Garzik wrote: > these tables should be const-ified Updated patch: <-- snip --> This patch contains the following cleanups: - make a needlessly global function static - make three needlessly global structs static const Since after moving the n

Re: [2.6 patch] drivers/net/smc-mca.c: cleanups

2005-02-20 Thread Jeff Garzik
Adrian Bunk wrote: This patch contains the following cleanups: - make a needlessly global function static - make three needlessly global structs static Since after moving the now-static stucts to smc-mca.c the file smc-mca.h was empty except for two #define's, I've also killed the rest of smc-mca

Re: [2.6 patch] drivers/net/smc-mca.c: cleanups

2005-02-20 Thread Herbert Xu
Arjan van de Ven <[EMAIL PROTECTED]> wrote: > >> I've used this technique in a few very >> small programs to reduce their size (I could strip off both their bss and >> data sections to save space). Also, I believe that the compiler is able >> to optimize code using consts, but this is pure specula

Re: [2.6 patch] drivers/net/smc-mca.c: cleanups

2005-02-19 Thread Arjan van de Ven
> > > This comment is applicable to similar changes, also. Use 'const' > > > whenever possible. > > > > does that even have meaning in C? In C++ it does, but afaik in C it > > doesn't. > > Yes it does. Often the variables declared this way will go into the text > section which is marked read-o

Re: [2.6 patch] drivers/net/smc-mca.c: cleanups

2005-02-19 Thread Willy Tarreau
On Sat, Feb 19, 2005 at 10:09:00AM +0100, Arjan van de Ven wrote: > On Sat, 2005-02-19 at 03:41 -0500, Jeff Garzik wrote: > > Adrian Bunk wrote: > > > This patch contains the following cleanups: > > > - make a needlessly global function static > > > - make three needlessly global structs static > >

Re: [2.6 patch] drivers/net/smc-mca.c: cleanups

2005-02-19 Thread Arjan van de Ven
On Sat, 2005-02-19 at 03:41 -0500, Jeff Garzik wrote: > Adrian Bunk wrote: > > This patch contains the following cleanups: > > - make a needlessly global function static > > - make three needlessly global structs static > > > > Since after moving the now-static stucts to smc-mca.c the file smc-mca

Re: [2.6 patch] drivers/net/smc-mca.c: cleanups

2005-02-19 Thread Jeff Garzik
Arjan van de Ven wrote: On Sat, 2005-02-19 at 03:41 -0500, Jeff Garzik wrote: Adrian Bunk wrote: This patch contains the following cleanups: - make a needlessly global function static - make three needlessly global structs static Since after moving the now-static stucts to smc-mca.c the file smc-mc

Re: [2.6 patch] drivers/net/smc-mca.c: cleanups

2005-02-19 Thread Jeff Garzik
Adrian Bunk wrote: This patch contains the following cleanups: - make a needlessly global function static - make three needlessly global structs static Since after moving the now-static stucts to smc-mca.c the file smc-mca.h was empty except for two #define's, I've also killed the rest of smc-mca

[2.6 patch] drivers/net/smc-mca.c: cleanups

2005-02-19 Thread Adrian Bunk
This patch contains the following cleanups: - make a needlessly global function static - make three needlessly global structs static Since after moving the now-static stucts to smc-mca.c the file smc-mca.h was empty except for two #define's, I've also killed the rest of smc-mca.h . Signed-off-b