From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jarkko Sakkinen Subject: Re: [PATCH 02/11] tpm/tpm_i2c_infineon: switch to i2c_lock_segment Date: Tue, 19 Jun 2018 15:56:18 +0300 Message-ID: <20180619125618.GB5609@linux.intel.com> References: <20180615101506.8012-1-peda@axentia.se> <20180615101506.8012-3-peda@axentia.se> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180615101506.8012-3-peda@axentia.se> Sender: linux-kernel-owner@vger.kernel.org To: Peter Rosin Cc: linux-kernel@vger.kernel.org, Peter Huewe , Jason Gunthorpe , Arnd Bergmann , Greg Kroah-Hartman , Brian Norris , Gregory Fong , Florian Fainelli , bcm-kernel-feedback-list@broadcom.com, Sekhar Nori , Kevin Hilman , Haavard Skinnemoen , Kukjin Kim , Krzysztof Kozlowski , Orson Zhai , Baolin Wang , Chunyan Zhang , Laxman Dewangan , Thierry Reding , Jonathan Hunter , Wo List-Id: linux-input@vger.kernel.org On Fri, Jun 15, 2018 at 12:14:57PM +0200, Peter Rosin wrote: > Locking the root adapter for __i2c_transfer will deadlock if the > device sits behind a mux-locked I2C mux. Switch to the finer-grained > i2c_lock_segment. If the device does not sit behind a mux-locked mux, > the two locking variants are equivalent. > > Signed-off-by: Peter Rosin Can you quickly explain (or give a reference) the difference with these functions? Not an expert in this area. Thanks. /Jarkko