On Sat, Oct 3, 2020 at 5:01 AM Coiby Xu <coiby.xu@xxxxxxxxx> 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.carpenter@xxxxxxxxxx> > Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > Reviewed-by: Alex Elder <elder@xxxxxxxxxx> > Signed-off-by: Coiby Xu <coiby.xu@xxxxxxxxx> > --- 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.sr@xxxxxxxxx> -- Thanks, _______________________________________________ greybus-dev mailing list greybus-dev@xxxxxxxxxxxxxxxx https://lists.linaro.org/mailman/listinfo/greybus-dev
|