From: Sergio Paracuellos <sergio.paracuellos@gmail.com>
To: linux-watchdog@vger.kernel.org
Cc: wim@linux-watchdog.org, linux@roeck-us.net, robh+dt@kernel.org,
krzysztof.kozlowski+dt@linaro.org, matthias.bgg@gmail.com,
arinc.unal@arinc9.com, tsbogend@alpha.franken.de,
p.zabel@pengutronix.de, linux-kernel@vger.kernel.org,
devicetree@vger.kernel.org, linux-mips@vger.kernel.org
Subject: [PATCH v3 0/4] watchdog: mt7621-wdt: avoid globals and arch dependencies
Date: Fri, 10 Feb 2023 18:38:37 +0100 [thread overview]
Message-ID: <20230210173841.705783-1-sergio.paracuellos@gmail.com> (raw)
Hi all,
This series make an update in the MT7621 SoC's watchdog driver code. This
SoC already provides a system controller node to access reset status
register needed for the watchdog. Instead of using MIPS architecture
dependent operations in header 'asm/mach-ralink/ralink_regs.h' use
a phandle to the system controller node and use it through regmap APIs
from the code. Driver is also using some globals that are not needed at
all if a driver data structure is used along the code. Hence, add all
new needed stuff inside a new driver data structure. With this changes
driver can be properly compile tested.
Thanks in advance for reviewing this!
V1 of this series here [0].
V2 os thise series here [1].
Changes in v3:
- rename phandler from 'ralink,sysctl' into 'mediatek,sysctl'.
- Drop error message added in PATHC 3 that modifies functionality and we
only want to maintain current functionaloty by now.
Changes in v2:
- Remove no needed compatible 'syscon' from bindings.
- Rewrite new syscon phandle description in bindings.
- Remove 'syscon' from compatible in 'mt7621.dtsi'.
- Split PATCH 3 into two different patches:
- PATCH 3: removes static globals using a driver data structure.
- PATCH 4: remove ralink architecture dependent includes and code.
Best regards,
Sergio Paracuellos
[0]: https://lore.kernel.org/linux-watchdog/20230210065621.598120-1-sergio.paracuellos@gmail.com/T/#t
[1]: https://lore.kernel.org/linux-watchdog/20230210121735.639089-1-sergio.paracuellos@gmail.com/T/#t
Sergio Paracuellos (4):
dt-bindings: watchdog: mt7621-wdt: add phandle to access system
controller registers
mips: dts: ralink: mt7621: add phandle to system controller node for
watchdog
watchdog: mt7621-wdt: avoid static global declarations
watchdog: mt7621-wdt: avoid ralink architecture dependent code
.../watchdog/mediatek,mt7621-wdt.yaml | 7 ++
arch/mips/boot/dts/ralink/mt7621.dtsi | 1 +
drivers/watchdog/Kconfig | 2 +
drivers/watchdog/mt7621_wdt.c | 119 ++++++++++++------
4 files changed, 88 insertions(+), 41 deletions(-)
--
2.25.1
next reply other threads:[~2023-02-10 17:38 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-10 17:38 Sergio Paracuellos [this message]
2023-02-10 17:38 ` [PATCH v3 1/4] dt-bindings: watchdog: mt7621-wdt: add phandle to access system controller registers Sergio Paracuellos
2023-02-10 17:38 ` [PATCH v3 2/4] mips: dts: ralink: mt7621: add phandle to system controller node for watchdog Sergio Paracuellos
2023-02-10 17:46 ` Arınç ÜNAL
2023-02-11 7:16 ` Sergio Paracuellos
2023-02-10 17:38 ` [PATCH v3 3/4] watchdog: mt7621-wdt: avoid static global declarations Sergio Paracuellos
2023-02-10 18:46 ` Guenter Roeck
2023-02-10 17:38 ` [PATCH v3 4/4] watchdog: mt7621-wdt: avoid ralink architecture dependent code Sergio Paracuellos
2023-02-10 18:47 ` Guenter Roeck
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230210173841.705783-1-sergio.paracuellos@gmail.com \
--to=sergio.paracuellos@gmail.com \
--cc=arinc.unal@arinc9.com \
--cc=devicetree@vger.kernel.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@vger.kernel.org \
--cc=linux-watchdog@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=matthias.bgg@gmail.com \
--cc=p.zabel@pengutronix.de \
--cc=robh+dt@kernel.org \
--cc=tsbogend@alpha.franken.de \
--cc=wim@linux-watchdog.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox