[RFC PATCH] dt-bindings: mfd: Add Renesas R2A11302FT PMIC

[RFC PATCH] dt-bindings: mfd: Add Renesas R2A11302FT PMIC

[Wolfram Sang]

Internal RFC for now.

Basic binding description for the above PMIC already referenced in
upstream DTs for the Renesas Lager and Koelsch boards.

Do you think we could / should upstream it? Passes dt_binding_check and
dtbs_check.

Fixes:
arch/arm/boot/dts/renesas/r8a7790-lager.dtb: /soc/spi@e6e10000/pmic@0: failed to match any schema with compatible: ['renesas,r2a11302ft']
arch/arm/boot/dts/renesas/r8a7791-koelsch.dtb: /soc/spi@e6e20000/pmic@0: failed to match any schema with compatible: ['renesas,r2a11302ft']

Notes:

Would have loved to add this to 'trivial-devices' but 'spi-cpol' and
'spi-cpha' are not allowed there.

There is no driver and there will likely never be one.

I don't have and couldn't find any datasheet.

Maybe Geert wants to be listed as maintainer? He is the only one who
ever playes with this device. All is fine with me...

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
 .../bindings/mfd/renesas,r2a11302ft.yaml      | 51 +++++++++++++++++++
 1 file changed, 51 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/mfd/renesas,r2a11302ft.yaml

diff --git a/Documentation/devicetree/bindings/mfd/renesas,r2a11302ft.yaml b/Documentation/devicetree/bindings/mfd/renesas,r2a11302ft.yaml
new file mode 100644
index 000000000000..fc94f26f2bcd
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/renesas,r2a11302ft.yaml
@@ -0,0 +1,51 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mfd/renesas,r2a11302ft.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Renesas R2A11302FT Power Supply ICs for R-Car
+
+maintainers:
+  - Wolfram Sang <wsa+renesas@sang-engineering.com>
+
+properties:
+  compatible:
+    const: renesas,r2a11302ft
+
+  reg:
+    maxItems: 1
+
+  spi-max-frequency:
+    maximum: 6000000
+
+  spi-cpol: true
+
+  spi-cpha: true
+
+required:
+  - compatible
+  - reg
+  - spi-cpol
+  - spi-cpha
+
+allOf:
+  - $ref: /schemas/spi/spi-peripheral-props.yaml#
+
+unevaluatedProperties: false
+
+examples:
+  - |
+    spi {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        pmic: pmic@0 {
+                compatible = "renesas,r2a11302ft";
+                reg = <0>;
+                spi-max-frequency = <6000000>;
+                spi-cpol;
+                spi-cpha;
+        };
+    };
+...
-- 
2.47.2

Re: [RFC PATCH] dt-bindings: mfd: Add Renesas R2A11302FT PMIC

[Geert Uytterhoeven]

Hi Wolfram,

On Thu, 2 Oct 2025 at 11:30, Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
> Basic binding description for the above PMIC already referenced in
> upstream DTs for the Renesas Lager and Koelsch boards.
>
> Do you think we could / should upstream it? Passes dt_binding_check and
> dtbs_check.
>
> Fixes:
> arch/arm/boot/dts/renesas/r8a7790-lager.dtb: /soc/spi@e6e10000/pmic@0: failed to match any schema with compatible: ['renesas,r2a11302ft']
> arch/arm/boot/dts/renesas/r8a7791-koelsch.dtb: /soc/spi@e6e20000/pmic@0: failed to match any schema with compatible: ['renesas,r2a11302ft']
>
> Notes:
>
> Would have loved to add this to 'trivial-devices' but 'spi-cpol' and
> 'spi-cpha' are not allowed there.
>
> There is no driver and there will likely never be one.
>
> I don't have and couldn't find any datasheet.
>
> Maybe Geert wants to be listed as maintainer? He is the only one who

Morimoto-san wrote the driver that is part of the R-Car H1 BSP.

> ever playes with this device. All is fine with me...

Am I? ;-)

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Re: [RFC PATCH] dt-bindings: mfd: Add Renesas R2A11302FT PMIC

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 676 bytes --]

Hi Geert,

> > Maybe Geert wants to be listed as maintainer? He is the only one who
> 
> Morimoto-san wrote the driver that is part of the R-Car H1 BSP.

Ah, okay, didn't know that. Thanks for the pointer! But even with that,
it doesn't make sense to upstream the driver for these old boards which
work fine without the driver for years, no?

> > ever playes with this device. All is fine with me...
> 
> Am I? ;-)

According to an old cover-letter, you read out the ID when enabling
MSIOF on Gen2. That made you the expert in my book :) But as said, I can
stay the maintainer, no worries.

Any other comment on the patch?

Happy hacking,

   Wolfram


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [RFC PATCH] dt-bindings: mfd: Add Renesas R2A11302FT PMIC

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 48 bytes --]

> +        pmic: pmic@0 {

The label can go...


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [RFC PATCH] dt-bindings: mfd: Add Renesas R2A11302FT PMIC

[Geert Uytterhoeven]

Hi Wolfram,

On Thu, 2 Oct 2025 at 11:30, Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
> Internal RFC for now.
>
> Basic binding description for the above PMIC already referenced in
> upstream DTs for the Renesas Lager and Koelsch boards.
>
> Do you think we could / should upstream it? Passes dt_binding_check and
> dtbs_check.
>
> Fixes:
> arch/arm/boot/dts/renesas/r8a7790-lager.dtb: /soc/spi@e6e10000/pmic@0: failed to match any schema with compatible: ['renesas,r2a11302ft']
> arch/arm/boot/dts/renesas/r8a7791-koelsch.dtb: /soc/spi@e6e20000/pmic@0: failed to match any schema with compatible: ['renesas,r2a11302ft']
>
> Notes:
>
> Would have loved to add this to 'trivial-devices' but 'spi-cpol' and
> 'spi-cpha' are not allowed there.
>
> There is no driver and there will likely never be one.
>
> I don't have and couldn't find any datasheet.
>
> Maybe Geert wants to be listed as maintainer? He is the only one who
> ever playes with this device. All is fine with me...
>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

Thanks for your patch!

> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mfd/renesas,r2a11302ft.yaml

> +
> +unevaluatedProperties: false

Perhaps this should be dropped? Or that can be done when/if actual
additional properties and subnodes are added.

Regardless:
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds