linux-kernel - Re: [PATCH] ARM: at91/dt: at91sam9m10g45ek add rtc node

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 03 Sep 2014 08:09:28 +0200
From:	Erik van Luijk <evanluijk@...eract.nl>
To:	Boris BREZILLON <boris.brezillon@...e-electrons.com>,
	Nicolas Ferre <nicolas.ferre@...el.com>
CC:	Jean-Christophe Plagniol-Villard <plagnioj@...osoft.com>,
	linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
	Alexandre Belloni <alexandre.belloni@...e-electrons.com>
Subject: Re: [PATCH] ARM: at91/dt: at91sam9m10g45ek add rtc node


Hi Boris, Nicolas,

Boris BREZILLON schreef op 2-9-2014 om 18:33:
> Hi Erik,
 >
 > On Tue,  2 Sep 2014 12:52:12 +0200
 > Erik van Luijk <evanluijk@...eract.nl> wrote:
 >
 >> Signed-off-by: Erik van Luijk <evanluijk@...eract.nl>
 >
 > Except for the nitpicks you'll find below, you have my
 >
 > Acked-by: Boris Brezillon <boris.brezillon@...e-electrons.com>
 >
 >> ---
 >>  arch/arm/boot/dts/at91sam9g45.dtsi     | 7 +++++++
 >>  arch/arm/boot/dts/at91sam9m10g45ek.dts | 3 +++
 >>  2 files changed, 10 insertions(+)
 >>
 >> diff --git a/arch/arm/boot/dts/at91sam9g45.dtsi 
b/arch/arm/boot/dts/at91sam9g45.dtsi
 >> index 932a669..bf62c18 100644
 >> --- a/arch/arm/boot/dts/at91sam9g45.dtsi
 >> +++ b/arch/arm/boot/dts/at91sam9g45.dtsi
 >> @@ -1070,6 +1070,13 @@
 >>                  status = "disabled";
 >>              };
 >>
 >> +            rtc@...ffdb0 {
 >> +                compatible = "atmel,at91rm9200-rtc";
 >> +                reg = <0xfffffdb0 0x30>;
 >
 > The RTC registers goes from 0xfffffdb0 to 0xfffffdc0, which means the
 > size is 0x10 and not 0x30. This is not important here, because the
 > following region is marked as reserved, but we prefer to keep the DT
 > consistent with the datasheet.

This is an inconsistency in the datasheet of the AT91SAM9G45 (and at 
least also
AT91SAM9G15, AT91SAM9G25, AT91SAM9G35 and AT91SAM9M10).
In the memory map (fig. 5-1) there is room for 16 bytes.
But in the 'Real-time Clock (RTC) User Interface' (table 15-1) there is 
room for
48 bytes (0x30) excluding the Reserved Registers (which don't belong to 
the RTC
on the AT91SAM9M10). Perhaps Nicolas can draw some attention to this at 
Atmel.

>
 >> +                interrupts = <1 IRQ_TYPE_LEVEL_HIGH 7>;
 >> +                status = "disabled";
 >> +            };
 >> +
 >>              spi0: spi@...a4000 {
 >>                  #address-cells = <1>;
 >>                  #size-cells = <0>;
 >> diff --git a/arch/arm/boot/dts/at91sam9m10g45ek.dts 
b/arch/arm/boot/dts/at91sam9m10g45ek.dts
 >> index 96ccc7d..2dd2347 100644
 >> --- a/arch/arm/boot/dts/at91sam9m10g45ek.dts
 >> +++ b/arch/arm/boot/dts/at91sam9m10g45ek.dts
 >> @@ -70,6 +70,9 @@
 >>
 >>              watchdog@...ffd40 {
 >>                  status = "okay";
 >> +
 >> +            rtc@...ffdb0 {
 >> +                status = "okay";
 >
 > Try to keep SoC and board changes in distinct commits.

Understood.

>
 >>              };
 >>
 >>              mmc0: mmc@...80000 {
 >
 >
 > Best Regards,
 >
 > Boris
 >

Regards,

Erik
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/