regulator: lp8788-ldo: make array en_mask static const, makes object smaller
authorColin Ian King <colin.king@canonical.com>
Fri, 6 Sep 2019 13:06:32 +0000 (14:06 +0100)
committerMark Brown <broonie@kernel.org>
Mon, 9 Sep 2019 09:53:48 +0000 (10:53 +0100)
Don't populate the array en_mask on the stack but instead make it
static const. Makes the object code smaller by 87 bytes.

Before:
   text    data     bss     dec     hex filename
  12967    3408       0   16375    3ff7 drivers/regulator/lp8788-ldo.o

After:
   text    data     bss     dec     hex filename
  12816    3472       0   16288    3fa0 drivers/regulator/lp8788-ldo.o

(gcc version 9.2.1, amd64)

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Link: https://lore.kernel.org/r/20190906130632.6709-1-colin.king@canonical.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/lp8788-ldo.c

index 1b00f363899660d08926ac8ec02a83d7617283af..00e9bb92c326b223a769328713e533010f87bb3b 100644 (file)
@@ -464,7 +464,7 @@ static int lp8788_config_ldo_enable_mode(struct platform_device *pdev,
 {
        struct lp8788 *lp = ldo->lp;
        enum lp8788_ext_ldo_en_id enable_id;
-       u8 en_mask[] = {
+       static const u8 en_mask[] = {
                [EN_ALDO1]   = LP8788_EN_SEL_ALDO1_M,
                [EN_ALDO234] = LP8788_EN_SEL_ALDO234_M,
                [EN_ALDO5]   = LP8788_EN_SEL_ALDO5_M,