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

[<Hari.PrasathGE@microchip.com>]

On 03/08/23 3:18 pm, Nancy.Lin wrote:
> [You don't often get email from nancy.lin@mediatek.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@mediatek.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@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel