From: Wolfram Sang <wsa@kernel.org>
To: Rob Herring <robh@kernel.org>
Cc: "Tomi Valkeinen" <tomi.valkeinen@ideasonboard.com>,
linux-media@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-i2c@vger.kernel.org,
"Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
"Luca Ceresoli" <luca.ceresoli@bootlin.com>,
"Andy Shevchenko" <andriy.shevchenko@intel.com>,
"Matti Vaittinen" <Matti.Vaittinen@fi.rohmeurope.com>,
"Laurent Pinchart" <laurent.pinchart+renesas@ideasonboard.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Peter Rosin" <peda@axentia.se>,
"Liam Girdwood" <lgirdwood@gmail.com>,
"Mark Brown" <broonie@kernel.org>,
"Sakari Ailus" <sakari.ailus@linux.intel.com>,
"Michael Tretter" <m.tretter@pengutronix.de>,
"Hans Verkuil" <hverkuil@xs4all.nl>,
"Mike Pagano" <mpagano@gentoo.org>,
"Krzysztof Hałasa" <khalasa@piap.pl>,
"Marek Vasut" <marex@denx.de>,
"Satish Nagireddy" <satish.nagireddy@getcruise.com>,
"Luca Ceresoli" <luca@lucaceresoli.net>,
"Andy Shevchenko" <andriy.shevchenko@linux.intel.com>
Subject: Re: [PATCH v11 1/7] i2c: add I2C Address Translator (ATR) support
Date: Tue, 25 Apr 2023 21:24:11 +0200 [thread overview]
Message-ID: <ZEgo2+0cA+tD2k7u@sai> (raw)
In-Reply-To: <20230425171347.GB1957523-robh@kernel.org>
[-- Attachment #1: Type: text/plain, Size: 554 bytes --]
> We do support some flags in the upper 16-bits of I2C addresses. Any of
> those possibly needed here?
I2C_OWN_SLAVE_ADDRESS definately not. ATR is only for addresses on the
remote bus while our own address is on the local bus.
I2C_TEN_BIT_ADDRESS, only in theory. I have never seen a 10-bit address
in the wild, so extremely unlikely that ATR hardware does support it.
But then again, there is a slight chance that we add more flags in the
future which might be relevant for ATR? So, u32 is probably a good
thing to use nonetheless.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2023-04-25 19:24 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-21 10:18 [PATCH v11 0/7] i2c-atr and FPDLink Tomi Valkeinen
2023-04-21 10:18 ` [PATCH v11 1/7] i2c: add I2C Address Translator (ATR) support Tomi Valkeinen
2023-04-24 10:31 ` Wolfram Sang
2023-04-24 10:51 ` Tomi Valkeinen
2023-04-24 11:01 ` Tomi Valkeinen
2023-04-24 11:32 ` Wolfram Sang
2023-04-24 11:39 ` Tomi Valkeinen
2023-04-24 11:43 ` Wolfram Sang
2023-04-25 17:13 ` Rob Herring
2023-04-25 19:24 ` Wolfram Sang [this message]
2023-04-21 10:18 ` [PATCH v11 2/7] dt-bindings: media: add TI DS90UB913 FPD-Link III Serializer Tomi Valkeinen
2023-04-21 10:18 ` [PATCH v11 3/7] dt-bindings: media: add TI DS90UB953 " Tomi Valkeinen
2023-04-21 10:18 ` [PATCH v11 4/7] dt-bindings: media: add TI DS90UB960 FPD-Link III Deserializer Tomi Valkeinen
2023-04-25 17:11 ` Rob Herring
2023-04-21 10:18 ` [PATCH v11 5/7] media: i2c: add DS90UB960 driver Tomi Valkeinen
2023-04-21 10:18 ` [PATCH v11 6/7] media: i2c: add DS90UB913 driver Tomi Valkeinen
2023-05-08 5:10 ` kernel test robot
2023-04-21 10:18 ` [PATCH v11 7/7] media: i2c: add DS90UB953 driver Tomi Valkeinen
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=ZEgo2+0cA+tD2k7u@sai \
--to=wsa@kernel.org \
--cc=Matti.Vaittinen@fi.rohmeurope.com \
--cc=andriy.shevchenko@intel.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=hverkuil@xs4all.nl \
--cc=khalasa@piap.pl \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=laurent.pinchart+renesas@ideasonboard.com \
--cc=lgirdwood@gmail.com \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=luca.ceresoli@bootlin.com \
--cc=luca@lucaceresoli.net \
--cc=m.tretter@pengutronix.de \
--cc=marex@denx.de \
--cc=mchehab@kernel.org \
--cc=mpagano@gentoo.org \
--cc=peda@axentia.se \
--cc=robh@kernel.org \
--cc=sakari.ailus@linux.intel.com \
--cc=satish.nagireddy@getcruise.com \
--cc=tomi.valkeinen@ideasonboard.com \
/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.