[PATCH v2] drm/bridge: tpd12s015: Fix irq registering in tpd12s015_probe

2020-11-03 Thread YueHaibing
e83f ("drm/bridge: Add driver for the TI TPD12S015 HDMI level shifter") Signed-off-by: YueHaibing --- v2: Add checking for >= 0 and update commit message --- drivers/gpu/drm/bridge/ti-tpd12s015.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/b

[PATCH] drm/amd/display: Make two functions static

2019-05-11 Thread YueHaibing
27; was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/dc/dcn10/dcn10_clk_mgr.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_clk_mgr.c b/drivers/gpu/drm/

[PATCH -next] drm/amd/display: Remove set but not used variable 'pixel_width'

2019-05-27 Thread YueHaibing
t-variable] It is never used and can be remove. Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/dc/dcn10/dcn10_dpp.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_dpp.c b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_dpp.c index 6f

[PATCH -next] drm/amdkfd: Make deallocate_hiq_sdma_mqd static

2019-05-27 Thread YueHaibing
Fix sparse warning: drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_device_queue_manager.c:1846:6: warning: symbol 'deallocate_hiq_sdma_mqd' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c

[PATCH -next] drm/stm: ltdc: remove set but not used variable 'src_h'

2019-01-21 Thread YueHaibing
^ ^ drivers/gpu/drm/stm/ltdc.c:694:6: warning: variable 'src_x' set but not used [-Wunused-but-set-variable] u32 src_x, src_y, src_w, src_h; ^ Signed-off-by: YueHaibing --- drivers/gpu/drm/stm/ltdc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) d

[PATCH -next] drm/xen-front: Drop pointless static qualifier in fb_destroy()

2019-01-26 Thread YueHaibing
There is no need to have the 'struct drm_framebuffer *fb' variable static since new value always be assigned before use it. Signed-off-by: YueHaibing --- drivers/gpu/drm/xen/xen_drm_front_kms.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/g

[PATCH -next] video: fbdev: Fix potential NULL pointer dereference

2019-01-30 Thread YueHaibing
There is a potential NULL pointer dereference in case fb_create_modedb() fails and returns NULL. Signed-off-by: YueHaibing --- drivers/video/fbdev/core/fbmon.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/video/fbdev/core/fbmon.c b/drivers/video/fbdev/core/fbmon.c index dd31289

[PATCH -next] drm/vmwgfx: Remove set but not used variable 'restart'

2019-02-14 Thread YueHaibing
fter commit dc366364c4ef ("drm/vmwgfx: Fix multiple command buffer context use") Signed-off-by: YueHaibing --- drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c b/drivers/gpu/drm/v

[PATCH -next] drm/shmob: Fix return value check in shmob_drm_probe

2018-12-18 Thread YueHaibing
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(). Fixes: 8f1597c8f1a5 ("drm: shmobile: Perform initialization/cleanup at probe/remove time") Signed-off-by:

Re: [PATCH -next] staging: fbtft: use strcmp() instead of strncmp() for subsystem lookup

