Re: [PATCH] - e1000_ethtool.c - convert macros to functions

2007-11-01 Thread Kok, Auke
Joe Perches wrote: > Minimal macro to function conversion in e1000_ethtool.c > > Adds functions reg_pattern_test and reg_set_and_check > Changes REG_PATTERN_TEST and REG_SET_AND_CHECK macros > to call these functions. > > Saves ~2.5KB > > Compiled x86, untested (no hardware) > > old: > > $ siz

[PATCH] - e1000_ethtool.c - convert macros to functions

2007-11-01 Thread Joe Perches
Minimal macro to function conversion in e1000_ethtool.c Adds functions reg_pattern_test and reg_set_and_check Changes REG_PATTERN_TEST and REG_SET_AND_CHECK macros to call these functions. Saves ~2.5KB Compiled x86, untested (no hardware) old: $ size drivers/net/e1000/e1000_ethtool.o text

Re: [PATCH] - e1000_ethtool.c - convert macros to functions

2007-10-31 Thread Kok, Auke
Joe Perches wrote: > On Wed, 2007-10-31 at 14:30 -0700, Kok, Auke wrote: >> Joe Perches wrote: >> that's not a bad idea, however see below: >> can't we keep the macro here (and just make it call the function instead of >> expanding). the resulting code is much more lenghty and contains all these >

Re: [PATCH] - e1000_ethtool.c - convert macros to functions

2007-10-31 Thread Joe Perches
On Wed, 2007-10-31 at 14:30 -0700, Kok, Auke wrote: > Joe Perches wrote: > > Convert REG_PATTERN_TEST and REG_SET_AND_CHECK macros to functions > > Reduces x86 defconfig image by about 3k Convert REG_PATTERN_TEST and REG_SET_AND_CHECK macros to functions Reduces x86 defconfig image by about 3k co

Re: [PATCH] - e1000_ethtool.c - convert macros to functions

2007-10-31 Thread Joe Perches
On Wed, 2007-10-31 at 14:30 -0700, Kok, Auke wrote: > Joe Perches wrote: > that's not a bad idea, however see below: > can't we keep the macro here (and just make it call the function instead of > expanding). the resulting code is much more lenghty and contains all these > logic > traps that the p

Re: [PATCH] - e1000_ethtool.c - convert macros to functions

2007-10-31 Thread Kok, Auke
Joe Perches wrote: > Convert REG_PATTERN_TEST and REG_SET_AND_CHECK macros to functions > Reduces x86 defconfig image by about 3k > > compiled, untested (no hardware) > > Signed-off-by: Joe Perches <[EMAIL PROTECTED]> > > New: > > $ size vmlinux >textdata bss dec hex filenam

[PATCH] - e1000_ethtool.c - convert macros to functions

2007-10-31 Thread Joe Perches
Convert REG_PATTERN_TEST and REG_SET_AND_CHECK macros to functions Reduces x86 defconfig image by about 3k compiled, untested (no hardware) Signed-off-by: Joe Perches <[EMAIL PROTECTED]> New: $ size vmlinux textdata bss dec hex filename 4792735 490626 606208 5889569 59de21