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:   Fri, 18 Mar 2022 11:59:08 +0100
From:   Wolfram Sang <wsa@kernel.org>
To:     Patrick Rudolph <patrick.rudolph@9elements.com>,
        Peter Rosin <peda@axentia.se>
Cc:     linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [v6 2/3] i2c: muxes: pca954x: Add MAX735x/MAX736x support
Message-ID: <YjRl/DUeLIJJxuOi@shikoro>
Mail-Followup-To: Wolfram Sang <wsa@kernel.org>,
        Patrick Rudolph <patrick.rudolph@9elements.com>,
        Peter Rosin <peda@axentia.se>, linux-i2c@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <20220216074613.235725-1-patrick.rudolph@9elements.com>
 <20220216074613.235725-3-patrick.rudolph@9elements.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="7Nma04pQlY1cRCe6"
Content-Disposition: inline
In-Reply-To: <20220216074613.235725-3-patrick.rudolph@9elements.com>
Precedence: bulk


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

On Wed, Feb 16, 2022 at 08:46:11AM +0100, Patrick Rudolph wrote:
> Add support for the following Maxim chips using the existing PCA954x
> driver:
> - MAX7356
> - MAX7357
> - MAX7358
> - MAX7367
> - MAX7368
> - MAX7369
>=20
> All added Maxim chips behave like the PCA954x, where a single SMBUS byte
> write selects up to 8 channels to be bridged to the primary bus.
>=20
> The MAX7357 exposes 6 additional registers at Power-On-Reset and is
> configured to:
>  - Disabled interrupts on bus locked up detection
>  - Enable bus locked-up clearing
>  - Disconnect only locked bus instead of all channels
>=20
> While the MAX7357/MAX7358 have interrupt support, they don't act as
> interrupt controller like the PCA9545 does. Thus don't enable IRQ support
> and handle them like the PCA9548.
>=20
> Tested using the MAX7357 and verified that the stalled bus is disconnected
> while the other channels remain operational.
>=20
> Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>

Peter, are you happy with this patch series?

All the best,

   Wolfram


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

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

iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmI0ZfwACgkQFA3kzBSg
KbYDDw//bVx/JYczUSag1uZnYX1KcJ+ylu1lZWgyL7TEQc8xeBluNsHUdkYXzway
9uGkQIcB6bLCAymsW88MnLCKGQImGnZT3KIxtCnGwp/K5tTzAxAZkVrvLy6zvnnG
xt+/rokDndBEQA8UThOp4iKlI5zCbpkCeNDXuu+AK95o0PNPP1w6smgCg86AnHzS
BpZM3FscZyFpaNdLY2vwvbEY/Po6E/KwJr12u2IAp3h2VCkmpf9L3cBfmY0A+y5u
ddrIwfTsTowa7rEX4HBzlU8Jo1pwN2NAVkeakMx2YE4Oq51aLO1M8w3LHouCCmwt
jZNMlG2R75eDvQlrggWoUHH+rV/nIPVyE29r9jKu+qiPzkqJjbdnjOzX3ftmPCNF
GmGOzTUITCgeyZVvuOzkK9qSYrK/xykM5C7rUMWKAxfUf0+k+CPJ+GJMEQquJ/ci
YRZsgjDsnvID27TNxhyjpUDGBsjG/tZNi+3wj6lbOURdr/1TgWQUfR38BxxtowG5
Ejm8mbXsQZ1cuIvPv7Swyk6soJSol+Cn5LPoN1dNSJgTMkUmztw34KGS46XRGrnU
PH/9CVeTXX4grWtQnfSc88r/6yrAaDHYnUwQOFxfIUteeaD7z0viVLnSXFWtvXXj
ibGjYU11QyrGp9Z4YAwg5khvGOcLoGNAhrlRy/Ew+TQDlyed7os=
=8TIT
-----END PGP SIGNATURE-----

--7Nma04pQlY1cRCe6--