On Sat, Oct 3, 2020 at 5:01 AM Coiby Xu <coiby...@gmail.com> wrote:
>
> This patch fix the following warnings from sparse,
>
> $ make C=2 drivers/staging/greybus/
> drivers/staging/greybus/audio_module.c:222:25: warning: incorrect type in 
> assignment (different base types)
> drivers/staging/greybus/audio_module.c:222:25:    expected restricted __le16 
> [usertype] data_cport
> drivers/staging/greybus/audio_module.c:222:25:    got unsigned short 
> [usertype] intf_cport_id
> drivers/staging/greybus/audio_topology.c:460:40: warning: restricted __le32 
> degrades to integer
> drivers/staging/greybus/audio_topology.c:691:41: warning: incorrect type in 
> assignment (different base types)
> drivers/staging/greybus/audio_topology.c:691:41:    expected unsigned int 
> access
> drivers/staging/greybus/audio_topology.c:691:41:    got restricted __le32 
> [usertype] access
> drivers/staging/greybus/audio_topology.c:746:44: warning: incorrect type in 
> assignment (different base types)
> drivers/staging/greybus/audio_topology.c:746:44:    expected unsigned int
> drivers/staging/greybus/audio_topology.c:746:44:    got restricted __le32
> drivers/staging/greybus/audio_topology.c:748:52: warning: incorrect type in 
> assignment (different base types)
> drivers/staging/greybus/audio_topology.c:748:52:    expected unsigned int
> drivers/staging/greybus/audio_topology.c:748:52:    got restricted __le32
> drivers/staging/greybus/audio_topology.c:802:42: warning: restricted __le32 
> degrades to integer
> drivers/staging/greybus/audio_topology.c:805:50: warning: incorrect type in 
> assignment (different base types)
> drivers/staging/greybus/audio_topology.c:805:50:    expected restricted __le32
> drivers/staging/greybus/audio_topology.c:805:50:    got unsigned int
> drivers/staging/greybus/audio_topology.c:814:50: warning: restricted __le32 
> degrades to integer
> drivers/staging/greybus/audio_topology.c:817:58: warning: incorrect type in 
> assignment (different base types)
> drivers/staging/greybus/audio_topology.c:817:58:    expected restricted __le32
> drivers/staging/greybus/audio_topology.c:817:58:    got unsigned int
> drivers/staging/greybus/audio_topology.c:889:25: warning: incorrect type in 
> assignment (different base types)
> drivers/staging/greybus/audio_topology.c:889:25:    expected unsigned int 
> access
> drivers/staging/greybus/audio_topology.c:889:25:    got restricted __le32 
> [usertype] access
>
> Suggested-by: Dan Carpenter <dan.carpen...@oracle.com>
> Reviewed-by: Dan Carpenter <dan.carpen...@oracle.com>
> Reviewed-by: Alex Elder <el...@linaro.org>
> Signed-off-by: Coiby Xu <coiby...@gmail.com>
> ---
Hi Coiby,

Thanks for sharing the patch. Sorry, I could not reply to the v1 series.
Now, I have gone through the patches. Looks good (all 3 patches).

Reviewed-by: Vaibhav Agarwal <vaibhav...@gmail.com>

--
Thanks,
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to