All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops
@ 2026-05-10 17:16 Marek Vasut
  2026-05-10 17:16 ` [PATCH 02/11] video: bridge: ps862x: " Marek Vasut
                   ` (10 more replies)
  0 siblings, 11 replies; 25+ messages in thread
From: Marek Vasut @ 2026-05-10 17:16 UTC (permalink / raw)
  To: u-boot; +Cc: Marek Vasut, Anatolij Gustschin, 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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
 drivers/video/bridge/anx6345.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/bridge/anx6345.c b/drivers/video/bridge/anx6345.c
index 8cee4c958bd..a5d2781aa48 100644
--- a/drivers/video/bridge/anx6345.c
+++ b/drivers/video/bridge/anx6345.c
@@ -403,7 +403,7 @@ static int anx6345_probe(struct udevice *dev)
 	return anx6345_enable(dev);
 }
 
-struct video_bridge_ops anx6345_ops = {
+static const struct video_bridge_ops anx6345_ops = {
 	.attach = anx6345_attach,
 	.set_backlight = anx6345_set_backlight,
 	.read_edid = anx6345_read_edid,
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH 02/11] video: bridge: ps862x: Staticize and constify driver ops
  2026-05-10 17:16 [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops Marek Vasut
@ 2026-05-10 17:16 ` Marek Vasut
  2026-05-11 13:07   ` Simon Glass
  2026-05-10 17:16 ` [PATCH 03/11] video: bridge: ptn3460: " Marek Vasut
                   ` (9 subsequent siblings)
  10 siblings, 1 reply; 25+ messages in thread
From: Marek Vasut @ 2026-05-10 17:16 UTC (permalink / raw)
  To: u-boot; +Cc: Marek Vasut, Anatolij Gustschin, 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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
 drivers/video/bridge/ps862x.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/bridge/ps862x.c b/drivers/video/bridge/ps862x.c
index efd03752281..a08227f8355 100644
--- a/drivers/video/bridge/ps862x.c
+++ b/drivers/video/bridge/ps862x.c
@@ -116,7 +116,7 @@ static int ps8622_probe(struct udevice *dev)
 	return 0;
 }
 
-struct video_bridge_ops ps8622_ops = {
+static const struct video_bridge_ops ps8622_ops = {
 	.attach = ps8622_attach,
 	.set_backlight = ps8622_set_backlight,
 };
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH 03/11] video: bridge: ptn3460: Staticize and constify driver ops
  2026-05-10 17:16 [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops Marek Vasut
  2026-05-10 17:16 ` [PATCH 02/11] video: bridge: ps862x: " Marek Vasut
@ 2026-05-10 17:16 ` Marek Vasut
  2026-05-11 13:07   ` Simon Glass
  2026-05-10 17:16 ` [PATCH 04/11] video: console: normal: " Marek Vasut
                   ` (8 subsequent siblings)
  10 siblings, 1 reply; 25+ messages in thread
From: Marek Vasut @ 2026-05-10 17:16 UTC (permalink / raw)
  To: u-boot; +Cc: Marek Vasut, Anatolij Gustschin, 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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
 drivers/video/bridge/ptn3460.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/bridge/ptn3460.c b/drivers/video/bridge/ptn3460.c
index 5851e1ef15e..ce576c0b282 100644
--- a/drivers/video/bridge/ptn3460.c
+++ b/drivers/video/bridge/ptn3460.c
@@ -15,7 +15,7 @@ static int ptn3460_attach(struct udevice *dev)
 	return video_bridge_set_active(dev, true);
 }
 
-struct video_bridge_ops ptn3460_ops = {
+static const struct video_bridge_ops ptn3460_ops = {
 	.attach = ptn3460_attach,
 };
 
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH 04/11] video: console: normal: Staticize and constify driver ops
  2026-05-10 17:16 [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops Marek Vasut
  2026-05-10 17:16 ` [PATCH 02/11] video: bridge: ps862x: " Marek Vasut
  2026-05-10 17:16 ` [PATCH 03/11] video: bridge: ptn3460: " Marek Vasut
@ 2026-05-10 17:16 ` Marek Vasut
  2026-05-11 13:07   ` Simon Glass
  2026-05-10 17:17 ` [PATCH 05/11] video: console: rotate: " Marek Vasut
                   ` (7 subsequent siblings)
  10 siblings, 1 reply; 25+ messages in thread
From: Marek Vasut @ 2026-05-10 17:16 UTC (permalink / raw)
  To: u-boot; +Cc: Marek Vasut, Anatolij Gustschin, 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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
 drivers/video/console_normal.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/console_normal.c b/drivers/video/console_normal.c
index 07db613ac53..6dc0a6eaf9d 100644
--- a/drivers/video/console_normal.c
+++ b/drivers/video/console_normal.c
@@ -133,7 +133,7 @@ static int console_set_cursor_visible(struct udevice *dev, bool visible,
 	return 0;
 }
 
-struct vidconsole_ops console_ops = {
+static const struct vidconsole_ops console_ops = {
 	.putc_xy	= console_putc_xy,
 	.move_rows	= console_move_rows,
 	.set_row	= console_set_row,
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH 05/11] video: console: rotate: Staticize and constify driver ops
  2026-05-10 17:16 [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops Marek Vasut
                   ` (2 preceding siblings ...)
  2026-05-10 17:16 ` [PATCH 04/11] video: console: normal: " Marek Vasut
@ 2026-05-10 17:17 ` Marek Vasut
  2026-05-11 13:08   ` Simon Glass
  2026-05-10 17:17 ` [PATCH 06/11] video: console: truetype: " Marek Vasut
                   ` (6 subsequent siblings)
  10 siblings, 1 reply; 25+ messages in thread
From: Marek Vasut @ 2026-05-10 17:17 UTC (permalink / raw)
  To: u-boot; +Cc: Marek Vasut, Anatolij Gustschin, 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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
 drivers/video/console_rotate.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/video/console_rotate.c b/drivers/video/console_rotate.c
index 886b25dcfaf..e478e0ef3bc 100644
--- a/drivers/video/console_rotate.c
+++ b/drivers/video/console_rotate.c
@@ -284,7 +284,7 @@ static int console_putc_xy_3(struct udevice *dev, uint x_frac, uint y, int cp)
 	return VID_TO_POS(fontdata->width);
 }
 
-struct vidconsole_ops console_ops_1 = {
+static const struct vidconsole_ops console_ops_1 = {
 	.putc_xy	= console_putc_xy_1,
 	.move_rows	= console_move_rows_1,
 	.set_row	= console_set_row_1,
@@ -293,7 +293,7 @@ struct vidconsole_ops console_ops_1 = {
 	.select_font	= console_simple_select_font,
 };
 
-struct vidconsole_ops console_ops_2 = {
+static const struct vidconsole_ops console_ops_2 = {
 	.putc_xy	= console_putc_xy_2,
 	.move_rows	= console_move_rows_2,
 	.set_row	= console_set_row_2,
@@ -302,7 +302,7 @@ struct vidconsole_ops console_ops_2 = {
 	.select_font	= console_simple_select_font,
 };
 
-struct vidconsole_ops console_ops_3 = {
+static const struct vidconsole_ops console_ops_3 = {
 	.putc_xy	= console_putc_xy_3,
 	.move_rows	= console_move_rows_3,
 	.set_row	= console_set_row_3,
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH 06/11] video: console: truetype: Staticize and constify driver ops
  2026-05-10 17:16 [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops Marek Vasut
                   ` (3 preceding siblings ...)
  2026-05-10 17:17 ` [PATCH 05/11] video: console: rotate: " Marek Vasut
@ 2026-05-10 17:17 ` Marek Vasut
  2026-05-11 13:07   ` Simon Glass
  2026-05-10 17:17 ` [PATCH 07/11] video: dw_mipi_dsi: " Marek Vasut
                   ` (5 subsequent siblings)
  10 siblings, 1 reply; 25+ messages in thread
From: Marek Vasut @ 2026-05-10 17:17 UTC (permalink / raw)
  To: u-boot
  Cc: Marek Vasut, Anatolij Gustschin, Heinrich Schuchardt, Simon Glass,
	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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
Cc: Heinrich Schuchardt <xypron.glpk@gmx.de>
Cc: Simon Glass <sjg@chromium.org>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
 drivers/video/console_truetype.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/console_truetype.c b/drivers/video/console_truetype.c
index eaf169e8386..b9b6f394fbc 100644
--- a/drivers/video/console_truetype.c
+++ b/drivers/video/console_truetype.c
@@ -1069,7 +1069,7 @@ static int console_truetype_probe(struct udevice *dev)
 	return 0;
 }
 
-struct vidconsole_ops console_truetype_ops = {
+static const struct vidconsole_ops console_truetype_ops = {
 	.putc_xy	= console_truetype_putc_xy,
 	.move_rows	= console_truetype_move_rows,
 	.set_row	= console_truetype_set_row,
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH 07/11] video: dw_mipi_dsi: Staticize and constify driver ops
  2026-05-10 17:16 [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops Marek Vasut
                   ` (4 preceding siblings ...)
  2026-05-10 17:17 ` [PATCH 06/11] video: console: truetype: " Marek Vasut
@ 2026-05-10 17:17 ` Marek Vasut
  2026-05-11 13:08   ` Simon Glass
  2026-05-10 17:17 ` [PATCH 08/11] video: imx: ldb: " Marek Vasut
                   ` (4 subsequent siblings)
  10 siblings, 1 reply; 25+ messages in thread
From: Marek Vasut @ 2026-05-10 17:17 UTC (permalink / raw)
  To: u-boot; +Cc: Marek Vasut, Anatolij Gustschin, 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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
 drivers/video/dw_mipi_dsi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/dw_mipi_dsi.c b/drivers/video/dw_mipi_dsi.c
index c74fe678d12..ec5d4c81812 100644
--- a/drivers/video/dw_mipi_dsi.c
+++ b/drivers/video/dw_mipi_dsi.c
@@ -837,7 +837,7 @@ static int dw_mipi_dsi_enable(struct udevice *dev)
 	return 0;
 }
 
-struct dsi_host_ops dw_mipi_dsi_ops = {
+static const struct dsi_host_ops dw_mipi_dsi_ops = {
 	.init = dw_mipi_dsi_init,
 	.enable = dw_mipi_dsi_enable,
 };
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH 08/11] video: imx: ldb: Staticize and constify driver ops
  2026-05-10 17:16 [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops Marek Vasut
                   ` (5 preceding siblings ...)
  2026-05-10 17:17 ` [PATCH 07/11] video: dw_mipi_dsi: " Marek Vasut
@ 2026-05-10 17:17 ` Marek Vasut
  2026-05-11 13:08   ` Simon Glass
  2026-05-10 17:17 ` [PATCH 09/11] video: stm32: " Marek Vasut
                   ` (3 subsequent siblings)
  10 siblings, 1 reply; 25+ messages in thread
From: Marek Vasut @ 2026-05-10 17:17 UTC (permalink / raw)
  To: u-boot; +Cc: Marek Vasut, Anatolij Gustschin, 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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
 drivers/video/imx/ldb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/imx/ldb.c b/drivers/video/imx/ldb.c
index e918341c0a3..32a327647f8 100644
--- a/drivers/video/imx/ldb.c
+++ b/drivers/video/imx/ldb.c
@@ -230,7 +230,7 @@ dis_clk:
 	return ret;
 }
 
-struct video_bridge_ops imx_ldb_ops = {
+static const struct video_bridge_ops imx_ldb_ops = {
 	.attach = imx_ldb_attach,
 	.set_backlight	= imx_ldb_set_backlight,
 };
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH 09/11] video: stm32: Staticize and constify driver ops
  2026-05-10 17:16 [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops Marek Vasut
                   ` (6 preceding siblings ...)
  2026-05-10 17:17 ` [PATCH 08/11] video: imx: ldb: " Marek Vasut
@ 2026-05-10 17:17 ` Marek Vasut
  2026-05-11  6:02   ` Patrice CHOTARD
                     ` (2 more replies)
  2026-05-10 17:17 ` [PATCH 10/11] video: tda19988: " Marek Vasut
                   ` (2 subsequent siblings)
  10 siblings, 3 replies; 25+ messages in thread
From: Marek Vasut @ 2026-05-10 17:17 UTC (permalink / raw)
  To: u-boot
  Cc: Marek Vasut, Anatolij Gustschin, Patrice Chotard,
	Patrick Delaunay, Raphael Gallais-Pou, Tom Rini, uboot-stm32

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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
Cc: Patrice Chotard <patrice.chotard@foss.st.com>
Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
Cc: Raphael Gallais-Pou <rgallaispou@gmail.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
Cc: uboot-stm32@st-md-mailman.stormreply.com
---
 drivers/video/stm32/stm32_dsi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/stm32/stm32_dsi.c b/drivers/video/stm32/stm32_dsi.c
index 5c4d8d2aab5..29c57a4ff89 100644
--- a/drivers/video/stm32/stm32_dsi.c
+++ b/drivers/video/stm32/stm32_dsi.c
@@ -511,7 +511,7 @@ err_reg:
 	return ret;
 }
 
-struct video_bridge_ops stm32_dsi_ops = {
+static const struct video_bridge_ops stm32_dsi_ops = {
 	.attach = stm32_dsi_attach,
 	.set_backlight = stm32_dsi_set_backlight,
 };
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH 10/11] video: tda19988: Staticize and constify driver ops
  2026-05-10 17:16 [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops Marek Vasut
                   ` (7 preceding siblings ...)
  2026-05-10 17:17 ` [PATCH 09/11] video: stm32: " Marek Vasut
@ 2026-05-10 17:17 ` Marek Vasut
  2026-05-11 13:08   ` Simon Glass
  2026-05-10 17:17 ` [PATCH 11/11] video: tegra: " Marek Vasut
  2026-05-11 13:07 ` [PATCH 01/11] video: bridge: anx6345: " Simon Glass
  10 siblings, 1 reply; 25+ messages in thread
From: Marek Vasut @ 2026-05-10 17:17 UTC (permalink / raw)
  To: u-boot; +Cc: Marek Vasut, Anatolij Gustschin, Liviu Dudau, 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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
Cc: Liviu Dudau <liviu.dudau@foss.arm.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
 drivers/video/tda19988.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/tda19988.c b/drivers/video/tda19988.c
index ebc8521c6ed..a4cc3a49232 100644
--- a/drivers/video/tda19988.c
+++ b/drivers/video/tda19988.c
@@ -522,7 +522,7 @@ static int tda19988_enable(struct udevice *dev, int panel_bpp,
 	return 0;
 }
 
-struct dm_display_ops tda19988_ops = {
+static const struct dm_display_ops tda19988_ops = {
 	.read_edid = tda19988_read_edid,
 	.enable = tda19988_enable,
 };
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [PATCH 11/11] video: tegra: Staticize and constify driver ops
  2026-05-10 17:16 [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops Marek Vasut
                   ` (8 preceding siblings ...)
  2026-05-10 17:17 ` [PATCH 10/11] video: tda19988: " Marek Vasut
@ 2026-05-10 17:17 ` Marek Vasut
  2026-05-10 17:41   ` Svyatoslav Ryhel
  2026-05-11 18:07   ` Simon Glass
  2026-05-11 13:07 ` [PATCH 01/11] video: bridge: anx6345: " Simon Glass
  10 siblings, 2 replies; 25+ messages in thread
From: Marek Vasut @ 2026-05-10 17:17 UTC (permalink / raw)
  To: u-boot
  Cc: Marek Vasut, Anatolij Gustschin, Svyatoslav Ryhel, Thierry Reding,
	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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
Cc: Svyatoslav Ryhel <clamor95@gmail.com>
Cc: Thierry Reding <treding@nvidia.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de
---
 drivers/video/tegra/dsi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/tegra/dsi.c b/drivers/video/tegra/dsi.c
index bc308869f4e..f53fabf6fd6 100644
--- a/drivers/video/tegra/dsi.c
+++ b/drivers/video/tegra/dsi.c
@@ -337,7 +337,7 @@ static ssize_t tegra_dsi_host_transfer(struct mipi_dsi_host *host,
 	return count;
 }
 
-struct mipi_dsi_host_ops tegra_dsi_bridge_host_ops = {
+static const struct mipi_dsi_host_ops tegra_dsi_bridge_host_ops = {
 	.transfer	= tegra_dsi_host_transfer,
 };
 
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 25+ messages in thread

* Re: [PATCH 11/11] video: tegra: Staticize and constify driver ops
  2026-05-10 17:17 ` [PATCH 11/11] video: tegra: " Marek Vasut
@ 2026-05-10 17:41   ` Svyatoslav Ryhel
  2026-05-11 18:07   ` Simon Glass
  1 sibling, 0 replies; 25+ messages in thread
From: Svyatoslav Ryhel @ 2026-05-10 17:41 UTC (permalink / raw)
  To: Marek Vasut; +Cc: u-boot, Anatolij Gustschin, Thierry Reding, Tom Rini

нд, 10 трав. 2026 р. о 20:17 Marek Vasut <marek.vasut+renesas@mailbox.org> пише:
>
> 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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
> Cc: Svyatoslav Ryhel <clamor95@gmail.com>
> Cc: Thierry Reding <treding@nvidia.com>
> Cc: Tom Rini <trini@konsulko.com>
> Cc: u-boot@lists.denx.de
> ---
>  drivers/video/tegra/dsi.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>

Acked-by: Svyatoslav Ryhel <clamor95@gmail.com>

Thank you!

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH 09/11] video: stm32: Staticize and constify driver ops
  2026-05-10 17:17 ` [PATCH 09/11] video: stm32: " Marek Vasut
@ 2026-05-11  6:02   ` Patrice CHOTARD
  2026-05-11 13:11   ` Simon Glass
  2026-05-14 15:03   ` Raphaël Gallais-Pou
  2 siblings, 0 replies; 25+ messages in thread
From: Patrice CHOTARD @ 2026-05-11  6:02 UTC (permalink / raw)
  To: Marek Vasut, u-boot
  Cc: Anatolij Gustschin, Patrick Delaunay, Raphael Gallais-Pou,
	Tom Rini, uboot-stm32



On 5/10/26 19:17, 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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
> Cc: Patrice Chotard <patrice.chotard@foss.st.com>
> Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
> Cc: Raphael Gallais-Pou <rgallaispou@gmail.com>
> Cc: Tom Rini <trini@konsulko.com>
> Cc: u-boot@lists.denx.de
> Cc: uboot-stm32@st-md-mailman.stormreply.com
> ---
>  drivers/video/stm32/stm32_dsi.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/video/stm32/stm32_dsi.c b/drivers/video/stm32/stm32_dsi.c
> index 5c4d8d2aab5..29c57a4ff89 100644
> --- a/drivers/video/stm32/stm32_dsi.c
> +++ b/drivers/video/stm32/stm32_dsi.c
> @@ -511,7 +511,7 @@ err_reg:
>  	return ret;
>  }
>  
> -struct video_bridge_ops stm32_dsi_ops = {
> +static const struct video_bridge_ops stm32_dsi_ops = {
>  	.attach = stm32_dsi_attach,
>  	.set_backlight = stm32_dsi_set_backlight,
>  };

Hi Marek

Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>

Thanks
Patrice

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops
  2026-05-10 17:16 [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops Marek Vasut
                   ` (9 preceding siblings ...)
  2026-05-10 17:17 ` [PATCH 11/11] video: tegra: " Marek Vasut
@ 2026-05-11 13:07 ` Simon Glass
  10 siblings, 0 replies; 25+ messages in thread
From: Simon Glass @ 2026-05-11 13:07 UTC (permalink / raw)
  To: marek.vasut+renesas; +Cc: u-boot, Anatolij Gustschin, Tom Rini

On 2026-05-10T17:16:56, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:
> video: bridge: anx6345: 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/video/bridge/anx6345.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH 06/11] video: console: truetype: Staticize and constify driver ops
  2026-05-10 17:17 ` [PATCH 06/11] video: console: truetype: " Marek Vasut
@ 2026-05-11 13:07   ` Simon Glass
  0 siblings, 0 replies; 25+ messages in thread
From: Simon Glass @ 2026-05-11 13:07 UTC (permalink / raw)
  To: marek.vasut+renesas
  Cc: u-boot, Anatolij Gustschin, Heinrich Schuchardt, Simon Glass,
	Tom Rini

On 2026-05-10T17:16:56, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:
> video: console: truetype: 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/video/console_truetype.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH 02/11] video: bridge: ps862x: Staticize and constify driver ops
  2026-05-10 17:16 ` [PATCH 02/11] video: bridge: ps862x: " Marek Vasut
@ 2026-05-11 13:07   ` Simon Glass
  0 siblings, 0 replies; 25+ messages in thread
From: Simon Glass @ 2026-05-11 13:07 UTC (permalink / raw)
  To: marek.vasut+renesas; +Cc: u-boot, Anatolij Gustschin, Tom Rini

On 2026-05-10T17:16:56, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:
> video: bridge: ps862x: 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/video/bridge/ps862x.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH 03/11] video: bridge: ptn3460: Staticize and constify driver ops
  2026-05-10 17:16 ` [PATCH 03/11] video: bridge: ptn3460: " Marek Vasut
@ 2026-05-11 13:07   ` Simon Glass
  0 siblings, 0 replies; 25+ messages in thread
From: Simon Glass @ 2026-05-11 13:07 UTC (permalink / raw)
  To: marek.vasut+renesas; +Cc: u-boot, Anatolij Gustschin, Tom Rini

On 2026-05-10T17:16:56, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:
> video: bridge: ptn3460: 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/video/bridge/ptn3460.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH 04/11] video: console: normal: Staticize and constify driver ops
  2026-05-10 17:16 ` [PATCH 04/11] video: console: normal: " Marek Vasut
@ 2026-05-11 13:07   ` Simon Glass
  0 siblings, 0 replies; 25+ messages in thread
From: Simon Glass @ 2026-05-11 13:07 UTC (permalink / raw)
  To: marek.vasut+renesas; +Cc: u-boot, Anatolij Gustschin, Tom Rini

On 2026-05-10T17:16:56, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:
> video: console: normal: 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/video/console_normal.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH 05/11] video: console: rotate: Staticize and constify driver ops
  2026-05-10 17:17 ` [PATCH 05/11] video: console: rotate: " Marek Vasut
@ 2026-05-11 13:08   ` Simon Glass
  0 siblings, 0 replies; 25+ messages in thread
