[PATCH][next] drm/xe/guc: Fix spelling mistake "sheduling" -> "scheduling"

2025-09-12 Thread Colin Ian King
There is a spelling mistake in a xe_gt_err error message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/xe/xe_guc_submit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c index a465594b61dc

[PATCH][next] drm/sysfb: Remove double assignment to pointer crtc_state

2025-09-03 Thread Colin Ian King
The declaration of pointer crtc_state includes an assignment to crtc_state. The double assignment of crtc_state is redundant and can be removed. Fixes: 061963cd9e5b ("drm/sysfb: Blit to CRTC destination format") Signed-off-by: Colin Ian King --- drivers/gpu/drm/sysfb/drm_sysfb_mod

[PATCH][next] drm/amd/amdgpu: Fix a less than zero check on a uint32_t struct field

2025-09-03 Thread Colin Ian King
e the MES INV_TLBS API for tlb invalidation on gfx12") Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/amdgpu/mes_v12_0.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/mes_v12_0.c b/drivers/gpu/drm/amd/amdgpu/mes_v12_0.c index cd

[PATCH][next] drm/amd/amdgpu: Fix missing error return on kzalloc failure

2025-09-02 Thread Colin Ian King
d buf allocations") Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c index 1d6e1d5de8fa..3696f48c233b 100644 --- a/drive

[PATCH][next] drm/bridge: synopsys: Fix spelling mistake "pixeel" -> "pixel"

2025-09-01 Thread Colin Ian King
There is a spelling mistake in a dev_warn message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/bridge/synopsys/dw-dp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/synopsys/dw-dp.c b/drivers/gpu/drm/bridge/synopsys/dw-dp.c index

[PATCH][next] drm/dp: remove redundant check on ret and return statement

2025-08-13 Thread Colin Ian King
There is a redundant check on return and a return statement after a previous return statement from the call to drm_dp_dpcd_write_byte. These statements are redundant and can be removed. Remove the statements and the now unused variable ret. Signed-off-by: Colin Ian King --- drivers/gpu/drm

[PATCH][next] drm/i915/bw: Remove space before newline

2025-08-01 Thread Colin Ian King
There is an extraneous space before a newline in a drm_dbg_kms message. Remove the space. Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/display/intel_bw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_bw.c b/drivers/gpu/drm/i915

[PATCH][next] drm/i915: remove redundant repeated checks on err

2025-08-01 Thread Colin Ian King
There are a couple of redundant repeated checks on err being non-zero that are always true because they are inside a previous check on err being non-zero. Remove the duplicated checks. Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/gem/selftests/i915_gem_migrate.c | 6 ++ 1 file

[PATCH 2/2][next] drm/msm: replace minor->dev->dev with dev->dev

2025-07-31 Thread Colin Ian King
The pointer dev has been set to minor->dev, so replace minor->dev->dev with dev->dev in the DRM_DEV_ERROR messages. Signed-off-by: Colin Ian King --- drivers/gpu/drm/msm/msm_debugfs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/msm/msm_

[PATCH 1/2][next][V2] Fix dereference of pointer minor before null check

2025-07-31 Thread Colin Ian King
evice") Signed-off-by: Colin Ian King --- drivers/gpu/drm/msm/msm_debugfs.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/msm/msm_debugfs.c b/drivers/gpu/drm/msm/msm_debugfs.c index bbda865addae..da618720cf8a 100644 --- a/drivers/gpu/drm/msm/msm_debug

[PATCH 0/2] drm/msm: Fix pointer dereference and use dev->dev

2025-07-31 Thread Colin Ian King
Fix a potential null pointer dereference issue and replace minor->dev->dev with dev->dev in error messages. Colin Ian King (2): Fix dereference of pointer minor before null check drm/msm: replace minor->dev->dev with dev->dev drivers/gpu/drm/msm/msm_debugfs.c | 11 +

[PATCH][next] drm/msm: Fix dereference of pointer minor before null check

2025-07-30 Thread Colin Ian King
it_minor() if it is not a GPU device") Signed-off-by: Colin Ian King --- drivers/gpu/drm/msm/msm_debugfs.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/msm/msm_debugfs.c b/drivers/gpu/drm/msm/msm_debugfs.c index bbda865addae..97dc70876442

[PATCH][next] drm/kmb: Fix dereference of pointer plane before a null check

