linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v1 0/3] Fixes for Mediatek dt-bindings
@ 2022-02-25 22:58 Nícolas F. R. A. Prado
  2022-02-25 22:58 ` [PATCH v1 1/3] dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp Nícolas F. R. A. Prado
                   ` (3 more replies)
  0 siblings, 4 replies; 11+ messages in thread
From: Nícolas F. R. A. Prado @ 2022-02-25 22:58 UTC (permalink / raw)
  To: Rob Herring
  Cc: AngeloGioacchino Del Regno, kernel, Nícolas F. R. A. Prado,
	Bjorn Andersson, Chunfeng Yun, Greg Kroah-Hartman,
	Krzysztof Kozlowski, Mathieu Poirier, Matthias Brugger,
	Mauro Carvalho Chehab, Tinghan Shen, Yunfei Dong, devicetree,
	linux-arm-kernel, linux-kernel, linux-media, linux-mediatek,
	linux-remoteproc, linux-usb


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


^ permalink raw reply	[flat|nested] 11+ messages in thread

* [PATCH v1 1/3] dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp
  2022-02-25 22:58 [PATCH v1 0/3] Fixes for Mediatek dt-bindings Nícolas F. R. A. Prado
@ 2022-02-25 22:58 ` Nícolas F. R. A. Prado
  2022-03-04 23:10   ` Rob Herring
  2022-04-22 23:00   ` (subset) " Bjorn Andersson
  2022-02-25 22:58 ` [PATCH v1 2/3] dt-bindings: usb: mtk-xhci: Allow wakeup interrupt-names to be optional Nícolas F. R. A. Prado
                   ` (2 subsequent siblings)
  3 siblings, 2 replies; 11+ messages in thread
From: Nícolas F. R. A. Prado @ 2022-02-25 22:58 UTC (permalink / raw)
  To: Rob Herring
  Cc: AngeloGioacchino Del Regno, kernel, Nícolas F. R. A. Prado,
	Bjorn Andersson, Krzysztof Kozlowski, Mathieu Poirier,
	Matthias Brugger, Tinghan Shen, devicetree, linux-arm-kernel,
	linux-kernel, linux-mediatek, linux-remoteproc

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


^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH v1 2/3] dt-bindings: usb: mtk-xhci: Allow wakeup interrupt-names to be optional
  2022-02-25 22:58 [PATCH v1 0/3] Fixes for Mediatek dt-bindings Nícolas F. R. A. Prado
  2022-02-25 22:58 ` [PATCH v1 1/3] dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp Nícolas F. R. A. Prado
@ 2022-02-25 22:58 ` Nícolas F. R. A. Prado
  2022-03-04 23:10   ` Rob Herring
  2022-02-25 22:58 ` [PATCH v1 3/3] media: dt-bindings: mtk-vcodec-encoder: Add power-domains property Nícolas F. R. A. Prado
  2022-04-04 15:22 ` [PATCH v1 0/3] Fixes for Mediatek dt-bindings Nícolas F. R. A. Prado
  3 siblings, 1 reply; 11+ messages in thread
From: Nícolas F. R. A. Prado @ 2022-02-25 22:58 UTC (permalink / raw)
  To: Rob Herring
  Cc: AngeloGioacchino Del Regno, kernel, Nícolas F. R. A. Prado,
	Chunfeng Yun, Greg Kroah-Hartman, Krzysztof Kozlowski,
	Matthias Brugger, devicetree, linux-arm-kernel, linux-kernel,
	linux-mediatek, linux-usb

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


^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH v1 3/3] media: dt-bindings: mtk-vcodec-encoder: Add power-domains property
  2022-02-25 22:58 [PATCH v1 0/3] Fixes for Mediatek dt-bindings Nícolas F. R. A. Prado
  2022-02-25 22:58 ` [PATCH v1 1/3] dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp Nícolas F. R. A. Prado
  2022-02-25 22:58 ` [PATCH v1 2/3] dt-bindings: usb: mtk-xhci: Allow wakeup interrupt-names to be optional Nícolas F. R. A. Prado
