* [PATCH v4] drm: atmel-hlcdc: replace dev_* print functions with drm_* variants
@ 2025-08-13 22:39 Eslam Khafagy
2025-08-18 9:16 ` Manikandan.M
0 siblings, 1 reply; 6+ messages in thread
From: Eslam Khafagy @ 2025-08-13 22:39 UTC (permalink / raw)
To: Manikandan Muralidharan, Dharma Balasubiramani, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, David Airlie, Simona Vetter,
Nicolas Ferre, Alexandre Belloni, Claudiu Beznea, dri-devel,
linux-arm-kernel, linux-kernel
Cc: skhan, eslam.medhat1993
Update the Atmel HLCDC code to use DRM print macros drm_*() instead of
dev_warn() and dev_err(). This change ensures consistency with DRM
subsystem logging conventions [1].
[1]
Link: https://docs.kernel.org/gpu/todo.html#convert-logging-to-drm-functions-with-drm-device-parameter
Signed-off-by: Eslam Khafagy <eslam.medhat1993@gmail.com>
---
.../gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 21 ++++++++++---------
drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 14 ++++++-------
.../gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 3 ++-
.../gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 6 +++---
4 files changed, 23 insertions(+), 21 deletions(-)
diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
index 0f7ffb3ced20..e0efc7309b1b 100644
--- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
+++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
@@ -20,6 +20,7 @@
#include <drm/drm_atomic_helper.h>
#include <drm/drm_crtc.h>
#include <drm/drm_modeset_helper_vtables.h>
+#include <drm/drm_print.h>
#include <drm/drm_probe_helper.h>
#include <drm/drm_vblank.h>
@@ -215,32 +216,32 @@ static void atmel_hlcdc_crtc_atomic_disable(struct drm_crtc *c,
if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
!(status & ATMEL_XLCDC_CM),
10, 1000))
- dev_warn(dev->dev, "Atmel LCDC status register CMSTS timeout\n");
+ drm_warn(dev, "Atmel LCDC status register CMSTS timeout\n");
regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_XLCDC_SD);
if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
status & ATMEL_XLCDC_SD,
10, 1000))
- dev_warn(dev->dev, "Atmel LCDC status register SDSTS timeout\n");
+ drm_warn(dev, "Atmel LCDC status register SDSTS timeout\n");
}
regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_DISP);
if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
!(status & ATMEL_HLCDC_DISP),
10, 1000))
- dev_warn(dev->dev, "Atmel LCDC status register DISPSTS timeout\n");
+ drm_warn(dev, "Atmel LCDC status register DISPSTS timeout\n");
regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_SYNC);
if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
!(status & ATMEL_HLCDC_SYNC),
10, 1000))
- dev_warn(dev->dev, "Atmel LCDC status register LCDSTS timeout\n");
+ drm_warn(dev, "Atmel LCDC status register LCDSTS timeout\n");
regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_PIXEL_CLK);
if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
!(status & ATMEL_HLCDC_PIXEL_CLK),
10, 1000))
- dev_warn(dev->dev, "Atmel LCDC status register CLKSTS timeout\n");
+ drm_warn(dev, "Atmel LCDC status register CLKSTS timeout\n");
clk_disable_unprepare(crtc->dc->hlcdc->sys_clk);
pinctrl_pm_select_sleep_state(dev->dev);
@@ -269,32 +270,32 @@ static void atmel_hlcdc_crtc_atomic_enable(struct drm_crtc *c,
if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
status & ATMEL_HLCDC_PIXEL_CLK,
10, 1000))
- dev_warn(dev->dev, "Atmel LCDC status register CLKSTS timeout\n");
+ drm_warn(dev, "Atmel LCDC status register CLKSTS timeout\n");
regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_HLCDC_SYNC);
if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
status & ATMEL_HLCDC_SYNC,
10, 1000))
- dev_warn(dev->dev, "Atmel LCDC status register LCDSTS timeout\n");
+ drm_warn(dev, "Atmel LCDC status register LCDSTS timeout\n");
regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_HLCDC_DISP);
if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
status & ATMEL_HLCDC_DISP,
10, 1000))
- dev_warn(dev->dev, "Atmel LCDC status register DISPSTS timeout\n");
+ drm_warn(dev, "Atmel LCDC status register DISPSTS timeout\n");
if (crtc->dc->desc->is_xlcdc) {
regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_XLCDC_CM);
if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
status & ATMEL_XLCDC_CM,
10, 1000))
- dev_warn(dev->dev, "Atmel LCDC status register CMSTS timeout\n");
+ drm_warn(dev, "Atmel LCDC status register CMSTS timeout\n");
regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_XLCDC_SD);
if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
!(status & ATMEL_XLCDC_SD),
10, 1000))
- dev_warn(dev->dev, "Atmel LCDC status register SDSTS timeout\n");
+ drm_warn(dev, "Atmel LCDC status register SDSTS timeout\n");
}
pm_runtime_put_sync(dev->dev);
diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
index fa8ad94e431a..acb017a2486b 100644
--- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
+++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
@@ -724,19 +724,19 @@ static int atmel_hlcdc_dc_modeset_init(struct drm_device *dev)
ret = atmel_hlcdc_create_outputs(dev);
if (ret) {
- dev_err(dev->dev, "failed to create HLCDC outputs: %d\n", ret);
+ drm_err(dev, "failed to create HLCDC outputs: %d\n", ret);
return ret;
}
ret = atmel_hlcdc_create_planes(dev);
if (ret) {
- dev_err(dev->dev, "failed to create planes: %d\n", ret);
+ drm_err(dev, "failed to create planes: %d\n", ret);
return ret;
}
ret = atmel_hlcdc_crtc_create(dev);
if (ret) {
- dev_err(dev->dev, "failed to create crtc\n");
+ drm_err(dev, "failed to create crtc\n");
return ret;
}
@@ -778,7 +778,7 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
ret = clk_prepare_enable(dc->hlcdc->periph_clk);
if (ret) {
- dev_err(dev->dev, "failed to enable periph_clk\n");
+ drm_err(dev, "failed to enable periph_clk\n");
return ret;
}
@@ -786,13 +786,13 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
ret = drm_vblank_init(dev, 1);
if (ret < 0) {
- dev_err(dev->dev, "failed to initialize vblank\n");
+ drm_err(dev, "failed to initialize vblank\n");
goto err_periph_clk_disable;
}
ret = atmel_hlcdc_dc_modeset_init(dev);
if (ret < 0) {
- dev_err(dev->dev, "failed to initialize mode setting\n");
+ drm_err(dev, "failed to initialize mode setting\n");
goto err_periph_clk_disable;
}
@@ -802,7 +802,7 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
ret = atmel_hlcdc_dc_irq_install(dev, dc->hlcdc->irq);
pm_runtime_put_sync(dev->dev);
if (ret < 0) {
- dev_err(dev->dev, "failed to install IRQ handler\n");
+ drm_err(dev, "failed to install IRQ handler\n");
goto err_periph_clk_disable;
}
diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
index 50fee6a93964..0b8a86afb096 100644
--- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
+++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
@@ -15,6 +15,7 @@
#include <drm/drm_bridge.h>
#include <drm/drm_encoder.h>
#include <drm/drm_of.h>
+#include <drm/drm_print.h>
#include <drm/drm_simple_kms_helper.h>
#include "atmel_hlcdc_dc.h"
@@ -92,7 +93,7 @@ static int atmel_hlcdc_attach_endpoint(struct drm_device *dev, int endpoint)
output->bus_fmt = atmel_hlcdc_of_bus_fmt(ep);
of_node_put(ep);
if (output->bus_fmt < 0) {
- dev_err(dev->dev, "endpoint %d: invalid bus width\n", endpoint);
+ drm_err(dev, "endpoint %d: invalid bus width\n", endpoint);
return -EINVAL;
}
diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
index 4a7ba0918eca..817284509b57 100644
--- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
+++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
@@ -1034,7 +1034,7 @@ static void atmel_hlcdc_irq_dbg(struct atmel_hlcdc_plane *plane,
if (isr &
(ATMEL_HLCDC_LAYER_OVR_IRQ(0) | ATMEL_HLCDC_LAYER_OVR_IRQ(1) |
ATMEL_HLCDC_LAYER_OVR_IRQ(2)))
- dev_dbg(plane->base.dev->dev, "overrun on plane %s\n",
+ drm_dbg(plane->base.dev, "overrun on plane %s\n",
desc->name);
}
@@ -1051,7 +1051,7 @@ static void atmel_xlcdc_irq_dbg(struct atmel_hlcdc_plane *plane,
if (isr &
(ATMEL_XLCDC_LAYER_OVR_IRQ(0) | ATMEL_XLCDC_LAYER_OVR_IRQ(1) |
ATMEL_XLCDC_LAYER_OVR_IRQ(2)))
- dev_dbg(plane->base.dev->dev, "overrun on plane %s\n",
+ drm_dbg(plane->base.dev, "overrun on plane %s\n",
desc->name);
}
@@ -1140,7 +1140,7 @@ static void atmel_hlcdc_plane_reset(struct drm_plane *p)
if (state) {
if (atmel_hlcdc_plane_alloc_dscrs(p, state)) {
kfree(state);
- dev_err(p->dev->dev,
+ drm_err(p->dev,
"Failed to allocate initial plane state\n");
return;
}
--
2.43.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v4] drm: atmel-hlcdc: replace dev_* print functions with drm_* variants
2025-08-13 22:39 [PATCH v4] drm: atmel-hlcdc: replace dev_* print functions with drm_* variants Eslam Khafagy
@ 2025-08-18 9:16 ` Manikandan.M
2025-09-13 22:19 ` Eslam Khafagy
2025-10-08 10:27 ` Dharma.B
0 siblings, 2 replies; 6+ messages in thread
From: Manikandan.M @ 2025-08-18 9:16 UTC (permalink / raw)
To: eslam.medhat1993, Dharma.B, maarten.lankhorst, mripard,
tzimmermann, airlied, simona, Nicolas.Ferre, alexandre.belloni,
claudiu.beznea, dri-devel, linux-arm-kernel, linux-kernel
Cc: skhan
On 14/08/25 4:09 am, Eslam Khafagy wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> Update the Atmel HLCDC code to use DRM print macros drm_*() instead of
> dev_warn() and dev_err(). This change ensures consistency with DRM
> subsystem logging conventions [1].
>
> [1]
> Link: https://docs.kernel.org/gpu/todo.html#convert-logging-to-drm-functions-with-drm-device-parameter
>
> Signed-off-by: Eslam Khafagy <eslam.medhat1993@gmail.com>
Reviewed-by: Manikandan Muralidharan <manikandan.m@microchip.com>
> ---
> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 21 ++++++++++---------
> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 14 ++++++-------
> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 3 ++-
> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 6 +++---
> 4 files changed, 23 insertions(+), 21 deletions(-)
>
> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
> index 0f7ffb3ced20..e0efc7309b1b 100644
> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
> @@ -20,6 +20,7 @@
> #include <drm/drm_atomic_helper.h>
> #include <drm/drm_crtc.h>
> #include <drm/drm_modeset_helper_vtables.h>
> +#include <drm/drm_print.h>
> #include <drm/drm_probe_helper.h>
> #include <drm/drm_vblank.h>
>
> @@ -215,32 +216,32 @@ static void atmel_hlcdc_crtc_atomic_disable(struct drm_crtc *c,
> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> !(status & ATMEL_XLCDC_CM),
> 10, 1000))
> - dev_warn(dev->dev, "Atmel LCDC status register CMSTS timeout\n");
> + drm_warn(dev, "Atmel LCDC status register CMSTS timeout\n");
>
> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_XLCDC_SD);
> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> status & ATMEL_XLCDC_SD,
> 10, 1000))
> - dev_warn(dev->dev, "Atmel LCDC status register SDSTS timeout\n");
> + drm_warn(dev, "Atmel LCDC status register SDSTS timeout\n");
> }
>
> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_DISP);
> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> !(status & ATMEL_HLCDC_DISP),
> 10, 1000))
> - dev_warn(dev->dev, "Atmel LCDC status register DISPSTS timeout\n");
> + drm_warn(dev, "Atmel LCDC status register DISPSTS timeout\n");
>
> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_SYNC);
> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> !(status & ATMEL_HLCDC_SYNC),
> 10, 1000))
> - dev_warn(dev->dev, "Atmel LCDC status register LCDSTS timeout\n");
> + drm_warn(dev, "Atmel LCDC status register LCDSTS timeout\n");
>
> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_PIXEL_CLK);
> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> !(status & ATMEL_HLCDC_PIXEL_CLK),
> 10, 1000))
> - dev_warn(dev->dev, "Atmel LCDC status register CLKSTS timeout\n");
> + drm_warn(dev, "Atmel LCDC status register CLKSTS timeout\n");
>
> clk_disable_unprepare(crtc->dc->hlcdc->sys_clk);
> pinctrl_pm_select_sleep_state(dev->dev);
> @@ -269,32 +270,32 @@ static void atmel_hlcdc_crtc_atomic_enable(struct drm_crtc *c,
> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> status & ATMEL_HLCDC_PIXEL_CLK,
> 10, 1000))
> - dev_warn(dev->dev, "Atmel LCDC status register CLKSTS timeout\n");
> + drm_warn(dev, "Atmel LCDC status register CLKSTS timeout\n");
>
> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_HLCDC_SYNC);
> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> status & ATMEL_HLCDC_SYNC,
> 10, 1000))
> - dev_warn(dev->dev, "Atmel LCDC status register LCDSTS timeout\n");
> + drm_warn(dev, "Atmel LCDC status register LCDSTS timeout\n");
>
> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_HLCDC_DISP);
> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> status & ATMEL_HLCDC_DISP,
> 10, 1000))
> - dev_warn(dev->dev, "Atmel LCDC status register DISPSTS timeout\n");
> + drm_warn(dev, "Atmel LCDC status register DISPSTS timeout\n");
>
> if (crtc->dc->desc->is_xlcdc) {
> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_XLCDC_CM);
> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> status & ATMEL_XLCDC_CM,
> 10, 1000))
> - dev_warn(dev->dev, "Atmel LCDC status register CMSTS timeout\n");
> + drm_warn(dev, "Atmel LCDC status register CMSTS timeout\n");
>
> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_XLCDC_SD);
> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> !(status & ATMEL_XLCDC_SD),
> 10, 1000))
> - dev_warn(dev->dev, "Atmel LCDC status register SDSTS timeout\n");
> + drm_warn(dev, "Atmel LCDC status register SDSTS timeout\n");
> }
>
> pm_runtime_put_sync(dev->dev);
> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
> index fa8ad94e431a..acb017a2486b 100644
> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
> @@ -724,19 +724,19 @@ static int atmel_hlcdc_dc_modeset_init(struct drm_device *dev)
>
> ret = atmel_hlcdc_create_outputs(dev);
> if (ret) {
> - dev_err(dev->dev, "failed to create HLCDC outputs: %d\n", ret);
> + drm_err(dev, "failed to create HLCDC outputs: %d\n", ret);
> return ret;
> }
>
> ret = atmel_hlcdc_create_planes(dev);
> if (ret) {
> - dev_err(dev->dev, "failed to create planes: %d\n", ret);
> + drm_err(dev, "failed to create planes: %d\n", ret);
> return ret;
> }
>
> ret = atmel_hlcdc_crtc_create(dev);
> if (ret) {
> - dev_err(dev->dev, "failed to create crtc\n");
> + drm_err(dev, "failed to create crtc\n");
> return ret;
> }
>
> @@ -778,7 +778,7 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
>
> ret = clk_prepare_enable(dc->hlcdc->periph_clk);
> if (ret) {
> - dev_err(dev->dev, "failed to enable periph_clk\n");
> + drm_err(dev, "failed to enable periph_clk\n");
> return ret;
> }
>
> @@ -786,13 +786,13 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
>
> ret = drm_vblank_init(dev, 1);
> if (ret < 0) {
> - dev_err(dev->dev, "failed to initialize vblank\n");
> + drm_err(dev, "failed to initialize vblank\n");
> goto err_periph_clk_disable;
> }
>
> ret = atmel_hlcdc_dc_modeset_init(dev);
> if (ret < 0) {
> - dev_err(dev->dev, "failed to initialize mode setting\n");
> + drm_err(dev, "failed to initialize mode setting\n");
> goto err_periph_clk_disable;
> }
>
> @@ -802,7 +802,7 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
> ret = atmel_hlcdc_dc_irq_install(dev, dc->hlcdc->irq);
> pm_runtime_put_sync(dev->dev);
> if (ret < 0) {
> - dev_err(dev->dev, "failed to install IRQ handler\n");
> + drm_err(dev, "failed to install IRQ handler\n");
> goto err_periph_clk_disable;
> }
>
> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
> index 50fee6a93964..0b8a86afb096 100644
> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
> @@ -15,6 +15,7 @@
> #include <drm/drm_bridge.h>
> #include <drm/drm_encoder.h>
> #include <drm/drm_of.h>
> +#include <drm/drm_print.h>
> #include <drm/drm_simple_kms_helper.h>
>
> #include "atmel_hlcdc_dc.h"
> @@ -92,7 +93,7 @@ static int atmel_hlcdc_attach_endpoint(struct drm_device *dev, int endpoint)
> output->bus_fmt = atmel_hlcdc_of_bus_fmt(ep);
> of_node_put(ep);
> if (output->bus_fmt < 0) {
> - dev_err(dev->dev, "endpoint %d: invalid bus width\n", endpoint);
> + drm_err(dev, "endpoint %d: invalid bus width\n", endpoint);
> return -EINVAL;
> }
>
> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> index 4a7ba0918eca..817284509b57 100644
> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> @@ -1034,7 +1034,7 @@ static void atmel_hlcdc_irq_dbg(struct atmel_hlcdc_plane *plane,
> if (isr &
> (ATMEL_HLCDC_LAYER_OVR_IRQ(0) | ATMEL_HLCDC_LAYER_OVR_IRQ(1) |
> ATMEL_HLCDC_LAYER_OVR_IRQ(2)))
> - dev_dbg(plane->base.dev->dev, "overrun on plane %s\n",
> + drm_dbg(plane->base.dev, "overrun on plane %s\n",
> desc->name);
> }
>
> @@ -1051,7 +1051,7 @@ static void atmel_xlcdc_irq_dbg(struct atmel_hlcdc_plane *plane,
> if (isr &
> (ATMEL_XLCDC_LAYER_OVR_IRQ(0) | ATMEL_XLCDC_LAYER_OVR_IRQ(1) |
> ATMEL_XLCDC_LAYER_OVR_IRQ(2)))
> - dev_dbg(plane->base.dev->dev, "overrun on plane %s\n",
> + drm_dbg(plane->base.dev, "overrun on plane %s\n",
> desc->name);
> }
>
> @@ -1140,7 +1140,7 @@ static void atmel_hlcdc_plane_reset(struct drm_plane *p)
> if (state) {
> if (atmel_hlcdc_plane_alloc_dscrs(p, state)) {
> kfree(state);
> - dev_err(p->dev->dev,
> + drm_err(p->dev,
> "Failed to allocate initial plane state\n");
> return;
> }
> --
> 2.43.0
>
--
Thanks and Regards,
Manikandan M.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v4] drm: atmel-hlcdc: replace dev_* print functions with drm_* variants
2025-08-18 9:16 ` Manikandan.M
@ 2025-09-13 22:19 ` Eslam Khafagy
2025-09-16 6:50 ` Manikandan.M
2025-10-08 10:27 ` Dharma.B
1 sibling, 1 reply; 6+ messages in thread
From: Eslam Khafagy @ 2025-09-13 22:19 UTC (permalink / raw)
To: Manikandan.M, Dharma Balasubiramani, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, David Airlie, Simona Vetter,
Nicolas Ferre, Alexandre Belloni, Claudiu Beznea, dri-devel,
linux-arm-kernel, open list
Cc: Eslam Khafagy
Hello Manikandan,
This patch has been reviewed for a while now. but i didn't get an
update if it's been pulled yet or not .
kindly can you kindly update if it's been pulled or not ?
thanks in advance.
On Mon, Aug 18, 2025 at 12:16 PM <Manikandan.M@microchip.com> wrote:
>
> On 14/08/25 4:09 am, Eslam Khafagy wrote:
> > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> >
> > Update the Atmel HLCDC code to use DRM print macros drm_*() instead of
> > dev_warn() and dev_err(). This change ensures consistency with DRM
> > subsystem logging conventions [1].
> >
> > [1]
> > Link: https://docs.kernel.org/gpu/todo.html#convert-logging-to-drm-functions-with-drm-device-parameter
> >
> > Signed-off-by: Eslam Khafagy <eslam.medhat1993@gmail.com>
>
> Reviewed-by: Manikandan Muralidharan <manikandan.m@microchip.com>
>
> > ---
> > .../gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 21 ++++++++++---------
> > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 14 ++++++-------
> > .../gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 3 ++-
> > .../gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 6 +++---
> > 4 files changed, 23 insertions(+), 21 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
> > index 0f7ffb3ced20..e0efc7309b1b 100644
> > --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
> > +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
> > @@ -20,6 +20,7 @@
> > #include <drm/drm_atomic_helper.h>
> > #include <drm/drm_crtc.h>
> > #include <drm/drm_modeset_helper_vtables.h>
> > +#include <drm/drm_print.h>
> > #include <drm/drm_probe_helper.h>
> > #include <drm/drm_vblank.h>
> >
> > @@ -215,32 +216,32 @@ static void atmel_hlcdc_crtc_atomic_disable(struct drm_crtc *c,
> > if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> > !(status & ATMEL_XLCDC_CM),
> > 10, 1000))
> > - dev_warn(dev->dev, "Atmel LCDC status register CMSTS timeout\n");
> > + drm_warn(dev, "Atmel LCDC status register CMSTS timeout\n");
> >
> > regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_XLCDC_SD);
> > if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> > status & ATMEL_XLCDC_SD,
> > 10, 1000))
> > - dev_warn(dev->dev, "Atmel LCDC status register SDSTS timeout\n");
> > + drm_warn(dev, "Atmel LCDC status register SDSTS timeout\n");
> > }
> >
> > regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_DISP);
> > if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> > !(status & ATMEL_HLCDC_DISP),
> > 10, 1000))
> > - dev_warn(dev->dev, "Atmel LCDC status register DISPSTS timeout\n");
> > + drm_warn(dev, "Atmel LCDC status register DISPSTS timeout\n");
> >
> > regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_SYNC);
> > if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> > !(status & ATMEL_HLCDC_SYNC),
> > 10, 1000))
> > - dev_warn(dev->dev, "Atmel LCDC status register LCDSTS timeout\n");
> > + drm_warn(dev, "Atmel LCDC status register LCDSTS timeout\n");
> >
> > regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_PIXEL_CLK);
> > if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> > !(status & ATMEL_HLCDC_PIXEL_CLK),
> > 10, 1000))
> > - dev_warn(dev->dev, "Atmel LCDC status register CLKSTS timeout\n");
> > + drm_warn(dev, "Atmel LCDC status register CLKSTS timeout\n");
> >
> > clk_disable_unprepare(crtc->dc->hlcdc->sys_clk);
> > pinctrl_pm_select_sleep_state(dev->dev);
> > @@ -269,32 +270,32 @@ static void atmel_hlcdc_crtc_atomic_enable(struct drm_crtc *c,
> > if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> > status & ATMEL_HLCDC_PIXEL_CLK,
> > 10, 1000))
> > - dev_warn(dev->dev, "Atmel LCDC status register CLKSTS timeout\n");
> > + drm_warn(dev, "Atmel LCDC status register CLKSTS timeout\n");
> >
> > regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_HLCDC_SYNC);
> > if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> > status & ATMEL_HLCDC_SYNC,
> > 10, 1000))
> > - dev_warn(dev->dev, "Atmel LCDC status register LCDSTS timeout\n");
> > + drm_warn(dev, "Atmel LCDC status register LCDSTS timeout\n");
> >
> > regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_HLCDC_DISP);
> > if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> > status & ATMEL_HLCDC_DISP,
> > 10, 1000))
> > - dev_warn(dev->dev, "Atmel LCDC status register DISPSTS timeout\n");
> > + drm_warn(dev, "Atmel LCDC status register DISPSTS timeout\n");
> >
> > if (crtc->dc->desc->is_xlcdc) {
> > regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_XLCDC_CM);
> > if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> > status & ATMEL_XLCDC_CM,
> > 10, 1000))
> > - dev_warn(dev->dev, "Atmel LCDC status register CMSTS timeout\n");
> > + drm_warn(dev, "Atmel LCDC status register CMSTS timeout\n");
> >
> > regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_XLCDC_SD);
> > if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
> > !(status & ATMEL_XLCDC_SD),
> > 10, 1000))
> > - dev_warn(dev->dev, "Atmel LCDC status register SDSTS timeout\n");
> > + drm_warn(dev, "Atmel LCDC status register SDSTS timeout\n");
> > }
> >
> > pm_runtime_put_sync(dev->dev);
> > diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
> > index fa8ad94e431a..acb017a2486b 100644
> > --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
> > +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
> > @@ -724,19 +724,19 @@ static int atmel_hlcdc_dc_modeset_init(struct drm_device *dev)
> >
> > ret = atmel_hlcdc_create_outputs(dev);
> > if (ret) {
> > - dev_err(dev->dev, "failed to create HLCDC outputs: %d\n", ret);
> > + drm_err(dev, "failed to create HLCDC outputs: %d\n", ret);
> > return ret;
> > }
> >
> > ret = atmel_hlcdc_create_planes(dev);
> > if (ret) {
> > - dev_err(dev->dev, "failed to create planes: %d\n", ret);
> > + drm_err(dev, "failed to create planes: %d\n", ret);
> > return ret;
> > }
> >
> > ret = atmel_hlcdc_crtc_create(dev);
> > if (ret) {
> > - dev_err(dev->dev, "failed to create crtc\n");
> > + drm_err(dev, "failed to create crtc\n");
> > return ret;
> > }
> >
> > @@ -778,7 +778,7 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
> >
> > ret = clk_prepare_enable(dc->hlcdc->periph_clk);
> > if (ret) {
> > - dev_err(dev->dev, "failed to enable periph_clk\n");
> > + drm_err(dev, "failed to enable periph_clk\n");
> > return ret;
> > }
> >
> > @@ -786,13 +786,13 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
> >
> > ret = drm_vblank_init(dev, 1);
> > if (ret < 0) {
> > - dev_err(dev->dev, "failed to initialize vblank\n");
> > + drm_err(dev, "failed to initialize vblank\n");
> > goto err_periph_clk_disable;
> > }
> >
> > ret = atmel_hlcdc_dc_modeset_init(dev);
> > if (ret < 0) {
> > - dev_err(dev->dev, "failed to initialize mode setting\n");
> > + drm_err(dev, "failed to initialize mode setting\n");
> > goto err_periph_clk_disable;
> > }
> >
> > @@ -802,7 +802,7 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
> > ret = atmel_hlcdc_dc_irq_install(dev, dc->hlcdc->irq);
> > pm_runtime_put_sync(dev->dev);
> > if (ret < 0) {
> > - dev_err(dev->dev, "failed to install IRQ handler\n");
> > + drm_err(dev, "failed to install IRQ handler\n");
> > goto err_periph_clk_disable;
> > }
> >
> > diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
> > index 50fee6a93964..0b8a86afb096 100644
> > --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
> > +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
> > @@ -15,6 +15,7 @@
> > #include <drm/drm_bridge.h>
> > #include <drm/drm_encoder.h>
> > #include <drm/drm_of.h>
> > +#include <drm/drm_print.h>
> > #include <drm/drm_simple_kms_helper.h>
> >
> > #include "atmel_hlcdc_dc.h"
> > @@ -92,7 +93,7 @@ static int atmel_hlcdc_attach_endpoint(struct drm_device *dev, int endpoint)
> > output->bus_fmt = atmel_hlcdc_of_bus_fmt(ep);
> > of_node_put(ep);
> > if (output->bus_fmt < 0) {
> > - dev_err(dev->dev, "endpoint %d: invalid bus width\n", endpoint);
> > + drm_err(dev, "endpoint %d: invalid bus width\n", endpoint);
> > return -EINVAL;
> > }
> >
> > diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> > index 4a7ba0918eca..817284509b57 100644
> > --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> > +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
> > @@ -1034,7 +1034,7 @@ static void atmel_hlcdc_irq_dbg(struct atmel_hlcdc_plane *plane,
> > if (isr &
> > (ATMEL_HLCDC_LAYER_OVR_IRQ(0) | ATMEL_HLCDC_LAYER_OVR_IRQ(1) |
> > ATMEL_HLCDC_LAYER_OVR_IRQ(2)))
> > - dev_dbg(plane->base.dev->dev, "overrun on plane %s\n",
> > + drm_dbg(plane->base.dev, "overrun on plane %s\n",
> > desc->name);
> > }
> >
> > @@ -1051,7 +1051,7 @@ static void atmel_xlcdc_irq_dbg(struct atmel_hlcdc_plane *plane,
> > if (isr &
> > (ATMEL_XLCDC_LAYER_OVR_IRQ(0) | ATMEL_XLCDC_LAYER_OVR_IRQ(1) |
> > ATMEL_XLCDC_LAYER_OVR_IRQ(2)))
> > - dev_dbg(plane->base.dev->dev, "overrun on plane %s\n",
> > + drm_dbg(plane->base.dev, "overrun on plane %s\n",
> > desc->name);
> > }
> >
> > @@ -1140,7 +1140,7 @@ static void atmel_hlcdc_plane_reset(struct drm_plane *p)
> > if (state) {
> > if (atmel_hlcdc_plane_alloc_dscrs(p, state)) {
> > kfree(state);
> > - dev_err(p->dev->dev,
> > + drm_err(p->dev,
> > "Failed to allocate initial plane state\n");
> > return;
> > }
> > --
> > 2.43.0
> >
>
> --
> Thanks and Regards,
> Manikandan M.
>
--
Eslam Medhat Khafagy
https://www.linkedin.com/in/eslam-khafagy-a8a68159/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v4] drm: atmel-hlcdc: replace dev_* print functions with drm_* variants
2025-09-13 22:19 ` Eslam Khafagy
@ 2025-09-16 6:50 ` Manikandan.M
2025-09-16 20:25 ` Eslam Khafagy
0 siblings, 1 reply; 6+ messages in thread
From: Manikandan.M @ 2025-09-16 6:50 UTC (permalink / raw)
To: eslam.medhat1993, Dharma.B, maarten.lankhorst, mripard,
tzimmermann, airlied, simona, Nicolas.Ferre, alexandre.belloni,
claudiu.beznea, dri-devel, linux-arm-kernel, linux-kernel
Cc: Manikandan.M
Hi Eslam Khafagy,
I am still awaiting approval for my commit access request to the
drm-misc repository. I will be able to apply your changes as soon as the
access is granted.Thank you for your patience.
On 14/09/25 3:49 am, Eslam Khafagy wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> Hello Manikandan,
> This patch has been reviewed for a while now. but i didn't get an
> update if it's been pulled yet or not .
>
> kindly can you kindly update if it's been pulled or not ?
> thanks in advance.
>
>
> On Mon, Aug 18, 2025 at 12:16 PM <Manikandan.M@microchip.com> wrote:
>>
>> On 14/08/25 4:09 am, Eslam Khafagy wrote:
>>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>>>
>>> Update the Atmel HLCDC code to use DRM print macros drm_*() instead of
>>> dev_warn() and dev_err(). This change ensures consistency with DRM
>>> subsystem logging conventions [1].
>>>
>>> [1]
>>> Link: https://docs.kernel.org/gpu/todo.html#convert-logging-to-drm-functions-with-drm-device-parameter
>>>
>>> Signed-off-by: Eslam Khafagy <eslam.medhat1993@gmail.com>
>>
>> Reviewed-by: Manikandan Muralidharan <manikandan.m@microchip.com>
>>
>>> ---
>>> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 21 ++++++++++---------
>>> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 14 ++++++-------
>>> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 3 ++-
>>> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 6 +++---
>>> 4 files changed, 23 insertions(+), 21 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
>>> index 0f7ffb3ced20..e0efc7309b1b 100644
>>> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
>>> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
>>> @@ -20,6 +20,7 @@
>>> #include <drm/drm_atomic_helper.h>
>>> #include <drm/drm_crtc.h>
>>> #include <drm/drm_modeset_helper_vtables.h>
>>> +#include <drm/drm_print.h>
>>> #include <drm/drm_probe_helper.h>
>>> #include <drm/drm_vblank.h>
>>>
>>> @@ -215,32 +216,32 @@ static void atmel_hlcdc_crtc_atomic_disable(struct drm_crtc *c,
>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>> !(status & ATMEL_XLCDC_CM),
>>> 10, 1000))
>>> - dev_warn(dev->dev, "Atmel LCDC status register CMSTS timeout\n");
>>> + drm_warn(dev, "Atmel LCDC status register CMSTS timeout\n");
>>>
>>> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_XLCDC_SD);
>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>> status & ATMEL_XLCDC_SD,
>>> 10, 1000))
>>> - dev_warn(dev->dev, "Atmel LCDC status register SDSTS timeout\n");
>>> + drm_warn(dev, "Atmel LCDC status register SDSTS timeout\n");
>>> }
>>>
>>> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_DISP);
>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>> !(status & ATMEL_HLCDC_DISP),
>>> 10, 1000))
>>> - dev_warn(dev->dev, "Atmel LCDC status register DISPSTS timeout\n");
>>> + drm_warn(dev, "Atmel LCDC status register DISPSTS timeout\n");
>>>
>>> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_SYNC);
>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>> !(status & ATMEL_HLCDC_SYNC),
>>> 10, 1000))
>>> - dev_warn(dev->dev, "Atmel LCDC status register LCDSTS timeout\n");
>>> + drm_warn(dev, "Atmel LCDC status register LCDSTS timeout\n");
>>>
>>> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_PIXEL_CLK);
>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>> !(status & ATMEL_HLCDC_PIXEL_CLK),
>>> 10, 1000))
>>> - dev_warn(dev->dev, "Atmel LCDC status register CLKSTS timeout\n");
>>> + drm_warn(dev, "Atmel LCDC status register CLKSTS timeout\n");
>>>
>>> clk_disable_unprepare(crtc->dc->hlcdc->sys_clk);
>>> pinctrl_pm_select_sleep_state(dev->dev);
>>> @@ -269,32 +270,32 @@ static void atmel_hlcdc_crtc_atomic_enable(struct drm_crtc *c,
>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>> status & ATMEL_HLCDC_PIXEL_CLK,
>>> 10, 1000))
>>> - dev_warn(dev->dev, "Atmel LCDC status register CLKSTS timeout\n");
>>> + drm_warn(dev, "Atmel LCDC status register CLKSTS timeout\n");
>>>
>>> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_HLCDC_SYNC);
>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>> status & ATMEL_HLCDC_SYNC,
>>> 10, 1000))
>>> - dev_warn(dev->dev, "Atmel LCDC status register LCDSTS timeout\n");
>>> + drm_warn(dev, "Atmel LCDC status register LCDSTS timeout\n");
>>>
>>> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_HLCDC_DISP);
>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>> status & ATMEL_HLCDC_DISP,
>>> 10, 1000))
>>> - dev_warn(dev->dev, "Atmel LCDC status register DISPSTS timeout\n");
>>> + drm_warn(dev, "Atmel LCDC status register DISPSTS timeout\n");
>>>
>>> if (crtc->dc->desc->is_xlcdc) {
>>> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_XLCDC_CM);
>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>> status & ATMEL_XLCDC_CM,
>>> 10, 1000))
>>> - dev_warn(dev->dev, "Atmel LCDC status register CMSTS timeout\n");
>>> + drm_warn(dev, "Atmel LCDC status register CMSTS timeout\n");
>>>
>>> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_XLCDC_SD);
>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>> !(status & ATMEL_XLCDC_SD),
>>> 10, 1000))
>>> - dev_warn(dev->dev, "Atmel LCDC status register SDSTS timeout\n");
>>> + drm_warn(dev, "Atmel LCDC status register SDSTS timeout\n");
>>> }
>>>
>>> pm_runtime_put_sync(dev->dev);
>>> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
>>> index fa8ad94e431a..acb017a2486b 100644
>>> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
>>> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
>>> @@ -724,19 +724,19 @@ static int atmel_hlcdc_dc_modeset_init(struct drm_device *dev)
>>>
>>> ret = atmel_hlcdc_create_outputs(dev);
>>> if (ret) {
>>> - dev_err(dev->dev, "failed to create HLCDC outputs: %d\n", ret);
>>> + drm_err(dev, "failed to create HLCDC outputs: %d\n", ret);
>>> return ret;
>>> }
>>>
>>> ret = atmel_hlcdc_create_planes(dev);
>>> if (ret) {
>>> - dev_err(dev->dev, "failed to create planes: %d\n", ret);
>>> + drm_err(dev, "failed to create planes: %d\n", ret);
>>> return ret;
>>> }
>>>
>>> ret = atmel_hlcdc_crtc_create(dev);
>>> if (ret) {
>>> - dev_err(dev->dev, "failed to create crtc\n");
>>> + drm_err(dev, "failed to create crtc\n");
>>> return ret;
>>> }
>>>
>>> @@ -778,7 +778,7 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
>>>
>>> ret = clk_prepare_enable(dc->hlcdc->periph_clk);
>>> if (ret) {
>>> - dev_err(dev->dev, "failed to enable periph_clk\n");
>>> + drm_err(dev, "failed to enable periph_clk\n");
>>> return ret;
>>> }
>>>
>>> @@ -786,13 +786,13 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
>>>
>>> ret = drm_vblank_init(dev, 1);
>>> if (ret < 0) {
>>> - dev_err(dev->dev, "failed to initialize vblank\n");
>>> + drm_err(dev, "failed to initialize vblank\n");
>>> goto err_periph_clk_disable;
>>> }
>>>
>>> ret = atmel_hlcdc_dc_modeset_init(dev);
>>> if (ret < 0) {
>>> - dev_err(dev->dev, "failed to initialize mode setting\n");
>>> + drm_err(dev, "failed to initialize mode setting\n");
>>> goto err_periph_clk_disable;
>>> }
>>>
>>> @@ -802,7 +802,7 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
>>> ret = atmel_hlcdc_dc_irq_install(dev, dc->hlcdc->irq);
>>> pm_runtime_put_sync(dev->dev);
>>> if (ret < 0) {
>>> - dev_err(dev->dev, "failed to install IRQ handler\n");
>>> + drm_err(dev, "failed to install IRQ handler\n");
>>> goto err_periph_clk_disable;
>>> }
>>>
>>> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
>>> index 50fee6a93964..0b8a86afb096 100644
>>> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
>>> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
>>> @@ -15,6 +15,7 @@
>>> #include <drm/drm_bridge.h>
>>> #include <drm/drm_encoder.h>
>>> #include <drm/drm_of.h>
>>> +#include <drm/drm_print.h>
>>> #include <drm/drm_simple_kms_helper.h>
>>>
>>> #include "atmel_hlcdc_dc.h"
>>> @@ -92,7 +93,7 @@ static int atmel_hlcdc_attach_endpoint(struct drm_device *dev, int endpoint)
>>> output->bus_fmt = atmel_hlcdc_of_bus_fmt(ep);
>>> of_node_put(ep);
>>> if (output->bus_fmt < 0) {
>>> - dev_err(dev->dev, "endpoint %d: invalid bus width\n", endpoint);
>>> + drm_err(dev, "endpoint %d: invalid bus width\n", endpoint);
>>> return -EINVAL;
>>> }
>>>
>>> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
>>> index 4a7ba0918eca..817284509b57 100644
>>> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
>>> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
>>> @@ -1034,7 +1034,7 @@ static void atmel_hlcdc_irq_dbg(struct atmel_hlcdc_plane *plane,
>>> if (isr &
>>> (ATMEL_HLCDC_LAYER_OVR_IRQ(0) | ATMEL_HLCDC_LAYER_OVR_IRQ(1) |
>>> ATMEL_HLCDC_LAYER_OVR_IRQ(2)))
>>> - dev_dbg(plane->base.dev->dev, "overrun on plane %s\n",
>>> + drm_dbg(plane->base.dev, "overrun on plane %s\n",
>>> desc->name);
>>> }
>>>
>>> @@ -1051,7 +1051,7 @@ static void atmel_xlcdc_irq_dbg(struct atmel_hlcdc_plane *plane,
>>> if (isr &
>>> (ATMEL_XLCDC_LAYER_OVR_IRQ(0) | ATMEL_XLCDC_LAYER_OVR_IRQ(1) |
>>> ATMEL_XLCDC_LAYER_OVR_IRQ(2)))
>>> - dev_dbg(plane->base.dev->dev, "overrun on plane %s\n",
>>> + drm_dbg(plane->base.dev, "overrun on plane %s\n",
>>> desc->name);
>>> }
>>>
>>> @@ -1140,7 +1140,7 @@ static void atmel_hlcdc_plane_reset(struct drm_plane *p)
>>> if (state) {
>>> if (atmel_hlcdc_plane_alloc_dscrs(p, state)) {
>>> kfree(state);
>>> - dev_err(p->dev->dev,
>>> + drm_err(p->dev,
>>> "Failed to allocate initial plane state\n");
>>> return;
>>> }
>>> --
>>> 2.43.0
>>>
>>
>> --
>> Thanks and Regards,
>> Manikandan M.
>>
>
>
> --
>
> Eslam Medhat Khafagy
>
> https://www.linkedin.com/in/eslam-khafagy-a8a68159/
--
Thanks and Regards,
Manikandan M.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v4] drm: atmel-hlcdc: replace dev_* print functions with drm_* variants
2025-09-16 6:50 ` Manikandan.M
@ 2025-09-16 20:25 ` Eslam Khafagy
0 siblings, 0 replies; 6+ messages in thread
From: Eslam Khafagy @ 2025-09-16 20:25 UTC (permalink / raw)
To: Manikandan.M, Dharma.B, maarten.lankhorst, mripard, tzimmermann,
airlied, simona, Nicolas.Ferre, alexandre.belloni, claudiu.beznea,
dri-devel, linux-arm-kernel, linux-kernel
Hi Manikandan,
Understood. thank you for the follow-up.
On 9/16/25 09:50, Manikandan.M@microchip.com wrote:
> Hi Eslam Khafagy,
>
> I am still awaiting approval for my commit access request to the
> drm-misc repository. I will be able to apply your changes as soon as the
> access is granted.Thank you for your patience.
>
> On 14/09/25 3:49 am, Eslam Khafagy wrote:
>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>>
>> Hello Manikandan,
>> This patch has been reviewed for a while now. but i didn't get an
>> update if it's been pulled yet or not .
>>
>> kindly can you kindly update if it's been pulled or not ?
>> thanks in advance.
>>
>>
>> On Mon, Aug 18, 2025 at 12:16 PM <Manikandan.M@microchip.com> wrote:
>>> On 14/08/25 4:09 am, Eslam Khafagy wrote:
>>>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>>>>
>>>> Update the Atmel HLCDC code to use DRM print macros drm_*() instead of
>>>> dev_warn() and dev_err(). This change ensures consistency with DRM
>>>> subsystem logging conventions [1].
>>>>
>>>> [1]
>>>> Link: https://docs.kernel.org/gpu/todo.html#convert-logging-to-drm-functions-with-drm-device-parameter
>>>>
>>>> Signed-off-by: Eslam Khafagy <eslam.medhat1993@gmail.com>
>>> Reviewed-by: Manikandan Muralidharan <manikandan.m@microchip.com>
>>>
>>>> ---
>>>> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 21 ++++++++++---------
>>>> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 14 ++++++-------
>>>> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 3 ++-
>>>> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 6 +++---
>>>> 4 files changed, 23 insertions(+), 21 deletions(-)
>>>>
>>>> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
>>>> index 0f7ffb3ced20..e0efc7309b1b 100644
>>>> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
>>>> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
>>>> @@ -20,6 +20,7 @@
>>>> #include <drm/drm_atomic_helper.h>
>>>> #include <drm/drm_crtc.h>
>>>> #include <drm/drm_modeset_helper_vtables.h>
>>>> +#include <drm/drm_print.h>
>>>> #include <drm/drm_probe_helper.h>
>>>> #include <drm/drm_vblank.h>
>>>>
>>>> @@ -215,32 +216,32 @@ static void atmel_hlcdc_crtc_atomic_disable(struct drm_crtc *c,
>>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>>> !(status & ATMEL_XLCDC_CM),
>>>> 10, 1000))
>>>> - dev_warn(dev->dev, "Atmel LCDC status register CMSTS timeout\n");
>>>> + drm_warn(dev, "Atmel LCDC status register CMSTS timeout\n");
>>>>
>>>> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_XLCDC_SD);
>>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>>> status & ATMEL_XLCDC_SD,
>>>> 10, 1000))
>>>> - dev_warn(dev->dev, "Atmel LCDC status register SDSTS timeout\n");
>>>> + drm_warn(dev, "Atmel LCDC status register SDSTS timeout\n");
>>>> }
>>>>
>>>> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_DISP);
>>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>>> !(status & ATMEL_HLCDC_DISP),
>>>> 10, 1000))
>>>> - dev_warn(dev->dev, "Atmel LCDC status register DISPSTS timeout\n");
>>>> + drm_warn(dev, "Atmel LCDC status register DISPSTS timeout\n");
>>>>
>>>> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_SYNC);
>>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>>> !(status & ATMEL_HLCDC_SYNC),
>>>> 10, 1000))
>>>> - dev_warn(dev->dev, "Atmel LCDC status register LCDSTS timeout\n");
>>>> + drm_warn(dev, "Atmel LCDC status register LCDSTS timeout\n");
>>>>
>>>> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_PIXEL_CLK);
>>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>>> !(status & ATMEL_HLCDC_PIXEL_CLK),
>>>> 10, 1000))
>>>> - dev_warn(dev->dev, "Atmel LCDC status register CLKSTS timeout\n");
>>>> + drm_warn(dev, "Atmel LCDC status register CLKSTS timeout\n");
>>>>
>>>> clk_disable_unprepare(crtc->dc->hlcdc->sys_clk);
>>>> pinctrl_pm_select_sleep_state(dev->dev);
>>>> @@ -269,32 +270,32 @@ static void atmel_hlcdc_crtc_atomic_enable(struct drm_crtc *c,
>>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>>> status & ATMEL_HLCDC_PIXEL_CLK,
>>>> 10, 1000))
>>>> - dev_warn(dev->dev, "Atmel LCDC status register CLKSTS timeout\n");
>>>> + drm_warn(dev, "Atmel LCDC status register CLKSTS timeout\n");
>>>>
>>>> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_HLCDC_SYNC);
>>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>>> status & ATMEL_HLCDC_SYNC,
>>>> 10, 1000))
>>>> - dev_warn(dev->dev, "Atmel LCDC status register LCDSTS timeout\n");
>>>> + drm_warn(dev, "Atmel LCDC status register LCDSTS timeout\n");
>>>>
>>>> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_HLCDC_DISP);
>>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>>> status & ATMEL_HLCDC_DISP,
>>>> 10, 1000))
>>>> - dev_warn(dev->dev, "Atmel LCDC status register DISPSTS timeout\n");
>>>> + drm_warn(dev, "Atmel LCDC status register DISPSTS timeout\n");
>>>>
>>>> if (crtc->dc->desc->is_xlcdc) {
>>>> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_XLCDC_CM);
>>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>>> status & ATMEL_XLCDC_CM,
>>>> 10, 1000))
>>>> - dev_warn(dev->dev, "Atmel LCDC status register CMSTS timeout\n");
>>>> + drm_warn(dev, "Atmel LCDC status register CMSTS timeout\n");
>>>>
>>>> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_XLCDC_SD);
>>>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>>>> !(status & ATMEL_XLCDC_SD),
>>>> 10, 1000))
>>>> - dev_warn(dev->dev, "Atmel LCDC status register SDSTS timeout\n");
>>>> + drm_warn(dev, "Atmel LCDC status register SDSTS timeout\n");
>>>> }
>>>>
>>>> pm_runtime_put_sync(dev->dev);
>>>> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
>>>> index fa8ad94e431a..acb017a2486b 100644
>>>> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
>>>> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
>>>> @@ -724,19 +724,19 @@ static int atmel_hlcdc_dc_modeset_init(struct drm_device *dev)
>>>>
>>>> ret = atmel_hlcdc_create_outputs(dev);
>>>> if (ret) {
>>>> - dev_err(dev->dev, "failed to create HLCDC outputs: %d\n", ret);
>>>> + drm_err(dev, "failed to create HLCDC outputs: %d\n", ret);
>>>> return ret;
>>>> }
>>>>
>>>> ret = atmel_hlcdc_create_planes(dev);
>>>> if (ret) {
>>>> - dev_err(dev->dev, "failed to create planes: %d\n", ret);
>>>> + drm_err(dev, "failed to create planes: %d\n", ret);
>>>> return ret;
>>>> }
>>>>
>>>> ret = atmel_hlcdc_crtc_create(dev);
>>>> if (ret) {
>>>> - dev_err(dev->dev, "failed to create crtc\n");
>>>> + drm_err(dev, "failed to create crtc\n");
>>>> return ret;
>>>> }
>>>>
>>>> @@ -778,7 +778,7 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
>>>>
>>>> ret = clk_prepare_enable(dc->hlcdc->periph_clk);
>>>> if (ret) {
>>>> - dev_err(dev->dev, "failed to enable periph_clk\n");
>>>> + drm_err(dev, "failed to enable periph_clk\n");
>>>> return ret;
>>>> }
>>>>
>>>> @@ -786,13 +786,13 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
>>>>
>>>> ret = drm_vblank_init(dev, 1);
>>>> if (ret < 0) {
>>>> - dev_err(dev->dev, "failed to initialize vblank\n");
>>>> + drm_err(dev, "failed to initialize vblank\n");
>>>> goto err_periph_clk_disable;
>>>> }
>>>>
>>>> ret = atmel_hlcdc_dc_modeset_init(dev);
>>>> if (ret < 0) {
>>>> - dev_err(dev->dev, "failed to initialize mode setting\n");
>>>> + drm_err(dev, "failed to initialize mode setting\n");
>>>> goto err_periph_clk_disable;
>>>> }
>>>>
>>>> @@ -802,7 +802,7 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
>>>> ret = atmel_hlcdc_dc_irq_install(dev, dc->hlcdc->irq);
>>>> pm_runtime_put_sync(dev->dev);
>>>> if (ret < 0) {
>>>> - dev_err(dev->dev, "failed to install IRQ handler\n");
>>>> + drm_err(dev, "failed to install IRQ handler\n");
>>>> goto err_periph_clk_disable;
>>>> }
>>>>
>>>> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
>>>> index 50fee6a93964..0b8a86afb096 100644
>>>> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
>>>> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
>>>> @@ -15,6 +15,7 @@
>>>> #include <drm/drm_bridge.h>
>>>> #include <drm/drm_encoder.h>
>>>> #include <drm/drm_of.h>
>>>> +#include <drm/drm_print.h>
>>>> #include <drm/drm_simple_kms_helper.h>
>>>>
>>>> #include "atmel_hlcdc_dc.h"
>>>> @@ -92,7 +93,7 @@ static int atmel_hlcdc_attach_endpoint(struct drm_device *dev, int endpoint)
>>>> output->bus_fmt = atmel_hlcdc_of_bus_fmt(ep);
>>>> of_node_put(ep);
>>>> if (output->bus_fmt < 0) {
>>>> - dev_err(dev->dev, "endpoint %d: invalid bus width\n", endpoint);
>>>> + drm_err(dev, "endpoint %d: invalid bus width\n", endpoint);
>>>> return -EINVAL;
>>>> }
>>>>
>>>> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
>>>> index 4a7ba0918eca..817284509b57 100644
>>>> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
>>>> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
>>>> @@ -1034,7 +1034,7 @@ static void atmel_hlcdc_irq_dbg(struct atmel_hlcdc_plane *plane,
>>>> if (isr &
>>>> (ATMEL_HLCDC_LAYER_OVR_IRQ(0) | ATMEL_HLCDC_LAYER_OVR_IRQ(1) |
>>>> ATMEL_HLCDC_LAYER_OVR_IRQ(2)))
>>>> - dev_dbg(plane->base.dev->dev, "overrun on plane %s\n",
>>>> + drm_dbg(plane->base.dev, "overrun on plane %s\n",
>>>> desc->name);
>>>> }
>>>>
>>>> @@ -1051,7 +1051,7 @@ static void atmel_xlcdc_irq_dbg(struct atmel_hlcdc_plane *plane,
>>>> if (isr &
>>>> (ATMEL_XLCDC_LAYER_OVR_IRQ(0) | ATMEL_XLCDC_LAYER_OVR_IRQ(1) |
>>>> ATMEL_XLCDC_LAYER_OVR_IRQ(2)))
>>>> - dev_dbg(plane->base.dev->dev, "overrun on plane %s\n",
>>>> + drm_dbg(plane->base.dev, "overrun on plane %s\n",
>>>> desc->name);
>>>> }
>>>>
>>>> @@ -1140,7 +1140,7 @@ static void atmel_hlcdc_plane_reset(struct drm_plane *p)
>>>> if (state) {
>>>> if (atmel_hlcdc_plane_alloc_dscrs(p, state)) {
>>>> kfree(state);
>>>> - dev_err(p->dev->dev,
>>>> + drm_err(p->dev,
>>>> "Failed to allocate initial plane state\n");
>>>> return;
>>>> }
>>>> --
>>>> 2.43.0
>>>>
>>> --
>>> Thanks and Regards,
>>> Manikandan M.
>>>
>>
>> --
>>
>> Eslam Medhat Khafagy
>>
>> https://www.linkedin.com/in/eslam-khafagy-a8a68159/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v4] drm: atmel-hlcdc: replace dev_* print functions with drm_* variants
2025-08-18 9:16 ` Manikandan.M
2025-09-13 22:19 ` Eslam Khafagy
@ 2025-10-08 10:27 ` Dharma.B
1 sibling, 0 replies; 6+ messages in thread
From: Dharma.B @ 2025-10-08 10:27 UTC (permalink / raw)
To: Manikandan.M, eslam.medhat1993, maarten.lankhorst, mripard,
tzimmermann, airlied, simona, Nicolas.Ferre, alexandre.belloni,
claudiu.beznea, dri-devel, linux-arm-kernel, linux-kernel
Cc: skhan
On 18/08/25 2:46 pm, Manikandan M - I67131 wrote:
> On 14/08/25 4:09 am, Eslam Khafagy wrote:
>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>>
>> Update the Atmel HLCDC code to use DRM print macros drm_*() instead of
>> dev_warn() and dev_err(). This change ensures consistency with DRM
>> subsystem logging conventions [1].
>>
>> [1]
>> Link: https://docs.kernel.org/gpu/todo.html#convert-logging-to-drm-functions-with-drm-device-parameter
>>
>> Signed-off-by: Eslam Khafagy <eslam.medhat1993@gmail.com>
>
> Reviewed-by: Manikandan Muralidharan <manikandan.m@microchip.com>
Applied to drm-misc-next, Thanks.
>
>> ---
>> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 21 ++++++++++---------
>> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 14 ++++++-------
>> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 3 ++-
>> .../gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 6 +++---
>> 4 files changed, 23 insertions(+), 21 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
>> index 0f7ffb3ced20..e0efc7309b1b 100644
>> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
>> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c
>> @@ -20,6 +20,7 @@
>> #include <drm/drm_atomic_helper.h>
>> #include <drm/drm_crtc.h>
>> #include <drm/drm_modeset_helper_vtables.h>
>> +#include <drm/drm_print.h>
>> #include <drm/drm_probe_helper.h>
>> #include <drm/drm_vblank.h>
>>
>> @@ -215,32 +216,32 @@ static void atmel_hlcdc_crtc_atomic_disable(struct drm_crtc *c,
>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>> !(status & ATMEL_XLCDC_CM),
>> 10, 1000))
>> - dev_warn(dev->dev, "Atmel LCDC status register CMSTS timeout\n");
>> + drm_warn(dev, "Atmel LCDC status register CMSTS timeout\n");
>>
>> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_XLCDC_SD);
>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>> status & ATMEL_XLCDC_SD,
>> 10, 1000))
>> - dev_warn(dev->dev, "Atmel LCDC status register SDSTS timeout\n");
>> + drm_warn(dev, "Atmel LCDC status register SDSTS timeout\n");
>> }
>>
>> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_DISP);
>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>> !(status & ATMEL_HLCDC_DISP),
>> 10, 1000))
>> - dev_warn(dev->dev, "Atmel LCDC status register DISPSTS timeout\n");
>> + drm_warn(dev, "Atmel LCDC status register DISPSTS timeout\n");
>>
>> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_SYNC);
>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>> !(status & ATMEL_HLCDC_SYNC),
>> 10, 1000))
>> - dev_warn(dev->dev, "Atmel LCDC status register LCDSTS timeout\n");
>> + drm_warn(dev, "Atmel LCDC status register LCDSTS timeout\n");
>>
>> regmap_write(regmap, ATMEL_HLCDC_DIS, ATMEL_HLCDC_PIXEL_CLK);
>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>> !(status & ATMEL_HLCDC_PIXEL_CLK),
>> 10, 1000))
>> - dev_warn(dev->dev, "Atmel LCDC status register CLKSTS timeout\n");
>> + drm_warn(dev, "Atmel LCDC status register CLKSTS timeout\n");
>>
>> clk_disable_unprepare(crtc->dc->hlcdc->sys_clk);
>> pinctrl_pm_select_sleep_state(dev->dev);
>> @@ -269,32 +270,32 @@ static void atmel_hlcdc_crtc_atomic_enable(struct drm_crtc *c,
>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>> status & ATMEL_HLCDC_PIXEL_CLK,
>> 10, 1000))
>> - dev_warn(dev->dev, "Atmel LCDC status register CLKSTS timeout\n");
>> + drm_warn(dev, "Atmel LCDC status register CLKSTS timeout\n");
>>
>> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_HLCDC_SYNC);
>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>> status & ATMEL_HLCDC_SYNC,
>> 10, 1000))
>> - dev_warn(dev->dev, "Atmel LCDC status register LCDSTS timeout\n");
>> + drm_warn(dev, "Atmel LCDC status register LCDSTS timeout\n");
>>
>> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_HLCDC_DISP);
>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>> status & ATMEL_HLCDC_DISP,
>> 10, 1000))
>> - dev_warn(dev->dev, "Atmel LCDC status register DISPSTS timeout\n");
>> + drm_warn(dev, "Atmel LCDC status register DISPSTS timeout\n");
>>
>> if (crtc->dc->desc->is_xlcdc) {
>> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_XLCDC_CM);
>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>> status & ATMEL_XLCDC_CM,
>> 10, 1000))
>> - dev_warn(dev->dev, "Atmel LCDC status register CMSTS timeout\n");
>> + drm_warn(dev, "Atmel LCDC status register CMSTS timeout\n");
>>
>> regmap_write(regmap, ATMEL_HLCDC_EN, ATMEL_XLCDC_SD);
>> if (regmap_read_poll_timeout(regmap, ATMEL_HLCDC_SR, status,
>> !(status & ATMEL_XLCDC_SD),
>> 10, 1000))
>> - dev_warn(dev->dev, "Atmel LCDC status register SDSTS timeout\n");
>> + drm_warn(dev, "Atmel LCDC status register SDSTS timeout\n");
>> }
>>
>> pm_runtime_put_sync(dev->dev);
>> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
>> index fa8ad94e431a..acb017a2486b 100644
>> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
>> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
>> @@ -724,19 +724,19 @@ static int atmel_hlcdc_dc_modeset_init(struct drm_device *dev)
>>
>> ret = atmel_hlcdc_create_outputs(dev);
>> if (ret) {
>> - dev_err(dev->dev, "failed to create HLCDC outputs: %d\n", ret);
>> + drm_err(dev, "failed to create HLCDC outputs: %d\n", ret);
>> return ret;
>> }
>>
>> ret = atmel_hlcdc_create_planes(dev);
>> if (ret) {
>> - dev_err(dev->dev, "failed to create planes: %d\n", ret);
>> + drm_err(dev, "failed to create planes: %d\n", ret);
>> return ret;
>> }
>>
>> ret = atmel_hlcdc_crtc_create(dev);
>> if (ret) {
>> - dev_err(dev->dev, "failed to create crtc\n");
>> + drm_err(dev, "failed to create crtc\n");
>> return ret;
>> }
>>
>> @@ -778,7 +778,7 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
>>
>> ret = clk_prepare_enable(dc->hlcdc->periph_clk);
>> if (ret) {
>> - dev_err(dev->dev, "failed to enable periph_clk\n");
>> + drm_err(dev, "failed to enable periph_clk\n");
>> return ret;
>> }
>>
>> @@ -786,13 +786,13 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
>>
>> ret = drm_vblank_init(dev, 1);
>> if (ret < 0) {
>> - dev_err(dev->dev, "failed to initialize vblank\n");
>> + drm_err(dev, "failed to initialize vblank\n");
>> goto err_periph_clk_disable;
>> }
>>
>> ret = atmel_hlcdc_dc_modeset_init(dev);
>> if (ret < 0) {
>> - dev_err(dev->dev, "failed to initialize mode setting\n");
>> + drm_err(dev, "failed to initialize mode setting\n");
>> goto err_periph_clk_disable;
>> }
>>
>> @@ -802,7 +802,7 @@ static int atmel_hlcdc_dc_load(struct drm_device *dev)
>> ret = atmel_hlcdc_dc_irq_install(dev, dc->hlcdc->irq);
>> pm_runtime_put_sync(dev->dev);
>> if (ret < 0) {
>> - dev_err(dev->dev, "failed to install IRQ handler\n");
>> + drm_err(dev, "failed to install IRQ handler\n");
>> goto err_periph_clk_disable;
>> }
>>
>> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
>> index 50fee6a93964..0b8a86afb096 100644
>> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
>> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c
>> @@ -15,6 +15,7 @@
>> #include <drm/drm_bridge.h>
>> #include <drm/drm_encoder.h>
>> #include <drm/drm_of.h>
>> +#include <drm/drm_print.h>
>> #include <drm/drm_simple_kms_helper.h>
>>
>> #include "atmel_hlcdc_dc.h"
>> @@ -92,7 +93,7 @@ static int atmel_hlcdc_attach_endpoint(struct drm_device *dev, int endpoint)
>> output->bus_fmt = atmel_hlcdc_of_bus_fmt(ep);
>> of_node_put(ep);
>> if (output->bus_fmt < 0) {
>> - dev_err(dev->dev, "endpoint %d: invalid bus width\n", endpoint);
>> + drm_err(dev, "endpoint %d: invalid bus width\n", endpoint);
>> return -EINVAL;
>> }
>>
>> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
>> index 4a7ba0918eca..817284509b57 100644
>> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
>> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
>> @@ -1034,7 +1034,7 @@ static void atmel_hlcdc_irq_dbg(struct atmel_hlcdc_plane *plane,
>> if (isr &
>> (ATMEL_HLCDC_LAYER_OVR_IRQ(0) | ATMEL_HLCDC_LAYER_OVR_IRQ(1) |
>> ATMEL_HLCDC_LAYER_OVR_IRQ(2)))
>> - dev_dbg(plane->base.dev->dev, "overrun on plane %s\n",
>> + drm_dbg(plane->base.dev, "overrun on plane %s\n",
>> desc->name);
>> }
>>
>> @@ -1051,7 +1051,7 @@ static void atmel_xlcdc_irq_dbg(struct atmel_hlcdc_plane *plane,
>> if (isr &
>> (ATMEL_XLCDC_LAYER_OVR_IRQ(0) | ATMEL_XLCDC_LAYER_OVR_IRQ(1) |
>> ATMEL_XLCDC_LAYER_OVR_IRQ(2)))
>> - dev_dbg(plane->base.dev->dev, "overrun on plane %s\n",
>> + drm_dbg(plane->base.dev, "overrun on plane %s\n",
>> desc->name);
>> }
>>
>> @@ -1140,7 +1140,7 @@ static void atmel_hlcdc_plane_reset(struct drm_plane *p)
>> if (state) {
>> if (atmel_hlcdc_plane_alloc_dscrs(p, state)) {
>> kfree(state);
>> - dev_err(p->dev->dev,
>> + drm_err(p->dev,
>> "Failed to allocate initial plane state\n");
>> return;
>> }
>> --
>> 2.43.0
>>
>
--
With Best Regards,
Dharma B.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2025-10-08 10:27 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-13 22:39 [PATCH v4] drm: atmel-hlcdc: replace dev_* print functions with drm_* variants Eslam Khafagy
2025-08-18 9:16 ` Manikandan.M
2025-09-13 22:19 ` Eslam Khafagy
2025-09-16 6:50 ` Manikandan.M
2025-09-16 20:25 ` Eslam Khafagy
2025-10-08 10:27 ` Dharma.B
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).