2025-07-30 Thread Colin Ian King
assigning old_plane_state and new_plance_state and then null pointer checking these. Fixes: 977697e20b3d ("drm/atomic: Pass the full state to planes atomic disable and update") Fixes: 37418bf14c13 ("drm: Use state helper instead of the plane state pointer") Signed-off-by: Colin Ia

[PATCH][next] drm/panel: Kconfig: Fix spelling mistake "pannel" -> "panel"

2025-07-24 Thread Colin Ian King
There is a spelling mistake in the LEDS_BD2606MVV config. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/panel/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig index 09b9f7ff9340..af90ca62e57d

[PATCH][next] drm/msm: remove extraneous semicolon after a statement

2025-07-18 Thread Colin Ian King
There is a statement that has an extraneous semicolon; remove it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/msm/msm_gpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c index c317b25a8162..33caefd0f877

[PATCH][next] drm/ttm: remove redundant ternaray operation on ret

2025-07-02 Thread Colin Ian King
. Clean up the code by replacing the ternary operator with just a simple return of NULL Signed-off-by: Colin Ian King --- drivers/gpu/drm/ttm/ttm_bo_util.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/ttm/ttm_bo_util.c b/drivers/gpu/drm/ttm/ttm_bo_util.c

[PATCH][next] drm/tidss: remove redundant assignment to variable ret

2025-07-02 Thread Colin Ian King
The assignment of zero to variable is redundant as the following continue statement loops back to the start of the loop where ret is assigned a new value from the return to the call to get_parent_dss_vp. Remove assignment. Signed-off-by: Colin Ian King --- drivers/gpu/drm/tidss/tidss_oldi.c | 1

[PATCH][next] drm/vmwgfx: fix missing assignment to ts

2025-06-23 Thread Colin Ian King
The assignment to ts is missing on the call to ktime_to_timespec64. Fix this by adding the missing assignment. Fixes: db6a94b26354 ("drm/vmwgfx: Implement dma_fence_ops properly") Signed-off-by: Colin Ian King --- drivers/gpu/drm/vmwgfx/vmwgfx_fence.c | 2 +- 1 file changed, 1 inser

[PATCH][next] drm/amd: Fix spelling mistake "correctalbe" -> "correctable"

2025-06-23 Thread Colin Ian King
There is a spelling mistake in a pr_info message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c index

[PATCH][next] drm/nouveau/gsp: Fix potential integer overflow on integer shifts

2025-05-22 Thread Colin Ian King
u/gsp: support deeper page tables in COPY_SERVER_RESERVED_PDES") Signed-off-by: Colin Ian King --- drivers/gpu/drm/nouveau/nvkm/subdev/gsp/rm/r535/vmm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/gsp/rm/r535/vmm.c b/drivers/gpu/d

[PATCH][next] drm/display: remove redundant ternary operator

2025-05-06 Thread Colin Ian King
The ternary operator is checking if ret is less than zero inside an if block that also checks if ret is less than zero. Since the nested ternary statement is always true then the -EIO return is never executed and can be removed. Signed-off-by: Colin Ian King --- drivers/gpu/drm/display

[PATCH][next] drm/display: dp: remove duplicated check of ret < 0

2025-04-22 Thread Colin Ian King
There is a redundant duplicated check for the condition of ret < 0 inside an if block that also checks for ret < 0. The ternary check is always true, so remove it and replace it with just return err. Signed-off-by: Colin Ian King --- drivers/gpu/drm/display/drm_dp_helper.c | 2 +-

[PATCH][next] dma-buf/sw_sync: remove redundant error check on variable err

2025-04-22 Thread Colin Ian King
The variable err is always zero at the point where it is being non-zero checked for an error. The check and error return path are redundant and can be removed. Issue detected by Coverity Scan static analysis. Signed-off-by: Colin Ian King --- drivers/dma-buf/sw_sync.c | 3 --- 1 file changed, 3

[PATCH][next] drm/amdgpu: Fix spelling mistake "rounter" -> "rounter"

2025-04-22 Thread Colin Ian King
There is a spelling mistake with the array utcl2_rounter_str, it appears it should be utcl2_router_str. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/amdgpu/gfx_v9_4_2.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_4_2

[PATCH][next] fbdev/carminefb: Fix spelling mistake of CARMINE_TOTAL_DIPLAY_MEM

2025-04-18 Thread Colin Ian King
There is a spelling mistake in macro CARMINE_TOTAL_DIPLAY_MEM. Fix it. Signed-off-by: Colin Ian King --- drivers/video/fbdev/carminefb.c | 8 drivers/video/fbdev/carminefb.h | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/video/fbdev/carminefb.c b/drivers

