Re: [PATCH 12/18] dt-bindings: pwm: add binding for mt8365 SoC
Hello, On Mon, Oct 23, 2023 at 11:44:11PM +0200, Uwe Kleine-König wrote: > On Mon, Oct 23, 2023 at 04:40:12PM +0200, Alexandre Mergnat wrote: > > Display PWM for MT8365 is compatible with MT8183. Then, add MT8365 binding > > along with MT8183 SoC. > > > > Signed-off-by: Alexandre Mergnat > > Acked-by: Uwe Kleine-König > > What is the merge plan here? Should the whole series go in via drm? I'll assume that Alexandre will send out a new round of this series and this patch isn't supposed to be applied as is to the pwm tree. So I'm marking this patch as "changes requested" in our patchwork. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König| Industrial Linux Solutions | https://www.pengutronix.de/ | signature.asc Description: PGP signature
Re: [PATCH 12/18] dt-bindings: pwm: add binding for mt8365 SoC
Il 23/10/23 16:40, Alexandre Mergnat ha scritto: Display PWM for MT8365 is compatible with MT8183. Then, add MT8365 binding along with MT8183 SoC. MT8365's Display PWM controller IP is the same as MT8183: blah blah blah :-) because saying that it is compatible doesn't mean that it is the same... That's a nitpick anyway, but whatever - you'll still have to send a v2 of this series, so you have an easy chance to edit the commit description of this one. Nitpicks apart... Reviewed-by: AngeloGioacchino Del Regno Signed-off-by: Alexandre Mergnat --- Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml b/Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml index efe0cacf55b7..e4069bcbf8d5 100644 --- a/Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml +++ b/Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml @@ -32,6 +32,7 @@ properties: - mediatek,mt8188-disp-pwm - mediatek,mt8192-disp-pwm - mediatek,mt8195-disp-pwm + - mediatek,mt8365-disp-pwm - const: mediatek,mt8183-disp-pwm reg:
Re: [PATCH 12/18] dt-bindings: pwm: add binding for mt8365 SoC
On Mon, Oct 23, 2023 at 04:40:12PM +0200, Alexandre Mergnat wrote: > Display PWM for MT8365 is compatible with MT8183. Then, add MT8365 binding > along with MT8183 SoC. > > Signed-off-by: Alexandre Mergnat Acked-by: Uwe Kleine-König What is the merge plan here? Should the whole series go in via drm? Thanks Uwe -- Pengutronix e.K. | Uwe Kleine-König| Industrial Linux Solutions | https://www.pengutronix.de/ | signature.asc Description: PGP signature
Re: [PATCH 12/18] dt-bindings: pwm: add binding for mt8365 SoC
On Mon, Oct 23, 2023 at 04:40:12PM +0200, Alexandre Mergnat wrote: > Display PWM for MT8365 is compatible with MT8183. Then, add MT8365 binding > along with MT8183 SoC. Additionally here, for the subject, "dt-bindings: pwm: mediatek,pwm-disp: add compatible for mt8365 SoC" Cheers Conor. > > Signed-off-by: Alexandre Mergnat > --- > Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml > b/Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml > index efe0cacf55b7..e4069bcbf8d5 100644 > --- a/Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml > +++ b/Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml > @@ -32,6 +32,7 @@ properties: >- mediatek,mt8188-disp-pwm >- mediatek,mt8192-disp-pwm >- mediatek,mt8195-disp-pwm > + - mediatek,mt8365-disp-pwm >- const: mediatek,mt8183-disp-pwm > >reg: > > -- > 2.25.1 > signature.asc Description: PGP signature
[PATCH 12/18] dt-bindings: pwm: add binding for mt8365 SoC
Display PWM for MT8365 is compatible with MT8183. Then, add MT8365 binding along with MT8183 SoC. Signed-off-by: Alexandre Mergnat --- Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml b/Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml index efe0cacf55b7..e4069bcbf8d5 100644 --- a/Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml +++ b/Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml @@ -32,6 +32,7 @@ properties: - mediatek,mt8188-disp-pwm - mediatek,mt8192-disp-pwm - mediatek,mt8195-disp-pwm + - mediatek,mt8365-disp-pwm - const: mediatek,mt8183-disp-pwm reg: -- 2.25.1
