Re: [PATCH v3 1/2] staging: slicoss: fix different address space warnings: 32 bits

2016-11-17 Thread Greg KH
On Thu, Nov 17, 2016 at 12:46:12PM +0100, Sergio Paracuellos wrote: > On Thu, Nov 17, 2016 at 12:33 PM, Dan Carpenter > wrote: > > Give it a shot and see if the warnings go away. I don't think the tag > > is correct. > > Just removing __iomem tag in shmem_data field of slic_shmemory struct > mak

Re: [PATCH v3 1/2] staging: slicoss: fix different address space warnings: 32 bits

2016-11-17 Thread Sergio Paracuellos
On Thu, Nov 17, 2016 at 12:33 PM, Dan Carpenter wrote: > Give it a shot and see if the warnings go away. I don't think the tag > is correct. Just removing __iomem tag in shmem_data field of slic_shmemory struct makes sparse happy. No warnings around. Should I send a v4 patch with the tag remove

Re: [PATCH v3 1/2] staging: slicoss: fix different address space warnings: 32 bits

2016-11-17 Thread Dan Carpenter
On Thu, Nov 17, 2016 at 12:22:00PM +0100, Sergio Paracuellos wrote: > On Thu, Nov 17, 2016 at 11:51 AM, Dan Carpenter > wrote: > > Hm... I seem to duplicate the Sparse warnings at all. That's weird. > > > > With this patch applied, sparse warnings dissapear for me... That's weird. > I understa

Re: [PATCH v3 1/2] staging: slicoss: fix different address space warnings: 32 bits

2016-11-17 Thread Sergio Paracuellos
On Thu, Nov 17, 2016 at 11:51 AM, Dan Carpenter wrote: > Hm... I seem to duplicate the Sparse warnings at all. That's weird. > With this patch applied, sparse warnings dissapear for me... That's weird. > Anyway as far as I can tell the warnings are false positives... Can't > you just remove t

Re: [PATCH v3 1/2] staging: slicoss: fix different address space warnings: 32 bits

2016-11-17 Thread Dan Carpenter
Hm... I seem to duplicate the Sparse warnings at all. That's weird. Anyway as far as I can tell the warnings are false positives... Can't you just remove the __iomem tag? Have you tested this change? regards, dan carpenter ___ devel mailing list de