From: Simon Glass @ 2026-05-11 13:08 UTC (permalink / raw)
  To: marek.vasut+renesas; +Cc: u-boot, Anatolij Gustschin, Tom Rini

On 2026-05-10T17:16:56, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:
> video: console: rotate: 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/video/console_rotate.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH 08/11] video: imx: ldb: Staticize and constify driver ops
  2026-05-10 17:17 ` [PATCH 08/11] video: imx: ldb: " Marek Vasut
@ 2026-05-11 13:08   ` Simon Glass
  0 siblings, 0 replies; 25+ messages in thread
From: Simon Glass @ 2026-05-11 13:08 UTC (permalink / raw)
  To: marek.vasut+renesas; +Cc: u-boot, Anatolij Gustschin, Tom Rini

On 2026-05-10T17:16:56, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:
> video: imx: ldb: 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/video/imx/ldb.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH 07/11] video: dw_mipi_dsi: Staticize and constify driver ops
  2026-05-10 17:17 ` [PATCH 07/11] video: dw_mipi_dsi: " Marek Vasut
@ 2026-05-11 13:08   ` Simon Glass
  0 siblings, 0 replies; 25+ messages in thread
From: Simon Glass @ 2026-05-11 13:08 UTC (permalink / raw)
  To: marek.vasut+renesas; +Cc: u-boot, Anatolij Gustschin, Tom Rini

