Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 5 Aug 2020 10:52:36 +0200
From:   <wsa@kernel.org>
To:     Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
Cc:     <linux-i2c@vger.kernel.org>, <devicetree@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>, <robh+dt@kernel.org>,
        <ludovic.desroches@microchip.com>, <nicolas.ferre@microchip.com>,
        <alexandre.belloni@bootlin.com>, <linux@armlinux.org.uk>,
        <kamel.bouhara@bootlin.com>
Subject: Re: [PATCH 0/4] i2c: core: add generic GPIO bus recovery
Message-ID: <20200805085236.GE1229@kunai>
Mail-Followup-To: <wsa@kernel.org>,
        Codrin Ciubotariu <codrin.ciubotariu@microchip.com>,
        <linux-i2c@vger.kernel.org>, <devicetree@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>, <robh+dt@kernel.org>,
        <ludovic.desroches@microchip.com>, <nicolas.ferre@microchip.com>,
        <alexandre.belloni@bootlin.com>, <linux@armlinux.org.uk>,
        <kamel.bouhara@bootlin.com>
References: <20200804095926.205643-1-codrin.ciubotariu@microchip.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="imjhCm/Pyz7Rq5F2"
Content-Disposition: inline
In-Reply-To: <20200804095926.205643-1-codrin.ciubotariu@microchip.com>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


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

Codrin, everyone

> This patch series was previously sent as a RFC. Significant changes
> since RFC:
> - "recovery" pinctrl state marked as deprecared in bindings;
> - move to "gpio" pinctrl state done after the call to prepare_recovery()
>   callback;
> - glitch protection when SDA gpio is taken at initialization;

Thanks for the fast update, now all merged for inclusion into 5.9. I
think it is really good, but to verify and double check, I think two
things would be even better..

One thing, I'll definately be doing is to add this feature to
i2c-sh_mobile.c and scope the results.

The other thing would be to convert the PXA driver and see if our
generic support can help their advanced use case or if we are missing
something. Codrin, do you have maybe time and interest to do that? That
would be awesome!

Happy hacking and kind regards,

   Wolfram


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

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

iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAl8qc1QACgkQFA3kzBSg
KbZCyg//Qg7gQ0nyIFtC1bC47ahB6lQVhGflL+6pItCAXkD7FMoEbFOTFDtDHAcy
/Uz+AmMkv96HY0sI65t4LI2mJZZcOL9WXyO8Aas1/Y7wAB9IxKz3reECmodn305S
PU0v4AlFTI3r0Zck7s4F0MPLQ3wVfNoyk6RYhJpAI8/aPEmC1im9BTq8MfAaXZEB
ghxbnqWLAiNMAwWUBuqQOkH+yD1C5+WRdyiyrWbcwGDnrbZQUmZrsZFarfzG8BHY
8bo4a49Nb1lZu0Y5Y06Iy6EerXEpeOWZza/J0T2pNfxW7yvZpKyDMgE2L2uww6GO
BLdZMPSMJuhVRJmRdS/edJSoQKSG1P3rBUCmLyuv2eoo2HOO567SunGfjuOSboRV
cgyQ6sXyN64YKfhvqtaLB1KAKJuLQ5vpJaqzEVZI3WA/Gn+6mBP4fh3XjhQhDEBt
0n+URh0t8zmdBZvW95+wAzR5IIzNoo7TpIoJAkkkz00eLDOZxEh9iPOAaGARvFKL
4tGLLFkCR4oPd7scvcySNFzqRFNSLlAXeDxcRMH4jddc3lY5Ntbq/mIoV5DUcUKo
R3wnpLzDx7HNGO10dNjBP+1yakGz7WespmHJF8zJf7l1xQ5zIRqVSvTYWV2LmGoB
uGTfGvISBXR5DQKmkk0V8a61Yqw94wckoO8Kn2JOrGzwLT2yu/k=
=nFwm
-----END PGP SIGNATURE-----

--imjhCm/Pyz7Rq5F2--