[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <11fa6a06-7498-bff3-2b40-3c5eb908f7a5@microchip.com>
Date: Thu, 15 Apr 2021 07:31:36 +0000
From: <Claudiu.Beznea@...rochip.com>
To: <robh@...nel.org>
CC: <Nicolas.Ferre@...rochip.com>, <alexandre.belloni@...tlin.com>,
<Ludovic.Desroches@...rochip.com>, <linux@...linux.org.uk>,
<linux-arm-kernel@...ts.infradead.org>,
<devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 16/24] dt-bindings: atmel-sysreg: add bindings for
sama7g5
On 12.04.2021 19:02, Rob Herring wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> On Fri, Apr 09, 2021 at 02:13:37PM +0300, Claudiu Beznea wrote:
>> Add RAM controller and RAM PHY controller DT bindings.
>>
>> Signed-off-by: Claudiu Beznea <claudiu.beznea@...rochip.com>
>> ---
>> .../devicetree/bindings/arm/atmel-sysregs.txt | 15 ++++++++++++++-
>> 1 file changed, 14 insertions(+), 1 deletion(-)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/atmel-sysregs.txt b/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
>> index 807264a78edc..7cd55a760d41 100644
>> --- a/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
>> +++ b/Documentation/devicetree/bindings/arm/atmel-sysregs.txt
>> @@ -45,7 +45,8 @@ RAMC SDRAM/DDR Controller required properties:
>> "atmel,at91sam9260-sdramc",
>> "atmel,at91sam9g45-ddramc",
>> "atmel,sama5d3-ddramc",
>> - "microchip,sam9x60-ddramc"
>> + "microchip,sam9x60-ddramc",
>> + "microchip,sama7g5-uddrc"
>> - reg: Should contain registers location and length
>>
>> Examples:
>> @@ -55,6 +56,18 @@ Examples:
>> reg = <0xffffe800 0x200>;
>> };
>>
>> +RAMC PHY Controller required properties:
>> +- compatible: Should be "microchip,sama7g5-ddr3phy", "syscon"
>> +- reg: Should contain registers location and length
>> +
>> +Example:
>> +
>> + ddr3phy: ddr3phy@...04000 {
>> + compatible = "microchip,sama7g5-ddr3phy", "syscon";
>> + reg = <0xe3804000 0x1000>;
>> + status = "okay";
>
> Really need an example for this? If so, drop 'status'.
The self-refresh code needs the address for this, this is why I've added
it. The status is not needed so, as you suggested, I will remove it.
Thank you,
Claudiu
>
>> +};
>> +
>> SHDWC Shutdown Controller
>>
>> required properties:
>> --
>> 2.25.1
>>
Powered by blists - more mailing lists