Re: [PATCH v4 5/5] dt-bindings: display: mediatek: dpi: correct power-domains property

From: Krzysztof Kozlowski
Date: Wed Oct 02 2024 - 02:04:37 EST


On Wed, Oct 02, 2024 at 11:09:07AM +0800, Macpaul Lin wrote:
>
>
> On 10/2/24 09:51, Rob Herring wrote:
> >
> >
> > External email : Please do not click links or open attachments until you
> > have verified the sender or the content.
> >
> > On Mon, Sep 30, 2024 at 04:38:54PM +0800, Macpaul Lin wrote:
> > > The MediaTek DPI module is typically associated with one of the
> > > following multimedia power domains:
> > > - POWER_DOMAIN_DISPLAY
> > > - POWER_DOMAIN_VDOSYS
> > > - POWER_DOMAIN_MM
> > > The specific power domain used varies depending on the SoC design.
> > >
> > > These power domains are shared by multiple devices within the SoC.
> > > In most cases, these power domains are enabled by other devices.
> > > As a result, the DPI module of legacy SoCs often functions correctly
> > > even without explicit configuration.
> > >
> > > It is recommended to explicitly add the appropriate power domain
> > > property to the DPI node in the device tree. Hence drop the
> > > compatible checking for specific SoCs.
> > >
> > > Fixes: 5474d49b2f79 ("dt-bindings: display: mediatek: dpi: Add power domains")
> > > Signed-off-by: Macpaul Lin <macpaul.lin@xxxxxxxxxxxx>
> > > Signed-off-by: Jitao Shi <jitao.shi@xxxxxxxxxxxx>
> > > Signed-off-by: Pablo Sun <pablo.sun@xxxxxxxxxxxx>
> > > ---
> > > .../display/mediatek/mediatek,dpi.yaml | 24 ++++++++-----------
> > > 1 file changed, 10 insertions(+), 14 deletions(-)
> >
> > You missed Krzysztof's R-by tag.
> >
>
> Oh, I just missed that reply for v3 in the mailbox
> and thought it still need to be reviewed.
> I just found Krzysztof's R-by tag in the mailbox right now.
>
> I'll send an update for this patch set.
> Thanks for the reminder.

Plaese, instead just start using b4...

Best regards,
Krzysztof