* [PATCH v2 01/19] backlight: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 02/19] backlight: ams369fg06: " Krzysztof Kozlowski
` (20 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 02/19] backlight: ams369fg06: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 01/19] " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 03/19] backlight: corgi_lcd: " Krzysztof Kozlowski
` (19 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 03/19] backlight: corgi_lcd: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 01/19] " Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 02/19] backlight: ams369fg06: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 04/19] backlight: hx8357: " Krzysztof Kozlowski
` (18 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 04/19] backlight: hx8357: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (2 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 03/19] backlight: corgi_lcd: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 05/19] backlight: ili922x: " Krzysztof Kozlowski
` (17 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 05/19] backlight: ili922x: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (3 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 04/19] backlight: hx8357: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 06/19] backlight: ili9320: " Krzysztof Kozlowski
` (16 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 06/19] backlight: ili9320: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (4 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 05/19] backlight: ili922x: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 07/19] backlight: jornada720_lcd: " Krzysztof Kozlowski
` (15 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 07/19] backlight: jornada720_lcd: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (5 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 06/19] backlight: ili9320: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 08/19] backlight: l4f00242t03: " Krzysztof Kozlowski
` (14 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 08/19] backlight: l4f00242t03: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (6 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 07/19] backlight: jornada720_lcd: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 09/19] backlight: lms283gf05: " Krzysztof Kozlowski
` (13 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 09/19] backlight: lms283gf05: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (7 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 08/19] backlight: l4f00242t03: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 10/19] backlight: lms501kf03: " Krzysztof Kozlowski
` (12 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 10/19] backlight: lms501kf03: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (8 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 09/19] backlight: lms283gf05: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 11/19] backlight: ltv350qv: " Krzysztof Kozlowski
` (11 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 11/19] backlight: ltv350qv: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (9 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 10/19] backlight: lms501kf03: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 12/19] backlight: otm3225a: " Krzysztof Kozlowski
` (10 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 12/19] backlight: otm3225a: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (10 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 11/19] backlight: ltv350qv: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 13/19] backlight: platform_lcd: " Krzysztof Kozlowski
` (9 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 13/19] backlight: platform_lcd: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (11 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 12/19] backlight: otm3225a: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 14/19] backlight: tdo24m: " Krzysztof Kozlowski
` (8 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 14/19] backlight: tdo24m: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (12 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 13/19] backlight: platform_lcd: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 15/19] HID: picoLCD: " Krzysztof Kozlowski
` (7 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 15/19] HID: picoLCD: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (13 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 14/19] backlight: tdo24m: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 16/19] fbdev: clps711x: " Krzysztof Kozlowski
` (6 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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.
Reviewed-by: Bruno Prémont <bonbons@linux-vserver.org>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 16/19] fbdev: clps711x: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (14 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 15/19] HID: picoLCD: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 17/19] fbdev: imx: " Krzysztof Kozlowski
` (5 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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, Thomas Zimmermann,
Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 17/19] fbdev: imx: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (15 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 16/19] fbdev: clps711x: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 7:05 ` Peng Fan
2024-04-24 6:33 ` [PATCH v2 18/19] fbdev: omap: lcd_ams_delta: " Krzysztof Kozlowski
` (4 subsequent siblings)
21 siblings, 1 reply; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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, Thomas Zimmermann,
Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* RE: [PATCH v2 17/19] fbdev: imx: Constify lcd_ops
2024-04-24 6:33 ` [PATCH v2 17/19] fbdev: imx: " Krzysztof Kozlowski
@ 2024-04-24 7:05 ` Peng Fan
0 siblings, 0 replies; 29+ messages in thread
From: Peng Fan @ 2024-04-24 7:05 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, Thomas Zimmermann
> Subject: [PATCH v2 17/19] fbdev: imx: Constify lcd_ops
>
> 'struct lcd_ops' is not modified by core backlight code, so it can be made
> const for increased code safety.
>
> Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
> ---
>
> 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.43.0
>
^ permalink raw reply [flat|nested] 29+ messages in thread
* [PATCH v2 18/19] fbdev: omap: lcd_ams_delta: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (16 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 17/19] fbdev: imx: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 6:33 ` [PATCH v2 19/19] const_structs.checkpatch: add lcd_ops Krzysztof Kozlowski
` (3 subsequent siblings)
21 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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, Thomas Zimmermann,
Krzysztof Kozlowski
'struct lcd_ops' is not modified by core backlight code, so it can be
made const for increased code safety.
Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
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.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [PATCH v2 19/19] const_structs.checkpatch: add lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (17 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 18/19] fbdev: omap: lcd_ams_delta: " Krzysztof Kozlowski
@ 2024-04-24 6:33 ` Krzysztof Kozlowski
2024-04-24 10:08 ` Daniel Thompson
2024-05-02 7:21 ` Lee Jones
2024-05-02 7:22 ` [PATCH v2 00/19] backlight: Constify lcd_ops Lee Jones
` (2 subsequent siblings)
21 siblings, 2 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-04-24 6:33 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, Thomas Weißschuh,
Krzysztof Kozlowski
'struct lcd_ops' is not modified by core code.
Suggested-by: Thomas Weißschuh <linux@weissschuh.net>
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
Patch making lcd_ops const in progress:
https://lore.kernel.org/r/20240414-video-backlight-lcd-ops-v1-0-9b37fcbf546a@kernel.org
Cc: Lee Jones <lee@kernel.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
Cc: linux-fbdev@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
---
scripts/const_structs.checkpatch | 1 +
1 file changed, 1 insertion(+)
diff --git a/scripts/const_structs.checkpatch b/scripts/const_structs.checkpatch
index fa96cfd16e99..52e5bfb61fd0 100644
--- a/scripts/const_structs.checkpatch
+++ b/scripts/const_structs.checkpatch
@@ -39,6 +39,7 @@ kgdb_arch
kgdb_io
kobj_type
kset_uevent_ops
+lcd_ops
lock_manager_operations
machine_desc
microcode_ops
--
2.43.0
^ permalink raw reply related [flat|nested] 29+ messages in thread
* Re: [PATCH v2 19/19] const_structs.checkpatch: add lcd_ops
2024-04-24 6:33 ` [PATCH v2 19/19] const_structs.checkpatch: add lcd_ops Krzysztof Kozlowski
@ 2024-04-24 10:08 ` Daniel Thompson
2024-05-02 7:21 ` Lee Jones
1 sibling, 0 replies; 29+ messages in thread
From: Daniel Thompson @ 2024-04-24 10:08 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, Thomas Weißschuh
On Wed, Apr 24, 2024 at 08:33:45AM +0200, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core code.
>
> Suggested-by: Thomas Weißschuh <linux@weissschuh.net>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Daniel.
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2 19/19] const_structs.checkpatch: add lcd_ops
2024-04-24 6:33 ` [PATCH v2 19/19] const_structs.checkpatch: add lcd_ops Krzysztof Kozlowski
2024-04-24 10:08 ` Daniel Thompson
@ 2024-05-02 7:21 ` Lee Jones
1 sibling, 0 replies; 29+ messages in thread
From: Lee Jones @ 2024-05-02 7:21 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: 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, Thomas Weißschuh
On Wed, 24 Apr 2024, Krzysztof Kozlowski wrote:
> 'struct lcd_ops' is not modified by core code.
>
> Suggested-by: Thomas Weißschuh <linux@weissschuh.net>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
>
> ---
>
> Patch making lcd_ops const in progress:
> https://lore.kernel.org/r/20240414-video-backlight-lcd-ops-v1-0-9b37fcbf546a@kernel.org
>
> Cc: Lee Jones <lee@kernel.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
> Cc: linux-fbdev@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> ---
> scripts/const_structs.checkpatch | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/scripts/const_structs.checkpatch b/scripts/const_structs.checkpatch
> index fa96cfd16e99..52e5bfb61fd0 100644
> --- a/scripts/const_structs.checkpatch
> +++ b/scripts/const_structs.checkpatch
> @@ -39,6 +39,7 @@ kgdb_arch
> kgdb_io
> kobj_type
> kset_uevent_ops
> +lcd_ops
Acked-by: Lee Jones <lee@kernel.org>
> lock_manager_operations
> machine_desc
> microcode_ops
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2 00/19] backlight: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (18 preceding siblings ...)
2024-04-24 6:33 ` [PATCH v2 19/19] const_structs.checkpatch: add lcd_ops Krzysztof Kozlowski
@ 2024-05-02 7:22 ` Lee Jones
2024-05-02 7:47 ` Krzysztof Kozlowski
2024-05-03 9:41 ` Lee Jones
2024-05-10 14:55 ` [GIT PULL] Immutable branch between Backlight, HID and fbdev due for the v6.10 merge window Lee Jones
21 siblings, 1 reply; 29+ messages in thread
From: Lee Jones @ 2024-05-02 7:22 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: 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, Thomas Zimmermann, Thomas Weißschuh
On Wed, 24 Apr 2024, Krzysztof Kozlowski wrote:
> Hi,
>
> Changes in v2:
> - Collect tags, including wrongly places Thomas' tag (which requires me
> to manually edit 15 other patches to drop it).
> - Combine here checkpatch patch:
> https://lore.kernel.org/all/20240414185440.288812-1-krzk@kernel.org/
> - Link to v1: https://lore.kernel.org/r/20240414-video-backlight-lcd-ops-v1-0-9b37fcbf546a@kernel.org
What's missing? Are we good to go?
> 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 (19):
> 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
> const_structs.checkpatch: add 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 +++---
> scripts/const_structs.checkpatch | 1 +
> 20 files changed, 23 insertions(+), 22 deletions(-)
> ---
> base-commit: a59668a9397e7245b26e9be85d23f242ff757ae8
> change-id: 20240414-video-backlight-lcd-ops-276d8439ffb8
>
> Best regards,
> --
> Krzysztof Kozlowski <krzk@kernel.org>
>
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2 00/19] backlight: Constify lcd_ops
2024-05-02 7:22 ` [PATCH v2 00/19] backlight: Constify lcd_ops Lee Jones
@ 2024-05-02 7:47 ` Krzysztof Kozlowski
0 siblings, 0 replies; 29+ messages in thread
From: Krzysztof Kozlowski @ 2024-05-02 7:47 UTC (permalink / raw)
To: Lee Jones
Cc: 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, Thomas Zimmermann, Thomas Weißschuh
On 02/05/2024 09:22, Lee Jones wrote:
> On Wed, 24 Apr 2024, Krzysztof Kozlowski wrote:
>
>> Hi,
>>
>> Changes in v2:
>> - Collect tags, including wrongly places Thomas' tag (which requires me
>> to manually edit 15 other patches to drop it).
>> - Combine here checkpatch patch:
>> https://lore.kernel.org/all/20240414185440.288812-1-krzk@kernel.org/
>> - Link to v1: https://lore.kernel.org/r/20240414-video-backlight-lcd-ops-v1-0-9b37fcbf546a@kernel.org
>
> What's missing? Are we good to go?
Nothing missing, you can take entire set (also last checkpatch patch
which you acked).
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2 00/19] backlight: Constify lcd_ops
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (19 preceding siblings ...)
2024-05-02 7:22 ` [PATCH v2 00/19] backlight: Constify lcd_ops Lee Jones
@ 2024-05-03 9:41 ` Lee Jones
2024-05-03 9:43 ` Lee Jones
2024-05-10 14:55 ` [GIT PULL] Immutable branch between Backlight, HID and fbdev due for the v6.10 merge window Lee Jones
21 siblings, 1 reply; 29+ messages in thread
From: Lee Jones @ 2024-05-03 9:41 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, Krzysztof Kozlowski
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Thomas Zimmermann,
Thomas Weißschuh
On Wed, 24 Apr 2024 08:33:26 +0200, Krzysztof Kozlowski wrote:
> Changes in v2:
> - Collect tags, including wrongly places Thomas' tag (which requires me
> to manually edit 15 other patches to drop it).
> - Combine here checkpatch patch:
> https://lore.kernel.org/all/20240414185440.288812-1-krzk@kernel.org/
> - Link to v1: https://lore.kernel.org/r/20240414-video-backlight-lcd-ops-v1-0-9b37fcbf546a@kernel.org
>
> [...]
Applied, thanks!
[01/19] backlight: Constify lcd_ops
commit: 9258e31adebf0ccf0797867841f3f9800695dba2
[02/19] backlight: ams369fg06: Constify lcd_ops
commit: 8053c4fa200c3fef07859bac9469ad3f26f5aba1
[03/19] backlight: corgi_lcd: Constify lcd_ops
commit: 18c5d4ab9f6312f2c9c6c409287d552112db810a
[04/19] backlight: hx8357: Constify lcd_ops
commit: 1d669c1998b559393ec2eaac0449f4989a255049
[05/19] backlight: ili922x: Constify lcd_ops
commit: e77fef89de954b1557cb91b64696cd4fc06c80ad
[06/19] backlight: ili9320: Constify lcd_ops
commit: 06cfc92faa1eabb2ea226c58d6fd0b5ab117ee39
[07/19] backlight: jornada720_lcd: Constify lcd_ops
commit: a54b4999dc204bc5839bb70602078c7c8e4a5010
[08/19] backlight: l4f00242t03: Constify lcd_ops
commit: 657e6c1b270e9f4a890059f5d08a08ea842fa1a8
[09/19] backlight: lms283gf05: Constify lcd_ops
commit: 66e5a10818fd332e973d36429e36f4c436a86a91
[10/19] backlight: lms501kf03: Constify lcd_ops
commit: 31c205d1e8426dd0cce0143c500ff1ff71fe64d1
[11/19] backlight: ltv350qv: Constify lcd_ops
commit: 24424f84d7568d9d794657622e080b1cba1e9290
[12/19] backlight: otm3225a: Constify lcd_ops
commit: 02949072ee8fb6141cd8ac2be9867ef466580ddb
[13/19] backlight: platform_lcd: Constify lcd_ops
commit: d217a8d5a39851caa16996756682715c9debb4a9
[14/19] backlight: tdo24m: Constify lcd_ops
commit: c7a1809d1982f671e66a4b1c1ffd8bdd5ba260aa
[15/19] HID: picoLCD: Constify lcd_ops
commit: 238724635763e7c5d82c0581b0c49e5dfdd5505a
[16/19] fbdev: clps711x: Constify lcd_ops
commit: 55d9a955375af3b3fd5725a9b5cbc658d4bdd244
[17/19] fbdev: imx: Constify lcd_ops
commit: a6abbb5783345c4c7cc9fbd583b81e167bd0207d
[18/19] fbdev: omap: lcd_ams_delta: Constify lcd_ops
commit: ca991e8e096c9f0cff0300289e2d4813192b8ef3
[19/19] const_structs.checkpatch: add lcd_ops
commit: f02aeccbec6108d768f54d31e7cb48b06c0e3814
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2 00/19] backlight: Constify lcd_ops
2024-05-03 9:41 ` Lee Jones
@ 2024-05-03 9:43 ` Lee Jones
2024-05-03 9:55 ` Lee Jones
0 siblings, 1 reply; 29+ messages in thread
From: Lee Jones @ 2024-05-03 9:43 UTC (permalink / raw)
To: Daniel Thompson, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam,
Krzysztof Kozlowski
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Thomas Zimmermann,
Thomas Weißschuh
On Fri, 03 May 2024, Lee Jones wrote:
> On Wed, 24 Apr 2024 08:33:26 +0200, Krzysztof Kozlowski wrote:
> > Changes in v2:
> > - Collect tags, including wrongly places Thomas' tag (which requires me
> > to manually edit 15 other patches to drop it).
> > - Combine here checkpatch patch:
> > https://lore.kernel.org/all/20240414185440.288812-1-krzk@kernel.org/
> > - Link to v1: https://lore.kernel.org/r/20240414-video-backlight-lcd-ops-v1-0-9b37fcbf546a@kernel.org
> >
> > [...]
>
> Applied, thanks!
>
> [01/19] backlight: Constify lcd_ops
> commit: 9258e31adebf0ccf0797867841f3f9800695dba2
> [02/19] backlight: ams369fg06: Constify lcd_ops
> commit: 8053c4fa200c3fef07859bac9469ad3f26f5aba1
> [03/19] backlight: corgi_lcd: Constify lcd_ops
> commit: 18c5d4ab9f6312f2c9c6c409287d552112db810a
> [04/19] backlight: hx8357: Constify lcd_ops
> commit: 1d669c1998b559393ec2eaac0449f4989a255049
> [05/19] backlight: ili922x: Constify lcd_ops
> commit: e77fef89de954b1557cb91b64696cd4fc06c80ad
> [06/19] backlight: ili9320: Constify lcd_ops
> commit: 06cfc92faa1eabb2ea226c58d6fd0b5ab117ee39
> [07/19] backlight: jornada720_lcd: Constify lcd_ops
> commit: a54b4999dc204bc5839bb70602078c7c8e4a5010
> [08/19] backlight: l4f00242t03: Constify lcd_ops
> commit: 657e6c1b270e9f4a890059f5d08a08ea842fa1a8
> [09/19] backlight: lms283gf05: Constify lcd_ops
> commit: 66e5a10818fd332e973d36429e36f4c436a86a91
> [10/19] backlight: lms501kf03: Constify lcd_ops
> commit: 31c205d1e8426dd0cce0143c500ff1ff71fe64d1
> [11/19] backlight: ltv350qv: Constify lcd_ops
> commit: 24424f84d7568d9d794657622e080b1cba1e9290
> [12/19] backlight: otm3225a: Constify lcd_ops
> commit: 02949072ee8fb6141cd8ac2be9867ef466580ddb
> [13/19] backlight: platform_lcd: Constify lcd_ops
> commit: d217a8d5a39851caa16996756682715c9debb4a9
> [14/19] backlight: tdo24m: Constify lcd_ops
> commit: c7a1809d1982f671e66a4b1c1ffd8bdd5ba260aa
> [15/19] HID: picoLCD: Constify lcd_ops
> commit: 238724635763e7c5d82c0581b0c49e5dfdd5505a
> [16/19] fbdev: clps711x: Constify lcd_ops
> commit: 55d9a955375af3b3fd5725a9b5cbc658d4bdd244
> [17/19] fbdev: imx: Constify lcd_ops
> commit: a6abbb5783345c4c7cc9fbd583b81e167bd0207d
> [18/19] fbdev: omap: lcd_ams_delta: Constify lcd_ops
> commit: ca991e8e096c9f0cff0300289e2d4813192b8ef3
> [19/19] const_structs.checkpatch: add lcd_ops
> commit: f02aeccbec6108d768f54d31e7cb48b06c0e3814
Please ensure you use checkpatch.pl before submitting:
====
ERROR: Does not appear to be a unified-diff format patch
total: 1 errors, 0 warnings, 0 lines checked
NOTE: For some of the reported defects, checkpatch may be able to
mechanically convert to the typical style using --fix or --fix-inplace.
"[PATCH v2 19/19] const_structs.checkpatch: add lcd_ops" has style problems, please review.
NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
WARNING: Missing commit description - Add an appropriate one
ERROR: Missing Signed-off-by: line(s)
total: 1 errors, 1 warnings, 7 lines checked
NOTE: For some of the reported defects, checkpatch may be able to
mechanically convert to the typical style using --fix or --fix-inplace.
Your patch has style problems, please review.
NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
====
Some of them are a little odd. I fixed the relevant ones
Submitted for build testing.
If successful, I'll submit a PR for the other maintainers soon.
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH v2 00/19] backlight: Constify lcd_ops
2024-05-03 9:43 ` Lee Jones
@ 2024-05-03 9:55 ` Lee Jones
0 siblings, 0 replies; 29+ messages in thread
From: Lee Jones @ 2024-05-03 9:55 UTC (permalink / raw)
To: Daniel Thompson, Jingoo Han, Helge Deller, Bruno Prémont,
Jiri Kosina, Benjamin Tissoires, Alexander Shiyan, Sascha Hauer,
Pengutronix Kernel Team, Shawn Guo, Fabio Estevam,
Krzysztof Kozlowski
Cc: dri-devel, linux-fbdev, linux-kernel, linux-input,
linux-arm-kernel, imx, linux-omap, Thomas Zimmermann,
Thomas Weißschuh
On Fri, 03 May 2024, Lee Jones wrote:
> On Fri, 03 May 2024, Lee Jones wrote:
>
> > On Wed, 24 Apr 2024 08:33:26 +0200, Krzysztof Kozlowski wrote:
> > > Changes in v2:
> > > - Collect tags, including wrongly places Thomas' tag (which requires me
> > > to manually edit 15 other patches to drop it).
> > > - Combine here checkpatch patch:
> > > https://lore.kernel.org/all/20240414185440.288812-1-krzk@kernel.org/
> > > - Link to v1: https://lore.kernel.org/r/20240414-video-backlight-lcd-ops-v1-0-9b37fcbf546a@kernel.org
> > >
> > > [...]
> >
> > Applied, thanks!
> >
> > [01/19] backlight: Constify lcd_ops
> > commit: 9258e31adebf0ccf0797867841f3f9800695dba2
> > [02/19] backlight: ams369fg06: Constify lcd_ops
> > commit: 8053c4fa200c3fef07859bac9469ad3f26f5aba1
> > [03/19] backlight: corgi_lcd: Constify lcd_ops
> > commit: 18c5d4ab9f6312f2c9c6c409287d552112db810a
> > [04/19] backlight: hx8357: Constify lcd_ops
> > commit: 1d669c1998b559393ec2eaac0449f4989a255049
> > [05/19] backlight: ili922x: Constify lcd_ops
> > commit: e77fef89de954b1557cb91b64696cd4fc06c80ad
> > [06/19] backlight: ili9320: Constify lcd_ops
> > commit: 06cfc92faa1eabb2ea226c58d6fd0b5ab117ee39
> > [07/19] backlight: jornada720_lcd: Constify lcd_ops
> > commit: a54b4999dc204bc5839bb70602078c7c8e4a5010
> > [08/19] backlight: l4f00242t03: Constify lcd_ops
> > commit: 657e6c1b270e9f4a890059f5d08a08ea842fa1a8
> > [09/19] backlight: lms283gf05: Constify lcd_ops
> > commit: 66e5a10818fd332e973d36429e36f4c436a86a91
> > [10/19] backlight: lms501kf03: Constify lcd_ops
> > commit: 31c205d1e8426dd0cce0143c500ff1ff71fe64d1
> > [11/19] backlight: ltv350qv: Constify lcd_ops
> > commit: 24424f84d7568d9d794657622e080b1cba1e9290
> > [12/19] backlight: otm3225a: Constify lcd_ops
> > commit: 02949072ee8fb6141cd8ac2be9867ef466580ddb
> > [13/19] backlight: platform_lcd: Constify lcd_ops
> > commit: d217a8d5a39851caa16996756682715c9debb4a9
> > [14/19] backlight: tdo24m: Constify lcd_ops
> > commit: c7a1809d1982f671e66a4b1c1ffd8bdd5ba260aa
> > [15/19] HID: picoLCD: Constify lcd_ops
> > commit: 238724635763e7c5d82c0581b0c49e5dfdd5505a
> > [16/19] fbdev: clps711x: Constify lcd_ops
> > commit: 55d9a955375af3b3fd5725a9b5cbc658d4bdd244
> > [17/19] fbdev: imx: Constify lcd_ops
> > commit: a6abbb5783345c4c7cc9fbd583b81e167bd0207d
> > [18/19] fbdev: omap: lcd_ams_delta: Constify lcd_ops
> > commit: ca991e8e096c9f0cff0300289e2d4813192b8ef3
> > [19/19] const_structs.checkpatch: add lcd_ops
> > commit: f02aeccbec6108d768f54d31e7cb48b06c0e3814
>
> Please ensure you use checkpatch.pl before submitting:
>
> ====
>
> ERROR: Does not appear to be a unified-diff format patch
>
> total: 1 errors, 0 warnings, 0 lines checked
>
> NOTE: For some of the reported defects, checkpatch may be able to
> mechanically convert to the typical style using --fix or --fix-inplace.
>
> "[PATCH v2 19/19] const_structs.checkpatch: add lcd_ops" has style problems, please review.
>
> NOTE: If any of the errors are false positives, please report
> them to the maintainer, see CHECKPATCH in MAINTAINERS.
> WARNING: Missing commit description - Add an appropriate one
>
> ERROR: Missing Signed-off-by: line(s)
>
> total: 1 errors, 1 warnings, 7 lines checked
>
> NOTE: For some of the reported defects, checkpatch may be able to
> mechanically convert to the typical style using --fix or --fix-inplace.
>
> Your patch has style problems, please review.
>
> NOTE: If any of the errors are false positives, please report
> them to the maintainer, see CHECKPATCH in MAINTAINERS.
>
> ====
>
> Some of them are a little odd. I fixed the relevant ones
Change of plan - I actually can't find anything to fix.
All of this reports seem bogus!
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 29+ messages in thread
* [GIT PULL] Immutable branch between Backlight, HID and fbdev due for the v6.10 merge window
2024-04-24 6:33 [PATCH v2 00/19] backlight: Constify lcd_ops Krzysztof Kozlowski
` (20 preceding siblings ...)
2024-05-03 9:41 ` Lee Jones
@ 2024-05-10 14:55 ` Lee Jones
21 siblings, 0 replies; 29+ messages in thread
From: Lee Jones @ 2024-05-10 14:55 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: 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, Thomas Zimmermann, Thomas Weißschuh
Enjoy!
The following changes since commit 4cece764965020c22cff7665b18a012006359095:
Linux 6.9-rc1 (2024-03-24 14:10:05 -0700)
are available in the Git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/lee/backlight.git ib-backlight-hid-fbdev-lcd-scripts-v6.10
for you to fetch changes up to 82b9007bc4f8c22975d640d7df6743366f25a353:
const_structs.checkpatch: add lcd_ops (2024-05-03 10:45:55 +0100)
----------------------------------------------------------------
Immutable branch between Backlight, HID and fbdev due for the v6.10 merge window
----------------------------------------------------------------
Krzysztof Kozlowski (19):
backlight: lcd: 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
const_structs.checkpatch: add 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 +++---
scripts/const_structs.checkpatch | 1 +
20 files changed, 23 insertions(+), 22 deletions(-)
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 29+ messages in thread