[PATCH v1 0/3] Fixes for Mediatek dt-bindings

[PATCH v1 0/3] Fixes for Mediatek dt-bindings

[Nícolas F. R. A. Prado]

This series has some fixes for Mediatek dt-bindings. It solves some
warnings printed by dtbs_check, both for already merged Devicetrees, as
well as some that would be introduced by the changes to mt8192.dtsi in
[1].

[1] https://lore.kernel.org/all/20220218091633.9368-1-allen-kh.cheng@mediatek.com/


Nícolas F. R. A. Prado (3):
  dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp
  dt-bindings: usb: mtk-xhci: Allow wakeup interrupt-names to be
    optional
  media: dt-bindings: mtk-vcodec-encoder: Add power-domains property

 .../devicetree/bindings/media/mediatek,vcodec-encoder.yaml     | 3 +++
 Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml      | 3 +++
 Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml   | 1 +
 3 files changed, 7 insertions(+)

-- 
2.35.1

[PATCH v1 1/3] dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp

[Nícolas F. R. A. Prado]

The SCP node can have an associated interrupt. Add a property for it.

Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>

---

 Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml b/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
index d21a25ee96e6..103ccea462f8 100644
--- a/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
@@ -41,6 +41,9 @@ properties:
   clock-names:
     const: main
 
+  interrupts:
+    maxItems: 1
+
 required:
   - compatible
   - reg
-- 
2.35.1

Re: [PATCH v1 1/3] dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp

[Rob Herring]

On Fri, 25 Feb 2022 17:58:52 -0500, Nícolas F. R. A. Prado wrote:
> The SCP node can have an associated interrupt. Add a property for it.
> 
> Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
> 
> ---
> 
>  Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml | 3 +++
>  1 file changed, 3 insertions(+)
> 

Acked-by: Rob Herring <robh@kernel.org>

Re: (subset) [PATCH v1 1/3] dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp

[Bjorn Andersson]

On Fri, 25 Feb 2022 17:58:52 -0500, Nícolas F. R. A. Prado wrote:
> The SCP node can have an associated interrupt. Add a property for it.
> 
> 

Applied, thanks!

[1/3] dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp
      commit: b7da6f517214c307efece604ac9dc58dc6123c07

Best regards,
-- 
Bjorn Andersson <bjorn.andersson@linaro.org>

[PATCH v1 2/3] dt-bindings: usb: mtk-xhci: Allow wakeup interrupt-names to be optional

[Nícolas F. R. A. Prado]

Add missing 'minItems: 1' to the interrupt-names property to allow the
second interrupt-names, 'wakeup', to be optional.

Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
---

 Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml b/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml
index 41efb51638d1..f069c1d77e62 100644
--- a/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml
+++ b/Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml
@@ -56,6 +56,7 @@ properties:
       - description: optional, wakeup interrupt used to support runtime PM
 
   interrupt-names:
+    minItems: 1
     items:
       - const: host
       - const: wakeup
-- 
2.35.1

Re: [PATCH v1 2/3] dt-bindings: usb: mtk-xhci: Allow wakeup interrupt-names to be optional

[Rob Herring]

On Fri, 25 Feb 2022 17:58:53 -0500, Nícolas F. R. A. Prado wrote:
> Add missing 'minItems: 1' to the interrupt-names property to allow the
> second interrupt-names, 'wakeup', to be optional.
> 
> Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
> ---
> 
>  Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 

Acked-by: Rob Herring <robh@kernel.org>

[PATCH v1 3/3] media: dt-bindings: mtk-vcodec-encoder: Add power-domains property

[Nícolas F. R. A. Prado]

The encoder node may be dependent on a power-domain. Add a property for
it.

Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
---

 .../devicetree/bindings/media/mediatek,vcodec-encoder.yaml     | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/media/mediatek,vcodec-encoder.yaml b/Documentation/devicetree/bindings/media/mediatek,vcodec-encoder.yaml
index e7b65a91c92c..c288913e5666 100644
--- a/Documentation/devicetree/bindings/media/mediatek,vcodec-encoder.yaml
+++ b/Documentation/devicetree/bindings/media/mediatek,vcodec-encoder.yaml
@@ -65,6 +65,9 @@ properties:
     description:
       Describes point to scp.
 
+  power-domains:
+    maxItems: 1
+
 required:
   - compatible
   - reg
-- 
2.35.1

Re: [PATCH v1 3/3] media: dt-bindings: mtk-vcodec-encoder: Add power-domains property

[Rob Herring]

On Fri, 25 Feb 2022 17:58:54 -0500, Nícolas F. R. A. Prado wrote:
> The encoder node may be dependent on a power-domain. Add a property for
> it.
> 
> Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
> ---
> 
>  .../devicetree/bindings/media/mediatek,vcodec-encoder.yaml     | 3 +++
>  1 file changed, 3 insertions(+)
> 

Acked-by: Rob Herring <robh@kernel.org>

Re: [PATCH v1 0/3] Fixes for Mediatek dt-bindings

