[PATCH] drm/loongson: Fix return value check for kzalloc

2025-04-14 Thread Chen Ni
kzalloc doesn't return ERR_PTR, delete unneeded IS_ERR test. Signed-off-by: Chen Ni --- drivers/gpu/drm/loongson/lsdc_gfxpll.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/loongson/lsdc_gfxpll.c b/drivers/gpu/drm/loongson/lsdc_gfxpll.c index 249c09d

[PATCH] drm/msm: Convert comma to semicolon

2025-04-10 Thread Chen Ni
Replace comma between expressions with semicolons. Using a ',' in place of a ';' can have unintended side effects. Although that is not the case here, it is seems best to use ';' unless ',' is intended. Found by inspection. No functional change intended. C

[PATCH] drm/komeda: Remove unnecessary NULL check before clk_prepare_enable()

2025-03-13 Thread Chen Ni
clk_prepare_enable() already checked NULL clock parameter. Remove unneeded NULL check for clk here. Signed-off-by: Chen Ni --- drivers/gpu/drm/arm/display/komeda/komeda_dev.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b

[PATCH] drm/omap: Remove unnecessary NULL check before clk_prepare_enable/clk_disable_unprepare

2025-03-13 Thread Chen Ni
clk_prepare_enable() and clk_disable_unprepare() already checked NULL clock parameter.Remove unneeded NULL check for clk here. Signed-off-by: Chen Ni --- drivers/gpu/drm/omapdrm/dss/venc.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/omapdrm/dss

[PATCH] drm/mxsfb: Remove unnecessary NULL check before clk_prepare_enable/clk_disable_unprepare

2025-03-13 Thread Chen Ni
clk_prepare_enable() and clk_disable_unprepare() already checked NULL clock parameter.Remove unneeded NULL check for clk here. Signed-off-by: Chen Ni --- drivers/gpu/drm/mxsfb/mxsfb_kms.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/mxsfb/mxsfb_kms.c

[PATCH] accel/qaic: Remove redundant 'flush_workqueue()' calls

2025-03-12 Thread Chen Ni
'destroy_workqueue()' already drains the queue before destroying it, so there is no need to flush it explicitly. Remove the redundant 'flush_workqueue()' calls. This was generated with coccinelle: @@ expression E; @@ - flush_workqueue(E); destroy_workqueue(E); Si

[PATCH] drm/bridge: anx7625: Remove redundant 'flush_workqueue()' calls

2025-03-12 Thread Chen Ni
'destroy_workqueue()' already drains the queue before destroying it, so there is no need to flush it explicitly. Remove the redundant 'flush_workqueue()' calls. This was generated with coccinelle: @@ expression E; @@ - flush_workqueue(E); destroy_workqueue(E); Si

[PATCH net-next] sfc: remove newlines in NL_SET_ERR_MSG_MOD

2024-09-18 Thread Chen Ni
Fix following coccicheck warning: ./drivers/net/ethernet/sfc/tc.c:1584:29-80: WARNING avoid newline at end of message in NL_SET_ERR_MSG_MOD Signed-off-by: Chen Ni --- drivers/net/ethernet/sfc/tc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/sfc/tc.c

[PATCH] drm/fsl-dcu: Remove redundant dev_err()

2024-09-18 Thread Chen Ni
There is no need to call the dev_err() function directly to print a custom message when handling an error from platform_get_irq() function as it is going to display an appropriate error message in case of a failure. Signed-off-by: Chen Ni --- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c | 4

[PATCH] drm/i915: convert comma to semicolon

2024-09-02 Thread Chen Ni
Replace a comma between expression statements by a semicolon. Signed-off-by: Chen Ni --- drivers/gpu/drm/i915/display/intel_hdmi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_hdmi.c b/drivers/gpu/drm/i915/display/intel_hdmi.c index

[PATCH] fbdev: imsttfb: convert comma to semicolon

2024-09-02 Thread Chen Ni
Replace a comma between expression statements by a semicolon. Signed-off-by: Chen Ni --- drivers/video/fbdev/imsttfb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/video/fbdev/imsttfb.c b/drivers/video/fbdev/imsttfb.c index 660499260f46..dc4e659e06af 100644

[PATCH] fbdev: pxa3xx-gcu: Convert comma to semicolon

2024-09-02 Thread Chen Ni
Replace a comma between expression statements by a semicolon. Signed-off-by: Chen Ni --- drivers/video/fbdev/pxa3xx-gcu.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/video/fbdev/pxa3xx-gcu.c b/drivers/video/fbdev/pxa3xx-gcu.c index 43c80316d84b..489088b4e467

