[PATCH 3/3] staging: emxx_udc: Added missing le16_to_cpu conversions

2018-11-07 Thread Carmeli Tamir
Fixed sparse tool warnings due to missing convesion from le16 to cpu endienness. Signed-off-by: Carmeli Tamir --- drivers/staging/emxx_udc/emxx_udc.c | 22 -- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers

[PATCH 2/3] staging: emxx_udc: Added missing __iomem modifier to handle p_regs

2018-11-07 Thread Carmeli Tamir
Since in nbu2ss_drv_probe() p_regs is assigned from mmio_base, which is marked as __iomem, p_regs also should be market with __iomem. Signed-off-by: Carmeli Tamir --- drivers/staging/emxx_udc/emxx_udc.c | 44 ++--- drivers/staging/emxx_udc/emxx_udc.h | 2 +- 2

[PATCH 1/3] staging: emxx_udc: Added static modifier to udc_controller

2018-11-07 Thread Carmeli Tamir
Added static modifier to the udc_controller, since it's only required within emxx_udc.c. Signed-off-by: Carmeli Tamir --- drivers/staging/emxx_udc/emxx_udc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emx

[PATCH 0/3] staging: emxx_udc: Fixed all meaningful sparse warnings

2018-11-07 Thread Carmeli Tamir
This series fixes all meaningful sparse (running make C=2... ) warnings. The number of warnings is reduced from ~280 to 12 (only warnings of type 'expression using sizeof(void)' are left). Resent the series to Greg's request. Carmeli Tamir (3): staging: emxx_udc: Added sta

[PATCH 3/3] Added missing le16_to_cpu conversions

2018-11-01 Thread Carmeli Tamir
Fixed sparse tool warnings due to missing convesion from le16 to cpu endienness. Previously posted without any feedback, now updated according to master. Signed-off-by: Carmeli Tamir --- drivers/staging/emxx_udc/emxx_udc.c | 22 -- 1 file changed, 12 insertions(+), 10

[PATCH 2/3] staging: emxx_udc: Added missing __iomem modifier to handle p_regs

2018-11-01 Thread Carmeli Tamir
Since in nbu2ss_drv_probe() p_regs is assigned from mmio_base, which is marked as __iomem, p_regs also should be market with __iomem. Previously posted without any feedback, now updated according to master. Signed-off-by: Carmeli Tamir --- drivers/staging/emxx_udc/emxx_udc.c | 44

[PATCH 1/3] staging: emxx_udc: Added static modifier to udc_controller

2018-11-01 Thread Carmeli Tamir
Added static modifier to the udc_controller, since it's only required within emxx_udc.c. Previously posted without any feedback, now updated according to master. Signed-off-by: Carmeli Tamir --- drivers/staging/emxx_udc/emxx_udc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 3/3] staging: emxx_udc: Added missing le16_to_cpu conversions

2018-10-17 Thread Carmeli Tamir
Fixed sparse tool warnings due to missing convesion from le16 to cpu endienness. Signed-off-by: Carmeli Tamir --- drivers/staging/emxx_udc/emxx_udc.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers

[PATCH 2/3] staging: emxx_udc: Added missing __iomem modifier to handle p_regs

2018-10-17 Thread Carmeli Tamir
Since in nbu2ss_drv_probe() p_regs is assigned from mmio_base, which is marked as __iomem, p_regs also should be market with __iomem. Signed-off-by: Carmeli Tamir --- drivers/staging/emxx_udc/emxx_udc.c | 44 ++--- drivers/staging/emxx_udc/emxx_udc.h | 2 +- 2

[PATCH 1/3] staging: emxx_udc: Added static modifier to udc_controller

2018-10-17 Thread Carmeli Tamir
Added static modifier to the udc_controller, since it's only required within emxx_udc.c. Signed-off-by: Carmeli Tamir --- drivers/staging/emxx_udc/emxx_udc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emx

[PATCH] STAGING/EMXX_UDC: Fixed all meaningful sparse errors.

2018-10-06 Thread Carmeli Tamir
Fixed all meaningful sparse errors: 1. Added static to udc_controller 2. Added mising __iomem modifier to handle p_regs 3. Added missing le16_to_cpu Signed-off-by: Tamir Carmeli --- drivers/staging/emxx_udc/emxx_udc.c | 69 +++-- drivers/staging/emxx_udc/emxx_

[PATCH] STAGING/EMXX_UDC: emxx_udc.c: Fixed all meaningful sparse errors: 1. Added static to udc_controller 2. Added mising __iomem modifier to handle p_regs 3. Added missing le16_to_cpu

2018-10-05 Thread Carmeli Tamir
Signed-off-by: Tamir Carmeli --- drivers/staging/emxx_udc/Makefile | 2 +- drivers/staging/emxx_udc/emxx_udc.c | 69 +++-- drivers/staging/emxx_udc/emxx_udc.h | 2 +- 3 files changed, 38 insertions(+), 35 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx

[PATCH] staging/rtl8723bs: fixed a style problem by removing else after return

2018-08-18 Thread Carmeli Tamir
Signed-off-by: Carmeli Tamir --- drivers/staging/rtl8723bs/hal/odm_CfoTracking.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8723bs/hal/odm_CfoTracking.c b/drivers/staging/rtl8723bs/hal/odm_CfoTracking.c index a733046..6912276 100644 --- a/drivers