[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <YldWhNA6SwNBGXa8@robh.at.kernel.org>
Date: Wed, 13 Apr 2022 18:02:28 -0500
From: Rob Herring <robh@...nel.org>
To: Rex-BC Chen <rex-bc.chen@...iatek.com>
Cc: matthias.bgg@...il.com, yongqiang.niu@...iatek.com,
allen-kh.cheng@...iatek.com, linux-kernel@...r.kernel.org,
krzk+dt@...nel.org, airlied@...ux.ie, devicetree@...r.kernel.org,
nancy.lin@...iatek.com, robh+dt@...nel.org,
dri-devel@...ts.freedesktop.org,
linux-arm-kernel@...ts.infradead.org, chunkuang.hu@...nel.org,
Project_Global_Chrome_Upstream_Group@...iatek.com,
p.zabel@...gutronix.de, jason-jh.lin@...iatek.com,
linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH V2 1/3] dt-bindings: display: mediatek: Update disp_aal
binding for MT8183
On Mon, 11 Apr 2022 11:58:41 +0800, Rex-BC Chen wrote:
> The driver data of MT8183 and MT8173 are different.
>
> For MT8173, the gamma module is inside disp_aal. When we need to adjust
> gamma value, we need to use "has_gamma" to control gamma function
> inside disp_aal to adjust the gamma value.
>
> For successors like MT8183, disp_gamma is separated from disp_aal. We
> just need to control disp_gamma directly and don't need to control gamma
> function inside disp_aal.
>
> With this modification, the driver doesn't require any functional changes.
> We only update the dt-binding and DTS node to make it clear.
>
> Signed-off-by: Rex-BC Chen <rex-bc.chen@...iatek.com>
> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
> ---
> .../devicetree/bindings/display/mediatek/mediatek,aal.yaml | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
Acked-by: Rob Herring <robh@...nel.org>
Powered by blists - more mailing lists