Received-SPF: pass (google.com: domain of linux-kernel+bounces-75219-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Date: Wed, 21 Feb 2024 18:43:55 +0100
From: Wolfram Sang <wsa@kernel.org>
To: =?utf-8?B?VGjDqW8=?= Lebrun <theo.lebrun@bootlin.com>
Cc: Linus Walleij <linus.walleij@linaro.org>, Rob Herring <robh@kernel.org>,
	Andi Shyti <andi.shyti@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 01/13] dt-bindings: i2c: nomadik: add timeout-usecs
 property bindings
Message-ID: <ZdY2WzKbElloXC4-@shikoro>
Mail-Followup-To: Wolfram Sang <wsa@kernel.org>,
	=?utf-8?B?VGjDqW8=?= Lebrun <theo.lebrun@bootlin.com>,
	Linus Walleij <linus.walleij@linaro.org>,
	Rob Herring <robh@kernel.org>, Andi Shyti <andi.shyti@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-1-19a336e91dca@bootlin.com>
 <20240216022704.GB850600-robh@kernel.org>
 <CZ6E24VPJKJG.35LACFD6ZV5KE@bootlin.com>
 <CACRpkdZZhhzg5SY7U5dv_OfLEVejRFom4V9nCfkQXunAw1ZXSw@mail.gmail.com>
 <CZ94LGRSF9KN.15ZO1VRMIQVR8@bootlin.com>
 <CZAX02IL1N1J.2GQR9D73GLRZB@bootlin.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="F60VzApEKksHKb8j"
Content-Disposition: inline
In-Reply-To: <CZAX02IL1N1J.2GQR9D73GLRZB@bootlin.com>


--F60VzApEKksHKb8j
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

Hi,

> > > > i2c-mpc (fsl,timeout) and i2c-gpio (i2c-gpio,timeout-ms). I agree this
> > > > prop has no reason to be compatible-specific.

Anyone up to convert these drivers to the new binding and mark the old
ones as deprecated?

> > > As Rob mentioned this isn't in the kernel schemas but in dtschema, so
> > > you need to patch this:
> > > https://github.com/robherring/dt-schema

@Rob: My memory fails a little bit about these two schemas: we have the
github one for generic bindings, not strictly related to Linux, right?
But why do we have then i2c.txt in ther kernel tree? Why don't we sync
regularly with the generic schema?

> Note: I've sent a draft patch to dt-schema. See:
> https://github.com/devicetree-org/dt-schema/pull/129

I used to argue that you can set this timeout to any value in userspace.
I have been convinced that it might make sense to set it early so it is
in use already when booting. So, for this pull request:

Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

All the best,

   Wolfram


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

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

iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmXWNlsACgkQFA3kzBSg
KbbCIQ//T+UfZwN5sfx3E29io7LplBxBq1Vc1X1yI094wCUlwjSfpIHwZb/eFi3l
JTOHIoWz6fWb8zhtUfsF5VXj+Mz9R2ZaxSZpwp3ZJsn+qdhR+DpulzqhrLS1Vce/
dD4DPsVn4fCeXijUtNFZz8fhfqYTW0/8dN1wAAkFTsREfvB99gYuPGU+Ib9f8ELQ
jP7czMAee4luC2CN2VNmOJZSu9tdEAaxy+ePOZq1PtA2ox6QbMxM98WIURyJ3LCQ
EpHONtSUXhY4wcVH4C0I/gpZuw5AWk3X6QRvIXyR/kA7TerokwK0F+6X0Ug6b0Hy
SpKn7vXsPkD1Up+QlJkFQu7psTqYiHHRNPMf5kr+s9CytknreWfburno1a+LeX1j
dyXXeoTNMRdVjmHgibV/0tAPtgOs0PxzicfsVtDrOTzJ9Qcg5aECGpszIMGD2FDK
w230+coTWbqiGMakQUkfOM7jayHUwC7WEl+xLXpbyjqAKRYwX3VrMxIW8RHRItQd
EdLd/FdBp82t9EGlUCabjlyMJNS1N+rQ8PLvKYPtRzl4/7IpQejYd8yZnaX99G6/
ndJQeRCIt9Wu//PCNkAlQXsyRBkApDk8Ai7tXuJQZC+0Y9Yvuk5/rKRlNwtcoeMX
8AgdyMQcQH6a/XmBm28kmb1s+mkAbwxDrke88y4hvascDGcJUTs=
=DKt0
-----END PGP SIGNATURE-----

--F60VzApEKksHKb8j--