* [PATCH 3/3] ARM: mvebu: dts: connect interrupt for WD on armada-38x [not found] <20171011022958.31268-1-chris.packham@alliedtelesis.co.nz> @ 2017-10-11 2:29 ` Chris Packham 2017-10-12 14:16 ` Gregory CLEMENT 0 siblings, 1 reply; 3+ messages in thread From: Chris Packham @ 2017-10-11 2:29 UTC (permalink / raw) To: linux-arm-kernel The watchdog timer interrupt is available internally on the Armada-38x SoCs. Connect it so that we can have the orion_wdt_irq generate debug information when a watchdog timeout occurs. Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz> --- arch/arm/boot/dts/armada-38x.dtsi | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/boot/dts/armada-38x.dtsi b/arch/arm/boot/dts/armada-38x.dtsi index 7ff0811e61db..9ac76c54c9e5 100644 --- a/arch/arm/boot/dts/armada-38x.dtsi +++ b/arch/arm/boot/dts/armada-38x.dtsi @@ -408,6 +408,7 @@ reg = <0x20300 0x34>, <0x20704 0x4>, <0x18260 0x4>; clocks = <&coreclk 2>, <&refclk>; clock-names = "nbclk", "fixed"; + interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>; }; cpurst: cpurst at 20800 { -- 2.14.2 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH 3/3] ARM: mvebu: dts: connect interrupt for WD on armada-38x 2017-10-11 2:29 ` [PATCH 3/3] ARM: mvebu: dts: connect interrupt for WD on armada-38x Chris Packham @ 2017-10-12 14:16 ` Gregory CLEMENT 2017-10-12 20:12 ` Chris Packham 0 siblings, 1 reply; 3+ messages in thread From: Gregory CLEMENT @ 2017-10-12 14:16 UTC (permalink / raw) To: linux-arm-kernel Hi Chris, On mer., oct. 11 2017, Chris Packham <chris.packham@alliedtelesis.co.nz> wrote: > The watchdog timer interrupt is available internally on the Armada-38x > SoCs. Connect it so that we can have the orion_wdt_irq generate debug > information when a watchdog timeout occurs. Given the patch 2, when an interrupt is provided by the dt, then the behavior of the watchdog changes. So we can't do it for all the platform. As you propose on your cover letter it is better to set it at board level. So instead of this patch you should update the binding documentation with this new resources. Thanks to this each board developer will be able to decide to use this feature or not. Thanks, Gregory > > Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz> > --- > arch/arm/boot/dts/armada-38x.dtsi | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/boot/dts/armada-38x.dtsi b/arch/arm/boot/dts/armada-38x.dtsi > index 7ff0811e61db..9ac76c54c9e5 100644 > --- a/arch/arm/boot/dts/armada-38x.dtsi > +++ b/arch/arm/boot/dts/armada-38x.dtsi > @@ -408,6 +408,7 @@ > reg = <0x20300 0x34>, <0x20704 0x4>, <0x18260 0x4>; > clocks = <&coreclk 2>, <&refclk>; > clock-names = "nbclk", "fixed"; > + interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>; > }; > > cpurst: cpurst at 20800 { > -- > 2.14.2 > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com ^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH 3/3] ARM: mvebu: dts: connect interrupt for WD on armada-38x 2017-10-12 14:16 ` Gregory CLEMENT @ 2017-10-12 20:12 ` Chris Packham 0 siblings, 0 replies; 3+ messages in thread From: Chris Packham @ 2017-10-12 20:12 UTC (permalink / raw) To: linux-arm-kernel On 13/10/17 03:17, Gregory CLEMENT wrote: > Hi Chris, > > On mer., oct. 11 2017, Chris Packham <chris.packham@alliedtelesis.co.nz> wrote: > >> The watchdog timer interrupt is available internally on the Armada-38x >> SoCs. Connect it so that we can have the orion_wdt_irq generate debug >> information when a watchdog timeout occurs. > > Given the patch 2, when an interrupt is provided by the dt, then the > behavior of the watchdog changes. So we can't do it for all the > platform. > > As you propose on your cover letter it is better to set it at board > level. > > So instead of this patch you should update the binding documentation > with this new resources. Thanks to this each board developer will be > able to decide to use this feature or not. Agreed. I'll drop this patch in favor of a documentation update in v2. > > Thanks, > > Gregory > >> >> Signed-off-by: Chris Packham <chris.packham@alliedtelesis.co.nz> >> --- >> arch/arm/boot/dts/armada-38x.dtsi | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/arm/boot/dts/armada-38x.dtsi b/arch/arm/boot/dts/armada-38x.dtsi >> index 7ff0811e61db..9ac76c54c9e5 100644 >> --- a/arch/arm/boot/dts/armada-38x.dtsi >> +++ b/arch/arm/boot/dts/armada-38x.dtsi >> @@ -408,6 +408,7 @@ >> reg = <0x20300 0x34>, <0x20704 0x4>, <0x18260 0x4>; >> clocks = <&coreclk 2>, <&refclk>; >> clock-names = "nbclk", "fixed"; >> + interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>; >> }; >> >> cpurst: cpurst at 20800 { >> -- >> 2.14.2 >> > ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-10-12 20:12 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20171011022958.31268-1-chris.packham@alliedtelesis.co.nz>
2017-10-11 2:29 ` [PATCH 3/3] ARM: mvebu: dts: connect interrupt for WD on armada-38x Chris Packham
2017-10-12 14:16 ` Gregory CLEMENT
2017-10-12 20:12 ` Chris Packham
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).