All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kim Seer Paller <kimseer.paller@analog.com>
To: Andy Shevchenko <andriy.shevchenko@intel.com>
Cc: "Jonathan Cameron" <jic23@kernel.org>,
	"David Lechner" <dlechner@baylibre.com>,
	"Nuno Sá" <nuno.sa@analog.com>,
	"Andy Shevchenko" <andy@kernel.org>,
	"Michael Hennerich" <Michael.Hennerich@analog.com>,
	"Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Philipp Zabel" <p.zabel@pengutronix.de>,
	linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux@analog.com, devicetree@vger.kernel.org
Subject: Re: [PATCH 1/3] iio: ABI: add DAC high_z powerdown mode and current output entries
Date: Fri, 17 Jul 2026 15:58:44 +0800	[thread overview]
Message-ID: <DK0OQSHG4QF2.CCD4OGSWXFBH@analog> (raw)
In-Reply-To: <alkcBaHV0gHY37wR@ashevche-desk.local>

On Fri Jul 17, 2026 at 1:59 AM PST, Andy Shevchenko wrote:
> On Thu, Jul 16, 2026 at 03:59:25PM +0800, Kim Seer Paller wrote:
> > Add high_z powerdown mode for DACs with high impedance output in
> > current mode (IDAC) and 15kohm_to_gnd resistor to GND. Also add
> > out_currentY_powerdown_mode, out_currentY_powerdown_mode_available,
> > and out_currentY_powerdown entries to document current output
> > powerdown support.
>
> ...
>
> >  		125kohm_to_gnd: connected to ground via an 125kOhm resistor,
> >  		500kohm_to_gnd: connected to ground via a 500kOhm resistor,
> >  		640kohm_to_gnd: connected to ground via a 640kOhm resistor,
> > -		three_state: left floating.
> > +		three_state: left floating,
> > +		high_z: left floating.
>
> It may be less churn if just located before three_state, but hey, why?!
> three_state is a synonym to high_z. What's the impediment to use it?

You are right, thanks. I added high_z because that is the term the datasheet
uses for current output powerdown mode. Since three_state means the same 
thing I will drop the high_z addition and use the existing three_state.

>
> -- 
> With Best Regards,
> Andy Shevchenko


  parent reply	other threads:[~2026-07-17  7:59 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-16  7:59 [PATCH 0/3] Add AD5710R/AD5711R DAC Kim Seer Paller
2026-07-16  7:59 ` [PATCH 1/3] iio: ABI: add DAC high_z powerdown mode and current output entries Kim Seer Paller
2026-07-16 17:59   ` Andy Shevchenko
2026-07-16 18:00     ` Andy Shevchenko
2026-07-17  7:58     ` Kim Seer Paller [this message]
2026-07-16  7:59 ` [PATCH 2/3] dt-bindings: iio: dac: add adi,ad5710r.yaml Kim Seer Paller
2026-07-16  8:13   ` sashiko-bot
2026-07-16  7:59 ` [PATCH 3/3] iio: dac: ad5710r: Add driver for AD5710R and AD5711R Kim Seer Paller
2026-07-16  8:12   ` sashiko-bot
2026-07-17  9:01   ` Uwe Kleine-König
2026-07-16 17:58 ` [PATCH 0/3] Add AD5710R/AD5711R DAC Andy Shevchenko
2026-07-17  7:50   ` Kim Seer Paller
2026-07-17  8:10     ` Andy Shevchenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DK0OQSHG4QF2.CCD4OGSWXFBH@analog \
    --to=kimseer.paller@analog.com \
    --cc=Michael.Hennerich@analog.com \
    --cc=andriy.shevchenko@intel.com \
    --cc=andy@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dlechner@baylibre.com \
    --cc=jic23@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@analog.com \
    --cc=nuno.sa@analog.com \
    --cc=p.zabel@pengutronix.de \
    --cc=robh@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.