* [PATCH 1/3] dt-bindings: power: add Amlogic A5 power domains
2024-06-27 11:47 [PATCH 0/3] Power: A5: add power domain driver Xianwei Zhao via B4 Relay
@ 2024-06-27 11:47 ` Xianwei Zhao via B4 Relay
2024-06-27 14:56 ` Conor Dooley
2024-06-27 11:47 ` [PATCH 2/3] pmdomain: amlogic: Add support for A5 power domains controller Xianwei Zhao via B4 Relay
` (3 subsequent siblings)
4 siblings, 1 reply; 9+ messages in thread
From: Xianwei Zhao via B4 Relay @ 2024-06-27 11:47 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Neil Armstrong,
Kevin Hilman, Jerome Brunet, Martin Blumenstingl, Jianxin Pan,
Ulf Hansson
Cc: devicetree, linux-arm-kernel, linux-amlogic, linux-kernel,
linux-pm, Xianwei Zhao, Hongyu Chen
From: Xianwei Zhao <xianwei.zhao@amlogic.com>
Add devicetree binding document and related header file for
Amlogic A5 secure power domains.
Signed-off-by: Hongyu Chen <hongyu.chen1@amlogic.com>
Signed-off-by: Xianwei Zhao <xianwei.zhao@amlogic.com>
---
.../bindings/power/amlogic,meson-sec-pwrc.yaml | 1 +
include/dt-bindings/power/amlogic,a5-pwrc.h | 21 +++++++++++++++++++++
2 files changed, 22 insertions(+)
diff --git a/Documentation/devicetree/bindings/power/amlogic,meson-sec-pwrc.yaml b/Documentation/devicetree/bindings/power/amlogic,meson-sec-pwrc.yaml
index 59915f9d4860..15d74138baa3 100644
--- a/Documentation/devicetree/bindings/power/amlogic,meson-sec-pwrc.yaml
+++ b/Documentation/devicetree/bindings/power/amlogic,meson-sec-pwrc.yaml
@@ -21,6 +21,7 @@ properties:
- amlogic,meson-a1-pwrc
- amlogic,meson-s4-pwrc
- amlogic,a4-pwrc
+ - amlogic,a5-pwrc
- amlogic,c3-pwrc
- amlogic,t7-pwrc
diff --git a/include/dt-bindings/power/amlogic,a5-pwrc.h b/include/dt-bindings/power/amlogic,a5-pwrc.h
new file mode 100644
index 000000000000..3a6f53eb959f
--- /dev/null
+++ b/include/dt-bindings/power/amlogic,a5-pwrc.h
@@ -0,0 +1,21 @@
+/* SPDX-License-Identifier: (GPL-2.0-only OR MIT) */
+/*
+ * Copyright (C) 2024 Amlogic, Inc. All rights reserved
+ */
+
+#ifndef _DT_BINDINGS_AMLOGIC_A5_POWER_H
+#define _DT_BINDINGS_AMLOGIC_A5_POWER_H
+
+#define PWRC_A5_NNA_ID 0
+#define PWRC_A5_AUDIO_ID 1
+#define PWRC_A5_SDIOA_ID 2
+#define PWRC_A5_EMMC_ID 3
+#define PWRC_A5_USB_COMB_ID 4
+#define PWRC_A5_ETH_ID 5
+#define PWRC_A5_RSA_ID 6
+#define PWRC_A5_AUDIO_PDM_ID 7
+#define PWRC_A5_DMC_ID 8
+#define PWRC_A5_SYS_WRAP_ID 9
+#define PWRC_A5_DSPA_ID 10
+
+#endif
--
2.37.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 1/3] dt-bindings: power: add Amlogic A5 power domains
2024-06-27 11:47 ` [PATCH 1/3] dt-bindings: power: add Amlogic A5 power domains Xianwei Zhao via B4 Relay
@ 2024-06-27 14:56 ` Conor Dooley
0 siblings, 0 replies; 9+ messages in thread
From: Conor Dooley @ 2024-06-27 14:56 UTC (permalink / raw)
To: xianwei.zhao
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Neil Armstrong,
Kevin Hilman, Jerome Brunet, Martin Blumenstingl, Jianxin Pan,
Ulf Hansson, devicetree, linux-arm-kernel, linux-amlogic,
linux-kernel, linux-pm, Hongyu Chen
[-- Attachment #1: Type: text/plain, Size: 402 bytes --]
On Thu, Jun 27, 2024 at 07:47:51PM +0800, Xianwei Zhao via B4 Relay wrote:
> From: Xianwei Zhao <xianwei.zhao@amlogic.com>
>
> Add devicetree binding document and related header file for
> Amlogic A5 secure power domains.
>
> Signed-off-by: Hongyu Chen <hongyu.chen1@amlogic.com>
> Signed-off-by: Xianwei Zhao <xianwei.zhao@amlogic.com>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 2/3] pmdomain: amlogic: Add support for A5 power domains controller
2024-06-27 11:47 [PATCH 0/3] Power: A5: add power domain driver Xianwei Zhao via B4 Relay
2024-06-27 11:47 ` [PATCH 1/3] dt-bindings: power: add Amlogic A5 power domains Xianwei Zhao via B4 Relay
@ 2024-06-27 11:47 ` Xianwei Zhao via B4 Relay
2024-06-27 11:55 ` Neil Armstrong
2024-06-27 11:47 ` [PATCH 3/3] arm64: dts: amlogic: a5: add power domain controller node Xianwei Zhao via B4 Relay
` (2 subsequent siblings)
4 siblings, 1 reply; 9+ messages in thread
From: Xianwei Zhao via B4 Relay @ 2024-06-27 11:47 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Neil Armstrong,
Kevin Hilman, Jerome Brunet, Martin Blumenstingl, Jianxin Pan,
Ulf Hansson
Cc: devicetree, linux-arm-kernel, linux-amlogic, linux-kernel,
linux-pm, Xianwei Zhao, Hongyu Chen
From: Xianwei Zhao <xianwei.zhao@amlogic.com>
Add support for the A5 power controller, whose registers are
in the secure domain and should be accessed via SMC.
Signed-off-by: Hongyu Chen <hongyu.chen1@amlogic.com>
Signed-off-by: Xianwei Zhao <xianwei.zhao@amlogic.com>
---
drivers/pmdomain/amlogic/meson-secure-pwrc.c | 26 ++++++++++++++++++++++++++
1 file changed, 26 insertions(+)
diff --git a/drivers/pmdomain/amlogic/meson-secure-pwrc.c b/drivers/pmdomain/amlogic/meson-secure-pwrc.c
index df5567418226..f6729eea6b8c 100644
--- a/drivers/pmdomain/amlogic/meson-secure-pwrc.c
+++ b/drivers/pmdomain/amlogic/meson-secure-pwrc.c
@@ -15,6 +15,7 @@
#include <dt-bindings/power/meson-s4-power.h>
#include <dt-bindings/power/amlogic,t7-pwrc.h>
#include <dt-bindings/power/amlogic,a4-pwrc.h>
+#include <dt-bindings/power/amlogic,a5-pwrc.h>
#include <linux/arm-smccc.h>
#include <linux/firmware/meson/meson_sm.h>
#include <linux/module.h>
@@ -155,6 +156,22 @@ static struct meson_secure_pwrc_domain_desc a4_pwrc_domains[] = {
SEC_PD(A4_AO_IR, GENPD_FLAG_ALWAYS_ON),
};
+static struct meson_secure_pwrc_domain_desc a5_pwrc_domains[] = {
+ SEC_PD(A5_NNA, 0),
+ SEC_PD(A5_AUDIO, 0),
+ SEC_PD(A5_SDIOA, 0),
+ SEC_PD(A5_EMMC, 0),
+ SEC_PD(A5_USB_COMB, 0),
+ SEC_PD(A5_ETH, 0),
+ SEC_PD(A5_RSA, 0),
+ SEC_PD(A5_AUDIO_PDM, 0),
+ /* DMC is for DDR PHY ana/dig and DMC, and should be always on */
+ SEC_PD(A5_DMC, GENPD_FLAG_ALWAYS_ON),
+ /* WRAP is secure_top, a lot of modules are included, and should be always on */
+ SEC_PD(A5_SYS_WRAP, GENPD_FLAG_ALWAYS_ON),
+ SEC_PD(A5_DSPA, 0),
+};
+
static struct meson_secure_pwrc_domain_desc c3_pwrc_domains[] = {
SEC_PD(C3_NNA, 0),
SEC_PD(C3_AUDIO, 0),
@@ -335,6 +352,11 @@ static struct meson_secure_pwrc_domain_data amlogic_secure_a4_pwrc_data = {
.count = ARRAY_SIZE(a4_pwrc_domains),
};
+static struct meson_secure_pwrc_domain_data amlogic_secure_a5_pwrc_data = {
+ .domains = a5_pwrc_domains,
+ .count = ARRAY_SIZE(a5_pwrc_domains),
+};
+
static struct meson_secure_pwrc_domain_data amlogic_secure_c3_pwrc_data = {
.domains = c3_pwrc_domains,
.count = ARRAY_SIZE(c3_pwrc_domains),
@@ -359,6 +381,10 @@ static const struct of_device_id meson_secure_pwrc_match_table[] = {
.compatible = "amlogic,a4-pwrc",
.data = &amlogic_secure_a4_pwrc_data,
},
+ {
+ .compatible = "amlogic,a5-pwrc",
+ .data = &amlogic_secure_a5_pwrc_data,
+ },
{
.compatible = "amlogic,c3-pwrc",
.data = &amlogic_secure_c3_pwrc_data,
--
2.37.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 2/3] pmdomain: amlogic: Add support for A5 power domains controller
2024-06-27 11:47 ` [PATCH 2/3] pmdomain: amlogic: Add support for A5 power domains controller Xianwei Zhao via B4 Relay
@ 2024-06-27 11:55 ` Neil Armstrong
0 siblings, 0 replies; 9+ messages in thread
From: Neil Armstrong @ 2024-06-27 11:55 UTC (permalink / raw)
To: xianwei.zhao, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Kevin Hilman, Jerome Brunet, Martin Blumenstingl, Jianxin Pan,
Ulf Hansson
Cc: devicetree, linux-arm-kernel, linux-amlogic, linux-kernel,
linux-pm, Hongyu Chen
On 27/06/2024 13:47, Xianwei Zhao via B4 Relay wrote:
> From: Xianwei Zhao <xianwei.zhao@amlogic.com>
>
> Add support for the A5 power controller, whose registers are
> in the secure domain and should be accessed via SMC.
>
> Signed-off-by: Hongyu Chen <hongyu.chen1@amlogic.com>
> Signed-off-by: Xianwei Zhao <xianwei.zhao@amlogic.com>
> ---
> drivers/pmdomain/amlogic/meson-secure-pwrc.c | 26 ++++++++++++++++++++++++++
> 1 file changed, 26 insertions(+)
>
> diff --git a/drivers/pmdomain/amlogic/meson-secure-pwrc.c b/drivers/pmdomain/amlogic/meson-secure-pwrc.c
> index df5567418226..f6729eea6b8c 100644
> --- a/drivers/pmdomain/amlogic/meson-secure-pwrc.c
> +++ b/drivers/pmdomain/amlogic/meson-secure-pwrc.c
> @@ -15,6 +15,7 @@
> #include <dt-bindings/power/meson-s4-power.h>
> #include <dt-bindings/power/amlogic,t7-pwrc.h>
> #include <dt-bindings/power/amlogic,a4-pwrc.h>
> +#include <dt-bindings/power/amlogic,a5-pwrc.h>
> #include <linux/arm-smccc.h>
> #include <linux/firmware/meson/meson_sm.h>
> #include <linux/module.h>
> @@ -155,6 +156,22 @@ static struct meson_secure_pwrc_domain_desc a4_pwrc_domains[] = {
> SEC_PD(A4_AO_IR, GENPD_FLAG_ALWAYS_ON),
> };
>
> +static struct meson_secure_pwrc_domain_desc a5_pwrc_domains[] = {
> + SEC_PD(A5_NNA, 0),
> + SEC_PD(A5_AUDIO, 0),
> + SEC_PD(A5_SDIOA, 0),
> + SEC_PD(A5_EMMC, 0),
> + SEC_PD(A5_USB_COMB, 0),
> + SEC_PD(A5_ETH, 0),
> + SEC_PD(A5_RSA, 0),
> + SEC_PD(A5_AUDIO_PDM, 0),
> + /* DMC is for DDR PHY ana/dig and DMC, and should be always on */
> + SEC_PD(A5_DMC, GENPD_FLAG_ALWAYS_ON),
> + /* WRAP is secure_top, a lot of modules are included, and should be always on */
> + SEC_PD(A5_SYS_WRAP, GENPD_FLAG_ALWAYS_ON),
> + SEC_PD(A5_DSPA, 0),
> +};
> +
> static struct meson_secure_pwrc_domain_desc c3_pwrc_domains[] = {
> SEC_PD(C3_NNA, 0),
> SEC_PD(C3_AUDIO, 0),
> @@ -335,6 +352,11 @@ static struct meson_secure_pwrc_domain_data amlogic_secure_a4_pwrc_data = {
> .count = ARRAY_SIZE(a4_pwrc_domains),
> };
>
> +static struct meson_secure_pwrc_domain_data amlogic_secure_a5_pwrc_data = {
> + .domains = a5_pwrc_domains,
> + .count = ARRAY_SIZE(a5_pwrc_domains),
> +};
> +
> static struct meson_secure_pwrc_domain_data amlogic_secure_c3_pwrc_data = {
> .domains = c3_pwrc_domains,
> .count = ARRAY_SIZE(c3_pwrc_domains),
> @@ -359,6 +381,10 @@ static const struct of_device_id meson_secure_pwrc_match_table[] = {
> .compatible = "amlogic,a4-pwrc",
> .data = &amlogic_secure_a4_pwrc_data,
> },
> + {
> + .compatible = "amlogic,a5-pwrc",
> + .data = &amlogic_secure_a5_pwrc_data,
> + },
> {
> .compatible = "amlogic,c3-pwrc",
> .data = &amlogic_secure_c3_pwrc_data,
>
Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 3/3] arm64: dts: amlogic: a5: add power domain controller node
2024-06-27 11:47 [PATCH 0/3] Power: A5: add power domain driver Xianwei Zhao via B4 Relay
2024-06-27 11:47 ` [PATCH 1/3] dt-bindings: power: add Amlogic A5 power domains Xianwei Zhao via B4 Relay
2024-06-27 11:47 ` [PATCH 2/3] pmdomain: amlogic: Add support for A5 power domains controller Xianwei Zhao via B4 Relay
@ 2024-06-27 11:47 ` Xianwei Zhao via B4 Relay
2024-06-27 11:56 ` Neil Armstrong
2024-07-09 11:14 ` [PATCH 0/3] Power: A5: add power domain driver Ulf Hansson
2024-07-09 15:28 ` (subset) " Neil Armstrong
4 siblings, 1 reply; 9+ messages in thread
From: Xianwei Zhao via B4 Relay @ 2024-06-27 11:47 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Neil Armstrong,
Kevin Hilman, Jerome Brunet, Martin Blumenstingl, Jianxin Pan,
Ulf Hansson
Cc: devicetree, linux-arm-kernel, linux-amlogic, linux-kernel,
linux-pm, Xianwei Zhao, Hongyu Chen
From: Xianwei Zhao <xianwei.zhao@amlogic.com>
Add power domain controller node for Amlogic A5 SoC.
Signed-off-by: Hongyu Chen <hongyu.chen1@amlogic.com>
Signed-off-by: Xianwei Zhao <xianwei.zhao@amlogic.com>
---
arch/arm64/boot/dts/amlogic/amlogic-a5.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm64/boot/dts/amlogic/amlogic-a5.dtsi b/arch/arm64/boot/dts/amlogic/amlogic-a5.dtsi
index 43f68a7da2f7..17a6316de891 100644
--- a/arch/arm64/boot/dts/amlogic/amlogic-a5.dtsi
+++ b/arch/arm64/boot/dts/amlogic/amlogic-a5.dtsi
@@ -4,6 +4,7 @@
*/
#include "amlogic-a4-common.dtsi"
+#include <dt-bindings/power/amlogic,a5-pwrc.h>
/ {
cpus {
#address-cells = <2>;
@@ -37,4 +38,13 @@ cpu3: cpu@300 {
enable-method = "psci";
};
};
+
+ sm: secure-monitor {
+ compatible = "amlogic,meson-gxbb-sm";
+
+ pwrc: power-controller {
+ compatible = "amlogic,a5-pwrc";
+ #power-domain-cells = <1>;
+ };
+ };
};
--
2.37.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 3/3] arm64: dts: amlogic: a5: add power domain controller node
2024-06-27 11:47 ` [PATCH 3/3] arm64: dts: amlogic: a5: add power domain controller node Xianwei Zhao via B4 Relay
@ 2024-06-27 11:56 ` Neil Armstrong
0 siblings, 0 replies; 9+ messages in thread
From: Neil Armstrong @ 2024-06-27 11:56 UTC (permalink / raw)
To: xianwei.zhao, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Kevin Hilman, Jerome Brunet, Martin Blumenstingl, Jianxin Pan,
Ulf Hansson
Cc: devicetree, linux-arm-kernel, linux-amlogic, linux-kernel,
linux-pm, Hongyu Chen
On 27/06/2024 13:47, Xianwei Zhao via B4 Relay wrote:
> From: Xianwei Zhao <xianwei.zhao@amlogic.com>
>
> Add power domain controller node for Amlogic A5 SoC.
>
> Signed-off-by: Hongyu Chen <hongyu.chen1@amlogic.com>
> Signed-off-by: Xianwei Zhao <xianwei.zhao@amlogic.com>
> ---
> arch/arm64/boot/dts/amlogic/amlogic-a5.dtsi | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/amlogic/amlogic-a5.dtsi b/arch/arm64/boot/dts/amlogic/amlogic-a5.dtsi
> index 43f68a7da2f7..17a6316de891 100644
> --- a/arch/arm64/boot/dts/amlogic/amlogic-a5.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/amlogic-a5.dtsi
> @@ -4,6 +4,7 @@
> */
>
> #include "amlogic-a4-common.dtsi"
> +#include <dt-bindings/power/amlogic,a5-pwrc.h>
> / {
> cpus {
> #address-cells = <2>;
> @@ -37,4 +38,13 @@ cpu3: cpu@300 {
> enable-method = "psci";
> };
> };
> +
> + sm: secure-monitor {
> + compatible = "amlogic,meson-gxbb-sm";
> +
> + pwrc: power-controller {
> + compatible = "amlogic,a5-pwrc";
> + #power-domain-cells = <1>;
> + };
> + };
> };
>
Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 0/3] Power: A5: add power domain driver
2024-06-27 11:47 [PATCH 0/3] Power: A5: add power domain driver Xianwei Zhao via B4 Relay
` (2 preceding siblings ...)
2024-06-27 11:47 ` [PATCH 3/3] arm64: dts: amlogic: a5: add power domain controller node Xianwei Zhao via B4 Relay
@ 2024-07-09 11:14 ` Ulf Hansson
2024-07-09 15:28 ` (subset) " Neil Armstrong
4 siblings, 0 replies; 9+ messages in thread
From: Ulf Hansson @ 2024-07-09 11:14 UTC (permalink / raw)
To: xianwei.zhao
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Neil Armstrong,
Kevin Hilman, Jerome Brunet, Martin Blumenstingl, Jianxin Pan,
devicetree, linux-arm-kernel, linux-amlogic, linux-kernel,
linux-pm, Hongyu Chen
On Thu, 27 Jun 2024 at 13:47, Xianwei Zhao via B4 Relay
<devnull+xianwei.zhao.amlogic.com@kernel.org> wrote:
>
> Add power controller driver support for Amlogic A5 SoC.
>
> Signed-off-by: Xianwei Zhao <xianwei.zhao@amlogic.com>
> ---
> Xianwei Zhao (3):
> dt-bindings: power: add Amlogic A5 power domains
> pmdomain: amlogic: Add support for A5 power domains controller
> arm64: dts: amlogic: a5: add power domain controller node
>
> .../bindings/power/amlogic,meson-sec-pwrc.yaml | 1 +
> arch/arm64/boot/dts/amlogic/amlogic-a5.dtsi | 10 +++++++++
> drivers/pmdomain/amlogic/meson-secure-pwrc.c | 26 ++++++++++++++++++++++
> include/dt-bindings/power/amlogic,a5-pwrc.h | 21 +++++++++++++++++
> 4 files changed, 58 insertions(+)
> ---
Patch 1 and 2 applied for next (patch 1 is available at the immutable
dt branch too), thanks!
Kind regards
Uffe
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: (subset) [PATCH 0/3] Power: A5: add power domain driver
2024-06-27 11:47 [PATCH 0/3] Power: A5: add power domain driver Xianwei Zhao via B4 Relay
` (3 preceding siblings ...)
2024-07-09 11:14 ` [PATCH 0/3] Power: A5: add power domain driver Ulf Hansson
@ 2024-07-09 15:28 ` Neil Armstrong
4 siblings, 0 replies; 9+ messages in thread
From: Neil Armstrong @ 2024-07-09 15:28 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Jianxin Pan, Ulf Hansson,
Xianwei Zhao
Cc: devicetree, linux-arm-kernel, linux-amlogic, linux-kernel,
linux-pm, Hongyu Chen
Hi,
On Thu, 27 Jun 2024 19:47:50 +0800, Xianwei Zhao wrote:
> Add power controller driver support for Amlogic A5 SoC.
>
>
Thanks, Applied to https://git.kernel.org/pub/scm/linux/kernel/git/amlogic/linux.git (v6.11/arm64-dt)
[3/3] arm64: dts: amlogic: a5: add power domain controller node
https://git.kernel.org/amlogic/c/c087cd0ef1b0c2adfabd216e28dfed242b5f4f07
These changes has been applied on the intermediate git tree [1].
The v6.11/arm64-dt branch will then be sent via a formal Pull Request to the Linux SoC maintainers
for inclusion in their intermediate git branches in order to be sent to Linus during
the next merge window, or sooner if it's a set of fixes.
In the cases of fixes, those will be merged in the current release candidate
kernel and as soon they appear on the Linux master branch they will be
backported to the previous Stable and Long-Stable kernels [2].
The intermediate git branches are merged daily in the linux-next tree [3],
people are encouraged testing these pre-release kernels and report issues on the
relevant mailing-lists.
If problems are discovered on those changes, please submit a signed-off-by revert
patch followed by a corrective changeset.
[1] https://git.kernel.org/pub/scm/linux/kernel/git/amlogic/linux.git
[2] https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
[3] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
--
Neil
^ permalink raw reply [flat|nested] 9+ messages in thread