* [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver
@ 2022-04-25 20:46 Peter Rosin
2022-04-25 20:46 ` [PATCH v2 1/2] dt-bindings: iio: ti-dac5571: Add ti,dac121c081 Peter Rosin
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Peter Rosin @ 2022-04-25 20:46 UTC (permalink / raw)
To: linux-kernel
Cc: Jonathan Cameron, Lars-Peter Clausen, Rob Herring,
Krzysztof Kozlowski, Sean Nyekjaer, linux-iio, devicetree
Hi!
The new chip works much like the other chips supported by the driver, so
this is just adding another compatible to the list.
Chenages since v1:
- Guenter Roeck noticed elsewhere that my mail setup was botched and
that my patches were clobbered. Hopefully fixed. *blush*
- added tags from Sean Nyekjaer and Rob Herring
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2 1/2] dt-bindings: iio: ti-dac5571: Add ti,dac121c081
2022-04-25 20:46 [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver Peter Rosin
@ 2022-04-25 20:46 ` Peter Rosin
2022-04-25 20:47 ` [PATCH v2 2/2] iio: dac: ti-dac5571: add support for ti,dac121c081 Peter Rosin
2022-04-28 19:44 ` [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver Jonathan Cameron
2 siblings, 0 replies; 8+ messages in thread
From: Peter Rosin @ 2022-04-25 20:46 UTC (permalink / raw)
To: linux-kernel
Cc: Jonathan Cameron, Lars-Peter Clausen, Rob Herring,
Krzysztof Kozlowski, Sean Nyekjaer, linux-iio, devicetree
It works much the same as ti,dac7571, so it fits in the "family" even
if the name is a little bit different.
Reviewed-by: Sean Nyekjaer <sean@geanix.com>
Acked-by: Rob Herring <robh@kernel.org>
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
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v2 2/2] iio: dac: ti-dac5571: add support for ti,dac121c081
2022-04-25 20:46 [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver Peter Rosin
2022-04-25 20:46 ` [PATCH v2 1/2] dt-bindings: iio: ti-dac5571: Add ti,dac121c081 Peter Rosin
@ 2022-04-25 20:47 ` Peter Rosin
2022-04-28 19:44 ` [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver Jonathan Cameron
2 siblings, 0 replies; 8+ messages in thread
From: Peter Rosin @ 2022-04-25 20:47 UTC (permalink / raw)
To: linux-kernel
Cc: Jonathan Cameron, Lars-Peter Clausen, Rob Herring,
Krzysztof Kozlowski, Sean Nyekjaer, linux-iio, devicetree
It fits right in as a single 12-bit DAC.
Reviewed-by: Sean Nyekjaer <sean@geanix.com>
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
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver
2022-04-25 20:46 [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver Peter Rosin
2022-04-25 20:46 ` [PATCH v2 1/2] dt-bindings: iio: ti-dac5571: Add ti,dac121c081 Peter Rosin
2022-04-25 20:47 ` [PATCH v2 2/2] iio: dac: ti-dac5571: add support for ti,dac121c081 Peter Rosin
@ 2022-04-28 19:44 ` Jonathan Cameron
2022-05-01 18:03 ` Jonathan Cameron
2 siblings, 1 reply; 8+ messages in thread
From: Jonathan Cameron @ 2022-04-28 19:44 UTC (permalink / raw)
To: Peter Rosin
Cc: linux-kernel, Lars-Peter Clausen, Rob Herring,
Krzysztof Kozlowski, Sean Nyekjaer, linux-iio, devicetree
On Mon, 25 Apr 2022 22:46:30 +0200
Peter Rosin <peda@axentia.se> wrote:
> Hi!
>
> The new chip works much like the other chips supported by the driver, so
> this is just adding another compatible to the list.
>
> Chenages since v1:
> - Guenter Roeck noticed elsewhere that my mail setup was botched and
> that my patches were clobbered. Hopefully fixed. *blush*
> - added tags from Sean Nyekjaer and Rob Herring
Applied. Thanks
Jonathan
>
> 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(-)
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver
2022-04-28 19:44 ` [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver Jonathan Cameron
@ 2022-05-01 18:03 ` Jonathan Cameron
2022-05-12 8:59 ` Peter Rosin
0 siblings, 1 reply; 8+ messages in thread
From: Jonathan Cameron @ 2022-05-01 18:03 UTC (permalink / raw)
To: Peter Rosin
Cc: linux-kernel, Lars-Peter Clausen, Rob Herring,
Krzysztof Kozlowski, Sean Nyekjaer, linux-iio, devicetree
On Thu, 28 Apr 2022 20:44:39 +0100
Jonathan Cameron <jic23@kernel.org> wrote:
> On Mon, 25 Apr 2022 22:46:30 +0200
> Peter Rosin <peda@axentia.se> wrote:
>
> > Hi!
> >
> > The new chip works much like the other chips supported by the driver, so
> > this is just adding another compatible to the list.
> >
> > Chenages since v1:
> > - Guenter Roeck noticed elsewhere that my mail setup was botched and
> > that my patches were clobbered. Hopefully fixed. *blush*
> > - added tags from Sean Nyekjaer and Rob Herring
>
> Applied. Thanks
Backed out temporarily because they have crossed with a fix (that I'd
forgotten about) and were requiring a non trivial merge in linux-next.
https://patchwork.kernel.org/project/linux-iio/patch/20220324234340.32402-1-laurent.pinchart@ideasonboard.com/
Hopefully that will get resolved in my upstream fairly soon and I can
fix this one up whilst applying it myself.
Thanks,
Jonathan
>
> Jonathan
>
> >
> > 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(-)
> >
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver
2022-05-01 18:03 ` Jonathan Cameron
@ 2022-05-12 8:59 ` Peter Rosin
2022-05-12 13:02 ` Jonathan Cameron
0 siblings, 1 reply; 8+ messages in thread
From: Peter Rosin @ 2022-05-12 8:59 UTC (permalink / raw)
To: Jonathan Cameron
Cc: linux-kernel, Lars-Peter Clausen, Rob Herring,
Krzysztof Kozlowski, Sean Nyekjaer, linux-iio, devicetree
HI!
2022-05-01 at 20:03, Jonathan Cameron wrote:
> On Thu, 28 Apr 2022 20:44:39 +0100
> Jonathan Cameron <jic23@kernel.org> wrote:
>
>> On Mon, 25 Apr 2022 22:46:30 +0200
>> Peter Rosin <peda@axentia.se> wrote:
>>
>>> Hi!
>>>
>>> The new chip works much like the other chips supported by the driver, so
>>> this is just adding another compatible to the list.
>>>
>>> Chenages since v1:
>>> - Guenter Roeck noticed elsewhere that my mail setup was botched and
>>> that my patches were clobbered. Hopefully fixed. *blush*
>>> - added tags from Sean Nyekjaer and Rob Herring
>>
>> Applied. Thanks
>
> Backed out temporarily because they have crossed with a fix (that I'd
> forgotten about) and were requiring a non trivial merge in linux-next.
>
> https://patchwork.kernel.org/project/linux-iio/patch/20220324234340.32402-1-laurent.pinchart@ideasonboard.com/
>
> Hopefully that will get resolved in my upstream fairly soon and I can
> fix this one up whilst applying it myself.
Looks like it might be time to bring this one in from the cold?
Cheers,
Peter
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver
2022-05-12 8:59 ` Peter Rosin
@ 2022-05-12 13:02 ` Jonathan Cameron
2022-05-22 11:57 ` Jonathan Cameron
0 siblings, 1 reply; 8+ messages in thread
From: Jonathan Cameron @ 2022-05-12 13:02 UTC (permalink / raw)
To: Peter Rosin
Cc: Jonathan Cameron, linux-kernel, Lars-Peter Clausen, Rob Herring,
Krzysztof Kozlowski, Sean Nyekjaer, linux-iio, devicetree
On Thu, 12 May 2022 10:59:48 +0200
Peter Rosin <peda@axentia.se> wrote:
> HI!
>
> 2022-05-01 at 20:03, Jonathan Cameron wrote:
> > On Thu, 28 Apr 2022 20:44:39 +0100
> > Jonathan Cameron <jic23@kernel.org> wrote:
> >
> >> On Mon, 25 Apr 2022 22:46:30 +0200
> >> Peter Rosin <peda@axentia.se> wrote:
> >>
> >>> Hi!
> >>>
> >>> The new chip works much like the other chips supported by the driver, so
> >>> this is just adding another compatible to the list.
> >>>
> >>> Chenages since v1:
> >>> - Guenter Roeck noticed elsewhere that my mail setup was botched and
> >>> that my patches were clobbered. Hopefully fixed. *blush*
> >>> - added tags from Sean Nyekjaer and Rob Herring
> >>
> >> Applied. Thanks
> >
> > Backed out temporarily because they have crossed with a fix (that I'd
> > forgotten about) and were requiring a non trivial merge in linux-next.
> >
> > https://patchwork.kernel.org/project/linux-iio/patch/20220324234340.32402-1-laurent.pinchart@ideasonboard.com/
> >
> > Hopefully that will get resolved in my upstream fairly soon and I can
> > fix this one up whilst applying it myself.
>
> Looks like it might be time to bring this one in from the cold?
Nearly. I have a pull request out to Greg. Once he's taken that
I'll fastforward the togreg branch to include the fix and
can then apply this.
Jonathan
>
> Cheers,
> Peter
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver
2022-05-12 13:02 ` Jonathan Cameron
@ 2022-05-22 11:57 ` Jonathan Cameron
0 siblings, 0 replies; 8+ messages in thread
From: Jonathan Cameron @ 2022-05-22 11:57 UTC (permalink / raw)
To: Jonathan Cameron
Cc: Peter Rosin, linux-kernel, Lars-Peter Clausen, Rob Herring,
Krzysztof Kozlowski, Sean Nyekjaer, linux-iio, devicetree
On Thu, 12 May 2022 14:02:32 +0100
Jonathan Cameron <Jonathan.Cameron@Huawei.com> wrote:
> On Thu, 12 May 2022 10:59:48 +0200
> Peter Rosin <peda@axentia.se> wrote:
>
> > HI!
> >
> > 2022-05-01 at 20:03, Jonathan Cameron wrote:
> > > On Thu, 28 Apr 2022 20:44:39 +0100
> > > Jonathan Cameron <jic23@kernel.org> wrote:
> > >
> > >> On Mon, 25 Apr 2022 22:46:30 +0200
> > >> Peter Rosin <peda@axentia.se> wrote:
> > >>
> > >>> Hi!
> > >>>
> > >>> The new chip works much like the other chips supported by the driver, so
> > >>> this is just adding another compatible to the list.
> > >>>
> > >>> Chenages since v1:
> > >>> - Guenter Roeck noticed elsewhere that my mail setup was botched and
> > >>> that my patches were clobbered. Hopefully fixed. *blush*
> > >>> - added tags from Sean Nyekjaer and Rob Herring
> > >>
> > >> Applied. Thanks
> > >
> > > Backed out temporarily because they have crossed with a fix (that I'd
> > > forgotten about) and were requiring a non trivial merge in linux-next.
> > >
> > > https://patchwork.kernel.org/project/linux-iio/patch/20220324234340.32402-1-laurent.pinchart@ideasonboard.com/
> > >
> > > Hopefully that will get resolved in my upstream fairly soon and I can
> > > fix this one up whilst applying it myself.
> >
> > Looks like it might be time to bring this one in from the cold?
>
> Nearly. I have a pull request out to Greg. Once he's taken that
> I'll fastforward the togreg branch to include the fix and
> can then apply this.
Now we are good. Applied to the togreg branch of iio.git and added the
tweak needed to provide data in the of_device_id table.
Unfortunately timing is against me this time, and unless the merge window
is delayed this is now 5.20 material.
Thanks,
Jonathan
>
> Jonathan
>
> >
> > Cheers,
> > Peter
>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2022-05-22 11:49 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-25 20:46 [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver Peter Rosin
2022-04-25 20:46 ` [PATCH v2 1/2] dt-bindings: iio: ti-dac5571: Add ti,dac121c081 Peter Rosin
2022-04-25 20:47 ` [PATCH v2 2/2] iio: dac: ti-dac5571: add support for ti,dac121c081 Peter Rosin
2022-04-28 19:44 ` [PATCH v2 0/2] Add support for texas dac121c081 to the dac5571 driver Jonathan Cameron
2022-05-01 18:03 ` Jonathan Cameron
2022-05-12 8:59 ` Peter Rosin
2022-05-12 13:02 ` Jonathan Cameron
2022-05-22 11:57 ` Jonathan Cameron
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).