From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 23 Nov 2015 22:51:15 -0800 (PST) From: Sugar Wu To: linux-sunxi Cc: atx@atx.name, linux-clk@vger.kernel.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, gpatchesrdh@mveas.com, mturquette@linaro.org, hdegoede@redhat.com, sboyd@codeaurora.org, mturquette@baylibre.com, emilio@elopez.com.ar, linux@arm.linux.org.uk, edubezval@gmail.com, rui.zhang@intel.com, wens@csie.org, galak@codeaurora.org, ijc+devicetree@hellion.org.uk, mark.rutland@arm.com, pawel.moll@arm.com, robh+dt@kernel.org Message-Id: In-Reply-To: <20151123124356.GW32142@lukather> References: <20151123124356.GW32142@lukather> Subject: Re: [PATCH v2 1/5] ARM: dts: sun8i: Add SID node MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="----=_Part_7799_1307141804.1448347875675" List-ID: ------=_Part_7799_1307141804.1448347875675 Content-Type: multipart/alternative; boundary="----=_Part_7800_1893462329.1448347875675" ------=_Part_7800_1893462329.1448347875675 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On Monday, November 23, 2015 at 8:43:59 PM UTC+8, Maxime Ripard wrote: > > Hi, > > On Mon, Nov 23, 2015 at 09:02:48AM +0100, Josef Gajdusek wrote: > > Add a node describing the Security ID memory to the > > Allwinner H3 .dtsi file. > > > > Signed-off-by: Josef Gajdusek > > > --- > > arch/arm/boot/dts/sun8i-h3.dtsi | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/arch/arm/boot/dts/sun8i-h3.dtsi > b/arch/arm/boot/dts/sun8i-h3.dtsi > > index 0faa38a..58de718 100644 > > --- a/arch/arm/boot/dts/sun8i-h3.dtsi > > +++ b/arch/arm/boot/dts/sun8i-h3.dtsi > > @@ -359,6 +359,13 @@ > > #size-cells = <0>; > > }; > > > > + sid: eeprom@01c14000 { > > + compatible = "allwinner,sun4i-a10-sid"; > > + reg = <0x01c14000 0x400>; > > The datasheet says it's 256 bytes wide, while the size here is of 1kB, > is it intentional? SID memory map is 0x01c14000 ~ 0x01c143FF, include 2048bits efuse space. H3 efuse space is SID_SRAM, its range is 0x01c14200 ~ +0x100. > > Thanks, > Maxime > > -- > Maxime Ripard, Free Electrons > Embedded Linux, Kernel and Android engineering > http://free-electrons.com > ------=_Part_7800_1893462329.1448347875675 Content-Type: text/html; charset=utf-8 Content-Transfer-Encoding: quoted-printable

On Monday, November 23, 2015 at 8:43:59 PM UTC+8, Maxime Ripard wro= te:
Hi,

On Mon, Nov 23, 2015 at 09:02:48AM +0100, Josef Gajdusek wrote:
> Add a node describing the Security ID memory to the
> Allwinner H3 .dtsi file.
>=20
> Signed-off-by: Josef Gajdusek <a...@atx.name>
> ---
> =C2=A0arch/arm/boot/dts/sun8i-h3.dtsi | 7 +++++++
> =C2=A01 file changed, 7 insertions(+)
>=20
> diff --git a/arch/arm/boot/dts/sun8i-h3.dtsi b/arch/arm/boot/= dts/sun8i-h3.dtsi
> index 0faa38a..58de718 100644
> --- a/arch/arm/boot/dts/sun8i-h3.dtsi
> +++ b/arch/arm/boot/dts/sun8i-h3.dtsi
> @@ -359,6 +359,13 @@
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0#size-cells =3D <0>;
> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0};
> =C2=A0
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0sid: eeprom@01c14000 {
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0compatible =3D "allwinner,sun4i-a10-sid";
> +=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0reg =3D <0x01c14000 0x400>;

The datasheet says it's 256 bytes wide, while the size here is of 1= kB,
is it intentional?
SID memory map is 0x01c14000 ~ 0x0= 1c143FF, include 2048bits efuse space.
H3 efuse space is SID_SRAM= , its range is =C2=A00x01c14200 ~ +0x100.
=C2=A0
Thanks,
Maxime

--=20
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.c= om
------=_Part_7800_1893462329.1448347875675-- ------=_Part_7799_1307141804.1448347875675--