Re: [Linaro-mm-sig] [RESEND][PATCH 0/5] Ion cleanup

2016-08-14 Thread Benjamin Gaignard
This series of patches look good for me, thanks for the clean up Reviewed-by: Benjamin Gaignard 2016-08-08 18:52 GMT+02:00 Laura Abbott : > I never saw any feedback or action on these patches so this is a resend. > Probably hit the window when everyone was busy/traveling. > > --- > > Hi, > > Thi

Re: [PATCH] Staging: rtl8723au: os_intfs: fixed case statement is variable issue

2016-08-14 Thread Johannes Berg
> It's a sparse defect. > > Try again after patching sparse with Jes' patch: > http://marc.info/?l=linux-sparse&m=147091200720267&w=3 Mine, not Jes's ;-) But there's another patch going into the kernel to fix it: https://www.ozlabs.org/~akpm/mmotm/broken-out/byteswap-dont-use-__builtin_bswap-w

Re: [PATCH] Staging: rtl8723au: os_intfs: fixed case statement is variable issue

2016-08-14 Thread Joe Perches
On Sat, 2016-08-13 at 17:26 +0800, sunbing wrote: > On Aug 12, 2016, at 22:30, Jes Sorensen wrote: > > sunbing writes: > > > On Aug 11, 2016, at 23:25, Jes Sorensen wrote: > > > > Bing Sun writes: > > > > > > > > > > Fixed sparse parse error: > > > > > Expected constant expression in case stat

Re: [PATCH] Staging: rtl8723au: os_intfs: fixed case statement is variable issue

2016-08-14 Thread Joe Perches
On Sun, 2016-08-14 at 14:15 +0200, Johannes Berg wrote: > > > > It's a sparse defect. > > > > Try again after patching sparse with Jes' patch: > > http://marc.info/?l=linux-sparse&m=147091200720267&w=3 > Mine, not Jes's ;-) Right, sorry 'bout that. > But there's another patch going into the ker

Re: [RFC 1/2] netvsc: reference counting fix

2016-08-14 Thread David Miller
From: Stephen Hemminger Date: Sat, 13 Aug 2016 11:35:59 -0700 > This is how I think it should be fixed, but not tested yet. > > Subjec: netvsc: use device not module reference counts > > Fix how the cross-device reference counting is handled. When VF is > associated with the synthetic interfac

Re: [PATCH] Staging: rtl8723au: os_intfs: fixed case statement is variable issue

2016-08-14 Thread Johannes Berg
On Sun, 2016-08-14 at 05:23 -0700, Joe Perches wrote: >  > Maybe this test should be sparse version checked after > sparse is updated. *If* sparse ever gets updated :) I don't think it's been updated much lately. That said, I'm not even sure how, and what version, etc. so obviously that'd have to