[0/3] make pin-controller use hwspinlock

Message ID 20181113095142.32015-1-benjamin.gaignard@st.com
Headers show
Series
  • make pin-controller use hwspinlock
Related show

Message

Benjamin Gaignard Nov. 13, 2018, 9:51 a.m.
This series allow to protect pin controller configuration registers
with a hwspinlock to avoid conflicting accesses between processors.

Benjamin Gaignard (3):
  dt-bindings: pinctrl: stm32: Document hwlocks properties
  pinctrl: stm32: protect configuration registers with a hwspinlock
  ARM: dts: stm32: Add hwlock for pinctrl

 .../bindings/pinctrl/st,stm32-pinctrl.txt          |  1 +
 arch/arm/boot/dts/stm32mp157-pinctrl.dtsi          |  1 +
 drivers/pinctrl/stm32/pinctrl-stm32.c              | 71 +++++++++++++++++++++-
 3 files changed, 72 insertions(+), 1 deletion(-)

-- 
2.15.0

Comments

Alexandre Torgue Nov. 13, 2018, 9:58 a.m. | #1
Hi Benjamin

On 11/13/18 10:51 AM, Benjamin Gaignard wrote:
> Define a hwspinlock to be used by pin-controller

> 

> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com>

> ---

>   arch/arm/boot/dts/stm32mp157-pinctrl.dtsi | 1 +

>   1 file changed, 1 insertion(+)

> 


In commit title please add for which SoC it is targeted. If you don't 
have to send new version for driver or dt-bindings, I will do it when 
I'll apply.

regards
Alex

> diff --git a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi

> index c4851271e810..2886e5a6ac27 100644

> --- a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi

> +++ b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi

> @@ -14,6 +14,7 @@

>   			ranges = <0 0x50002000 0xa400>;

>   			interrupt-parent = <&exti>;

>   			st,syscfg = <&exti 0x60 0xff>;

> +			hwlocks = <&hsem 0>;

>   			pins-are-numbered;

>   

>   			gpioa: gpio@50002000 {

>
Linus Walleij Nov. 19, 2018, 12:16 p.m. | #2
On Tue, Nov 13, 2018 at 10:51 AM Benjamin Gaignard
<benjamin.gaignard@linaro.org> wrote:

> Add hwlocks as optional property

>

> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com>


Patch applied.

Yours,
Linus Walleij