public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Josua Mayer <josua@solid-run.com>
To: "Russell King (Oracle)" <linux@armlinux.org.uk>
Cc: Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Shawn Guo <shawnguo@kernel.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Fabio Estevam <festevam@gmail.com>,
	NXP Linux Team <linux-imx@nxp.com>,
	Yazan Shhady <yazan.shhady@solid-run.com>,
	Jon Nettleton <jon@solid-run.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] ARM: dts: imx6qdl-hummingboard: Add rtc0 and rtc1 aliases to fix hctosys
Date: Fri, 19 Jan 2024 12:08:00 +0000	[thread overview]
Message-ID: <79f9bd25-a05f-43b2-8d93-5d51adae1824@solid-run.com> (raw)
In-Reply-To: <ZalMsJZKrpwncEDp@shell.armlinux.org.uk>

Am 18.01.24 um 17:07 schrieb Russell King (Oracle):
> On Thu, Jan 18, 2024 at 04:01:10PM +0100, Josua Mayer wrote:
>> HummingBoard has two RTCs, first integrated within SoC that can be used to
>> wake up from sleep - and a second on the carrier board including back-up
>> battery which is intended for keeping time during power-off.
>>
>> Add aliases for both, ensuring that the battery-backed clock is primary
>> rtc and used by default during boot for restoring system time.
> Given that the snvs RTC isn't battery backed, should we even be enabling
> that in DT?
In imx6qdl.dtsi it is not disabled.
According to Jon it is useful because it can wake up the soc from sleep,
whereas the external rtc can't.
>
> Also, have you seen any issues such as:
>
> [    0.933249] rtc-pcf8523 0-0068: failed to set xtal load capacitance: -11
> [    0.933505] rtc-pcf8523: probe of 0-0068 failed with error -11
>
> which seems to be exhibiting itself on my SolidSense board?
Not on my HummingBoard Gate Rev. 1.4., but indeed on my solidsense
unit too, which is probably same age as yours.
Only tested imx6dl-hummingboard2-emmc-som-v15.dtb,
but solidsense one should make no difference.

  reply	other threads:[~2024-01-19 12:08 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-18 15:01 [PATCH] ARM: dts: imx6qdl-hummingboard: Add rtc0 and rtc1 aliases to fix hctosys Josua Mayer
2024-01-18 15:10 ` Fabio Estevam
2024-01-18 16:07 ` Russell King (Oracle)
2024-01-19 12:08   ` Josua Mayer [this message]
2024-01-19 13:46     ` Josua Mayer
2024-01-19 14:33       ` Russell King (Oracle)
2024-01-29 19:26         ` Josua Mayer
2024-01-29 22:07           ` Russell King (Oracle)
2024-02-05  9:56 ` Shawn Guo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=79f9bd25-a05f-43b2-8d93-5d51adae1824@solid-run.com \
    --to=josua@solid-run.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=festevam@gmail.com \
    --cc=jon@solid-run.com \
    --cc=kernel@pengutronix.de \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-imx@nxp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=robh+dt@kernel.org \
    --cc=s.hauer@pengutronix.de \
    --cc=shawnguo@kernel.org \
    --cc=yazan.shhady@solid-run.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox