Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 18 Apr 2023 15:06:10 +0200
From:   Wolfram Sang <wsa@kernel.org>
To:     Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Cc:     linux-media@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-i2c@vger.kernel.org,
        Rob Herring <robh+dt@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 =?utf-8?Q?Ha=C5=82asa?= <khalasa@piap.pl>,
        Marek Vasut <marex@denx.de>,
        Satish Nagireddy <satish.nagireddy@getcruise.com>,
        Rob Herring <robh@kernel.org>,
        Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH v10 5/8] dt-bindings: media: add TI DS90UB960 FPD-Link
 III Deserializer
Message-ID: <ZD6VwpRya6SGBAt5@shikoro>
Mail-Followup-To: Wolfram Sang <wsa@kernel.org>,
        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,
        Rob Herring <robh+dt@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 =?utf-8?Q?Ha=C5=82asa?= <khalasa@piap.pl>,
        Marek Vasut <marex@denx.de>,
        Satish Nagireddy <satish.nagireddy@getcruise.com>,
        Rob Herring <robh@kernel.org>,
        Laurent Pinchart <laurent.pinchart@ideasonboard.com>
References: <20230222132907.594690-1-tomi.valkeinen@ideasonboard.com>
 <20230222132907.594690-6-tomi.valkeinen@ideasonboard.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="mi+Z0PG+MUqRZwKk"
Content-Disposition: inline
In-Reply-To: <20230222132907.594690-6-tomi.valkeinen@ideasonboard.com>
Precedence: bulk


--mi+Z0PG+MUqRZwKk
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline


> +  i2c-alias-pool:
> +    $ref: /schemas/types.yaml#/definitions/uint16-array
> +    description:
> +      I2C alias pool is a pool of I2C addresses on the main I2C bus that can be
> +      used to access the remote peripherals on the serializer's I2C bus. The
> +      addresses must be available, not used by any other peripheral. Each
> +      remote peripheral is assigned an alias from the pool, and transactions to
> +      that address will be forwarded to the remote peripheral, with the address
> +      translated to the remote peripheral's real address. This property is not
> +      needed if there are no I2C addressable remote peripherals.

After some initial discussion with Tomi on IRC, this question is
probably more for Luca:

Why is "i2c-alias-pool" in the drivers binding and not a regular i2c
binding? Same question for the implementation of the alias-pool
handling. Shouldn't this be in the i2c-atr library? I'd think managing
the list of aliases would look all the same in the drivers otherwise?


--mi+Z0PG+MUqRZwKk
Content-Type: application/pgp-signature; name="signature.asc"

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

iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmQ+lb4ACgkQFA3kzBSg
KbYYSA/9EHTjpfBICYkwQW1dIEF2SbICH4VCTgofpP1fIAbbZ0s6cCNmk/k2TYph
+ZTjCmBLERErgJVS3LRX7oFokQHC98uDYswJ6U2SVv1Q/qT56/qNLVvsD2d4q4gl
9V7gmoSWIKLzeEthKQYKj7MYdbfwulJfauhm9svSRN6S+6rS/H7jEtaZpYDd5I44
y3zhfNQDul4ktn+n2isKVXqP5IRMgI5WsB5IgGVhH50jy78LqmKn3b1mn/ao1P00
9uCup1vdzsJB0VC4NExNkzRRfYcKHhwMdbU+wAQ+1sUdiO695qKY21iAUuWZGak3
m9bjWdYsOsmLmHtruRniY8srFgQSBH+3Z/o241+pHuFOO42vh/oWpFieRfPJG5vN
n5pVhMU538L0yVBrovAgxVMklqfAlGGXB+Q0suhXd8yzOYcYu9zJjve1f/FxOlEI
nP4vsM3pwQ+G6Is1sEKPe/WxD1W46La/9TVO9nKF61gdeVVNs+UczR60jEJUVgsz
riKuF/rEDz/RqWEmQxO4phmki79l/nYLtNCUtRWvYhWOwl/HERr8ctdyPQyRmHqp
7h5nR0/ragtc7rn4A8pmK0UuQM5gOuANwsz6J5BDSBU73BYhCh4nrAkfNMbIcEQ0
FeoSXAepQjG5Iv3acvPaMdwqamyT83qP2dznc7fom2naH+wkud0=
=Fqdo
-----END PGP SIGNATURE-----

--mi+Z0PG+MUqRZwKk--