On 2026-05-10T17:16:56, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:
> video: dw_mipi_dsi: 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/video/dw_mipi_dsi.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH 10/11] video: tda19988: Staticize and constify driver ops
  2026-05-10 17:17 ` [PATCH 10/11] video: tda19988: " Marek Vasut
@ 2026-05-11 13:08   ` Simon Glass
  0 siblings, 0 replies; 25+ messages in thread
From: Simon Glass @ 2026-05-11 13:08 UTC (permalink / raw)
  To: marek.vasut+renesas; +Cc: u-boot, Anatolij Gustschin, Liviu Dudau, Tom Rini

On 2026-05-10T17:16:56, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:
> video: tda19988: 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/video/tda19988.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH 09/11] video: stm32: Staticize and constify driver ops
  2026-05-10 17:17 ` [PATCH 09/11] video: stm32: " Marek Vasut
  2026-05-11  6:02   ` Patrice CHOTARD
@ 2026-05-11 13:11   ` Simon Glass
  2026-05-14 15:03   ` Raphaël Gallais-Pou
  2 siblings, 0 replies; 25+ messages in thread
From: Simon Glass @ 2026-05-11 13:11 UTC (permalink / raw)
  To: marek.vasut+renesas
  Cc: u-boot, Anatolij Gustschin, Patrice Chotard, Patrick Delaunay,
	Raphael Gallais-Pou, Tom Rini, uboot-stm32

