dt-bindings: rtc: at91: rename rtt bindings file

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

Posted by Alexandre Belloni 4 years, 3 months ago

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 <alexandre.belloni@bootlin.com>
---
 .../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

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

Posted by Nicolas Ferre 4 years, 3 months ago

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 <alexandre.belloni@bootlin.com>

Yes, clearer indeed:
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>

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

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

Posted by Rob Herring 4 years, 3 months ago

On Tue, Mar 8, 2022 at 9:57 AM Alexandre Belloni
<alexandre.belloni@bootlin.com> 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 <alexandre.belloni@bootlin.com>
> ---
>  .../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#

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

Posted by Rob Herring 4 years, 3 months ago

On Wed, Mar 9, 2022 at 3:05 PM Rob Herring <robh+dt@kernel.org> wrote:
>
> On Tue, Mar 8, 2022 at 9:57 AM Alexandre Belloni
> <alexandre.belloni@bootlin.com> 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 <alexandre.belloni@bootlin.com>
> > ---
> >  .../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

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

Posted by Alexandre Belloni 4 years, 3 months ago

On 23/03/2022 09:39:07-0500, Rob Herring wrote:
> On Wed, Mar 9, 2022 at 3:05 PM Rob Herring <robh+dt@kernel.org> wrote:
> >
> > On Tue, Mar 8, 2022 at 9:57 AM Alexandre Belloni
> > <alexandre.belloni@bootlin.com> 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 <alexandre.belloni@bootlin.com>
> > > ---
> > >  .../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