linux-kernel - Re: [PATCH v3] drm/mediatek: fix uninitialized symbol

Open Source and information security mailing list archives

Message-ID: <688a97b5-ce0c-cf6b-c78b-aa7410347d1d@microchip.com>
Date:   Thu, 3 Aug 2023 10:15:54 +0000
From:   <Hari.PrasathGE@...rochip.com>
To:     <nancy.lin@...iatek.com>, <chunkuang.hu@...nel.org>,
        <p.zabel@...gutronix.de>, <matthias.bgg@...il.com>,
        <angelogioacchino.delregno@...labora.com>
CC:     <airlied@...il.com>, <daniel@...ll.ch>,
        <dri-devel@...ts.freedesktop.org>,
        <linux-mediatek@...ts.infradead.org>,
        <linux-kernel@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        <Project_Global_Chrome_Upstream_Group@...iatek.com>,
        <singo.chang@...iatek.com>
Subject: Re: [PATCH v3] drm/mediatek: fix uninitialized symbol

On 03/08/23 3:18 pm, Nancy.Lin wrote:
> [You don't often get email from nancy.lin@...iatek.com. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
> 
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> fix Smatch static checker warning
>    -Fix uninitialized symbol comp_pdev in mtk_ddp_comp_init.
> 
> Fixes: 0d9eee9118b7 ("drm/mediatek: Add drm ovl_adaptor sub driver for MT8195")
> Signed-off-by: Nancy.Lin <nancy.lin@...iatek.com>
> ---
> v3: fix reviewer comment in v2
> v2: add Fixes tag
> ---
>   drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c | 16 ++++++++--------
>   1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c b/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c
> index f114da4d36a9..771f4e173353 100644
> --- a/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c
> +++ b/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c
> @@ -563,14 +563,15 @@ int mtk_ddp_comp_init(struct device_node *node, struct mtk_ddp_comp *comp,
>          /* Not all drm components have a DTS device node, such as ovl_adaptor,
>           * which is the drm bring up sub driver
>           */
> -       if (node) {
> -               comp_pdev = of_find_device_by_node(node);
> -               if (!comp_pdev) {
> -                       DRM_INFO("Waiting for device %s\n", node->full_name);
> -                       return -EPROBE_DEFER;
> -               }
> -               comp->dev = &comp_pdev->dev;
> +       if (!node)
> +               return 0;
> +
> +       comp_pdev = of_find_device_by_node(node);
> +       if (!comp_pdev) {
> +               DRM_INFO("Waiting for device %s\n", node->full_name);
> +               return -EPROBE_DEFER;
>          }
> +       comp->dev = &comp_pdev->dev;
> 
>          if (type == MTK_DISP_AAL ||
>              type == MTK_DISP_BLS ||
> @@ -580,7 +581,6 @@ int mtk_ddp_comp_init(struct device_node *node, struct mtk_ddp_comp *comp,
>              type == MTK_DISP_MERGE ||
>              type == MTK_DISP_OVL ||
>              type == MTK_DISP_OVL_2L ||
> -           type == MTK_DISP_OVL_ADAPTOR ||

Is this also a related change ?

>              type == MTK_DISP_PWM ||
>              type == MTK_DISP_RDMA ||
>              type == MTK_DPI ||
> --
> 2.18.0
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@...ts.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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