[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAAOTY__EAW-Tj93oSnN1TTB0sH3VMnHwwRaVv-Nm7cEGBeuvcQ@mail.gmail.com>
Date: Thu, 15 Aug 2024 21:57:07 +0800
From: Chun-Kuang Hu <chunkuang.hu@...nel.org>
To: Javier Carrasco <javier.carrasco.cruz@...il.com>
Cc: Chun-Kuang Hu <chunkuang.hu@...nel.org>, Philipp Zabel <p.zabel@...gutronix.de>,
David Airlie <airlied@...il.com>, Daniel Vetter <daniel@...ll.ch>,
Matthias Brugger <matthias.bgg@...il.com>,
AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
"Nancy.Lin" <nancy.lin@...iatek.com>, dri-devel@...ts.freedesktop.org,
linux-mediatek@...ts.infradead.org, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, stable@...r.kernel.org
Subject: Re: [PATCH 0/3] drm/mediatek: fixes for ovl_adaptor
Hi, Javier:
Javier Carrasco <javier.carrasco.cruz@...il.com> 於 2024年6月25日 週二 上午12:44寫道:
>
> The main fix is a possible memory leak on an early exit in the
> for_each_child_of_node() loop. That fix has been divided into a patch
> that can be backported (a simple of_node_put()), and another one that
> uses the scoped variant of the macro, removing the need for any
> of_node_put(). That prevents mistakes if new break/return instructions
> are added, but the macro might not be available in older kernels.
>
> When at it, an unused header has been dropped.
For this series, applied to mediatek-drm-next [1], thanks.
Regards,
CK
[1] https://git.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git/log/?h=mediatek-drm-next
>
> Signed-off-by: Javier Carrasco <javier.carrasco.cruz@...il.com>
> ---
> Javier Carrasco (3):
> drm/mediatek: ovl_adaptor: drop unused mtk_crtc.h header
> drm/mediatek: ovl_adaptor: add missing of_node_put()
> drm/mediatek: ovl_adaptor: use scoped variant of for_each_child_of_node()
>
> drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
> ---
> base-commit: f76698bd9a8ca01d3581236082d786e9a6b72bb7
> change-id: 20240624-mtk_disp_ovl_adaptor_scoped-0702a6b23443
>
> Best regards,
> --
> Javier Carrasco <javier.carrasco.cruz@...il.com>
>
Powered by blists - more mailing lists