2022-03-08 18:50:15

by Alexandre Belloni

[permalink] [raw]
Subject: [PATCH] dt-bindings: rtc: at91: rename rtt bindings file

atmel,at91sam9-rtc is a confuing name for this file as it is documenting
the RTT used as an RTC and not the other regular RTC (atmel,at91rm9200-rtc
and atmel,at91sam9x5-rtc)

Signed-off-by: Alexandre Belloni <[email protected]>
---
.../rtc/{atmel,at91sam9-rtc.yaml => atmel,at91sam9260-rtt.yaml} | 0
1 file changed, 0 insertions(+), 0 deletions(-)
rename Documentation/devicetree/bindings/rtc/{atmel,at91sam9-rtc.yaml => atmel,at91sam9260-rtt.yaml} (100%)

diff --git a/Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.yaml b/Documentation/devicetree/bindings/rtc/atmel,at91sam9260-rtt.yaml
similarity index 100%
rename from Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.yaml
rename to Documentation/devicetree/bindings/rtc/atmel,at91sam9260-rtt.yaml
--
2.35.1


2022-03-08 23:22:38

by Nicolas Ferre

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: rtc: at91: rename rtt bindings file

On 08/03/2022 at 16:57, Alexandre Belloni wrote:
> atmel,at91sam9-rtc is a confuing name for this file as it is documenting

confusing ^

> the RTT used as an RTC and not the other regular RTC (atmel,at91rm9200-rtc
> and atmel,at91sam9x5-rtc)
>
> Signed-off-by: Alexandre Belloni <[email protected]>

Yes, clearer indeed:
Acked-by: Nicolas Ferre <[email protected]>

True that this driver's name was confusing from day 1...

Thanks Alexandre, best regards,
Nicolas

> ---
> .../rtc/{atmel,at91sam9-rtc.yaml => atmel,at91sam9260-rtt.yaml} | 0
> 1 file changed, 0 insertions(+), 0 deletions(-)
> rename Documentation/devicetree/bindings/rtc/{atmel,at91sam9-rtc.yaml => atmel,at91sam9260-rtt.yaml} (100%)
>
> diff --git a/Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.yaml b/Documentation/devicetree/bindings/rtc/atmel,at91sam9260-rtt.yaml
> similarity index 100%
> rename from Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.yaml
> rename to Documentation/devicetree/bindings/rtc/atmel,at91sam9260-rtt.yaml
> --
> 2.35.1
>


--
Nicolas Ferre

2022-03-09 22:40:59

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: rtc: at91: rename rtt bindings file

On Tue, Mar 8, 2022 at 9:57 AM Alexandre Belloni
<[email protected]> wrote:
>
> atmel,at91sam9-rtc is a confuing name for this file as it is documenting
> the RTT used as an RTC and not the other regular RTC (atmel,at91rm9200-rtc
> and atmel,at91sam9x5-rtc)
>
> Signed-off-by: Alexandre Belloni <[email protected]>
> ---
> .../rtc/{atmel,at91sam9-rtc.yaml => atmel,at91sam9260-rtt.yaml} | 0
> 1 file changed, 0 insertions(+), 0 deletions(-)
> rename Documentation/devicetree/bindings/rtc/{atmel,at91sam9-rtc.yaml => atmel,at91sam9260-rtt.yaml} (100%)
>
> diff --git a/Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.yaml b/Documentation/devicetree/bindings/rtc/atmel,at91sam9260-rtt.yaml
> similarity index 100%
> rename from Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.yaml
> rename to Documentation/devicetree/bindings/rtc/atmel,at91sam9260-rtt.yaml

Now failing in -next:

./Documentation/devicetree/bindings/rtc/atmel,at91sam9260-rtt.yaml:
$id: relative path/filename doesn't match actual path or filename
expected: http://devicetree.org/schemas/rtc/atmel,at91sam9260-rtt.yaml#

2022-03-24 16:55:03

by Alexandre Belloni

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: rtc: at91: rename rtt bindings file


On 23/03/2022 09:39:07-0500, Rob Herring wrote:
> On Wed, Mar 9, 2022 at 3:05 PM Rob Herring <[email protected]> wrote:
> >
> > On Tue, Mar 8, 2022 at 9:57 AM Alexandre Belloni
> > <[email protected]> wrote:
> > >
> > > atmel,at91sam9-rtc is a confuing name for this file as it is documenting
> > > the RTT used as an RTC and not the other regular RTC (atmel,at91rm9200-rtc
> > > and atmel,at91sam9x5-rtc)
> > >
> > > Signed-off-by: Alexandre Belloni <[email protected]>
> > > ---
> > > .../rtc/{atmel,at91sam9-rtc.yaml => atmel,at91sam9260-rtt.yaml} | 0
> > > 1 file changed, 0 insertions(+), 0 deletions(-)
> > > rename Documentation/devicetree/bindings/rtc/{atmel,at91sam9-rtc.yaml => atmel,at91sam9260-rtt.yaml} (100%)
> > >
> > > diff --git a/Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.yaml b/Documentation/devicetree/bindings/rtc/atmel,at91sam9260-rtt.yaml
> > > similarity index 100%
> > > rename from Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.yaml
> > > rename to Documentation/devicetree/bindings/rtc/atmel,at91sam9260-rtt.yaml
> >
> > Now failing in -next:
> >
> > ./Documentation/devicetree/bindings/rtc/atmel,at91sam9260-rtt.yaml:
> > $id: relative path/filename doesn't match actual path or filename
> > expected: http://devicetree.org/schemas/rtc/atmel,at91sam9260-rtt.yaml#
>
> Still failing...

Sorry, this is fixed now.

>
> Rob

--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

2022-03-25 02:50:11

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: rtc: at91: rename rtt bindings file

On Wed, Mar 9, 2022 at 3:05 PM Rob Herring <[email protected]> wrote:
>
> On Tue, Mar 8, 2022 at 9:57 AM Alexandre Belloni
> <[email protected]> wrote:
> >
> > atmel,at91sam9-rtc is a confuing name for this file as it is documenting
> > the RTT used as an RTC and not the other regular RTC (atmel,at91rm9200-rtc
> > and atmel,at91sam9x5-rtc)
> >
> > Signed-off-by: Alexandre Belloni <[email protected]>
> > ---
> > .../rtc/{atmel,at91sam9-rtc.yaml => atmel,at91sam9260-rtt.yaml} | 0
> > 1 file changed, 0 insertions(+), 0 deletions(-)
> > rename Documentation/devicetree/bindings/rtc/{atmel,at91sam9-rtc.yaml => atmel,at91sam9260-rtt.yaml} (100%)
> >
> > diff --git a/Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.yaml b/Documentation/devicetree/bindings/rtc/atmel,at91sam9260-rtt.yaml
> > similarity index 100%
> > rename from Documentation/devicetree/bindings/rtc/atmel,at91sam9-rtc.yaml
> > rename to Documentation/devicetree/bindings/rtc/atmel,at91sam9260-rtt.yaml
>
> Now failing in -next:
>
> ./Documentation/devicetree/bindings/rtc/atmel,at91sam9260-rtt.yaml:
> $id: relative path/filename doesn't match actual path or filename
> expected: http://devicetree.org/schemas/rtc/atmel,at91sam9260-rtt.yaml#

Still failing...

Rob