[PATCH 0/2] Add support for texas dac121c081 to the dac5571 driver

[PATCH 0/2] Add support for texas dac121c081 to the dac5571 driver

[Peter Rosin]

Hi!

The new chip works much like the other chips supported by the driver, so
this is just adding another compatible to the list.

Cheers,
Peter

Peter Rosin (2):
   dt-bindings: iio: ti-dac5571: Add ti,dac121c081
   iio: dac: ti-dac5571: add support for ti,dac121c081

  Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml | 1 +
  drivers/iio/dac/Kconfig                                   | 2 +-
  drivers/iio/dac/ti-dac5571.c                              | 3 +++
  3 files changed, 5 insertions(+), 1 deletion(-)

-- 
2.20.1

[PATCH 1/2] dt-bindings: iio: ti-dac5571: Add ti,dac121c081

[Peter Rosin]

It works much the same as ti,dac7571, so it fits in the "family" even
if the name is a little bit different.

Signed-off-by: Peter Rosin <peda@axentia.se>
---
  Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml | 1 +
  1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml b/Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml
index 714191724f7c..88298bc43b81 100644
--- a/Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml
+++ b/Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml
@@ -21,6 +21,7 @@ properties:
        - ti,dac5573
        - ti,dac6573
        - ti,dac7573
+      - ti,dac121c081
  
    reg:
      maxItems: 1
-- 
2.20.1

[PATCH 2/2] iio: dac: ti-dac5571: add support for ti,dac121c081

[Peter Rosin]

It fits right in as a single 12-bit DAC.

Signed-off-by: Peter Rosin <peda@axentia.se>
---
  drivers/iio/dac/Kconfig      | 2 +-
  drivers/iio/dac/ti-dac5571.c | 3 +++
  2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/iio/dac/Kconfig b/drivers/iio/dac/Kconfig
index c0bf0d84197f..d6248b188645 100644
--- a/drivers/iio/dac/Kconfig
+++ b/drivers/iio/dac/Kconfig
@@ -417,7 +417,7 @@ config TI_DAC5571
  	help
  	  Driver for the Texas Instruments
  	  DAC5571, DAC6571, DAC7571, DAC5574, DAC6574, DAC7574, DAC5573,
-	  DAC6573, DAC7573, DAC8571, DAC8574.
+	  DAC6573, DAC7573, DAC8571, DAC8574, DAC121C081.
  
  	  If compiled as a module, it will be called ti-dac5571.
  
diff --git a/drivers/iio/dac/ti-dac5571.c b/drivers/iio/dac/ti-dac5571.c
index 4a3b8d875518..ce542bd8b69e 100644
--- a/drivers/iio/dac/ti-dac5571.c
+++ b/drivers/iio/dac/ti-dac5571.c
@@ -13,6 +13,7 @@
   * https://www.ti.com/lit/ds/symlink/dac5573.pdf
   * https://www.ti.com/lit/ds/symlink/dac6573.pdf
   * https://www.ti.com/lit/ds/symlink/dac7573.pdf
+ * https://www.ti.com/lit/ds/symlink/dac121c081.pdf
   */
  
  #include <linux/iio/iio.h>
@@ -394,6 +395,7 @@ static const struct of_device_id dac5571_of_id[] = {
  	{.compatible = "ti,dac5573"},
  	{.compatible = "ti,dac6573"},
  	{.compatible = "ti,dac7573"},
+	{.compatible = "ti,dac121c081"},
  	{}
  };
  MODULE_DEVICE_TABLE(of, dac5571_of_id);
@@ -408,6 +410,7 @@ static const struct i2c_device_id dac5571_id[] = {
  	{"dac5573", quad_8bit},
  	{"dac6573", quad_10bit},
  	{"dac7573", quad_12bit},
+	{"dac121c081", single_12bit},
  	{}
  };
  MODULE_DEVICE_TABLE(i2c, dac5571_id);
-- 
2.20.1

Re: [PATCH 1/2] dt-bindings: iio: ti-dac5571: Add ti,dac121c081

[Sean Nyekjaer]

