From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH 1/2] arm64: dts: r8a7795: update register size for thermal Date: Fri, 5 Jan 2018 09:47:11 +0100 Message-ID: References: <20180104213108.24080-1-niklas.soderlund+renesas@ragnatech.se> <20180104213108.24080-2-niklas.soderlund+renesas@ragnatech.se> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <20180104213108.24080-2-niklas.soderlund+renesas-1zkq55x86MTxsAP9Fp7wbw@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: =?UTF-8?Q?Niklas_S=C3=B6derlund?= Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Simon Horman , Magnus Damm , Linux-Renesas List-Id: devicetree@vger.kernel.org On Thu, Jan 4, 2018 at 10:31 PM, Niklas S=C3=B6derlund wrote: > To be able to read fused calibration values from hardware the size of > the register resource of TSC1 needs to be incremented to cover one more > register which holds the information if the calibration values have been > fused or not. > > Signed-off-by: Niklas S=C3=B6derlund Reviewed-by: Geert Uytterhoeven > --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi > @@ -2385,7 +2385,7 @@ > > tsc: thermal@e6198000 { > compatible =3D "renesas,r8a7795-thermal"; > - reg =3D <0 0xe6198000 0 0x68>, > + reg =3D <0 0xe6198000 0 0x6c>, Perhaps we should just make it 0x100? I'd be very surprised if that would be smaller than the granularity of the address decoder circuitry. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k= .org In personal conversations with technical people, I call myself a hacker. Bu= t when I'm talking to journalists I just say "programmer" or something like t= hat. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html