[PATCH -next v3 2/2] staging: kpc2000: replace white spaces with tabs for kpc2000_spi.c

2019-05-28 Thread Mao Wenan
There are multiple wrong formats in kpc2000_spi.c, is time to do clean work for it. Signed-off-by: Mao Wenan --- drivers/staging/kpc2000/kpc2000_spi.c | 722 +- 1 file changed, 361 insertions(+), 361 deletions(-) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b

[PATCH -next v3 1/2] staging: kpc2000: report error status to spi core

2019-05-28 Thread Mao Wenan
There is an error condition that's not reported to the spi core in kp_spi_transfer_one_message(). It should restore status value to m->status, and return it in error path. Signed-off-by: Mao Wenan --- drivers/staging/kpc2000/kpc2000_spi.c | 8 ++-- 1 file changed, 6 insertio

[PATCH -next v3 0/2] cleanup for kpc2000_spi.c

2019-05-28 Thread Mao Wenan
patch to fix the error condition path and do some clean work for kpc2000_spi.c. Mao Wenan (2): staging: kpc2000: report error status to spi core staging: kpc2000: replace white spaces with tabs for kpc2000_spi.c drivers/staging/kpc2000/kpc2000_spi.c | 718 +- 1 file

[PATCH -next v2] staging: kpc2000: Remove set but not used variable ‘status’

2019-05-25 Thread Mao Wenan
status = 0; ^~ The variable 'status' is not used any more, remve it. Signed-off-by: Mao Wenan --- v2: change the subject of the patch. --- drivers/staging/kpc2000/kpc_spi/spi_driver.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/kpc2000/kpc_spi/spi_driver.c b/dri

[PATCH net] staging: Remove set but not used variable ‘status’

2019-05-24 Thread Mao Wenan
status = 0; ^~ The variable 'status' is not used any more, remve it. Signed-off-by: Mao Wenan --- drivers/staging/kpc2000/kpc_spi/spi_driver.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/kpc2000/kpc_spi/spi_driver.c b/drivers/staging/kpc2000/kpc_spi/spi_driver.

[PATCH v2] staging: davinci: drop pointless static qualifier in vpfe_resizer_init()

2019-03-11 Thread Mao Wenan
There is no need to have the 'T *v' variable static since new value always be assigned before use it. Signed-off-by: Mao Wenan --- v1->v2: remove additional space character between resource_size_t and res_len. drivers/staging/media/davinci_vpfe/dm365_resizer.c | 2 +- 1 fi

[PATCH] staging: davinci: drop pointless static qualifier in vpfe_resizer_init()

2019-03-11 Thread Mao Wenan
There is no need to have the 'T *v' variable static since new value always be assigned before use it. Signed-off-by: Mao Wenan --- drivers/staging/media/davinci_vpfe/dm365_resizer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/da

[PATCH net] staging: rtl8188eu: use is_zero_ether_addr() instead of memcmp()

2019-03-08 Thread Mao Wenan
Using is_zero_ether_addr() instead of directly use memcmp() to determine if the ethernet address is all zeros. Signed-off-by: Mao Wenan --- drivers/staging/rtl8188eu/core/rtw_mlme.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_mlme.c

[PATCH] staging: rtl8712: assign initial value to a static variable in r8712_efuse_pg_packet_write()

2019-02-26 Thread Mao Wenan
repeat_times is a static variable, but each time when it enters r8712_efuse_pg_packet_write(), it is set to zero, this value is not consistent with last calling, so next behavior is not our expect. Signed-off-by: Mao Wenan --- drivers/staging/rtl8712/rtl8712_efuse.c | 3 +-- 1 file changed, 1