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:   Thu, 11 Aug 2022 23:09:57 +0200
From:   Wolfram Sang <wsa@kernel.org>
To:     luca.ceresoli@bootlin.com, Peter Rosin <peda@axentia.se>
Cc:     linux-doc@vger.kernel.org, linux-i2c@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH 7/9] docs: i2c: i2c-topology: reorder sections more
 logically
Message-ID: <YvVwJf2s45yjUWfM@shikoro>
Mail-Followup-To: Wolfram Sang <wsa@kernel.org>, luca.ceresoli@bootlin.com,
        Peter Rosin <peda@axentia.se>, linux-doc@vger.kernel.org,
        linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20220808141708.1021103-1-luca.ceresoli@bootlin.com>
 <20220808141708.1021103-8-luca.ceresoli@bootlin.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="bwOb1WpJ03KiHaqV"
Content-Disposition: inline
In-Reply-To: <20220808141708.1021103-8-luca.ceresoli@bootlin.com>
Precedence: bulk


--bwOb1WpJ03KiHaqV
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Aug 08, 2022 at 04:17:06PM +0200, luca.ceresoli@bootlin.com wrote:
> From: Luca Ceresoli <luca.ceresoli@bootlin.com>
>=20
> The sequence of sections is a bit confusing here:
>=20
>  * we list the mux locking scheme for existing drivers before introducing
>    what mux locking schemes are
>  * we list the caveats for each locking scheme (which are tricky) before
>    the example of the simple use case
>=20
> Restructure it entirely with the following logic:
>=20
>  * Intro ("I2C muxes and complex topologies")
>  * Locking
>    - mux-locked
>      - example
>      - caveats
>    - parent-locked
>      - example
>      - caveats
>  * Complex examples
>  * Mux type of existing device drivers
>=20
> While there, also apply some other improvements:
>=20
>  * convert the caveat list from a table (with only one column carrying
>    content) to a bullet list.
>  * add a small introductory text to bridge the gap from listing the use
>    cases to telling about the hardware components to handle them and then
>    the device drivers that implement those.
>  * make empty lines usage more uniform
>=20
> Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>

Peter, are you fine with this change?


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

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

iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmL1cCUACgkQFA3kzBSg
KbYt7Q//ftQMPUZDT/hXAu+k28HHX9zb7QRLzTxZxUeXt4YhMUL8gVcWx9UvoWx/
1HSoZnmJhUcnxI4uu/4OFuuAo3P6beqkiMGyWz5LaKgpqJm5JsnLBExct1mIJ9ir
wiNdjMILC+vdbO0b7qlm5EdcPZSDBWMtRoGJu5lm/7fj85JE83hGERz8CVk+kK3D
jDdTDQ8+LoJ5gNec/miySGtCTgVOUVBwzM/j5qnQxLoUZFcmB6xA99FqeniVd/np
PSlXrlnN5aW+zipdHQK+hJGqWEQjZFOIVBACwBOTviiIw/zJuhID+417lzu4sOly
UpBjERW1eb+Jzvs/XeR/eGVq6ujx3n3fQKbmOPAA44Mg8sth90pAbFucGYJpeqay
YToZklQGMWt0loRJF800fDw9JD3rf+rB4L8Pg7ARKeuqszbtYDRUDJz3aQqaIeL1
74PuXY98dTSdZtpP5qPgd1SlPHVTduSFsCfKQt1oyMo+/tytBRyAJLquyFSvXjRe
jn8fHISP9yaRrR1nqeodHqej/q75+ImLZFQBGVau34kQQ/VkE4VLkzD8inKC+024
FkAPcM/90Gfzg3F0F8hYxejRD50KtZWA6SEEiCeuCTuUWL1bYU8ioCfW95E2oXFi
1hrLGwSncCYGWureZzi3xrwr9uJDK3JYRt/wzwUlqjEGAYBlX3U=
=Yf7W
-----END PGP SIGNATURE-----

--bwOb1WpJ03KiHaqV--