Re: [PATCH] iio:adc:at91-sama5d2: make array startup_lookup static

2017-07-10 Thread Colin Ian King
On 10/07/17 21:59, Jonathan Cameron wrote: > On Mon, 10 Jul 2017 17:56:16 +0100 > Colin King wrote: > >> From: Colin Ian King >> >> Don't populate array startup_lookup on the stack but instead make >> it static. Makes the object code smaller. Also add in missing 'int' type >> specifier to clea

Re: [PATCH] iio:adc:at91-sama5d2: make array startup_lookup static

2017-07-10 Thread Jonathan Cameron
On Mon, 10 Jul 2017 17:56:16 +0100 Colin King wrote: > From: Colin Ian King > > Don't populate array startup_lookup on the stack but instead make > it static. Makes the object code smaller. Also add in missing 'int' type > specifier to clean up a checkpatch warning. > > Before: >text

[PATCH] iio:adc:at91-sama5d2: make array startup_lookup static

2017-07-10 Thread Colin King
From: Colin Ian King Don't populate array startup_lookup on the stack but instead make it static. Makes the object code smaller. Also add in missing 'int' type specifier to clean up a checkpatch warning. Before: textdata bss dec hex filename 102972800 128 13225

Re: [PATCH] iio:adc:at91-sama5d2: make array startup_lookup static to reduce code size

2017-07-09 Thread Jonathan Cameron
On Fri, 7 Jul 2017 17:08:35 +0100 Colin King wrote: > From: Colin Ian King > > Making the const array startup_lookup static rather having it on the > stack saves 69 bytes. Add in missing int specifier to clean up a > checkpatch warning. > > Before: >text data bss dec

[PATCH] iio:adc:at91-sama5d2: make array startup_lookup static to reduce code size

2017-07-07 Thread Colin King
From: Colin Ian King Making the const array startup_lookup static rather having it on the stack saves 69 bytes. Add in missing int specifier to clean up a checkpatch warning. Before: textdata bss dec hex filename 102972800 128 1322533a9 drivers/iio/adc/at91-sam