[PATCH] fbdev/hyperv_fb: Convert comma to semicolon

2024-09-02 Thread Chen Ni
Replace a comma between expression statements by a semicolon. Fixes: d786e00d19f9 ("drivers: hv, hyperv_fb: Untangle and refactor Hyper-V panic notifiers") Signed-off-by: Chen Ni --- drivers/video/fbdev/hyperv_fb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v2] backlight: l4f00242t03: Add check for spi_setup

2024-07-05 Thread Chen Ni
Add check for the return value of spi_setup() and return the error if it fails in order to catch the error. Signed-off-by: Chen Ni --- Changelog: v1 -> v2: 1. Use dev_err_probe() to match other error paths --- drivers/video/backlight/l4f00242t03.c | 5 - 1 file changed, 4 insertions(+)

[PATCH] backlight: l4f00242t03: Add check for spi_setup

2024-07-05 Thread Chen Ni
Add check for the return value of spi_setup() and return the error if it fails in order to catch the error. Signed-off-by: Chen Ni --- drivers/video/backlight/l4f00242t03.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/video/backlight/l4f00242t03.c b/drivers

[PATCH] fbdev: omap2: Return clk_prepare_enable to transfer the error

2024-06-27 Thread Chen Ni
Return clk_prepare_enable() in order to transfer the error if it fails. Signed-off-by: Chen Ni --- drivers/video/fbdev/omap2/omapfb/dss/venc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/video/fbdev/omap2/omapfb/dss/venc.c b/drivers/video/fbdev/omap2/omapfb

[PATCH] drm/qxl: Add check for drm_cvt_mode

2024-06-21 Thread Chen Ni
Add check for the return value of drm_cvt_mode() and return the error if it fails in order to avoid NULL pointer dereference. Fixes: 1b043677d4be ("drm/qxl: add qxl_add_mode helper function") Signed-off-by: Chen Ni --- drivers/gpu/drm/qxl/qxl_display.c | 3 +++ 1 file changed, 3

[PATCH 2/2] drm/panel-xinpeng-xpp055c272: add check for mipi_dsi_dcs_enter_sleep_mode

2024-06-20 Thread Chen Ni
Add check for the return value of mipi_dsi_dcs_enter_sleep_mode() and return the error if it fails in order to catch the error. Signed-off-by: Chen Ni --- drivers/gpu/drm/panel/panel-xinpeng-xpp055c272.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/panel

[PATCH 1/2] drm/panel: ltk050h3146w: add check for mipi_dsi_dcs_enter_sleep_mode

2024-06-20 Thread Chen Ni
Add check for the return value of mipi_dsi_dcs_enter_sleep_mode() and return the error if it fails in order to catch the error. Signed-off-by: Chen Ni --- drivers/gpu/drm/panel/panel-leadtek-ltk050h3146w.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/panel

[PATCH] drm/panel: sitronix-st7789v: Add check for of_drm_get_panel_orientation

2024-05-28 Thread Chen Ni
Add check for the return value of of_drm_get_panel_orientation() and return the error if it fails in order to catch the error. Fixes: b27c0f6d208d ("drm/panel: sitronix-st7789v: add panel orientation support") Signed-off-by: Chen Ni --- drivers/gpu/drm/panel/panel-sitronix-st7

[PATCH] drm/bochs: Add check for drm_simple_display_pipe_init

2024-05-23 Thread Chen Ni
Add check for the return value of drm_simple_display_pipe_init() and return the error if it fails in order to catch the error. Signed-off-by: Chen Ni --- drivers/gpu/drm/tiny/bochs.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/tiny/bochs.c b/drivers

[PATCH] drm/exynos: Add check for dma_set_max_seg_size

2024-04-29 Thread Chen Ni
Add check for the return value of dma_set_max_seg_size() and return the error if it fails in order to catch the error. Fixes: ddfd4ab6bb08 ("drm/exynos: Fix dma_parms allocation") Signed-off-by: Chen Ni --- drivers/gpu/drm/exynos/exynos_drm_dma.c | 5 - 1 file changed, 4 insert

[PATCH] drm/tegra: dsi: Add missing check for of_find_device_by_node

2023-10-24 Thread Chen Ni
Add check for the return value of of_find_device_by_node() and return the error if it fails in order to avoid NULL pointer dereference. Fixes: e94236cde4d5 ("drm/tegra: dsi: Add ganged mode support") Signed-off-by: Chen Ni --- drivers/gpu/drm/tegra/dsi.c | 4 +++- 1 file changed, 3