From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752543AbbAMLn4 (ORCPT ); Tue, 13 Jan 2015 06:43:56 -0500 Received: from comal.ext.ti.com ([198.47.26.152]:41085 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752483AbbAMLnz (ORCPT ); Tue, 13 Jan 2015 06:43:55 -0500 Message-ID: <54B504EF.4000809@ti.com> Date: Tue, 13 Jan 2015 13:43:43 +0200 From: Tomi Valkeinen User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-Version: 1.0 To: "Lad, Prabhakar" , LFBDEV , Jean-Christophe Plagniol-Villard , Maxime Ripard CC: LKML Subject: Re: [PATCH 2/2] fbdev: ssd1307fb: check return value while setting offset References: <1420705079-3741-1-git-send-email-prabhakar.csengg@gmail.com> <1420705079-3741-3-git-send-email-prabhakar.csengg@gmail.com> In-Reply-To: <1420705079-3741-3-git-send-email-prabhakar.csengg@gmail.com> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="SgxtnFdlnkLLVHcgB9cb0tTaHKiLItvp3" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --SgxtnFdlnkLLVHcgB9cb0tTaHKiLItvp3 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 08/01/15 10:17, Lad, Prabhakar wrote: > this patch checks the return value of write command while > setting the display offset. >=20 > Signed-off-by: Lad, Prabhakar > --- > drivers/video/fbdev/ssd1307fb.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >=20 > diff --git a/drivers/video/fbdev/ssd1307fb.c b/drivers/video/fbdev/ssd1= 307fb.c > index 70e3ce8..a7ecaa6 100644 > --- a/drivers/video/fbdev/ssd1307fb.c > +++ b/drivers/video/fbdev/ssd1307fb.c > @@ -342,7 +342,7 @@ static int ssd1307fb_ssd1306_init(struct ssd1307fb_= par *par) > =20 > /* set display offset value */ > ret =3D ssd1307fb_write_cmd(par->client, SSD1307FB_SET_DISPLAY_OFFSET= ); > - ret =3D ssd1307fb_write_cmd(par->client, 0x20); > + ret =3D ret & ssd1307fb_write_cmd(par->client, 0x20); > if (ret < 0) > return ret; Hrm, what's that supposed to do? If both calls to ssd1307fb_write_cmd() return an error, they are anded, resulting in a garbage error code... Tomi --SgxtnFdlnkLLVHcgB9cb0tTaHKiLItvp3 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJUtQTvAAoJEPo9qoy8lh718tIQAIRmfycemgx9Wd6ps2UMP//L yym+wjko7E4MbmxA7ReqUdvngSCOivMaps71EQRzysorqnZD5HUSw1TOthbCEZi2 +h0npkW3yMKhNgU7lc8M5liG0VehCnTNYSc0ME023N0dbjJkh/CNBv/y/XjHKCmu qYZLmmVldUWT8MLFNN3u5ij/mTOmznCKyfOM8GivQp234NDQfQ81oyeOsxiZ0zt6 lAsPa0odUyI2ZC+ZSecbG0dcC1218Hc0wQZ0h2L9wjXNtOPXRhZrR4vMu6+ZTb68 +67rzlPzZ2mcVLOcF+i0CO2nVSldy9hxLm2AKe3jQqWpfEqPA2guY6xbOKeC8RSp U8IrZ2j9Rb7DhfSE16RyoGfA1aUzQAVfQIxFZvnfo/rV5VBjWlZ6ePD7V8Ms+hdk uNNYMz+8BPdm+eIQuQoik5UOwcpHTLDS3idXRzk7tPs9WAU4h4dMd/BgmVMlIGC4 DVWUGoCiLu29fHvVV2oJhyjXcC8HsmluOCRO1moAT2A5zF7BRZLyzJNO1LPIemC+ MT3crZ2b8jbYtoRgq9zeJeemndZErevjPIYRTWar2ABz6JlHyAR33wtvGBzC2t1E RxmuXV+M1eQ9UFgd5BWe6+8GWPtSIxt55x6FBUWQu5bRLbxJQ1NXHerVeh9MHpdD 6BC5EvhCjzNkh85NFtpz =iq+G -----END PGP SIGNATURE----- --SgxtnFdlnkLLVHcgB9cb0tTaHKiLItvp3--