Re: [PATCH 1/1] drivers: staging: cxt1e1: linux.c - missing __user annotation

From: Greg KH
Date: Sun Jul 27 2014 - 23:14:18 EST


On Mon, Jul 28, 2014 at 08:38:29AM +0530, Anil Belur wrote:
> From: Anil Belur <askb23@xxxxxxxxx>
>
> - Some of the functions internally call copy_{to,from}_user() but does
> not use '__user'. this patch fixes missing __user annotations.
> - this patch fixes the following sparse errors:
> drivers/staging/cxt1e1/linux.c:488:33: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:488:33: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:488:33: got void *data
> drivers/staging/cxt1e1/linux.c:499:26: warning: incorrect type in argument 1 (different address spaces)
> drivers/staging/cxt1e1/linux.c:499:26: expected void [noderef] <asn:1>*to
> drivers/staging/cxt1e1/linux.c:499:26: got void *data
> drivers/staging/cxt1e1/linux.c:512:33: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:512:33: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:512:33: got void *data
> drivers/staging/cxt1e1/linux.c:534:33: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:534:33: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:534:33: got void *data
> drivers/staging/cxt1e1/linux.c:550:33: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:550:33: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:550:33: got void *data
> drivers/staging/cxt1e1/linux.c:558:26: warning: incorrect type in argument 1 (different address spaces)
> drivers/staging/cxt1e1/linux.c:558:26: expected void [noderef] <asn:1>*to
> drivers/staging/cxt1e1/linux.c:558:26: got void *data
> drivers/staging/cxt1e1/linux.c:571:33: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:571:33: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:571:33: got void *data
> drivers/staging/cxt1e1/linux.c:580:26: warning: incorrect type in argument 1 (different address spaces)
> drivers/staging/cxt1e1/linux.c:580:26: expected void [noderef] <asn:1>*to
> drivers/staging/cxt1e1/linux.c:580:26: got void *data
> drivers/staging/cxt1e1/linux.c:593:33: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:593:33: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:593:33: got void *data
> drivers/staging/cxt1e1/linux.c:601:26: warning: incorrect type in argument 1 (different address spaces)
> drivers/staging/cxt1e1/linux.c:601:26: expected void [noderef] <asn:1>*to
> drivers/staging/cxt1e1/linux.c:601:26: got void *data
> drivers/staging/cxt1e1/linux.c:612:33: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:612:33: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:612:33: got void *data
> drivers/staging/cxt1e1/linux.c:620:26: warning: incorrect type in argument 1 (different address spaces)
> drivers/staging/cxt1e1/linux.c:620:26: expected void [noderef] <asn:1>*to
> drivers/staging/cxt1e1/linux.c:620:26: got void *data
> drivers/staging/cxt1e1/linux.c:631:33: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:631:33: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:631:33: got void *data
> drivers/staging/cxt1e1/linux.c:647:33: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:647:33: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:647:33: got void *data
> drivers/staging/cxt1e1/linux.c:673:34: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:673:34: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:673:34: got void *data
> drivers/staging/cxt1e1/linux.c:681:26: warning: incorrect type in argument 1 (different address spaces)
> drivers/staging/cxt1e1/linux.c:681:26: expected void [noderef] <asn:1>*to
> drivers/staging/cxt1e1/linux.c:681:26: got void *data
> drivers/staging/cxt1e1/linux.c:691:47: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:691:47: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:691:47: got void *data
> drivers/staging/cxt1e1/linux.c:738:33: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:738:33: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:738:33: got void *data
> drivers/staging/cxt1e1/linux.c:785:33: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:785:33: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:785:33: got void *data
> drivers/staging/cxt1e1/linux.c:839:14: warning: incorrect type in assignment (different address spaces)
> drivers/staging/cxt1e1/linux.c:839:14: expected void *data
> drivers/staging/cxt1e1/linux.c:839:14: got void [noderef] <asn:1>*
> drivers/staging/cxt1e1/linux.c:840:34: warning: incorrect type in argument 2 (different address spaces)
> drivers/staging/cxt1e1/linux.c:840:34: expected void const [noderef] <asn:1>*from
> drivers/staging/cxt1e1/linux.c:840:34: got void *data
> drivers/staging/cxt1e1/linux.c:891:42: warning: incorrect type in argument 1 (different address spaces)
> drivers/staging/cxt1e1/linux.c:891:42: expected void [noderef] <asn:1>*to
> drivers/staging/cxt1e1/linux.c:891:42: got void *data
>
> Signed-off-by: Anil Belur <askb23@xxxxxxxxx>
> ---
> drivers/staging/cxt1e1/linux.c | 30 +++++++++++++++---------------
> 1 file changed, 15 insertions(+), 15 deletions(-)

This file is no longer in my tree, or in linux-next :(
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/