On Mon, Apr 25, 2022 at 08:02:25AM +0200, Peter Rosin wrote:
> It works much the same as ti,dac7571, so it fits in the "family" even
> if the name is a little bit different.
> 
> Signed-off-by: Peter Rosin <peda@axentia.se>
Reviewed-by: Sean Nyekjaer <sean@geanix.com>
> ---
>  Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml b/Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml
> index 714191724f7c..88298bc43b81 100644
> --- a/Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml
> @@ -21,6 +21,7 @@ properties:
>        - ti,dac5573
>        - ti,dac6573
>        - ti,dac7573
> +      - ti,dac121c081
>    reg:
>      maxItems: 1
> -- 
> 2.20.1
> 

Re: [PATCH 2/2] iio: dac: ti-dac5571: add support for ti,dac121c081

[Sean Nyekjaer]

On Mon, Apr 25, 2022 at 08:02:34AM +0200, Peter Rosin wrote:
> It fits right in as a single 12-bit DAC.
> 
> Signed-off-by: Peter Rosin <peda@axentia.se>
Reviewed-by: Sean Nyekjaer <sean@geanix.com>
> ---
>  drivers/iio/dac/Kconfig      | 2 +-
>  drivers/iio/dac/ti-dac5571.c | 3 +++
>  2 files changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/iio/dac/Kconfig b/drivers/iio/dac/Kconfig
> index c0bf0d84197f..d6248b188645 100644
> --- a/drivers/iio/dac/Kconfig
> +++ b/drivers/iio/dac/Kconfig
> @@ -417,7 +417,7 @@ config TI_DAC5571
>  	help
>  	  Driver for the Texas Instruments
>  	  DAC5571, DAC6571, DAC7571, DAC5574, DAC6574, DAC7574, DAC5573,
> -	  DAC6573, DAC7573, DAC8571, DAC8574.
> +	  DAC6573, DAC7573, DAC8571, DAC8574, DAC121C081.
>  	  If compiled as a module, it will be called ti-dac5571.
> diff --git a/drivers/iio/dac/ti-dac5571.c b/drivers/iio/dac/ti-dac5571.c
> index 4a3b8d875518..ce542bd8b69e 100644
> --- a/drivers/iio/dac/ti-dac5571.c
> +++ b/drivers/iio/dac/ti-dac5571.c
> @@ -13,6 +13,7 @@
>   * https://www.ti.com/lit/ds/symlink/dac5573.pdf
>   * https://www.ti.com/lit/ds/symlink/dac6573.pdf
>   * https://www.ti.com/lit/ds/symlink/dac7573.pdf
> + * https://www.ti.com/lit/ds/symlink/dac121c081.pdf
>   */
>  #include <linux/iio/iio.h>
> @@ -394,6 +395,7 @@ static const struct of_device_id dac5571_of_id[] = {
>  	{.compatible = "ti,dac5573"},
>  	{.compatible = "ti,dac6573"},
>  	{.compatible = "ti,dac7573"},
> +	{.compatible = "ti,dac121c081"},
>  	{}
>  };
>  MODULE_DEVICE_TABLE(of, dac5571_of_id);
> @@ -408,6 +410,7 @@ static const struct i2c_device_id dac5571_id[] = {
>  	{"dac5573", quad_8bit},
>  	{"dac6573", quad_10bit},
>  	{"dac7573", quad_12bit},
> +	{"dac121c081", single_12bit},
>  	{}
>  };
>  MODULE_DEVICE_TABLE(i2c, dac5571_id);
> -- 
> 2.20.1
> 

Re: [PATCH 1/2] dt-bindings: iio: ti-dac5571: Add ti,dac121c081

[Rob Herring]

On Mon, 25 Apr 2022 08:02:25 +0200, Peter Rosin wrote:
> It works much the same as ti,dac7571, so it fits in the "family" even
> if the name is a little bit different.
> 
> Signed-off-by: Peter Rosin <peda@axentia.se>
> ---
>   Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml | 1 +
>   1 file changed, 1 insertion(+)
> 

Acked-by: Rob Herring <robh@kernel.org>