| Message ID | 20181113144805.1054-2-benjamin.gaignard@st.com |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | Make STM32 interrupt controller use hwspinlock | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | warning | "total: 0 errors, 1 warnings, 10 lines checked" |
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 >
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.
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...
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 --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 {
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(+)