From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751953AbdBIHk7 (ORCPT ); Thu, 9 Feb 2017 02:40:59 -0500 Received: from mail.free-electrons.com ([62.4.15.54]:34636 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751840AbdBIHk6 (ORCPT ); Thu, 9 Feb 2017 02:40:58 -0500 Date: Thu, 9 Feb 2017 08:34:48 +0100 From: Maxime Ripard To: Wei Yongjun Cc: Alessandro Zummo , Alexandre Belloni , Chen-Yu Tsai , Wei Yongjun , rtc-linux@googlegroups.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH -next] rtc: sun6i: Fix return value check in sun6i_rtc_clk_init() Message-ID: <20170209073448.ua6qtptmpdjkren6@lukather> References: <20170209001613.21665-1-weiyj.lk@gmail.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="b2oeurynxati5w6f" Content-Disposition: inline In-Reply-To: <20170209001613.21665-1-weiyj.lk@gmail.com> User-Agent: Mutt/1.6.2-neo (2016-08-21) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --b2oeurynxati5w6f Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Feb 09, 2017 at 12:16:13AM +0000, Wei Yongjun wrote: > From: Wei Yongjun >=20 > In case of error, the function of_io_request_and_map() returns ERR_PTR() > and never returns NULL. The NULL test in the return value check should > be replaced with IS_ERR(). >=20 > Fixes: 847b8bf62eb4 ("rtc: sun6i: Expose the 32kHz oscillator") > Signed-off-by: Wei Yongjun Acked-by: Maxime Ripard Thanks, Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com --b2oeurynxati5w6f Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIcBAEBCAAGBQJYnBuUAAoJEBx+YmzsjxAg5+wQAK+KsQG1sqOvxlGk/pSN3f7C DXF2aZHuohz4daQWSjINQOFimKYmmIn0hh5S7vA7klHBIXVFgajVF4jg5hSwYhAk aSqHXLsDU0PUTCuBZqmJYP+rY3i1fjxa+pq0u1u8onh+IIz0Ajq4OBFP7FJSuqQT vrDmhqr/tYlUJi8RE702BKWroyHRgQFL9oDGdijQhPdgTp21RHCu6RU6nGH5Iryn s7GPS06v+hOuhbAX+5S5bsz1EB6QqT6Tud6Xz7pAaTDg8LcnxFLK1jc6qpmdul7e wMQq2m/i3tPUcyCGwdJ5NKY0bVURHP9Y5ocKCDn0HbVHz3Jg65s5t1KYJBJj5R6m e4ybmSb7h7J/xaoc46erd3O73326MZFIpc58IQzEZ9NJGD1K/18WuZLBNBIWw24M AFsYA0nRDjUdjdorFDerFXVFppVzJRvq09cOoBqtXsLpxO3wgYZ+Wi7JbQBy0Esv /LfDzaBi0BQ6qr5G53FqMy2YhPA9kYilDzqI22ERwY0NyGCgmaBNZYcd9eZnSLJ4 vj1ReNBQOmeAdiHR1X6EtrhBWfI5Vj4Flvp2u/V/v2abc9M1+eK7cN1uyuZRTt1d 8xLZWuZc8Adu6gDjCaFhu9vfb33ReZJZbWQTR8hrQV0zPGF65xtXv8oFf0i0zp+p bdFBb+cWRnsfdBbqPK5q =tCZC -----END PGP SIGNATURE----- --b2oeurynxati5w6f--