[PATCH][next] drm/exynos: Fix spelling mistake "enqueu" -> "enqueue"

2025-04-18 Thread Colin Ian King
There is a spelling mistake in a DRM_DEV_DEBUG_KMS message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/exynos/exynos_drm_fimc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimc.c b/drivers/gpu/drm/exynos/exynos_drm_fimc.c

[PATCH][next] accel/habanalabs: Make the read-only array edma_queues_id static const

2025-04-17 Thread Colin Ian King
Don't populate the read-only array edma_queues_id on the stack at run time, instead make it static const. Signed-off-by: Colin Ian King --- drivers/accel/habanalabs/gaudi2/gaudi2.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/accel/habanalabs/g

[PATCH][next] ice: make const read-only array dflt_rules static

2025-03-17 Thread Colin Ian King
Don't populate the const read-only array dflt_rules on the stack at run time, instead make it static. Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/intel_memory_region.c| 2 +- drivers/net/ethernet/intel/ice/ice_ethtool_fdir.c | 2 +- 2 files changed, 2 insertions(

[PATCH][next] drm/connector: Fix spelling mistake "provded" -> "provided"

2025-02-28 Thread Colin Ian King
There are spelling mistakes in drm_err messages. Fix them. Signed-off-by: Colin Ian King --- drivers/gpu/drm/drm_connector.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c index 5f24d6b41cc6..9cf347f70643

[PATCH][next] drm/bridge: Fix spelling mistake "gettin" -> "getting"

2025-02-28 Thread Colin Ian King
There is a spelling mistake in a dev_err message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/bridge/sii902x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/sii902x.c b/drivers/gpu/drm/bridge/sii902x.c index 2100a687096e..914a2609a685

[PATCH][next] drm/xe: Fix uninitialized pointer def

2025-02-26 Thread Colin Ian King
g errors from xe_heci_gsc_init()") Signed-off-by: Colin Ian King --- drivers/gpu/drm/xe/xe_heci_gsc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/xe/xe_heci_gsc.c b/drivers/gpu/drm/xe/xe_heci_gsc.c index 3ea325d3db99..27d11e06a82b 100644 --- a/drivers/gpu/drm/xe/x

[PATCH][next] drm/amdgpu: Fix spelling mistake "initiailize" -> "initialize" and grammar

2025-02-26 Thread Colin Ian King
There is a spelling mistake and a grammatical error in a dev_err message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/amdgpu/amdgpu_cper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cper.c b/drivers/gpu/drm/amd/amdgpu

[PATCH][next] drm/amd/display: Fix spelling mistake "oustanding" -> "outstanding"

2025-02-17 Thread Colin Ian King
There is a spelling mistake in max_oustanding_when_urgent_expected, fix it. Signed-off-by: Colin Ian King --- .../dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.c | 6 +++--- .../dc/dml2/dml21/src/dml2_core/dml2_core_shared_types.h| 2 +- 2 files changed, 4 insertions(+), 4 deletions

[PATCH][next] drm/i915/selftest: Fix spelling mistake "ofset" -> "offset"

2025-02-02 Thread Colin Ian King
There is a spelling mistake in an error message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/selftests/i915_gem_gtt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/selftests/i915_gem_gtt.c b/drivers/gpu/drm/i915/selftests

[PATCH][next] drm/amd/display: remove extraneous ; after statements

2025-01-15 Thread Colin Ian King
There are a couple of statements with two following semicolons, replace these with just one semicolon. Signed-off-by: Colin Ian King --- .../dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.c| 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd

[PATCH][next] drm/amd/display: Fix spelling mistake "tolarance" -> "tolerance"

2024-12-19 Thread Colin Ian King
There is a spelling mistake in a dml2_printf message. Fix it. Signed-off-by: Colin Ian King --- .../display/dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_core

[PATCH][next] drm/amd/amdgpu: Fix spelling mistake "versoin" -> "version"

2024-11-13 Thread Colin Ian King
There is a spelling mistake in a dev_err message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/amdgpu/mes_v11_0.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c b/drivers/gpu/drm/amd/amdgpu/mes_v11_0.c index

[PATCH][V2] drm/msm/a5xx: remove null pointer check on pdev

