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:   Sat, 17 Sep 2022 23:02:58 +0200
From:   Wolfram Sang <wsa@kernel.org>
To:     Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc:     Tharun Kumar P <tharunkumar.pasumarthi@microchip.com>,
        linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org,
        krzk@kernel.org, jarkko.nikula@linux.intel.com, robh@kernel.org,
        semen.protsenko@linaro.org, sven@svenpeter.dev, jsd@semihalf.com,
        rafal@milecki.pl, olof@lixom.net, arnd@arndb.de,
        UNGLinuxDriver@microchip.com
Subject: Re: [PATCH v4 i2c-master] i2c: microchip: pci1xxxx: Add driver for
 I2C host controller in multifunction endpoint of pci1xxxx switch
Message-ID: <YyY2Aic+WIylNA02@shikoro>
Mail-Followup-To: Wolfram Sang <wsa@kernel.org>,
        Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
        Tharun Kumar P <tharunkumar.pasumarthi@microchip.com>,
        linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org,
        krzk@kernel.org, jarkko.nikula@linux.intel.com, robh@kernel.org,
        semen.protsenko@linaro.org, sven@svenpeter.dev, jsd@semihalf.com,
        rafal@milecki.pl, olof@lixom.net, arnd@arndb.de,
        UNGLinuxDriver@microchip.com
References: <20220907161143.897289-1-tharunkumar.pasumarthi@microchip.com>
 <YxjHgiqrGauhg1oS@smile.fi.intel.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="HvLyCNN6/FfT9NAp"
Content-Disposition: inline
In-Reply-To: <YxjHgiqrGauhg1oS@smile.fi.intel.com>
Precedence: bulk


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

On Wed, Sep 07, 2022 at 07:32:02PM +0300, Andy Shevchenko wrote:
> On Wed, Sep 07, 2022 at 09:41:43PM +0530, Tharun Kumar P wrote:
> > Microchip pci1xxxx is an unmanaged PCIe3.1a Switch for Consumer,
> > Industrial and Automotive applications. This switch has multiple
> > downstream ports. In one of the Switch's Downstream port, there
> > is a multifunction endpoint for peripherals which includes an I2C
> > host controller. The I2C function in the endpoint operates at 100KHz,
> > 400KHz and 1 MHz and has buffer depth of 128 bytes.
> > This patch provides the I2C controller driver for the I2C function
> > of the switch.
>=20
> There are still possible nit-picks, but it's good enough to be applied
> as is now.
>=20
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Thanks a lot! However,  no need to quote all the 1300+ lines for that
IMHO.


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

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

iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmMmNgEACgkQFA3kzBSg
KbYzJxAAnvIf/2CGGk6vOeU7nzsiriYPVMp4bia4ytyRSdNyBWHV2IbeOsllwtEa
iKbJRORRtCz0IYibzCuWqoYiL0Nz2LX+1dG/hDsBE4kdC4CYri3IGbj6Z+TgUpuV
KWez2RW94h1LncSus+vGBhp/tJ6Yl9AJ73mZS6VkUf6CVRhKDBS+xCdUmougnKgT
QuNbjl/RC0tI6LronN5TuTnFAmZijSJyehLZBaY9uf55OBu6EzTzAT/nVh58iewm
pPwjf7myjh1A3bfDBWZBbcpku5TP1j/XsrofNC1mikMGIW6RmnFSGWX+oUq4/bDP
EgANSHgGB8Gz8rmg0x/m3lSP5VJK1CQirbWL/XGJV7/vrmMg770kvawv12E0DbLI
b4yBHRicw0GOrcVfHIyFuDFJifs65MjH4+SMlx/hMXtNcgL6CKezgBAOfTgHtWbG
BcxRfLWy6oX5Nq6RbPCe3pxvZcH0651O9SsGOtjlfDRaTfJ7Ul9sEVQROojn15Ep
kRwMl+ZDlAGfmJ+xZaauNeKLmMdsbmkKHbKse1rQ06MM1dX0h/4xjGl3sN4KnCvy
jXNMPMxcW3rFvkHPuPZkWpxGcoC4oDkjQNUsGsYdY+uQ3fCYClLUrfwQm+7qp+e7
cjcm8FajFKDB8NmPHY2L9wR6Q6pjIQgaYrGbGFP/7AudGM6f7LE=
=8x0N
-----END PGP SIGNATURE-----

--HvLyCNN6/FfT9NAp--