@ 2022-02-25 22:58 ` Nícolas F. R. A. Prado
  2022-03-04 23:10   ` Rob Herring
  2022-04-04 15:22 ` [PATCH v1 0/3] Fixes for Mediatek dt-bindings Nícolas F. R. A. Prado
  3 siblings, 1 reply; 11+ messages in thread
From: Nícolas F. R. A. Prado @ 2022-02-25 22:58 UTC (permalink / raw)
  To: Rob Herring
  Cc: AngeloGioacchino Del Regno, kernel, Nícolas F. R. A. Prado,
	Krzysztof Kozlowski, Matthias Brugger, Mauro Carvalho Chehab,
	Yunfei Dong, devicetree, linux-arm-kernel, linux-kernel,
	linux-media, linux-mediatek

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


^ permalink raw reply related	[flat|nested] 11+ messages in thread

* Re: [PATCH v1 1/3] dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp
  2022-02-25 22:58 ` [PATCH v1 1/3] dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp Nícolas F. R. A. Prado
@ 2022-03-04 23:10   ` Rob Herring
  2022-04-22 23:00   ` (subset) " Bjorn Andersson
  1 sibling, 0 replies; 11+ messages in thread
From: Rob Herring @ 2022-03-04 23:10 UTC (permalink / raw)
  To: Nícolas F. R. A. Prado
  Cc: Matthias Brugger, Bjorn Andersson, linux-mediatek, linux-kernel,
	linux-arm-kernel, Mathieu Poirier, kernel, Rob Herring,
	Krzysztof Kozlowski, linux-remoteproc,
	AngeloGioacchino Del Regno, devicetree, Tinghan Shen

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>

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH v1 2/3] dt-bindings: usb: mtk-xhci: Allow wakeup interrupt-names to be optional
  2022-02-25 22:58 ` [PATCH v1 2/3] dt-bindings: usb: mtk-xhci: Allow wakeup interrupt-names to be optional Nícolas F. R. A. Prado
@ 2022-03-04 23:10   ` Rob Herring
  0 siblings, 0 replies; 11+ messages in thread
From: Rob Herring @ 2022-03-04 23:10 UTC (permalink / raw)
  To: Nícolas F. R. A. Prado
  Cc: Krzysztof Kozlowski, devicetree, Matthias Brugger, linux-usb,
	linux-mediatek, linux-arm-kernel, AngeloGioacchino Del Regno,
	Greg Kroah-Hartman, linux-kernel, kernel, Rob Herring,
	Chunfeng Yun

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>

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH v1 3/3] media: dt-bindings: mtk-vcodec-encoder: Add power-domains property
  2022-02-25 22:58 ` [PATCH v1 3/3] media: dt-bindings: mtk-vcodec-encoder: Add power-domains property Nícolas F. R. A. Prado
@ 2022-03-04 23:10   ` Rob Herring
  0 siblings, 0 replies; 11+ messages in thread
From: Rob Herring @ 2022-03-04 23:10 UTC (permalink / raw)
  To: Nícolas F. R. A. Prado
  Cc: Matthias Brugger, Krzysztof Kozlowski, Rob Herring, Yunfei Dong,
	linux-media, kernel, linux-mediatek, AngeloGioacchino Del Regno,
	linux-arm-kernel, linux-kernel, Mauro Carvalho Chehab,
	devicetree

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>

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH v1 0/3] Fixes for Mediatek dt-bindings
  2022-02-25 22:58 [PATCH v1 0/3] Fixes for Mediatek dt-bindings Nícolas F. R. A. Prado
                   ` (2 preceding siblings ...)
  2022-02-25 22:58 ` [PATCH v1 3/3] media: dt-bindings: mtk-vcodec-encoder: Add power-domains property Nícolas F. R. A. Prado
@ 2022-04-04 15:22 ` Nícolas F. R. A. Prado
  2022-04-22 15:16   ` Bjorn Andersson
  3 siblings, 1 reply; 11+ messages in thread