2018-12-19 Thread YueHaibing
sorry, Pls ignore this. The title need fix. On 2018/12/18 21:09, YueHaibing wrote: > strncmp() stops comparing when either the end of one of the first two > arguments is reached or when 'n' characters have been compared, whichever > comes first.That means that strncmp(s1, s

Re: [PATCH v2 -next] staging: fbtft: use strcmp() instead of strncmp() for

2018-12-19 Thread YueHaibing
On 2018/12/18 21:37, Dan Carpenter wrote: > On Tue, Dec 18, 2018 at 09:25:08PM +0800, YueHaibing wrote: >> strncmp() stops comparing when either the end of one of the first two >> arguments is reached or when 'n' characters have been compared, whichever >> comes f

[PATCH v3 -next] staging: fbtft: fix strncmp() size warning

2018-12-19 Thread YueHaibing
voids that the following warning is reported by smatch: drivers/staging/fbtft/fbtft_device.c:1458 fbtft_device_init() error: strncmp() '"list"' too small (5 vs 32) Signed-off-by: YueHaibing --- v2-v3: fix messed patch title --- drivers/staging/fbtft/fbtft_device.c | 2 +- 1 f

[PATCH -next] staging: fbtft: use strcmp() instead of strncmp() for subsystem lookup

2018-12-19 Thread YueHaibing
tch avoids that the following warning is reported by smatch: drivers/staging/fbtft/fbtft_device.c:1458 fbtft_device_init() error: strncmp() '"list"' too small (5 vs 32) Signed-off-by: YueHaibing --- drivers/staging/fbtft/fbtft_device.c | 2 +- 1 file changed, 1 insertion(+

[PATCH v2 -next] staging: fbtft: use strcmp() instead of strncmp() for

2018-12-19 Thread YueHaibing
tch avoids that the following warning is reported by smatch: drivers/staging/fbtft/fbtft_device.c:1458 fbtft_device_init() error: strncmp() '"list"' too small (5 vs 32) Signed-off-by: YueHaibing --- v2: fix patch title --- drivers/staging/fbtft/fbtft_device.c | 2 +- 1 f

[PATCH -next] drm: Fix error handling in drm_legacy_addctx

2019-01-01 Thread YueHaibing
'ctx->handle' is unsigned, it never less than zero. This patch use int 'tmp_handle' to handle the err condition. Fixes: 62968144e673 ("drm: convert drm context code to use Linux idr") Signed-off-by: YueHaibing --- drivers/gpu/drm/drm_context.c | 15 +---

[PATCH -next] drm/vmwgfx: Remove set but not used variable 'srf'

2019-01-16 Thread YueHaibing
ce commit 543831cfc976 ("drm/vmwgfx: Break out surface and context management to separate files") Signed-off-by: YueHaibing --- drivers/gpu/drm/vmwgfx/vmwgfx_surface.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_surface.c b/drivers/gpu/drm/vmwgfx

[PATCH -next] drm: Remove set but not used variable 'gem'

2019-02-18 Thread YueHaibing
struct drm_gem_cma_object *gem; ^ Signed-off-by: YueHaibing --- drivers/gpu/drm/vc4/vc4_txp.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_txp.c b/drivers/gpu/drm/vc4/vc4_txp.c index aa279b5b0de7..afb1c4ec4f18 100644 --- a/drivers/gpu/drm/vc4/vc4_txp.c +++ b/drivers/g

[PATCH -next] drm/bridge: cdns: Remove set but not used variable 'bpp'

2019-02-18 Thread YueHaibing
Fixes gcc '-Wunused-but-set-variable' warning: drivers/gpu/drm/bridge/cdns-dsi.c: In function 'cdns_dsi_bridge_enable': drivers/gpu/drm/bridge/cdns-dsi.c:986:6: warning: variable 'bpp' set but not used [-Wunused-but-set-variable] It never used since introducti

[PATCH -next] drm/qxl: remove set but not used variable 'bo_old'

2019-02-18 Thread YueHaibing
e after 4979904c62b9 ("drm/qxl: use shadow bo directly") Signed-off-by: YueHaibing --- drivers/gpu/drm/qxl/qxl_display.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/gpu/drm/qxl/qxl_display.c b/drivers/gpu/drm/qxl/qxl_display.c index 08c725544a2f..

[PATCH -next] drm/radeon: remove set but not used variable 'vbi_time_out'

2019-02-18 Thread YueHaibing
ince introduction. Signed-off-by: YueHaibing --- drivers/gpu/drm/radeon/si_dpm.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/radeon/si_dpm.c b/drivers/gpu/drm/radeon/si_dpm.c index 0a785ef0ab66..de8bd245497f 100644 --- a/drivers/gpu/drm/radeon/si_dpm.

[PATCH -next] drm/ttm: remove set but not used variable 'bdev'

2019-02-19 Thread YueHaibing
ot used any more and can be removed. Signed-off-by: YueHaibing --- drivers/gpu/drm/ttm/ttm_execbuf_util.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/ttm/ttm_execbuf_util.c b/drivers/gpu/drm/ttm/ttm_execbuf_util.c index 93860346c426..0075eb9a0b52 100644 --- a/drivers/gpu/drm/t

[PATCH -next] drm/amdgpu: remove set but not used variables 'vm, bo'

2019-02-20 Thread YueHaibing
e] drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c:838:20: warning: variable 'vm' set but not used [-Wunused-but-set-variable] They're never used since introduction in a46a2cd103a8 ("drm/amdgpu: Add GPUVM memory management functions for KFD") Signed-off-by: YueHaibing --- drivers/g

[PATCH -next] drm/nouveau/dmem: remove set but not used variable 'drm'

2019-02-21 Thread YueHaibing
^ Signed-off-by: YueHaibing --- drivers/gpu/drm/nouveau/nouveau_dmem.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_dmem.c b/drivers/gpu/drm/nouveau/nouveau_dmem.c index aa9fec80492d..900a302b7ce9 100644 --- a/drivers/gpu/drm/nouveau/no

[PATCH -next] drm/amdgpu: remove set but not used variable 'vbi_time_out'

2019-02-26 Thread YueHaibing
;s never used since introduction in 841686df9f7d ("drm/amdgpu: add SI DPM support (v4)"), so can be removed Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/amdgpu/si_dpm.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/si_dpm.c

Re: [PATCH -next] fbdev: omap2: omapfb: trivial code cleanup

2019-03-03 Thread YueHaibing
On 2019/3/1 22:50, Bartlomiej Zolnierkiewicz wrote: > > Hi, > > On 03/01/2019 02:53 AM, Yue Haibing wrote: >> From: YueHaibing >> >> After commit 60d2fa0dad06 ("fbdev: omap2: no need to check >> return value of debugfs_create functions"

Re: [PATCH] drm/nouveau: Fix DEVICE_PRIVATE dependencies

2019-05-30 Thread Yuehaibing
Hi all, Friendly ping: Who can take this? On 2019/4/17 22:26, Yue Haibing wrote: > From: YueHaibing > > During randconfig builds, I occasionally run into an invalid configuration > > WARNING: unmet direct dependencies detected for DEVICE_PRIVATE > Depends on [n]: ARCH_

[PATCH -next] drm/amd/display: Use kmemdup in dc_copy_stream()

2019-06-02 Thread YueHaibing
Use kmemdup rather than duplicating its implementation. Detected by coccinelle. Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/dc/core/dc_stream.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_stream.c b/drivers/gpu

[PATCH -next] drm/bridge: sii902x: Make sii902x_audio_digital_mute static

2019-06-14 Thread YueHaibing
Fix sparse warning: drivers/gpu/drm/bridge/sii902x.c:665:5: warning: symbol 'sii902x_audio_digital_mute' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/bridge/sii902x.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletio

[PATCH -next] drm/sti: Remove duplicated include from sti_drv.c

2019-06-21 Thread YueHaibing
Remove duplicated include. Signed-off-by: YueHaibing --- drivers/gpu/drm/sti/sti_drv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/sti/sti_drv.c b/drivers/gpu/drm/sti/sti_drv.c index bb6ae6dd66c9..2edd666fb44a 100644 --- a/drivers/gpu/drm/sti/sti_drv.c +++ b/drivers/gpu

[PATCH -next] drm/amdgpu: fix debugfs_simple_attr.cocci warnings

2019-06-27 Thread YueHaibing
: scripts/coccinelle/api/debugfs/debugfs_simple_attr.cocci Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c

[PATCH -next] drm/amdgpu: remove set but not used variable 'psp_enabled'

2019-06-27 Thread YueHaibing
uction in commit c6b6a42175f5 ("drm/amdgpu: add navi10 common ip block (v3)") Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/amdgpu/nv.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/nv.c b/drivers/gpu/drm/amd/amdgpu/nv.c index af20ffb55c54..8b9fa3d

[PATCH -next] drm/i915: Remove set but not used variable 'encoder'

2019-07-05 Thread YueHaibing
used, so can be removed.Also remove related variable 'dig_port' Signed-off-by: YueHaibing --- drivers/gpu/drm/i915/display/intel_dp.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index 8f7188

[PATCH -next] drm/i915: Remove set but not used variable 'intel_dig_port'

2019-07-05 Thread YueHaibing
e] struct intel_digital_port *intel_dig_port; It is never used, so can be removed. Signed-off-by: YueHaibing --- drivers/gpu/drm/i915/display/intel_ddi.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c index a4172595c

[PATCH -next] drm/komeda: remove set but not used variable 'old'

2019-07-09 Thread YueHaibing
used since commit 990dee3aa456 ("drm/komeda: Computing image enhancer internally") Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/arm/display/komeda/komeda_plane.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_pl

[PATCH -next] drm/amdgpu: remove duplicated include from gfx_v9_0.c

2019-07-10 Thread YueHaibing
Remove duplicated include. Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c index 5ba332376710..822f45161240 100644 --- a/drivers/gpu/drm/amd

[PATCH] drm/amdgpu: Fix build without CONFIG_HMM_MIRROR

2019-07-10 Thread YueHaibing
mirror; Fixes: 7590f6d211ec ("drm/amdgpu: Prepare for hmm_range_register API change") Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/amdgpu/amdgpu_mn.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_mn.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_

[PATCH -next] drm/vboxvideo: Remove unused including

2019-03-20 Thread YueHaibing
Remove including that don't need it. Signed-off-by: YueHaibing --- drivers/gpu/drm/vboxvideo/vbox_drv.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/vboxvideo/vbox_drv.h b/drivers/gpu/drm/vboxvideo/vbox_drv.h index 6dea8bf5f045..ece31f395540 100644 --- a/drivers/gp

[PATCH -next] drm/ttm: remove set but not used variable 'vgdev'

2019-03-20 Thread YueHaibing
tio/virtgpu_ttm.c: In function 'virtio_gpu_bo_swap_notify': drivers/gpu/drm/virtio/virtgpu_ttm.c:300:28: warning: variable 'vgdev' set but not used [-Wunused-but-set-variable] It is never used since introduction in dc5698e80cf7 ("Add virtio gpu driver.") Signed-off-b

[PATCH -next] drm/ttm: remove set but not used variable 'rdev'

2019-03-21 Thread YueHaibing
deon/radeon_ttm.c: In function 'radeon_move_ram_vram': drivers/gpu/drm/radeon/radeon_ttm.c:301:24: warning: variable 'rdev' set but not used [-Wunused-but-set-variable] It's not used since a987fcaa805f ("ttm: Make parts of a struct ttm_bo_device global.") Signed-off-by: YueH

[PATCH -next] drm/vmwgfx: Remove set but not used variable 'fb_offset, fb_depth'

2019-03-24 Thread YueHaibing
b.c:645:19: warning: variable 'fb_depth' set but not used [-Wunused-but-set-variable] They're not used any more, so can be removed. Signed-off-by: YueHaibing --- drivers/gpu/drm/vmwgfx/vmwgfx_fb.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm

Re: [PATCH -next] drm/ttm: remove set but not used variable 'vgdev'

2019-03-26 Thread YueHaibing
On 2019/3/25 17:02, Daniel Vetter wrote: > On Wed, Mar 20, 2019 at 02:03:08AM +0000, YueHaibing wrote: >> Fixes gcc '-Wunused-but-set-variable' warning: >> >> drivers/gpu/drm/virtio/virtgpu_ttm.c: In function 'virtio_gpu_init_mem_type': >> drive

[PATCH v2 -next] drm/virtio: remove set but not used variable 'vgdev'

2019-03-26 Thread YueHaibing
tio/virtgpu_ttm.c: In function 'virtio_gpu_bo_swap_notify': drivers/gpu/drm/virtio/virtgpu_ttm.c:300:28: warning: variable 'vgdev' set but not used [-Wunused-but-set-variable] It is never used since introduction in dc5698e80cf7 ("Add virtio gpu driver.") Signed-off-b

[PATCH -next] drm/omap: Add NULL entry to the end of_device_id array

2019-03-27 Thread YueHaibing
Make sure of_device_id tables are NULL terminated Found by coccinelle spatch "misc/of_table.cocci" Signed-off-by: YueHaibing --- drivers/gpu/drm/omapdrm/dss/omapdss-boot-init.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/omapdrm/dss/omapdss-boot-init.c b/drive

[PATCH -next] drm/omap: Use PTR_ERR_OR_ZERO in hdmi4_core_init()

2019-03-29 Thread YueHaibing
Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR Signed-off-by: YueHaibing --- drivers/gpu/drm/omapdrm/dss/hdmi4_core.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/gpu/drm/omapdrm/dss/hdmi4_core.c b/drivers/gpu/drm/omapdrm/dss/hdmi4_core.c index

[PATCH -next] video: fbdev: remove set but not used variable 'size'

2019-03-31 Thread YueHaibing
;pvr2fb: Fix oops when pseudo_palette is written") Signed-off-by: YueHaibing --- drivers/video/fbdev/pvr2fb.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/video/fbdev/pvr2fb.c b/drivers/video/fbdev/pvr2fb.c index 41390c8e0f67..4e4d6a0df978 100644 --- a/drivers/video/fbde

[PATCH -next] video: fbdev: atmel_lcdfb: remove set but not used variable 'pdata'

2019-04-04 Thread YueHaibing
t 42110e91de7f ("video: atmel_lcdfb: introduce atmel_lcdfb_power_control") Signed-off-by: YueHaibing --- drivers/video/fbdev/atmel_lcdfb.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/video/fbdev/atmel_lcdfb.c b/drivers/video/fbdev/atmel_lcdfb.c index b8786ff62b87..

[PATCH -next] fbdev: mxsfb: remove set but not used variable 'line_count'

2019-04-04 Thread YueHaibing
in commit f0a523b5e5e2 ("video: Add i.MX23/28 framebuffer driver") and can be removed. Signed-off-by: YueHaibing --- drivers/video/fbdev/mxsfb.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/video/fbdev/mxsfb.c b/drivers/video/fbdev/mxsfb.c index a017200a16b3..1fdd1eb38fe0 1

Re: [PATCH] drm/msm/a6xx: Fix build with !CONFIG_DEBUG_FS

2019-04-04 Thread YueHaibing
On 2019/4/3 23:36, Jordan Crouse wrote: > On Wed, Apr 03, 2019 at 02:48:11PM +0800, Yue Haibing wrote: >> From: YueHaibing >> >> When building CONFIG_DEBUG_FS is not set >> gcc warn this: >> >> drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c: In function a

[PATCH -next] drm/nouveau/secboot: Make acr_r352_ls_gpccs_func static

2019-07-17 Thread YueHaibing
Fix sparse warning: drivers/gpu/drm/nouveau/nvkm/subdev/secboot/acr_r352.c:1092:1: warning: symbol 'acr_r352_ls_gpccs_func' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/nouveau/nvkm/subdev/secboot/acr_r352.c | 2

[PATCH -next] drm/i915/dsi: remove set but not used variable 'hfront_porch'

2019-07-18 Thread YueHaibing
never used and can be removed. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/i915/display/icl_dsi.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/display/icl_dsi.c b/drivers/gpu/drm/i915/display/icl_dsi.c index 4d952accf

[PATCH -next] drm/i915/icl: Remove set but not used variable 'src_y'

2019-07-18 Thread YueHaibing
-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/i915/display/intel_sprite.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_sprite.c b/drivers/gpu/drm/i915/display/intel_sprite.c index 34586f29be60..9c3367491f04 100644 ---

Re: [PATCH -next] drm/komeda: remove set but not used variable 'old'

2019-07-21 Thread Yuehaibing
On 2019/7/19 21:04, Daniel Vetter wrote: > On Thu, Jul 18, 2019 at 08:51:50PM +0200, Daniel Vetter wrote: >> On Tue, Jul 09, 2019 at 09:58:08PM +0800, YueHaibing wrote: >>> Fixes gcc '-Wunused-but-set-variable' warning: >>> >>> drivers/gpu/drm/arm/d

[PATCH v2 -next] drm/komeda: remove set but not used variable 'old'

2019-07-21 Thread YueHaibing
used since commit 990dee3aa456 ("drm/komeda: Computing image enhancer internally") Reported-by: Hulk Robot Signed-off-by: YueHaibing --- v2: fix compile err --- drivers/gpu/drm/arm/display/komeda/komeda_plane.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers

[PATCH -next] drm/amd/display: Make pow_buffer_ptr static

2019-07-23 Thread YueHaibing
Fix sparse warning: drivers/gpu/drm/amd/amdgpu/../display/modules/color/color_gamma.c:62:5: warning: symbol 'pow_buffer_ptr' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/modules/color/color_gamma.c | 2

[PATCH -next] drm/mediatek: Remove duplicated include from mtk_drm_drv.c

2019-07-23 Thread YueHaibing
Remove duplicated include. Signed-off-by: YueHaibing --- drivers/gpu/drm/mediatek/mtk_drm_drv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/mediatek/mtk_drm_drv.c b/drivers/gpu/drm/mediatek/mtk_drm_drv.c index 2ee809a6f3dc..e3b64a266dcf 100644 --- a/drivers/gpu/drm

[PATCH -next] drm/mga: remove set but not used variable 'buf_priv'

2019-07-23 Thread YueHaibing
eported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/mga/mga_state.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/mga/mga_state.c b/drivers/gpu/drm/mga/mga_state.c index 77a0b006f066..0dec4062e5a2 100644 --- a/drivers/gpu/drm/mga/mga_state.c +++ b/drivers/gpu

[PATCH] drm/amd/display: remove duplicated include from dc_link.c

2019-07-26 Thread YueHaibing
Remove duplicated include. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/dc/core/dc_link.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link.c b/drivers/gpu/drm/amd/display/dc/core/dc_link.c index 193d6f1

[PATCH] drm/bridge: lvds-encoder: Fix build error

2019-07-28 Thread YueHaibing
("Fix lvds-encoder since the panel_bridge rework.") Signed-off-by: YueHaibing --- drivers/gpu/drm/bridge/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig index a6eec90..2926750 100644 --- a/drivers/gp

Re: [PATCH] drm/bridge: lvds-encoder: Fix build error

2019-07-28 Thread Yuehaibing
Pls drop this one, will resend new. On 2019/7/29 14:35, YueHaibing wrote: > If CONFIG_DRM_LVDS_ENCODER=y but CONFIG_DRM_KMS_HELPER=m, > build fails: > > drivers/gpu/drm/bridge/lvds-encoder.o: In function `lvds_encoder_probe': > lvds-encoder.c:(.text+0x155): un

[PATCH] drm/bridge: lvds-encoder: Fix build error while CONFIG_DRM_KMS_HELPER=m

2019-07-28 Thread YueHaibing
7; drivers/gpu/drm/bridge/tc358764.o:(.rodata+0x270): undefined reference to `drm_atomic_helper_connector_destroy_state' Reported-by: Hulk Robot Fixes: dbb58bfd9ae6 drm/bridge: ("Fix lvds-encoder since the panel_bridge rework.") Signed-off-by: YueHaibing --- drivers/gpu/drm/bridge/Kc

[PATCH v2] drm/bridge: lvds-encoder: Fix build error while CONFIG_DRM_KMS_HELPER=m

2019-07-29 Thread YueHaibing
ix lvds-encoder since the panel_bridge rework.") Signed-off-by: YueHaibing --- v2: remove tc358764 log in commit log, also fix Fixes tag --- drivers/gpu/drm/bridge/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig in

[PATCH] drm/amdkfd: remove set but not used variable 'pdd'

2019-07-29 Thread YueHaibing
nce commit 5b87245faf57 ("drm/amdkfd: Simplify kfd2kgd interface") Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/amdkfd/kfd_process.c | 11 --- 1 file changed, 11 deletions(-) diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_process.c b/drivers/gpu/drm/amd/

[PATCH] drm/bridge: tc358764: Fix build error

2019-07-29 Thread YueHaibing
S_HELPER to fix this, and change to select DRM_PANEL to avoid recursive dependency. Reported-by: Hulk Robot Fixes: f38b7cca6d0e ("drm/bridge: tc358764: Add DSI to LVDS bridge driver") Signed-off-by: YueHaibing --- drivers/gpu/drm/bridge/Kconfig | 3 ++- 1 file changed, 2 insertions(+

[PATCH -next] drm/rockchip: Make analogix_dp_atomic_check static

2019-07-31 Thread YueHaibing
Fix sparse warning: drivers/gpu/drm/bridge/analogix/analogix_dp_core.c:1151:5: warning: symbol 'analogix_dp_atomic_check' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 4 ++-- 1 fi

[PATCH 3/3] drm/amd/powerplay: Make two functions static

2019-10-28 Thread YueHaibing
ntrol_fini' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/powerplay/arcturus_ppt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c b/drivers/g

[PATCH -next] drm/amd/display: Make calculate_integer_scaling static

2019-10-28 Thread YueHaibing
Fix sparse warning: drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_resource.c:963:6: warning: symbol 'calculate_integer_scaling' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/dc/core/dc_resource.c | 2

Re: [PATCH 3/3] drm/amd/powerplay: Make two functions static

2019-10-28 Thread Yuehaibing
On 2019/10/28 22:06, Alex Deucher wrote: > On Mon, Oct 28, 2019 at 9:37 AM YueHaibing wrote: >> >> Fix sparse warnings: >> >> drivers/gpu/drm/amd/amdgpu/../powerplay/arcturus_ppt.c:2050:5: >> warning: symbol 'arcturus_i2c_eeprom_control_init'

[PATCH -next] gpu: host1x: Fix compile test failure

2019-10-31 Thread YueHaibing
Fixes: 52499a6ad2ae ("gpu: host1x: select IOMMU_IOVA") Signed-off-by: YueHaibing --- drivers/gpu/host1x/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/host1x/Kconfig b/drivers/gpu/host1x/Kconfig index cf987a3..354232d 100644 --- a/drivers/gpu/host

Re: [PATCH -next] gpu: host1x: Fix compile test failure

2019-11-01 Thread Yuehaibing
On 2019/10/31 6:26, Dmitry Osipenko wrote: > 30.10.2019 16:54, YueHaibing пишет: >> If IOMMU_SUPPORT is not set, but IOMMU_IOVA is m and >> COMPILE_TEST is y, building fails: >> >> drivers/gpu/host1x/dev.o: In function `host1x_remove': >> dev.c

Re: [PATCH -next] gpu: host1x: Fix compile test failure

2019-11-01 Thread Yuehaibing
On 2019/11/1 17:47, Thierry Reding wrote: > On Thu, Oct 31, 2019 at 09:33:38PM +0800, Yuehaibing wrote: >> On 2019/10/31 6:26, Dmitry Osipenko wrote: >>> 30.10.2019 16:54, YueHaibing пишет: >>>> If IOMMU_SUPPORT is not set, but IOMMU_IOVA is m and >>

[PATCH -next] drm/vmwgfx: remove set but not used variable 'srf'

2019-11-01 Thread YueHaibing
drivers/gpu/drm/vmwgfx/vmwgfx_surface.c:339:22: warning: variable srf set but not used [-Wunused-but-set-variable] 'srf' is never used, so can be removed. Signed-off-by: YueHaibing --- drivers/gpu/drm/vmwgfx/vmwgfx_surface.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drive

[PATCH -next] gpu: host1x: Remove dev_err() on platform_get_irq() failure

2019-11-03 Thread YueHaibing
platform_get_irq() will call dev_err() itself on failure, so there is no need for the driver to also do this. This is detected by coccinelle. Signed-off-by: YueHaibing --- drivers/gpu/host1x/dev.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/host1x/dev.c b

[PATCH -next] drm/amd/display: remove set but not used variable 'ds_port'

2019-11-10 Thread YueHaibing
t is never used, so can be removed. Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c b/drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c index 65de32f..b814b74 10

[PATCH -next] drm/amd/display: remove set but not used variable 'bpc'

2019-11-10 Thread YueHaibing
used since commit e49f69363adf ("drm/amd/display: use proper formula to calculate bandwidth from timing") Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/dc/core/dc_link.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link.c b/drivers

[PATCH -next] drm/amd/display: Fix old-style declaration

2019-11-12 Thread YueHaibing
Fix a build warning: drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc.c:75:1: warning: 'static' is not at beginning of declaration [-Wold-style-declaration] Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/dc/core/dc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH v2 -next] drm/amd/display: remove set but not used variable 'bpc'

2019-11-12 Thread YueHaibing
used since commit e49f69363adf ("drm/amd/display: use proper formula to calculate bandwidth from timing"), this also remove get_color_depth(), which is only used here. Signed-off-by: YueHaibing --- v2: also remove unused get_color_depth() --- drivers/gpu/drm/amd/display/dc/core/dc_link.c | 1

Re: [PATCH -next] drm/amd/display: Fix old-style declaration

2019-11-12 Thread Yuehaibing
On 2019/11/12 10:39, Joe Perches wrote: > On Mon, 2019-11-11 at 20:28 +0800, YueHaibing wrote: >> Fix a build warning: >> >> drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc.c:75:1: >> warning: 'static' is not at beginning of declaration >> [-Wold-styl

[PATCH -next] drm/nouveau/nv04: Use match_string() helper to simplify the code

2019-12-31 Thread YueHaibing
match_string() returns the array index of a matching string. Use it instead of the open-coded implementation. Signed-off-by: YueHaibing --- drivers/gpu/drm/nouveau/dispnv04/tvnv17.c | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/nouveau

[PATCH -next] drm/i915: Add missing include file

2020-01-08 Thread YueHaibing
upper_32_bits(mul_u32_u32(prandom_u32_state(state), ep_ro)); Reported-by: Hulk Robot Fixes: 7ce5b6850b47 ("drm/i915/selftests: Use mul_u32_u32() for 32b x 32b -> 64b result") Signed-off-by: YueHaibing --- drivers/gpu/drm/i915/selftests/i915_random.h | 1 + 1 file changed, 1 insertion(+

[PATCH -next] drm/amd/display: remove set but not used variable 'core_freesync'

2019-10-07 Thread YueHaibing
ed-but-set-variable] It is not used since commit 98e6436d3af5 ("drm/amd/display: Refactor FreeSync module") Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/modules/freesync/freesync.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers

[PATCH -next] drm/amdgpu: remove duplicated include from mmhub_v1_0.c

2019-10-07 Thread YueHaibing
Remove duplicated include. Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c b/drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c index 4c7e8c64a94e..6965e1e6fa9e 100644 --- a/drivers/gpu/drm

[PATCH -next] drm/qxl: Fix randbuild error

2019-10-07 Thread YueHaibing
If DEM_QXL is y and DRM_TTM_HELPER is m, building fails: drivers/gpu/drm/qxl/qxl_object.o: undefined reference to `drm_gem_ttm_print_info' Select DRM_TTM_HELPER to fix this. Fixes: 78d54f1f6a33 ("drm/qxl: use drm_gem_ttm_print_info") Signed-off-by: YueHaibing --- drivers/gpu/

[PATCH -next] drm/vkms: Remove duplicated include from vkms_drv.c

2019-10-10 Thread YueHaibing
Remove duplicated include. Signed-off-by: YueHaibing --- drivers/gpu/drm/vkms/vkms_drv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/vkms/vkms_drv.c b/drivers/gpu/drm/vkms/vkms_drv.c index 54703463d966..d1fe144aa289 100644 --- a/drivers/gpu/drm/vkms/vkms_drv.c +++ b

[PATCH -next] drm/amd/display: Make dc_link_detect_helper static

2019-10-18 Thread YueHaibing
Fix sparse warning: drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link.c:746:6: warning: symbol 'dc_link_detect_helper' was not declared. Should it be static? Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/dc/core/dc_link.c | 3 ++- 1 file

[PATCH -next] drm/amdgpu: remove set but not used variable 'adev'

2019-10-23 Thread YueHaibing
: variable adev set but not used [-Wunused-but-set-variable] It is never used, so can removed it. Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c b/drivers/gpu/drm/amd/amdgpu

[RFC PATCH -next] drm/amd/display: Remove set but not unused variable 'stream_status'

2020-02-10 Thread YueHaibing
("drm/amd/display: Split program front end part that occur outside lock") involved this unused variable. Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/gpu/drm/amd/

[PATCH -next] drm/amd/display: remove set but not used variable 'mc_vm_apt_default'

2020-02-21 Thread YueHaibing
. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/dc/dcn21/dcn21_hubp.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/dcn21/dcn21_hubp.c b/drivers/gpu/drm/amd/display/dc/dcn21/dcn21_hubp.c index aa7b0e7..d285ba6 100644

[PATCH -next] drm/tidss: Drop pointless static qualifier in dispc_find_csc()

2020-02-27 Thread YueHaibing
There is no need to have the 'const struct dispc_csc_coef *coef' variable static since new value always be assigned before use it. Signed-off-by: YueHaibing --- drivers/gpu/drm/tidss/tidss_dispc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu

[PATCH -next] drm/ch7006: Use match_string() helper to simplify the code

2020-01-12 Thread YueHaibing
match_string() returns the array index of a matching string. Use it instead of the open-coded implementation. Signed-off-by: YueHaibing --- drivers/gpu/drm/i2c/ch7006_drv.c | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/i2c/ch7006_drv.c b

[PATCH -next] drm/ttm: Remove set but not used variable 'mem'

2020-01-12 Thread YueHaibing
/nouveau_ttm.c:106:22: warning: variable mem set but not used [-Wunused-but-set-variable] They are not used any more, so remove it. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/nouveau/nouveau_ttm.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/gpu/drm/nouveau

[PATCH] drm/nouveau: Fix copy-paste error in nouveau_fence_wait_uevent_handler

2020-01-12 Thread YueHaibing
Like other cases, it should use rcu protected 'chan' rather than 'fence->channel' in nouveau_fence_wait_uevent_handler. Fixes: 0ec5f02f0e2c ("drm/nouveau: prevent stale fence->channel pointers, and protect with rcu") Signed-off-by: YueHaibing --- drivers

[PATCH -next] drm/nv04/disp: remove set but not used variable 'width'

2020-01-17 Thread YueHaibing
drivers/gpu/drm/nouveau/dispnv04/arb.c: In function nv04_calc_arb: drivers/gpu/drm/nouveau/dispnv04/arb.c:56:21: warning: variable width set but not used [-Wunused-but-set-variable] 'width' is never used, so remove it. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drive

[PATCH -next] video: fbdev: pxa168fb: remove unnecessary platform_get_irq

2020-01-17 Thread YueHaibing
commit 640ba2444fa9 ("drivers/video/pxa168fb.c: use devm_ functions") left behind this, it can be removed. Signed-off-by: YueHaibing --- drivers/video/fbdev/pxa168fb.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/video/fbdev/pxa168fb.c b/drivers/video/fbdev/pxa168

[PATCH -next] drm/nouveau/kms/nv50: remove set but not unused variable 'nv_connector'

2020-01-17 Thread YueHaibing
tom") left behind this. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/nouveau/dispnv50/disp.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/nouveau/dispnv50/disp.c b/drivers/gpu/drm/nouveau/dispnv50/disp.c index 5fabe2b..a82b354 100644 --- a/

[PATCH -next] crypto: ccree - remove set but not used variable 'du_size'

2020-02-04 Thread YueHaibing
ccree - fix FDE descriptor sequence") involved this unused variable, so remove it. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/crypto/ccree/cc_cipher.c | 8 1 file changed, 8 deletions(-) diff --git a/drivers/crypto/ccree/cc_cipher.c b/drivers/crypto/ccree/cc_c

[PATCH -next] drm/panfrost: Remove set but not used variable 'bo'

2020-02-04 Thread YueHaibing
2d8dc0 ("drm/panfrost: Add the panfrost_gem_mapping concept") involved this unused variable. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/gpu/drm/panfrost/panfrost_job.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/gpu/drm/panfrost/pa

[PATCH -next] drm/amd/display: remove set but not used variable 'msg_out'

2019-11-26 Thread YueHaibing
/hdcp/hdcp_psp.c: In function mod_hdcp_hdcp2_enable_dp_stream_encryption: drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp_psp.c:710:77: warning: variable msg_out set but not used [-Wunused-but-set-variable] It is never used, so remove it. Reported-by: Hulk Robot Signed-off-by: YueHaibing

[PATCH -next] drm/amd/powerplay: remove set but not used variable 'stretch_amount2'

2019-11-26 Thread YueHaibing
removed. Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/powerplay/smumgr/vegam_smumgr.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/vegam_smumgr.c b/drivers/gpu/drm/amd/powerplay/smumgr/vegam_smumgr.c index 50896e9

[PATCH -next] drm/tegra: sor: Fix unused function warnings

2019-12-06 Thread YueHaibing
_runtime_resume' defined but not used [-Wunused-function] drivers/gpu/drm/tegra/sor.c:3915:12: warning: 'tegra_sor_runtime_suspend' defined but not used [-Wunused-function] Mark these functions as __maybe_unused so gcc can drop them silently. Reported-by: Hulk Robot Signed-off-by

[PATCH -next] drm/amdgpu/display: fix build error without CONFIG_DRM_AMD_DC_DSC_SUPPORT

2019-08-27 Thread YueHaibing
;drm/amd/display: Make init_hw and init_pipes generic for seamless boot") Signed-off-by: YueHaibing --- drivers/gpu/drm/amd/display/dc/dcn20/dcn20_hwseq.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_hwseq.c b/drivers/gpu/drm/amd/dis

  1   2   >