Re: [PATCH] arm64: tegra: Enable built-in RTC on Jetson AGX Thor

Thierry Reding <[email protected]>
Newsgroups org.kernel.vger.linux-tegra
Message-ID <aoQ4vGb2-FwwXss9@orome>
On Mon, Aug 17, 2026 at 03:41:31PM +0100, Jon Hunter wrote:
> 
> On 17/08/2026 15:32, Thierry Reding wrote:
> > From: Thierry Reding <[email protected]>
> > 
> > The built-in RTC is mostly useful for testing suspend/resume cycles.
> > 
> > Signed-off-by: Thierry Reding <[email protected]>
> > ---
> >   arch/arm64/boot/dts/nvidia/tegra264-p4071-0000+p3834.dtsi | 6 ++++++
> >   1 file changed, 6 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/nvidia/tegra264-p4071-0000+p3834.dtsi b/arch/arm64/boot/dts/nvidia/tegra264-p4071-0000+p3834.dtsi
> > index c4ba3322ed9a..113ecf297378 100644
> > --- a/arch/arm64/boot/dts/nvidia/tegra264-p4071-0000+p3834.dtsi
> > +++ b/arch/arm64/boot/dts/nvidia/tegra264-p4071-0000+p3834.dtsi
> > @@ -13,6 +13,12 @@ chosen {
> >   		stdout-path = "serial0:115200n8";
> >   	};
> > +	bus@0 {
> > +		rtc@c2c0000 {
> > +			status = "okay";
> > +		};
> > +	};
> > +
> 
> 
> Any reason we should not just enable in tegra264.dtsi by default? I see that
> we do that for tegra234.

This is more of a development type of option and not everybody may want
to have it enabled, especially since it's not always hooked up to do
anything useful. We commonly use it on the devkit for suspend/resume
testing, so it's useful there, but even then there's usually another RTC
on a PMIC or something that's considered the primary one.

So in production I suspect people may not want to enable the built-in
RTC. If we enabled it by default, then people would have to explicitly
disable it again.

I don't feel very strongly either way, and I could easily be swayed.

Thierry
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAmqEOdIACgkQ3SOs138+
s6Fr4w/+PLD0POiyV3a0Z2AcwWegT/paZCc1bnS2IH+KlxcOWJ2M8/R/unzMuELG
o/9XB4cOeesLcoiBDwCugit0qurUZOXu9Q9JiPexu3MMwsx4KuESwngoLM9rB1Yb
rHVDchoOLlVYltjEq1AN3i5LsqIKQm8s1BRZDHaLw6G3M36hl4URJisNTY5uXMri
QCh2t6jTpGDWryA8Gd9ELAomoJqCk39S+SSzwTGDohE7+35ddR4Un4cN/Qw5NcsC
jX8v2KFbQNHtmvq8Ah5tiehbCNkNgjiv6/aQRCIsuauz+F3w7P4fYg4mzBqbnQJP
0f5q63tqrgFmpup2H84xcT7XIUiS9C9T39Vz0yUnaIfte8VBeTeaF3Eg7RtthF1U
NOKETBnho6wMAemfr6jo+YUE238X5rZn34JGSpXZkGREoag2BauZcbfZr8oSiY/2
N/9HxMJC5xV+MpqHI+1xW2LMHLrmeBdX6J2NKTzpG61pmYiB2YDGoc1DFeXjWerg
A3FM0tJpS/OmLdc67Gja3HZ9igGrd8TfzO6J/iWg0zml4Ld5Cxjx/EaeoUo85CFe
r4MsEShSgpOtJh7A3UPJIokyQqiUOy8AAFbx1qgwwkixWrjMDWFOJ17W/acvCA9I
ja39FSiQdU5KVNFIrL2VucDuoWOFurGoqNK18Wd1bOodoN0/8hg=
=+2Vq
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.