On 2026-05-10T17:16:56, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:
> video: stm32: 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>
> Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
>
> drivers/video/stm32/stm32_dsi.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH 11/11] video: tegra: Staticize and constify driver ops
  2026-05-10 17:17 ` [PATCH 11/11] video: tegra: " Marek Vasut
  2026-05-10 17:41   ` Svyatoslav Ryhel
@ 2026-05-11 18:07   ` Simon Glass
  1 sibling, 0 replies; 25+ messages in thread
From: Simon Glass @ 2026-05-11 18:07 UTC (permalink / raw)
  To: marek.vasut+renesas
  Cc: u-boot, Anatolij Gustschin, Svyatoslav Ryhel, Thierry Reding,
	Tom Rini

On 2026-05-10T17:16:56, Marek Vasut <marek.vasut+renesas@mailbox.org> wrote:
> video: tegra: 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>
> Acked-by: Svyatoslav Ryhel <clamor95@gmail.com>
>
> drivers/video/tegra/dsi.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* Re: [PATCH 09/11] video: stm32: Staticize and constify driver ops
  2026-05-10 17:17 ` [PATCH 09/11] video: stm32: " Marek Vasut
  2026-05-11  6:02   ` Patrice CHOTARD
  2026-05-11 13:11   ` Simon Glass
