From: Peter Griffin <peter.griffin@linaro.org>
To: arnd@arndb.de, robh+dt@kernel.org,
krzysztof.kozlowski+dt@linaro.org, linux@roeck-us.net,
wim@linux-watchdog.org, conor+dt@kernel.org,
alim.akhtar@samsung.com, jaewon02.kim@samsung.com,
chanho61.park@samsung.com, semen.protsenko@linaro.org
Cc: kernel-team@android.com, peter.griffin@linaro.org,
tudor.ambarus@linaro.org, andre.draszik@linaro.org,
saravanak@google.com, willmcvicker@google.com,
linux-fsd@tesla.com, linux-watchdog@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org
Subject: [PATCH 0/9] Add exynos_pmu_update/read/write() APIs to exynos-pmu
Date: Mon, 22 Jan 2024 22:57:01 +0000 [thread overview]
Message-ID: <20240122225710.1952066-1-peter.griffin@linaro.org> (raw)
Hi folks,
This series adds 4 new exynos_pmu_update/read/write() APIs to the exynos-pmu
driver. The APIs are intended to be used by drivers that need to read and
write PMU registers on Exynos based SoCs.
Examples of such Exynos drivers are:
* watchdog
* usb phy
* mipi phy
* ufs phy
Currently upstream most drivers that require PMU register access have a phandle
to the pmu_system_controller in their dt node and obtain a regmap using the
syscon_regmap_lookup_by_phandle() API.
On newer Exynos SoCs this approach is insufficient, for the following reasons
1) Newer Exynos Socs have special set/clear bit hardware on PMU registers.
The intention is this should be used when the register can be accessed by
multiple masters. Support is added for this hw via the QUIRK_HAS_ATOMIC_BITSETHW
and is implemented in exynos_pmu_set_bit_atomic() function.
2) On some SoCs the PMU registers can only be written from secure state (el3).
As Linux needs to write some of these registers a SMC read/write interface
is provided in the secure monitor, and the register offset is checked against
an allowlist. This is done for security hardenning reasons so that normal world
can't tamper with the power to root of trust IPs. Support for this is added
via QUIRK_PMU_ALIVE_WRITE_SEC. This SMC interface is known to be implemented on
gs101 and derivative based SoCs firmware.
The above (2) is a similar mechanism to what drivers/soc/tegra/pmc.c is
providing for it's SoCs where PMC registers can only be accessed from secure
state on some SoCs.
As an example of a consumer of these APIs I have updated the s3c2410_wdt
watchdog driver to use these new exynos_pmu_*() APIs. As the
samsung,syscon-phandle logic has been removed from the s3c2410_wdt.c driver
I have also updated the bindings documentation to mark this as deprecated, and
updated all existing platforms DT to remove samsung,syscon-phandle from the
Watchdog DT node. Please let me know if there is alternative process I should
follow with regards to to deprecating this and updating existing platforms.
This series has been tested on Pixel 6 / gs101. If the various maintainers/
contributors of fsd, exynos850, exynosautov9 etc can test these patches on
your respective systems that would also be most appreciated!
The expectation is this series would be merged via Krzysztofs Samsung Exynos
tree.
Many thanks,
Peter.
Peter Griffin (9):
dt-bindings: watchdog: samsung-wdt: deprecate samsung,syscon-phandle
soc: samsung: exynos-pmu: Add exynos_pmu_update/read/write APIs and
SoC quirks
watchdog: s3c2410_wdt: update to use new exynos_pmu_*() apis
arm64: dts: fsd: remove deprecated samsung,syscon-phandle
arm64: dts: exynosautov9: remove deprecated samsung,syscon-phandle
arm64: dts: exynos850: remove deprecated samsung,syscon-phandle
arm64: dts: exynos7: remove deprecated samsung,syscon-phandle
ARM: dts: samsung: exynos4: remove deprecated samsung,syscon-phandle
ARM: dts: exynos5250: remove deprecated samsung,syscon-phandle
.../bindings/watchdog/samsung-wdt.yaml | 15 +-
arch/arm/boot/dts/samsung/exynos4x12.dtsi | 1 -
arch/arm/boot/dts/samsung/exynos5250.dtsi | 1 -
arch/arm64/boot/dts/exynos/exynos7.dtsi | 1 -
arch/arm64/boot/dts/exynos/exynos850.dtsi | 2 -
arch/arm64/boot/dts/exynos/exynosautov9.dtsi | 2 -
arch/arm64/boot/dts/tesla/fsd.dtsi | 3 -
drivers/soc/samsung/exynos-pmu.c | 209 +++++++++++++++++-
drivers/soc/samsung/exynos-pmu.h | 4 +
drivers/watchdog/Kconfig | 1 +
drivers/watchdog/s3c2410_wdt.c | 25 +--
include/linux/soc/samsung/exynos-pmu.h | 28 +++
12 files changed, 245 insertions(+), 47 deletions(-)
--
2.43.0.429.g432eaa2c6b-goog
next reply other threads:[~2024-01-22 22:57 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-01-22 22:57 Peter Griffin [this message]
2024-01-22 22:57 ` [PATCH 1/9] dt-bindings: watchdog: samsung-wdt: deprecate samsung,syscon-phandle Peter Griffin
2024-01-23 11:09 ` Krzysztof Kozlowski
2024-01-22 22:57 ` [PATCH 2/9] soc: samsung: exynos-pmu: Add exynos_pmu_update/read/write APIs and SoC quirks Peter Griffin
2024-01-23 8:10 ` Arnd Bergmann
2024-01-24 10:21 ` Peter Griffin
2024-01-23 11:17 ` Krzysztof Kozlowski
2024-01-24 10:41 ` Peter Griffin
2024-01-23 18:56 ` Sam Protsenko
2024-01-24 10:02 ` Peter Griffin
2024-01-24 20:23 ` Sam Protsenko
2024-01-25 10:48 ` Peter Griffin
2024-01-26 17:26 ` kernel test robot
2024-01-26 23:07 ` kernel test robot
2024-01-22 22:57 ` [PATCH 3/9] watchdog: s3c2410_wdt: update to use new exynos_pmu_*() apis Peter Griffin
2024-01-23 10:33 ` Guenter Roeck
2024-01-23 15:35 ` Peter Griffin
2024-01-23 11:19 ` Krzysztof Kozlowski
2024-01-23 17:30 ` Peter Griffin
2024-01-23 18:12 ` Krzysztof Kozlowski
2024-01-24 3:37 ` Saravana Kannan
2024-01-24 6:27 ` Krzysztof Kozlowski
2024-01-24 21:27 ` Saravana Kannan
2024-01-25 7:37 ` Krzysztof Kozlowski
2024-01-25 11:46 ` Lee Jones
2024-01-26 2:23 ` Saravana Kannan
2024-01-26 8:43 ` Lee Jones
2024-01-26 2:17 ` Saravana Kannan
2024-01-25 13:19 ` Peter Griffin
2024-01-30 20:27 ` Rob Herring
2024-01-26 17:04 ` kernel test robot
2024-01-22 22:57 ` [PATCH 4/9] arm64: dts: fsd: remove deprecated samsung,syscon-phandle Peter Griffin
2024-01-22 22:57 ` [PATCH 5/9] arm64: dts: exynosautov9: " Peter Griffin
2024-01-22 22:57 ` [PATCH 6/9] arm64: dts: exynos850: " Peter Griffin
2024-01-22 22:57 ` [PATCH 7/9] arm64: dts: exynos7: " Peter Griffin
2024-01-22 22:57 ` [PATCH 8/9] ARM: dts: samsung: exynos4: " Peter Griffin
2024-01-22 22:57 ` [PATCH 9/9] ARM: dts: exynos5250: " Peter Griffin
2024-01-23 7:41 ` Arnd Bergmann
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=20240122225710.1952066-1-peter.griffin@linaro.org \
--to=peter.griffin@linaro.org \
--cc=alim.akhtar@samsung.com \
--cc=andre.draszik@linaro.org \
--cc=arnd@arndb.de \
--cc=chanho61.park@samsung.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=jaewon02.kim@samsung.com \
--cc=kernel-team@android.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-fsd@tesla.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=linux-watchdog@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=robh+dt@kernel.org \
--cc=saravanak@google.com \
--cc=semen.protsenko@linaro.org \
--cc=tudor.ambarus@linaro.org \
--cc=willmcvicker@google.com \
--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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).