diff mbox series

[2/3] dt-bindings: clock: mediatek: Remove compatible for MT8188 VPPSYS

Message ID 20231109082201.7698-3-yu-chang.lee@mediatek.com (mailing list archive)
State New
Headers show
Series Migrate MT8188 VPPSYS 0/1 to mtk-mmsys driver | expand

Commit Message

Yu-chang Lee (李禹璋) Nov. 9, 2023, 8:22 a.m. UTC
Remove VPPSYS0, VPPSYS1 compatible on Mediatek MT8188.

Signed-off-by: yu-chang.lee <yu-chang.lee@mediatek.com>
---
 .../devicetree/bindings/clock/mediatek,mt8188-clock.yaml        | 2 --
 1 file changed, 2 deletions(-)

Comments

AngeloGioacchino Del Regno Nov. 9, 2023, 9:15 a.m. UTC | #1
Il 09/11/23 09:22, yu-chang.lee ha scritto:
> Remove VPPSYS0, VPPSYS1 compatible on Mediatek MT8188.
> 
> Signed-off-by: yu-chang.lee <yu-chang.lee@mediatek.com>

The mail recipients are wrong/incomplete. This patch cannot ever possibly get
upstream if you don't Cc the right people/lists.

Please use scripts/get_maintainer.pl to get the right list of recipients and
resend.

Thanks,
Angelo

> ---
>   .../devicetree/bindings/clock/mediatek,mt8188-clock.yaml        | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/clock/mediatek,mt8188-clock.yaml b/Documentation/devicetree/bindings/clock/mediatek,mt8188-clock.yaml
> index d7214d97b2ba..860570320545 100644
> --- a/Documentation/devicetree/bindings/clock/mediatek,mt8188-clock.yaml
> +++ b/Documentation/devicetree/bindings/clock/mediatek,mt8188-clock.yaml
> @@ -43,8 +43,6 @@ properties:
>         - mediatek,mt8188-vdecsys
>         - mediatek,mt8188-vdecsys-soc
>         - mediatek,mt8188-vencsys
> -      - mediatek,mt8188-vppsys0
> -      - mediatek,mt8188-vppsys1
>         - mediatek,mt8188-wpesys
>         - mediatek,mt8188-wpesys-vpp0
>
Yu-chang Lee (李禹璋) Nov. 9, 2023, 11:14 a.m. UTC | #2
On Thu, 2023-11-09 at 10:15 +0100, AngeloGioacchino Del Regno wrote:
> Il 09/11/23 09:22, yu-chang.lee ha scritto:
> > Remove VPPSYS0, VPPSYS1 compatible on Mediatek MT8188.
> > 
> > Signed-off-by: yu-chang.lee <yu-chang.lee@mediatek.com>
> 
> The mail recipients are wrong/incomplete. This patch cannot ever
> possibly get
> upstream if you don't Cc the right people/lists.
> 
> Please use scripts/get_maintainer.pl to get the right list of
> recipients and
> resend.
> 
> Thanks,
> Angelo
> 
Thank for your time and patience, I already resend the same series with
list script provided.

Thanks

Best Regards,
Yu Chang

> > ---
> >   .../devicetree/bindings/clock/mediatek,mt8188-clock.yaml        | 
> > 2 --
> >   1 file changed, 2 deletions(-)
> > 
> > diff --git
> > a/Documentation/devicetree/bindings/clock/mediatek,mt8188-
> > clock.yaml
> > b/Documentation/devicetree/bindings/clock/mediatek,mt8188-
> > clock.yaml
> > index d7214d97b2ba..860570320545 100644
> > --- a/Documentation/devicetree/bindings/clock/mediatek,mt8188-
> > clock.yaml
> > +++ b/Documentation/devicetree/bindings/clock/mediatek,mt8188-
> > clock.yaml
> > @@ -43,8 +43,6 @@ properties:
> >         - mediatek,mt8188-vdecsys
> >         - mediatek,mt8188-vdecsys-soc
> >         - mediatek,mt8188-vencsys
> > -      - mediatek,mt8188-vppsys0
> > -      - mediatek,mt8188-vppsys1
> >         - mediatek,mt8188-wpesys
> >         - mediatek,mt8188-wpesys-vpp0
> >   
> 
>
Krzysztof Kozlowski Nov. 9, 2023, 4:27 p.m. UTC | #3
On 09/11/2023 09:22, yu-chang.lee wrote:
> Remove VPPSYS0, VPPSYS1 compatible on Mediatek MT8188.
> 
> Signed-off-by: yu-chang.lee <yu-chang.lee@mediatek.com>
> ---

Please use scripts/get_maintainers.pl to get a list of necessary people
and lists to CC. It might happen, that command when run on an older
kernel, gives you outdated entries. Therefore please be sure you base
your patches on recent Linux kernel.

You missed at least devicetree list (maybe more), so this won't be
tested by automated tooling. Performing review on untested code might be
a waste of time, thus I will skip this patch entirely till you follow
the process allowing the patch to be tested.

Please kindly resend and include all necessary To/Cc entries.

Best regards,
Krzysztof
Krzysztof Kozlowski Nov. 9, 2023, 4:30 p.m. UTC | #4
On 09/11/2023 12:14, Yu-chang Lee (李禹璋) wrote:
> On Thu, 2023-11-09 at 10:15 +0100, AngeloGioacchino Del Regno wrote:
>> Il 09/11/23 09:22, yu-chang.lee ha scritto:
>>> Remove VPPSYS0, VPPSYS1 compatible on Mediatek MT8188.
>>>
>>> Signed-off-by: yu-chang.lee <yu-chang.lee@mediatek.com>
>>
>> The mail recipients are wrong/incomplete. This patch cannot ever
>> possibly get
>> upstream if you don't Cc the right people/lists.
>>
>> Please use scripts/get_maintainer.pl to get the right list of
>> recipients and
>> resend.
>>
>> Thanks,
>> Angelo
>>
> Thank for your time and patience, I already resend the same series with
> list script provided.

Oh, really? Please point me to this submission where you Cc all
maintainers presented by get_maintainer.pl.

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/clock/mediatek,mt8188-clock.yaml b/Documentation/devicetree/bindings/clock/mediatek,mt8188-clock.yaml
index d7214d97b2ba..860570320545 100644
--- a/Documentation/devicetree/bindings/clock/mediatek,mt8188-clock.yaml
+++ b/Documentation/devicetree/bindings/clock/mediatek,mt8188-clock.yaml
@@ -43,8 +43,6 @@  properties:
       - mediatek,mt8188-vdecsys
       - mediatek,mt8188-vdecsys-soc
       - mediatek,mt8188-vencsys
-      - mediatek,mt8188-vppsys0
-      - mediatek,mt8188-vppsys1
       - mediatek,mt8188-wpesys
       - mediatek,mt8188-wpesys-vpp0