2024-11-12 Thread Colin Ian King
The call chain on a5xx_gpu_init is such that pdev is not going to be null, so the null check on pdev can be removed. This also cleans up a static analysis warning where pdev is dereferenced before the null check which cannot actually occur. Signed-off-by: Colin Ian King --- V2: rewrite Subject

[PATCH] drm/msm/a5xx: Fix dereference of pointer pdev before null check on pdev

2024-11-12 Thread Colin Ian King
t;) Signed-off-by: Colin Ian King --- drivers/gpu/drm/msm/adreno/a5xx_gpu.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/adreno/a5xx_gpu.c b/drivers/gpu/drm/msm/adreno/a5xx_gpu.c index ee89db72e36e..e83081346059 100644 --- a/drivers/gpu/drm/msm/adreno/

[PATCH][next] drm/amd/display: Fix spelling mistake "tunndeling" -> "tunneling"

2024-10-16 Thread Colin Ian King
There is a spelling mistake in a dm_error message. Fix it. Signed-off-by: Colin Ian King --- .../gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c b

[PATCH][next] drm/omap: clean up error exit path on omap_encoder allocation failure

2024-10-15 Thread Colin Ian King
redundant omap_encoder_destroy call, and just return NULL. Signed-off-by: Colin Ian King --- drivers/gpu/drm/omapdrm/omap_encoder.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/gpu/drm/omapdrm/omap_encoder.c b/drivers/gpu/drm/omapdrm/omap_encoder.c index

[PATCH][next] drm/amdgpu: Fix potential integer overflow on shift of a int

2024-10-10 Thread Colin Ian King
: f0b19b84d391 ("drm/amdgpu: add amdgpu_jpeg_sched_mask debugfs") Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/amdgpu/amdgpu_jpeg.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_jpeg.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_jp

[PATCH][next] drm/xe/guc: Fix inverted logic on snapshot->copy check

2024-10-09 Thread Colin Ian King
check logic. Fixes: d8ce1a977226 ("drm/xe/guc: Use a two stage dump for GuC logs and add more info") Signed-off-by: Colin Ian King --- drivers/gpu/drm/xe/xe_guc_log.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/xe/xe_guc_log.c b/drivers/

[PATCH][next] fbdev: clps711x-fb: remove redundant call to unregister_framebuffer

2024-10-09 Thread Colin Ian King
Currently the call to unregister_framebuffer is unreachable code because the previous statement returns from the function. The call is redundant and can be removed. Fixes: 36462ac19308 ("fbdev: clps711x-fb: Replace check_fb in favor of struct fb_info.lcd_dev") Signed-off-by: Coli

[PATCH][next] drm/amdgpu: Fix spelling mistake "initializtion" -> "initialization"

2024-10-02 Thread Colin Ian King
There is a spelling mistake in a dev_err message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/amdgpu/amdgpu_xgmi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_xgmi.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_xgmi.c index

[PATCH][next] drm/i915/display: Fix spelling mistake "Uncomressed" -> "Uncompressed"

2024-10-02 Thread Colin Ian King
There is a spelling mistake in a drm_WARN message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/display/intel_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display

[PATCH][next] drm/nouveau/gsp: remove extraneous ; after mutex

2024-09-17 Thread Colin Ian King
The mutex field has two following semicolons, replace this with just one semicolon. Signed-off-by: Colin Ian King --- drivers/gpu/drm/nouveau/include/nvkm/subdev/gsp.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/include/nvkm/subdev/gsp.h b

[PATCH][next] drm/msm8998: make const arrays ratio_list and band_list static

2024-09-12 Thread Colin Ian King
Don't populate the const read-only arrays ratio_list and band_list on the stack at run time, instead make them static. Signed-off-by: Colin Ian King --- drivers/gpu/drm/msm/hdmi/hdmi_phy_8998.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/msm

[PATCH][next] accel/habanalabs/gaudi2: Make read-only array edma_queues_id static const

2024-09-12 Thread Colin Ian King
Don't populate the read-only array edma_queues_id on the stack at run time, instead make it static const. Signed-off-by: Colin Ian King --- drivers/accel/habanalabs/gaudi2/gaudi2.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/accel/habanalabs/g

[PATCH][next] drm/amd/display: Fix spelling mistake "recompte" -> "recompute"

2024-08-28 Thread Colin Ian King
There is a spelling mistake in a DRM_DEBUG_DRIVER message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c b

[PATCH][next] drm/tegra: hdmi: make read-only const array possible_nvram_sizes static

