[PATCH] usb: phy: msm: Fix return value check in msm_otg_probe()

2014-07-19 Thread weiyj_lk
From: Wei Yongjun In case of error, the function devm_ioremap_nocache() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Signed-off-by: Wei Yongjun --- drivers/usb/phy/phy-msm-usb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 d

[PATCH -next] usb: gadget: Fix return value check in r8a66597_probe()

2014-07-19 Thread weiyj_lk
From: Wei Yongjun In case of error, the function devm_ioremap_resource() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- drivers/usb/gadget/udc/r8a66597-udc.c | 4 ++-- 1 file changed, 2 insertio

[PATCH -next] usb: gadget: Fix return value check in ep_write()

2014-07-19 Thread weiyj_lk
From: Wei Yongjun In case of error, the function memdup_user() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- drivers/usb/gadget/legacy/inode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH -next] usb: chipidea: debug: fix sparse non static symbol warnings

2014-07-19 Thread weiyj_lk
From: Wei Yongjun Fixes the following sparse warnings: drivers/usb/chipidea/debug.c:211:5: warning: symbol 'ci_otg_show' was not declared. Should it be static? drivers/usb/chipidea/debug.c:334:5: warning: symbol 'ci_registers_show' was not declared. Should it be static? Signed-off-by: Wei Yon

[PATCH -next] usb: gadget: Remove redundant dev_err call in r8a66597_sudmac_ioremap()

2014-07-20 Thread weiyj_lk
From: Wei Yongjun There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Signed-off-by: Wei Yongjun --- drivers/usb/gadget/udc/r8a66597-udc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/usb/

[PATCH] usb: gadget: f_midi: fix missing unlock on error in f_midi_alloc()

2016-02-05 Thread weiyj_lk
From: Wei Yongjun Add the missing unlock before return from function f_midi_alloc() when kfifo_alloc() failed. Signed-off-by: Wei Yongjun --- drivers/usb/gadget/function/f_midi.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/usb/gadget/function/f_midi.c b/drive

[PATCH] usb: chipidea: fix return value check in ci_hdrc_pci_probe()

2016-02-06 Thread weiyj_lk
From: Wei Yongjun In case of error, the function usb_phy_generic_register() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- drivers/usb/chipidea/ci_hdrc_pci.c | 4 ++-- 1 file changed, 2 insertio

[PATCH] usb: gadget: xudc: fix return value check in xudc_probe()

2015-04-16 Thread weiyj_lk
From: Wei Yongjun In case of error, the function devm_ioremap_resource() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by: Wei Yongjun --- drivers/usb/gadget/udc/udc-xilinx.c | 4 ++-- 1 file changed, 2 insertions

[PATCH -next] usb: gadget: uvc: remove unused including

2015-04-16 Thread weiyj_lk
From: Wei Yongjun Remove including that don't need it. Signed-off-by: Wei Yongjun --- drivers/usb/gadget/function/uvc.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/usb/gadget/function/uvc.h b/drivers/usb/gadget/function/uvc.h index 3390ecd..eedf0b1 100644 --- a/drivers/usb/gad