[Nícolas F. R. A. Prado]

Hi Matthias,

Maybe you could pick this series through your tree? It addresses some dtc
warnings that are introduced with the new nodes in mt8192.dtsi that you just
picked up on your v5.18-next/dts64 branch.

Thanks,
Nícolas

On Fri, Feb 25, 2022 at 05:58:51PM -0500, Nícolas F. R. A. Prado wrote:
> 
> This series has some fixes for Mediatek dt-bindings. It solves some
> warnings printed by dtbs_check, both for already merged Devicetrees, as
> well as some that would be introduced by the changes to mt8192.dtsi in
> [1].
> 
> [1] https://lore.kernel.org/all/20220218091633.9368-1-allen-kh.cheng@mediatek.com/
> 
> 
> Nícolas F. R. A. Prado (3):
>   dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp
>   dt-bindings: usb: mtk-xhci: Allow wakeup interrupt-names to be
>     optional
>   media: dt-bindings: mtk-vcodec-encoder: Add power-domains property
> 
>  .../devicetree/bindings/media/mediatek,vcodec-encoder.yaml     | 3 +++
>  Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml      | 3 +++
>  Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml   | 1 +
>  3 files changed, 7 insertions(+)
> 
> -- 
> 2.35.1

Re: [PATCH v1 0/3] Fixes for Mediatek dt-bindings

[Bjorn Andersson]

On Mon 04 Apr 10:22 CDT 2022, N?colas F. R. A. Prado wrote:

> Hi Matthias,
> 
> Maybe you could pick this series through your tree? It addresses some dtc
> warnings that are introduced with the new nodes in mt8192.dtsi that you just
> picked up on your v5.18-next/dts64 branch.
> 

I don't see a reply from Matthias, so I've picked up this (the
remoteproc) fix in our tree.

Thanks,
Bjorn

> Thanks,
> Nícolas
> 
> On Fri, Feb 25, 2022 at 05:58:51PM -0500, Nícolas F. R. A. Prado wrote:
> > 
> > This series has some fixes for Mediatek dt-bindings. It solves some
> > warnings printed by dtbs_check, both for already merged Devicetrees, as
> > well as some that would be introduced by the changes to mt8192.dtsi in
> > [1].
> > 
> > [1] https://lore.kernel.org/all/20220218091633.9368-1-allen-kh.cheng@mediatek.com/
> > 
> > 
> > Nícolas F. R. A. Prado (3):
> >   dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp
> >   dt-bindings: usb: mtk-xhci: Allow wakeup interrupt-names to be
> >     optional
> >   media: dt-bindings: mtk-vcodec-encoder: Add power-domains property
> > 
> >  .../devicetree/bindings/media/mediatek,vcodec-encoder.yaml     | 3 +++
> >  Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml      | 3 +++
> >  Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml   | 1 +
> >  3 files changed, 7 insertions(+)
> > 
> > -- 
> > 2.35.1

Re: [PATCH v1 0/3] Fixes for Mediatek dt-bindings

[Matthias Brugger]

On 22/04/2022 17:16, Bjorn Andersson wrote:
> On Mon 04 Apr 10:22 CDT 2022, N?colas F. R. A. Prado wrote:
> 
>> Hi Matthias,
>>
>> Maybe you could pick this series through your tree? It addresses some dtc
>> warnings that are introduced with the new nodes in mt8192.dtsi that you just
>> picked up on your v5.18-next/dts64 branch.
>>
> 
> I don't see a reply from Matthias, so I've picked up this (the
> remoteproc) fix in our tree.
> 

That's totally fine. I'm reluctant to take random dt-binding patches as it can 
provoke merge conflicts when there are other patches on the same binding taken 
by the driver maintainer. Normally I leave it to the driver maintainer to take 
such kind of patches.

Regards,
Matthias

> Thanks,
> Bjorn
> 
>> Thanks,
>> Nícolas
>>
>> On Fri, Feb 25, 2022 at 05:58:51PM -0500, Nícolas F. R. A. Prado wrote:
>>>
>>> This series has some fixes for Mediatek dt-bindings. It solves some
>>> warnings printed by dtbs_check, both for already merged Devicetrees, as
>>> well as some that would be introduced by the changes to mt8192.dtsi in
>>> [1].
>>>
>>> [1] https://lore.kernel.org/all/20220218091633.9368-1-allen-kh.cheng@mediatek.com/
>>>
>>>
>>> Nícolas F. R. A. Prado (3):
>>>    dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp
>>>    dt-bindings: usb: mtk-xhci: Allow wakeup interrupt-names to be
>>>      optional
>>>    media: dt-bindings: mtk-vcodec-encoder: Add power-domains property
>>>
>>>   .../devicetree/bindings/media/mediatek,vcodec-encoder.yaml     | 3 +++
>>>   Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml      | 3 +++
>>>   Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml   | 1 +
>>>   3 files changed, 7 insertions(+)
>>>
>>> -- 
>>> 2.35.1