Re: [PATCH v3 1/4] drm/mediatek: Remove freeing not dynamic allocated memory

Subject: Re: [PATCH v3 1/4] drm/mediatek: Remove freeing not dynamic allocated memory
From: Alexandre Mergnat <amergnat@xxxxxxxxxxxx>
Date: Wed, 21 Jun 2023 14:12:54 +0200
Cc: Singo Chang <singo.chang@xxxxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, dri-devel@xxxxxxxxxxxxxxxxxxxxx, Project_Global_Chrome_Upstream_Group@xxxxxxxxxxxx, Jason-ch Chen <jason-ch.chen@xxxxxxxxxxxx>, Nancy Lin <nancy.lin@xxxxxxxxxxxx>, Johnson Wang <johnson.wang@xxxxxxxxxxxx>, Shawn Sung <shawn.sung@xxxxxxxxxxxx>, Matthias Brugger <matthias.bgg@xxxxxxxxx>, linux-mediatek@xxxxxxxxxxxxxxxxxxx, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20230621102247.10116-2-jason-jh.lin@mediatek.com>
References: <20230621102247.10116-1-jason-jh.lin@mediatek.com> <20230621102247.10116-2-jason-jh.lin@mediatek.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0

On 21/06/2023 12:22, Jason-JH.Lin wrote:

Fixing the coverity issue of:
mtk_drm_cmdq_pkt_destroy frees address of mtk_crtc->cmdq_handle

So remove the free function.


Reviewed-by: Alexandre Mergnat <amergnat@xxxxxxxxxxxx>

--
Regards,
Alexandre

References:
- [PATCH v3 0/4] Fix mediatek-drm coverity issues
  - From: Jason-JH.Lin
- [PATCH v3 1/4] drm/mediatek: Remove freeing not dynamic allocated memory
  - From: Jason-JH.Lin