| Message ID | 20230522101849.297499-1-biju.das.jz@bp.renesas.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Add Renesas PMIC RAA215300 and built-in RTC support | expand |
Hi all, I am planning to split this series into 3 patch series from next version onwards. Patch set 1: i2c core patch + PMIC patches, as PMIC driver patch depend upon i2c core patch. Patch set 2: RTC patchset Patch set 3: Device tree changes. Hope it is ok to everyone. Cheers, Biju > -----Original Message----- > From: Biju Das <biju.das.jz@bp.renesas.com> > Sent: Monday, May 22, 2023 11:19 AM > To: Wolfram Sang <wsa@kernel.org>; Alessandro Zummo > <a.zummo@towertech.it>; Alexandre Belloni > <alexandre.belloni@bootlin.com>; Rob Herring <robh+dt@kernel.org>; > Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> > Cc: Biju Das <biju.das.jz@bp.renesas.com>; Liam Girdwood > <lgirdwood@gmail.com>; Mark Brown <broonie@kernel.org>; Geert > Uytterhoeven <geert+renesas@glider.be>; Magnus Damm > <magnus.damm@gmail.com>; devicetree@vger.kernel.org; linux- > rtc@vger.kernel.org; linux-kernel@vger.kernel.org; linux-renesas- > soc@vger.kernel.org; Fabrizio Castro <fabrizio.castro.jz@renesas.com> > Subject: [PATCH v5 00/11] Add Renesas PMIC RAA215300 and built-in RTC > support > > 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. > > Enhance i2c_new_ancillary_device() to instantiate a real device. > (eg: Instantiate rtc device from PMIC driver) > > 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://jpn01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatchw > ork.kernel.org%2Fproject%2Flinux-renesas- > soc%2Fpatch%2F20230505091720.115675-1- > biju.das.jz%40bp.renesas.com%2F&data=05%7C01%7Cbiju.das.jz%40bp.renesas. > com%7C086829526c5a4b09386808db5aadf404%7C53d82571da1947e49cb4625a166a4a2 > a%7C0%7C0%7C638203475387281433%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwM > DAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata= > bZIDac5ULa%2F%2BX9u8ZBlXjAaka8vl%2B8tYKOgcXIOpIrk%3D&reserved=0 > > https://jpn01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatchw > ork.kernel.org%2Fproject%2Flinux-renesas- > soc%2Fpatch%2F20230505172530.357455-5- > biju.das.jz%40bp.renesas.com%2F&data=05%7C01%7Cbiju.das.jz%40bp.renesas. > com%7C086829526c5a4b09386808db5aadf404%7C53d82571da1947e49cb4625a166a4a2 > a%7C0%7C0%7C638203475387281433%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwM > DAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata= > aKyxhnzMXfirtEcR3XuGAhl%2FUPc6J0Bkx1iEqdvorB8%3D&reserved=0 > > 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. > > Logs: > [ 15.447305] rtc-isl1208 3-006f: registered as rtc0 > [ 15.479493] rtc-isl1208 3-006f: setting system clock to 2023-04- > 27T19:31:02 UTC (1682623862) > > root@smarc-rzv2l:~# hwclock -r > 2023-04-27 19:33:05.499001+00:00 > root@smarc-rzv2l:~# hwclock -r > 2023-04-27 19:33:06.936688+00:00 > root@smarc-rzv2l:~# > > Biju Das (11): > i2c: Enhance i2c_new_ancillary_device API > 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 > 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 > > .../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/gpu/drm/bridge/adv7511/adv7511_drv.c | 6 +- > drivers/i2c/i2c-core-base.c | 92 ++++++++---- > drivers/media/i2c/adv748x/adv748x-core.c | 2 +- > drivers/media/i2c/adv7604.c | 3 +- > drivers/regulator/Kconfig | 7 + > drivers/regulator/Makefile | 1 + > drivers/regulator/raa215300.c | 102 ++++++++++++++ > drivers/rtc/rtc-isl1208.c | 133 ++++++++++++++---- > include/linux/i2c.h | 3 +- > 13 files changed, 484 insertions(+), 106 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
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. Enhance i2c_new_ancillary_device() to instantiate a real device. (eg: Instantiate rtc device from PMIC driver) 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/20230505091720.115675-1-biju.das.jz@bp.renesas.com/ https://patchwork.kernel.org/project/linux-renesas-soc/patch/20230505172530.357455-5-biju.das.jz@bp.renesas.com/ 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. Logs: [ 15.447305] rtc-isl1208 3-006f: registered as rtc0 [ 15.479493] rtc-isl1208 3-006f: setting system clock to 2023-04-27T19:31:02 UTC (1682623862) root@smarc-rzv2l:~# hwclock -r 2023-04-27 19:33:05.499001+00:00 root@smarc-rzv2l:~# hwclock -r 2023-04-27 19:33:06.936688+00:00 root@smarc-rzv2l:~# Biju Das (11): i2c: Enhance i2c_new_ancillary_device API 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 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 .../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/gpu/drm/bridge/adv7511/adv7511_drv.c | 6 +- drivers/i2c/i2c-core-base.c | 92 ++++++++---- drivers/media/i2c/adv748x/adv748x-core.c | 2 +- drivers/media/i2c/adv7604.c | 3 +- drivers/regulator/Kconfig | 7 + drivers/regulator/Makefile | 1 + drivers/regulator/raa215300.c | 102 ++++++++++++++ drivers/rtc/rtc-isl1208.c | 133 ++++++++++++++---- include/linux/i2c.h | 3 +- 13 files changed, 484 insertions(+), 106 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