2022-11-03 19:54:08

by Nícolas F. R. A. Prado

[permalink] [raw]
Subject: Re: [PATCH v27 01/11] dt-bindings: arm: mediatek: mmsys: add vdosys1 compatible for MT8195

On Thu, Nov 03, 2022 at 11:25:02AM +0800, Nancy.Lin wrote:
> Add vdosys1 mmsys compatible for MT8195 platform.
>
> For MT8195, VDOSYS0 and VDOSYS1 are 2 display HW pipelines binding to
> 2 different power domains, different clock drivers and different
> mediatek-drm drivers.
>
> Signed-off-by: Nancy.Lin <[email protected]>
> Change-Id: I8065205ce44baa417f58176adaae9e1eefa6eadb

No Change-Id tag, please. There are some other commits with it as well.

Otherwise,

Reviewed-by: N?colas F. R. A. Prado <[email protected]>

Thanks,
N?colas


2022-11-04 09:32:30

by Nancy Lin (林欣螢)

[permalink] [raw]
Subject: Re: [PATCH v27 01/11] dt-bindings: arm: mediatek: mmsys: add vdosys1 compatible for MT8195

Dear Nicolas,

Thanks for the review.

On Thu, 2022-11-03 at 15:39 -0400, Nícolas F. R. A. Prado wrote:
> On Thu, Nov 03, 2022 at 11:25:02AM +0800, Nancy.Lin wrote:
> > Add vdosys1 mmsys compatible for MT8195 platform.
> >
> > For MT8195, VDOSYS0 and VDOSYS1 are 2 display HW pipelines binding
> > to
> > 2 different power domains, different clock drivers and different
> > mediatek-drm drivers.
> >
> > Signed-off-by: Nancy.Lin <[email protected]>
> > Change-Id: I8065205ce44baa417f58176adaae9e1eefa6eadb
>
> No Change-Id tag, please. There are some other commits with it as
> well.
>
> Otherwise,
>
> Reviewed-by: Nícolas F. R. A. Prado <[email protected]>
>
> Thanks,
> Nícolas

Sorry I forgot to remove Change-Id. I will check all the commits in the
series.

Thanks,
Nancy