On Thu, Feb 06, 2014 at 11:13:46AM -0500, Steven Rostedt wrote: > Having a discussion about sparse warnings in the kernel, and that we > should clean them up, I decided to pick a random file to do so. This > happened to be devres.c which gives the following warnings: > > CHECK lib/devres.c > lib/devres.c:83:9: warning: cast removes address space of expression > lib/devres.c:117:31: warning: incorrect type in return expression (different > address spaces) > lib/devres.c:117:31: expected void [noderef] <asn:2>* > lib/devres.c:117:31: got void * > lib/devres.c:125:31: warning: incorrect type in return expression (different > address spaces) > lib/devres.c:125:31: expected void [noderef] <asn:2>* > lib/devres.c:125:31: got void * > lib/devres.c:136:26: warning: incorrect type in assignment (different address > spaces) > lib/devres.c:136:26: expected void [noderef] <asn:2>*[assigned] dest_ptr > lib/devres.c:136:26: got void * > lib/devres.c:226:9: warning: cast removes address space of expression > > Mostly it's just the use of typecasting to void * without adding > __force, or returning ERR_PTR(-ESOMEERR) without typecasting to a > __iomem type. > > I added a helper macro IOMEM_ERR_PTR() that does the typecast to make > the code a little nicer than adding ugly typecasts to the code. > > This is applied against linux-next. > > Signed-off-by: Steven Rostedt <rost...@goodmis.org>
Acked-by: Greg Kroah-Hartman <gre...@linuxfoundation.org> -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/