mbox series

[0/3] pinctrl/actions: Constify static variables

Message ID 20200830224311.36994-1-rikard.falkeborn@gmail.com
Headers show
Series pinctrl/actions: Constify static variables | expand

Message

Rikard Falkeborn Aug. 30, 2020, 10:43 p.m. UTC
Constify a couple of static variables which are not modified to allow
the compiler to put them in read-only memory. Patch 1/3 is probably
the most important one since those structs contain function pointer.
The patches are independent, and can be applied in any order. 
Compile-tested only.

Rikard Falkeborn (3):
  pinctrl: actions: pinctrl-owl: Constify owl_pinctrl_ops and
    owl_pinmux_ops
  pinctrl: actions: pinctrl-s700: Constify s700_padinfo[]
  pinctrl: actions: pinctrl-s900: Constify s900_padinfo[]

 drivers/pinctrl/actions/pinctrl-owl.c  | 4 ++--
 drivers/pinctrl/actions/pinctrl-s700.c | 2 +-
 drivers/pinctrl/actions/pinctrl-s900.c | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

Comments

Manivannan Sadhasivam Aug. 31, 2020, 9:15 a.m. UTC | #1
On 0831, Rikard Falkeborn wrote:
> Constify a couple of static variables which are not modified to allow
> the compiler to put them in read-only memory. Patch 1/3 is probably
> the most important one since those structs contain function pointer.
> The patches are independent, and can be applied in any order. 
> Compile-tested only.
> 

For the series,

Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>

Thanks,
Mani

> Rikard Falkeborn (3):
>   pinctrl: actions: pinctrl-owl: Constify owl_pinctrl_ops and
>     owl_pinmux_ops
>   pinctrl: actions: pinctrl-s700: Constify s700_padinfo[]
>   pinctrl: actions: pinctrl-s900: Constify s900_padinfo[]
> 
>  drivers/pinctrl/actions/pinctrl-owl.c  | 4 ++--
>  drivers/pinctrl/actions/pinctrl-s700.c | 2 +-
>  drivers/pinctrl/actions/pinctrl-s900.c | 2 +-
>  3 files changed, 4 insertions(+), 4 deletions(-)
> 
> -- 
> 2.28.0
>
Linus Walleij Sept. 12, 2020, 10:15 a.m. UTC | #2
On Mon, Aug 31, 2020 at 12:43 AM Rikard Falkeborn
<rikard.falkeborn@gmail.com> wrote:

> Constify a couple of static variables which are not modified to allow
> the compiler to put them in read-only memory. Patch 1/3 is probably
> the most important one since those structs contain function pointer.
> The patches are independent, and can be applied in any order.
> Compile-tested only.
>
> Rikard Falkeborn (3):
>   pinctrl: actions: pinctrl-owl: Constify owl_pinctrl_ops and
>     owl_pinmux_ops
>   pinctrl: actions: pinctrl-s700: Constify s700_padinfo[]
>   pinctrl: actions: pinctrl-s900: Constify s900_padinfo[]

Patch series applied,
thanks!

Yours,
Linus Walleij