* [PATCH 02/14] power: domain: bcm6328: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
@ 2026-05-08 12:22 ` Marek Vasut
2026-05-13 10:00 ` Peng Fan
2026-05-08 12:22 ` [PATCH 03/14] power: domain: imx8-legacy: " Marek Vasut
` (12 subsequent siblings)
13 siblings, 1 reply; 31+ messages in thread
From: Marek Vasut @ 2026-05-08 12:22 UTC (permalink / raw)
To: u-boot; +Cc: Marek Vasut, Jaehoon Chung, Peng Fan, Tom Rini
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
drivers/power/domain/bcm6328-power-domain.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/domain/bcm6328-power-domain.c b/drivers/power/domain/bcm6328-power-domain.c
index 36b5a933748..5b449f4c29d 100644
--- a/drivers/power/domain/bcm6328-power-domain.c
+++ b/drivers/power/domain/bcm6328-power-domain.c
@@ -57,7 +57,7 @@ static const struct udevice_id bcm6328_power_domain_ids[] = {
{ /* sentinel */ }
};
-struct power_domain_ops bcm6328_power_domain_ops = {
+static const struct power_domain_ops bcm6328_power_domain_ops = {
.off = bcm6328_power_domain_off,
.on = bcm6328_power_domain_on,
.request = bcm6328_power_domain_request,
--
2.53.0
^ permalink raw reply related [flat|nested] 31+ messages in thread* [PATCH 03/14] power: domain: imx8-legacy: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
2026-05-08 12:22 ` [PATCH 02/14] power: domain: bcm6328: " Marek Vasut
@ 2026-05-08 12:22 ` Marek Vasut
2026-05-13 10:01 ` Peng Fan
2026-05-08 12:22 ` [PATCH 04/14] power: domain: imx8: " Marek Vasut
` (11 subsequent siblings)
13 siblings, 1 reply; 31+ messages in thread
From: Marek Vasut @ 2026-05-08 12:22 UTC (permalink / raw)
To: u-boot; +Cc: Marek Vasut, Jaehoon Chung, Peng Fan, Tom Rini
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
drivers/power/domain/imx8-power-domain-legacy.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/domain/imx8-power-domain-legacy.c b/drivers/power/domain/imx8-power-domain-legacy.c
index 713a51d7807..a646f667039 100644
--- a/drivers/power/domain/imx8-power-domain-legacy.c
+++ b/drivers/power/domain/imx8-power-domain-legacy.c
@@ -347,7 +347,7 @@ static const struct udevice_id imx8_power_domain_ids[] = {
{ }
};
-struct power_domain_ops imx8_power_domain_ops = {
+static const struct power_domain_ops imx8_power_domain_ops = {
.on = imx8_power_domain_on,
.off = imx8_power_domain_off,
.of_xlate = imx8_power_domain_of_xlate,
--
2.53.0
^ permalink raw reply related [flat|nested] 31+ messages in thread* [PATCH 04/14] power: domain: imx8: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
2026-05-08 12:22 ` [PATCH 02/14] power: domain: bcm6328: " Marek Vasut
2026-05-08 12:22 ` [PATCH 03/14] power: domain: imx8-legacy: " Marek Vasut
@ 2026-05-08 12:22 ` Marek Vasut
2026-05-13 10:01 ` Peng Fan
2026-05-08 12:22 ` [PATCH 05/14] power: domain: imx8m: " Marek Vasut
` (10 subsequent siblings)
13 siblings, 1 reply; 31+ messages in thread
From: Marek Vasut @ 2026-05-08 12:22 UTC (permalink / raw)
To: u-boot; +Cc: Marek Vasut, Jaehoon Chung, Peng Fan, Tom Rini
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
drivers/power/domain/imx8-power-domain.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/domain/imx8-power-domain.c b/drivers/power/domain/imx8-power-domain.c
index e8dcc057fee..5740cd686db 100644
--- a/drivers/power/domain/imx8-power-domain.c
+++ b/drivers/power/domain/imx8-power-domain.c
@@ -51,7 +51,7 @@ static const struct udevice_id imx8_power_domain_ids[] = {
{ }
};
-struct power_domain_ops imx8_power_domain_ops_v2 = {
+static const struct power_domain_ops imx8_power_domain_ops_v2 = {
.on = imx8_power_domain_on,
.off = imx8_power_domain_off,
};
--
2.53.0
^ permalink raw reply related [flat|nested] 31+ messages in thread* [PATCH 05/14] power: domain: imx8m: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
` (2 preceding siblings ...)
2026-05-08 12:22 ` [PATCH 04/14] power: domain: imx8: " Marek Vasut
@ 2026-05-08 12:22 ` Marek Vasut
2026-05-13 10:01 ` Peng Fan
2026-05-08 12:22 ` [PATCH 06/14] power: domain: imx8mp-mediamix: " Marek Vasut
` (9 subsequent siblings)
13 siblings, 1 reply; 31+ messages in thread
From: Marek Vasut @ 2026-05-08 12:22 UTC (permalink / raw)
To: u-boot
Cc: Marek Vasut, Frieder Schrempf, Jaehoon Chung, Miquel Raynal,
Patrice Chotard, Peng Fan, Tom Rini, Yao Zi
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Frieder Schrempf <frieder.schrempf@kontron.de>
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Miquel Raynal <miquel.raynal@bootlin.com>
Cc: Patrice Chotard <patrice.chotard@foss.st.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: Yao Zi <me@ziyao.cc>
Cc: u-boot@lists.denx.de
---
drivers/power/domain/imx8m-power-domain.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/domain/imx8m-power-domain.c b/drivers/power/domain/imx8m-power-domain.c
index 1c731b897cc..5fdb95fb6a7 100644
--- a/drivers/power/domain/imx8m-power-domain.c
+++ b/drivers/power/domain/imx8m-power-domain.c
@@ -558,7 +558,7 @@ static const struct udevice_id imx8m_power_domain_ids[] = {
{ }
};
-struct power_domain_ops imx8m_power_domain_ops = {
+static const struct power_domain_ops imx8m_power_domain_ops = {
.on = imx8m_power_domain_on,
.off = imx8m_power_domain_off,
.of_xlate = imx8m_power_domain_of_xlate,
--
2.53.0
^ permalink raw reply related [flat|nested] 31+ messages in thread* Re: [PATCH 05/14] power: domain: imx8m: Staticize and constify driver ops
2026-05-08 12:22 ` [PATCH 05/14] power: domain: imx8m: " Marek Vasut
@ 2026-05-13 10:01 ` Peng Fan
0 siblings, 0 replies; 31+ messages in thread
From: Peng Fan @ 2026-05-13 10:01 UTC (permalink / raw)
To: Marek Vasut
Cc: u-boot, Frieder Schrempf, Jaehoon Chung, Miquel Raynal,
Patrice Chotard, Peng Fan, Tom Rini, Yao Zi
On Fri, May 08, 2026 at 02:22:05PM +0200, Marek Vasut wrote:
>Set the ops structure as static const. The structure is not accessible
>from outside of this driver and is not going to be modified at runtime.
>
>Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
^ permalink raw reply [flat|nested] 31+ messages in thread
* [PATCH 06/14] power: domain: imx8mp-mediamix: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
` (3 preceding siblings ...)
2026-05-08 12:22 ` [PATCH 05/14] power: domain: imx8m: " Marek Vasut
@ 2026-05-08 12:22 ` Marek Vasut
2026-05-13 10:02 ` Peng Fan
2026-05-08 12:22 ` [PATCH 07/14] power: domain: meson-ee-pwrc: " Marek Vasut
` (8 subsequent siblings)
13 siblings, 1 reply; 31+ messages in thread
From: Marek Vasut @ 2026-05-08 12:22 UTC (permalink / raw)
To: u-boot; +Cc: Marek Vasut, Jaehoon Chung, Peng Fan, Tom Rini
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
drivers/power/domain/imx8mp-mediamix.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/domain/imx8mp-mediamix.c b/drivers/power/domain/imx8mp-mediamix.c
index 504c22f7d36..66ea5d8e60c 100644
--- a/drivers/power/domain/imx8mp-mediamix.c
+++ b/drivers/power/domain/imx8mp-mediamix.c
@@ -194,7 +194,7 @@ static const struct udevice_id imx8mp_mediamix_ids[] = {
{ }
};
-struct power_domain_ops imx8mp_mediamix_ops = {
+static const struct power_domain_ops imx8mp_mediamix_ops = {
.on = imx8mp_mediamix_on,
.off = imx8mp_mediamix_off,
.of_xlate = imx8mp_mediamix_of_xlate,
--
2.53.0
^ permalink raw reply related [flat|nested] 31+ messages in thread* [PATCH 07/14] power: domain: meson-ee-pwrc: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
` (4 preceding siblings ...)
2026-05-08 12:22 ` [PATCH 06/14] power: domain: imx8mp-mediamix: " Marek Vasut
@ 2026-05-08 12:22 ` Marek Vasut
2026-05-13 10:02 ` Peng Fan
2026-05-08 12:22 ` [PATCH 08/14] power: domain: meson-gx-pwrc: " Marek Vasut
` (7 subsequent siblings)
13 siblings, 1 reply; 31+ messages in thread
From: Marek Vasut @ 2026-05-08 12:22 UTC (permalink / raw)
To: u-boot
Cc: Marek Vasut, Jaehoon Chung, Neil Armstrong, Peng Fan, Tom Rini,
u-boot-amlogic
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Neil Armstrong <neil.armstrong@linaro.org>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot-amlogic@groups.io
Cc: u-boot@lists.denx.de
---
drivers/power/domain/meson-ee-pwrc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/domain/meson-ee-pwrc.c b/drivers/power/domain/meson-ee-pwrc.c
index 4d9f3bba644..6361f3a6c59 100644
--- a/drivers/power/domain/meson-ee-pwrc.c
+++ b/drivers/power/domain/meson-ee-pwrc.c
@@ -386,7 +386,7 @@ static int meson_ee_pwrc_of_xlate(struct power_domain *power_domain,
return 0;
}
-struct power_domain_ops meson_ee_pwrc_ops = {
+static const struct power_domain_ops meson_ee_pwrc_ops = {
.off = meson_ee_pwrc_off,
.on = meson_ee_pwrc_on,
.of_xlate = meson_ee_pwrc_of_xlate,
--
2.53.0
^ permalink raw reply related [flat|nested] 31+ messages in thread* [PATCH 08/14] power: domain: meson-gx-pwrc: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
` (5 preceding siblings ...)
2026-05-08 12:22 ` [PATCH 07/14] power: domain: meson-ee-pwrc: " Marek Vasut
@ 2026-05-08 12:22 ` Marek Vasut
2026-05-13 10:02 ` Peng Fan
2026-05-08 12:22 ` [PATCH 09/14] power: domain: meson-secure-pwrc: " Marek Vasut
` (6 subsequent siblings)
13 siblings, 1 reply; 31+ messages in thread
From: Marek Vasut @ 2026-05-08 12:22 UTC (permalink / raw)
To: u-boot
Cc: Marek Vasut, Jaehoon Chung, Neil Armstrong, Peng Fan, Tom Rini,
u-boot-amlogic
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Neil Armstrong <neil.armstrong@linaro.org>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot-amlogic@groups.io
Cc: u-boot@lists.denx.de
---
drivers/power/domain/meson-gx-pwrc-vpu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/domain/meson-gx-pwrc-vpu.c b/drivers/power/domain/meson-gx-pwrc-vpu.c
index 1c56e8508c3..325296b0dd7 100644
--- a/drivers/power/domain/meson-gx-pwrc-vpu.c
+++ b/drivers/power/domain/meson-gx-pwrc-vpu.c
@@ -262,7 +262,7 @@ static int meson_pwrc_vpu_of_xlate(struct power_domain *power_domain,
return 0;
}
-struct power_domain_ops meson_gx_pwrc_vpu_ops = {
+static const struct power_domain_ops meson_gx_pwrc_vpu_ops = {
.off = meson_pwrc_vpu_off,
.on = meson_pwrc_vpu_on,
.of_xlate = meson_pwrc_vpu_of_xlate,
--
2.53.0
^ permalink raw reply related [flat|nested] 31+ messages in thread* [PATCH 09/14] power: domain: meson-secure-pwrc: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
` (6 preceding siblings ...)
2026-05-08 12:22 ` [PATCH 08/14] power: domain: meson-gx-pwrc: " Marek Vasut
@ 2026-05-08 12:22 ` Marek Vasut
2026-05-13 10:02 ` Peng Fan
2026-05-08 12:22 ` [PATCH 10/14] power: domain: mtk: " Marek Vasut
` (5 subsequent siblings)
13 siblings, 1 reply; 31+ messages in thread
From: Marek Vasut @ 2026-05-08 12:22 UTC (permalink / raw)
To: u-boot
Cc: Marek Vasut, Jaehoon Chung, Neil Armstrong, Peng Fan, Tom Rini,
u-boot-amlogic
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Neil Armstrong <neil.armstrong@linaro.org>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot-amlogic@groups.io
Cc: u-boot@lists.denx.de
---
drivers/power/domain/meson-secure-pwrc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/domain/meson-secure-pwrc.c b/drivers/power/domain/meson-secure-pwrc.c
index f70f8e02423..1b82b58f3e5 100644
--- a/drivers/power/domain/meson-secure-pwrc.c
+++ b/drivers/power/domain/meson-secure-pwrc.c
@@ -120,7 +120,7 @@ static struct meson_secure_pwrc_domain_desc a1_pwrc_domains[] = {
SEC_PD(RSA),
};
-struct power_domain_ops meson_secure_pwrc_ops = {
+static const struct power_domain_ops meson_secure_pwrc_ops = {
.on = meson_secure_pwrc_on,
.off = meson_secure_pwrc_off,
.of_xlate = meson_secure_pwrc_of_xlate,
--
2.53.0
^ permalink raw reply related [flat|nested] 31+ messages in thread* [PATCH 10/14] power: domain: mtk: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
` (7 preceding siblings ...)
2026-05-08 12:22 ` [PATCH 09/14] power: domain: meson-secure-pwrc: " Marek Vasut
@ 2026-05-08 12:22 ` Marek Vasut
2026-05-08 12:58 ` David Lechner
2026-05-13 10:03 ` Peng Fan
2026-05-08 12:22 ` [PATCH 11/14] power: domain: sandbox: " Marek Vasut
` (4 subsequent siblings)
13 siblings, 2 replies; 31+ messages in thread
From: Marek Vasut @ 2026-05-08 12:22 UTC (permalink / raw)
To: u-boot
Cc: Marek Vasut, Chunfeng Yun, David Lechner, GSS_MTK_Uboot_upstream,
Igor Belwon, Jaehoon Chung, Julien Stephan, Peng Fan, Ryder Lee,
Tom Rini, Weijie Gao
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Chunfeng Yun <chunfeng.yun@mediatek.com>
Cc: David Lechner <dlechner@baylibre.com>
Cc: GSS_MTK_Uboot_upstream <GSS_MTK_Uboot_upstream@mediatek.com>
Cc: Igor Belwon <igor.belwon@mentallysanemainliners.org>
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Julien Stephan <jstephan@baylibre.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Ryder Lee <ryder.lee@mediatek.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: Weijie Gao <weijie.gao@mediatek.com>
Cc: u-boot@lists.denx.de
---
drivers/power/domain/mtk-power-domain.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/domain/mtk-power-domain.c b/drivers/power/domain/mtk-power-domain.c
index 2d1ba1855a5..24dd540897d 100644
--- a/drivers/power/domain/mtk-power-domain.c
+++ b/drivers/power/domain/mtk-power-domain.c
@@ -392,7 +392,7 @@ static const struct udevice_id mtk_power_domain_ids[] = {
{ /* sentinel */ }
};
-struct power_domain_ops mtk_power_domain_ops = {
+static const struct power_domain_ops mtk_power_domain_ops = {
.off = scpsys_power_off,
.on = scpsys_power_on,
.request = scpsys_power_request,
--
2.53.0
^ permalink raw reply related [flat|nested] 31+ messages in thread* Re: [PATCH 10/14] power: domain: mtk: Staticize and constify driver ops
2026-05-08 12:22 ` [PATCH 10/14] power: domain: mtk: " Marek Vasut
@ 2026-05-08 12:58 ` David Lechner
2026-05-13 10:03 ` Peng Fan
1 sibling, 0 replies; 31+ messages in thread
From: David Lechner @ 2026-05-08 12:58 UTC (permalink / raw)
To: Marek Vasut, u-boot
Cc: Chunfeng Yun, GSS_MTK_Uboot_upstream, Igor Belwon, Jaehoon Chung,
Julien Stephan, Peng Fan, Ryder Lee, Tom Rini, Weijie Gao
On 5/8/26 7:22 AM, Marek Vasut wrote:
> Set the ops structure as static const. The structure is not accessible
> from outside of this driver and is not going to be modified at runtime.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
> ---
Reviewed-by: David Lechner <dlechner@baylibre.com>
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [PATCH 10/14] power: domain: mtk: Staticize and constify driver ops
2026-05-08 12:22 ` [PATCH 10/14] power: domain: mtk: " Marek Vasut
2026-05-08 12:58 ` David Lechner
@ 2026-05-13 10:03 ` Peng Fan
1 sibling, 0 replies; 31+ messages in thread
From: Peng Fan @ 2026-05-13 10:03 UTC (permalink / raw)
To: Marek Vasut
Cc: u-boot, Chunfeng Yun, David Lechner, GSS_MTK_Uboot_upstream,
Igor Belwon, Jaehoon Chung, Julien Stephan, Peng Fan, Ryder Lee,
Tom Rini, Weijie Gao
On Fri, May 08, 2026 at 02:22:10PM +0200, Marek Vasut wrote:
>Set the ops structure as static const. The structure is not accessible
>from outside of this driver and is not going to be modified at runtime.
>
>Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
^ permalink raw reply [flat|nested] 31+ messages in thread
* [PATCH 11/14] power: domain: sandbox: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
` (8 preceding siblings ...)
2026-05-08 12:22 ` [PATCH 10/14] power: domain: mtk: " Marek Vasut
@ 2026-05-08 12:22 ` Marek Vasut
2026-05-08 17:49 ` Simon Glass
2026-05-13 10:03 ` Peng Fan
2026-05-08 12:22 ` [PATCH 12/14] power: domain: scmi: " Marek Vasut
` (3 subsequent siblings)
13 siblings, 2 replies; 31+ messages in thread
From: Marek Vasut @ 2026-05-08 12:22 UTC (permalink / raw)
To: u-boot; +Cc: Marek Vasut, Jaehoon Chung, Peng Fan, Tom Rini
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
drivers/power/domain/sandbox-power-domain.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/domain/sandbox-power-domain.c b/drivers/power/domain/sandbox-power-domain.c
index a8031657638..a30826a3b4d 100644
--- a/drivers/power/domain/sandbox-power-domain.c
+++ b/drivers/power/domain/sandbox-power-domain.c
@@ -78,7 +78,7 @@ static const struct udevice_id sandbox_power_domain_ids[] = {
{ }
};
-struct power_domain_ops sandbox_power_domain_ops = {
+static const struct power_domain_ops sandbox_power_domain_ops = {
.request = sandbox_power_domain_request,
.rfree = sandbox_power_domain_free,
.on = sandbox_power_domain_on,
--
2.53.0
^ permalink raw reply related [flat|nested] 31+ messages in thread* Re: [PATCH 11/14] power: domain: sandbox: Staticize and constify driver ops
2026-05-08 12:22 ` [PATCH 11/14] power: domain: sandbox: " Marek Vasut
@ 2026-05-08 17:49 ` Simon Glass
2026-05-13 10:03 ` Peng Fan
1 sibling, 0 replies; 31+ messages in thread
From: Simon Glass @ 2026-05-08 17:49 UTC (permalink / raw)
To: marek.vasut+renesas; +Cc: u-boot, Jaehoon Chung, Peng Fan, Tom Rini
On 2026-05-08T12:22:01, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:
> power: domain: sandbox: Staticize and constify driver ops
>
> Set the ops structure as static const. The structure is not accessible
> from outside of this driver and is not going to be modified at runtime.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
>
> drivers/power/domain/sandbox-power-domain.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [PATCH 11/14] power: domain: sandbox: Staticize and constify driver ops
2026-05-08 12:22 ` [PATCH 11/14] power: domain: sandbox: " Marek Vasut
2026-05-08 17:49 ` Simon Glass
@ 2026-05-13 10:03 ` Peng Fan
1 sibling, 0 replies; 31+ messages in thread
From: Peng Fan @ 2026-05-13 10:03 UTC (permalink / raw)
To: Marek Vasut; +Cc: u-boot, Jaehoon Chung, Peng Fan, Tom Rini
On Fri, May 08, 2026 at 02:22:11PM +0200, Marek Vasut wrote:
>Set the ops structure as static const. The structure is not accessible
>from outside of this driver and is not going to be modified at runtime.
>
>Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
^ permalink raw reply [flat|nested] 31+ messages in thread
* [PATCH 12/14] power: domain: scmi: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
` (9 preceding siblings ...)
2026-05-08 12:22 ` [PATCH 11/14] power: domain: sandbox: " Marek Vasut
@ 2026-05-08 12:22 ` Marek Vasut
2026-05-13 10:03 ` Peng Fan
2026-05-08 12:22 ` [PATCH 13/14] power: domain: tegra186: " Marek Vasut
` (2 subsequent siblings)
13 siblings, 1 reply; 31+ messages in thread
From: Marek Vasut @ 2026-05-08 12:22 UTC (permalink / raw)
To: u-boot; +Cc: Marek Vasut, Jaehoon Chung, Peng Fan, Tom Rini
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
drivers/power/domain/scmi-power-domain.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/domain/scmi-power-domain.c b/drivers/power/domain/scmi-power-domain.c
index e8c0ba8878e..6dcc259ad8f 100644
--- a/drivers/power/domain/scmi-power-domain.c
+++ b/drivers/power/domain/scmi-power-domain.c
@@ -179,7 +179,7 @@ static int scmi_power_domain_probe(struct udevice *dev)
return 0;
}
-struct power_domain_ops scmi_power_domain_ops = {
+static const struct power_domain_ops scmi_power_domain_ops = {
.on = scmi_power_domain_on,
.off = scmi_power_domain_off,
};
--
2.53.0
^ permalink raw reply related [flat|nested] 31+ messages in thread* [PATCH 13/14] power: domain: tegra186: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
` (10 preceding siblings ...)
2026-05-08 12:22 ` [PATCH 12/14] power: domain: scmi: " Marek Vasut
@ 2026-05-08 12:22 ` Marek Vasut
2026-05-13 10:03 ` Peng Fan
2026-05-08 12:22 ` [PATCH 14/14] power: domain: zynqmp: " Marek Vasut
2026-05-13 10:00 ` [PATCH 01/14] power: domain: apple: " Peng Fan
13 siblings, 1 reply; 31+ messages in thread
From: Marek Vasut @ 2026-05-08 12:22 UTC (permalink / raw)
To: u-boot; +Cc: Marek Vasut, Jaehoon Chung, Peng Fan, Tom Rini
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
drivers/power/domain/tegra186-power-domain.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/domain/tegra186-power-domain.c b/drivers/power/domain/tegra186-power-domain.c
index 334c460c805..3865cd4cf47 100644
--- a/drivers/power/domain/tegra186-power-domain.c
+++ b/drivers/power/domain/tegra186-power-domain.c
@@ -55,7 +55,7 @@ static int tegra186_power_domain_off(struct power_domain *power_domain)
return tegra186_power_domain_common(power_domain, false);
}
-struct power_domain_ops tegra186_power_domain_ops = {
+static const struct power_domain_ops tegra186_power_domain_ops = {
.on = tegra186_power_domain_on,
.off = tegra186_power_domain_off,
};
--
2.53.0
^ permalink raw reply related [flat|nested] 31+ messages in thread* [PATCH 14/14] power: domain: zynqmp: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
` (11 preceding siblings ...)
2026-05-08 12:22 ` [PATCH 13/14] power: domain: tegra186: " Marek Vasut
@ 2026-05-08 12:22 ` Marek Vasut
2026-05-11 6:35 ` Michal Simek
2026-05-13 10:04 ` Peng Fan
2026-05-13 10:00 ` [PATCH 01/14] power: domain: apple: " Peng Fan
13 siblings, 2 replies; 31+ messages in thread
From: Marek Vasut @ 2026-05-08 12:22 UTC (permalink / raw)
To: u-boot
Cc: Marek Vasut, Jaehoon Chung, Michal Simek, Naman Trivedi, Peng Fan,
Senthil Nathan Thangaraj, Tom Rini, Venkatesh Yadav Abbarapu
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Jaehoon Chung <jh80.chung@samsung.com>
Cc: Michal Simek <michal.simek@amd.com>
Cc: Naman Trivedi <naman.trivedimanojbhai@amd.com>
Cc: Peng Fan <peng.fan@nxp.com>
Cc: Senthil Nathan Thangaraj <senthilnathan.thangaraj@amd.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: Venkatesh Yadav Abbarapu <venkatesh.abbarapu@amd.com>
Cc: u-boot@lists.denx.de
---
drivers/power/domain/zynqmp-power-domain.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/power/domain/zynqmp-power-domain.c b/drivers/power/domain/zynqmp-power-domain.c
index a54de5c1439..0acfc54e787 100644
--- a/drivers/power/domain/zynqmp-power-domain.c
+++ b/drivers/power/domain/zynqmp-power-domain.c
@@ -57,7 +57,7 @@ static int zynqmp_power_domain_off(struct power_domain *power_domain)
return 0;
}
-struct power_domain_ops zynqmp_power_domain_ops = {
+static const struct power_domain_ops zynqmp_power_domain_ops = {
.request = zynqmp_power_domain_request,
.rfree = zynqmp_power_domain_free,
.on = zynqmp_power_domain_on,
--
2.53.0
^ permalink raw reply related [flat|nested] 31+ messages in thread* Re: [PATCH 14/14] power: domain: zynqmp: Staticize and constify driver ops
2026-05-08 12:22 ` [PATCH 14/14] power: domain: zynqmp: " Marek Vasut
@ 2026-05-11 6:35 ` Michal Simek
2026-05-13 10:04 ` Peng Fan
1 sibling, 0 replies; 31+ messages in thread
From: Michal Simek @ 2026-05-11 6:35 UTC (permalink / raw)
To: Marek Vasut, u-boot
Cc: Jaehoon Chung, Naman Trivedi, Peng Fan, Senthil Nathan Thangaraj,
Tom Rini, Venkatesh Yadav Abbarapu
On 5/8/26 14:22, Marek Vasut wrote:
> Set the ops structure as static const. The structure is not accessible
> from outside of this driver and is not going to be modified at runtime.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
> ---
> Cc: Jaehoon Chung <jh80.chung@samsung.com>
> Cc: Michal Simek <michal.simek@amd.com>
> Cc: Naman Trivedi <naman.trivedimanojbhai@amd.com>
> Cc: Peng Fan <peng.fan@nxp.com>
> Cc: Senthil Nathan Thangaraj <senthilnathan.thangaraj@amd.com>
> Cc: Tom Rini <trini@konsulko.com>
> Cc: Venkatesh Yadav Abbarapu <venkatesh.abbarapu@amd.com>
> Cc: u-boot@lists.denx.de
> ---
> drivers/power/domain/zynqmp-power-domain.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/power/domain/zynqmp-power-domain.c b/drivers/power/domain/zynqmp-power-domain.c
> index a54de5c1439..0acfc54e787 100644
> --- a/drivers/power/domain/zynqmp-power-domain.c
> +++ b/drivers/power/domain/zynqmp-power-domain.c
> @@ -57,7 +57,7 @@ static int zynqmp_power_domain_off(struct power_domain *power_domain)
> return 0;
> }
>
> -struct power_domain_ops zynqmp_power_domain_ops = {
> +static const struct power_domain_ops zynqmp_power_domain_ops = {
> .request = zynqmp_power_domain_request,
> .rfree = zynqmp_power_domain_free,
> .on = zynqmp_power_domain_on,
Reviewed-by: Michal Simek <michal.simek@amd.com>
Thanks,
Michal
^ permalink raw reply [flat|nested] 31+ messages in thread* Re: [PATCH 14/14] power: domain: zynqmp: Staticize and constify driver ops
2026-05-08 12:22 ` [PATCH 14/14] power: domain: zynqmp: " Marek Vasut
2026-05-11 6:35 ` Michal Simek
@ 2026-05-13 10:04 ` Peng Fan
1 sibling, 0 replies; 31+ messages in thread
From: Peng Fan @ 2026-05-13 10:04 UTC (permalink / raw)
To: Marek Vasut
Cc: u-boot, Jaehoon Chung, Michal Simek, Naman Trivedi, Peng Fan,
Senthil Nathan Thangaraj, Tom Rini, Venkatesh Yadav Abbarapu
On Fri, May 08, 2026 at 02:22:14PM +0200, Marek Vasut wrote:
>Set the ops structure as static const. The structure is not accessible
>from outside of this driver and is not going to be modified at runtime.
>
>Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [PATCH 01/14] power: domain: apple: Staticize and constify driver ops
2026-05-08 12:22 [PATCH 01/14] power: domain: apple: Staticize and constify driver ops Marek Vasut
` (12 preceding siblings ...)
2026-05-08 12:22 ` [PATCH 14/14] power: domain: zynqmp: " Marek Vasut
@ 2026-05-13 10:00 ` Peng Fan
13 siblings, 0 replies; 31+ messages in thread
From: Peng Fan @ 2026-05-13 10:00 UTC (permalink / raw)
To: Marek Vasut; +Cc: u-boot, Jaehoon Chung, Peng Fan, Tom Rini
On Fri, May 08, 2026 at 02:22:01PM +0200, Marek Vasut wrote:
>Set the ops structure as static const. The structure is not accessible
>from outside of this driver and is not going to be modified at runtime.
>
>Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
^ permalink raw reply [flat|nested] 31+ messages in thread