Re: New sparse warning from min_t(): expression using sizeof(void)

2018-04-23 Thread Kalle Valo
Kalle Valo writes: > Joey Pabalinas writes: > >> On Sat, Apr 21, 2018 at 10:50:51AM +0300, Kalle Valo wrote: >>> Is there any way to fix it? With ath10k I use sparse a lot and because >>> of these warnings sparse is now very annoying to use. >> >> I submitted a sparse patch [1] for this not too

Re: New sparse warning from min_t(): expression using sizeof(void)

2018-04-23 Thread Joey Pabalinas
On Mon, Apr 23, 2018 at 09:20:14AM +0300, Luciano Coelho wrote: > Thanks, this solves the problem for me, but I'm still getting a lot of > this: > > ./include/linux/mm.h:533:24: warning: constant 0xc900 is so big > it is unsigned long > > Is there a patch in sparse to solve this one

Re: New sparse warning from min_t(): expression using sizeof(void)

2018-04-22 Thread Luciano Coelho
On Sat, 2018-04-21 at 11:54 +0300, Kalle Valo wrote: > Joey Pabalinas writes: > > > On Sat, Apr 21, 2018 at 10:50:51AM +0300, Kalle Valo wrote: > > > Is there any way to fix it? With ath10k I use sparse a lot and > > > because > > > of these warnings sparse is now very annoying to use. > > > > I

Re: New sparse warning from min_t(): expression using sizeof(void)

2018-04-21 Thread Kalle Valo
Joey Pabalinas writes: > On Sat, Apr 21, 2018 at 10:50:51AM +0300, Kalle Valo wrote: >> Is there any way to fix it? With ath10k I use sparse a lot and because >> of these warnings sparse is now very annoying to use. > > I submitted a sparse patch [1] for this not too long ago, but in the > meanti

Re: New sparse warning from min_t(): expression using sizeof(void)

2018-04-21 Thread Joey Pabalinas
On Sat, Apr 21, 2018 at 10:50:51AM +0300, Kalle Valo wrote: > Is there any way to fix it? With ath10k I use sparse a lot and because > of these warnings sparse is now very annoying to use. I submitted a sparse patch [1] for this not too long ago, but in the meantime you can still curl the patch [2

New sparse warning from min_t(): expression using sizeof(void)

2018-04-21 Thread Kalle Valo
Hi Kees&Linus, after upgrading to v4.17-rc1 I started to see this sparse warning from min_t(): drivers/net/wireless/ath/ath10k/wmi.c:4620:31: warning: expression using sizeof(void) I counted 167 such warnings just from ath10k alone and Luca told me that he sees similar sparse warnings with iwlw