* [PATCH 0/2] Driver for watchdog timer on Intel Lightning Mountain SoC @ 2020-06-08 5:49 Dilip Kota 2020-06-08 5:49 ` [PATCH 1/2] dt-bindings: watchdog: intel: Add YAML Schemas for Watchdog timer Dilip Kota 2020-06-08 5:49 ` [PATCH 2/2] watchdog: intel: Watchdog timer support on Lightning Mountain Dilip Kota 0 siblings, 2 replies; 13+ messages in thread From: Dilip Kota @ 2020-06-08 5:49 UTC (permalink / raw) To: wim, linux, linux-watchdog, devicetree, robbh Cc: linux-kernel, andriy.shevchenko, cheol.yong.kim, qi-ming.wu, yixin.zhu, Dilip Kota This patch series adds watchdog timer driver and respective yaml schemas for watchdog timer on Intel Lightning Mountain SoC. This patch series is rebased and tested on mainline linux kernel 5.7: base-commit: 3d77e6a8804a ("Linux 5.7") tags: v5.7 Dilip Kota (2): dt-bindings: watchdog: intel: Add YAML Schemas for Watchdog timer watchdog: intel: Watchdog timer support on Lightning Mountain .../bindings/watchdog/intel,lgm-gptc-wdt.yaml | 75 ++++ drivers/watchdog/Kconfig | 13 + drivers/watchdog/Makefile | 1 + drivers/watchdog/intel_lgm_gptc_wdt.c | 420 +++++++++++++++++++++ 4 files changed, 509 insertions(+) create mode 100644 Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml create mode 100644 drivers/watchdog/intel_lgm_gptc_wdt.c -- 2.11.0 ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 1/2] dt-bindings: watchdog: intel: Add YAML Schemas for Watchdog timer 2020-06-08 5:49 [PATCH 0/2] Driver for watchdog timer on Intel Lightning Mountain SoC Dilip Kota @ 2020-06-08 5:49 ` Dilip Kota 2020-06-08 13:37 ` Guenter Roeck 2020-06-08 5:49 ` [PATCH 2/2] watchdog: intel: Watchdog timer support on Lightning Mountain Dilip Kota 1 sibling, 1 reply; 13+ messages in thread From: Dilip Kota @ 2020-06-08 5:49 UTC (permalink / raw) To: wim, linux, linux-watchdog, devicetree, robbh Cc: linux-kernel, andriy.shevchenko, cheol.yong.kim, qi-ming.wu, yixin.zhu, Dilip Kota Add YAML schemas for the watchdog timer on Intel Lightning Mountain SoC. Signed-off-by: Dilip Kota <eswara.kota@linux.intel.com> --- .../bindings/watchdog/intel,lgm-gptc-wdt.yaml | 75 ++++++++++++++++++++++ 1 file changed, 75 insertions(+) create mode 100644 Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml diff --git a/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml b/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml new file mode 100644 index 0000000000000..83dc39a5090c1 --- /dev/null +++ b/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml @@ -0,0 +1,75 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/watchdog/intel,lgm-gptc-wdt.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Intel Lightning Mountain Watchdog timer. + +maintainers: + - Dilip Kota <eswara.kota@linux.intel.com> + +description: | + Intel Lightning Mountain SoC has General Purpose Timer Counter(GPTC) which can + be configured as Clocksource, real time clock and Watchdog timer. + Each General Purpose Timer Counter has three timers. And total four General + Purpose Timer Counters are present on Lightning Mountain SoC which sums up + to 12 timers. + Lightning Mountain has four CPUs and each CPU is configured with one GPTC + timer as watchdog timer. Total four timers are configured as watchdog timers + on Lightning Mountain SoC. + +allOf: + - $ref: "watchdog.yaml#" + +properties: + compatible: + enum: + - intel,lgm-gptc-wdt + + reg: + maxItems: 1 + + clocks: + items: + - description: Frequency clock + - description: Core clock + + clock-names: + items: + - const: freq + - const: gptc + + intel,wdt-rst-hndl: + $ref: /schemas/types.yaml#/definitions/phandle + description: Watchdog timer registers handle + + intel,timer-cfg: + description: Watchdog Timer id and CPU id + allOf: + - $ref: /schemas/types.yaml#/definitions/uint32-array + - minItems: 2 + maxItems: 4 + items: + minimum: 0 + maximum: 3 + +required: + - compatible + - reg + - clocks + - clock-names + - intel,wdt-rst-hndl + - intel,timer-cfg + +examples: + - | + watchdog@e2500000 { + compatible = "intel,lgm-gptc-wdt"; + reg = <0xe2500000 0x10000>; + intel,wdt-rst-hndl = <&sysconf>; + clocks = <&cgu0 31>, <&cgu0 136>; + clock-names = "freq", "gptc"; + timeout-sec = <30>; + intel,timer-cfg = <1 0 2 1>; + }; -- 2.11.0 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 1/2] dt-bindings: watchdog: intel: Add YAML Schemas for Watchdog timer 2020-06-08 5:49 ` [PATCH 1/2] dt-bindings: watchdog: intel: Add YAML Schemas for Watchdog timer Dilip Kota @ 2020-06-08 13:37 ` Guenter Roeck 2020-06-09 8:57 ` Dilip Kota 0 siblings, 1 reply; 13+ messages in thread From: Guenter Roeck @ 2020-06-08 13:37 UTC (permalink / raw) To: Dilip Kota, wim, linux-watchdog, devicetree, robbh Cc: linux-kernel, andriy.shevchenko, cheol.yong.kim, qi-ming.wu, yixin.zhu On 6/7/20 10:49 PM, Dilip Kota wrote: > Add YAML schemas for the watchdog timer on Intel Lightning > Mountain SoC. > > Signed-off-by: Dilip Kota <eswara.kota@linux.intel.com> > --- > .../bindings/watchdog/intel,lgm-gptc-wdt.yaml | 75 ++++++++++++++++++++++ > 1 file changed, 75 insertions(+) > create mode 100644 Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml > > diff --git a/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml b/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml > new file mode 100644 > index 0000000000000..83dc39a5090c1 > --- /dev/null > +++ b/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml > @@ -0,0 +1,75 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/watchdog/intel,lgm-gptc-wdt.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Intel Lightning Mountain Watchdog timer. > + > +maintainers: > + - Dilip Kota <eswara.kota@linux.intel.com> > + > +description: | > + Intel Lightning Mountain SoC has General Purpose Timer Counter(GPTC) which can > + be configured as Clocksource, real time clock and Watchdog timer. > + Each General Purpose Timer Counter has three timers. And total four General > + Purpose Timer Counters are present on Lightning Mountain SoC which sums up > + to 12 timers. > + Lightning Mountain has four CPUs and each CPU is configured with one GPTC > + timer as watchdog timer. Total four timers are configured as watchdog timers > + on Lightning Mountain SoC. > + Why not just one ? The watchdog subsystem does not monitor individual CPUs, it monitors the system. Guenter > +allOf: > + - $ref: "watchdog.yaml#" > + > +properties: > + compatible: > + enum: > + - intel,lgm-gptc-wdt > + > + reg: > + maxItems: 1 > + > + clocks: > + items: > + - description: Frequency clock > + - description: Core clock > + > + clock-names: > + items: > + - const: freq > + - const: gptc > + > + intel,wdt-rst-hndl: > + $ref: /schemas/types.yaml#/definitions/phandle > + description: Watchdog timer registers handle > + > + intel,timer-cfg: > + description: Watchdog Timer id and CPU id > + allOf: > + - $ref: /schemas/types.yaml#/definitions/uint32-array > + - minItems: 2 > + maxItems: 4 > + items: > + minimum: 0 > + maximum: 3 > + > +required: > + - compatible > + - reg > + - clocks > + - clock-names > + - intel,wdt-rst-hndl > + - intel,timer-cfg > + > +examples: > + - | > + watchdog@e2500000 { > + compatible = "intel,lgm-gptc-wdt"; > + reg = <0xe2500000 0x10000>; > + intel,wdt-rst-hndl = <&sysconf>; > + clocks = <&cgu0 31>, <&cgu0 136>; > + clock-names = "freq", "gptc"; > + timeout-sec = <30>; > + intel,timer-cfg = <1 0 2 1>; > + }; > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/2] dt-bindings: watchdog: intel: Add YAML Schemas for Watchdog timer 2020-06-08 13:37 ` Guenter Roeck @ 2020-06-09 8:57 ` Dilip Kota 2020-06-09 13:46 ` Guenter Roeck 0 siblings, 1 reply; 13+ messages in thread From: Dilip Kota @ 2020-06-09 8:57 UTC (permalink / raw) To: Guenter Roeck, wim, linux-watchdog, devicetree, robbh Cc: linux-kernel, andriy.shevchenko, cheol.yong.kim, qi-ming.wu, yixin.zhu On 6/8/2020 9:37 PM, Guenter Roeck wrote: > On 6/7/20 10:49 PM, Dilip Kota wrote: >> Add YAML schemas for the watchdog timer on Intel Lightning >> Mountain SoC. >> >> Signed-off-by: Dilip Kota <eswara.kota@linux.intel.com> >> --- >> .../bindings/watchdog/intel,lgm-gptc-wdt.yaml | 75 ++++++++++++++++++++++ >> 1 file changed, 75 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml >> >> diff --git a/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml b/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml >> new file mode 100644 >> index 0000000000000..83dc39a5090c1 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml >> @@ -0,0 +1,75 @@ >> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/watchdog/intel,lgm-gptc-wdt.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Intel Lightning Mountain Watchdog timer. >> + >> +maintainers: >> + - Dilip Kota <eswara.kota@linux.intel.com> >> + >> +description: | >> + Intel Lightning Mountain SoC has General Purpose Timer Counter(GPTC) which can >> + be configured as Clocksource, real time clock and Watchdog timer. >> + Each General Purpose Timer Counter has three timers. And total four General >> + Purpose Timer Counters are present on Lightning Mountain SoC which sums up >> + to 12 timers. >> + Lightning Mountain has four CPUs and each CPU is configured with one GPTC >> + timer as watchdog timer. Total four timers are configured as watchdog timers >> + on Lightning Mountain SoC. >> + > Why not just one ? The watchdog subsystem does not monitor individual CPUs, > it monitors the system. Intel Atom based Lightning Mountain SoC, system has four CPUs. On Lightning Mountain SoC ,Watchdog subsystem is combination of GPTC timers and reset controller unit. On Lightning Mountain SoC, each CPU is configured with one GPTC timer, so that if any of the CPU hangs or freezes, the watchdog daemon running on respective CPU cannot reset/ping or pet the watchdog timer. This causes the watchdog timeout. On watchdog timeout, reset controller triggers the reset to respective CPU. ____________________ ----------------------------->| | | ------------>| Reset controller unit | | | |___________________| | | | | ______________________|__________|______ | GPTC | | | | ___________ ________|_ ______|____ | | | timer 1 | | timer 2 | | timer 3 | | | |_________| |_________| |_________| | |______________________________________| Regards, -Dilip > Guenter > > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/2] dt-bindings: watchdog: intel: Add YAML Schemas for Watchdog timer 2020-06-09 8:57 ` Dilip Kota @ 2020-06-09 13:46 ` Guenter Roeck 2020-06-10 7:54 ` Dilip Kota 0 siblings, 1 reply; 13+ messages in thread From: Guenter Roeck @ 2020-06-09 13:46 UTC (permalink / raw) To: Dilip Kota, wim, linux-watchdog, devicetree, robbh Cc: linux-kernel, andriy.shevchenko, cheol.yong.kim, qi-ming.wu, yixin.zhu On 6/9/20 1:57 AM, Dilip Kota wrote: > > On 6/8/2020 9:37 PM, Guenter Roeck wrote: >> On 6/7/20 10:49 PM, Dilip Kota wrote: >>> Add YAML schemas for the watchdog timer on Intel Lightning >>> Mountain SoC. >>> >>> Signed-off-by: Dilip Kota <eswara.kota@linux.intel.com> >>> --- >>> .../bindings/watchdog/intel,lgm-gptc-wdt.yaml | 75 ++++++++++++++++++++++ >>> 1 file changed, 75 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml >>> >>> diff --git a/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml b/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml >>> new file mode 100644 >>> index 0000000000000..83dc39a5090c1 >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml >>> @@ -0,0 +1,75 @@ >>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >>> +%YAML 1.2 >>> +--- >>> +$id: http://devicetree.org/schemas/watchdog/intel,lgm-gptc-wdt.yaml# >>> +$schema: http://devicetree.org/meta-schemas/core.yaml# >>> + >>> +title: Intel Lightning Mountain Watchdog timer. >>> + >>> +maintainers: >>> + - Dilip Kota <eswara.kota@linux.intel.com> >>> + >>> +description: | >>> + Intel Lightning Mountain SoC has General Purpose Timer Counter(GPTC) which can >>> + be configured as Clocksource, real time clock and Watchdog timer. >>> + Each General Purpose Timer Counter has three timers. And total four General >>> + Purpose Timer Counters are present on Lightning Mountain SoC which sums up >>> + to 12 timers. >>> + Lightning Mountain has four CPUs and each CPU is configured with one GPTC >>> + timer as watchdog timer. Total four timers are configured as watchdog timers >>> + on Lightning Mountain SoC. >>> + >> Why not just one ? The watchdog subsystem does not monitor individual CPUs, >> it monitors the system. > > Intel Atom based Lightning Mountain SoC, system has four CPUs. On Lightning Mountain SoC ,Watchdog subsystem is combination of GPTC timers and reset controller unit. On Lightning Mountain SoC, each CPU is configured with one GPTC timer, so that if any of the CPU hangs or freezes, the watchdog daemon running on respective CPU cannot reset/ping or pet the watchdog timer. This causes the watchdog timeout. On watchdog timeout, reset controller triggers the reset to respective CPU. > A system watchdog driver should not duplicate functionality from kernel/watchdog.c, which monitors individual CPUs. If the SoC does nto provide a system watchdog timer (which I think is unlikely), it should stick with that. A watchdog resetting an individual CPU instead of the entire system isn't something I would want to see in the watchdog subsystem. Guenter > > ____________________ > ----------------------------->| | > | ------------>| Reset controller unit | > | | |___________________| > | | > | | > ______________________|__________|______ > | GPTC | | | > | ___________ ________|_ ______|____ | > | | timer 1 | | timer 2 | | timer 3 | | > | |_________| |_________| |_________| | > |______________________________________| > > Regards, > -Dilip > > >> Guenter >> >> ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/2] dt-bindings: watchdog: intel: Add YAML Schemas for Watchdog timer 2020-06-09 13:46 ` Guenter Roeck @ 2020-06-10 7:54 ` Dilip Kota 2020-06-10 13:05 ` Guenter Roeck 0 siblings, 1 reply; 13+ messages in thread From: Dilip Kota @ 2020-06-10 7:54 UTC (permalink / raw) To: Guenter Roeck, wim, linux-watchdog, devicetree, robbh Cc: linux-kernel, andriy.shevchenko, cheol.yong.kim, qi-ming.wu, yixin.zhu On 6/9/2020 9:46 PM, Guenter Roeck wrote: > On 6/9/20 1:57 AM, Dilip Kota wrote: >> On 6/8/2020 9:37 PM, Guenter Roeck wrote: >>> On 6/7/20 10:49 PM, Dilip Kota wrote: >>>> Add YAML schemas for the watchdog timer on Intel Lightning >>>> Mountain SoC. >>>> >>>> Signed-off-by: Dilip Kota <eswara.kota@linux.intel.com> >>>> --- >>>> .../bindings/watchdog/intel,lgm-gptc-wdt.yaml | 75 ++++++++++++++++++++++ >>>> 1 file changed, 75 insertions(+) >>>> create mode 100644 Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml >>>> >>>> diff --git a/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml b/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml >>>> new file mode 100644 >>>> index 0000000000000..83dc39a5090c1 >>>> --- /dev/null >>>> +++ b/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml >>>> @@ -0,0 +1,75 @@ >>>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >>>> +%YAML 1.2 >>>> +--- >>>> +$id: http://devicetree.org/schemas/watchdog/intel,lgm-gptc-wdt.yaml# >>>> +$schema: http://devicetree.org/meta-schemas/core.yaml# >>>> + >>>> +title: Intel Lightning Mountain Watchdog timer. >>>> + >>>> +maintainers: >>>> + - Dilip Kota <eswara.kota@linux.intel.com> >>>> + >>>> +description: | >>>> + Intel Lightning Mountain SoC has General Purpose Timer Counter(GPTC) which can >>>> + be configured as Clocksource, real time clock and Watchdog timer. >>>> + Each General Purpose Timer Counter has three timers. And total four General >>>> + Purpose Timer Counters are present on Lightning Mountain SoC which sums up >>>> + to 12 timers. >>>> + Lightning Mountain has four CPUs and each CPU is configured with one GPTC >>>> + timer as watchdog timer. Total four timers are configured as watchdog timers >>>> + on Lightning Mountain SoC. >>>> + >>> Why not just one ? The watchdog subsystem does not monitor individual CPUs, >>> it monitors the system. >> Intel Atom based Lightning Mountain SoC, system has four CPUs. On Lightning Mountain SoC ,Watchdog subsystem is combination of GPTC timers and reset controller unit. On Lightning Mountain SoC, each CPU is configured with one GPTC timer, so that if any of the CPU hangs or freezes, the watchdog daemon running on respective CPU cannot reset/ping or pet the watchdog timer. This causes the watchdog timeout. On watchdog timeout, reset controller triggers the reset to respective CPU. >> > A system watchdog driver should not duplicate functionality > from kernel/watchdog.c, which monitors individual CPUs. > If the SoC does nto provide a system watchdog timer (which > I think is unlikely), it should stick with that. A watchdog > resetting an individual CPU instead of the entire system > isn't something I would want to see in the watchdog subsystem. My bad here, complete hardware reset happens on watchdog timeout not a single CPU or core. Could you please clarify: The complete system means, you mean, "a watchdog susbsystem should monitor all the cores/cpus in the SoC. Not like each core/cpu in SoC having a wdt". Regards, Dilip > > Guenter > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/2] dt-bindings: watchdog: intel: Add YAML Schemas for Watchdog timer 2020-06-10 7:54 ` Dilip Kota @ 2020-06-10 13:05 ` Guenter Roeck 2020-06-11 9:38 ` Dilip Kota 0 siblings, 1 reply; 13+ messages in thread From: Guenter Roeck @ 2020-06-10 13:05 UTC (permalink / raw) To: Dilip Kota, wim, linux-watchdog, devicetree, robbh Cc: linux-kernel, andriy.shevchenko, cheol.yong.kim, qi-ming.wu, yixin.zhu On 6/10/20 12:54 AM, Dilip Kota wrote: > > On 6/9/2020 9:46 PM, Guenter Roeck wrote: >> On 6/9/20 1:57 AM, Dilip Kota wrote: >>> On 6/8/2020 9:37 PM, Guenter Roeck wrote: >>>> On 6/7/20 10:49 PM, Dilip Kota wrote: >>>>> Add YAML schemas for the watchdog timer on Intel Lightning >>>>> Mountain SoC. >>>>> >>>>> Signed-off-by: Dilip Kota <eswara.kota@linux.intel.com> >>>>> --- >>>>> .../bindings/watchdog/intel,lgm-gptc-wdt.yaml | 75 ++++++++++++++++++++++ >>>>> 1 file changed, 75 insertions(+) >>>>> create mode 100644 Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml >>>>> >>>>> diff --git a/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml b/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml >>>>> new file mode 100644 >>>>> index 0000000000000..83dc39a5090c1 >>>>> --- /dev/null >>>>> +++ b/Documentation/devicetree/bindings/watchdog/intel,lgm-gptc-wdt.yaml >>>>> @@ -0,0 +1,75 @@ >>>>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >>>>> +%YAML 1.2 >>>>> +--- >>>>> +$id: http://devicetree.org/schemas/watchdog/intel,lgm-gptc-wdt.yaml# >>>>> +$schema: http://devicetree.org/meta-schemas/core.yaml# >>>>> + >>>>> +title: Intel Lightning Mountain Watchdog timer. >>>>> + >>>>> +maintainers: >>>>> + - Dilip Kota <eswara.kota@linux.intel.com> >>>>> + >>>>> +description: | >>>>> + Intel Lightning Mountain SoC has General Purpose Timer Counter(GPTC) which can >>>>> + be configured as Clocksource, real time clock and Watchdog timer. >>>>> + Each General Purpose Timer Counter has three timers. And total four General >>>>> + Purpose Timer Counters are present on Lightning Mountain SoC which sums up >>>>> + to 12 timers. >>>>> + Lightning Mountain has four CPUs and each CPU is configured with one GPTC >>>>> + timer as watchdog timer. Total four timers are configured as watchdog timers >>>>> + on Lightning Mountain SoC. >>>>> + >>>> Why not just one ? The watchdog subsystem does not monitor individual CPUs, >>>> it monitors the system. >>> Intel Atom based Lightning Mountain SoC, system has four CPUs. On Lightning Mountain SoC ,Watchdog subsystem is combination of GPTC timers and reset controller unit. On Lightning Mountain SoC, each CPU is configured with one GPTC timer, so that if any of the CPU hangs or freezes, the watchdog daemon running on respective CPU cannot reset/ping or pet the watchdog timer. This causes the watchdog timeout. On watchdog timeout, reset controller triggers the reset to respective CPU. >>> >> A system watchdog driver should not duplicate functionality >> from kernel/watchdog.c, which monitors individual CPUs. >> If the SoC does nto provide a system watchdog timer (which >> I think is unlikely), it should stick with that. A watchdog >> resetting an individual CPU instead of the entire system >> isn't something I would want to see in the watchdog subsystem. > My bad here, complete hardware reset happens on watchdog timeout not a single CPU or core. > Could you please clarify: The complete system means, you mean, "a watchdog susbsystem should monitor all the cores/cpus in the SoC. Not like each core/cpu in SoC having a wdt". > No, the watchdog subsystem does not monitor "all cores". Again, that is the responsibility of kernel/watchdog.c. Guenter > Regards, > Dilip >> >> Guenter >> ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/2] dt-bindings: watchdog: intel: Add YAML Schemas for Watchdog timer 2020-06-10 13:05 ` Guenter Roeck @ 2020-06-11 9:38 ` Dilip Kota 2020-06-11 17:09 ` Guenter Roeck 0 siblings, 1 reply; 13+ messages in thread From: Dilip Kota @ 2020-06-11 9:38 UTC (permalink / raw) To: Guenter Roeck, wim, linux-watchdog, devicetree, robbh Cc: linux-kernel, andriy.shevchenko, cheol.yong.kim, qi-ming.wu, yixin.zhu On 6/10/2020 9:05 PM, Guenter Roeck wrote: > On 6/10/20 12:54 AM, Dilip Kota wrote: >> On 6/9/2020 9:46 PM, Guenter Roeck wrote: >>> On 6/9/20 1:57 AM, Dilip Kota wrote: >>>> On 6/8/2020 9:37 PM, Guenter Roeck wrote: >>>>> On 6/7/20 10:49 PM, Dilip Kota wrote: [...] >>>>>> + >>>>>> +description: | >>>>>> + Intel Lightning Mountain SoC has General Purpose Timer Counter(GPTC) which can >>>>>> + be configured as Clocksource, real time clock and Watchdog timer. >>>>>> + Each General Purpose Timer Counter has three timers. And total four General >>>>>> + Purpose Timer Counters are present on Lightning Mountain SoC which sums up >>>>>> + to 12 timers. >>>>>> + Lightning Mountain has four CPUs and each CPU is configured with one GPTC >>>>>> + timer as watchdog timer. Total four timers are configured as watchdog timers >>>>>> + on Lightning Mountain SoC. >>>>>> + >>>>> Why not just one ? The watchdog subsystem does not monitor individual CPUs, >>>>> it monitors the system. >>>> Intel Atom based Lightning Mountain SoC, system has four CPUs. On Lightning Mountain SoC ,Watchdog subsystem is combination of GPTC timers and reset controller unit. On Lightning Mountain SoC, each CPU is configured with one GPTC timer, so that if any of the CPU hangs or freezes, the watchdog daemon running on respective CPU cannot reset/ping or pet the watchdog timer. This causes the watchdog timeout. On watchdog timeout, reset controller triggers the reset to respective CPU. >>>> >>> A system watchdog driver should not duplicate functionality >>> from kernel/watchdog.c, which monitors individual CPUs. >>> If the SoC does nto provide a system watchdog timer (which >>> I think is unlikely), it should stick with that. A watchdog >>> resetting an individual CPU instead of the entire system >>> isn't something I would want to see in the watchdog subsystem. >> My bad here, complete hardware reset happens on watchdog timeout not a single CPU or core. >> Could you please clarify: The complete system means, you mean, "a watchdog susbsystem should monitor all the cores/cpus in the SoC. Not like each core/cpu in SoC having a wdt". >> > No, the watchdog subsystem does not monitor "all cores". > Again, that is the responsibility of kernel/watchdog.c. I am a bit confused here. I have gone through the kernel/watchdog.c code and i see hrtimers are used and panic is triggered for lockup on CPU/core. It looks similar to the watchdog subsystem which uses wdt and triggers hardware reset on timeout, whereas kernel/watchdog.c using hrtimers and triggers panic on timeout. To my understanding Watchdog timer recovers the hardware from software hangs or freeze states on the CPU / cores. Also, what does system mean in your statement " watchdog subsystem monitors the system"? What all comes under the system other than the cores/cpus. And also i see there is no other watchdog subsystem in Lightning Mountain architecture. Regards, Dilip > > Guenter > >> Regards, >> Dilip >>> Guenter >>> ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/2] dt-bindings: watchdog: intel: Add YAML Schemas for Watchdog timer 2020-06-11 9:38 ` Dilip Kota @ 2020-06-11 17:09 ` Guenter Roeck 0 siblings, 0 replies; 13+ messages in thread From: Guenter Roeck @ 2020-06-11 17:09 UTC (permalink / raw) To: Dilip Kota Cc: wim, linux-watchdog, devicetree, robbh, linux-kernel, andriy.shevchenko, cheol.yong.kim, qi-ming.wu, yixin.zhu On Thu, Jun 11, 2020 at 05:38:14PM +0800, Dilip Kota wrote: > > On 6/10/2020 9:05 PM, Guenter Roeck wrote: > > On 6/10/20 12:54 AM, Dilip Kota wrote: > > > On 6/9/2020 9:46 PM, Guenter Roeck wrote: > > > > On 6/9/20 1:57 AM, Dilip Kota wrote: > > > > > On 6/8/2020 9:37 PM, Guenter Roeck wrote: > > > > > > On 6/7/20 10:49 PM, Dilip Kota wrote: > [...] > > > > > > > + > > > > > > > +description: | > > > > > > > + Intel Lightning Mountain SoC has General Purpose Timer Counter(GPTC) which can > > > > > > > + be configured as Clocksource, real time clock and Watchdog timer. > > > > > > > + Each General Purpose Timer Counter has three timers. And total four General > > > > > > > + Purpose Timer Counters are present on Lightning Mountain SoC which sums up > > > > > > > + to 12 timers. > > > > > > > + Lightning Mountain has four CPUs and each CPU is configured with one GPTC > > > > > > > + timer as watchdog timer. Total four timers are configured as watchdog timers > > > > > > > + on Lightning Mountain SoC. > > > > > > > + > > > > > > Why not just one ? The watchdog subsystem does not monitor individual CPUs, > > > > > > it monitors the system. > > > > > Intel Atom based Lightning Mountain SoC, system has four CPUs. On Lightning Mountain SoC ,Watchdog subsystem is combination of GPTC timers and reset controller unit. On Lightning Mountain SoC, each CPU is configured with one GPTC timer, so that if any of the CPU hangs or freezes, the watchdog daemon running on respective CPU cannot reset/ping or pet the watchdog timer. This causes the watchdog timeout. On watchdog timeout, reset controller triggers the reset to respective CPU. > > > > > > > > > A system watchdog driver should not duplicate functionality > > > > from kernel/watchdog.c, which monitors individual CPUs. > > > > If the SoC does nto provide a system watchdog timer (which > > > > I think is unlikely), it should stick with that. A watchdog > > > > resetting an individual CPU instead of the entire system > > > > isn't something I would want to see in the watchdog subsystem. > > > My bad here, complete hardware reset happens on watchdog timeout not a single CPU or core. > > > Could you please clarify: The complete system means, you mean, "a watchdog susbsystem should monitor all the cores/cpus in the SoC. Not like each core/cpu in SoC having a wdt". > > > > > No, the watchdog subsystem does not monitor "all cores". > > Again, that is the responsibility of kernel/watchdog.c. > I am a bit confused here. > I have gone through the kernel/watchdog.c code and i see hrtimers are used > and panic is triggered for lockup on CPU/core. > It looks similar to the watchdog subsystem which uses wdt and triggers > hardware reset on timeout, whereas kernel/watchdog.c using hrtimers and > triggers panic on timeout. > To my understanding Watchdog timer recovers the hardware from software hangs > or freeze states on the CPU / cores. > Also, what does system mean in your statement " watchdog subsystem monitors > the system"? What all comes under the system other than the cores/cpus. > And also i see there is no other watchdog subsystem in Lightning Mountain > architecture. > From my perspective, we are not going to duplicate functionality covered by kernel/watchdog.c, which means we are not going to support per cpu core watchdog drivers in drivers/watchdog. If you insist doing it anyway, please disuss with Wim. Thanks, Guenter ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 2/2] watchdog: intel: Watchdog timer support on Lightning Mountain 2020-06-08 5:49 [PATCH 0/2] Driver for watchdog timer on Intel Lightning Mountain SoC Dilip Kota 2020-06-08 5:49 ` [PATCH 1/2] dt-bindings: watchdog: intel: Add YAML Schemas for Watchdog timer Dilip Kota @ 2020-06-08 5:49 ` Dilip Kota 2020-06-08 13:36 ` Guenter Roeck 1 sibling, 1 reply; 13+ messages in thread From: Dilip Kota @ 2020-06-08 5:49 UTC (permalink / raw) To: wim, linux, linux-watchdog, devicetree, robbh Cc: linux-kernel, andriy.shevchenko, cheol.yong.kim, qi-ming.wu, yixin.zhu, Dilip Kota On Intel Lightning Mountain SoC, General Purpose Timer Counter(GPTC) programmable as clocksource, real time clock or watchdog timer. This driver configures GPTC as Watchdog timer and triggers reset signal to CPU on timeout. Signed-off-by: Dilip Kota <eswara.kota@linux.intel.com> --- drivers/watchdog/Kconfig | 13 ++ drivers/watchdog/Makefile | 1 + drivers/watchdog/intel_lgm_gptc_wdt.c | 420 ++++++++++++++++++++++++++++++++++ 3 files changed, 434 insertions(+) create mode 100644 drivers/watchdog/intel_lgm_gptc_wdt.c diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index 0663c604bd642..8009c11e75dda 100644 --- a/drivers/watchdog/Kconfig +++ b/drivers/watchdog/Kconfig @@ -1789,6 +1789,19 @@ config IMGPDC_WDT To compile this driver as a loadable module, choose M here. The module will be called imgpdc_wdt. +config INTEL_LGM_GPTC_WDT + tristate "INTEL LGM SoC Watchdog" + depends on X86 || COMPILE_TEST + depends on OF && HAS_IOMEM + select REGMAP + select MFD_SYSCON + select WATCHDOG_CORE + help + Driver for Watchdog Timer on Intel Lightning Mountain SoC. + + To compile this driver as a loadable module, choose M here. + The module will be called intel_lgm_gptc_wdt. + config LANTIQ_WDT tristate "Lantiq SoC watchdog" depends on LANTIQ diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile index 6de2e4ceef190..92c99e4c46eb7 100644 --- a/drivers/watchdog/Makefile +++ b/drivers/watchdog/Makefile @@ -166,6 +166,7 @@ obj-$(CONFIG_TXX9_WDT) += txx9wdt.o obj-$(CONFIG_OCTEON_WDT) += octeon-wdt.o octeon-wdt-y := octeon-wdt-main.o octeon-wdt-nmi.o obj-$(CONFIG_LANTIQ_WDT) += lantiq_wdt.o +obj-$(CONFIG_INTEL_LGM_GPTC_WDT) += intel_lgm_gptc_wdt.o obj-$(CONFIG_LOONGSON1_WDT) += loongson1_wdt.o obj-$(CONFIG_RALINK_WDT) += rt2880_wdt.o obj-$(CONFIG_IMGPDC_WDT) += imgpdc_wdt.o diff --git a/drivers/watchdog/intel_lgm_gptc_wdt.c b/drivers/watchdog/intel_lgm_gptc_wdt.c new file mode 100644 index 0000000000000..52be7cc194f8f --- /dev/null +++ b/drivers/watchdog/intel_lgm_gptc_wdt.c @@ -0,0 +1,420 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (C) 2020 Intel Corporation. + */ + +#include <linux/bitfield.h> +#include <linux/clk.h> +#include <linux/cpu.h> +#include <linux/io.h> +#include <linux/kernel.h> +#include <linux/mfd/syscon.h> +#include <linux/module.h> +#include <linux/of_device.h> +#include <linux/platform_device.h> +#include <linux/regmap.h> +#include <linux/watchdog.h> + +#define GPTC_CLC 0x00 +#define GPTC_CLC_SUSPEND BIT(4) +#define GPTC_CLC_RMC GENMASK(15, 8) + +/* divider 10 to produce 200 / 10 = 20 MHz clock */ +#define CLC_RMC_DIV 10 + +#define GPTC_CON(X) (0x10 + (X) * 0x20) +#define GPTC_CON_CNT_UP BIT(1) +#define GPTC_CON_ONESHOT BIT(3) +#define GPTC_CON_EXT BIT(4) + +#define GPTC_RUN(X) (0x18 + (X) * 0x20) +#define GPTC_RUN_EN BIT(0) +#define GPTC_RUN_STOP BIT(1) +#define GPTC_RUN_RELOAD BIT(2) + +#define GPTC_RLD(X) (0x20 + (X) * 0x20) +#define GPTC_CNT(X) (0x28 + (X) * 0x20) + +#define GPTC_IRNENCLR 0xF0 +#define GPTC_IRNEN 0xF4 +#define GPTC_IRNCR 0xFC + +/* Watchdog Timeout Reset register offset and bitfeilds */ +#define BIA_WDT_RST_EN 0x1E0 +#define BIA_WDT BIT(6) + +#define MAX_TIMERID 2 +#define MAX_CPUID 3 +#define TIMER_MARGIN_SEC 300 + +static bool nowayout = WATCHDOG_NOWAYOUT; +module_param(nowayout, bool, 0); +MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started\n" + " (default=" __MODULE_STRING(WATCHDOG_NOWAYOUT) ")"); + +struct lgm_gptc_timer { + struct lgm_gptc_wdt *wdt_node; + struct watchdog_device wdd; + unsigned int tid; + unsigned int cpuid; + unsigned int frequency; + unsigned int cycles; + bool enable; +}; + +struct lgm_gptc_wdt { + struct device *dev; + void __iomem *gptc_base; + struct regmap *rst_hndl; + struct clk *freqclk; + struct clk *gateclk; + unsigned int fpifreq; + enum cpuhp_state state; +}; + +DEFINE_PER_CPU(struct lgm_gptc_timer, lgm_timer_per_cpu); + +static void lgm_gptc_wdt_cfg_timer(struct lgm_gptc_timer *timer) +{ + struct lgm_gptc_wdt *wdt_node = timer->wdt_node; + void __iomem *base = wdt_node->gptc_base; + u32 val; + + val = readl(base + GPTC_CON(timer->tid)); + val &= ~GPTC_CON_CNT_UP; + val |= GPTC_CON_EXT; + val &= ~GPTC_CON_ONESHOT; + writel(val, base + GPTC_CON(timer->tid)); + writel(U32_MAX, base + GPTC_RLD(timer->tid)); + writel(BIT(timer->tid * 2), base + GPTC_IRNEN); +} + +static void lgm_gptc_wdt_init(struct lgm_gptc_wdt *wdt_node) +{ + void __iomem *base = wdt_node->gptc_base; + + writel(GPTC_CLC_SUSPEND | FIELD_PREP(GPTC_CLC_RMC, CLC_RMC_DIV), + base + GPTC_CLC); + writel(0xFF, base + GPTC_IRNENCLR); + writel(0xFF, base + GPTC_IRNCR); +} + +static void lgm_gptc_wdt_timer_stop(struct lgm_gptc_timer *timer) +{ + struct lgm_gptc_wdt *wdt_node = timer->wdt_node; + + writel(GPTC_RUN_STOP, wdt_node->gptc_base + GPTC_RUN(timer->tid)); + regmap_write(wdt_node->rst_hndl, BIA_WDT_RST_EN, 0x00); +} + +static void lgm_gptc_wdt_timer_set_timeout(struct lgm_gptc_timer *timer, + u32 timeout) +{ + unsigned long cycles; + struct lgm_gptc_wdt *wdt_node = timer->wdt_node; + + cycles = timeout * timer->frequency / CLC_RMC_DIV; + if (cycles > U32_MAX) + cycles = U32_MAX; + + timer->cycles = cycles; + writel(cycles, wdt_node->gptc_base + GPTC_RLD(timer->tid)); +} + +static int lgm_gptc_wdt_start(struct watchdog_device *wdt_dev) +{ + struct lgm_gptc_timer *timer = watchdog_get_drvdata(wdt_dev); + struct lgm_gptc_wdt *wdt_node = timer->wdt_node; + void __iomem *base = wdt_node->gptc_base; + u32 val; + + val = readl(base + GPTC_CON(timer->tid)); + val &= ~GPTC_CON_ONESHOT; + writel(val, base + GPTC_CON(timer->tid)); + + writel(timer->cycles, base + GPTC_RLD(timer->tid)); + writel(GPTC_RUN_EN | GPTC_RUN_RELOAD, base + GPTC_RUN(timer->tid)); + writel(BIT(timer->tid * 2), base + GPTC_IRNEN); + + /* Enable WDT reset */ + regmap_read(wdt_node->rst_hndl, BIA_WDT_RST_EN, &val); + val |= BIT(timer->cpuid) | BIA_WDT; + regmap_write(wdt_node->rst_hndl, BIA_WDT_RST_EN, val); + + return 0; +} + +static int lgm_gptc_wdt_stop(struct watchdog_device *wdt_dev) +{ + struct lgm_gptc_timer *timer = watchdog_get_drvdata(wdt_dev); + + lgm_gptc_wdt_timer_stop(timer); + + return 0; +} + +static int lgm_gptc_wdt_set_timeout(struct watchdog_device *wdt_dev, + unsigned int new_timeout) +{ + struct lgm_gptc_timer *timer = watchdog_get_drvdata(wdt_dev); + + wdt_dev->timeout = new_timeout; + lgm_gptc_wdt_timer_set_timeout(timer, new_timeout); + + return 0; +} + +static uint32_t lgm_gptc_wdt_get_timeleft(struct watchdog_device *wdt_dev) +{ + struct lgm_gptc_timer *timer = watchdog_get_drvdata(wdt_dev); + struct lgm_gptc_wdt *wdt_node = timer->wdt_node; + u32 cycles, interval; + + cycles = readl(wdt_node->gptc_base + GPTC_CNT(timer->tid)); + interval = cycles / timer->frequency / CLC_RMC_DIV; + dev_dbg(wdt_node->dev, "timeleft=%u cycles=%u\n", interval, cycles); + + return interval; +} + +static int lgm_gptc_wdt_ping(struct watchdog_device *wdt_dev) +{ + return lgm_gptc_wdt_start(wdt_dev); +} + +static const struct watchdog_info lgm_gptc_wdt_info = { + .options = WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING | WDIOF_MAGICCLOSE, + .identity = "Hardware Watchdog for Intel LGM", +}; + +static const struct watchdog_ops lgm_gptc_wdt_ops = { + .start = lgm_gptc_wdt_start, + .stop = lgm_gptc_wdt_stop, + .ping = lgm_gptc_wdt_ping, + .set_timeout = lgm_gptc_wdt_set_timeout, + .get_timeleft = lgm_gptc_wdt_get_timeleft, + .owner = THIS_MODULE, +}; + +static int lgm_gptc_wdt_reg_wdd(struct lgm_gptc_timer *timer) +{ + struct lgm_gptc_wdt *wdt_node = timer->wdt_node; + struct watchdog_device *wdd; + int ret; + + wdd = &timer->wdd; + wdd->id = timer->cpuid; + wdd->info = &lgm_gptc_wdt_info; + wdd->ops = &lgm_gptc_wdt_ops; + wdd->min_timeout = 1; + wdd->max_timeout = U32_MAX; + wdd->max_hw_heartbeat_ms = wdd->max_timeout * 1000; + wdd->min_hw_heartbeat_ms = wdd->min_timeout * 1000; + wdd->timeout = TIMER_MARGIN_SEC; + watchdog_init_timeout(wdd, 0, wdt_node->dev); + lgm_gptc_wdt_timer_set_timeout(timer, wdd->timeout); + watchdog_set_drvdata(wdd, timer); + watchdog_set_nowayout(wdd, nowayout); + ret = watchdog_register_device(wdd); + if (ret) { + dev_err(wdt_node->dev, + "Watchdog timer register fail: %d\n", ret); + return ret; + } + + return 0; +} + +static int lgm_gptc_wdt_reg_cpu(unsigned int cpu) +{ + struct lgm_gptc_timer *timer = per_cpu_ptr(&lgm_timer_per_cpu, cpu); + + if (!timer->enable) + return 0; + + lgm_gptc_wdt_cfg_timer(timer); + lgm_gptc_wdt_reg_wdd(timer); + + return 0; +} + +static int lgm_gptc_wdt_unreg_cpu(unsigned int cpu) +{ + struct lgm_gptc_timer *timer = per_cpu_ptr(&lgm_timer_per_cpu, cpu); + + if (!timer->enable) + return 0; + + lgm_gptc_wdt_timer_stop(timer); + watchdog_unregister_device(&timer->wdd); + + return 0; +} + +static int lgm_gptc_wdt_clk_enable(struct lgm_gptc_wdt *wdt_node) +{ + int ret; + + wdt_node->gateclk = devm_clk_get(wdt_node->dev, "gptc"); + if (IS_ERR(wdt_node->gateclk)) { + ret = PTR_ERR(wdt_node->gateclk); + if (ret != -EPROBE_DEFER) { + dev_err(wdt_node->dev, + "Failed to get gptc clk: %d\n", ret); + } + + return ret; + } + + wdt_node->freqclk = devm_clk_get(wdt_node->dev, "freq"); + if (IS_ERR(wdt_node->freqclk)) { + ret = PTR_ERR(wdt_node->freqclk); + if (ret != -EPROBE_DEFER) { + dev_err(wdt_node->dev, + "Failed to get freq clk: %d\n", ret); + } + + return ret; + } + + ret = clk_prepare_enable(wdt_node->gateclk); + if (ret) { + dev_err(wdt_node->dev, "Gate clk enable failed: %d\n", ret); + return ret; + } + + ret = clk_prepare_enable(wdt_node->freqclk); + if (ret) { + dev_err(wdt_node->dev, "fpi clk enable failed: %d\n", ret); + clk_disable_unprepare(wdt_node->gateclk); + return ret; + } + + wdt_node->fpifreq = clk_get_rate(wdt_node->freqclk); + + return 0; +} + +static void lgm_gptc_wdt_clk_disable(struct lgm_gptc_wdt *wdt_node) +{ + clk_disable_unprepare(wdt_node->gateclk); + clk_disable_unprepare(wdt_node->freqclk); +} + +static int lgm_gptc_wdt_of_parse_timer(struct lgm_gptc_wdt *wdt_node) +{ + struct device_node *np = wdt_node->dev->of_node; + struct lgm_gptc_timer *timer; + u32 i, nr_timers, args[4]; + int ret; + + ret = of_property_read_variable_u32_array(np, "intel,timer-cfg", + args, 2, 4); + if (ret < 0) { + dev_err(wdt_node->dev, "Failed to get timer cfg: %d\n", ret); + return ret; + } + + nr_timers = ret / 2; + if (!nr_timers) { + dev_err(wdt_node->dev, "No timer defined\n"); + return -EINVAL; + } + + for (i = 0; i < nr_timers; i++) { + if (args[i * 2] > MAX_TIMERID) { + dev_err(wdt_node->dev, + "Invalid timer id %d\n", args[i * 2]); + return -EINVAL; + } + + if (args[i * 2 + 1] > MAX_CPUID) { + dev_err(wdt_node->dev, + "Invalid CPU-id : %d\n", args[i * 2 + 1]); + return -EINVAL; + } + + timer = per_cpu_ptr(&lgm_timer_per_cpu, args[i * 2 + 1]); + timer->cpuid = args[i * 2 + 1]; + timer->tid = args[i * 2]; + timer->enable = true; + timer->wdt_node = wdt_node; + timer->frequency = wdt_node->fpifreq; + timer++; + } + + return 0; +} + +static int lgm_gptc_wdt_drv_probe(struct platform_device *pdev) +{ + struct device *dev = &pdev->dev; + struct lgm_gptc_wdt *wdt_node; + int ret; + + wdt_node = devm_kzalloc(dev, sizeof(*wdt_node), GFP_KERNEL); + if (!wdt_node) + return -ENOMEM; + + wdt_node->dev = dev; + wdt_node->gptc_base = devm_platform_ioremap_resource(pdev, 0); + if (IS_ERR(wdt_node->gptc_base)) + return PTR_ERR(wdt_node->gptc_base); + + wdt_node->rst_hndl = syscon_regmap_lookup_by_phandle(dev->of_node, + "intel,wdt-rst-hndl"); + if (IS_ERR(wdt_node->rst_hndl)) { + dev_err(dev, "No phandle of reset controller\n"); + return PTR_ERR(wdt_node->rst_hndl); + } + + ret = lgm_gptc_wdt_clk_enable(wdt_node); + if (ret) + return ret; + + ret = lgm_gptc_wdt_of_parse_timer(wdt_node); + if (ret) + return ret; + + lgm_gptc_wdt_init(wdt_node); + ret = cpuhp_setup_state(CPUHP_AP_ONLINE_DYN, "LGM GPTC WDT REG", + lgm_gptc_wdt_reg_cpu, lgm_gptc_wdt_unreg_cpu); + if (ret < 0) { + dev_err(dev, "CPU hotplug setup failed: %d\n", ret); + return ret; + } + + wdt_node->state = ret; + platform_set_drvdata(pdev, wdt_node); + + return 0; +} + +static int lgm_gptc_wdt_drv_remove(struct platform_device *dev) +{ + struct lgm_gptc_wdt *wdt_node = platform_get_drvdata(dev); + + cpuhp_remove_state(wdt_node->state); + lgm_gptc_wdt_clk_disable(wdt_node); + + return 0; +} + +static const struct of_device_id lgm_gptc_wdt_drv_match[] = { + { .compatible = "intel,lgm-gptc-wdt" }, + {} +}; + +static struct platform_driver lgm_gptc_wdt_driver = { + .driver = { + .name = "intel,lgm-gptc-wdt", + .of_match_table = lgm_gptc_wdt_drv_match, + }, + .probe = lgm_gptc_wdt_drv_probe, + .remove = lgm_gptc_wdt_drv_remove, +}; + +module_platform_driver(lgm_gptc_wdt_driver); + +MODULE_LICENSE("GPL v2"); +MODULE_DESCRIPTION("Driver for Watchdog Timer on Intel Lightning Mountain SoC"); -- 2.11.0 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 2/2] watchdog: intel: Watchdog timer support on Lightning Mountain 2020-06-08 5:49 ` [PATCH 2/2] watchdog: intel: Watchdog timer support on Lightning Mountain Dilip Kota @ 2020-06-08 13:36 ` Guenter Roeck 2020-06-09 8:57 ` Dilip Kota 0 siblings, 1 reply; 13+ messages in thread From: Guenter Roeck @ 2020-06-08 13:36 UTC (permalink / raw) To: Dilip Kota, wim, linux-watchdog, devicetree, robbh Cc: linux-kernel, andriy.shevchenko, cheol.yong.kim, qi-ming.wu, yixin.zhu On 6/7/20 10:49 PM, Dilip Kota wrote: > On Intel Lightning Mountain SoC, General Purpose Timer Counter(GPTC) > programmable as clocksource, real time clock or watchdog timer. > > This driver configures GPTC as Watchdog timer and triggers reset signal > to CPU on timeout. > > Signed-off-by: Dilip Kota <eswara.kota@linux.intel.com> > --- > drivers/watchdog/Kconfig | 13 ++ > drivers/watchdog/Makefile | 1 + > drivers/watchdog/intel_lgm_gptc_wdt.c | 420 ++++++++++++++++++++++++++++++++++ > 3 files changed, 434 insertions(+) > create mode 100644 drivers/watchdog/intel_lgm_gptc_wdt.c > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > index 0663c604bd642..8009c11e75dda 100644 > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > @@ -1789,6 +1789,19 @@ config IMGPDC_WDT > To compile this driver as a loadable module, choose M here. > The module will be called imgpdc_wdt. > > +config INTEL_LGM_GPTC_WDT > + tristate "INTEL LGM SoC Watchdog" > + depends on X86 || COMPILE_TEST > + depends on OF && HAS_IOMEM > + select REGMAP > + select MFD_SYSCON > + select WATCHDOG_CORE > + help > + Driver for Watchdog Timer on Intel Lightning Mountain SoC. > + > + To compile this driver as a loadable module, choose M here. > + The module will be called intel_lgm_gptc_wdt. > + > config LANTIQ_WDT > tristate "Lantiq SoC watchdog" > depends on LANTIQ > diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile > index 6de2e4ceef190..92c99e4c46eb7 100644 > --- a/drivers/watchdog/Makefile > +++ b/drivers/watchdog/Makefile > @@ -166,6 +166,7 @@ obj-$(CONFIG_TXX9_WDT) += txx9wdt.o > obj-$(CONFIG_OCTEON_WDT) += octeon-wdt.o > octeon-wdt-y := octeon-wdt-main.o octeon-wdt-nmi.o > obj-$(CONFIG_LANTIQ_WDT) += lantiq_wdt.o > +obj-$(CONFIG_INTEL_LGM_GPTC_WDT) += intel_lgm_gptc_wdt.o > obj-$(CONFIG_LOONGSON1_WDT) += loongson1_wdt.o > obj-$(CONFIG_RALINK_WDT) += rt2880_wdt.o > obj-$(CONFIG_IMGPDC_WDT) += imgpdc_wdt.o > diff --git a/drivers/watchdog/intel_lgm_gptc_wdt.c b/drivers/watchdog/intel_lgm_gptc_wdt.c > new file mode 100644 > index 0000000000000..52be7cc194f8f > --- /dev/null > +++ b/drivers/watchdog/intel_lgm_gptc_wdt.c > @@ -0,0 +1,420 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Copyright (C) 2020 Intel Corporation. > + */ > + > +#include <linux/bitfield.h> > +#include <linux/clk.h> > +#include <linux/cpu.h> > +#include <linux/io.h> > +#include <linux/kernel.h> > +#include <linux/mfd/syscon.h> > +#include <linux/module.h> > +#include <linux/of_device.h> > +#include <linux/platform_device.h> > +#include <linux/regmap.h> > +#include <linux/watchdog.h> > + > +#define GPTC_CLC 0x00 > +#define GPTC_CLC_SUSPEND BIT(4) > +#define GPTC_CLC_RMC GENMASK(15, 8) > + > +/* divider 10 to produce 200 / 10 = 20 MHz clock */ > +#define CLC_RMC_DIV 10 > + > +#define GPTC_CON(X) (0x10 + (X) * 0x20) > +#define GPTC_CON_CNT_UP BIT(1) > +#define GPTC_CON_ONESHOT BIT(3) > +#define GPTC_CON_EXT BIT(4) > + > +#define GPTC_RUN(X) (0x18 + (X) * 0x20) > +#define GPTC_RUN_EN BIT(0) > +#define GPTC_RUN_STOP BIT(1) > +#define GPTC_RUN_RELOAD BIT(2) > + > +#define GPTC_RLD(X) (0x20 + (X) * 0x20) > +#define GPTC_CNT(X) (0x28 + (X) * 0x20) > + > +#define GPTC_IRNENCLR 0xF0 > +#define GPTC_IRNEN 0xF4 > +#define GPTC_IRNCR 0xFC > + > +/* Watchdog Timeout Reset register offset and bitfeilds */ > +#define BIA_WDT_RST_EN 0x1E0 > +#define BIA_WDT BIT(6) > + > +#define MAX_TIMERID 2 > +#define MAX_CPUID 3 > +#define TIMER_MARGIN_SEC 300 > + > +static bool nowayout = WATCHDOG_NOWAYOUT; > +module_param(nowayout, bool, 0); > +MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started\n" > + " (default=" __MODULE_STRING(WATCHDOG_NOWAYOUT) ")"); > + > +struct lgm_gptc_timer { > + struct lgm_gptc_wdt *wdt_node; > + struct watchdog_device wdd; > + unsigned int tid; > + unsigned int cpuid; > + unsigned int frequency; > + unsigned int cycles; > + bool enable; > +}; > + > +struct lgm_gptc_wdt { > + struct device *dev; > + void __iomem *gptc_base; > + struct regmap *rst_hndl; > + struct clk *freqclk; > + struct clk *gateclk; > + unsigned int fpifreq; > + enum cpuhp_state state; > +}; > + > +DEFINE_PER_CPU(struct lgm_gptc_timer, lgm_timer_per_cpu); > + This is unusual. You'll have to provide a very detailed explanation why this is needed. Guenter > +static void lgm_gptc_wdt_cfg_timer(struct lgm_gptc_timer *timer) > +{ > + struct lgm_gptc_wdt *wdt_node = timer->wdt_node; > + void __iomem *base = wdt_node->gptc_base; > + u32 val; > + > + val = readl(base + GPTC_CON(timer->tid)); > + val &= ~GPTC_CON_CNT_UP; > + val |= GPTC_CON_EXT; > + val &= ~GPTC_CON_ONESHOT; > + writel(val, base + GPTC_CON(timer->tid)); > + writel(U32_MAX, base + GPTC_RLD(timer->tid)); > + writel(BIT(timer->tid * 2), base + GPTC_IRNEN); > +} > + > +static void lgm_gptc_wdt_init(struct lgm_gptc_wdt *wdt_node) > +{ > + void __iomem *base = wdt_node->gptc_base; > + > + writel(GPTC_CLC_SUSPEND | FIELD_PREP(GPTC_CLC_RMC, CLC_RMC_DIV), > + base + GPTC_CLC); > + writel(0xFF, base + GPTC_IRNENCLR); > + writel(0xFF, base + GPTC_IRNCR); > +} > + > +static void lgm_gptc_wdt_timer_stop(struct lgm_gptc_timer *timer) > +{ > + struct lgm_gptc_wdt *wdt_node = timer->wdt_node; > + > + writel(GPTC_RUN_STOP, wdt_node->gptc_base + GPTC_RUN(timer->tid)); > + regmap_write(wdt_node->rst_hndl, BIA_WDT_RST_EN, 0x00); > +} > + > +static void lgm_gptc_wdt_timer_set_timeout(struct lgm_gptc_timer *timer, > + u32 timeout) > +{ > + unsigned long cycles; > + struct lgm_gptc_wdt *wdt_node = timer->wdt_node; > + > + cycles = timeout * timer->frequency / CLC_RMC_DIV; > + if (cycles > U32_MAX) > + cycles = U32_MAX; > + > + timer->cycles = cycles; > + writel(cycles, wdt_node->gptc_base + GPTC_RLD(timer->tid)); > +} > + > +static int lgm_gptc_wdt_start(struct watchdog_device *wdt_dev) > +{ > + struct lgm_gptc_timer *timer = watchdog_get_drvdata(wdt_dev); > + struct lgm_gptc_wdt *wdt_node = timer->wdt_node; > + void __iomem *base = wdt_node->gptc_base; > + u32 val; > + > + val = readl(base + GPTC_CON(timer->tid)); > + val &= ~GPTC_CON_ONESHOT; > + writel(val, base + GPTC_CON(timer->tid)); > + > + writel(timer->cycles, base + GPTC_RLD(timer->tid)); > + writel(GPTC_RUN_EN | GPTC_RUN_RELOAD, base + GPTC_RUN(timer->tid)); > + writel(BIT(timer->tid * 2), base + GPTC_IRNEN); > + > + /* Enable WDT reset */ > + regmap_read(wdt_node->rst_hndl, BIA_WDT_RST_EN, &val); > + val |= BIT(timer->cpuid) | BIA_WDT; > + regmap_write(wdt_node->rst_hndl, BIA_WDT_RST_EN, val); > + > + return 0; > +} > + > +static int lgm_gptc_wdt_stop(struct watchdog_device *wdt_dev) > +{ > + struct lgm_gptc_timer *timer = watchdog_get_drvdata(wdt_dev); > + > + lgm_gptc_wdt_timer_stop(timer); > + > + return 0; > +} > + > +static int lgm_gptc_wdt_set_timeout(struct watchdog_device *wdt_dev, > + unsigned int new_timeout) > +{ > + struct lgm_gptc_timer *timer = watchdog_get_drvdata(wdt_dev); > + > + wdt_dev->timeout = new_timeout; > + lgm_gptc_wdt_timer_set_timeout(timer, new_timeout); > + > + return 0; > +} > + > +static uint32_t lgm_gptc_wdt_get_timeleft(struct watchdog_device *wdt_dev) > +{ > + struct lgm_gptc_timer *timer = watchdog_get_drvdata(wdt_dev); > + struct lgm_gptc_wdt *wdt_node = timer->wdt_node; > + u32 cycles, interval; > + > + cycles = readl(wdt_node->gptc_base + GPTC_CNT(timer->tid)); > + interval = cycles / timer->frequency / CLC_RMC_DIV; > + dev_dbg(wdt_node->dev, "timeleft=%u cycles=%u\n", interval, cycles); > + > + return interval; > +} > + > +static int lgm_gptc_wdt_ping(struct watchdog_device *wdt_dev) > +{ > + return lgm_gptc_wdt_start(wdt_dev); > +} > + > +static const struct watchdog_info lgm_gptc_wdt_info = { > + .options = WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING | WDIOF_MAGICCLOSE, > + .identity = "Hardware Watchdog for Intel LGM", > +}; > + > +static const struct watchdog_ops lgm_gptc_wdt_ops = { > + .start = lgm_gptc_wdt_start, > + .stop = lgm_gptc_wdt_stop, > + .ping = lgm_gptc_wdt_ping, > + .set_timeout = lgm_gptc_wdt_set_timeout, > + .get_timeleft = lgm_gptc_wdt_get_timeleft, > + .owner = THIS_MODULE, > +}; > + > +static int lgm_gptc_wdt_reg_wdd(struct lgm_gptc_timer *timer) > +{ > + struct lgm_gptc_wdt *wdt_node = timer->wdt_node; > + struct watchdog_device *wdd; > + int ret; > + > + wdd = &timer->wdd; > + wdd->id = timer->cpuid; > + wdd->info = &lgm_gptc_wdt_info; > + wdd->ops = &lgm_gptc_wdt_ops; > + wdd->min_timeout = 1; > + wdd->max_timeout = U32_MAX; > + wdd->max_hw_heartbeat_ms = wdd->max_timeout * 1000; > + wdd->min_hw_heartbeat_ms = wdd->min_timeout * 1000; > + wdd->timeout = TIMER_MARGIN_SEC; > + watchdog_init_timeout(wdd, 0, wdt_node->dev); > + lgm_gptc_wdt_timer_set_timeout(timer, wdd->timeout); > + watchdog_set_drvdata(wdd, timer); > + watchdog_set_nowayout(wdd, nowayout); > + ret = watchdog_register_device(wdd); > + if (ret) { > + dev_err(wdt_node->dev, > + "Watchdog timer register fail: %d\n", ret); > + return ret; > + } > + > + return 0; > +} > + > +static int lgm_gptc_wdt_reg_cpu(unsigned int cpu) > +{ > + struct lgm_gptc_timer *timer = per_cpu_ptr(&lgm_timer_per_cpu, cpu); > + > + if (!timer->enable) > + return 0; > + > + lgm_gptc_wdt_cfg_timer(timer); > + lgm_gptc_wdt_reg_wdd(timer); > + > + return 0; > +} > + > +static int lgm_gptc_wdt_unreg_cpu(unsigned int cpu) > +{ > + struct lgm_gptc_timer *timer = per_cpu_ptr(&lgm_timer_per_cpu, cpu); > + > + if (!timer->enable) > + return 0; > + > + lgm_gptc_wdt_timer_stop(timer); > + watchdog_unregister_device(&timer->wdd); > + > + return 0; > +} > + > +static int lgm_gptc_wdt_clk_enable(struct lgm_gptc_wdt *wdt_node) > +{ > + int ret; > + > + wdt_node->gateclk = devm_clk_get(wdt_node->dev, "gptc"); > + if (IS_ERR(wdt_node->gateclk)) { > + ret = PTR_ERR(wdt_node->gateclk); > + if (ret != -EPROBE_DEFER) { > + dev_err(wdt_node->dev, > + "Failed to get gptc clk: %d\n", ret); > + } > + > + return ret; > + } > + > + wdt_node->freqclk = devm_clk_get(wdt_node->dev, "freq"); > + if (IS_ERR(wdt_node->freqclk)) { > + ret = PTR_ERR(wdt_node->freqclk); > + if (ret != -EPROBE_DEFER) { > + dev_err(wdt_node->dev, > + "Failed to get freq clk: %d\n", ret); > + } > + > + return ret; > + } > + > + ret = clk_prepare_enable(wdt_node->gateclk); > + if (ret) { > + dev_err(wdt_node->dev, "Gate clk enable failed: %d\n", ret); > + return ret; > + } > + > + ret = clk_prepare_enable(wdt_node->freqclk); > + if (ret) { > + dev_err(wdt_node->dev, "fpi clk enable failed: %d\n", ret); > + clk_disable_unprepare(wdt_node->gateclk); > + return ret; > + } > + > + wdt_node->fpifreq = clk_get_rate(wdt_node->freqclk); > + > + return 0; > +} > + > +static void lgm_gptc_wdt_clk_disable(struct lgm_gptc_wdt *wdt_node) > +{ > + clk_disable_unprepare(wdt_node->gateclk); > + clk_disable_unprepare(wdt_node->freqclk); > +} > + > +static int lgm_gptc_wdt_of_parse_timer(struct lgm_gptc_wdt *wdt_node) > +{ > + struct device_node *np = wdt_node->dev->of_node; > + struct lgm_gptc_timer *timer; > + u32 i, nr_timers, args[4]; > + int ret; > + > + ret = of_property_read_variable_u32_array(np, "intel,timer-cfg", > + args, 2, 4); > + if (ret < 0) { > + dev_err(wdt_node->dev, "Failed to get timer cfg: %d\n", ret); > + return ret; > + } > + > + nr_timers = ret / 2; > + if (!nr_timers) { > + dev_err(wdt_node->dev, "No timer defined\n"); > + return -EINVAL; > + } > + > + for (i = 0; i < nr_timers; i++) { > + if (args[i * 2] > MAX_TIMERID) { > + dev_err(wdt_node->dev, > + "Invalid timer id %d\n", args[i * 2]); > + return -EINVAL; > + } > + > + if (args[i * 2 + 1] > MAX_CPUID) { > + dev_err(wdt_node->dev, > + "Invalid CPU-id : %d\n", args[i * 2 + 1]); > + return -EINVAL; > + } > + > + timer = per_cpu_ptr(&lgm_timer_per_cpu, args[i * 2 + 1]); > + timer->cpuid = args[i * 2 + 1]; > + timer->tid = args[i * 2]; > + timer->enable = true; > + timer->wdt_node = wdt_node; > + timer->frequency = wdt_node->fpifreq; > + timer++; > + } > + > + return 0; > +} > + > +static int lgm_gptc_wdt_drv_probe(struct platform_device *pdev) > +{ > + struct device *dev = &pdev->dev; > + struct lgm_gptc_wdt *wdt_node; > + int ret; > + > + wdt_node = devm_kzalloc(dev, sizeof(*wdt_node), GFP_KERNEL); > + if (!wdt_node) > + return -ENOMEM; > + > + wdt_node->dev = dev; > + wdt_node->gptc_base = devm_platform_ioremap_resource(pdev, 0); > + if (IS_ERR(wdt_node->gptc_base)) > + return PTR_ERR(wdt_node->gptc_base); > + > + wdt_node->rst_hndl = syscon_regmap_lookup_by_phandle(dev->of_node, > + "intel,wdt-rst-hndl"); > + if (IS_ERR(wdt_node->rst_hndl)) { > + dev_err(dev, "No phandle of reset controller\n"); > + return PTR_ERR(wdt_node->rst_hndl); > + } > + > + ret = lgm_gptc_wdt_clk_enable(wdt_node); > + if (ret) > + return ret; > + > + ret = lgm_gptc_wdt_of_parse_timer(wdt_node); > + if (ret) > + return ret; > + > + lgm_gptc_wdt_init(wdt_node); > + ret = cpuhp_setup_state(CPUHP_AP_ONLINE_DYN, "LGM GPTC WDT REG", > + lgm_gptc_wdt_reg_cpu, lgm_gptc_wdt_unreg_cpu); > + if (ret < 0) { > + dev_err(dev, "CPU hotplug setup failed: %d\n", ret); > + return ret; > + } > + > + wdt_node->state = ret; > + platform_set_drvdata(pdev, wdt_node); > + > + return 0; > +} > + > +static int lgm_gptc_wdt_drv_remove(struct platform_device *dev) > +{ > + struct lgm_gptc_wdt *wdt_node = platform_get_drvdata(dev); > + > + cpuhp_remove_state(wdt_node->state); > + lgm_gptc_wdt_clk_disable(wdt_node); > + > + return 0; > +} > + > +static const struct of_device_id lgm_gptc_wdt_drv_match[] = { > + { .compatible = "intel,lgm-gptc-wdt" }, > + {} > +}; > + > +static struct platform_driver lgm_gptc_wdt_driver = { > + .driver = { > + .name = "intel,lgm-gptc-wdt", > + .of_match_table = lgm_gptc_wdt_drv_match, > + }, > + .probe = lgm_gptc_wdt_drv_probe, > + .remove = lgm_gptc_wdt_drv_remove, > +}; > + > +module_platform_driver(lgm_gptc_wdt_driver); > + > +MODULE_LICENSE("GPL v2"); > +MODULE_DESCRIPTION("Driver for Watchdog Timer on Intel Lightning Mountain SoC"); > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 2/2] watchdog: intel: Watchdog timer support on Lightning Mountain 2020-06-08 13:36 ` Guenter Roeck @ 2020-06-09 8:57 ` Dilip Kota 2020-06-09 13:42 ` Guenter Roeck 0 siblings, 1 reply; 13+ messages in thread From: Dilip Kota @ 2020-06-09 8:57 UTC (permalink / raw) To: Guenter Roeck, wim, linux-watchdog, devicetree, robbh Cc: linux-kernel, andriy.shevchenko, cheol.yong.kim, qi-ming.wu, yixin.zhu On 6/8/2020 9:36 PM, Guenter Roeck wrote: > On 6/7/20 10:49 PM, Dilip Kota wrote: >> On Intel Lightning Mountain SoC, General Purpose Timer Counter(GPTC) >> programmable as clocksource, real time clock or watchdog timer. >> >> This driver configures GPTC as Watchdog timer and triggers reset signal >> to CPU on timeout. >> >> Signed-off-by: Dilip Kota <eswara.kota@linux.intel.com> >> --- >> drivers/watchdog/Kconfig | 13 ++ >> drivers/watchdog/Makefile | 1 + >> drivers/watchdog/intel_lgm_gptc_wdt.c | 420 ++++++++++++++++++++++++++++++++++ >> 3 files changed, 434 insertions(+) >> create mode 100644 drivers/watchdog/intel_lgm_gptc_wdt.c >> >> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig >> index 0663c604bd642..8009c11e75dda 100644 >> --- a/drivers/watchdog/Kconfig >> +++ b/drivers/watchdog/Kconfig >> @@ -1789,6 +1789,19 @@ config IMGPDC_WDT >> To compile this driver as a loadable module, choose M here. >> The module will be called imgpdc_wdt. >> >> +config INTEL_LGM_GPTC_WDT >> + tristate "INTEL LGM SoC Watchdog" >> + depends on X86 || COMPILE_TEST >> + depends on OF && HAS_IOMEM >> + select REGMAP >> + select MFD_SYSCON >> + select WATCHDOG_CORE >> + help >> + Driver for Watchdog Timer on Intel Lightning Mountain SoC. >> + >> + To compile this driver as a loadable module, choose M here. >> + The module will be called intel_lgm_gptc_wdt. >> + >> config LANTIQ_WDT >> tristate "Lantiq SoC watchdog" >> depends on LANTIQ >> diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile >> index 6de2e4ceef190..92c99e4c46eb7 100644 >> --- a/drivers/watchdog/Makefile >> +++ b/drivers/watchdog/Makefile >> @@ -166,6 +166,7 @@ obj-$(CONFIG_TXX9_WDT) += txx9wdt.o >> obj-$(CONFIG_OCTEON_WDT) += octeon-wdt.o >> octeon-wdt-y := octeon-wdt-main.o octeon-wdt-nmi.o >> obj-$(CONFIG_LANTIQ_WDT) += lantiq_wdt.o >> +obj-$(CONFIG_INTEL_LGM_GPTC_WDT) += intel_lgm_gptc_wdt.o >> obj-$(CONFIG_LOONGSON1_WDT) += loongson1_wdt.o >> obj-$(CONFIG_RALINK_WDT) += rt2880_wdt.o >> obj-$(CONFIG_IMGPDC_WDT) += imgpdc_wdt.o >> diff --git a/drivers/watchdog/intel_lgm_gptc_wdt.c b/drivers/watchdog/intel_lgm_gptc_wdt.c >> new file mode 100644 >> index 0000000000000..52be7cc194f8f >> --- /dev/null >> +++ b/drivers/watchdog/intel_lgm_gptc_wdt.c >> @@ -0,0 +1,420 @@ >> +// SPDX-License-Identifier: GPL-2.0 >> +/* >> + * Copyright (C) 2020 Intel Corporation. >> + */ >> + >> +#include <linux/bitfield.h> >> +#include <linux/clk.h> >> +#include <linux/cpu.h> >> +#include <linux/io.h> >> +#include <linux/kernel.h> >> +#include <linux/mfd/syscon.h> >> +#include <linux/module.h> >> +#include <linux/of_device.h> >> +#include <linux/platform_device.h> >> +#include <linux/regmap.h> >> +#include <linux/watchdog.h> >> + >> +#define GPTC_CLC 0x00 >> +#define GPTC_CLC_SUSPEND BIT(4) >> +#define GPTC_CLC_RMC GENMASK(15, 8) >> + >> +/* divider 10 to produce 200 / 10 = 20 MHz clock */ >> +#define CLC_RMC_DIV 10 >> + >> +#define GPTC_CON(X) (0x10 + (X) * 0x20) >> +#define GPTC_CON_CNT_UP BIT(1) >> +#define GPTC_CON_ONESHOT BIT(3) >> +#define GPTC_CON_EXT BIT(4) >> + >> +#define GPTC_RUN(X) (0x18 + (X) * 0x20) >> +#define GPTC_RUN_EN BIT(0) >> +#define GPTC_RUN_STOP BIT(1) >> +#define GPTC_RUN_RELOAD BIT(2) >> + >> +#define GPTC_RLD(X) (0x20 + (X) * 0x20) >> +#define GPTC_CNT(X) (0x28 + (X) * 0x20) >> + >> +#define GPTC_IRNENCLR 0xF0 >> +#define GPTC_IRNEN 0xF4 >> +#define GPTC_IRNCR 0xFC >> + >> +/* Watchdog Timeout Reset register offset and bitfeilds */ >> +#define BIA_WDT_RST_EN 0x1E0 >> +#define BIA_WDT BIT(6) >> + >> +#define MAX_TIMERID 2 >> +#define MAX_CPUID 3 >> +#define TIMER_MARGIN_SEC 300 >> + >> +static bool nowayout = WATCHDOG_NOWAYOUT; >> +module_param(nowayout, bool, 0); >> +MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started\n" >> + " (default=" __MODULE_STRING(WATCHDOG_NOWAYOUT) ")"); >> + >> +struct lgm_gptc_timer { >> + struct lgm_gptc_wdt *wdt_node; >> + struct watchdog_device wdd; >> + unsigned int tid; >> + unsigned int cpuid; >> + unsigned int frequency; >> + unsigned int cycles; >> + bool enable; >> +}; >> + >> +struct lgm_gptc_wdt { >> + struct device *dev; >> + void __iomem *gptc_base; >> + struct regmap *rst_hndl; >> + struct clk *freqclk; >> + struct clk *gateclk; >> + unsigned int fpifreq; >> + enum cpuhp_state state; >> +}; >> + >> +DEFINE_PER_CPU(struct lgm_gptc_timer, lgm_timer_per_cpu); >> + > This is unusual. You'll have to provide a very detailed explanation > why this is needed. Sure will add it. It is required for the hotplug cpu support, and hotplug cpu is added because, the cpus on Lightning Mountain SoC can be online and offline dynamically. If CPUs come to online after the watchdog driver probe, hotplug CPU support helps to configure watchdog timer once CPU is online. Regards, -Dilip > > Guenter > > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 2/2] watchdog: intel: Watchdog timer support on Lightning Mountain 2020-06-09 8:57 ` Dilip Kota @ 2020-06-09 13:42 ` Guenter Roeck 0 siblings, 0 replies; 13+ messages in thread From: Guenter Roeck @ 2020-06-09 13:42 UTC (permalink / raw) To: Dilip Kota, wim, linux-watchdog, devicetree, robbh Cc: linux-kernel, andriy.shevchenko, cheol.yong.kim, qi-ming.wu, yixin.zhu On 6/9/20 1:57 AM, Dilip Kota wrote: > > On 6/8/2020 9:36 PM, Guenter Roeck wrote: >> On 6/7/20 10:49 PM, Dilip Kota wrote: >>> On Intel Lightning Mountain SoC, General Purpose Timer Counter(GPTC) >>> programmable as clocksource, real time clock or watchdog timer. >>> >>> This driver configures GPTC as Watchdog timer and triggers reset signal >>> to CPU on timeout. >>> >>> Signed-off-by: Dilip Kota <eswara.kota@linux.intel.com> >>> --- >>> drivers/watchdog/Kconfig | 13 ++ >>> drivers/watchdog/Makefile | 1 + >>> drivers/watchdog/intel_lgm_gptc_wdt.c | 420 ++++++++++++++++++++++++++++++++++ >>> 3 files changed, 434 insertions(+) >>> create mode 100644 drivers/watchdog/intel_lgm_gptc_wdt.c >>> >>> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig >>> index 0663c604bd642..8009c11e75dda 100644 >>> --- a/drivers/watchdog/Kconfig >>> +++ b/drivers/watchdog/Kconfig >>> @@ -1789,6 +1789,19 @@ config IMGPDC_WDT >>> To compile this driver as a loadable module, choose M here. >>> The module will be called imgpdc_wdt. >>> +config INTEL_LGM_GPTC_WDT >>> + tristate "INTEL LGM SoC Watchdog" >>> + depends on X86 || COMPILE_TEST >>> + depends on OF && HAS_IOMEM >>> + select REGMAP >>> + select MFD_SYSCON >>> + select WATCHDOG_CORE >>> + help >>> + Driver for Watchdog Timer on Intel Lightning Mountain SoC. >>> + >>> + To compile this driver as a loadable module, choose M here. >>> + The module will be called intel_lgm_gptc_wdt. >>> + >>> config LANTIQ_WDT >>> tristate "Lantiq SoC watchdog" >>> depends on LANTIQ >>> diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile >>> index 6de2e4ceef190..92c99e4c46eb7 100644 >>> --- a/drivers/watchdog/Makefile >>> +++ b/drivers/watchdog/Makefile >>> @@ -166,6 +166,7 @@ obj-$(CONFIG_TXX9_WDT) += txx9wdt.o >>> obj-$(CONFIG_OCTEON_WDT) += octeon-wdt.o >>> octeon-wdt-y := octeon-wdt-main.o octeon-wdt-nmi.o >>> obj-$(CONFIG_LANTIQ_WDT) += lantiq_wdt.o >>> +obj-$(CONFIG_INTEL_LGM_GPTC_WDT) += intel_lgm_gptc_wdt.o >>> obj-$(CONFIG_LOONGSON1_WDT) += loongson1_wdt.o >>> obj-$(CONFIG_RALINK_WDT) += rt2880_wdt.o >>> obj-$(CONFIG_IMGPDC_WDT) += imgpdc_wdt.o >>> diff --git a/drivers/watchdog/intel_lgm_gptc_wdt.c b/drivers/watchdog/intel_lgm_gptc_wdt.c >>> new file mode 100644 >>> index 0000000000000..52be7cc194f8f >>> --- /dev/null >>> +++ b/drivers/watchdog/intel_lgm_gptc_wdt.c >>> @@ -0,0 +1,420 @@ >>> +// SPDX-License-Identifier: GPL-2.0 >>> +/* >>> + * Copyright (C) 2020 Intel Corporation. >>> + */ >>> + >>> +#include <linux/bitfield.h> >>> +#include <linux/clk.h> >>> +#include <linux/cpu.h> >>> +#include <linux/io.h> >>> +#include <linux/kernel.h> >>> +#include <linux/mfd/syscon.h> >>> +#include <linux/module.h> >>> +#include <linux/of_device.h> >>> +#include <linux/platform_device.h> >>> +#include <linux/regmap.h> >>> +#include <linux/watchdog.h> >>> + >>> +#define GPTC_CLC 0x00 >>> +#define GPTC_CLC_SUSPEND BIT(4) >>> +#define GPTC_CLC_RMC GENMASK(15, 8) >>> + >>> +/* divider 10 to produce 200 / 10 = 20 MHz clock */ >>> +#define CLC_RMC_DIV 10 >>> + >>> +#define GPTC_CON(X) (0x10 + (X) * 0x20) >>> +#define GPTC_CON_CNT_UP BIT(1) >>> +#define GPTC_CON_ONESHOT BIT(3) >>> +#define GPTC_CON_EXT BIT(4) >>> + >>> +#define GPTC_RUN(X) (0x18 + (X) * 0x20) >>> +#define GPTC_RUN_EN BIT(0) >>> +#define GPTC_RUN_STOP BIT(1) >>> +#define GPTC_RUN_RELOAD BIT(2) >>> + >>> +#define GPTC_RLD(X) (0x20 + (X) * 0x20) >>> +#define GPTC_CNT(X) (0x28 + (X) * 0x20) >>> + >>> +#define GPTC_IRNENCLR 0xF0 >>> +#define GPTC_IRNEN 0xF4 >>> +#define GPTC_IRNCR 0xFC >>> + >>> +/* Watchdog Timeout Reset register offset and bitfeilds */ >>> +#define BIA_WDT_RST_EN 0x1E0 >>> +#define BIA_WDT BIT(6) >>> + >>> +#define MAX_TIMERID 2 >>> +#define MAX_CPUID 3 >>> +#define TIMER_MARGIN_SEC 300 >>> + >>> +static bool nowayout = WATCHDOG_NOWAYOUT; >>> +module_param(nowayout, bool, 0); >>> +MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started\n" >>> + " (default=" __MODULE_STRING(WATCHDOG_NOWAYOUT) ")"); >>> + >>> +struct lgm_gptc_timer { >>> + struct lgm_gptc_wdt *wdt_node; >>> + struct watchdog_device wdd; >>> + unsigned int tid; >>> + unsigned int cpuid; >>> + unsigned int frequency; >>> + unsigned int cycles; >>> + bool enable; >>> +}; >>> + >>> +struct lgm_gptc_wdt { >>> + struct device *dev; >>> + void __iomem *gptc_base; >>> + struct regmap *rst_hndl; >>> + struct clk *freqclk; >>> + struct clk *gateclk; >>> + unsigned int fpifreq; >>> + enum cpuhp_state state; >>> +}; >>> + >>> +DEFINE_PER_CPU(struct lgm_gptc_timer, lgm_timer_per_cpu); >>> + >> This is unusual. You'll have to provide a very detailed explanation >> why this is needed. > Sure will add it. > It is required for the hotplug cpu support, and hotplug cpu is added because, the cpus on Lightning Mountain SoC can be online and offline dynamically. > If CPUs come to online after the watchdog driver probe, hotplug CPU support helps to configure watchdog timer once CPU is online. > There is another watchdog in the system monitoring individual CPUs. The watchdog subsystem monitors the system, not individual CPUs. Individual CUPs are monitored with kernel/watchdog.c, and we should keep it that way. Guenter ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2020-06-11 17:09 UTC | newest] Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-06-08 5:49 [PATCH 0/2] Driver for watchdog timer on Intel Lightning Mountain SoC Dilip Kota 2020-06-08 5:49 ` [PATCH 1/2] dt-bindings: watchdog: intel: Add YAML Schemas for Watchdog timer Dilip Kota 2020-06-08 13:37 ` Guenter Roeck 2020-06-09 8:57 ` Dilip Kota 2020-06-09 13:46 ` Guenter Roeck 2020-06-10 7:54 ` Dilip Kota 2020-06-10 13:05 ` Guenter Roeck 2020-06-11 9:38 ` Dilip Kota 2020-06-11 17:09 ` Guenter Roeck 2020-06-08 5:49 ` [PATCH 2/2] watchdog: intel: Watchdog timer support on Lightning Mountain Dilip Kota 2020-06-08 13:36 ` Guenter Roeck 2020-06-09 8:57 ` Dilip Kota 2020-06-09 13:42 ` Guenter Roeck
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).