2024-08-22 Thread Colin Ian King
Don't populate the const read-only array possible_nvram_sizes on the stack at run time, instead make it static. Signed-off-by: Colin Ian King --- drivers/gpu/drm/tegra/hdmi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/tegra/hdmi.c b/drivers/gpu/drm/

[PATCH][next] drm/amd/display: remove extraneous ; after statements

2024-08-02 Thread Colin Ian King
There are a several statements with two following semicolons, replace these with just one semicolon. Signed-off-by: Colin Ian King --- .../drm/amd/display/dc/dml2/dml21/dml21_translation_helper.c | 2 +- .../dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.c| 2 +- .../display/dc/dml2

[PATCH][next] drm/amd/display: Fix spelling mistake "tolarance" -> "tolerance"

2024-07-24 Thread Colin Ian King
There is a spelling mistake in a dml2_printf message. Fix it. Signed-off-by: Colin Ian King --- .../display/dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_core

[PATCH][next] drm/i915/dp: Make read-only array bw_gbps static const

2024-07-22 Thread Colin Ian King
Don't populate the read-only array bw_gbps on the stack at run time, instead make it static const. Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/display/intel_dp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/driver

[PATCH][next] drm/amd/display: Fix a handful of spelling mistakes

2024-05-31 Thread Colin Ian King
There are a few spelling mistakes in dml2_printf messages. Fix them. Signed-off-by: Colin Ian King --- .../dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.c | 6 +++--- .../display/dc/dml2/dml21/src/dml2_core/dml2_core_shared.c | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions

[PATCH][next] drm/xe: Fix spelling mistake "forcebly" -> "forcibly"

2024-04-26 Thread Colin Ian King
There is a spelling mistake in a drm_dbg message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/xe/xe_device.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c index bb4fbae5edb6..b61f8356e23e 100644

[PATCH][next] drm/amd/display: Fix spelling various spelling mistakes

2024-04-24 Thread Colin Ian King
There are various spelling mistakes in dml2_printf messages, fix them. Signed-off-by: Colin Ian King --- .../dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.c | 6 +++--- .../display/dc/dml2/dml21/src/dml2_core/dml2_core_shared.c | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions

[PATCH][next] drm/amdgpu: Fix spelling mistake "PRORITY" -> "PRIORITY"

2024-04-24 Thread Colin Ian King
There are spelling mistakes in a literal string and enums, fix these. Currently there are no uses of the enums that got renamed in this fix. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/amdgpu/mes_v11_0.c| 2 +- drivers/gpu/drm/amd/include/mes_api_def.h | 2 +- drivers/gpu

[PATCH][next] drm/i915: remove redundant assignement to variable err

2024-04-15 Thread Colin Ian King
:1075:5: warning: Value stored to 'err' is never read [deadcode.DeadStores] Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c b/d

[PATCH][next] drm/panthor: Fix spelling mistake "readyness" -> "readiness"

2024-03-26 Thread Colin Ian King
There is a spelling mistake in a drm_err message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/panthor/panthor_gpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/panthor/panthor_gpu.c b/drivers/gpu/drm/panthor/panthor_gpu.c index 6dbbc4cfbe7e

[PATCH][next] drm/nouveau/gr/gf100: Remove second semicolon

2024-03-15 Thread Colin Ian King
There is a statement with two semicolons. Remove the second one, it is redundant. Signed-off-by: Colin Ian King --- drivers/gpu/drm/nouveau/nvkm/engine/gr/gf100.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/gr/gf100.c b/drivers/gpu

[PATCH][next] drm/gma500: Fix spelling mistake "patter" -> "pattern"

2024-03-14 Thread Colin Ian King
There is a spelling mistake in a DRM_DEBUG_KMS message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/gma500/cdv_intel_dp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/gma500/cdv_intel_dp.c b/drivers/gpu/drm/gma500/cdv_intel_dp.c index

[PATCH][next] drm/qxl: remove unused variable num_relocs

2024-03-07 Thread Colin Ian King
[-Wunused-but-set-variable] Signed-off-by: Colin Ian King --- drivers/gpu/drm/qxl/qxl_ioctl.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/qxl/qxl_ioctl.c b/drivers/gpu/drm/qxl/qxl_ioctl.c index dd0f834d881c..506ae1f5e099 100644 --- a/drivers/gpu/drm/qxl/q

[PATCH][next] drm/msm: remove unused variable 'out'

