diff mbox series

[v2,2/5] dt-bindings: mfd: mediatek,mt6357: Describe Auxiliary ADC subdev

Message ID 20240604123008.327424-3-angelogioacchino.delregno@collabora.com (mailing list archive)
State Handled Elsewhere
Headers show
Series MediaTek MT6357/8/9 PMIC Auxiliary ADC support | expand

Commit Message

AngeloGioacchino Del Regno June 4, 2024, 12:30 p.m. UTC
Describe the PMIC-integrated Auxiliary Analog to Digital Converter
subdevice node.
Full description is available in the mediatek,mt6359-auxadc.yaml
binding relative to that hardware.

Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
---
 .../devicetree/bindings/mfd/mediatek,mt6357.yaml       | 10 ++++++++++
 1 file changed, 10 insertions(+)

Comments

AngeloGioacchino Del Regno Sept. 12, 2024, 8:52 a.m. UTC | #1
Il 04/06/24 14:30, AngeloGioacchino Del Regno ha scritto:
> Describe the PMIC-integrated Auxiliary Analog to Digital Converter
> subdevice node.
> Full description is available in the mediatek,mt6359-auxadc.yaml
> binding relative to that hardware.
> 

Hello,

I just realized (indeed too late) that while all of the other commits of this
series are upstream, this patch was not picked, causing dts validation warnings.

Should I resend or can you just simply pick it?

Cheers,
Angelo

> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
> ---
>   .../devicetree/bindings/mfd/mediatek,mt6357.yaml       | 10 ++++++++++
>   1 file changed, 10 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/mediatek,mt6357.yaml b/Documentation/devicetree/bindings/mfd/mediatek,mt6357.yaml
> index 37423c2e0fdf..e3513cad25f6 100644
> --- a/Documentation/devicetree/bindings/mfd/mediatek,mt6357.yaml
> +++ b/Documentation/devicetree/bindings/mfd/mediatek,mt6357.yaml
> @@ -37,6 +37,11 @@ properties:
>     "#interrupt-cells":
>       const: 2
>   
> +  adc:
> +    type: object
> +    $ref: /schemas/iio/adc/mediatek,mt6359-auxadc.yaml
> +    unevaluatedProperties: false
> +
>     regulators:
>       type: object
>       $ref: /schemas/regulator/mediatek,mt6357-regulator.yaml
> @@ -83,6 +88,11 @@ examples:
>               interrupt-controller;
>               #interrupt-cells = <2>;
>   
> +            pmic_adc: adc {
> +                    compatible = "mediatek,mt6357-auxadc";
> +                    #io-channel-cells = <1>;
> +            };
> +
>               regulators {
>                   mt6357_vproc_reg: buck-vproc {
>                       regulator-name = "vproc";
Lee Jones Sept. 12, 2024, 3:20 p.m. UTC | #2
On Thu, 12 Sep 2024, AngeloGioacchino Del Regno wrote:

> Il 04/06/24 14:30, AngeloGioacchino Del Regno ha scritto:
> > Describe the PMIC-integrated Auxiliary Analog to Digital Converter
> > subdevice node.
> > Full description is available in the mediatek,mt6359-auxadc.yaml
> > binding relative to that hardware.
> > 
> 
> Hello,
> 
> I just realized (indeed too late) that while all of the other commits of this
> series are upstream, this patch was not picked, causing dts validation warnings.
> 
> Should I resend or can you just simply pick it?

It's lacking a DT review.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/mfd/mediatek,mt6357.yaml b/Documentation/devicetree/bindings/mfd/mediatek,mt6357.yaml
index 37423c2e0fdf..e3513cad25f6 100644
--- a/Documentation/devicetree/bindings/mfd/mediatek,mt6357.yaml
+++ b/Documentation/devicetree/bindings/mfd/mediatek,mt6357.yaml
@@ -37,6 +37,11 @@  properties:
   "#interrupt-cells":
     const: 2
 
+  adc:
+    type: object
+    $ref: /schemas/iio/adc/mediatek,mt6359-auxadc.yaml
+    unevaluatedProperties: false
+
   regulators:
     type: object
     $ref: /schemas/regulator/mediatek,mt6357-regulator.yaml
@@ -83,6 +88,11 @@  examples:
             interrupt-controller;
             #interrupt-cells = <2>;
 
+            pmic_adc: adc {
+                    compatible = "mediatek,mt6357-auxadc";
+                    #io-channel-cells = <1>;
+            };
+
             regulators {
                 mt6357_vproc_reg: buck-vproc {
                     regulator-name = "vproc";