Hi Rob, On Fri, Jan 27, 2017 at 02:19:13PM -0600, Rob Herring wrote: > On Mon, Jan 23, 2017 at 03:00:45PM +0100, Maxime Ripard wrote: > > The ARM Mali Utgard GPU family is embedded into a number of SoCs from > > Allwinner, Amlogic, Mediatek or Rockchip. > > > > Add a binding for the GPU of that family. > > > > Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > > Signed-off-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> > > > > --- > > > > Changes from v1: > > - Dropped the arm,mali-utgard compatible > > - Made the clocks mandatory > > - Added Linus Walleij Reviewed-by, and the ST compatible for the Mali > > --- > > Documentation/devicetree/bindings/gpu/arm,mali-utgard.txt | 79 ++++++++- > > 1 file changed, 79 insertions(+), 0 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-utgard.txt > > > > diff --git a/Documentation/devicetree/bindings/gpu/arm,mali-utgard.txt b/Documentation/devicetree/bindings/gpu/arm,mali-utgard.txt > > new file mode 100644 > > index 000000000000..ba0edcdd1b00 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/gpu/arm,mali-utgard.txt > > @@ -0,0 +1,79 @@ > > +ARM Mali Utgard GPU > > +=================== > > + > > +Required properties: > > + - compatible > > + * Must be one of the following: > > + + "arm,mali-300" > > + + "arm,mali-400" > > + + "arm,mali-450" > > + * And, optionally, one of the vendor specific compatible: > > + + allwinner,sun4i-a10-mali > > + + allwinner,sun7i-a20-mali > > And stericsson,db8500-mali? Good catch, I'll add it. > > + > > + - reg: Physical base address and length of the GPU registers > > + > > + - interrupts: an entry for each entry in interrupt-names. > > + See ../interrupt-controller/interrupts.txt for details. > > + > > + - interrupt-names: > > + * ppX: Pixel Processor X interrupt (X from 0 to 7) > > + * ppmmuX: Pixel Processor X MMU interrupt (X from 0 to 7) > > + * pp: Pixel Processor broadcast interrupt (mali-450 only) > > + * gp: Geometry Processor interrupt > > + * gpmmu: Geometry Processor MMU interrupt > > + > > + - clocks: an entry for each entry in clock-names > > + - clock-names: > > + * bus: bus clock for the GPU > > + * core: clock driving the GPU itself > > assigned-clocks? I didn't put it in there because it's optional, and was looking orthogonal to me, since it depends on the clock frequency out of reset and out of the bootloader running (and it's documented already as part of the generic clock bindings). Thanks, Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
Attachment:
signature.asc
Description: PGP signature
