diff mbox series

[1/3] dt-bindings: interrupt-controller: stm32: Document hwlock properties

Message ID 20181113144805.1054-2-benjamin.gaignard@st.com (mailing list archive)
State New, archived
Headers show
Series Make STM32 interrupt controller use hwspinlock | expand

Commit Message

Benjamin Gaignard Nov. 13, 2018, 2:48 p.m. UTC
Add hwlocks as optional property

Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com>
---
 .../devicetree/bindings/interrupt-controller/st,stm32-exti.txt        | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Rob Herring (Arm) Dec. 4, 2018, 8:47 p.m. UTC | #1
On Tue, Nov 13, 2018 at 03:48:03PM +0100, Benjamin Gaignard wrote:
> Add hwlocks as optional property
> 
> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com>

Should be linaro.org S-o-b?

> ---
>  .../devicetree/bindings/interrupt-controller/st,stm32-exti.txt        | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/interrupt-controller/st,stm32-exti.txt b/Documentation/devicetree/bindings/interrupt-controller/st,stm32-exti.txt
> index 6a36bf66d932..cd01b2292ec6 100644
> --- a/Documentation/devicetree/bindings/interrupt-controller/st,stm32-exti.txt
> +++ b/Documentation/devicetree/bindings/interrupt-controller/st,stm32-exti.txt
> @@ -14,6 +14,10 @@ Required properties:
>    (only needed for exti controller with multiple exti under
>    same parent interrupt: st,stm32-exti and st,stm32h7-exti)
>  
> +Optional properties:
> +
> +- hwlocks: reference to a phandle of a hardware spinlock provider node.
> +
>  Example:
>  
>  exti: interrupt-controller@40013c00 {
> -- 
> 2.15.0
>
Marc Zyngier Dec. 13, 2018, 2:23 p.m. UTC | #2
On 04/12/2018 20:47, Rob Herring wrote:
> On Tue, Nov 13, 2018 at 03:48:03PM +0100, Benjamin Gaignard wrote:
>> Add hwlocks as optional property
>>
>> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com>
> 
> Should be linaro.org S-o-b?

What is the status on this? I'm not going to queue it if there is a
doubt on the SoB...

Thanks,

	M.
Benjamin Gaignard Dec. 13, 2018, 2:55 p.m. UTC | #3
Le jeu. 13 déc. 2018 à 15:23, Marc Zyngier <marc.zyngier@arm.com> a écrit :
>
> On 04/12/2018 20:47, Rob Herring wrote:
> > On Tue, Nov 13, 2018 at 03:48:03PM +0100, Benjamin Gaignard wrote:
> >> Add hwlocks as optional property
> >>
> >> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com>
> >
> > Should be linaro.org S-o-b?
>
> What is the status on this? I'm not going to queue it if there is a
> doubt on the SoB...

I will fix that in version 2 (even if I don't really understand the issue)

Benjamin
>
> Thanks,
>
>         M.
> --
> Jazz is not dead. It just smells funny...
Marc Zyngier Dec. 13, 2018, 5:19 p.m. UTC | #4
On Thu, 13 Dec 2018 14:55:06 +0000,
Benjamin Gaignard <benjamin.gaignard@linaro.org> wrote:
> 
> Le jeu. 13 déc. 2018 à 15:23, Marc Zyngier <marc.zyngier@arm.com> a écrit :
> >
> > On 04/12/2018 20:47, Rob Herring wrote:
> > > On Tue, Nov 13, 2018 at 03:48:03PM +0100, Benjamin Gaignard wrote:
> > >> Add hwlocks as optional property
> > >>
> > >> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com>
> > >
> > > Should be linaro.org S-o-b?
> >
> > What is the status on this? I'm not going to queue it if there is a
> > doubt on the SoB...
> 
> I will fix that in version 2 (even if I don't really understand the
> issue)

It depends which hat you had on at the time you wrote the patches.

Thanks,

	M.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/interrupt-controller/st,stm32-exti.txt b/Documentation/devicetree/bindings/interrupt-controller/st,stm32-exti.txt
index 6a36bf66d932..cd01b2292ec6 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/st,stm32-exti.txt
+++ b/Documentation/devicetree/bindings/interrupt-controller/st,stm32-exti.txt
@@ -14,6 +14,10 @@  Required properties:
   (only needed for exti controller with multiple exti under
   same parent interrupt: st,stm32-exti and st,stm32h7-exti)
 
+Optional properties:
+
+- hwlocks: reference to a phandle of a hardware spinlock provider node.
+
 Example:
 
 exti: interrupt-controller@40013c00 {