linux-kernel - Re: [PATCH v1 1/1] drm/panel: boe-th101mb31ig002 : Fix the way to get porch parameters

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAA8EJpp8LvzKdHR2y-f-qtfx-fLSOwxG_CXTCrtV6JWnzX_YmQ@mail.gmail.com>
Date: Mon, 15 Jul 2024 14:32:26 +0300
From: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
To: Zhaoxiong Lv <lvzhaoxiong@...qin.corp-partner.google.com>
Cc: neil.armstrong@...aro.org, robh@...nel.org, 
	krzysztof.kozlowski+dt@...aro.org, conor+dt@...nel.org, airlied@...il.com, 
	mripard@...nel.org, dianders@...gle.com, hsinyi@...gle.com, 
	awarnecke002@...mail.com, quic_jesszhan@...cinc.com, 
	dri-devel@...ts.freedesktop.org, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 1/1] drm/panel: boe-th101mb31ig002 : Fix the way to get
 porch parameters

On Mon, 15 Jul 2024 at 06:19, Zhaoxiong Lv
<lvzhaoxiong@...qin.corp-partner.google.com> wrote:
>
> The current driver can only obtain the porch parameters
> of boe-th101mb31ig002. Modify it to obtain the porch
> parameters of the panel currently being used.
>
> Fixes: 24179ff9a2e4524 ("drm/panel: boe-th101mb31ig002 : Make it compatible with other panel.")
>
> Signed-off-by: Zhaoxiong Lv <lvzhaoxiong@...qin.corp-partner.google.com>

No empty line between Fixed and Sign-off-by


> ---
>  drivers/gpu/drm/panel/panel-boe-th101mb31ig002-28a.c | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/panel/panel-boe-th101mb31ig002-28a.c b/drivers/gpu/drm/panel/panel-boe-th101mb31ig002-28a.c
> index b55cf80c5522..d4e4abd103bb 100644
> --- a/drivers/gpu/drm/panel/panel-boe-th101mb31ig002-28a.c
> +++ b/drivers/gpu/drm/panel/panel-boe-th101mb31ig002-28a.c
> @@ -312,15 +312,14 @@ static int boe_th101mb31ig002_get_modes(struct drm_panel *panel,
>         struct boe_th101mb31ig002 *ctx = container_of(panel,
>                                                       struct boe_th101mb31ig002,
>                                                       panel);
> +       const struct drm_display_mode *desc_mode = ctx->desc->modes;
>         struct drm_display_mode *mode;
>
> -       mode = drm_mode_duplicate(connector->dev,
> -                                 &boe_th101mb31ig002_default_mode);
> +       mode = drm_mode_duplicate(connector->dev, desc_mode);
>         if (!mode) {
>                 dev_err(panel->dev, "Failed to add mode %ux%u@%u\n",
> -                       boe_th101mb31ig002_default_mode.hdisplay,
> -                       boe_th101mb31ig002_default_mode.vdisplay,
> -                       drm_mode_vrefresh(&boe_th101mb31ig002_default_mode));
> +                       desc_mode->hdisplay, desc_mode->vdisplay,
> +                       drm_mode_vrefresh(desc_mode));
>                 return -ENOMEM;

Please add a followup switching to drm_connector_helper_get_modes_fixed()

>         }
>
> --
> 2.17.1
>


-- 
With best wishes
Dmitry