@ 2026-05-14 15:03   ` Raphaël Gallais-Pou
  2 siblings, 0 replies; 25+ messages in thread
From: Raphaël Gallais-Pou @ 2026-05-14 15:03 UTC (permalink / raw)
  To: Marek Vasut, u-boot
  Cc: Anatolij Gustschin, Patrice Chotard, Patrick Delaunay, Tom Rini,
	uboot-stm32



On 5/10/26 19:17, 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: Anatolij Gustschin <ag.dev.uboot@gmail.com>
> Cc: Patrice Chotard <patrice.chotard@foss.st.com>
> Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
> Cc: Raphael Gallais-Pou <rgallaispou@gmail.com>
> Cc: Tom Rini <trini@konsulko.com>
> Cc: u-boot@lists.denx.de
> Cc: uboot-stm32@st-md-mailman.stormreply.com
> ---Hi Marek,

Reviewed-by: Raphaël Gallais-Pou <rgallaispou@gmail.com>

Thanks,
Best regards,
Raphaël

^ permalink raw reply	[flat|nested] 25+ messages in thread

end of thread, other threads:[~2026-05-14 15:03 UTC | newest]

Thread overview: 25+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-10 17:16 [PATCH 01/11] video: bridge: anx6345: Staticize and constify driver ops Marek Vasut
2026-05-10 17:16 ` [PATCH 02/11] video: bridge: ps862x: " Marek Vasut
2026-05-11 13:07   ` Simon Glass
2026-05-10 17:16 ` [PATCH 03/11] video: bridge: ptn3460: " Marek Vasut
2026-05-11 13:07   ` Simon Glass
2026-05-10 17:16 ` [PATCH 04/11] video: console: normal: " Marek Vasut
2026-05-11 13:07   ` Simon Glass
2026-05-10 17:17 ` [PATCH 05/11] video: console: rotate: " Marek Vasut
2026-05-11 13:08   ` Simon Glass
2026-05-10 17:17 ` [PATCH 06/11] video: console: truetype: " Marek Vasut
2026-05-11 13:07   ` Simon Glass
2026-05-10 17:17 ` [PATCH 07/11] video: dw_mipi_dsi: " Marek Vasut
2026-05-11 13:08   ` Simon Glass
2026-05-10 17:17 ` [PATCH 08/11] video: imx: ldb: " Marek Vasut
2026-05-11 13:08   ` Simon Glass
2026-05-10 17:17 ` [PATCH 09/11] video: stm32: " Marek Vasut
2026-05-11  6:02   ` Patrice CHOTARD
2026-05-11 13:11   ` Simon Glass
2026-05-14 15:03   ` Raphaël Gallais-Pou
2026-05-10 17:17 ` [PATCH 10/11] video: tda19988: " Marek Vasut
2026-05-11 13:08   ` Simon Glass
2026-05-10 17:17 ` [PATCH 11/11] video: tegra: " Marek Vasut
2026-05-10 17:41   ` Svyatoslav Ryhel
2026-05-11 18:07   ` Simon Glass
2026-05-11 13:07 ` [PATCH 01/11] video: bridge: anx6345: " Simon Glass

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.