Hi,

Here's a revised patchset that introduces a few updates to the
jz4740-rtc driver.

Patch [1/4] used to break ABI, it does not anymore.
Patch [2/4] did not change, patch [3/4] is new.

Patch [3/4] has been updated to use dev_err_probe(), use __clk_hw_get()
instead of looking up the parent's clock by name, and will now register
the CLK32K clock when the #clock-cells device property is present
instead of doing it based on the compatible string.

V2 had an extra patch to add support for fine-tuning the RTC; but since
it was not tested enough I decided to drop it from the V3 until it's
ready for prime time.

Cheers,
-Paul

Paul Cercueil (4):
  dt-bindings: rtc: Add #clock-cells property
  rtc: jz4740: Use readl_poll_timeout
  rtc: jz4740: Use dev_err_probe()
  rtc: jz4740: Register clock provider for the CLK32K pin

 .../devicetree/bindings/rtc/ingenic,rtc.yaml  | 29 ++++++
 drivers/rtc/Kconfig                           |  2 +-
 drivers/rtc/rtc-jz4740.c                      | 94 ++++++++++++++-----
 3 files changed, 99 insertions(+), 26 deletions(-)

-- 
2.39.0

On Sun, 29 Jan 2023 12:04:38 +0000, Paul Cercueil wrote:
> Here's a revised patchset that introduces a few updates to the
> jz4740-rtc driver.
> 
> Patch [1/4] used to break ABI, it does not anymore.
> Patch [2/4] did not change, patch [3/4] is new.
> 
> Patch [3/4] has been updated to use dev_err_probe(), use __clk_hw_get()
> instead of looking up the parent's clock by name, and will now register
> the CLK32K clock when the #clock-cells device property is present
> instead of doing it based on the compatible string.
> 
> [...]

Applied, thanks!

[1/4] dt-bindings: rtc: Add #clock-cells property
      commit: 4737a703528c769c4fde6b68462f656f91f4ad99
[2/4] rtc: jz4740: Use readl_poll_timeout
      commit: d644b133f78d6d8efd36f7b1703bebca09036f0b
[3/4] rtc: jz4740: Use dev_err_probe()
      commit: ff6fd3770e9687d7b849a0e826a32563bfcb98da
[4/4] rtc: jz4740: Register clock provider for the CLK32K pin
      commit: 5ddfa148de8cf5491fd1c89522c7cad859db8c88

Best regards,

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com