[PATCH v3 3/3] staging: fbtft: access screen buffer directly

2015-10-07 Thread Lars Svensson
In fbtft-bus.c:fbtft_write_vmem16_bus9(), ioread8() is used for accessing the provided screen array. Since screen_buffer actually points to an ordinary buffer, instead access it directly. Signed-off-by: Lars Svensson --- drivers/staging/fbtft/fbtft-bus.c | 8 1 file changed, 4

[PATCH v3 1/3] fb.h: Provide alternate screen_base pointer

2015-10-07 Thread Lars Svensson
Some drivers use member screen_base of struct fb_info to store non- __iomem pointers, creating the need for ugly __force typecasts to avoid sparse warnings. This adds an alternate pointer without the __iomem qualifyer for this use. Signed-off-by: Lars Svensson --- Patch v3: bugfix in fbtft

[PATCH v3 2/3] staging: fbtft: use alternate screen pointer

2015-10-07 Thread Lars Svensson
Member screen_base in struct fb_info is declared with __iomem qualifier causing sparse warnings when used as a regular ponter. To avoid the warnings, instead use alternate non-__iomem pointer, screen_buffer, troughout the driver. Signed-off-by: Lars Svensson --- drivers/staging/fbtft

[PATCH v2 1/2] fb.h: Provide alternate screen_base pointer

2015-10-06 Thread Lars Svensson
Some drivers use member screen_base of struct fb_info to store non- __iomem pointers, creating the need for ugly __force typecasts to avoid sparse warnings. This adds an alternate pointer without the __iomem qualifyer for this use. Signed-off-by: Lars Svensson --- include/linux/fb.h | 5

[PATCH v2 2/2] staging: fbtft: use alternate screen pointer

2015-10-06 Thread Lars Svensson
Member screen_base in struct fb_info is declared with __iomem qualifier causing sparse warnings when used as a regular ponter. To avoid the warnings, instead use alternate non-__iomem pointer, screen_buffer, troughout the driver. Signed-off-by: Lars Svensson --- v2: removed all references to

[PATCH 2/2] staging: fbtft: fix sparse warning in call to vfree()

2015-10-06 Thread Lars Svensson
:expected void const *addr fbtft-core.c:922:39:got char [noderef] *screen_base Signed-off-by: Lars Svensson --- drivers/staging/fbtft/fbtft-core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/fbtft/fbtft-core.c b/drivers/staging/fbtft/fbtft-core.c

[PATCH 1/2] fb.h: Provide alternate screen_base pointer

2015-10-06 Thread Lars Svensson
Some drivers use member screen_base of struct fb_info to store non- __iomem pointers, creating the need for ugly __force typecasts to avoid sparse warnings. This adds an alternate pointer without the __iomem qualifyer for this use. Signed-off-by: Lars Svensson --- New version of my previous

[PATCH] staging: fbtft: add typecast in call to vfree()

2015-10-01 Thread Lars Svensson
void const *addr fbtft-core.c:922:39:got char [noderef] *screen_base Signed-off-by: Lars Svensson --- drivers/staging/fbtft/fbtft-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fbtft/fbtft-core.c b/drivers/staging/fbtft/fbtft-core.c index 7f5fa3d

[PATCH V2] staging: rtl8723au: Remove unneeded endianness conversions

2015-09-24 Thread Lars Svensson
/rtl8723au/core/rtw_security.c:776:22: \ warning: cast to restricted __le32 drivers/staging/rtl8723au/core/rtw_security.c:777:24: \ warning: cast to restricted __le32 Signed-off-by: Lars Svensson --- Patch V2: Reworked as adviced. --- drivers/staging/rtl8723au/core/rtw_security.c | 9 - 1 file

Re: [PATCH] staging: rtl8723au: Mark type casts to __le32 as intentional

2015-09-23 Thread Lars Svensson
On Tue, Sep 22, 2015 at 11:30:54PM +0200, Larry Finger wrote: > > You may have silenced the Sparse warnings, but the code was not wrong. Your > version is also correct; however, you end up with some really ugly casts. > > Here is my analysis of these two, identical sections: > > The output of g

[PATCH] staging: rtl8723au: Mark type casts to __le32 as intentional

2015-09-22 Thread Lars Svensson
to restricted __le32 drivers/staging/rtl8723au/core/rtw_security.c:776:22: \ warning: cast to restricted __le32 drivers/staging/rtl8723au/core/rtw_security.c:777:24: \ warning: cast to restricted __le32 Signed-off-by: Lars Svensson --- drivers/staging/rtl8723au/core/rtw_security.c | 15

[PATCH] staging: iio_simple_dummy: Fix indentation errors

2015-08-05 Thread Lars Svensson
Fixing indentation errors in drivers/staging/iio/iio_simple_dummy_events.c. Signed-off-by: Lars Svensson --- drivers/staging/iio/iio_simple_dummy_events.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/iio/iio_simple_dummy_events.c b/drivers/staging/iio