Re: [PATCH 3/7] rtc: pm8xxx: add support for uefi offset

Sudeep Holla
2025-01-20T17:13:49Z

On Mon, Jan 20, 2025 at 04:17:36PM +0100, Johan Hovold wrote:
> On Mon, Jan 20, 2025 at 04:15:54PM +0100, Johan Hovold wrote:
> > On Mon, Jan 20, 2025 at 03:10:24PM +0000, Sudeep Holla wrote:
> > > On Mon, Jan 20, 2025 at 03:41:48PM +0100, Johan Hovold wrote:
> > > > On many Qualcomm platforms the PMIC RTC control and time registers are
> > > > read-only so that the RTC time can not be updated. Instead an offset
> > > > needs be stored in some machine-specific non-volatile memory, which the
> > > > driver can take into account.
> > > >
> > > > Add support for storing a 32-bit offset from the GPS time epoch in a
> > > > UEFI variable so that the RTC time can be set on such platforms.
> > > >
> > >
> > > Why can't the UEFI runtime services be used here ?
> > > In short, why is drivers/rtc/rtc-efi.c not working or no attempts are
> > > made to check if that works just fine ?
> >
> > As I alluded to in the commit message, the Qualcomm UEFI implementation
>
> s/commit message/cover letter/
>

Ah OK, Sorry for missing that and for you dealing with this mess 😄.
Never ending or very long list of quirks on Linux for the platform I imagine.

--
Regards,
Sudeep