Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-17 Thread Thierry Reding
On Mon, Jun 17, 2013 at 10:59:25AM +0300, Dan Carpenter wrote: > On Thu, Jun 13, 2013 at 11:39:25PM +0200, Thierry Reding wrote: > > On Thu, Jun 13, 2013 at 01:14:15PM -0700, Andrew Morton wrote: > > > On Thu, 13 Jun 2013 20:24:48 +0200 Thierry Reding > > > wrote: > > > > > > > On Thu, Jun 13,

Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-17 Thread Dan Carpenter
On Thu, Jun 13, 2013 at 11:39:25PM +0200, Thierry Reding wrote: > On Thu, Jun 13, 2013 at 01:14:15PM -0700, Andrew Morton wrote: > > On Thu, 13 Jun 2013 20:24:48 +0200 Thierry Reding > > wrote: > > > > > On Thu, Jun 13, 2013 at 10:55:17AM +0530, Sachin Kamat wrote: > > > > On 13 June 2013

Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-17 Thread Dan Carpenter
On Thu, Jun 13, 2013 at 11:39:25PM +0200, Thierry Reding wrote: On Thu, Jun 13, 2013 at 01:14:15PM -0700, Andrew Morton wrote: On Thu, 13 Jun 2013 20:24:48 +0200 Thierry Reding thierry.red...@gmail.com wrote: On Thu, Jun 13, 2013 at 10:55:17AM +0530, Sachin Kamat wrote: On 13 June

Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-17 Thread Thierry Reding
On Mon, Jun 17, 2013 at 10:59:25AM +0300, Dan Carpenter wrote: On Thu, Jun 13, 2013 at 11:39:25PM +0200, Thierry Reding wrote: On Thu, Jun 13, 2013 at 01:14:15PM -0700, Andrew Morton wrote: On Thu, 13 Jun 2013 20:24:48 +0200 Thierry Reding thierry.red...@gmail.com wrote: On Thu,

Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-13 Thread Thierry Reding
On Thu, Jun 13, 2013 at 01:14:15PM -0700, Andrew Morton wrote: > On Thu, 13 Jun 2013 20:24:48 +0200 Thierry Reding > wrote: > > > On Thu, Jun 13, 2013 at 10:55:17AM +0530, Sachin Kamat wrote: > > > On 13 June 2013 03:01, Marc Kleine-Budde wrote: > > > > Commit 75096579c3ac ("lib: devres:

Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-13 Thread Andrew Morton
On Thu, 13 Jun 2013 20:24:48 +0200 Thierry Reding wrote: > On Thu, Jun 13, 2013 at 10:55:17AM +0530, Sachin Kamat wrote: > > On 13 June 2013 03:01, Marc Kleine-Budde wrote: > > > Commit 75096579c3ac ("lib: devres: Introduce devm_ioremap_resource()") > > > introduced devm_ioremap_resource() and

Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-13 Thread Marc Kleine-Budde
On 06/13/2013 08:24 PM, Thierry Reding wrote: > On Thu, Jun 13, 2013 at 10:55:17AM +0530, Sachin Kamat wrote: >> On 13 June 2013 03:01, Marc Kleine-Budde wrote: >>> Commit 75096579c3ac ("lib: devres: Introduce devm_ioremap_resource()") >>> introduced devm_ioremap_resource() and encourage to check

Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-13 Thread Thierry Reding
On Thu, Jun 13, 2013 at 10:55:17AM +0530, Sachin Kamat wrote: > On 13 June 2013 03:01, Marc Kleine-Budde wrote: > > Commit 75096579c3ac ("lib: devres: Introduce devm_ioremap_resource()") > > introduced devm_ioremap_resource() and encourage to check its return value > > with > > IS_ERR(). This

Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-13 Thread Thierry Reding
On Thu, Jun 13, 2013 at 10:55:17AM +0530, Sachin Kamat wrote: On 13 June 2013 03:01, Marc Kleine-Budde m...@pengutronix.de wrote: Commit 75096579c3ac (lib: devres: Introduce devm_ioremap_resource()) introduced devm_ioremap_resource() and encourage to check its return value with IS_ERR().

Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-13 Thread Marc Kleine-Budde
On 06/13/2013 08:24 PM, Thierry Reding wrote: On Thu, Jun 13, 2013 at 10:55:17AM +0530, Sachin Kamat wrote: On 13 June 2013 03:01, Marc Kleine-Budde m...@pengutronix.de wrote: Commit 75096579c3ac (lib: devres: Introduce devm_ioremap_resource()) introduced devm_ioremap_resource() and encourage

Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-13 Thread Andrew Morton
On Thu, 13 Jun 2013 20:24:48 +0200 Thierry Reding thierry.red...@gmail.com wrote: On Thu, Jun 13, 2013 at 10:55:17AM +0530, Sachin Kamat wrote: On 13 June 2013 03:01, Marc Kleine-Budde m...@pengutronix.de wrote: Commit 75096579c3ac (lib: devres: Introduce devm_ioremap_resource())

Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-13 Thread Thierry Reding
On Thu, Jun 13, 2013 at 01:14:15PM -0700, Andrew Morton wrote: On Thu, 13 Jun 2013 20:24:48 +0200 Thierry Reding thierry.red...@gmail.com wrote: On Thu, Jun 13, 2013 at 10:55:17AM +0530, Sachin Kamat wrote: On 13 June 2013 03:01, Marc Kleine-Budde m...@pengutronix.de wrote: Commit

Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-12 Thread Sachin Kamat
On 13 June 2013 03:01, Marc Kleine-Budde wrote: > Commit 75096579c3ac ("lib: devres: Introduce devm_ioremap_resource()") > introduced devm_ioremap_resource() and encourage to check its return value > with > IS_ERR(). This however leads to the following sparse warnings, as >

[RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-12 Thread Marc Kleine-Budde
Commit 75096579c3ac ("lib: devres: Introduce devm_ioremap_resource()") introduced devm_ioremap_resource() and encourage to check its return value with IS_ERR(). This however leads to the following sparse warnings, as devm_ioremap_resource() returns a void __iomem pointer:

[RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-12 Thread Marc Kleine-Budde
Commit 75096579c3ac (lib: devres: Introduce devm_ioremap_resource()) introduced devm_ioremap_resource() and encourage to check its return value with IS_ERR(). This however leads to the following sparse warnings, as devm_ioremap_resource() returns a void __iomem pointer:

Re: [RFC: PATCH] err.h: silence warning when using IS_ERR on void __iomem *

2013-06-12 Thread Sachin Kamat
On 13 June 2013 03:01, Marc Kleine-Budde m...@pengutronix.de wrote: Commit 75096579c3ac (lib: devres: Introduce devm_ioremap_resource()) introduced devm_ioremap_resource() and encourage to check its return value with IS_ERR(). This however leads to the following sparse warnings, as