2024-03-07 Thread Colin Ian King
unused-but-set-variable] Signed-off-by: Colin Ian King --- drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c b/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c index 1f5245fc2cdc..d4e1ebfcb021 100644 --- a/drive

[PATCH][next] staging: fbtft: remove unused variable 'count'

2024-02-29 Thread Colin Ian King
ed-but-set-variable] Signed-off-by: Colin Ian King --- drivers/staging/fbtft/fbtft-core.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/fbtft/fbtft-core.c b/drivers/staging/fbtft/fbtft-core.c index 68add4d598ae..38845f23023f 100644 --- a/drivers/staging/fbtft/fbtft-core.c +++

[PATCH][next] drm/i915/dp: Fix spelling mistake "redect" -> "reject"

2024-02-28 Thread Colin Ian King
There is a spelling mistake in a drm_dbg_kms message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/display/intel_dp_tunnel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp_tunnel.c b/drivers/gpu/drm/i915/display

[PATCH][next] agp/amd64: remove redundant assignment to variable i

2024-02-15 Thread Colin Ian King
never read [deadcode.DeadStores] Signed-off-by: Colin Ian King --- drivers/char/agp/amd64-agp.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/char/agp/amd64-agp.c b/drivers/char/agp/amd64-agp.c index ce8651436609..47bd3b8a54b4 100644 --- a/drivers/char/agp/amd64-agp.c +++ b/drivers/char

[PATCH][next] gpu: host1x: remove redundant assignment to variable space

2024-02-15 Thread Colin Ian King
'space' during its initialization is never read [deadcode.DeadStores] Signed-off-by: Colin Ian King --- drivers/gpu/host1x/cdma.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/host1x/cdma.c b/drivers/gpu/host1x/cdma.c index d1336e438f4f..407ed9b9c

[PATCH][next] drm/msm/dp: Fix spelling mistake "enale" -> "enable"

2024-02-12 Thread Colin Ian King
There is a spelling mistake in a drm_dbg_dp message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/msm/dp/dp_ctrl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/dp/dp_ctrl.c b/drivers/gpu/drm/msm/dp/dp_ctrl.c index 03bbdf865d26

[PATCH][next] drm/i915/gvt: remove redundant assignment to pointer map

2024-02-09 Thread Colin Ian King
during its initialization is never read [deadcode.DeadStores] Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/gvt/interrupt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/gvt/interrupt.c b/drivers/gpu/drm/i915/gvt/interrupt.c index c8e7dfc9f791..8c

[PATCH][next] drm/nouveau/fifo/gk104: remove redundant variable ret

2024-01-16 Thread Colin Ian King
read from 'ret' [deadcode.DeadStores] Signed-off-by: Colin Ian King --- drivers/gpu/drm/nouveau/nvif/fifo.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nvif/fifo.c b/drivers/gpu/drm/nouveau/nvif/fifo.c index a463289962b2..e96de14ce87e

[PATCH][next] accel/habanalabs: remove redundant assignment to pointer 'input'

2024-01-06 Thread Colin Ian King
From: Colin Ian King The pointer input is assigned a value that is not read, it is being re-assigned again later with the same value. Resolve this by moving the declaration to input into the if block. Cleans up clang scan build warning: warning: Value stored to 'input' during its init

[PATCH][next] drm/xe: Fix spelling mistake "gueue" -> "queue"

2024-01-02 Thread Colin Ian King
There is a spelling mistake in a drm_info message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/xe/xe_wait_user_fence.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/xe/xe_wait_user_fence.c b/drivers/gpu/drm/xe/xe_wait_user_fence.c index

[PATCH] drivers: video: logo: use %u format specifier for unsigned int values

2023-12-19 Thread Colin Ian King
Signed-off-by: Colin Ian King --- drivers/video/logo/pnmtologo.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/video/logo/pnmtologo.c b/drivers/video/logo/pnmtologo.c index ada5ef6e51b7..2434a25afb64 100644 --- a/drivers/video/logo/pnmtologo.c +++ b/drivers/vi

[PATCH] drm/nouveau/therm: remove redundant duty == target check

2023-12-19 Thread Colin Ian King
:93:17: warning: Opposite inner 'if' condition leads to a dead code block. [oppositeInnerCondition] Fixes: e4311ee51d1e ("drm/nouveau/therm: remove ineffective workarounds for alarm bugs") Signed-off-by: Colin Ian King --- drivers/gpu/drm/nouveau/nvkm/subdev/therm/fan.c | 2 -

