ntp: Consolidate the RTC update implementation
authorThomas Gleixner <tglx@linutronix.de>
Sun, 6 Dec 2020 21:46:21 +0000 (22:46 +0100)
committerThomas Gleixner <tglx@linutronix.de>
Fri, 11 Dec 2020 09:40:53 +0000 (10:40 +0100)
commit76e87d96b30b5fee91b381fbc444a3eabcd9469a
treef1dbbeda81261f514e041e7d2bd1f0ddbca8e88b
parent69eca258c85000564577642ba28335eb4e1df8f0
ntp: Consolidate the RTC update implementation

The code for the legacy RTC and the RTC class based update are pretty much
the same. Consolidate the common parts into one function and just invoke
the actual setter functions.

For RTC class based devices the update code checks whether the offset is
valid for the device, which is usually not the case for the first
invocation. If it's not the same it stores the correct offset and lets the
caller try again. That's not much different from the previous approach
where the first invocation had a pretty low probability to actually hit the
allowed window.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Jason Gunthorpe <jgg@nvidia.com>
Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Link: https://lore.kernel.org/r/20201206220542.355743355@linutronix.de
kernel/time/ntp.c