From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756117Ab3HOLJv (ORCPT ); Thu, 15 Aug 2013 07:09:51 -0400 Received: from top.free-electrons.com ([176.31.233.9]:42177 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1755918Ab3HOLJo (ORCPT ); Thu, 15 Aug 2013 07:09:44 -0400 Message-ID: <520CB6F5.1040309@free-electrons.com> Date: Thu, 15 Aug 2013 13:09:41 +0200 From: Alexandre Belloni Organization: Free Electrons User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130803 Thunderbird/17.0.8 MIME-Version: 1.0 To: Wolfram Sang CC: linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org, marex@denx.de Subject: Re: [PATCH] i2c: mxs: Better error handling References: <1372841962-26310-1-git-send-email-alexandre.belloni@free-electrons.com> <20130815100331.GD2987@katana> In-Reply-To: <20130815100331.GD2987@katana> X-Enigmail-Version: 1.4.6 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 Hi Wolfram, On 15/08/2013 12:03, Wolfram Sang wrote: > On Wed, Jul 03, 2013 at 10:59:22AM +0200, Alexandre Belloni wrote: >> This patch fixes two error handling cases in mxs_i2c_xfer_msg(): >> >> - in PIO mode, mxs_i2c_pio_setup_xfer() may return an error but i2c->cmd_err >> will not be set so mxs_i2c_xfer_msg() fails silently. >> >> - in DMA mode, mxs_i2c_dma_setup_xfer() may return an error and have >> i2c->cmd_err set to -ENXIO. In that case, we want to go through the >> MXS_I2C_CTRL1_CLR_GOT_A_NAK before returning. > > A short explanation why we want that would be perfect. > >> >> Signed-off-by: Alexandre Belloni > > I'd like to postpone this until I applied Marek's PIO rework. Could you > respin after that? > Actually, this patch won't be needed after Marek's PIO rework. You can drop it now. - -- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (GNU/Linux) iQIcBAEBCAAGBQJSDLb1AAoJEKbNnwlvZCyzBOoQAL5NbGfmv2toVVcOHIkGpe0X L+4NqviBObeP2Kh/JWPm8zfuLhIHTU+MNGxFpL5JLzGh+5NVqEA5M7bQjwxmZxPS OCEgzKwMzrTbxM3mOAxALO3LtEOUa6xBzzcKbaeWq48m2yA2KRt9niEqvN+cAHVl rqFyrcv54AUOfdMS923bBNqMpTdpCmdy164msFV+yQqljd4Igg0T54KqmHmw2+4m asfJtDAa674sGizZdB0bSZTo0pGhD2TJEdqU17rsDa/PruFgI8/lw4LOXtpL017M zwx7miO6gkUymEiyOtfufOBkEUSfPmOPd/BcasgAs8loo01IPttn2vMWAV+Pku+D XzbvFvSrlHiMQze6GCejv3W5dMySyDMnWma5ZGJ9DXkzE677UK7i7ehZSxn4huVn w5JNgXfCsOexjTRmtoUad652kQl/GKDL5DdJr75wcZzp/6RfyGz3fu4mfUYiqFue vW/Xbkswc334v2ueuQsdzlEEFfGaiddlsyHHfEEOsdO7djoGRXBoUxI+S1bDFdLO WXCLRO11Ju0IXiTQ9uBBSibP0nSmu9W6/s1Ufp1F3OWqAj+H7CEZpqjg4hQkMIh+ 997svOMknIuJzLhvBQbGj8FJ1FQhNDRaF49tFrpBhZaXp3a+a1rS31kZaoTx2etT 35VZG9zLAOGkfCPU3JEu =bjCC -----END PGP SIGNATURE-----