[PATCH] drm/amd/display: remove redundant initialization of variable remainder

2023-12-19 Thread Colin Ian King
Variable remainder is being initialized with a value that is never read, the assignment is redundant and can be removed. Also add a newline after the declaration to clean up the coding style. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/display/dc/basics/conversion.c | 3 ++- 1 file

[PATCH][next] drm/i915/selftests: Fix spelling mistake "initialiased" -> "initialised"

2023-12-09 Thread Colin Ian King
There is a spelling mistake in a pr_err error message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/gem/selftests/i915_gem_dmabuf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/gem/selftests/i915_gem_dmabuf.c b/drivers/gpu/drm/i915

[PATCH][next] drm/gma500: Fix spelling mistake "patter" -> "pattern"

2023-12-09 Thread Colin Ian King
There is a spelling mistake in a DRM_DEBUG_KMS message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/gma500/cdv_intel_dp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/gma500/cdv_intel_dp.c b/drivers/gpu/drm/gma500/cdv_intel_dp.c index

[PATCH][next] drm/amd/display: Fix spelling mistake "SMC_MSG_AllowZstatesEntr" -> "SMC_MSG_AllowZstatesEntry"

2023-12-07 Thread Colin Ian King
There is a spelling mistake in a smu_print message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/display/dc/clk_mgr/dcn35/dcn35_smu.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/clk_mgr/dcn35/dcn35_smu.c b/drivers

[PATCH][next] drm/imagination: Fix a couple of spelling mistakes in literal strings

2023-11-24 Thread Colin Ian King
There are a couple of spelling mistakes in literal strings in the stid_fmts array. Fix these. Signed-off-by: Colin Ian King --- drivers/gpu/drm/imagination/pvr_rogue_fwif_sf.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/imagination/pvr_rogue_fwif_sf.h

[PATCH][next] agp/amd64: remove redundant assignment to variable i

2023-10-23 Thread Colin Ian King
The variable i is being initialized to 0 and then again in the following for-loop. The initialization outside the for-loop is redundant and can be removed. Signed-off-by: Colin Ian King --- drivers/char/agp/amd64-agp.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/char/agp/amd64

[PATCH][next] drm/amd/display: Fix a handful of spelling mistakes in dml_print output

2023-10-17 Thread Colin Ian King
There are a few spelling mistakes and an minor grammatical issue in some dml_print messages. Fix these. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/display/dc/dml2/display_mode_core.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/amd/display

[PATCH][next] drm/amd: Fix spelling mistake "throtting" -> "throttling"

2023-08-23 Thread Colin Ian King
There is a spelling mistake in variable throtting_events, rename it to throttling_events. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/pm/swsmu/smu11/arcturus_ppt.c| 6 +++--- drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c | 6 +++--- drivers/gpu/drm/amd/pm/swsmu/smu13

[PATCH][next][V2] agp/amd64: remove redundant assignment to variable i

2023-07-26 Thread Colin Ian King
The variable i is being initialized with a value that is never read, it is being re-assigned in the for-loop statement immediately afterwards. Remove it. Signed-off-by: Colin Ian King --- drivers/char/agp/amd64-agp.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/char/agp/amd64

[PATCH][V2][next] accel/qaic: remove redundant pointer pexec

2023-07-26 Thread Colin Ian King
Pointer pexec is being assigned a value however it is never read. The assignment is redundant and can be removed. Replace sizeof(*pexec) with sizeof the type and remove the declaration of pointer pexec. Signed-off-by: Colin Ian King --- V2: completely remove the declaration of pexec and replace

[PATCH][next] accel/qaic: remove redundant assignment to pointer pexec

2023-07-25 Thread Colin Ian King
Pointer pexec is being assigned a value however it is never read. The assignment is redundant and can be removed. Signed-off-by: Colin Ian King --- drivers/accel/qaic/qaic_data.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/accel/qaic/qaic_data.c b/drivers/accel/qaic/qaic_data.c

[PATCH][next] drm/nouveau/bios/therm: make read-only array duty_lut static const

2023-07-12 Thread Colin Ian King
Don't populate the read-only array on the stack, instead make it static const. Signed-off-by: Colin Ian King --- drivers/gpu/drm/nouveau/nvkm/subdev/bios/therm.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/bios/therm.c b/dr

[PATCH][next][V2] video: fbdev: kyro: make some const read-only arrays static and reduce type size

