2023-03-13 15:03:58

by Chun-Kuang Hu

[permalink] [raw]
Subject: Re: [PATCH 01/21] dt-bindings: display: mediatek: aal: add binding for MT8365 SoC

Hi, Alexandre:

Alexandre Mergnat <[email protected]> 於 2023年3月9日 週四 下午10:23寫道:
>
> Display Adaptive Ambient Light for MT8365 is compatible with another SoC.
> Then, add MT8365 binding along with MT8183 SoC.

Reviewed-by: Chun-Kuang Hu <[email protected]>

>
> Signed-off-by: Alexandre Mergnat <[email protected]>
> ---
> Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml
> index d4d585485e7b..d47bc72f09c0 100644
> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml
> +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml
> @@ -33,6 +33,7 @@ properties:
> - mediatek,mt8186-disp-aal
> - mediatek,mt8192-disp-aal
> - mediatek,mt8195-disp-aal
> + - mediatek,mt8365-disp-aal
> - const: mediatek,mt8183-disp-aal
>
> reg:
>
> --
> b4 0.10.1


2023-03-31 14:57:28

by Matthias Brugger

[permalink] [raw]
Subject: Re: [PATCH 01/21] dt-bindings: display: mediatek: aal: add binding for MT8365 SoC

Hi Chun-Kuang Hu,

On 13/03/2023 16:02, Chun-Kuang Hu wrote:
> Hi, Alexandre:
>
> Alexandre Mergnat <[email protected]> 於 2023年3月9日 週四 下午10:23寫道:
>>
>> Display Adaptive Ambient Light for MT8365 is compatible with another SoC.
>> Then, add MT8365 binding along with MT8183 SoC.
>
> Reviewed-by: Chun-Kuang Hu <[email protected]>
>

I'm a bit puzzled that you give your reviewed by while I would have expected
that you will take the display binding patches. Will you take these or do you
want someone else to take them?

Regards,
Matthias

>>
>> Signed-off-by: Alexandre Mergnat <[email protected]>
>> ---
>> Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml
>> index d4d585485e7b..d47bc72f09c0 100644
>> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml
>> +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml
>> @@ -33,6 +33,7 @@ properties:
>> - mediatek,mt8186-disp-aal
>> - mediatek,mt8192-disp-aal
>> - mediatek,mt8195-disp-aal
>> + - mediatek,mt8365-disp-aal
>> - const: mediatek,mt8183-disp-aal
>>
>> reg:
>>
>> --
>> b4 0.10.1

2023-04-03 03:32:46

by Chun-Kuang Hu

[permalink] [raw]
Subject: Re: [PATCH 01/21] dt-bindings: display: mediatek: aal: add binding for MT8365 SoC

Hi, Matthias:

Matthias Brugger <[email protected]> 於 2023年3月31日 週五 下午10:55寫道:
>
> Hi Chun-Kuang Hu,
>
> On 13/03/2023 16:02, Chun-Kuang Hu wrote:
> > Hi, Alexandre:
> >
> > Alexandre Mergnat <[email protected]> 於 2023年3月9日 週四 下午10:23寫道:
> >>
> >> Display Adaptive Ambient Light for MT8365 is compatible with another SoC.
> >> Then, add MT8365 binding along with MT8183 SoC.
> >
> > Reviewed-by: Chun-Kuang Hu <[email protected]>
> >
>
> I'm a bit puzzled that you give your reviewed by while I would have expected
> that you will take the display binding patches. Will you take these or do you
> want someone else to take them?

I usually apply whole series together, and I've question about
"[07/21] dt-bindings: display: mediatek: dpi: add binding for MT8365"
in this series. This is just the first version, so maybe I would apply
partial patches in later version.

Regards,
Chun-Kuang.


>
> Regards,
> Matthias
>
> >>
> >> Signed-off-by: Alexandre Mergnat <[email protected]>
> >> ---
> >> Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml | 1 +
> >> 1 file changed, 1 insertion(+)
> >>
> >> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml
> >> index d4d585485e7b..d47bc72f09c0 100644
> >> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml
> >> +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,aal.yaml
> >> @@ -33,6 +33,7 @@ properties:
> >> - mediatek,mt8186-disp-aal
> >> - mediatek,mt8192-disp-aal
> >> - mediatek,mt8195-disp-aal
> >> + - mediatek,mt8365-disp-aal
> >> - const: mediatek,mt8183-disp-aal
> >>
> >> reg:
> >>
> >> --
> >> b4 0.10.1