From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.0 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SPF_PASS,URIBL_BLOCKED,USER_AGENT_NEOMUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 365D6C10F12 for ; Mon, 15 Apr 2019 12:06:23 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 06407206BA for ; Mon, 15 Apr 2019 12:06:23 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="gWqdB7SC" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 06407206BA Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=the-dreams.de Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=jAuJyvCwOEalsn0YF1Pfd1jChF6TUL1DpxXLfxPjC/o=; b=gWqdB7SCP1voIA5/KD+jA6GKi YMzqokgaGTHgOmXk0ZW9NpnZZTEQkeHyR0+Ruq87xsm3TvfBK6UkPd0aEGX0d+ZjB+dsoMK6JDTfK L9rP8C5AQutX3TeeVwWdb6u4drGae2o3w7dT3kHhkWmUQl+X5PXykjER1isSCjxo2uOCP0m+aNDah T5+6XuJzWEAb29z6f6x3gEW0d4ALlIdUXrfTEpdp4PCGPw3biUPwIVsIHehAPRUwK7xKNR6Ow6514 w4iuSg/xU6Yk2182bBVRhk+os148iXjsZor6vR+76loLQ1+gQpBKGRRqj1l+zHlQ8QH4SJH6F0sBJ XMngAS6/w==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1hG0NP-00073b-0W; Mon, 15 Apr 2019 12:06:19 +0000 Received: from sauhun.de ([88.99.104.3] helo=pokefinder.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1hG0NI-0006w2-F4 for linux-arm-kernel@lists.infradead.org; Mon, 15 Apr 2019 12:06:15 +0000 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 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 In-Reply-To: <20190403131510.GE9224@smile.fi.intel.com> User-Agent: NeoMutt/20170113 (1.7.2) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190415_050613_406755_43A6409F X-CRM114-Status: GOOD ( 19.63 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Linus Walleij , linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, Wolfram Sang , linux-i2c@vger.kernel.org, linux-tegra@vger.kernel.org, Stefan Lengfeld , linux-omap@vger.kernel.org, Peter Rosin , linux-arm-kernel@lists.infradead.org Content-Type: multipart/mixed; boundary="===============6753860946095898747==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============6753860946095898747== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="rigvuvst2m7kwesb" Content-Disposition: inline --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-- --===============6753860946095898747== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============6753860946095898747==--