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
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 <[email protected]>
---
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
It fits right in as a single 12-bit DAC.
Signed-off-by: Peter Rosin <[email protected]>
---
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
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 <[email protected]>
Reviewed-by: Sean Nyekjaer <[email protected]>
> ---
> 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
>
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 <[email protected]>
> ---
> Documentation/devicetree/bindings/iio/dac/ti,dac5571.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
Acked-by: Rob Herring <[email protected]>
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 <[email protected]>
Reviewed-by: Sean Nyekjaer <[email protected]>
> ---
> 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
>