Messages in this thread Patch in this message | | | Date | Mon, 18 Dec 2017 17:21:34 +0100 | From | Boris Brezillon <> | Subject | Re: [PATCH] mtd: mtdswap: make array 'name' static const, shrinks object size |
| |
On Tue, 28 Nov 2017 18:59:46 +0000 Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com> > > Don't populate the read-only array 'name' on the stack but instead > make it static and constify it. Makes the object code smaller by 35 > bytes: > > Before: > text data bss dec hex filename > 26304 4444 352 31100 797c drivers/mtd/mtdswap.o > > After: > text data bss dec hex filename > 26205 4508 352 31065 7959 drivers/mtd/mtdswap.o > > (gcc version 7.2.0 x86_64) > > Signed-off-by: Colin Ian King <colin.king@canonical.com>
Applied.
Thanks,
Boris
> --- > drivers/mtd/mtdswap.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/mtd/mtdswap.c b/drivers/mtd/mtdswap.c > index f07492c6f4b2..7eb0e1f4f980 100644 > --- a/drivers/mtd/mtdswap.c > +++ b/drivers/mtd/mtdswap.c > @@ -1223,8 +1223,9 @@ static int mtdswap_show(struct seq_file *s, void *data) > unsigned int max[MTDSWAP_TREE_CNT]; > unsigned int i, cw = 0, cwp = 0, cwecount = 0, bb_cnt, mapped, pages; > uint64_t use_size; > - char *name[] = {"clean", "used", "low", "high", "dirty", "bitflip", > - "failing"}; > + static const char * const name[] = { > + "clean", "used", "low", "high", "dirty", "bitflip", "failing" > + }; > > mutex_lock(&d->mbd_dev->lock); >
| |