Re: [PATCH v6 1/6] dt-bindings: iio: dac: ad5504: add output-range and missing gpios

David Lechner <dlechner-rdvid1DuHRBWk0Htik3J/[email protected]>
Newsgroups gmane.linux.kernel.iio,gmane.linux.drivers.devicetree,gmane.linux.kernel
Message-ID <[email protected]>
On 8/22/26 2:36 PM, Taha Ed-Dafili wrote:
>> This is one that is starting to emerging as a common name, so I would
>> call it clear-gpios instead to match the same name on chips with a
>> similar function.
> 
> Worth noting Nuno asked for the opposite in v2 review, specifically
> to match the datasheet's /CLR pin name:

Normally I advocate for matching the pin name too. But there are some common
things like reset-gpios where we try to always use the same name. This clear
input seems to be one of those common things.

> 
> https://lore.kernel.org/linux-iio/a79296dc96215a579c29c5c9eda464f818f050d6.camel-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org/
> 
> Let me know which name you'd prefer given that context, and I'll
> update it in v7.
> 
> Best Regards,
> Taha Ed-Dafili


There is only one binding currently with clr-gpios (adi,ltc2688).

And only one with clear-gpios (adi,ad5791). Plus at least two more coming
soon [1] [2].

In [3], there is a /CLR pin, but we called it reset-gpios because it also
resets the control registers.

I didn't re-check the datasheets on all of these, but hopefully we are being
consistent.

[1]: https://lore.kernel.org/linux-iio/20260820-ad5529r-driver-v9-2-ba62e0b2a816-OyLXuOCK7orQT0dZR+AlfA@public.gmane.org/
[2]: https://lore.kernel.org/linux-iio/20260802-dac8163-work-v3-1-3ecc7bc66d0d-hi6Y0CQ0nG0@public.gmane.org/
[3]: https://lore.kernel.org/linux-iio/20240714133000.5866-5-kimseer.paller-OyLXuOCK7orQT0dZR+AlfA@public.gmane.org/
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.