From: <Manikandan.M@microchip.com>
To: <eslam.medhat1993@gmail.com>, <Dharma.B@microchip.com>,
<maarten.lankhorst@linux.intel.com>, <mripard@kernel.org>,
<tzimmermann@suse.de>, <airlied@gmail.com>, <simona@ffwll.ch>,
<Nicolas.Ferre@microchip.com>, <alexandre.belloni@bootlin.com>,
<claudiu.beznea@tuxon.dev>, <dri-devel@lists.freedesktop.org>,
<linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>
Cc: <Manikandan.M@microchip.com>
Subject: Re: [PATCH v4] drm: atmel-hlcdc: replace dev_* print functions with drm_* variants
Date: Tue, 16 Sep 2025 06:50:58 +0000 [thread overview]
Message-ID: <4e7faa33-6e39-4b9c-b5df-bccaa9cbb1cc@microchip.com> (raw)
In-Reply-To: <CAFRctSHGG-7tx0o5-hi2pZbsw-9jGV1FZrFquOOnFAB=i+O8Gw@mail.gmail.com>
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.
next prev parent reply other threads:[~2025-09-16 6:51 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
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 [this message]
2025-09-16 20:25 ` Eslam Khafagy
2025-10-08 10:27 ` Dharma.B
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4e7faa33-6e39-4b9c-b5df-bccaa9cbb1cc@microchip.com \
--to=manikandan.m@microchip.com \
--cc=Dharma.B@microchip.com \
--cc=Nicolas.Ferre@microchip.com \
--cc=airlied@gmail.com \
--cc=alexandre.belloni@bootlin.com \
--cc=claudiu.beznea@tuxon.dev \
--cc=dri-devel@lists.freedesktop.org \
--cc=eslam.medhat1993@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mripard@kernel.org \
--cc=simona@ffwll.ch \
--cc=tzimmermann@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox