Received: by 2002:a25:4158:0:0:0:0:0 with SMTP id o85csp2102940yba; Mon, 15 Apr 2019 05:07:42 -0700 (PDT) X-Google-Smtp-Source: APXvYqz9npOyCJiT2+4IlZpcr5s0bkd5LqkJgaoVNTTB8mszvxcWjwd/LycvtcUwLNT3kvuVphOo X-Received: by 2002:a63:d5f:: with SMTP id 31mr67178061pgn.208.1555330062297; Mon, 15 Apr 2019 05:07:42 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1555330062; cv=none; d=google.com; s=arc-20160816; b=Fle+IZa9gm5eZoQIOOz94jEevX3kvY419jKVVDFj4atM9xnZIzPOlYOTNY4NmNbuhR wTv9t1xVA5Ur0L7fYsQDYGx+910kwMrLi1maEQ/fTA77AQf5XTkfRGD5s0Qvg2l2PVkP uAz4CttTa016mj18xpxf6T5J4nfDbNHlmnN5fdrZN0grTCGcTZIybQm0bVrUPCdwPgI/ Xpa91EZXeUI9TVIPERdTNqJ34chuzDAmyyO1LSRAG41YAU7uC2477E88WedNHDcadzcb lbX/dCDJRKRTswLMcI1AJDPa7XExXgAGNP4PskutTaSGbXjVHt5EwnU6rCzcBuZ/jrOP WZZg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=W/DIk4MEgMidI02Vsy6h/dTG+5hi4C8s4I74x9wIPe4=; b=cXHxe1u9qSpW1HQK9qOt9X4AHC258DyyHJCrX8zD0pSOYH0luxiDe+A9tilEIGLSIA 5SSXh49Z9u15A4pK3YGt9AW1tFik29oo13TYPJBNOu78oam9bsSarc+Deex6trL/r49F 71cgAFIOf00NJkrYqN+No7RZ4YyHGIF2EXlvHvrgGYoVkO/P7fasH09mxHj9XZx99f5P eGl/oNQ1jUIOmeLdL3BJgdhWo+2ZhXa3gbeL3XtNIOJx9bppxXcEF4p9BZmxx7Z3Nx1Z MY2HO6q2NXocyhC+WwLcxll6+EWrMvSPhCCf4EWuJCTCs6tmnqUrNNxQz4A1nhPnG7S9 ucMQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f20si45248814pfd.51.2019.04.15.05.07.26; Mon, 15 Apr 2019 05:07:42 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727418AbfDOMGP (ORCPT + 99 others); Mon, 15 Apr 2019 08:06:15 -0400 Received: from sauhun.de ([88.99.104.3]:41152 "EHLO pokefinder.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726094AbfDOMGO (ORCPT ); Mon, 15 Apr 2019 08:06:14 -0400 Received: from localhost (p54B331F8.dip0.t-ipconnect.de [84.179.49.248]) by pokefinder.org (Postfix) with ESMTPSA id BBF772C0114; Mon, 15 Apr 2019 14:06:11 +0200 (CEST) Date: Mon, 15 Apr 2019 14:06:11 +0200 From: Wolfram Sang To: Andy Shevchenko Cc: Wolfram Sang , linux-i2c@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Peter Rosin , Stefan Lengfeld , linux-omap@vger.kernel.org, linux-tegra@vger.kernel.org, Linus Walleij Subject: Re: [PATCH 00/12] i2c: core: introduce atomic transfers Message-ID: <20190415120611.euidifhshkf5yqns@ninjato> References: <20190403124019.8947-1-wsa+renesas@sang-engineering.com> <20190403131510.GE9224@smile.fi.intel.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="rigvuvst2m7kwesb" Content-Disposition: inline In-Reply-To: <20190403131510.GE9224@smile.fi.intel.com> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --rigvuvst2m7kwesb Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Apr 03, 2019 at 04:15:10PM +0300, Andy Shevchenko wrote: > On Wed, Apr 03, 2019 at 02:40:07PM +0200, Wolfram Sang wrote: > > This series adds support for very late atomic transfers to the I2C subs= ystem. > > It finally reached a state which I think is ready-to-apply. This is mai= nly > > because of two things: > >=20 > > a) we decided to respect the current locking scheme and to not give ato= mic > > transfers a priority. The code needed for that would have been either > > incomplete or very invasive. And we cannot guarantee successful transfe= rs > > anyhow. See [1] for the discussion and other write-ups for design choic= es. > >=20 > > b) thanks to a discussion with Peter Zijlstra[2], the conditions when t= o allow > > atomic transfers became much clearer. The new helper i2c_in_atomic_xfer= _mode() > > adds readability, too. > >=20 > > In detail, changes since RFC v2: > >=20 > > * dropped coding style patch because already applied > > * added new patch 1 to drop in_atomic() and have better conditions when > > to enter the atomic path > > * added support to the mux-core > > * simplified omap conversion a little > > * added new conversions for ocores, stu300, and algo-bit/gpio > > * typo corrections found by Simon and Stefan > > * added tags to drivers > > * dropped tags from core patches because that part changed too much > >=20 > > All tested on a Renesas Lager board (R-Car H2). Sadly, the i2c-sh_mobil= e driver > > cannot be converted now because of other work needed first. I tested wi= th the > > i2c-gpio driver, though. The other driver patches are build tested. A b= ranch > > can be found here: > >=20 > > git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git renesas/i2c= /atomic_xfer > >=20 > > I am happy for reviews and comments. Please note if you review (especia= lly the > > core parts), I'd like to have a short summary of your review even if th= ere is > > no proposed change. Like what you did, what you think about it, etc. So= me stuff > > in here is subtle, so if you went through the effort to double check my > > assumptions you should name it :) > >=20 >=20 > Thank you! >=20 > FWIW, >=20 > Reviewed-by Andy Shevchenko >=20 > for patches 1-5,12. Thanks for the review, Andy! May I ask you once more to tag the patches individually so patchwork can pick them up for me? --rigvuvst2m7kwesb Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAly0c7MACgkQFA3kzBSg KbaPdg/5AQODvYA/brIEzEebBWxkjKYMSE6klgfCnyxiyMGQpdbuqY8OcnA+hwI/ 1R813o4sfcHaia62ssxzlPJ7Uipc5yItgMc+yX/KNGJWFmC0JJkbNsFhbEnDGho2 6OEQF/gLqXgpxrd024fb/melBhZhDzAD/BaH5zoPPJ8u8MUymeW4ft8MqnN27aYR Ah0TRQ5CbsqOE2X8sh2laQFypFW7CJuV15vk0RAq+GvyTM/RCG71LWRqIgNv9lKk KRdCnHbiRTyoxeApI4l45+D8jAu4ogxgMSY3B+rEd91Clk/GgiM2nydeLmuyvfYB oueTE1K6fqTYqOOMvkhbufW5bkVQyuhgjwlN9EnnG2ZTGQ3k2Xn3a8kp4TpRbvUL k9QgaBazdN2GngwI6Laa9ePrrVnYdqJM+qexmTX89FRMsLJOveVf/38bHLWOmEP1 TEGVCI6EiOwe6zFdZIHPZ6vWaBD5GQVPNZI3j0orbnia4MfZ2R/+8hBUpI7lj6wm yUKdhgChaGFRPHYvbPC/dWZ1QQQknsBWi6bUyJe5fzBZvq949EUpBPlWzzWeomPt pEi6M3foHyE3wUUH4H8Ecwla8pV6SekKEbshMZwmirF4ZamEoKTKjixNoik7uI8l tWEg2JSw9ku4bkiDM1MiGYZHam2Ea+mMC3n4XOeCVRrlyrbWCU0= =0JbO -----END PGP SIGNATURE----- --rigvuvst2m7kwesb--