Re: [PATCH 1/2] dt-bindings: gpu: mali-midgard: Add resets property
From: Neil Armstrong
Date: Wed Dec 12 2018 - 04:48:34 EST
On 11/12/2018 17:58, Rob Herring wrote:
> On Mon, Dec 10, 2018 at 7:22 AM Neil Armstrong <narmstrong@xxxxxxxxxxxx> wrote:
>> The Amlogic ARM Mali Midgard requires reset controls to power on and
>> software reset the GPU, adds these as optional in the bindings.
>> Signed-off-by: Neil Armstrong <narmstrong@xxxxxxxxxxxx>
>> Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt | 2 ++
>> 1 file changed, 2 insertions(+)
>> diff --git a/Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt b/Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt
>> index 18a2cde2e5f3..24d83ec952f1 100644
>> --- a/Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt
>> +++ b/Documentation/devicetree/bindings/gpu/arm,mali-midgard.txt
>> @@ -37,6 +37,8 @@ Optional properties:
>> - operating-points-v2 : Refer to Documentation/devicetree/bindings/opp/opp.txt
>> for details.
>> +- resets : Phandle to the reset controls for the Mali Midgard device.
> How many? Sounds like 1, but the dts has 2. There's also a similar
> patch for Allwinner. I don't want to see a different number of
> resets and different names for each vendor. So first figure out how
> many resets the IP block has. You'll have to find someone with access
> to the TRM.
I can answer for Amlogic, they have a reset for the bus interface and the GPU IP itself.
You are asking something impossible ;-) We can assume it has a single reset interface,
but vendors can add more to reset some glue HW. So we will need some variant per-vendor,
sorry to annoy you !
>  https://patchwork.ozlabs.org/patch/1003654/