[PATCH v2] dt-bindings: iio: dac: mcp47feb02: Fix I2C address in example

Ariana Lazar posted 1 patch 1 month, 3 weeks ago
Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[PATCH v2] dt-bindings: iio: dac: mcp47feb02: Fix I2C address in example
Posted by Ariana Lazar 1 month, 3 weeks ago
Change example reg value from 0 to 0x60 in order to use a valid I2C address

Signed-off-by: Ariana Lazar <ariana.lazar@microchip.com>
---
Changes in v2:
- drop fixes from commit message
- keep just the address fix
- Link to v1: https://lore.kernel.org/r/20260420-mcp47feb02-fix6-v1-1-ae1808fea11d@microchip.com
---
 Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml b/Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml
index d2466aa6bda2106a8b695347a0edf38462294d03..350e80e4dbe05dd10e3c6fb00f74d58c56c54bc8 100644
--- a/Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml
+++ b/Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml
@@ -280,9 +280,9 @@ examples:
 
         #address-cells = <1>;
         #size-cells = <0>;
-        dac@0 {
+        dac@60 {
           compatible = "microchip,mcp47feb02";
-          reg = <0>;
+          reg = <0x60>;
           vdd-supply = <&vdac_vdd>;
           vref-supply = <&vref_reg>;
 

---
base-commit: d2a4ec19d2a2e54c23b5180e939994d3da4a6b91
change-id: 20260417-mcp47feb02-fix6-bd6694d84750

Best regards,
-- 
Ariana Lazar <ariana.lazar@microchip.com>
Re: [PATCH v2] dt-bindings: iio: dac: mcp47feb02: Fix I2C address in example
Posted by Conor Dooley 1 month, 3 weeks ago
On Wed, Apr 22, 2026 at 02:56:59PM +0300, Ariana Lazar wrote:
> Change example reg value from 0 to 0x60 in order to use a valid I2C address
> 
> Signed-off-by: Ariana Lazar <ariana.lazar@microchip.com>
> ---
> Changes in v2:
> - drop fixes from commit message
> - keep just the address fix
> - Link to v1: https://lore.kernel.org/r/20260420-mcp47feb02-fix6-v1-1-ae1808fea11d@microchip.com

Acked-by: Conor Dooley <conor.dooley@microchip.com>
pw-bot: not-applicable

This really should be in series with your other patches, so that
Jonathan doesn't have to figure out which order they apply in.

> ---
>  Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml b/Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml
> index d2466aa6bda2106a8b695347a0edf38462294d03..350e80e4dbe05dd10e3c6fb00f74d58c56c54bc8 100644
> --- a/Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml
> @@ -280,9 +280,9 @@ examples:
>  
>          #address-cells = <1>;
>          #size-cells = <0>;
> -        dac@0 {
> +        dac@60 {
>            compatible = "microchip,mcp47feb02";
> -          reg = <0>;
> +          reg = <0x60>;
>            vdd-supply = <&vdac_vdd>;
>            vref-supply = <&vref_reg>;
>  
> 
> ---
> base-commit: d2a4ec19d2a2e54c23b5180e939994d3da4a6b91
> change-id: 20260417-mcp47feb02-fix6-bd6694d84750
> 
> Best regards,
> -- 
> Ariana Lazar <ariana.lazar@microchip.com>
> 
Re: [PATCH v2] dt-bindings: iio: dac: mcp47feb02: Fix I2C address in example
Posted by Jonathan Cameron 1 month, 3 weeks ago
On Wed, 22 Apr 2026 17:54:25 +0100
Conor Dooley <conor@kernel.org> wrote:

> On Wed, Apr 22, 2026 at 02:56:59PM +0300, Ariana Lazar wrote:
> > Change example reg value from 0 to 0x60 in order to use a valid I2C address
> > 
> > Signed-off-by: Ariana Lazar <ariana.lazar@microchip.com>
> > ---
> > Changes in v2:
> > - drop fixes from commit message
> > - keep just the address fix
> > - Link to v1: https://lore.kernel.org/r/20260420-mcp47feb02-fix6-v1-1-ae1808fea11d@microchip.com  
> 
> Acked-by: Conor Dooley <conor.dooley@microchip.com>
> pw-bot: not-applicable
> 
> This really should be in series with your other patches, so that
> Jonathan doesn't have to figure out which order they apply in.
There doesn't seem to be an order... Either order of this
or the next one is failing. I'll apply them by hand.
Too many things in flight so I'm keen to merge some stuff.

So please check result.

Applied.

> 
> > ---
> >  Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml b/Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml
> > index d2466aa6bda2106a8b695347a0edf38462294d03..350e80e4dbe05dd10e3c6fb00f74d58c56c54bc8 100644
> > --- a/Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml
> > +++ b/Documentation/devicetree/bindings/iio/dac/microchip,mcp47feb02.yaml
> > @@ -280,9 +280,9 @@ examples:
> >  
> >          #address-cells = <1>;
> >          #size-cells = <0>;
> > -        dac@0 {
> > +        dac@60 {
> >            compatible = "microchip,mcp47feb02";
> > -          reg = <0>;
> > +          reg = <0x60>;
> >            vdd-supply = <&vdac_vdd>;
> >            vref-supply = <&vref_reg>;
> >  
> > 
> > ---
> > base-commit: d2a4ec19d2a2e54c23b5180e939994d3da4a6b91
> > change-id: 20260417-mcp47feb02-fix6-bd6694d84750
> > 
> > Best regards,
> > -- 
> > Ariana Lazar <ariana.lazar@microchip.com>
> >