From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752164AbdDCIM4 (ORCPT ); Mon, 3 Apr 2017 04:12:56 -0400 Received: from mail-wr0-f182.google.com ([209.85.128.182]:33805 "EHLO mail-wr0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751843AbdDCIMv (ORCPT ); Mon, 3 Apr 2017 04:12:51 -0400 Subject: Re: [PATCH v2 1/5] dt-bindings: gpu: add bindings for the ARM Mali Midgard GPU To: Guillaume Tucker , Rob Herring , Mark Rutland References: Cc: devicetree@vger.kernel.org, Sjoerd Simons , Wookey , linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, John Reitan , Enric Balletbo i Serra , linux-arm-kernel@lists.infradead.org From: Neil Armstrong Organization: Baylibre Message-ID: Date: Mon, 3 Apr 2017 10:12:44 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 04/02/2017 09:59 AM, Guillaume Tucker wrote: > The ARM Mali Midgard GPU family is present in a number of SoCs > from many different vendors such as Samsung Exynos and Rockchip. > > Import the device tree bindings documentation from the r16p0 > release of the Mali Midgard GPU kernel driver: > > https://developer.arm.com/-/media/Files/downloads/mali-drivers/kernel/mali-midgard-gpu/TX011-SW-99002-r16p0-00rel0.tgz > > The following optional bindings have been omitted in this initial > version as they are only used in very specific cases: > > * snoop_enable_smc > * snoop_disable_smc > * jm_config > * power_model > * system-coherency > * ipa-model > > The example has been simplified accordingly. > > The compatible string definition has been limited to > "arm,mali-midgard" to avoid checkpatch.pl warnings and to match > what the driver actually expects (as of r16p0 out-of-tree). > > CC: John Reitan > Signed-off-by: Guillaume Tucker > --- > .../devicetree/bindings/gpu/arm,mali-midgard.txt | 53 ++++++++++++++++++++++ > 1 file changed, 53 insertions(+) > create mode 100644 Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt > > diff --git a/Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt b/Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt > new file mode 100644 > index 000000000000..da8fc6d21bbf > --- /dev/null > +++ b/Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt > @@ -0,0 +1,53 @@ > +# > +# (C) COPYRIGHT 2013-2016 ARM Limited. > +# Copyright (C) 2017 Collabora Ltd > +# > +# This program is free software and is provided to you under the terms of the > +# GNU General Public License version 2 as published by the Free Software > +# Foundation, and any use by you of this program is subject to the terms > +# of such GNU licence. > +# Hi Guillaume, This is unnecessary, please remove. > + > + > +ARM Mali Midgard GPU > +==================== > + > +Required properties: > + > +- compatible : Should be "arm,mali-midgard". > +- reg : Physical base address of the device and length of the register area. > +- interrupts : Contains the three IRQ lines required by Mali Midgard devices. > +- interrupt-names : Contains the names of IRQ resources in the order they were > + provided in the interrupts property. Must contain: "JOB, "MMU", "GPU". Please follow the bindings introduced for the utgard family : https://patchwork.kernel.org/patch/9553745/ - an entry for each mali-midgard revision, i.e. "arm,mali-t820" - an entry for each vendor specific wrapping if necessary, i.e. "amlogic,meson-gxm-mali" - low-case for interrupt names > + > +Optional: > + > +- clocks : Phandle to clock for the Mali Midgard device. > +- clock-names : Shall be "clk_mali". > +- mali-supply : Phandle to regulator for the Mali device. Refer to > + Documentation/devicetree/bindings/regulator/regulator.txt for details. > +- operating-points : Refer to Documentation/devicetree/bindings/power/opp.txt > + for details. Please add : * Must be one of the following: "arm,mali-t820" * And, optionally, one of the vendor specific compatible: "amlogic,meson-gxm-mali" with my Ack for the amlogic platform. > + > +Example for a Mali-T602: > + > +gpu@0xfc010000 { > + compatible = "arm,mali-midgard"; > + reg = <0xfc010000 0x4000>; > + interrupts = <0 36 4>, <0 37 4>, <0 38 4>; > + interrupt-names = "JOB", "MMU", "GPU"; > + > + clocks = <&pclk_mali>; > + clock-names = "clk_mali"; > + mali-supply = <&vdd_mali>; > + operating-points = < > + /* KHz uV */ > + 533000 1250000, > + 450000 1150000, > + 400000 1125000, > + 350000 1075000, > + 266000 1025000, > + 160000 925000, > + 100000 912500, > + >; > +}; > Thanks, Neil