[PATCH -next] drm/mediatek/mtk_mipi_tx: remove redundant dev_err call in mtk_mipi_tx_probe()

2016-07-13 Thread weiyj...@163.com
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/gpu/drm/mediatek/mtk_mipi_tx.c | 7 ++- 1 file changed, 2 insertions(+), 5

[PATCH -next] drm/hisilicon: Remove redundant dev_err call in ade_dts_parse()

2016-07-13 Thread weiyj...@163.com
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/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 4 +--- 1 file changed, 1 insertion(+),

[PATCH -next] drm/hisilicon: Fix return value check in ade_dts_parse()

2016-07-13 Thread weiyj...@163.com
From: Wei Yongjun In case of error, the function devm_clk_get() 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/gpu/drm/hisilicon/kirin/kirin_drm_ade.c |

[PATCH -next] drm/hisilicon: Fix non static symbol warning

2016-07-13 Thread weiyj...@163.com
From: Wei Yongjun Fixes the following sparse warning: drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c:107:5: warning: symbol 'ade_get_channel_formats' was not declared. Should it be static? Signed-off-by: Wei Yongjun ---

[PATCH -next] drm: atmel-hlcdc: fix non static symbol warning

2016-07-13 Thread weiyj...@163.com
From: Wei Yongjun Fixes the following sparse warning: drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c:390:6: warning: symbol 'atmel_hlcdc_crtc_reset' was not declared. Should it be static? Signed-off-by: Wei Yongjun ---

[PATCH -next] drm/msm/hdmi: use PTR_ERR_OR_ZERO() to simplify the code

2016-07-12 Thread weiyj...@163.com
From: Wei Yongjun Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR. Generated by: scripts/coccinelle/api/ptr_ret.cocci Signed-off-by: Wei Yongjun --- drivers/gpu/drm/msm/hdmi/hdmi.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git

[PATCH -next] drm/nouveau/device: fix return value check in nvkm_device_tegra_probe_iommu()

2016-07-06 Thread weiyj...@163.com
From: Wei Yongjun In case of error, the function iommu_domain_alloc() 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 ---

[PATCH -next] drm/tegra: sor: Fix invalid kfree of devm_kzalloc allocated data

2016-07-04 Thread weiyj...@163.com
From: Wei Yongjun data allocated with devm_kzalloc should not be freed using kfree, because doing so causes a dangling pointer, and a subsequent double free, use devm_kfree instead. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/tegra/sor.c | 2 +- 1 file

[PATCH -next] drm/msm/dsi: Fix return value check in msm_dsi_host_set_display_mode()

2016-06-18 Thread weiyj...@163.com
From: Wei Yongjun In case of error, the function drm_mode_duplicate() 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/gpu/drm/msm/dsi/dsi_host.c | 4 ++-- 1

[PATCH -next] drm/hisilicon: Fix return value check in ade_dts_parse()

2016-06-17 Thread weiyj...@163.com
From: Wei Yongjun In case of error, the function devm_clk_get() 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/gpu/drm/hisilicon/kirin/kirin_drm_ade.c |

[PATCH] drm/i915: Fix missing unlock on error in i915_ppgtt_info()

2016-06-14 Thread weiyj...@163.com
From: Wei Yongjun Add the missing unlock before return from function i915_ppgtt_info() in the error handling case. Fixes: 1d2ac403ae3b(drm: Protect dev->filelist with its own mutex) Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/i915_debugfs.c | 4 ++-- 1

[PATCH -next] drm/i915/audio: remove duplicated include from intel_audio.c

2015-04-16 Thread weiyj...@163.com
From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/intel_audio.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c index 2396cc7..d00d488

[PATCH] drm/sti: fix return value check in sti_dvo_probe()

2015-04-16 Thread weiyj...@163.com
From: Wei Yongjun In case of error, the function devm_ioremap_nocache() returns NULL not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/sti/sti_dvo.c | 4 ++-- 1 file

[PATCH -next] drm/i915: Fix missing unlock on error in i915_gem_init_hw()

2014-12-05 Thread weiyj...@163.com
From: Wei Yongjun Add the missing unlock before return from function i915_gem_init_hw() in the error handling case. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/i915_gem.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

[PATCH] drm/msm: Fix missing unlock on error in msm_fbdev_create()

2014-08-14 Thread weiyj...@163.com
From: Wei Yongjun Add the missing unlock before return from function msm_fbdev_create() in the error handling case. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/msm/msm_fbdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] drm: sti: Fix return value check in sti_drm_platform_probe()

2014-08-14 Thread weiyj...@163.com
From: Wei Yongjun In case of error, the function platform_device_register_resndata() 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 ---

[PATCH] drm: sti: hda: fix return value check in sti_hda_probe()

2014-08-14 Thread weiyj...@163.com
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/gpu/drm/sti/sti_hda.c | 8

[PATCH] drm: sti: hdmi: fix return value check in sti_hdmi_probe()

2014-08-14 Thread weiyj...@163.com
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/gpu/drm/sti/sti_hdmi.c | 8

[PATCH] drm: sti: tvout: fix return value check in sti_tvout_probe()

2014-08-14 Thread weiyj...@163.com
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/gpu/drm/sti/sti_tvout.c | 4 ++-- 1

[PATCH -next] drm/i915: Remove duplicated include from intel_dsi_panel_vbt.c

2014-07-28 Thread weiyj...@163.com
From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/intel_dsi_panel_vbt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c b/drivers/gpu/drm/i915/intel_dsi_panel_vbt.c

[PATCH -next] drm: Remove duplicated include from drm_plane_helper.c

2014-07-28 Thread weiyj...@163.com
From: Wei Yongjun Remove duplicated include. Signed-off-by: Wei Yongjun --- drivers/gpu/drm/drm_plane_helper.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/drm_plane_helper.c b/drivers/gpu/drm/drm_plane_helper.c index 6d13314..64ce96c