From: David Lechner <dlechner@baylibre.com>
To: Conor Dooley <conor@kernel.org>, Jonathan Cameron <jic23@kernel.org>
Cc: "Janani Sunil" <janani.sunil@analog.com>,
"Lars-Peter Clausen" <lars@metafoo.de>,
"Michael Hennerich" <Michael.Hennerich@analog.com>,
"Nuno Sá" <nuno.sa@analog.com>,
"Andy Shevchenko" <andy@kernel.org>,
"Rob Herring" <robh@kernel.org>,
"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>,
"Philipp Zabel" <p.zabel@pengutronix.de>,
"Jonathan Corbet" <corbet@lwn.net>,
"Shuah Khan" <skhan@linuxfoundation.org>,
"Mark Brown" <broonie@kernel.org>,
"Marius Cristea" <marius.cristea@microchip.com>,
"Marcus Folkesson" <marcus.folkesson@gmail.com>,
"Kent Gustavsson" <kent@minoris.se>,
linux-iio@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org,
"Janani Sunil" <jan.sun97@gmail.com>,
linux-spi@vger.kernel.org, "Kent Gustavsson" <nedo80@gmail.com>
Subject: Re: [PATCH v7 2/7] dt-bindings: iio: adc: microchip,mcp3564: Add spi-device-addr
Date: Wed, 29 Jul 2026 16:40:00 -0500 [thread overview]
Message-ID: <ea9c043f-27c9-4095-9488-e81e0bbb5273@baylibre.com> (raw)
In-Reply-To: <20260729-hundredth-suffice-f28afa67bc06@spud>
On 7/29/26 3:47 PM, Conor Dooley wrote:
...
> Only going to reply to this for now, I'll reply to the other thread of
> this conversation later so as not to end up duplicating discussion, but
> I am thinking of things like what if someone puts a mcp3654r and an
> ad5529r on the same chip select? I think that's actually perfectly
> functional on a hardware level, provided the fuses/pins are set up
I'm not sure this works in general since SPI devices aren't going to
have a standard way to specify the address.
For example, ad5529r has the address at bits [14:12] in a 16-bit
"instruction phase" while mcp3654r has the address at [7:6] in the
"command byte".
An ad5529r instruction could inadvertently be interpreted as a
command by mcp3654r. So I don't think they could exist on the
same CS.
> correctly (and if it is not, mixing the microchip devices is possible
> and probably mixing future ADI ones will be too).
>
Yes, I suppose this could be possible, but I don't think they would
be considered a single logical device in that case. So would be
covered by the device@cs,spi-addr binding that we already discussed.
Example:
adc@0,0 {
compatible = "adi,adc1";
...
};
adc@0,1 {
compatible = "adi,adc2";
...
};
next prev parent reply other threads:[~2026-07-29 21:40 UTC|newest]
Thread overview: 54+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-22 7:54 [PATCH v7 0/7] iio: dac: Add support for AD5529R DAC Janani Sunil
2026-07-22 7:54 ` [PATCH v7 1/7] spi: dt-bindings: Add spi-device-addr peripheral property Janani Sunil
2026-07-22 16:39 ` Conor Dooley
2026-07-23 5:48 ` Marcus Folkesson
2026-07-24 23:32 ` Jonathan Cameron
2026-07-25 20:55 ` David Lechner
2026-07-25 22:04 ` Jonathan Cameron
2026-07-28 16:10 ` Conor Dooley
2026-07-28 17:06 ` Nuno Sá
2026-07-28 17:17 ` Conor Dooley
2026-07-28 18:39 ` David Lechner
2026-07-28 21:05 ` Jonathan Cameron
2026-07-28 21:46 ` David Lechner
2026-07-29 22:47 ` Jonathan Cameron
2026-07-28 16:55 ` Nuno Sá
2026-07-22 7:54 ` [PATCH v7 2/7] dt-bindings: iio: adc: microchip,mcp3564: Add spi-device-addr Janani Sunil
2026-07-22 8:05 ` sashiko-bot
2026-07-22 16:40 ` Conor Dooley
2026-07-25 20:57 ` David Lechner
2026-07-25 22:07 ` Jonathan Cameron
2026-07-28 16:00 ` Conor Dooley
2026-07-28 21:17 ` Jonathan Cameron
2026-07-29 20:47 ` Conor Dooley
2026-07-29 21:40 ` David Lechner [this message]
2026-07-30 17:41 ` Conor Dooley
2026-07-29 23:09 ` Jonathan Cameron
2026-07-30 10:25 ` Nuno Sá
2026-07-30 17:34 ` Jonathan Cameron
2026-07-30 18:09 ` Conor Dooley
2026-07-31 15:38 ` Nuno Sá
2026-07-30 18:02 ` Conor Dooley
2026-07-22 7:54 ` [PATCH v7 3/7] iio: adc: mcp3564: Add support for spi-device-addr Janani Sunil
2026-07-22 16:45 ` Conor Dooley
2026-07-22 17:02 ` Marius.Cristea
2026-07-22 17:12 ` Conor Dooley
2026-07-22 17:06 ` Marius.Cristea
2026-07-22 7:54 ` [PATCH v7 4/7] dt-bindings: iio: adc: microchip,mcp3911: Add spi-device-addr Janani Sunil
2026-07-22 8:05 ` sashiko-bot
2026-07-22 16:41 ` Conor Dooley
2026-07-23 5:46 ` Marcus Folkesson
2026-07-22 7:54 ` [PATCH v7 5/7] iio: adc: mcp3911: Add support for spi-device-addr Janani Sunil
2026-07-22 16:41 ` Conor Dooley
2026-07-22 16:43 ` Conor Dooley
2026-07-22 17:18 ` Marius.Cristea
2026-07-22 17:19 ` Marius.Cristea
2026-07-23 5:44 ` Marcus Folkesson
2026-07-22 7:54 ` [PATCH v7 6/7] dt-bindings: iio: dac: Add AD5529R Janani Sunil
2026-07-22 8:11 ` sashiko-bot
2026-07-22 16:39 ` Conor Dooley
2026-07-25 20:38 ` David Lechner
2026-07-22 7:54 ` [PATCH v7 7/7] iio: dac: Add AD5529R DAC driver support Janani Sunil
2026-07-22 8:14 ` sashiko-bot
2026-07-24 23:57 ` Jonathan Cameron
2026-07-25 21:24 ` David Lechner
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=ea9c043f-27c9-4095-9488-e81e0bbb5273@baylibre.com \
--to=dlechner@baylibre.com \
--cc=Michael.Hennerich@analog.com \
--cc=andy@kernel.org \
--cc=broonie@kernel.org \
--cc=conor+dt@kernel.org \
--cc=conor@kernel.org \
--cc=corbet@lwn.net \
--cc=devicetree@vger.kernel.org \
--cc=jan.sun97@gmail.com \
--cc=janani.sunil@analog.com \
--cc=jic23@kernel.org \
--cc=kent@minoris.se \
--cc=krzk+dt@kernel.org \
--cc=lars@metafoo.de \
--cc=linux-doc@vger.kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-spi@vger.kernel.org \
--cc=marcus.folkesson@gmail.com \
--cc=marius.cristea@microchip.com \
--cc=nedo80@gmail.com \
--cc=nuno.sa@analog.com \
--cc=p.zabel@pengutronix.de \
--cc=robh@kernel.org \
--cc=skhan@linuxfoundation.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.