* [PATCH 00/18] backlight: Constify lcd_ops
@ 2024-04-14 16:35 Krzysztof Kozlowski
2024-04-14 16:35 ` [PATCH 01/18] " Krzysztof Kozlowski
` (19 more replies)
0 siblings, 20 replies; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:35 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
Hi,
Dependencies
============
All further patches depend on the first patch. Therefore everything
could go via backlight tree (please ack) or via cross-tree pulls. Or
whatever maintainer choose, just coordinate this with backlight.
Best regards,
Krzysztof
---
Krzysztof Kozlowski (18):
backlight: Constify lcd_ops
backlight: ams369fg06: Constify lcd_ops
backlight: corgi_lcd: Constify lcd_ops
backlight: hx8357: Constify lcd_ops
backlight: ili922x: Constify lcd_ops
backlight: ili9320: Constify lcd_ops
backlight: jornada720_lcd: Constify lcd_ops
backlight: l4f00242t03: Constify lcd_ops
backlight: lms283gf05: Constify lcd_ops
backlight: lms501kf03: Constify lcd_ops
backlight: ltv350qv: Constify lcd_ops
backlight: otm3225a: Constify lcd_ops
backlight: platform_lcd: Constify lcd_ops
backlight: tdo24m: Constify lcd_ops
HID: picoLCD: Constify lcd_ops
fbdev: clps711x: Constify lcd_ops
fbdev: imx: Constify lcd_ops
fbdev: omap: lcd_ams_delta: Constify lcd_ops
drivers/hid/hid-picolcd_lcd.c | 2 +-
drivers/video/backlight/ams369fg06.c | 2 +-
drivers/video/backlight/corgi_lcd.c | 2 +-
drivers/video/backlight/hx8357.c | 2 +-
drivers/video/backlight/ili922x.c | 2 +-
drivers/video/backlight/ili9320.c | 2 +-
drivers/video/backlight/jornada720_lcd.c | 2 +-
drivers/video/backlight/l4f00242t03.c | 2 +-
drivers/video/backlight/lcd.c | 4 ++--
drivers/video/backlight/lms283gf05.c | 2 +-
drivers/video/backlight/lms501kf03.c | 2 +-
drivers/video/backlight/ltv350qv.c | 2 +-
drivers/video/backlight/otm3225a.c | 2 +-
drivers/video/backlight/platform_lcd.c | 2 +-
drivers/video/backlight/tdo24m.c | 2 +-
drivers/video/fbdev/clps711x-fb.c | 2 +-
drivers/video/fbdev/imxfb.c | 2 +-
drivers/video/fbdev/omap/lcd_ams_delta.c | 2 +-
include/linux/lcd.h | 6 +++---
19 files changed, 22 insertions(+), 22 deletions(-)
---
base-commit: 9ed46da14b9b9b2ad4edb3b0c545b6dbe5c00d39
change-id: 20240414-video-backlight-lcd-ops-276d8439ffb8
Best regards,
--
Krzysztof Kozlowski <krzk@kernel.org>
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH 01/18] backlight: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
@ 2024-04-14 16:35 ` Krzysztof Kozlowski
2024-04-15 12:46 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 02/18] backlight: ams369fg06: " Krzysztof Kozlowski
` (18 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:35 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' passed in lcd_device_register() is not modified by core
backlight code, so it can be made const for code safety. This allows
drivers to also define the structure as const.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/lcd.c | 4 ++--
include/linux/lcd.h | 6 +++---
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/video/backlight/lcd.c b/drivers/video/backlight/lcd.c
index ba4771cbd781..ceec90ca758b 100644
--- a/drivers/video/backlight/lcd.c
+++ b/drivers/video/backlight/lcd.c
@@ -191,7 +191,7 @@ static const struct class lcd_class = {
* or a pointer to the newly allocated device.
*/
struct lcd_device *lcd_device_register(const char *name, struct device *parent,
- void *devdata, struct lcd_ops *ops)
+ void *devdata, const struct lcd_ops *ops)
{
struct lcd_device *new_ld;
int rc;
@@ -279,7 +279,7 @@ static int devm_lcd_device_match(struct device *dev, void *res, void *data)
*/
struct lcd_device *devm_lcd_device_register(struct device *dev,
const char *name, struct device *parent,
- void *devdata, struct lcd_ops *ops)
+ void *devdata, const struct lcd_ops *ops)
{
struct lcd_device **ptr, *lcd;
diff --git a/include/linux/lcd.h b/include/linux/lcd.h
index 238fb1dfed98..68703a51dc53 100644
--- a/include/linux/lcd.h
+++ b/include/linux/lcd.h
@@ -61,7 +61,7 @@ struct lcd_device {
points to something in the body of that driver, it is also invalid. */
struct mutex ops_lock;
/* If this is NULL, the backing module is unloaded */
- struct lcd_ops *ops;
+ const struct lcd_ops *ops;
/* Serialise access to set_power method */
struct mutex update_lock;
/* The framebuffer notifier block */
@@ -102,10 +102,10 @@ static inline void lcd_set_power(struct lcd_device *ld, int power)
}
extern struct lcd_device *lcd_device_register(const char *name,
- struct device *parent, void *devdata, struct lcd_ops *ops);
+ struct device *parent, void *devdata, const struct lcd_ops *ops);
extern struct lcd_device *devm_lcd_device_register(struct device *dev,
const char *name, struct device *parent,
- void *devdata, struct lcd_ops *ops);
+ void *devdata, const struct lcd_ops *ops);
extern void lcd_device_unregister(struct lcd_device *ld);
extern void devm_lcd_device_unregister(struct device *dev,
struct lcd_device *ld);
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 02/18] backlight: ams369fg06: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
2024-04-14 16:35 ` [PATCH 01/18] " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 12:47 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 03/18] backlight: corgi_lcd: " Krzysztof Kozlowski
` (17 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/ams369fg06.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/ams369fg06.c b/drivers/video/backlight/ams369fg06.c
index 522dd81110b8..57ec205d2bd2 100644
--- a/drivers/video/backlight/ams369fg06.c
+++ b/drivers/video/backlight/ams369fg06.c
@@ -427,7 +427,7 @@ static int ams369fg06_set_brightness(struct backlight_device *bd)
return ret;
}
-static struct lcd_ops ams369fg06_lcd_ops = {
+static const struct lcd_ops ams369fg06_lcd_ops = {
.get_power = ams369fg06_get_power,
.set_power = ams369fg06_set_power,
};
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 03/18] backlight: corgi_lcd: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
2024-04-14 16:35 ` [PATCH 01/18] " Krzysztof Kozlowski
2024-04-14 16:36 ` [PATCH 02/18] backlight: ams369fg06: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 12:48 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 04/18] backlight: hx8357: " Krzysztof Kozlowski
` (16 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/corgi_lcd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/corgi_lcd.c b/drivers/video/backlight/corgi_lcd.c
index dd765098ad98..aad1680c9075 100644
--- a/drivers/video/backlight/corgi_lcd.c
+++ b/drivers/video/backlight/corgi_lcd.c
@@ -380,7 +380,7 @@ static int corgi_lcd_get_power(struct lcd_device *ld)
return lcd->power;
}
-static struct lcd_ops corgi_lcd_ops = {
+static const struct lcd_ops corgi_lcd_ops = {
.get_power = corgi_lcd_get_power,
.set_power = corgi_lcd_set_power,
.set_mode = corgi_lcd_set_mode,
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 04/18] backlight: hx8357: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (2 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 03/18] backlight: corgi_lcd: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 12:49 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 05/18] backlight: ili922x: " Krzysztof Kozlowski
` (15 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/hx8357.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/hx8357.c b/drivers/video/backlight/hx8357.c
index 339d9128fbde..cdd7b7686723 100644
--- a/drivers/video/backlight/hx8357.c
+++ b/drivers/video/backlight/hx8357.c
@@ -559,7 +559,7 @@ static int hx8357_get_power(struct lcd_device *lcdev)
return lcd->state;
}
-static struct lcd_ops hx8357_ops = {
+static const struct lcd_ops hx8357_ops = {
.set_power = hx8357_set_power,
.get_power = hx8357_get_power,
};
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 05/18] backlight: ili922x: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (3 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 04/18] backlight: hx8357: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 12:49 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 06/18] backlight: ili9320: " Krzysztof Kozlowski
` (14 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/ili922x.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/ili922x.c b/drivers/video/backlight/ili922x.c
index c8e0e655dc86..7683e209ad6b 100644
--- a/drivers/video/backlight/ili922x.c
+++ b/drivers/video/backlight/ili922x.c
@@ -472,7 +472,7 @@ static int ili922x_get_power(struct lcd_device *ld)
return ili->power;
}
-static struct lcd_ops ili922x_ops = {
+static const struct lcd_ops ili922x_ops = {
.get_power = ili922x_get_power,
.set_power = ili922x_set_power,
};
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 06/18] backlight: ili9320: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (4 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 05/18] backlight: ili922x: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 12:49 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 07/18] backlight: jornada720_lcd: " Krzysztof Kozlowski
` (13 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/ili9320.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/ili9320.c b/drivers/video/backlight/ili9320.c
index 2acd2708f8ca..3e318d1891b6 100644
--- a/drivers/video/backlight/ili9320.c
+++ b/drivers/video/backlight/ili9320.c
@@ -161,7 +161,7 @@ static int ili9320_get_power(struct lcd_device *ld)
return lcd->power;
}
-static struct lcd_ops ili9320_ops = {
+static const struct lcd_ops ili9320_ops = {
.get_power = ili9320_get_power,
.set_power = ili9320_set_power,
};
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 07/18] backlight: jornada720_lcd: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (5 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 06/18] backlight: ili9320: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 12:52 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 08/18] backlight: l4f00242t03: " Krzysztof Kozlowski
` (12 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/jornada720_lcd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/jornada720_lcd.c b/drivers/video/backlight/jornada720_lcd.c
index 6796a7c2db25..5c64fa61e810 100644
--- a/drivers/video/backlight/jornada720_lcd.c
+++ b/drivers/video/backlight/jornada720_lcd.c
@@ -81,7 +81,7 @@ static int jornada_lcd_set_power(struct lcd_device *ld, int power)
return 0;
}
-static struct lcd_ops jornada_lcd_props = {
+static const struct lcd_ops jornada_lcd_props = {
.get_contrast = jornada_lcd_get_contrast,
.set_contrast = jornada_lcd_set_contrast,
.get_power = jornada_lcd_get_power,
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 08/18] backlight: l4f00242t03: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (6 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 07/18] backlight: jornada720_lcd: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 12:53 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 09/18] backlight: lms283gf05: " Krzysztof Kozlowski
` (11 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/l4f00242t03.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/l4f00242t03.c b/drivers/video/backlight/l4f00242t03.c
index bd5137ee203b..dd0874f8c7ff 100644
--- a/drivers/video/backlight/l4f00242t03.c
+++ b/drivers/video/backlight/l4f00242t03.c
@@ -158,7 +158,7 @@ static int l4f00242t03_lcd_power_set(struct lcd_device *ld, int power)
return 0;
}
-static struct lcd_ops l4f_ops = {
+static const struct lcd_ops l4f_ops = {
.set_power = l4f00242t03_lcd_power_set,
.get_power = l4f00242t03_lcd_power_get,
};
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 09/18] backlight: lms283gf05: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (7 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 08/18] backlight: l4f00242t03: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 12:54 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 10/18] backlight: lms501kf03: " Krzysztof Kozlowski
` (10 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/lms283gf05.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/lms283gf05.c b/drivers/video/backlight/lms283gf05.c
index 36856962ed83..a65490e83d3d 100644
--- a/drivers/video/backlight/lms283gf05.c
+++ b/drivers/video/backlight/lms283gf05.c
@@ -139,7 +139,7 @@ static int lms283gf05_power_set(struct lcd_device *ld, int power)
return 0;
}
-static struct lcd_ops lms_ops = {
+static const struct lcd_ops lms_ops = {
.set_power = lms283gf05_power_set,
.get_power = NULL,
};
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 10/18] backlight: lms501kf03: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (8 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 09/18] backlight: lms283gf05: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 12:54 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 11/18] backlight: ltv350qv: " Krzysztof Kozlowski
` (9 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/lms501kf03.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/lms501kf03.c b/drivers/video/backlight/lms501kf03.c
index 5c46df8022bf..8aebe0af3391 100644
--- a/drivers/video/backlight/lms501kf03.c
+++ b/drivers/video/backlight/lms501kf03.c
@@ -304,7 +304,7 @@ static int lms501kf03_set_power(struct lcd_device *ld, int power)
return lms501kf03_power(lcd, power);
}
-static struct lcd_ops lms501kf03_lcd_ops = {
+static const struct lcd_ops lms501kf03_lcd_ops = {
.get_power = lms501kf03_get_power,
.set_power = lms501kf03_set_power,
};
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 11/18] backlight: ltv350qv: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (9 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 10/18] backlight: lms501kf03: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 12:54 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 12/18] backlight: otm3225a: " Krzysztof Kozlowski
` (8 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/ltv350qv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/ltv350qv.c b/drivers/video/backlight/ltv350qv.c
index d54f501e4285..cdc4c087f230 100644
--- a/drivers/video/backlight/ltv350qv.c
+++ b/drivers/video/backlight/ltv350qv.c
@@ -217,7 +217,7 @@ static int ltv350qv_get_power(struct lcd_device *ld)
return lcd->power;
}
-static struct lcd_ops ltv_ops = {
+static const struct lcd_ops ltv_ops = {
.get_power = ltv350qv_get_power,
.set_power = ltv350qv_set_power,
};
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 12/18] backlight: otm3225a: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (10 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 11/18] backlight: ltv350qv: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 12:54 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 13/18] backlight: platform_lcd: " Krzysztof Kozlowski
` (7 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/otm3225a.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/otm3225a.c b/drivers/video/backlight/otm3225a.c
index 4568020abf3c..efe52fa08b07 100644
--- a/drivers/video/backlight/otm3225a.c
+++ b/drivers/video/backlight/otm3225a.c
@@ -205,7 +205,7 @@ static int otm3225a_get_power(struct lcd_device *ld)
return dd->power;
}
-static struct lcd_ops otm3225a_ops = {
+static const struct lcd_ops otm3225a_ops = {
.set_power = otm3225a_set_power,
.get_power = otm3225a_get_power,
};
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 13/18] backlight: platform_lcd: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (11 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 12/18] backlight: otm3225a: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 12:55 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 14/18] backlight: tdo24m: " Krzysztof Kozlowski
` (6 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/platform_lcd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/platform_lcd.c b/drivers/video/backlight/platform_lcd.c
index dc37494baf42..76872f5c34c5 100644
--- a/drivers/video/backlight/platform_lcd.c
+++ b/drivers/video/backlight/platform_lcd.c
@@ -62,7 +62,7 @@ static int platform_lcd_match(struct lcd_device *lcd, struct fb_info *info)
return plcd->us->parent == info->device;
}
-static struct lcd_ops platform_lcd_ops = {
+static const struct lcd_ops platform_lcd_ops = {
.get_power = platform_lcd_get_power,
.set_power = platform_lcd_set_power,
.check_fb = platform_lcd_match,
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 14/18] backlight: tdo24m: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (12 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 13/18] backlight: platform_lcd: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 12:55 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 15/18] HID: picoLCD: " Krzysztof Kozlowski
` (5 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/video/backlight/tdo24m.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/backlight/tdo24m.c b/drivers/video/backlight/tdo24m.c
index fc6fbaf85594..c413b3c68e95 100644
--- a/drivers/video/backlight/tdo24m.c
+++ b/drivers/video/backlight/tdo24m.c
@@ -322,7 +322,7 @@ static int tdo24m_set_mode(struct lcd_device *ld, struct fb_videomode *m)
return lcd->adj_mode(lcd, mode);
}
-static struct lcd_ops tdo24m_ops = {
+static const struct lcd_ops tdo24m_ops = {
.get_power = tdo24m_get_power,
.set_power = tdo24m_set_power,
.set_mode = tdo24m_set_mode,
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 15/18] HID: picoLCD: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (13 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 14/18] backlight: tdo24m: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-16 8:07 ` Bruno Prémont
2024-04-14 16:36 ` [PATCH 16/18] fbdev: clps711x: " Krzysztof Kozlowski
` (4 subsequent siblings)
19 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
Depends on the first patch in the series.
---
drivers/hid/hid-picolcd_lcd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/hid/hid-picolcd_lcd.c b/drivers/hid/hid-picolcd_lcd.c
index 0c4b76de8ae5..061a33ba7b1d 100644
--- a/drivers/hid/hid-picolcd_lcd.c
+++ b/drivers/hid/hid-picolcd_lcd.c
@@ -46,7 +46,7 @@ static int picolcd_check_lcd_fb(struct lcd_device *ldev, struct fb_info *fb)
return fb && fb == picolcd_fbinfo((struct picolcd_data *)lcd_get_data(ldev));
}
-static struct lcd_ops picolcd_lcdops = {
+static const struct lcd_ops picolcd_lcdops = {
.get_contrast = picolcd_get_contrast,
.set_contrast = picolcd_set_contrast,
.check_fb = picolcd_check_lcd_fb,
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 16/18] fbdev: clps711x: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (14 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 15/18] HID: picoLCD: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-14 16:36 ` [PATCH 17/18] fbdev: imx: " Krzysztof Kozlowski
` (3 subsequent siblings)
19 siblings, 0 replies; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
Depends on the first patch in the series.
---
drivers/video/fbdev/clps711x-fb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/fbdev/clps711x-fb.c b/drivers/video/fbdev/clps711x-fb.c
index dcfd1fbbc7e1..6171a98a48fd 100644
--- a/drivers/video/fbdev/clps711x-fb.c
+++ b/drivers/video/fbdev/clps711x-fb.c
@@ -197,7 +197,7 @@ static int clps711x_lcd_set_power(struct lcd_device *lcddev, int blank)
return 0;
}
-static struct lcd_ops clps711x_lcd_ops = {
+static const struct lcd_ops clps711x_lcd_ops = {
.check_fb = clps711x_lcd_check_fb,
.get_power = clps711x_lcd_get_power,
.set_power = clps711x_lcd_set_power,
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 17/18] fbdev: imx: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (15 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 16/18] fbdev: clps711x: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-14 16:36 ` [PATCH 18/18] fbdev: omap: lcd_ams_delta: " Krzysztof Kozlowski
` (2 subsequent siblings)
19 siblings, 0 replies; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
Depends on the first patch in the series.
---
drivers/video/fbdev/imxfb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/fbdev/imxfb.c b/drivers/video/fbdev/imxfb.c
index a4dbc72f93c3..4ebfe9b9df60 100644
--- a/drivers/video/fbdev/imxfb.c
+++ b/drivers/video/fbdev/imxfb.c
@@ -857,7 +857,7 @@ static int imxfb_lcd_set_power(struct lcd_device *lcddev, int power)
return 0;
}
-static struct lcd_ops imxfb_lcd_ops = {
+static const struct lcd_ops imxfb_lcd_ops = {
.check_fb = imxfb_lcd_check_fb,
.get_contrast = imxfb_lcd_get_contrast,
.set_contrast = imxfb_lcd_set_contrast,
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH 18/18] fbdev: omap: lcd_ams_delta: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (16 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 17/18] fbdev: imx: " Krzysztof Kozlowski
@ 2024-04-14 16:36 ` Krzysztof Kozlowski
2024-04-15 6:56 ` [PATCH 00/18] backlight: " Thomas Zimmermann
2024-04-15 12:58 ` Daniel Thompson
19 siblings, 0 replies; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-14 16:36 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
Depends on the first patch in the series.
---
drivers/video/fbdev/omap/lcd_ams_delta.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/fbdev/omap/lcd_ams_delta.c b/drivers/video/fbdev/omap/lcd_ams_delta.c
index 6f860c814d2c..97e2b71b64d7 100644
--- a/drivers/video/fbdev/omap/lcd_ams_delta.c
+++ b/drivers/video/fbdev/omap/lcd_ams_delta.c
@@ -76,7 +76,7 @@ static int ams_delta_lcd_get_contrast(struct lcd_device *dev)
return ams_delta_lcd & AMS_DELTA_MAX_CONTRAST;
}
-static struct lcd_ops ams_delta_lcd_ops = {
+static const struct lcd_ops ams_delta_lcd_ops = {
.get_power = ams_delta_lcd_get_power,
.set_power = ams_delta_lcd_set_power,
.get_contrast = ams_delta_lcd_get_contrast,
--
2.34.1
^ permalink raw reply related [flat|nested] 38+ messages in thread
* Re: [PATCH 00/18] backlight: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (17 preceding siblings ...)
2024-04-14 16:36 ` [PATCH 18/18] fbdev: omap: lcd_ams_delta: " Krzysztof Kozlowski
@ 2024-04-15 6:56 ` Thomas Zimmermann
2024-04-15 7:48 ` Krzysztof Kozlowski
2024-04-15 12:58 ` Daniel Thompson
19 siblings, 1 reply; 38+ messages in thread
From: Thomas Zimmermann @ 2024-04-15 6:56 UTC (permalink / raw)
To: Krzysztof Kozlowski, Lee Jones, Daniel Thompson, Jingoo Han,
Helge Deller, Bruno Prémont, Jiri Kosina,
Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap
Hi
for patches 16, 17 and 18:
Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
Best regards
Thomas
Am 14.04.24 um 18:35 schrieb Krzysztof Kozlowski:
> Hi,
>
> Dependencies
> ============
> All further patches depend on the first patch. Therefore everything
> could go via backlight tree (please ack) or via cross-tree pulls. Or
> whatever maintainer choose, just coordinate this with backlight.
>
> Best regards,
> Krzysztof
>
> ---
> Krzysztof Kozlowski (18):
> backlight: Constify lcd_ops
> backlight: ams369fg06: Constify lcd_ops
> backlight: corgi_lcd: Constify lcd_ops
> backlight: hx8357: Constify lcd_ops
> backlight: ili922x: Constify lcd_ops
> backlight: ili9320: Constify lcd_ops
> backlight: jornada720_lcd: Constify lcd_ops
> backlight: l4f00242t03: Constify lcd_ops
> backlight: lms283gf05: Constify lcd_ops
> backlight: lms501kf03: Constify lcd_ops
> backlight: ltv350qv: Constify lcd_ops
> backlight: otm3225a: Constify lcd_ops
> backlight: platform_lcd: Constify lcd_ops
> backlight: tdo24m: Constify lcd_ops
> HID: picoLCD: Constify lcd_ops
> fbdev: clps711x: Constify lcd_ops
> fbdev: imx: Constify lcd_ops
> fbdev: omap: lcd_ams_delta: Constify lcd_ops
>
> drivers/hid/hid-picolcd_lcd.c | 2 +-
> drivers/video/backlight/ams369fg06.c | 2 +-
> drivers/video/backlight/corgi_lcd.c | 2 +-
> drivers/video/backlight/hx8357.c | 2 +-
> drivers/video/backlight/ili922x.c | 2 +-
> drivers/video/backlight/ili9320.c | 2 +-
> drivers/video/backlight/jornada720_lcd.c | 2 +-
> drivers/video/backlight/l4f00242t03.c | 2 +-
> drivers/video/backlight/lcd.c | 4 ++--
> drivers/video/backlight/lms283gf05.c | 2 +-
> drivers/video/backlight/lms501kf03.c | 2 +-
> drivers/video/backlight/ltv350qv.c | 2 +-
> drivers/video/backlight/otm3225a.c | 2 +-
> drivers/video/backlight/platform_lcd.c | 2 +-
> drivers/video/backlight/tdo24m.c | 2 +-
> drivers/video/fbdev/clps711x-fb.c | 2 +-
> drivers/video/fbdev/imxfb.c | 2 +-
> drivers/video/fbdev/omap/lcd_ams_delta.c | 2 +-
> include/linux/lcd.h | 6 +++---
> 19 files changed, 22 insertions(+), 22 deletions(-)
> ---
> base-commit: 9ed46da14b9b9b2ad4edb3b0c545b6dbe5c00d39
> change-id: 20240414-video-backlight-lcd-ops-276d8439ffb8
>
> Best regards,
--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 00/18] backlight: Constify lcd_ops
2024-04-15 6:56 ` [PATCH 00/18] backlight: " Thomas Zimmermann
@ 2024-04-15 7:48 ` Krzysztof Kozlowski
2024-04-16 12:17 ` Lee Jones
0 siblings, 1 reply; 38+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-15 7:48 UTC (permalink / raw)
To: Thomas Zimmermann, Lee Jones, Daniel Thompson, Jingoo Han,
Helge Deller, Bruno Prémont, Jiri Kosina,
Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap
On 15/04/2024 08:56, Thomas Zimmermann wrote:
> Hi
>
> for patches 16, 17 and 18:
>
> Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
This does not work like this. Toolset will apply your review tag for
EVERYTHING. You must provide tag under each individual patch.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 01/18] backlight: Constify lcd_ops
2024-04-14 16:35 ` [PATCH 01/18] " Krzysztof Kozlowski
@ 2024-04-15 12:46 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:46 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:35:59PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' passed in lcd_device_register() is not modified by core
> backlight code, so it can be made const for code safety. This allows
> drivers to also define the structure as const.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 02/18] backlight: ams369fg06: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 02/18] backlight: ams369fg06: " Krzysztof Kozlowski
@ 2024-04-15 12:47 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:47 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:36:00PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 03/18] backlight: corgi_lcd: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 03/18] backlight: corgi_lcd: " Krzysztof Kozlowski
@ 2024-04-15 12:48 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:48 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:36:01PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 04/18] backlight: hx8357: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 04/18] backlight: hx8357: " Krzysztof Kozlowski
@ 2024-04-15 12:49 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:49 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:36:02PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 05/18] backlight: ili922x: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 05/18] backlight: ili922x: " Krzysztof Kozlowski
@ 2024-04-15 12:49 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:49 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:36:03PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 06/18] backlight: ili9320: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 06/18] backlight: ili9320: " Krzysztof Kozlowski
@ 2024-04-15 12:49 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:49 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:36:04PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 07/18] backlight: jornada720_lcd: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 07/18] backlight: jornada720_lcd: " Krzysztof Kozlowski
@ 2024-04-15 12:52 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:52 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:36:05PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 08/18] backlight: l4f00242t03: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 08/18] backlight: l4f00242t03: " Krzysztof Kozlowski
@ 2024-04-15 12:53 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:53 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:36:06PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 09/18] backlight: lms283gf05: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 09/18] backlight: lms283gf05: " Krzysztof Kozlowski
@ 2024-04-15 12:54 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:54 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:36:07PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 10/18] backlight: lms501kf03: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 10/18] backlight: lms501kf03: " Krzysztof Kozlowski
@ 2024-04-15 12:54 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:54 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:36:08PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 11/18] backlight: ltv350qv: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 11/18] backlight: ltv350qv: " Krzysztof Kozlowski
@ 2024-04-15 12:54 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:54 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:36:09PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 12/18] backlight: otm3225a: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 12/18] backlight: otm3225a: " Krzysztof Kozlowski
@ 2024-04-15 12:54 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:54 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:36:10PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 13/18] backlight: platform_lcd: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 13/18] backlight: platform_lcd: " Krzysztof Kozlowski
@ 2024-04-15 12:55 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:55 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:36:11PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 14/18] backlight: tdo24m: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 14/18] backlight: tdo24m: " Krzysztof Kozlowski
@ 2024-04-15 12:55 ` Daniel Thompson
0 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:55 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:36:12PM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 00/18] backlight: Constify lcd_ops
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
` (18 preceding siblings ...)
2024-04-15 6:56 ` [PATCH 00/18] backlight: " Thomas Zimmermann
@ 2024-04-15 12:58 ` Daniel Thompson
19 siblings, 0 replies; 38+ messages in thread
From: Daniel Thompson @ 2024-04-15 12:58 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, Apr 14, 2024 at 06:35:58PM +0200, Krzysztof Kozlowski wrote:
> Hi,
>
> Dependencies
> ============
> All further patches depend on the first patch. Therefore everything
> could go via backlight tree (please ack) or via cross-tree pulls. Or
> whatever maintainer choose, just coordinate this with backlight.
Thanks for the tidy up.
I've added my Reviewed-by: to all the backlight patches (for Lee) and
I'm happy with the other patches too... but I didn't want my R-b on the
HID and fbdev patches to be confused for an ack.
Daniel.
> ---
> Krzysztof Kozlowski (18):
> backlight: Constify lcd_ops
> backlight: ams369fg06: Constify lcd_ops
> backlight: corgi_lcd: Constify lcd_ops
> backlight: hx8357: Constify lcd_ops
> backlight: ili922x: Constify lcd_ops
> backlight: ili9320: Constify lcd_ops
> backlight: jornada720_lcd: Constify lcd_ops
> backlight: l4f00242t03: Constify lcd_ops
> backlight: lms283gf05: Constify lcd_ops
> backlight: lms501kf03: Constify lcd_ops
> backlight: ltv350qv: Constify lcd_ops
> backlight: otm3225a: Constify lcd_ops
> backlight: platform_lcd: Constify lcd_ops
> backlight: tdo24m: Constify lcd_ops
> HID: picoLCD: Constify lcd_ops
> fbdev: clps711x: Constify lcd_ops
> fbdev: imx: Constify lcd_ops
> fbdev: omap: lcd_ams_delta: Constify lcd_ops
>
> drivers/hid/hid-picolcd_lcd.c | 2 +-
> drivers/video/backlight/ams369fg06.c | 2 +-
> drivers/video/backlight/corgi_lcd.c | 2 +-
> drivers/video/backlight/hx8357.c | 2 +-
> drivers/video/backlight/ili922x.c | 2 +-
> drivers/video/backlight/ili9320.c | 2 +-
> drivers/video/backlight/jornada720_lcd.c | 2 +-
> drivers/video/backlight/l4f00242t03.c | 2 +-
> drivers/video/backlight/lcd.c | 4 ++--
> drivers/video/backlight/lms283gf05.c | 2 +-
> drivers/video/backlight/lms501kf03.c | 2 +-
> drivers/video/backlight/ltv350qv.c | 2 +-
> drivers/video/backlight/otm3225a.c | 2 +-
> drivers/video/backlight/platform_lcd.c | 2 +-
> drivers/video/backlight/tdo24m.c | 2 +-
> drivers/video/fbdev/clps711x-fb.c | 2 +-
> drivers/video/fbdev/imxfb.c | 2 +-
> drivers/video/fbdev/omap/lcd_ams_delta.c | 2 +-
> include/linux/lcd.h | 6 +++---
> 19 files changed, 22 insertions(+), 22 deletions(-)
> ---
> base-commit: 9ed46da14b9b9b2ad4edb3b0c545b6dbe5c00d39
> change-id: 20240414-video-backlight-lcd-ops-276d8439ffb8
>
> Best regards,
> --
> Krzysztof Kozlowski <krzk@kernel.org>
>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 15/18] HID: picoLCD: Constify lcd_ops
2024-04-14 16:36 ` [PATCH 15/18] HID: picoLCD: " Krzysztof Kozlowski
@ 2024-04-16 8:07 ` Bruno Prémont
0 siblings, 0 replies; 38+ messages in thread
From: Bruno Prémont @ 2024-04-16 8:07 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Lee Jones, Daniel Thompson, Jingoo Han, Helge Deller,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam, dri-devel,
linux-fbdev, linux-kernel, linux-input, linux-arm-kernel, imx,
linux-omap
On Sun, 14 Apr 2024 18:36:13 +0200 Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core backlight code, so it can be
> made const for increased code safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Bruno Prémont <bonbons@linux-vserver.org>
> ---
>
> Depends on the first patch in the series.
> ---
> drivers/hid/hid-picolcd_lcd.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/hid/hid-picolcd_lcd.c b/drivers/hid/hid-picolcd_lcd.c
> index 0c4b76de8ae5..061a33ba7b1d 100644
> --- a/drivers/hid/hid-picolcd_lcd.c
> +++ b/drivers/hid/hid-picolcd_lcd.c
> @@ -46,7 +46,7 @@ static int picolcd_check_lcd_fb(struct lcd_device *ldev, struct fb_info *fb)
> return fb && fb == picolcd_fbinfo((struct picolcd_data *)lcd_get_data(ldev));
> }
>
> -static struct lcd_ops picolcd_lcdops = {
> +static const struct lcd_ops picolcd_lcdops = {
> .get_contrast = picolcd_get_contrast,
> .set_contrast = picolcd_set_contrast,
> .check_fb = picolcd_check_lcd_fb,
>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH 00/18] backlight: Constify lcd_ops
2024-04-15 7:48 ` Krzysztof Kozlowski
@ 2024-04-16 12:17 ` Lee Jones
0 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2024-04-16 12:17 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Thomas Zimmermann, Daniel Thompson, Jingoo Han, Helge Deller,
Bruno Prémont, Jiri Kosina, Benjamin Tissoires,
Alexander Shiyan, Sascha Hauer, Pengutronix Kernel Team,
Shawn Guo, Fabio Estevam, dri-devel, linux-fbdev, linux-kernel,
linux-input, linux-arm-kernel, imx, linux-omap
On Mon, 15 Apr 2024, Krzysztof Kozlowski wrote:
> On 15/04/2024 08:56, Thomas Zimmermann wrote:
> > Hi
> >
> > for patches 16, 17 and 18:
> >
> > Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
>
> This does not work like this. Toolset will apply your review tag for
> EVERYTHING. You must provide tag under each individual patch.
And for that to not happen now, you have to resubmit the set. :)
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 38+ messages in thread
end of thread, other threads:[~2024-04-16 12:17 UTC | newest]
Thread overview: 38+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-04-14 16:35 [PATCH 00/18] backlight: Constify lcd_ops Krzysztof Kozlowski
2024-04-14 16:35 ` [PATCH 01/18] " Krzysztof Kozlowski
2024-04-15 12:46 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 02/18] backlight: ams369fg06: " Krzysztof Kozlowski
2024-04-15 12:47 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 03/18] backlight: corgi_lcd: " Krzysztof Kozlowski
2024-04-15 12:48 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 04/18] backlight: hx8357: " Krzysztof Kozlowski
2024-04-15 12:49 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 05/18] backlight: ili922x: " Krzysztof Kozlowski
2024-04-15 12:49 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 06/18] backlight: ili9320: " Krzysztof Kozlowski
2024-04-15 12:49 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 07/18] backlight: jornada720_lcd: " Krzysztof Kozlowski
2024-04-15 12:52 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 08/18] backlight: l4f00242t03: " Krzysztof Kozlowski
2024-04-15 12:53 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 09/18] backlight: lms283gf05: " Krzysztof Kozlowski
2024-04-15 12:54 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 10/18] backlight: lms501kf03: " Krzysztof Kozlowski
2024-04-15 12:54 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 11/18] backlight: ltv350qv: " Krzysztof Kozlowski
2024-04-15 12:54 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 12/18] backlight: otm3225a: " Krzysztof Kozlowski
2024-04-15 12:54 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 13/18] backlight: platform_lcd: " Krzysztof Kozlowski
2024-04-15 12:55 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 14/18] backlight: tdo24m: " Krzysztof Kozlowski
2024-04-15 12:55 ` Daniel Thompson
2024-04-14 16:36 ` [PATCH 15/18] HID: picoLCD: " Krzysztof Kozlowski
2024-04-16 8:07 ` Bruno Prémont
2024-04-14 16:36 ` [PATCH 16/18] fbdev: clps711x: " Krzysztof Kozlowski
2024-04-14 16:36 ` [PATCH 17/18] fbdev: imx: " Krzysztof Kozlowski
2024-04-14 16:36 ` [PATCH 18/18] fbdev: omap: lcd_ams_delta: " Krzysztof Kozlowski
2024-04-15 6:56 ` [PATCH 00/18] backlight: " Thomas Zimmermann
2024-04-15 7:48 ` Krzysztof Kozlowski
2024-04-16 12:17 ` Lee Jones
2024-04-15 12:58 ` Daniel Thompson
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 NNTP newsgroup(s).