This patch series aims to add support for Renesas PMIC RAA215300 and
built-in RTC found on this PMIC device.
The details of PMIC can be found here[1].
Renesas PMIC RAA215300 exposes two separate i2c devices, one for the main
device and another for rtc device.
PMIC driver enables RTC block and instantiate RTC device using
i2c_new_client_device().
The built-in RTC found on PMIC RAA215300 is the same as ISL1208.
However, the external oscillator bit is inverted on PMIC version
0x11. The PMIC driver detects PMIC version and instantiate appropriate
RTC device.
[1]
https://www.renesas.com/in/en/products/power-power-management/multi-channel-power-management-ics-pmics/ssdsoc-power-management-ics-pmic-and-pmus/raa215300-high-performance-9-channel-pmic-supporting-ddr-memory-built-charger-and-rtc
Ref:
https://patchwork.kernel.org/project/linux-renesas-soc/patch/[email protected]/
https://patchwork.kernel.org/project/linux-renesas-soc/patch/[email protected]/
v6->v7:
* Updated the test logs by running rtc_test.
* Dropped patch#1 and instead using i2c_new_client_device() to instantiate
RTC driver.
* Replaced i2c_new_ancillary_device->i2c_new_client_device for RTC client
instantiation.
* Introduced raa215300_clk_present() to find crystal or clock source is
connected to the PMIC device.
* Registered clk_dev, so that without any code change sibling device can
use the same clk API's for retrieving the clk details.
* The optional irq is shared to sibling device by struct i2c_board_info.
* Added code changes for clearing all blocks except RTC. Currently IRQ
handling only supported in RTC driver. Enabling other blocks will lead
to IRQ storm, if there is any fault.
* In probe() latched registers are cleared and PMIC interrupts are masked
to avoid IRQ storm.
* Dropped Rb tag from Geert for patch#9 and patch#10 as new changes introduced.
* In isl1208_set_xtoscb(), override the xtosc bit only if there is a
mismatch.
* Handling of xtosb_val inverted in probe() and the same change is adapted in
isl1208_set_xtoscb().
* Simplified isl1208_clk_present().
* Removed unnecessary blank line before isl1208_rtc_set_time()
* Dropped parsing the parent node for retrieving clk details as
clk_dev is registered in sibling PMIC driver.
v5->v6:
* Added Rb tag from Hans Verkuil and Geert.
* Updated commit description and comment related to i2c_new_ancillary_device()
* Fixed the issue related to assigning wrong parent device by adding check
for aux_device_name.
* Retained Rb tags for patch#1 as changes are trivial.
* Added Rb tag from Geert for patch#2.
* Updated the example with replacing node name "clock-xtal"->"x2-clock".
* Updated the driver header comment block in C++ style
* Dropped cache for registers.
* Dropped check for I2C functionality.
* Replaced "clock-xtal"->"x2-clock" for x2 node.
* Sorted nodes in alphabetical order.
* Added Rb tag from Geert for patch#6,#10 and #11
* Replaced u8->int for xtosb_val parameter in isl1208_set_xtoscb().
* Introduced isl1208_clk_present() for checking the presence of "xin" and
"clkin" for determining internal oscillator is enabled or not.
* Parsing of parent node is moved from probe->isl1208_clk_present()
* Added comment for parsing parent node for getting clock resource.
* Replaced XOR->NOT to make the operation more clear for the inverted case.
v4->v5:
* Replaced parameter dev->parent in __i2c_new_client_device() and
__i2c_new_dummy_device().
* Improved error message in __i2c_new_dummy_device() by printing device name.
* Updated comment for ancillary's device parent
* Dropped aux_device_name check in i2c_new_ancillary_device().
* Replaced oneOf->enum for clock-names as it is simpler.
* Added Rb tag from Conor for binding patches#3 and #9.
* Added Rb tag from Geert for patches#4, #5 and #6.
* Replaced "unsigned long"->"kernel_ulong_t" in isl1208_id[].
* Fixed the typo in commit description.
* Replaced the variable int_osc_en->xtosb_val for isl1208_set_xtoscb() and
changed the data type from bool->u8.
* Replaced devm_clk_get->devm_clk_get_optional() in probe.
* IS_ERR() related error is propagated and check for NULL to find out
if a clock is present.
* -ENOENT means clock not present, so any other errors are propagated.
* Dropped bool inverted parameter from isl1208_set_xtoscb() instead
using xor to compute the value of xtoscb.
* Added description for clocks in PMIC bindings patch.
* Dropped clk.h and started using of_property_present to check the
clocks.
v3->v4:
* Dropped Rb tag from Geert for patch#1 as there are new changes.
* Introduced __i2c_new_dummy_device() to share the code between
i2c_new_dummy_device and i2c_new_ancillary_device().
* Introduced __i2c_new_client_device() to pass parent dev
parameter, so that the ancillary device can assign its parent during
creation.
* Added minItems to interrupt-names in binding patch.
* Added interrupt-names in conditional schema check.
* Documented clock and clock-names properties.
* Dropped unused name variable from struct isl1208_config.
* Make similar I2C and DT-based matching.
* Drop enum isl1208_id and split the array isl1208_configs[].
* Introduced isl1208_set_xtoscb() to set XTOSCB bit.
* Added support for internal oscillator enable/disable.
* Moved PMIC bindings from mfd->regulator.
* Dropped minItems from reg.
* Dropped renesas,rtc-enabled property and instead used clock-names property
to find RTC is enabled or not.
* Added reg-names in required property.
* Updated the example.
* Moved from mfd->regulator as it doesn't use MFD APIs
* Dropped handling "renesas,rtc-enabled" property and instead used
clock-names to determine RTC is enabled or not and then instantiating
RTC device.
* Added clock nodes.
v2->v3:
* Enhanced i2c_new_ancillary_device() to instantiate a real ancillary_device().
* RTC device is instantiated by PMIC driver and dropped isl1208_probe_helper().
* Added "TYPE_RAA215300_RTC_A0" to handle inverted oscillator bit case.
* Added more detailed description for renesas,rtc-enabled property.
* Added support for handling "renesas,rtc-enabled" property.
* Based on PMIC version, it instantiates rtc device by calling i2c_new_
ancillary_device().
* Updated the logs.
RFC->v2:
* Dropped the cross-links from bindings and used a single compatible
with separate i2c addresses for pmic main and rtc device.
* Dropped patch#4 and split patch#3 from this series and send as
separate patch to ML [2].
* Added RTC platform driver and mfd cell entry to the PMIC driver.RTC
platform driver creates rtc device by using i2c_new_ancillary_device()
and register the rtc device by calling the helper function provided
by rtc-isl2108 driver.
* Updated reg property in bindings.
* Added optional reg-names, interrupts and renesas,rtc-enabled
properties.
* Fixed the node name in the binding example
* Dropped the cross link property renesas,raa215300-rtc.
* Updated the binding example
* Dropped MODULE_SOFTDEP from the driver as it is added in RTC platform
driver.
* Dropped compatible "renesas,raa215300-isl1208" and "renesas,raa215300-pmic" property.
* Updated the comment polarity->bit for External Oscillator.
* Added raa215300_rtc_probe_helper() for registering raa215300_rtc device and
added the helper function isl1208_probe_helper() to share the code.
* Updated pmic device node on the SoM dtsi based on the bindings.
RTC test logs with, out of tree patch for enabling RTC IRQ with IRQ alarm
call back and HW modification done on RZ/G2L SMARC EVK to enable PMIC INT#
by wiring R83 on SoM module and PMOD0 PIN7:
root@smarc-rzg2l:~# /rtc_test.sh
Wed Jun 14 19:30:00 UTC 2023
TAP version 13
1..8
# Starting 8 tests from 1 test cases.
# RUN rtc.date_read ...
# rtctest.c:52:date_read:Current RTC date/time is 14/06/2023 19:30:01.
# OK rtc.date_read
ok 1 rtc.date_read
# RUN rtc.date_read_loop ...
# rtctest.c:95:date_read_loop:Continuously reading RTC time for 30s (with 11ms breaks after every read).
# rtctest.c:122:date_read_loop:Performed 2047 RTC time reads.
# OK rtc.date_read_loop
ok 2 rtc.date_read_loop
# RUN rtc.uie_read ...
# OK rtc.uie_read
ok 3 rtc.uie_read
# RUN rtc.uie_select ...
# OK rtc.uie_select
ok 4 rtc.uie_select
# RUN rtc.alarm_alm_set ...
# rtctest.c:221:alarm_alm_set:Alarm time now set to 19:30:41.
# rtctest.c:241:alarm_alm_set:data: 1a0
# OK rtc.alarm_alm_set
ok 5 rtc.alarm_alm_set
# RUN rtc.alarm_wkalm_set ...
# rtctest.c:281:alarm_wkalm_set:Alarm time now set to 14/06/2023 19:30:44.
# OK rtc.alarm_wkalm_set
ok 6 rtc.alarm_wkalm_set
# RUN rtc.alarm_alm_set_minute ...
# rtctest.c:331:alarm_alm_set_minute:Alarm time now set to 19:31:00.
# rtctest.c:351:alarm_alm_set_minute:data: 1a0
# OK rtc.alarm_alm_set_minute
ok 7 rtc.alarm_alm_set_minute
# RUN rtc.alarm_wkalm_set_minute ...
# rtctest.c:391:alarm_wkalm_set_minute:Alarm time now set to 14/06/2023 19:32:00.
# OK rtc.alarm_wkalm_set_minute
ok 8 rtc.alarm_wkalm_set_minute
# PASSED: 8 / 8 tests passed.
# Totals: pass:8 fail:0 xfail:0 xpass:0 skip:0 error:0
cat/proc/interrupts | grep rtx
78: 30 0 rzg2l-irqc 5 Level rtc-isl1208
Biju Das (10):
regulator: dt-bindings: Add Renesas RAA215300 PMIC bindings
regulator: Add Renesas PMIC RAA215300 driver
arm64: dts: renesas: rzg2l-smarc-som: Enable PMIC and built-in RTC
dt-bindings: rtc: isl1208: Convert to json-schema
dt-bindings: rtc: isil,isl1208: Document clock and clock-names
properties
rtc: isl1208: Drop name variable
rtc: isl1208: Make similar I2C and DT-based matching table
rtc: isl1208: Drop enum isl1208_id and split isl1208_configs[]
rtc: isl1208: Add isl1208_set_xtoscb()
rtc: isl1208: Add support for the built-in RTC on the PMIC RAA215300
.../bindings/regulator/renesas,raa215300.yaml | 85 ++++++++
.../devicetree/bindings/rtc/isil,isl1208.txt | 38 ----
.../devicetree/bindings/rtc/isil,isl1208.yaml | 100 +++++++++
.../boot/dts/renesas/rzg2l-smarc-som.dtsi | 18 ++
drivers/regulator/Kconfig | 7 +
drivers/regulator/Makefile | 1 +
drivers/regulator/raa215300.c | 190 ++++++++++++++++++
drivers/rtc/rtc-isl1208.c | 130 +++++++++---
8 files changed, 499 insertions(+), 70 deletions(-)
create mode 100644 Documentation/devicetree/bindings/regulator/renesas,raa215300.yaml
delete mode 100644 Documentation/devicetree/bindings/rtc/isil,isl1208.txt
create mode 100644 Documentation/devicetree/bindings/rtc/isil,isl1208.yaml
create mode 100644 drivers/regulator/raa215300.c
--
2.25.1
On Fri, Jun 23, 2023 at 03:09:38PM +0100, Biju Das wrote:
> v6->v7:
> * Updated the test logs by running rtc_test.
> * Dropped patch#1 and instead using i2c_new_client_device() to instantiate
> RTC driver.
> * Replaced i2c_new_ancillary_device->i2c_new_client_device for RTC client
> instantiation.
This means that there's now no longer any interdependencies for the
regulator patches AIUI?
Hi Mark,
> -----Original Message-----
> From: Mark Brown <[email protected]>
> Sent: Friday, June 23, 2023 4:06 PM
> To: Biju Das <[email protected]>
> Cc: Wolfram Sang <[email protected]>; Alessandro Zummo
> <[email protected]>; Alexandre Belloni
> <[email protected]>; Rob Herring <[email protected]>;
> Krzysztof Kozlowski <[email protected]>; Liam Girdwood
> <[email protected]>; Geert Uytterhoeven <[email protected]>;
> Magnus Damm <[email protected]>; Trent Piepho <[email protected]>;
> [email protected]; [email protected]; linux-
> [email protected]; [email protected]; Fabrizio
> Castro <[email protected]>
> Subject: Re: [PATCH v7 00/10] Add Renesas PMIC RAA215300 and built-in
> RTC support
>
> On Fri, Jun 23, 2023 at 03:09:38PM +0100, Biju Das wrote:
>
> > v6->v7:
> > * Updated the test logs by running rtc_test.
> > * Dropped patch#1 and instead using i2c_new_client_device() to
> instantiate
> > RTC driver.
> > * Replaced i2c_new_ancillary_device->i2c_new_client_device for RTC
> client
> > instantiation.
>
> This means that there's now no longer any interdependencies for the
> regulator patches AIUI?
Yes, that is correct.
Cheers,
Biju
On Fri, 23 Jun 2023 15:09:38 +0100, Biju Das wrote:
> This patch series aims to add support for Renesas PMIC RAA215300 and
> built-in RTC found on this PMIC device.
>
> The details of PMIC can be found here[1].
>
> Renesas PMIC RAA215300 exposes two separate i2c devices, one for the main
> device and another for rtc device.
>
> [...]
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next
Thanks!
[01/10] regulator: dt-bindings: Add Renesas RAA215300 PMIC bindings
commit: fff8f6b0723159f09eb2c067e626fb96402c0e53
[02/10] regulator: Add Renesas PMIC RAA215300 driver
commit: 7bce16630837c705f72e8fd53a11ae8c236236f4
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
On Fri, 23 Jun 2023 15:09:38 +0100, Biju Das wrote:
> This patch series aims to add support for Renesas PMIC RAA215300 and
> built-in RTC found on this PMIC device.
>
> The details of PMIC can be found here[1].
>
> Renesas PMIC RAA215300 exposes two separate i2c devices, one for the main
> device and another for rtc device.
>
> [...]
Applied, thanks!
[04/10] dt-bindings: rtc: isl1208: Convert to json-schema
commit: ac739bac5201d4308cba2525dacb5da654b3ff31
[05/10] dt-bindings: rtc: isil,isl1208: Document clock and clock-names properties
commit: 138f352556d791d7e0ca3ac9a4f4815123af8c82
[06/10] rtc: isl1208: Drop name variable
commit: 380960c40a1d106bba3476c9a010eaf28195115d
[07/10] rtc: isl1208: Make similar I2C and DT-based matching table
commit: fbc06a53561c64ec6d7f9a1b3bc04597de4cbb2d
[08/10] rtc: isl1208: Drop enum isl1208_id and split isl1208_configs[]
commit: 5923fc75d0dfcebce53894ddada7e2440d756f8b
[09/10] rtc: isl1208: Add isl1208_set_xtoscb()
commit: 262f72b4656e182eefaab91ab24a7575dda5524f
[10/10] rtc: isl1208: Add support for the built-in RTC on the PMIC RAA215300
commit: fdd63f65ac25d0851dade4c7ba94a7a882b8d9c2
Best regards,
--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com