Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752674Ab3EPLuG (ORCPT ); Thu, 16 May 2013 07:50:06 -0400 Received: from metis.ext.pengutronix.de ([92.198.50.35]:60023 "EHLO metis.ext.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751827Ab3EPLuE (ORCPT ); Thu, 16 May 2013 07:50:04 -0400 Message-ID: <5194C7E8.4060804@pengutronix.de> Date: Thu, 16 May 2013 13:50:00 +0200 From: Marc Kleine-Budde User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130509 Thunderbird/17.0.6 MIME-Version: 1.0 To: Laurent Navet CC: linux-can@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] drivers: net: can: grcan: use devm_ioremap_resource() References: <1368458871-7402-1-git-send-email-laurent.navet@gmail.com> <51935176.7020601@pengutronix.de> In-Reply-To: <51935176.7020601@pengutronix.de> X-Enigmail-Version: 1.5.1 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="----enig2NJUWBQCCIVDADDCFLDWQ" X-SA-Exim-Connect-IP: 2001:6f8:1178:4:5054:ff:fe8d:eefb X-SA-Exim-Mail-From: mkl@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2427 Lines: 70 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) ------enig2NJUWBQCCIVDADDCFLDWQ Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: quoted-printable On 05/15/2013 11:12 AM, Marc Kleine-Budde wrote: > On 05/13/2013 05:27 PM, Laurent Navet wrote: >> Replace a call to deprecated devm_request_and_ioremap by devm_ioremap_= resource. >> dev_err() message is no more needed since it's already displayed in >> devm_ioremap_resource(). >> >> Signed-off-by: Laurent Navet >=20 > Tnx, applied to linux-can-next. The "IS_ERR(base)" ... > + base =3D devm_ioremap_resource(&ofdev->dev, res); > + if (IS_ERR(base)) { > + err =3D PTR_ERR(base); > goto exit_error; > } =2E.. introduces this sparse warning: > drivers/net/can/grcan.c:1687:20: warning: incorrect type in argument 1 = (different address spaces) > drivers/net/can/grcan.c:1687:20: expected void const *ptr > drivers/net/can/grcan.c:1687:20: got void [noderef] *[assigne= d] base > drivers/net/can/grcan.c:1688:31: warning: incorrect type in argument 1 = (different address spaces) > drivers/net/can/grcan.c:1688:31: expected void const *ptr > drivers/net/can/grcan.c:1688:31: got void [noderef] *[assigne= d] base I think it's the __iomem annotation of the void pointer that devm_ioremap_resource() returns. Is there a clever way to improve IS_ERR and PTR_ERR? Marc --=20 Pengutronix e.K. | Marc Kleine-Budde | Industrial Linux Solutions | Phone: +49-231-2826-924 | Vertretung West/Dortmund | Fax: +49-5121-206917-5555 | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | ------enig2NJUWBQCCIVDADDCFLDWQ 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.4.12 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iEYEARECAAYFAlGUx+gACgkQjTAFq1RaXHN3pwCfeMypx0rU/Ky2c8F+qDL5Ts0x OVsAoI9r/nHQ70eAka2/9HK8rLT3Zi1z =ivLw -----END PGP SIGNATURE----- ------enig2NJUWBQCCIVDADDCFLDWQ-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/