| Message ID | 20231023-display-support-v1-12-5c860ed5c33b@baylibre.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Add display support for the MT8365-EVK board | expand |
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 <amergnat@baylibre.com> > --- > 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 >
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 <amergnat@baylibre.com> Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> What is the merge plan here? Should the whole series go in via drm? Thanks Uwe
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 <angelogioacchino.delregno@collabora.com> > > Signed-off-by: Alexandre Mergnat <amergnat@baylibre.com> > --- > 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: >
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 <amergnat@baylibre.com> > > Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> > > 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
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:
Display PWM for MT8365 is compatible with MT8183. Then, add MT8365 binding along with MT8183 SoC. Signed-off-by: Alexandre Mergnat <amergnat@baylibre.com> --- Documentation/devicetree/bindings/pwm/mediatek,pwm-disp.yaml | 1 + 1 file changed, 1 insertion(+)