2023-07-12 Thread Colin Ian King
Don't populate the const read-only arrays on the stack but instead make them static const. Use smaller types to use less storage for the arrays. Also makes the object code a little smaller. Signed-off-by: Colin Ian King --- V2: Use smaller int types, kudos to Helge Deller for suggesting

[PATCH][next] video: fbdev: kyro: make some const read-only arrays static

2023-07-12 Thread Colin Ian King
Don't populate the const read-only arrays on the stack but instead make them static const. Also makes the object code a little smaller. Signed-off-by: Colin Ian King --- drivers/video/fbdev/kyro/STG4000InitDevice.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --

[PATCH][next] drm/edid: make read-only const array static

2023-06-27 Thread Colin Ian King
Don't populate the const array on the stack, instead make it static. Signed-off-by: Colin Ian King --- drivers/gpu/drm/drm_edid.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index e0dbd9140726..39b13417b749 1

[PATCH][next] drm/i915/mtl: Fix spelling mistake "initate" -> "initiate"

2023-06-26 Thread Colin Ian King
There is a spelling mistake in a drm_dbg_kms message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/display/intel_pmdemand.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_pmdemand.c b/drivers/gpu/drm/i915/display

[PATCH][next] drm/connector: Fix spelling mistake "provded" -> "provided"

2023-06-09 Thread Colin Ian King
There are two spelling mistakes in drm_err messages. Fix them. Signed-off-by: Colin Ian King --- drivers/gpu/drm/drm_connector.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c index 3ed4cfcb350c

[PATCH][next] drm/i915/mtl: Fix spelling mistake "initate" -> "initiate"

2023-06-08 Thread Colin Ian King
There is a spelling mistake in a drm_dbg_kms message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/display/intel_pmdemand.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_pmdemand.c b/drivers/gpu/drm/i915/display

[PATCH][next] drm/amdgpu: Fix a couple of spelling mistakes in info and debug messages

2023-05-22 Thread Colin Ian King
There are a couple of spelling mistakes, one in a dev_info message and the other in a dev_debug message. Fix them. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_xcp.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions

[PATCH][next] drm/amd/pm: Fix spelling mistake "aquire" -> "acquire"

2023-04-17 Thread Colin Ian King
There is a spelling mistake in the smu_i2c_bus_access prototype. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/amd/pm/powerplay/inc/hwmgr.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/pm/powerplay/inc/hwmgr.h b/drivers/gpu/drm/amd/pm

[PATCH][next] drm/msm/mdss: Fix spelling mistake "Unuspported" -> "Unsupported"

2023-03-29 Thread Colin Ian King
There is a spelling mistake in a dev_error message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/msm/msm_mdss.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/msm_mdss.c b/drivers/gpu/drm/msm/msm_mdss.c index c15d1e2dc718..7cb301854e64

[PATCH][next] drm/msm/dp: Fix spelling mistake "Capabiity" -> "Capability"

2023-03-14 Thread Colin Ian King
There is a spelling mistake in a drm_dbg_dp message. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/msm/dp/dp_link.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/dp/dp_link.c b/drivers/gpu/drm/msm/dp/dp_link.c index 5a4817ac086f

[PATCH][next] habanalabs: Fix spelling mistake "maped" -> "mapped"

2023-03-14 Thread Colin Ian King
There is a spelling mistake in a dev_err message. Fix it. Signed-off-by: Colin Ian King --- drivers/accel/habanalabs/common/memory_mgr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/accel/habanalabs/common/memory_mgr.c b/drivers/accel/habanalabs/common

[PATCH][next] drm/sti: Fix spelling mistake "top_cmd_offest" -> "top_cmd_offset"

2023-02-13 Thread Colin Ian King
There is a spelling mistake in a variable name. Fix it. Signed-off-by: Colin Ian King --- drivers/gpu/drm/sti/sti_hqvdp.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/sti/sti_hqvdp.c b/drivers/gpu/drm/sti/sti_hqvdp.c index 02b77279f6e4

[PATCH][next] habanalabs: Fix spelling mistake "offest" -> "offset"

2023-02-13 Thread Colin Ian King
There is a spelling mistake in a dev_err message. Fix it. Signed-off-by: Colin Ian King --- drivers/accel/habanalabs/common/command_submission.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/accel/habanalabs/common/command_submission.c b/drivers/accel/habanalabs

  1   2   3   >