2022-02-26 01:39:56

by Nícolas F. R. A. Prado

[permalink] [raw]
Subject: [PATCH v1 0/3] Fixes for Mediatek dt-bindings


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/[email protected]/


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


2022-02-26 01:47:39

by Nícolas F. R. A. Prado

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

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

Signed-off-by: Nícolas F. R. A. Prado <[email protected]>
---

.../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

2022-02-26 02:18:47

by Nícolas F. R. A. Prado

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

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 <[email protected]>
---

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

2022-02-26 02:28:42

by Nícolas F. R. A. Prado

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

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

Signed-off-by: Nícolas F. R. A. Prado <[email protected]>

---

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

2022-03-04 23:14:08

by Rob Herring (Arm)

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

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 <[email protected]>
> ---
>
> .../devicetree/bindings/media/mediatek,vcodec-encoder.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>

Acked-by: Rob Herring <[email protected]>

2022-03-04 23:14:14

by Rob Herring (Arm)

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

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 <[email protected]>
>
> ---
>
> Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>

Acked-by: Rob Herring <[email protected]>

2022-03-04 23:14:18

by Rob Herring (Arm)

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

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 <[email protected]>
> ---
>
> Documentation/devicetree/bindings/usb/mediatek,mtk-xhci.yaml | 1 +
> 1 file changed, 1 insertion(+)
>

Acked-by: Rob Herring <[email protected]>

2022-04-05 03:05:10

by Nícolas F. R. A. Prado

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

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/[email protected]/
>
>
> 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

2022-04-22 18:31:50

by Matthias Brugger

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



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/[email protected]/
>>>
>>>
>>> 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

2022-04-22 19:01:55

by Bjorn Andersson

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

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/[email protected]/
> >
> >
> > 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

2022-04-22 23:30:34

by Bjorn Andersson

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

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 <[email protected]>