From: Wei Yongjun
Fix the return value check which testing the wrong variable.
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/i915/selftests/i915_sw_fence.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/selftests/i915_sw_fence.c
b/drivers/gpu
From: Wei Yongjun <weiyongj...@huawei.com>
Fixes the following sparse warnings:
sound/soc/codecs/rt5665.c:4928:19: warning:
symbol 'rt5665_i2c_driver' was not declared. Should it be static?
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
sound/soc/codecs/rt5665.c | 2 +-
1
From: Wei Yongjun
Fixes the following sparse warnings:
sound/soc/codecs/rt5665.c:4928:19: warning:
symbol 'rt5665_i2c_driver' was not declared. Should it be static?
Signed-off-by: Wei Yongjun
---
sound/soc/codecs/rt5665.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
From: Wei Yongjun <weiyongj...@huawei.com>
Fixes the following sparse warning:
drivers/ras/ras.c:32:12: warning:
symbol 'parse_ras_param' was not declared. Should it be static?
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/ras/ras.c | 2 +-
1 file changed, 1 ins
From: Wei Yongjun
Fixes the following sparse warning:
drivers/ras/ras.c:32:12: warning:
symbol 'parse_ras_param' was not declared. Should it be static?
Signed-off-by: Wei Yongjun
---
drivers/ras/ras.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/ras/ras.c b
Commit-ID: 216646e4d82e847791f0ba66c439dedd36cb119f
Gitweb: http://git.kernel.org/tip/216646e4d82e847791f0ba66c439dedd36cb119f
Author: Wei Yongjun <weiyongj...@huawei.com>
AuthorDate: Thu, 27 Apr 2017 15:21:13 +
Committer: Thomas Gleixner <t...@linutronix.de>
CommitDate:
Commit-ID: 216646e4d82e847791f0ba66c439dedd36cb119f
Gitweb: http://git.kernel.org/tip/216646e4d82e847791f0ba66c439dedd36cb119f
Author: Wei Yongjun
AuthorDate: Thu, 27 Apr 2017 15:21:13 +
Committer: Thomas Gleixner
CommitDate: Sun, 30 Apr 2017 11:21:16 +0200
irqchip/mbigen: Fix
From: Wei Yongjun <weiyongj...@huawei.com>
In case of error, the function devm_ioremap() returns NULL pointer
not ERR_PTR(). Use devm_ioremap_resource() instead of devm_ioremap()
to fix the IS_ERR() test issue.
Fixes: 76e1f77f9c26 ("irqchip/mbigen: Introduce mbigen_of_create_domain
From: Wei Yongjun
In case of error, the function devm_ioremap() returns NULL pointer
not ERR_PTR(). Use devm_ioremap_resource() instead of devm_ioremap()
to fix the IS_ERR() test issue.
Fixes: 76e1f77f9c26 ("irqchip/mbigen: Introduce mbigen_of_create_domain()")
Signed-off-by: W
From: Wei Yongjun <weiyongj...@huawei.com>
In case of error, the function platform_device_register_simple()
returns ERR_PTR() and never returns NULL. The NULL test in the
return value check should be replaced with IS_ERR().
Fixes: 049a59db34eb ("firmware: Google VPD sysfs driver&q
From: Wei Yongjun
In case of error, the function platform_device_register_simple()
returns ERR_PTR() and never returns NULL. The NULL test in the
return value check should be replaced with IS_ERR().
Fixes: 049a59db34eb ("firmware: Google VPD sysfs driver")
Signed-off-by: W
From: Wei Yongjun <weiyongj...@huawei.com>
convert list_for_each() to list_for_each_entry() where
applicable.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/auxdisplay/panel.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/auxdispl
From: Wei Yongjun
convert list_for_each() to list_for_each_entry() where
applicable.
Signed-off-by: Wei Yongjun
---
drivers/auxdisplay/panel.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/auxdisplay/panel.c b/drivers/auxdisplay/panel.c
index e0c014c..7a8b8fb
From: Wei Yongjun <weiyongj...@huawei.com>
Use the builtin_platform_driver() macro to make the code simpler.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/irqchip/qcom-irq-combiner.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/
From: Wei Yongjun
Use the builtin_platform_driver() macro to make the code simpler.
Signed-off-by: Wei Yongjun
---
drivers/irqchip/qcom-irq-combiner.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/irqchip/qcom-irq-combiner.c
b/drivers/irqchip/qcom-irq
From: Wei Yongjun <weiyongj...@huawei.com>
The error return code PTR_ERR(mc) is always 0 since mc is
equal to 0 in this error handling case.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/gpu/drm/nouveau/nvkm/subdev/secboot/gm20b.c | 2 +-
1 file changed, 1 ins
From: Wei Yongjun
The error return code PTR_ERR(mc) is always 0 since mc is
equal to 0 in this error handling case.
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/nouveau/nvkm/subdev/secboot/gm20b.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/nouveau
From: Wei Yongjun <weiyongj...@huawei.com>
sizeof() when applied to a pointer typed expression gives the
size of the pointer, not that of the pointed data.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/phy/phy-qcom-qmp.c | 2 +-
1 file changed, 1 insertion(+)
From: Wei Yongjun
sizeof() when applied to a pointer typed expression gives the
size of the pointer, not that of the pointed data.
Signed-off-by: Wei Yongjun
---
drivers/phy/phy-qcom-qmp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/phy/phy-qcom-qmp.c b/drivers
From: Wei Yongjun <weiyongj...@huawei.com>
Since we will remove items off the list using list_del() we need
to use a safe version of the list_for_each() macro aptly named
list_for_each_safe().
This is detected by Coccinelle semantic patch.
Fixes: b8c722ddd548 ("ASoC: Intel: Skylake:
From: Wei Yongjun
Since we will remove items off the list using list_del() we need
to use a safe version of the list_for_each() macro aptly named
list_for_each_safe().
This is detected by Coccinelle semantic patch.
Fixes: b8c722ddd548 ("ASoC: Intel: Skylake: Add support for deferred
DSP m
From: Wei Yongjun <weiyongj...@huawei.com>
Fix the retrn value check which testing the wrong variable
in cdn_dp_probe().
Fixes: 1a0f7ed3abe2 ("drm/rockchip: cdn-dp: add cdn DP support for rk3399")
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/gpu/drm/rock
From: Wei Yongjun
Fix the retrn value check which testing the wrong variable
in cdn_dp_probe().
Fixes: 1a0f7ed3abe2 ("drm/rockchip: cdn-dp: add cdn DP support for rk3399")
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/rockchip/cdn-dp-core.c | 2 +-
1 file changed, 1 insertion(+),
From: Wei Yongjun <weiyongj...@huawei.com>
In case of error, the function of_iomap() returns NULL pointer
not ERR_PTR(). The IS_ERR() test in the return value check should
be replaced with NULL test.
Fixes: e78f3d15e115 ("phy: qcom-qmp: new qmp phy driver for qcom-chipsets")
Si
From: Wei Yongjun
In case of error, the function of_iomap() returns NULL pointer
not ERR_PTR(). The IS_ERR() test in the return value check should
be replaced with NULL test.
Fixes: e78f3d15e115 ("phy: qcom-qmp: new qmp phy driver for qcom-chipsets")
Signed-off-by: Wei Yongjun
---
d
From: Wei Yongjun <weiyongj...@huawei.com>
In case of error, the function platform_device_register_simple() returns
ERR_PTR() and never returns NULL. The NULL test in the return value
check should be replaced with IS_ERR().
Fixes: d384d6f43d1e ("firmware: google memconsole: Add coreb
From: Wei Yongjun
In case of error, the function platform_device_register_simple() returns
ERR_PTR() and never returns NULL. The NULL test in the return value
check should be replaced with IS_ERR().
Fixes: d384d6f43d1e ("firmware: google memconsole: Add coreboot support")
Signed-o
From: Wei Yongjun <weiyongj...@huawei.com>
Add the missing unlock before return from function etnaviv_gpu_submit()
in the error handling case.
Fixes: f3cd1b064f11 ("drm/etnaviv: (re-)protect fence allocation with
GPU mutex")
Signed-off-by: Wei Yongjun <weiyongj...@huawei.co
From: Wei Yongjun
Add the missing unlock before return from function etnaviv_gpu_submit()
in the error handling case.
Fixes: f3cd1b064f11 ("drm/etnaviv: (re-)protect fence allocation with
GPU mutex")
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 3 ++-
1 fi
From: Wei Yongjun <weiyongj...@huawei.com>
module_w1_family() makes the code simpler by eliminating
boilerplate code.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/w1/slaves/w1_ds2405.c | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --g
From: Wei Yongjun
module_w1_family() makes the code simpler by eliminating
boilerplate code.
Signed-off-by: Wei Yongjun
---
drivers/w1/slaves/w1_ds2405.c | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/drivers/w1/slaves/w1_ds2405.c b/drivers/w1/slaves
From: Wei Yongjun <weiyongj...@huawei.com>
Use kmemdup rather than duplicating its implementation.
Generated by: scripts/coccinelle/api/memdup.cocci
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/w1/masters/ds2490.c | 3 +--
1 file changed, 1 insertion(+), 2 deleti
From: Wei Yongjun
Use kmemdup rather than duplicating its implementation.
Generated by: scripts/coccinelle/api/memdup.cocci
Signed-off-by: Wei Yongjun
---
drivers/w1/masters/ds2490.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/w1/masters/ds2490.c b/drivers
From: Wei Yongjun <weiyongj...@huawei.com>
Fixes the following sparse warning:
drivers/gpu/drm/bridge/ti-tfp410.c:223:24: warning:
symbol 'tfp410_platform_driver' was not declared. Should it be static?
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/gpu/drm/bridge
From: Wei Yongjun
Fixes the following sparse warning:
drivers/gpu/drm/bridge/ti-tfp410.c:223:24: warning:
symbol 'tfp410_platform_driver' was not declared. Should it be static?
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/bridge/ti-tfp410.c | 2 +-
1 file changed, 1 insertion(+), 1
From: Wei Yongjun <weiyongj...@huawei.com>
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 <weiyongj...@huawei.com>
---
drivers/bus/da8xx-mstpri.c | 4 +---
1 file changed, 1 ins
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/bus/da8xx-mstpri.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/bus/da8xx
From: Wei Yongjun <weiyongj...@huawei.com>
Remove duplicated include.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
arch/x86/kernel/cpu/intel_rdt_rdtgroup.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/x86/kernel/cpu/intel_rdt_rdtgroup.c
b/arch/x8
From: Wei Yongjun
Remove duplicated include.
Signed-off-by: Wei Yongjun
---
arch/x86/kernel/cpu/intel_rdt_rdtgroup.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/x86/kernel/cpu/intel_rdt_rdtgroup.c
b/arch/x86/kernel/cpu/intel_rdt_rdtgroup.c
index 8af04af..759577d 100644
--- a/arch
From: Wei Yongjun <weiyongj...@huawei.com>
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 <weiyongj...@huawei.com>
---
drivers/memory/da8xx-ddrctl.c | 4 +---
1 file changed,
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/memory/da8xx-ddrctl.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/memory
From: Wei Yongjun <weiyongj...@huawei.com>
Remove duplicated include.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
arch/x86/kernel/itmt.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/x86/kernel/itmt.c b/arch/x86/kernel/itmt.c
index f73f475..b5ccc9b 100644
--
From: Wei Yongjun
Remove duplicated include.
Signed-off-by: Wei Yongjun
---
arch/x86/kernel/itmt.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/x86/kernel/itmt.c b/arch/x86/kernel/itmt.c
index f73f475..b5ccc9b 100644
--- a/arch/x86/kernel/itmt.c
+++ b/arch/x86/kernel/itmt.c
From: Wei Yongjun <weiyongj...@huawei.com>
In case of error, the function of_io_request_and_map() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check should
be replaced with IS_ERR().
Fixes: 847b8bf62eb4 ("rtc: sun6i: Expose the 32kHz oscillator")
Si
From: Wei Yongjun
In case of error, the function of_io_request_and_map() returns ERR_PTR()
and never returns NULL. The NULL test in the return value check should
be replaced with IS_ERR().
Fixes: 847b8bf62eb4 ("rtc: sun6i: Expose the 32kHz oscillator")
Signed-off-by: Wei Yongjun
--
From: Wei Yongjun <weiyongj...@huawei.com>
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 <weiyongj...@huawei.com>
---
drivers/mtd/spi-nor/aspeed-smc.c | 8 ++--
1 fil
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/mtd/spi-nor/aspeed-smc.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/mtd
From: Wei Yongjun <weiyongj...@huawei.com>
Fixes the following sparse warnings:
drivers/reset/reset-uniphier.c:68:34: warning:
symbol 'uniphier_sld3_sys_reset_data' was not declared. Should it be static?
drivers/reset/reset-uniphier.c:73:34: warning:
symbol 'uniphier_pro4_sys_rese
From: Wei Yongjun
Fixes the following sparse warnings:
drivers/reset/reset-uniphier.c:68:34: warning:
symbol 'uniphier_sld3_sys_reset_data' was not declared. Should it be static?
drivers/reset/reset-uniphier.c:73:34: warning:
symbol 'uniphier_pro4_sys_reset_data' was not declared. Should
From: Wei Yongjun <weiyongj...@huawei.com>
Add the missing platform_driver_unregister() and remove the duplicate
platform_device_unregister(force_pdev) in the error handling case.
Fixes: 00194826e6be ("tpm_tis: Clean up the force=1 module parameter")
Signed-off-by: Wei Y
From: Wei Yongjun
Add the missing platform_driver_unregister() and remove the duplicate
platform_device_unregister(force_pdev) in the error handling case.
Fixes: 00194826e6be ("tpm_tis: Clean up the force=1 module parameter")
Signed-off-by: Wei Yongjun
---
drivers/char/tpm/tpm_tis.c
From: Wei Yongjun <weiyongj...@huawei.com>
Remove duplicated include.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/bus/qcom-ebi2.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/bus/qcom-ebi2.c b/drivers/bus/qcom-ebi2.c
index a644424..374fe1d 100644
-
From: Wei Yongjun
Remove duplicated include.
Signed-off-by: Wei Yongjun
---
drivers/bus/qcom-ebi2.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/bus/qcom-ebi2.c b/drivers/bus/qcom-ebi2.c
index a644424..374fe1d 100644
--- a/drivers/bus/qcom-ebi2.c
+++ b/drivers/bus/qcom-ebi2.c
From: Wei Yongjun <weiyongj...@huawei.com>
Remove duplicated include.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
arch/mips/kernel/syscall.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/mips/kernel/syscall.c b/arch/mips/kernel/syscall.c
index 735733f..c8
From: Wei Yongjun
Remove duplicated include.
Signed-off-by: Wei Yongjun
---
arch/mips/kernel/syscall.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/mips/kernel/syscall.c b/arch/mips/kernel/syscall.c
index 735733f..c86ddba 100644
--- a/arch/mips/kernel/syscall.c
+++ b/arch/mips
From: Wei Yongjun <weiyongj...@huawei.com>
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 <weiyongj...@huawei.com>
---
drivers/soc/zte/zx2967_pm_domains.c | 4 +---
1 fil
From: Wei Yongjun <weiyongj...@huawei.com>
Remove .owner field if calls are used which set it automatically.
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/soc/zte/zx296718_pm_domains.c | 1 -
1 fil
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/soc/zte/zx2967_pm_domains.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/soc
From: Wei Yongjun
Remove .owner field if calls are used which set it automatically.
Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci
Signed-off-by: Wei Yongjun
---
drivers/soc/zte/zx296718_pm_domains.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/soc/zte
From: Wei Yongjun <weiyongj...@huawei.com>
It's not necessary to unregister a component registered
with devm_snd_soc_register_component(). Also removed
pointness clk_disable_unprepare() from error path and
snd_soc_unregister_platform() from the remove.
Fixes: f8260afa444b ("ASoC:
From: Wei Yongjun
It's not necessary to unregister a component registered
with devm_snd_soc_register_component(). Also removed
pointness clk_disable_unprepare() from error path and
snd_soc_unregister_platform() from the remove.
Fixes: f8260afa444b ("ASoC: sunxi: Add support for the SPDIF
From: Wei Yongjun <weiyongj...@huawei.com>
Fixes the following sparse warnings:
sound/soc/codecs/hdmi-codec.c:34:1: warning:
symbol 'hdmi_device_list' was not declared. Should it be static?
sound/soc/codecs/hdmi-codec.c:80:33: warning:
symbol 'hdmi_codec_stereo_chmaps' was not declared.
From: Wei Yongjun
Fixes the following sparse warnings:
sound/soc/codecs/hdmi-codec.c:34:1: warning:
symbol 'hdmi_device_list' was not declared. Should it be static?
sound/soc/codecs/hdmi-codec.c:80:33: warning:
symbol 'hdmi_codec_stereo_chmaps' was not declared. Should it be static?
sound/soc
From: Wei Yongjun <weiyongj...@huawei.com>
Fix to return error code -ENODEV from the of_find_compatible_node()
error handling case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
v1 -> v2: fix file module
---
arc
From: Wei Yongjun
Fix to return error code -ENODEV from the of_find_compatible_node()
error handling case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
---
v1 -> v2: fix file module
---
arch/arm/mach-hisi/platmcpm.c | 2 ++
1 file changed, 2 insertions(+)
d
From: Wei Yongjun <weiyongj...@huawei.com>
Fixes the following sparse warning:
drivers/gpu/drm/mxsfb/mxsfb_drv.c:129:38: warning:
symbol 'mxsfb_funcs' was not declared. Should it be static?
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/gpu/drm/mxsfb/mxsfb_drv
From: Wei Yongjun
Fixes the following sparse warning:
drivers/gpu/drm/mxsfb/mxsfb_drv.c:129:38: warning:
symbol 'mxsfb_funcs' was not declared. Should it be static?
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/mxsfb/mxsfb_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
From: Wei Yongjun <weiyongj...@huawei.com>
Fix to return a negative error code from the error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/gpu/drm/mxsfb/mxsfb_drv.c | 1 +
1 file changed, 1 insert
From: Wei Yongjun
Fix to return a negative error code from the error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/mxsfb/mxsfb_drv.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/mxsfb/mxsfb_drv.c
b
From: Wei Yongjun <weiyongj...@huawei.com>
'entries' is malloced in nvme_nvm_get_l2p_tbl() and should be freed
before leaving from the error handling cases, otherwise it will cause
memory leak.
Fixes: 8e79b5cb1d3b ("lightnvm: move block provisioning to targets")
Signed-off
From: Wei Yongjun
'entries' is malloced in nvme_nvm_get_l2p_tbl() and should be freed
before leaving from the error handling cases, otherwise it will cause
memory leak.
Fixes: 8e79b5cb1d3b ("lightnvm: move block provisioning to targets")
Signed-off-by: Wei Yongjun
---
drivers
From: Wei Yongjun <weiyongj...@huawei.com>
It's not necessary to free memory allocated with devm_kzalloc
and using kfree leads to a double free.
Fixes: cfad6425382e ("eeprom: Add IDT 89HPESx EEPROM/CSR driver")
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
--
From: Wei Yongjun
It's not necessary to free memory allocated with devm_kzalloc
and using kfree leads to a double free.
Fixes: cfad6425382e ("eeprom: Add IDT 89HPESx EEPROM/CSR driver")
Signed-off-by: Wei Yongjun
---
drivers/misc/eeprom/idt_89hpesx.c | 4
1 file changed, 4
From: Wei Yongjun <weiyongj...@huawei.com>
Fix to return error code -ENODEV from the of_find_compatible_node()
error handling case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
arch/arm/mach-hisi/platmcpm.c | 2 ++
1 fil
From: Wei Yongjun
Fix to return error code -ENODEV from the of_find_compatible_node()
error handling case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun
---
arch/arm/mach-hisi/platmcpm.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/mach-hisi
From: Wei Yongjun <weiyongj...@huawei.com>
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 <weiyongj...@huawei.com>
---
drivers/mtd/nand/mtk_nand.c | 1 -
1 file changed, 1 delet
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/mtd/nand/mtk_nand.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/mtd/nand/mtk_nand.c b/drivers/mtd
From: Wei Yongjun <weiyongj...@huawei.com>
PTR_ERR should access the value just tested by IS_ERR, otherwise
the wrong error code will be returned.
Fixes: d1667b86795a ("drm/hisilicon/hibmc: Add support for frame buffer")
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
From: Wei Yongjun
PTR_ERR should access the value just tested by IS_ERR, otherwise
the wrong error code will be returned.
Fixes: d1667b86795a ("drm/hisilicon/hibmc: Add support for frame buffer")
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c | 2
From: Wei Yongjun <weiyongj...@huawei.com>
Fix typo in parameter description.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
arch/powerpc/platforms/pseries/cmm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/platforms/pseries/cmm.c
b/
From: Wei Yongjun
Fix typo in parameter description.
Signed-off-by: Wei Yongjun
---
arch/powerpc/platforms/pseries/cmm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/platforms/pseries/cmm.c
b/arch/powerpc/platforms/pseries/cmm.c
index 66e7227..7af18da
From: Wei Yongjun <weiyongj...@huawei.com>
Fix to return a negative error code from the kthread_run() error
handling case instead of 0, as done elsewhere in this function.
Fixes: cdd5de500b2c ("soc: ti: Add wkup_m3_ipc driver")
Signed-off-by: Wei Yongjun <weiyongj...@huawei.co
From: Wei Yongjun
Fix to return a negative error code from the kthread_run() error
handling case instead of 0, as done elsewhere in this function.
Fixes: cdd5de500b2c ("soc: ti: Add wkup_m3_ipc driver")
Signed-off-by: Wei Yongjun
---
drivers/soc/ti/wkup_m3_ipc.c | 1 +
1 file
From: Wei Yongjun <weiyongj...@huawei.com>
Fixes the following sparse warning:
drivers/gpu/drm/drm_atomic_helper.c:1360:6: warning:
symbol 'release_crtc_commit' was not declared. Should it be static?
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/gpu/drm/drm_ato
From: Wei Yongjun
Fixes the following sparse warning:
drivers/gpu/drm/drm_atomic_helper.c:1360:6: warning:
symbol 'release_crtc_commit' was not declared. Should it be static?
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/drm_atomic_helper.c | 2 +-
1 file changed, 1 insertion(+), 1
From: Wei Yongjun <weiyongj...@huawei.com>
In case of error, the function kthread_run() 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 <weiyongj...@huawei.com>
---
kernel/trace/trace_ben
From: Wei Yongjun
In case of error, the function kthread_run() 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
---
kernel/trace/trace_benchmark.c | 4 ++--
1 file changed, 2 insertions(+), 2
From: Wei Yongjun <weiyongj...@huawei.com>
Fix a typo in MODULE_DEVICE_TABLE(). 'pmic_acpi_ids' should be
'bxtwc_acpi_ids'.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/mfd/intel_soc_pmic_bxtwc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a
From: Wei Yongjun
Fix a typo in MODULE_DEVICE_TABLE(). 'pmic_acpi_ids' should be
'bxtwc_acpi_ids'.
Signed-off-by: Wei Yongjun
---
drivers/mfd/intel_soc_pmic_bxtwc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mfd/intel_soc_pmic_bxtwc.c
b/drivers/mfd
On 10/30/2016 11:57 AM, Jens Axboe wrote:
> On 10/29/2016 10:10 AM, Wei Yongjun wrote:
>> From: Wei Yongjun <weiyongj...@huawei.com>
>>
>> Fixes the following sparse warning:
>>
>> drivers/block/brd.c:411:15: warning:
>> symbol 'rd_size' was not dec
On 10/30/2016 11:57 AM, Jens Axboe wrote:
> On 10/29/2016 10:10 AM, Wei Yongjun wrote:
>> From: Wei Yongjun
>>
>> Fixes the following sparse warning:
>>
>> drivers/block/brd.c:411:15: warning:
>> symbol 'rd_size' was not declared. Should it be static?
&
From: Wei Yongjun <weiyongj...@huawei.com>
Fixes the following sparse warnings:
sound/soc/codecs/wm8580.c:988:33: warning:
symbol 'wm8580_data' was not declared. Should it be static?
sound/soc/codecs/wm8580.c:992:33: warning:
symbol 'wm8581_data' was not declared. Should it be static?
From: Wei Yongjun
Fixes the following sparse warnings:
sound/soc/codecs/wm8580.c:988:33: warning:
symbol 'wm8580_data' was not declared. Should it be static?
sound/soc/codecs/wm8580.c:992:33: warning:
symbol 'wm8581_data' was not declared. Should it be static?
Signed-off-by: Wei Yongjun
From: Wei Yongjun <weiyongj...@huawei.com>
Fixes the following sparse warnings:
sound/soc/codecs/wm8580.c:988:33: warning:
symbol 'wm8580_data' was not declared. Should it be static?
sound/soc/codecs/wm8580.c:992:33: warning:
symbol 'wm8581_data' was not declared. Should it be static?
From: Wei Yongjun
Fixes the following sparse warnings:
sound/soc/codecs/wm8580.c:988:33: warning:
symbol 'wm8580_data' was not declared. Should it be static?
sound/soc/codecs/wm8580.c:992:33: warning:
symbol 'wm8581_data' was not declared. Should it be static?
Signed-off-by: Wei Yongjun
From: Wei Yongjun <weiyongj...@huawei.com>
Fixes the following sparse warning:
drivers/block/brd.c:411:15: warning:
symbol 'rd_size' was not declared. Should it be static?
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
drivers/block/brd.c | 2 +-
1 file changed, 1 ins
From: Wei Yongjun
Fixes the following sparse warning:
drivers/block/brd.c:411:15: warning:
symbol 'rd_size' was not declared. Should it be static?
Signed-off-by: Wei Yongjun
---
drivers/block/brd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/block/brd.c b
From: Wei Yongjun <weiyongj...@huawei.com>
Make sure of_device_id tables are NULL terminated.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
arch/arm/mach-mediatek/platsmp.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/mach-mediatek/platsmp.c b/arch/arm/m
From: Wei Yongjun
Make sure of_device_id tables are NULL terminated.
Signed-off-by: Wei Yongjun
---
arch/arm/mach-mediatek/platsmp.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/mach-mediatek/platsmp.c b/arch/arm/mach-mediatek/platsmp.c
index b821e34..e6cffc7 100644
From: Wei Yongjun <weiyongj...@huawei.com>
Make sure dmi_system_id tables are NULL terminated.
Signed-off-by: Wei Yongjun <weiyongj...@huawei.com>
---
sound/soc/intel/atom/sst/sst_acpi.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/soc/intel/atom/sst/sst_acpi.c
b/sou
301 - 400 of 1371 matches
Mail list logo