mbox series

[0/3] dt-bindings: Convert Arm Mali GPUs to DT schema

Message ID 20190820195959.6126-1-robh@kernel.org (mailing list archive)
Headers show
Series dt-bindings: Convert Arm Mali GPUs to DT schema | expand

Message

Rob Herring Aug. 20, 2019, 7:59 p.m. UTC
This series converts the various Arm Mali GPU bindings to use the DT
schema format.

The Midgard and Bifrost bindings generate warnings on 'interrupt-names'
because there's all different ordering. The Utgard binding generates 
warnings on Rockchip platforms because 'clock-names' order is reversed.

Rob

Rob Herring (3):
  dt-bindings: Convert Arm Mali Midgard GPU to DT schema
  dt-bindings: Convert Arm Mali Bifrost GPU to DT schema
  dt-bindings: Convert Arm Mali Utgard GPU to DT schema

 .../bindings/gpu/arm,mali-bifrost.txt         |  92 ----------
 .../bindings/gpu/arm,mali-bifrost.yaml        | 115 ++++++++++++
 .../bindings/gpu/arm,mali-midgard.txt         | 119 -------------
 .../bindings/gpu/arm,mali-midgard.yaml        | 165 +++++++++++++++++
 .../bindings/gpu/arm,mali-utgard.txt          | 129 --------------
 .../bindings/gpu/arm,mali-utgard.yaml         | 166 ++++++++++++++++++
 6 files changed, 446 insertions(+), 340 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-bifrost.txt
 create mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml
 delete mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt
 create mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml
 delete mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-utgard.txt
 create mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml

Comments

Neil Armstrong Aug. 21, 2019, 7:13 a.m. UTC | #1
Hi,

On 20/08/2019 21:59, Rob Herring wrote:
> This series converts the various Arm Mali GPU bindings to use the DT
> schema format.
> 
> The Midgard and Bifrost bindings generate warnings on 'interrupt-names'
> because there's all different ordering. The Utgard binding generates 
> warnings on Rockchip platforms because 'clock-names' order is reversed.

Except the fact clocks should be required on all amlogic bindings,
Reviewed-by: Neil Armstrong <narmstrong@baylibre.com>
I'll send a fixup patch to enforce clocks.

Thanks,
Neil

> 
> Rob
> 
> Rob Herring (3):
>   dt-bindings: Convert Arm Mali Midgard GPU to DT schema
>   dt-bindings: Convert Arm Mali Bifrost GPU to DT schema
>   dt-bindings: Convert Arm Mali Utgard GPU to DT schema
> 
>  .../bindings/gpu/arm,mali-bifrost.txt         |  92 ----------
>  .../bindings/gpu/arm,mali-bifrost.yaml        | 115 ++++++++++++
>  .../bindings/gpu/arm,mali-midgard.txt         | 119 -------------
>  .../bindings/gpu/arm,mali-midgard.yaml        | 165 +++++++++++++++++
>  .../bindings/gpu/arm,mali-utgard.txt          | 129 --------------
>  .../bindings/gpu/arm,mali-utgard.yaml         | 166 ++++++++++++++++++
>  6 files changed, 446 insertions(+), 340 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-bifrost.txt
>  create mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml
>  delete mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt
>  create mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml
>  delete mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-utgard.txt
>  create mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml
>
Maxime Ripard Aug. 21, 2019, 7:31 a.m. UTC | #2
Hi Rob,

On Tue, Aug 20, 2019 at 02:59:56PM -0500, Rob Herring wrote:
> This series converts the various Arm Mali GPU bindings to use the DT
> schema format.
>
> The Midgard and Bifrost bindings generate warnings on 'interrupt-names'
> because there's all different ordering. The Utgard binding generates
> warnings on Rockchip platforms because 'clock-names' order is reversed.

Thank for taking care of that one, it was on my radar but I didn't
really want to actually do it :)

Acked-by: Maxime Ripard <maxime.ripard@bootlin.com>

Maxime

--
Maxime Ripard, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
Heiko Stuebner Aug. 21, 2019, 5:24 p.m. UTC | #3
Am Dienstag, 20. August 2019, 21:59:56 CEST schrieb Rob Herring:
> This series converts the various Arm Mali GPU bindings to use the DT
> schema format.
> 
> The Midgard and Bifrost bindings generate warnings on 'interrupt-names'
> because there's all different ordering. The Utgard binding generates 
> warnings on Rockchip platforms because 'clock-names' order is reversed.

Are you planning on sending fixes for these, should I just change the
ordering myself?


> Rob Herring (3):
>   dt-bindings: Convert Arm Mali Midgard GPU to DT schema
>   dt-bindings: Convert Arm Mali Bifrost GPU to DT schema
>   dt-bindings: Convert Arm Mali Utgard GPU to DT schema

Acked-by: Heiko Stuebner <heiko@sntech.de>


Thanks for doing the conversion
Heiko


>  .../bindings/gpu/arm,mali-bifrost.txt         |  92 ----------
>  .../bindings/gpu/arm,mali-bifrost.yaml        | 115 ++++++++++++
>  .../bindings/gpu/arm,mali-midgard.txt         | 119 -------------
>  .../bindings/gpu/arm,mali-midgard.yaml        | 165 +++++++++++++++++
>  .../bindings/gpu/arm,mali-utgard.txt          | 129 --------------
>  .../bindings/gpu/arm,mali-utgard.yaml         | 166 ++++++++++++++++++
>  6 files changed, 446 insertions(+), 340 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-bifrost.txt
>  create mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml
>  delete mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt
>  create mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml
>  delete mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-utgard.txt
>  create mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml
> 
>
Rob Herring Aug. 21, 2019, 5:43 p.m. UTC | #4
On Wed, Aug 21, 2019 at 12:24 PM Heiko Stuebner <heiko@sntech.de> wrote:
>
> Am Dienstag, 20. August 2019, 21:59:56 CEST schrieb Rob Herring:
> > This series converts the various Arm Mali GPU bindings to use the DT
> > schema format.
> >
> > The Midgard and Bifrost bindings generate warnings on 'interrupt-names'
> > because there's all different ordering. The Utgard binding generates
> > warnings on Rockchip platforms because 'clock-names' order is reversed.
>
> Are you planning on sending fixes for these, should I just change the
> ordering myself?

I wasn't planning on it. I just add warnings. :)

>
> > Rob Herring (3):
> >   dt-bindings: Convert Arm Mali Midgard GPU to DT schema
> >   dt-bindings: Convert Arm Mali Bifrost GPU to DT schema
> >   dt-bindings: Convert Arm Mali Utgard GPU to DT schema
>
> Acked-by: Heiko Stuebner <heiko@sntech.de>

Thanks.

Rob