mbox series

[0/2] pinctrl: bcm: Constify static ops structs

Message ID 20210605185908.39982-1-rikard.falkeborn@gmail.com
Headers show
Series pinctrl: bcm: Constify static ops structs | expand

Message

Rikard Falkeborn June 5, 2021, 6:59 p.m. UTC 
Constify a number of pinctrl_ops and pinmux_ops that are never modified
to allow the compiler to put them in read-only memory.

Rikard Falkeborn (2):
  pinctrl: bcm: Constify static pinctrl_ops
  pinctrl: bcm: Constify static pinmux_ops

 drivers/pinctrl/bcm/pinctrl-bcm6318.c  | 4 ++--
 drivers/pinctrl/bcm/pinctrl-bcm63268.c | 4 ++--
 drivers/pinctrl/bcm/pinctrl-bcm6328.c  | 4 ++--
 drivers/pinctrl/bcm/pinctrl-bcm6358.c  | 4 ++--
 drivers/pinctrl/bcm/pinctrl-bcm6362.c  | 4 ++--
 drivers/pinctrl/bcm/pinctrl-bcm6368.c  | 4 ++--
 drivers/pinctrl/bcm/pinctrl-bcm63xx.h  | 4 ++--
 7 files changed, 14 insertions(+), 14 deletions(-)

Comments

Florian Fainelli June 6, 2021, 3:35 a.m. UTC | #1 
On 6/5/2021 11:59 AM, Rikard Falkeborn wrote:
> These are only assigned, either directly or via the bcm63xx_pinctrl_soc

> struct, to the pctlops field in the pinctrl_desc struct and never

> modified, so make them const to allow the compiler to put them in

> read-only memory.

> 

> Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com>


Acked-by: Florian Fainelli <f.fainelli@gmail.com>

-- 
Florian
Linus Walleij June 9, 2021, 9:18 a.m. UTC | #2 
On Sat, Jun 5, 2021 at 8:59 PM Rikard Falkeborn
<rikard.falkeborn@gmail.com> wrote:

> Constify a number of pinctrl_ops and pinmux_ops that are never modified

> to allow the compiler to put them in read-only memory.


Patches applied!

Yours,
Linus Walleij