[PATCH v2] drm: gm12u320: Fix the timeout usage for usb_bulk_msg()

2023-09-03 Thread Jinjie Ruan
it consistent here and so change IDLE_TIMEOUT to msecs_to_jiffies(IDLE_TIMEOUT) where it is used. Fixes: e4f86e437164 ("drm: Add Grain Media GM12U320 driver v2") Signed-off-by: Jinjie Ruan Suggested-by: Hans de Goede --- v2: - Remove the msecs_to_jiffies() also for IDLE_TIMEOUT. - Upda

[PATCH] drm: gm12u320: Fix the timeout usage for usb_bulk_msg()

2023-09-03 Thread Jinjie Ruan
The timeout arg of usb_bulk_msg() is ms already and it has convert it to jiffies by msecs_to_jiffies() in usb_start_wait_urb(). So fix the usage by remove the redundant msecs_to_jiffies() in the macros. Fixes: 77b8cabf3d52 ("drm/gm12u320: Move driver to drm/tiny") Signed-off-by: J

[PATCH] drm/mediatek: dp: Remove redundant dev_err_probe() for platform_get_irq()

2023-09-01 Thread Jinjie Ruan
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. Fixes: 828c91231fbe ("drm/mediatek: dp: Don't register HPD interrupt handler for eDP case") Signed-off-by: Jinjie Ruan --- drivers/gpu/drm/med

[PATCH -next] drm/i915/gvt: Use list_for_each_entry() helper

2023-08-28 Thread Jinjie Ruan
Convert list_for_each() to list_for_each_entry() so that the pos list_head pointer and list_entry() call are no longer needed, which can reduce a few lines of code. No functional changed. Signed-off-by: Jinjie Ruan --- drivers/gpu/drm/i915/gvt/dmabuf.c | 8 ++-- 1 file changed, 2 insertions

[PATCH -next] drm: xlnx: zynqmp_dpsub: Use devm_clk_get_enabled() helper function

2023-08-25 Thread Jinjie Ruan
The devm_clk_get_enabled() helper: - calls devm_clk_get() - calls clk_prepare_enable() and registers what is needed in order to call clk_disable_unprepare() when needed, as a managed resource. This simplifies the code. Signed-off-by: Jinjie Ruan --- drivers/gpu/drm/xlnx

[PATCH -next] fbdev/core: Use list_for_each_entry() helper

2023-08-23 Thread Jinjie Ruan
Convert list_for_each() to list_for_each_entry() so that the pos list_head pointer and list_entry() call are no longer needed, which can reduce a few lines of code. No functional changed. Signed-off-by: Jinjie Ruan --- drivers/video/fbdev/core/fbsysfs.c | 8 ++-- drivers/video/fbdev/core

[PATCH -next 2/2] drm/tegra: Use PTR_ERR_OR_ZERO() to simplify code

2023-08-22 Thread Jinjie Ruan
Return PTR_ERR_OR_ZERO() instead of return 0 or PTR_ERR() to simplify code. Signed-off-by: Jinjie Ruan --- drivers/gpu/drm/tegra/drm.c | 5 + drivers/gpu/drm/tegra/gem.c | 5 + 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/tegra/drm.c b/drivers/gpu/drm

[PATCH -next 0/2] drm: Use PTR_ERR_OR_ZERO() to simplify code

2023-08-22 Thread Jinjie Ruan
PTR_ERR_OR_ZERO() return the error code within @ptr if it is an error pointer, otherwise return 0. So use it to simplify code. Jinjie Ruan (2): drm/bridge: Use PTR_ERR_OR_ZERO() to simplify code drm/tegra: Use PTR_ERR_OR_ZERO() to simplify code drivers/gpu/drm/bridge/tc358762.c | 10

[PATCH -next 1/2] drm/bridge: Use PTR_ERR_OR_ZERO() to simplify code

2023-08-22 Thread Jinjie Ruan
Return PTR_ERR_OR_ZERO() instead of return 0 or PTR_ERR() to simplify code. Signed-off-by: Jinjie Ruan --- drivers/gpu/drm/bridge/tc358762.c | 10 ++ drivers/gpu/drm/bridge/tc358764.c | 5 + 2 files changed, 3 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/bridge