From: Nícolas F. R. A. Prado @ 2022-04-04 15:22 UTC (permalink / raw)
  To: Matthias Brugger
  Cc: AngeloGioacchino Del Regno, kernel, Bjorn Andersson,
	Chunfeng Yun, Greg Kroah-Hartman, Krzysztof Kozlowski,
	Mathieu Poirier, Mauro Carvalho Chehab, Tinghan Shen,
	Yunfei Dong, devicetree, linux-arm-kernel, linux-kernel,
	linux-media, linux-mediatek, linux-remoteproc, linux-usb,
	Rob Herring

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH v1 0/3] Fixes for Mediatek dt-bindings
  2022-04-04 15:22 ` [PATCH v1 0/3] Fixes for Mediatek dt-bindings Nícolas F. R. A. Prado
@ 2022-04-22 15:16   ` Bjorn Andersson
  2022-04-22 15:40     ` Matthias Brugger
  0 siblings, 1 reply; 11+ messages in thread
From: Bjorn Andersson @ 2022-04-22 15:16 UTC (permalink / raw)
  To: N?colas F. R. A. Prado
  Cc: Matthias Brugger, AngeloGioacchino Del Regno, kernel,
	Chunfeng Yun, Greg Kroah-Hartman, Krzysztof Kozlowski,
	Mathieu Poirier, Mauro Carvalho Chehab, Tinghan Shen,
	Yunfei Dong, devicetree, linux-arm-kernel, linux-kernel,
	linux-media, linux-mediatek, linux-remoteproc, linux-usb,
	Rob Herring

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH v1 0/3] Fixes for Mediatek dt-bindings
  2022-04-22 15:16   ` Bjorn Andersson
@ 2022-04-22 15:40     ` Matthias Brugger
  0 siblings, 0 replies; 11+ messages in thread
From: Matthias Brugger @ 2022-04-22 15:40 UTC (permalink / raw)
  To: Bjorn Andersson, N?colas F. R. A. Prado
  Cc: AngeloGioacchino Del Regno, kernel, Chunfeng Yun,
	Greg Kroah-Hartman, Krzysztof Kozlowski, Mathieu Poirier,
	Mauro Carvalho Chehab, Tinghan Shen, Yunfei Dong, devicetree,
	linux-arm-kernel, linux-kernel, linux-media, linux-mediatek,
	linux-remoteproc, linux-usb, Rob Herring



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

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: (subset) [PATCH v1 1/3] dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp
  2022-02-25 22:58 ` [PATCH v1 1/3] dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp Nícolas F. R. A. Prado
  2022-03-04 23:10   ` Rob Herring
@ 2022-04-22 23:00   ` Bjorn Andersson
  1 sibling, 0 replies; 11+ messages in thread
From: Bjorn Andersson @ 2022-04-22 23:00 UTC (permalink / raw)
  To: Rob Herring, Nícolas F . R . A . Prado
  Cc: Tinghan Shen, devicetree, linux-remoteproc, linux-kernel,
	Krzysztof Kozlowski, Matthias Brugger, linux-mediatek, kernel,
	Mathieu Poirier, AngeloGioacchino Del Regno, linux-arm-kernel

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>

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2022-04-22 23:22 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-25 22:58 [PATCH v1 0/3] Fixes for Mediatek dt-bindings Nícolas F. R. A. Prado
2022-02-25 22:58 ` [PATCH v1 1/3] dt-bindings: remoteproc: mediatek: Add interrupts property to mtk,scp Nícolas F. R. A. Prado
2022-03-04 23:10   ` Rob Herring
2022-04-22 23:00   ` (subset) " Bjorn Andersson
2022-02-25 22:58 ` [PATCH v1 2/3] dt-bindings: usb: mtk-xhci: Allow wakeup interrupt-names to be optional Nícolas F. R. A. Prado
2022-03-04 23:10   ` Rob Herring
2022-02-25 22:58 ` [PATCH v1 3/3] media: dt-bindings: mtk-vcodec-encoder: Add power-domains property Nícolas F. R. A. Prado
2022-03-04 23:10   ` Rob Herring
2022-04-04 15:22 ` [PATCH v1 0/3] Fixes for Mediatek dt-bindings Nícolas F. R. A. Prado
2022-04-22 15:16   ` Bjorn Andersson
2022-04-22 15:40     ` Matthias Brugger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).