Re: [PATCH 3/8] dt-bindings: gpu: mali-midgard: Add h6 mali gpu compatible

Clément Péron
2019-04-11T08:03:58Z

Hi,

On Thu, 11 Apr 2019 at 09:27, Maxime Ripard <maxime.ripard@bootlin.com> wrote:
>
> On Thu, Apr 11, 2019 at 01:25:38AM +0200, Clément Péron wrote:
> > This add the H6 mali compatible in the dt-bindings to later support
> > specific implementation.
> >
> > Signed-off-by: Clément Péron <peron.clem@gmail.com>
> > ---
> > .../devicetree/bindings/gpu/arm,mali-midgard.txt | 8 +++++++-
> > 1 file changed, 7 insertions(+), 1 deletion(-)
> >
> > diff --git a/Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt b/Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt
> > index 2e8bbce35695..9e71146b5c8a 100644
> > --- a/Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt
> > +++ b/Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt
> > @@ -15,6 +15,7 @@ Required properties:
> > + "arm,mali-t860"
> > + "arm,mali-t880"
> > * which must be preceded by one of the following vendor specifics:
> > + + "allwinner,sun50i-h6-mali"
> > + "amlogic,meson-gxm-mali"
> > + "rockchip,rk3288-mali"
> > + "rockchip,rk3399-mali"
> > @@ -49,9 +50,14 @@ Vendor-specific bindings
> > ------------------------
> >
> > The Mali GPU is integrated very differently from one SoC to
> > -another. In order to accomodate those differences, you have the option
> > +another. In order to accommodate those differences, you have the option
> > to specify one more vendor-specific compatible, among:
> >
> > +- "allwinner,sun50i-h6-mali"
> > + Required properties:
> > + - resets: Should contain phandle of :
> > + + GPU reset line
> > +
>
> And clocks, I assume?
Yes you're right clocks are also required for the H6.

Clement
>
> Maxime
>
> --
> Maxime Ripard, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com