[PATCH linux-next v3] staging/rtl8821ae: fix sparse address space warning

2014-04-23 Thread Anders Darander
Change the definition of pci_mem_start|end from correct from pci_io(un)map's point of view. Signed-off-by: Anders Darander --- Changes v2 - v3 * Fix a style error v1 -> v2: * Change the struct definition instead of casting all pci_iomap and pci_iounm

[PATCH linux-next v2] staging/rtl8821ae: fix sparse address space warning

2014-04-23 Thread Anders Darander
Change the definition of pci_mem_start|end from correct from pci_io(un)map's point of view. Signed-off-by: Anders Darander --- Changes v1 -> v2: * Change the struct definition instead of casting all pci_iomap and pci_iounmap calls. drivers/staging/rtl8821ae/pci

[PATCH][linux-next] staging/rtl8821ae: fix sparse address space warning

2014-04-23 Thread Anders Darander
When unmapping the pci memory, the pointer was explicitly casted to void*, thus omitting the __iomem designation. Signed-off-by: Anders Darander --- drivers/staging/rtl8821ae/pci.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8821ae/pci.c b/drivers

Re: [PATCH v2] Staging: ft1000-usb: fixed a few code style issues

2014-03-31 Thread Anders Darander
* Greg KH [140401 00:53]: > On Mon, Mar 31, 2014 at 08:26:28AM +0200, Anders Darander wrote: > > Fixed a few code style issues, particularly: > > Add do {} while (0) wrapper around seq_* macros. > > Change printk(KERN_WARN...) to netdev_warn(...). > That's 2 differ

[PATCH] Staging: ft1000-usb: wrap macros in do while

2014-03-31 Thread Anders Darander
Fix coding style issue by wrapping the macro in a do {} while (0). Signed-off-by: Anders Darander --- drivers/staging/ft1000/ft1000-usb/ft1000_proc.c | 21 - 1 file changed, 12 insertions(+), 9 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_proc.c b

[PATCH] Staging: ft1000-usb: use netdev_warn

2014-03-31 Thread Anders Darander
Fix coding style issue by replacing printk with netdev_warn. Signed-off-by: Anders Darander --- drivers/staging/ft1000/ft1000-usb/ft1000_proc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_proc.c b/drivers/staging/ft1000

[PATCH v2] Staging: ft1000-usb: fixed a few code style issues

2014-03-30 Thread Anders Darander
Fixed a few code style issues, particularly: Add do {} while (0) wrapper around seq_* macros. Change printk(KERN_WARN...) to netdev_warn(...). Signed-off-by: Anders Darander --- Changes: v2: * Fix syntax errors (adding missing ; and }). drivers/staging/ft1000/ft1000-usb/ft1000_proc.c | 25

[PATCH] Staging: ft1000-usb: fixed a few code style issues

2014-03-30 Thread Anders Darander
Fixed a few code style issues, particularly: Add do {} while (0) wrapper around seq_* macros. Change printk(KERN_WARN...) to netdev_warn(...). Signed-off-by: Anders Darander --- drivers/staging/ft1000/ft1000-usb/ft1000_proc.c | 25 ++--- 1 file changed, 14 insertions(+), 11