• Gustavo A. R. Silva's avatar
    pinctrl: nomadik: abx500: constify gpio_chip structure · d3761023
    Gustavo A. R. Silva authored
    This structure is only used to copy into another structure, so declare
    it as const.
    
    This issue was detected using Coccinelle and the following semantic patch:
    
    @r disable optional_qualifier@
    identifier i;
    position p;
    @@
    static struct gpio_chip i@p = { ... };
    
    @ok@
    identifier r.i;
    expression e;
    position p;
    @@
    e = i@p;
    
    @bad@
    position p != {r.p,ok.p};
    identifier r.i;
    struct gpio_chip e;
    @@
    e@i@p
    
    @depends on !bad disable optional_qualifier@
    identifier r.i;
    @@
    static
    +const
     struct gpio_chip i = { ... };
    
    In the following log you can see a significant difference in the code size
    and data segment, hence in the dec segment. This log is the output
    of the size command, before and after the code change:
    
    before:
       text    data     bss     dec     hex filename
      17545    5376       0   22921    5989 drivers/pinctrl/nomadik/pinctrl-abx500.o
    
    after:
                bss     dec     hex filename
      17273    5320       0   22593    5841 drivers/pinctrl/nomadik/pinctrl-abx500.o
    Signed-off-by: default avatarGustavo A. R. Silva <garsilva@embeddedor.com>
    Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
    d3761023
pinctrl-abx500.c 31.6 KB