Received-SPF: pass (google.com: domain of linux-kernel+bounces-84845-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Wed, 28 Feb 2024 11:49:24 +0100
From: Wolfram Sang <wsa@kernel.org>
To: =?utf-8?B?VGjDqW8=?= Lebrun <theo.lebrun@bootlin.com>
Cc: Linus Walleij <linus.walleij@linaro.org>,
	Andi Shyti <andi.shyti@kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
	linux-arm-kernel@lists.infradead.org, linux-i2c@vger.kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-mips@vger.kernel.org,
	Gregory Clement <gregory.clement@bootlin.com>,
	Vladimir Kondratiev <vladimir.kondratiev@mobileye.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Tawfik Bayouk <tawfik.bayouk@mobileye.com>
Subject: Re: [PATCH 09/13] i2c: nomadik: fetch timeout-usecs property from
 devicetree
Message-ID: <Zd8PtLsUc0G8KR97@shikoro>
Mail-Followup-To: Wolfram Sang <wsa@kernel.org>,
	=?utf-8?B?VGjDqW8=?= Lebrun <theo.lebrun@bootlin.com>,
	Linus Walleij <linus.walleij@linaro.org>,
	Andi Shyti <andi.shyti@kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
	linux-arm-kernel@lists.infradead.org, linux-i2c@vger.kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-mips@vger.kernel.org,
	Gregory Clement <gregory.clement@bootlin.com>,
	Vladimir Kondratiev <vladimir.kondratiev@mobileye.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Tawfik Bayouk <tawfik.bayouk@mobileye.com>
References: <20240215-mbly-i2c-v1-0-19a336e91dca@bootlin.com>
 <20240215-mbly-i2c-v1-9-19a336e91dca@bootlin.com>
 <Zd3SJMBp23ybgdsJ@shikoro>
 <CZFWIJE9978P.G3TZC2YIUST9@bootlin.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="FC1UhtSW8OWBZjC4"
Content-Disposition: inline
In-Reply-To: <CZFWIJE9978P.G3TZC2YIUST9@bootlin.com>


--FC1UhtSW8OWBZjC4
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi Th=C3=A9o,

> That sounds good. I have not used this prop in the DTS as it does not
> make much sense for an eval board. The target is production boards.

=2E..

> My upcoming question is how to move forward on this series. I can do the
> patch to i2c_parse_fw_timings() in the next revision. That way it gets
> added alongside the first user of this feature. Would it work for you?

Hmmm, to be honest I have a bit of an issue with the 'no user' problem.
There is a driver which uses this feature, okay. But there is no
upstream hardware which uses this driver with this new feature. This
makes maintaining harder ("Who uses this feature?" - "Someone" - "How do
they use it? Can we modify it?" - "Dunno").

Kind regards,

   Wolfram


--FC1UhtSW8OWBZjC4
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmXfD7AACgkQFA3kzBSg
KbZNpRAAplZcmoN+v+398lrwCBrPt7fW87i3u0pGvvs36OOmYI47d+qOyYjBaEdN
M52Dr0jkq4NrpXVgxXk2ty2m3Tv7F6pnPwKU5+e7TzVeUoCQgJwq/vJV1ppgBY+O
1iNfCKh6ZtzNrhuC6X+Rjxt0dZ/qc0zjjjQP4yUEYDCs5AHqc3gd5YopbJ1N0eGd
YMKEPrY9FntcE5argMTXW1OZ/jlRMhrRziD9d+a3odezohS5kITrTuwg9b+Yckm0
/3paBaWYXFFeAgwYPkg5vEIJHq39Y2ljKG5VhpKfMEdg2F2OWkWgxvOiRxEhlFua
rvHkb3UFHoFx4kyheWXEjx79i5KwP/wiCrMv/gxu1Gz4l6HanCMZfoRSUhsOscbH
0qiQBXust4YSa9BEpJD3CYOf5Z1SrWdACRgYwAmb7yUmIQLqlLbBPjxWfyvCKk5d
nDfz34EdiE8Sc5RWJrzwFv1JOZLwg/L0m0i++8GnqKL5jvTgj0+dcg6ScetIINKW
nayMd+ZqNBoA9g738JhlU/Xycy/CNxRda96mCbnFLZWXsEPX3bv4pyQRKapgkZxt
UIxVw3xtwpifhg7/3EwmV5xKDi3m+OuOM+E8sA0VloVNxnpscANrQVC9V5G3YDr1
GNpgoNp1XnZOTcCBGi1gdUUZDQnwybWDNMn+cdAlUZAM/N3WQe4=
=fDeJ
-----END PGP SIGNATURE-----

--FC1UhtSW8OWBZjC4--