Re: [PATCH] drm/mediatek/dp: fix memory leak on ->get_edid callback audio detection

[Chun-Kuang Hu]

Hi, Jani:

Jani Nikula <jani.nikula@xxxxxxxxx> 於 2023年9月22日 週五 下午5:44寫道：
>
> On Tue, 19 Sep 2023, Chen-Yu Tsai <wenst@xxxxxxxxxxxx> wrote:
> > On Tue, Sep 19, 2023 at 7:02 PM Jani Nikula <jani.nikula@xxxxxxxxx> wrote:
> >>
> >> On Fri, 15 Sep 2023, Chen-Yu Tsai <wenst@xxxxxxxxxxxx> wrote:
> >> > On Thu, Sep 14, 2023 at 11:53 PM Jani Nikula <jani.nikula@xxxxxxxxx> wrote:
> >> >>
> >> >> The sads returned by drm_edid_to_sad() needs to be freed.
> >> >>
> >> >> Fixes: e71a8ebbe086 ("drm/mediatek: dp: Audio support for MT8195")
> >> >> Cc: Guillaume Ranquet <granquet@xxxxxxxxxxxx>
> >> >> Cc: Bo-Chen Chen <rex-bc.chen@xxxxxxxxxxxx>
> >> >> Cc: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx>
> >> >> Cc: Dmitry Osipenko <dmitry.osipenko@xxxxxxxxxxxxx>
> >> >> Cc: Chun-Kuang Hu <chunkuang.hu@xxxxxxxxxx>
> >> >> Cc: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx>
> >> >> Cc: Matthias Brugger <matthias.bgg@xxxxxxxxx>
> >> >> Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx
> >> >> Cc: linux-mediatek@xxxxxxxxxxxxxxxxxxx
> >> >> Cc: linux-kernel@xxxxxxxxxxxxxxx
> >> >> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> >> >> Cc: <stable@xxxxxxxxxxxxxxx> # v6.1+
> >> >> Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx>
> >> >
> >> > Looks correct to me.
> >> >
> >> > Reviewed-by: Chen-Yu Tsai <wenst@xxxxxxxxxxxx>
> >>
> >> Thanks for the reviews Chen-Yu and Guillaume. Will you push this to
> >> drm-misc-next or shall I?
> >
> > Patches for the MediaTek drm driver go through their own separate tree,
> > maintained by CK (Chun-Kuang).
>
> Chun-Kuang, can you confirm picking up these two patches, please?
>
> MAINTAINERS does not list a separate git repository for MediaTek drm
> drivers, so I don't know where that would be. It should probably be
> added to MAINTAINERS.

Applied to mediatek-drm-next [1], thanks.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git/log/?h=mediatek-drm-next

Regards,
Chun-Kuang.

>
> Thanks,
> Jani.
>
> >
> > ChenYu
>
> --
> Jani Nikula, Intel