Re: [PATCH] drm/amdgpu: Fix code with incorrect enum type

2022-04-08 Thread Григорий
Alex Deucher I will do it. Initially, I came across an error that different enum types are used. Now I looked at the files, and indeed AMDGPU_GFX_PIPE_PRIO_* is used instead of AMDGPU_RING_PRIO_*. The question remains whether it is worth increasing the priority to AMDGPU_RING_PRIO_MAX=3 ? Regards,

[PATCH] drm/amd/display: Fix indenting mistakes in dcn10_hw_sequencer.c

2022-04-08 Thread Haowen Bai
Smatch reports the following: drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:2174 dcn10_enable_vblanks_synchronization() warn: if statement not indented Signed-off-by: Haowen Bai --- drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c | 14 +++--- 1 file change

Re: [PATCH v2] drm/hyperv: Added error message for fb size greater then allocated

2022-04-08 Thread Saurabh Singh Sengar
On Thu, Apr 07, 2022 at 09:28:53AM -0700, Deepak Rawat wrote: > On Wed, Apr 6, 2022 at 11:27 PM Saurabh Sengar > wrote: > > > > Added error message when the size of requested framebuffer is more then > > the allocated size by vmbus mmio region for framebuffer > > > > Signed-off-by: Saurabh Sengar

[PATCH] drm/amd/display: Fix pointer dereferenced before checking

2022-04-08 Thread Haowen Bai
The pointer dc is dereferencing pointer plane_state before plane_state is being null checked. Fix this by assigning plane_state->ctx->dc to dc only if plane_state is not NULL, otherwise just NULL. Signed-off-by: Haowen Bai --- drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c | 2 +- 1 f

Re: [PATCH 1/1] drm/vc4: hdmi: Replace drm_detect_hdmi_monitor() with is_hdmi

2022-04-08 Thread Maxime Ripard
Hi Jose, On Wed, Apr 06, 2022 at 06:55:14PM +0200, José Expósito wrote: > Once EDID is parsed, the monitor HDMI support information is cached in > drm_display_info.is_hdmi by drm_parse_hdmi_vsdb_video(). > > This driver calls drm_detect_hdmi_monitor() to receive the same > information and stores

Re: [PATCH 4/5] drm/solomon: Move device info from ssd130x-i2c to the core driver

2022-04-08 Thread Neil Armstrong
Hi, On 07/04/2022 22:02, Javier Martinez Canillas wrote: These are declared in the ssd130x-i2c transport driver but the information is not I2C specific and could be used by other SSD130x transport drivers. Move them to the ssd130x core driver and just set the OF device entries to an ID that cou

Re: [PATCH v3 00/17] fbcon cleanups

2022-04-08 Thread Daniel Vetter
On Tue, Apr 05, 2022 at 11:03:18PM +0200, Daniel Vetter wrote: > Hi all, > > Finally got around to respin this. Changes: > - Bunch more acks and r-b, still not yet all patches. > - one tiny fix for a bisect issue, end result was all fine > - I dropped the last to patches to make registered_fb priv

Re: [PATCH 1/5] drm/ttm: Add common debugfs code for resource managers

2022-04-08 Thread Daniel Vetter
On Thu, Apr 07, 2022 at 02:15:52PM +, Zack Rusin wrote: > On Thu, 2022-04-07 at 08:01 +0200, Christian König wrote: > > Am 07.04.22 um 04:56 schrieb Zack Rusin: > > > From: Zack Rusin > > > > > > Drivers duplicate the code required to add debugfs entries for > > > various > > > ttm resource m

Re: [Intel-gfx] [PATCH 0/4] drm/i915/dg2: Add support for render/media decompression

2022-04-08 Thread Jani Nikula
On Mon, 04 Apr 2022, Imre Deak wrote: > This is a rebased version of patches 15-17 of [1], adding DG2 display > engine support for decompressing render and media compressed > framebuffers. > > The dependency patches from [1] should be merged already to drm-tip. > > It addresses the review comments

[PATCH] drm/amdgpu: Fix incorrect enum type

2022-04-08 Thread Grigory Vasilyev
Instead of the 'amdgpu_ring_priority_level' type, the 'amdgpu_gfx_pipe_priority' type was used, which is an error when setting ring priority. This is a minor error, but may cause problems in the future. Instead of AMDGPU_RING_PRIO_2 = 2, we can use AMDGPU_RING_PRIO_MAX = 3, but AMDGPU_RING_PRIO_2

Re: [PATCH 1/1] drm/i915: Inherit submitter nice when scheduling requests

2022-04-08 Thread Daniel Vetter
On Thu, Apr 07, 2022 at 04:16:27PM +0100, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > Inherit submitter nice at point of request submission to account for long > running processes getting either externally or self re-niced. > > This accounts for the current processing landscape where comput

Re: [Intel-gfx] [PATCH 0/4] drm/i915/dg2: Add support for render/media decompression

2022-04-08 Thread Jani Nikula
On Fri, 08 Apr 2022, Jani Nikula wrote: > On Mon, 04 Apr 2022, Imre Deak wrote: >> This is a rebased version of patches 15-17 of [1], adding DG2 display >> engine support for decompressing render and media compressed >> framebuffers. >> >> The dependency patches from [1] should be merged already

Re: [RFC] drm/kms: control display brightness through drm_connector properties

2022-04-08 Thread Daniel Vetter
On Thu, Apr 07, 2022 at 05:05:52PM -0400, Alex Deucher wrote: > On Thu, Apr 7, 2022 at 1:43 PM Hans de Goede wrote: > > > > Hi Simon, > > > > On 4/7/22 18:51, Simon Ser wrote: > > > Very nice plan! Big +1 for the overall approach. > > > > Thanks. > > > > > On Thursday, April 7th, 2022 at 17:38, Ha

Re: [PATCH v9 00/23] drm/rockchip: RK356x VOP2 support

2022-04-08 Thread Sascha Hauer
On Wed, Apr 06, 2022 at 11:47:22AM +0200, Piotr Oniszczuk wrote: > > > > Wiadomość napisana przez Piotr Oniszczuk w dniu > > 01.04.2022, o godz. 15:05: > > Sascha > > > > Now works perfectly! > > (hd playback with 3.5...5.5% cpu while rendering to drm plane) > > > > Fantastic work of You! >

Re: [Intel-gfx] [PATCH 2/2] drm/i915: fix i915_gem_object_wait_moving_fence

2022-04-08 Thread Matthew Auld
On 08/04/2022 06:00, Lucas De Marchi wrote: On Thu, Apr 07, 2022 at 05:45:32PM +0100, Matthew Auld wrote: All of CI is just failing with the following, which prevents loading of the module:    i915 :03:00.0: [drm] *ERROR* Scratch setup failed Best guess is that this comes from the pin_map(

Re: linux-next: build failure after merge of the drm-misc tree

2022-04-08 Thread Christian König
Am 08.04.22 um 03:10 schrieb Stephen Rothwell: Hi all, After merging the drm-misc tree, today's linux-next build (x86_64 allmodconfig) failed like this: In file included from include/drm/drm_gem.h:38, from include/drm/ttm/ttm_bo_api.h:34, from drivers/gpu/drm

Re: [PATCH v2] drm/mediatek: dpi: Use mt8183 output formats for mt8192

2022-04-08 Thread AngeloGioacchino Del Regno
Il 08/04/22 03:39, Nícolas F. R. A. Prado ha scritto: The configuration for mt8192 was incorrectly using the output formats from mt8173. Since the output formats for mt8192 are instead the same ones as for mt8183, which require two bus samples per pixel, the pixelclock and DDR edge setting were m

Re: [PATCH] drm/bridge: anx7625: Use uint8 for lane-swing arrays

2022-04-08 Thread AngeloGioacchino Del Regno
Il 08/04/22 03:30, Nícolas F. R. A. Prado ha scritto: As defined in the anx7625 dt-binding, the analogix,lane0-swing and analogix,lane1-swing properties are uint8 arrays. Yet, the driver was reading the array as if it were of uint32 and masking to 8-bit before writing to the registers. This means

Re: [RFC] drm/kms: control display brightness through drm_connector properties

2022-04-08 Thread Simon Ser
Hi Hans, Thanks for your details replies! On Thursday, April 7th, 2022 at 19:43, Hans de Goede wrote: > > On Thursday, April 7th, 2022 at 17:38, Hans de Goede > > wrote: > > > >> The drm_connector brightness properties > >> === > >> > >> bl_brightness: rw

Re: [PATCH 4/5] drm/solomon: Move device info from ssd130x-i2c to the core driver

2022-04-08 Thread Javier Martinez Canillas
Hello Neil, Thanks for your feedback. On 4/8/22 09:49, Neil Armstrong wrote: [snip] >> >> +static struct ssd130x_deviceinfo ssd130x_variants[] = { >> +{ >> +.default_vcomh = 0x40, >> +.default_dclk_div = 1, >> +.default_dclk_frq = 5, >> +.

Re: [PATCH 1/1] drm/i915: Inherit submitter nice when scheduling requests

2022-04-08 Thread Tvrtko Ursulin
On 08/04/2022 08:58, Daniel Vetter wrote: On Thu, Apr 07, 2022 at 04:16:27PM +0100, Tvrtko Ursulin wrote: From: Tvrtko Ursulin Inherit submitter nice at point of request submission to account for long running processes getting either externally or self re-niced. This accounts for the curren

Re: [PATCH 1/2] drm/i915: fix broken build

2022-04-08 Thread Matthew Auld
On 07/04/2022 17:49, Christian König wrote: Am 07.04.22 um 18:45 schrieb Matthew Auld: I guess this was missed in the conversion or something. Fixes: 7bc80a5462c3 ("dma-buf: add enum dma_resv_usage v4") Signed-off-by: Matthew Auld Cc: Christian König Cc: Daniel Vetter My best guess is that

Re: [PATCH 1/5] drm/ttm: Add common debugfs code for resource managers

2022-04-08 Thread Christian König
Am 08.04.22 um 09:56 schrieb Daniel Vetter: On Thu, Apr 07, 2022 at 02:15:52PM +, Zack Rusin wrote: On Thu, 2022-04-07 at 08:01 +0200, Christian König wrote: Am 07.04.22 um 04:56 schrieb Zack Rusin: From: Zack Rusin Drivers duplicate the code required to add debugfs entries for various t

[PATCH v2] drm/i915: fix i915_gem_object_wait_moving_fence

2022-04-08 Thread Matthew Auld
All of CI is just failing with the following, which prevents loading of the module: i915 :03:00.0: [drm] *ERROR* Scratch setup failed Best guess is that this comes from the pin_map() for the scratch page, which does an i915_gem_object_wait_moving_fence() somewhere. It looks like this now

Re: [PATCH 1/2] drm/i915: fix broken build

2022-04-08 Thread Christian König
Am 08.04.22 um 10:32 schrieb Matthew Auld: On 07/04/2022 17:49, Christian König wrote: Am 07.04.22 um 18:45 schrieb Matthew Auld: I guess this was missed in the conversion or something. Fixes: 7bc80a5462c3 ("dma-buf: add enum dma_resv_usage v4") Signed-off-by: Matthew Auld Cc: Christian Kö

Re: [PATCH v2] drm/i915: fix i915_gem_object_wait_moving_fence

2022-04-08 Thread Christian König
Am 08.04.22 um 10:42 schrieb Matthew Auld: All of CI is just failing with the following, which prevents loading of the module: i915 :03:00.0: [drm] *ERROR* Scratch setup failed Best guess is that this comes from the pin_map() for the scratch page, which does an i915_gem_object_wait_mov

Re: [Intel-gfx] [PATCH v9 4/9] drm/i915/gt: Pass the -EINVAL when emit_pte doesn't update any PTE

2022-04-08 Thread Intel
On 4/5/22 17:08, Ramalingam C wrote: When emit_pte doesn't update any PTE with return value as 0, interpret it as -EINVAL. v2: Add missing goto [Thomas] Signed-off-by: Ramalingam C --- drivers/gpu/drm/i915/gt/intel_migrate.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) d

Re: [PATCH 12/15] drm/i915: drop bo->moving dependency

2022-04-08 Thread Jani Nikula
On Thu, 07 Apr 2022, "Christian König" wrote: > That should now be handled by the common dma_resv framework. > > Signed-off-by: Christian König > Reviewed-by: Daniel Vetter > Cc: intel-...@lists.freedesktop.org So, where are the i915 maintainer acks for merging this (and the other patches in th

Re: [Intel-gfx] [PATCH 2/2] drm/i915: fix i915_gem_object_wait_moving_fence

2022-04-08 Thread Tvrtko Ursulin
On 07/04/2022 17:45, Matthew Auld wrote: All of CI is just failing with the following, which prevents loading of the module: i915 :03:00.0: [drm] *ERROR* Scratch setup failed Best guess is that this comes from the pin_map() for the scratch page, which does an i915_gem_object_wait_mov

Re: [PATCH 1/2] drm/dp: Export drm_dp_dpcd_access()

2022-04-08 Thread Imre Deak
On Thu, Apr 07, 2022 at 10:32:11PM +0300, Jani Nikula wrote: > On Thu, 07 Apr 2022, Imre Deak wrote: > > The next patch needs a way to read a DPCD register without the preceding > > wake-up read in drm_dp_dpcd_read(). Export drm_dp_dpcd_access() to allow > > this. > > I think I'd rather you added

Re: [Intel-gfx] [PATCH 2/2] drm/i915: fix i915_gem_object_wait_moving_fence

2022-04-08 Thread Christian König
Am 08.04.22 um 11:08 schrieb Tvrtko Ursulin: On 07/04/2022 17:45, Matthew Auld wrote: All of CI is just failing with the following, which prevents loading of the module: i915 :03:00.0: [drm] *ERROR* Scratch setup failed Best guess is that this comes from the pin_map() for the scratch

Re: [Intel-gfx] [PATCH 2/2] drm/i915: fix i915_gem_object_wait_moving_fence

2022-04-08 Thread Tvrtko Ursulin
On 08/04/2022 10:12, Christian König wrote: Am 08.04.22 um 11:08 schrieb Tvrtko Ursulin: On 07/04/2022 17:45, Matthew Auld wrote: All of CI is just failing with the following, which prevents loading of the module: i915 :03:00.0: [drm] *ERROR* Scratch setup failed Best guess is tha

Re: [Intel-gfx] [PATCH v2] drm/i915: fix i915_gem_object_wait_moving_fence

2022-04-08 Thread Petri Latvala
On Fri, Apr 08, 2022 at 09:42:05AM +0100, Matthew Auld wrote: > All of CI is just failing with the following, which prevents loading of > the module: > > i915 :03:00.0: [drm] *ERROR* Scratch setup failed > > Best guess is that this comes from the pin_map() for the scratch page, > which do

Re: [PATCH 12/15] drm/i915: drop bo->moving dependency

2022-04-08 Thread Christian König
Am 08.04.22 um 11:05 schrieb Jani Nikula: On Thu, 07 Apr 2022, "Christian König" wrote: That should now be handled by the common dma_resv framework. Signed-off-by: Christian König Reviewed-by: Daniel Vetter Cc: intel-...@lists.freedesktop.org So, where are the i915 maintainer acks for mergi

[PATCH] drm/amdgpu: Fix NULL pointer dereference

2022-04-08 Thread Grigory Vasilyev
The code below check for NULL, but is no check at this place, which is potentially dangerous. Signed-off-by: Grigory Vasilyev --- drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c b/d

Re: [Intel-gfx] [PATCH 2/2] drm/i915: fix i915_gem_object_wait_moving_fence

2022-04-08 Thread Christian König
Am 08.04.22 um 11:23 schrieb Tvrtko Ursulin: On 08/04/2022 10:12, Christian König wrote: Am 08.04.22 um 11:08 schrieb Tvrtko Ursulin: On 07/04/2022 17:45, Matthew Auld wrote: All of CI is just failing with the following, which prevents loading of the module: i915 :03:00.0: [drm] *

Re: [PATCH 12/15] drm/i915: drop bo->moving dependency

2022-04-08 Thread Daniel Vetter
On Fri, 8 Apr 2022 at 11:27, Christian König wrote: > > Am 08.04.22 um 11:05 schrieb Jani Nikula: > > On Thu, 07 Apr 2022, "Christian König" > > wrote: > >> That should now be handled by the common dma_resv framework. > >> > >> Signed-off-by: Christian König > >> Reviewed-by: Daniel Vetter > >

Re: [PATCH v18 3/3] drm/ingenic: Add dw-hdmi driver specialization for jz4780

2022-04-08 Thread Paul Cercueil
Hi, Le jeu., avril 7 2022 at 13:16:11 +0200, H. Nikolaus Schaller a écrit : From: Paul Boddie A specialisation of the generic Synopsys HDMI driver is employed for JZ4780 HDMI support. This requires a new driver, plus device tree and configuration modifications. Here we add Kconfig DRM_INGEN

Re: [PATCH 12/15] drm/i915: drop bo->moving dependency

2022-04-08 Thread Christian König
Am 08.04.22 um 11:33 schrieb Daniel Vetter: On Fri, 8 Apr 2022 at 11:27, Christian König wrote: Am 08.04.22 um 11:05 schrieb Jani Nikula: On Thu, 07 Apr 2022, "Christian König" wrote: That should now be handled by the common dma_resv framework. Signed-off-by: Christian König Reviewed-by: D

Re: [Intel-gfx] [PATCH 1/1] drm/i915/uc: Use platform specific defaults for GuC/HuC enabling

2022-04-08 Thread Tvrtko Ursulin
On 07/04/2022 21:49, John Harrison wrote: On 4/7/2022 08:49, Tvrtko Ursulin wrote: On 03/06/2021 17:48, Matthew Brost wrote: From: John Harrison The meaning of 'default' for the enable_guc module parameter has been updated to accurately reflect what is supported on current platforms. So sta

Re: [PATCH v2] drm/i915: fix i915_gem_object_wait_moving_fence

2022-04-08 Thread Matthew Auld
On 08/04/2022 09:59, Christian König wrote: Am 08.04.22 um 10:42 schrieb Matthew Auld: All of CI is just failing with the following, which prevents loading of the module: i915 :03:00.0: [drm] *ERROR* Scratch setup failed Best guess is that this comes from the pin_map() for the scratch

Re: [PATCH 1/1] drm/i915: Inherit submitter nice when scheduling requests

2022-04-08 Thread Dave Airlie
On Fri, 8 Apr 2022 at 18:25, Tvrtko Ursulin wrote: > > > On 08/04/2022 08:58, Daniel Vetter wrote: > > On Thu, Apr 07, 2022 at 04:16:27PM +0100, Tvrtko Ursulin wrote: > >> From: Tvrtko Ursulin > >> > >> Inherit submitter nice at point of request submission to account for long > >> running process

[PATCH] drivers: Fix spelling mistake "writting" -> "writing"

2022-04-08 Thread cgel . zte
From: Lv Ruyi There are some spelling mistakes in the comments. Fix it. Reported-by: Zeal Robot Signed-off-by: Lv Ruyi --- drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 2 +- drivers/gpu/drm/i915/i915_request.c | 2 +- drivers/net/ethernet/sfc/mcdi_pcol.h

Re: [RFC] drm/kms: control display brightness through drm_connector properties

2022-04-08 Thread Hans de Goede
Hi Daniel, On 4/8/22 10:07, Daniel Vetter wrote: > On Thu, Apr 07, 2022 at 05:05:52PM -0400, Alex Deucher wrote: >> On Thu, Apr 7, 2022 at 1:43 PM Hans de Goede wrote: >>> >>> Hi Simon, >>> >>> On 4/7/22 18:51, Simon Ser wrote: Very nice plan! Big +1 for the overall approach. >>> >>> Thanks.

Re: [PATCH] drm/amdgpu: Fix NULL pointer dereference

2022-04-08 Thread Simon Ser
Is amdgpu_display_get_fb_info ever called with NULL tiling_flags/tmz_surface? If not, there's no point in adding NULL checks.

[PATCH] drm/bridge: anx7625: remove unnecessary flush_workqueue()

2022-04-08 Thread cgel . zte
From: Lv Ruyi All work currently pending will be done by calling destroy_workqueue, so there is unnecessary to flush it explicitly. Reported-by: Zeal Robot Signed-off-by: Lv Ruyi --- drivers/gpu/drm/bridge/analogix/anx7625.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/b

Re: [RFC] drm/kms: control display brightness through drm_connector properties

2022-04-08 Thread Hans de Goede
Hi, On 4/8/22 11:58, Hans de Goede wrote: > Hi Daniel, > > On 4/8/22 10:07, Daniel Vetter wrote: >> On Thu, Apr 07, 2022 at 05:05:52PM -0400, Alex Deucher wrote: >>> On Thu, Apr 7, 2022 at 1:43 PM Hans de Goede wrote: Hi Simon, On 4/7/22 18:51, Simon Ser wrote: > Very nic

Re: [PATCH 12/15] drm/i915: drop bo->moving dependency

2022-04-08 Thread Jani Nikula
On Fri, 08 Apr 2022, Christian König wrote: > Am 08.04.22 um 11:05 schrieb Jani Nikula: >> On Thu, 07 Apr 2022, "Christian König" >> wrote: >>> That should now be handled by the common dma_resv framework. >>> >>> Signed-off-by: Christian König >>> Reviewed-by: Daniel Vetter >>> Cc: intel-...@l

Re: [PATCH] drm/bridge: anx7625: Use irq flags from devicetree

2022-04-08 Thread AngeloGioacchino Del Regno
Il 08/04/22 03:33, Nícolas F. R. A. Prado ha scritto: Read the irq flags, like which edge to trigger on, from the devicetree and use those when registering the irq instead of hardcoding them. In case none was specified, fallback to falling edge trigger. Signed-off-by: Nícolas F. R. A. Prado --

Re: [RFC] drm/kms: control display brightness through drm_connector properties

2022-04-08 Thread Simon Ser
Would it be an option to only support the KMS prop for Good devices, and continue using the suboptimal existing sysfs API for Bad devices? (I'm just throwing ideas around to see what sticks, feel free to ignore.)

Re: [PATCH] drivers: Fix spelling mistake "writting" -> "writing"

2022-04-08 Thread Jani Nikula
On Fri, 08 Apr 2022, cgel@gmail.com wrote: > From: Lv Ruyi > > There are some spelling mistakes in the comments. Fix it. Please prefer splitting by driver. This isn't even split by subsystem. I presume there are very few maintainers willing to pick this up as it is. BR, Jani. > > Reported-b

Re: [RFC] drm/kms: control display brightness through drm_connector properties

2022-04-08 Thread Hans de Goede
Hi, On 4/8/22 11:58, Hans de Goede wrote: > Hi Daniel, > > On 4/8/22 10:07, Daniel Vetter wrote: >> On Thu, Apr 07, 2022 at 05:05:52PM -0400, Alex Deucher wrote: >>> On Thu, Apr 7, 2022 at 1:43 PM Hans de Goede wrote: Hi Simon, On 4/7/22 18:51, Simon Ser wrote: > Very nic

[PATCH] drm/exynos: fix IS_ERR() vs NULL check in probe

2022-04-08 Thread Dan Carpenter
The of_drm_find_bridge() does not return error pointers, it returns NULL on error. Fixes: dd8b6803bc49 ("exynos: drm: dsi: Attach in_bridge in MIC driver") Signed-off-by: Dan Carpenter --- -EPROBE_DEFER is the correct return, right? drivers/gpu/drm/exynos/exynos_drm_mic.c | 4 ++-- 1 file chang

Re: [RFC] drm/kms: control display brightness through drm_connector properties

2022-04-08 Thread Hans de Goede
Hi, On 4/8/22 12:16, Simon Ser wrote: > Would it be an option to only support the KMS prop for Good devices, > and continue using the suboptimal existing sysfs API for Bad devices? > > (I'm just throwing ideas around to see what sticks, feel free to ignore.) Currently suid-root or pkexec helpers

Re: [PATCH 1/1] drm/i915: Inherit submitter nice when scheduling requests

2022-04-08 Thread Tvrtko Ursulin
On 08/04/2022 10:50, Dave Airlie wrote: On Fri, 8 Apr 2022 at 18:25, Tvrtko Ursulin wrote: On 08/04/2022 08:58, Daniel Vetter wrote: On Thu, Apr 07, 2022 at 04:16:27PM +0100, Tvrtko Ursulin wrote: From: Tvrtko Ursulin Inherit submitter nice at point of request submission to account for

Re: [Intel-gfx] [PATCH] drm/i915/display/debugfs: Add connector debugfs for "output_bpc"

2022-04-08 Thread Modem, Bhanuprakash
On Mon-04-04-2022 09:11 pm, Daniel Vetter wrote: On Mon, Apr 04, 2022 at 01:46:23PM +0300, Jani Nikula wrote: On Mon, 04 Apr 2022, "Modem, Bhanuprakash" wrote: On Fri-01-04-2022 06:10 pm, Jani Nikula wrote: On Tue, 29 Mar 2022, Bhanuprakash Modem wrote: This new debugfs will expose the conn

Re: [PATCH 12/15] drm/i915: drop bo->moving dependency

2022-04-08 Thread Christian König
Am 08.04.22 um 12:15 schrieb Jani Nikula: On Fri, 08 Apr 2022, Christian König wrote: Am 08.04.22 um 11:05 schrieb Jani Nikula: On Thu, 07 Apr 2022, "Christian König" wrote: That should now be handled by the common dma_resv framework. Signed-off-by: Christian König Reviewed-by: Daniel V

Re: [PATCH v2] drm/i915: fix i915_gem_object_wait_moving_fence

2022-04-08 Thread Matthew Auld
On 08/04/2022 10:48, Matthew Auld wrote: On 08/04/2022 09:59, Christian König wrote: Am 08.04.22 um 10:42 schrieb Matthew Auld: All of CI is just failing with the following, which prevents loading of the module: i915 :03:00.0: [drm] *ERROR* Scratch setup failed Best guess is that thi

Re: [PATCH 5/5] drm/solomon: Add SSD130x OLED displays SPI support

2022-04-08 Thread Mark Brown
On Thu, Apr 07, 2022 at 10:02:04PM +0200, Javier Martinez Canillas wrote: > The ssd130x driver only provides the core support for these devices but it > does not have any bus transport logic. Add a driver to interface over SPI. > > There is a difference in the communication protocol when using 4-w

[PATCH v10 19/24] arm64: dts: rockchip: enable vop2 and hdmi tx on quartz64a

2022-04-08 Thread Sascha Hauer
From: Michael Riesch Enable the RK356x Video Output Processor (VOP) 2 on the Pine64 Quartz64 Model A. Signed-off-by: Michael Riesch Signed-off-by: Sascha Hauer --- Notes: Changes since v5: - Drop reg property from single endpoint node Changes since v4: - Sort nodes alphab

[PATCH v10 21/24] drm/rockchip: Make VOP driver optional

2022-04-08 Thread Sascha Hauer
With upcoming VOP2 support VOP won't be the only choice anymore, so make the VOP driver optional. This also adds a dependency from ROCKCHIP_ANALOGIX_DP to ROCKCHIP_VOP, because that driver currently only links and works with the VOP driver. Signed-off-by: Sascha Hauer --- Notes: Changes sin

[PATCH v10 01/24] clk: rk3568: Mark hclk_vo as critical

2022-04-08 Thread Sascha Hauer
Whenever pclk_vo is enabled hclk_vo must be enabled as well. This is described in the Reference Manual as: | 2.8.6 NIU Clock gating reliance | | A part of niu clocks have a dependence on another niu clock in order to | sharing the internal bus. When these clocks are in use, another niu | clock mus

[PATCH v10 18/24] arm64: dts: rockchip: rk3568-evb: Enable VOP2 and hdmi

2022-04-08 Thread Sascha Hauer
This enabled the VOP2 display controller along with hdmi and the required port routes which is enough to get a picture out of the hdmi port of the board. Signed-off-by: Sascha Hauer --- Notes: Changes since v5: - Drop reg property from single endpoint node Changes since v4:

[PATCH v10 13/24] drm/rockchip: dw_hdmi: Set cur_ctr to 0 always

2022-04-08 Thread Sascha Hauer
From: Douglas Anderson Jitter was improved by lowering the MPLL bandwidth to account for high frequency noise in the rk3288 PLL. In each case MPLL bandwidth was lowered only enough to get us a comfortable margin. We believe that lowering the bandwidth like this is safe given sufficient testing.

[PATCH v10 12/24] drm/rockchip: dw_hdmi: drop mode_valid hook

2022-04-08 Thread Sascha Hauer
The driver checks if the pixel clock of the given mode matches an entry in the mpll config table. The frequencies in the mpll table are meant as a frequency range up to which the entry works, not as a frequency that must match the pixel clock. The downstream Kernel also does not have this check, so

[PATCH v10 09/24] drm/rockchip: dw_hdmi: add regulator support

2022-04-08 Thread Sascha Hauer
The RK3568 has HDMI_TX_AVDD0V9 and HDMI_TX_AVDD_1V8 supply inputs needed for the HDMI port. add support for these to the driver for boards which have them supplied by switchable regulators. Signed-off-by: Sascha Hauer Reviewed-by: Dmitry Osipenko --- drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c

[PATCH v10 10/24] dt-bindings: display: rockchip: dw-hdmi: Add regulator support

2022-04-08 Thread Sascha Hauer
The RK3568 has HDMI_TX_AVDD0V9 and HDMI_TX_AVDD_1V8 supply inputs needed for the HDMI port. Add the binding for these supplies. Signed-off-by: Sascha Hauer Acked-by: Rob Herring --- Notes: Changes since v4: - Add Robs Ack .../bindings/display/rockchip/rockchip,dw-hdmi.yaml | 11

[PATCH v10 04/24] drm/rockchip: dw_hdmi: rename vpll clock to reference clock

2022-04-08 Thread Sascha Hauer
"vpll" is a misnomer. A clock input to a device should be named after the usage in the device, not after the clock that drives it. On the rk3568 the same clock is driven by the HPLL. To fix that, this patch renames the vpll clock to ref clock. The clock name "vpll" is left for compatibility to old

[PATCH v10 17/24] arm64: dts: rockchip: rk356x: Add HDMI nodes

2022-04-08 Thread Sascha Hauer
Add support for the HDMI port found on RK3568. Signed-off-by: Sascha Hauer --- Notes: Changes since v7: - Rename hclk to niu Changes since v5: - Drop unnecessary #size-cells/#address-cells from nodes with only single endpoint arch/arm64/boot/dts/rockchip/rk356x.dtsi | 32

[PATCH v10 16/24] arm64: dts: rockchip: rk356x: Add VOP2 nodes

2022-04-08 Thread Sascha Hauer
The VOP2 is the display output controller on the RK3568. Add the node for it to the dtsi file along with the required display-subsystem node and the iommu node. Signed-off-by: Sascha Hauer Acked-by: Rob Herring --- Notes: Changes since v6: - Change RK3568_ prefix to ROCKCHIP_ prefix

[PATCH v10 02/24] drm/rockchip: Embed drm_encoder into rockchip_decoder

2022-04-08 Thread Sascha Hauer
The VOP2 driver needs rockchip specific information for a drm_encoder. This patch creates a struct rockchip_encoder with a struct drm_encoder embedded in it. This is used throughout the rockchip driver instead of struct drm_encoder directly. The information the VOP2 drivers needs is the of_graph

[PATCH v10 24/24] dt-bindings: display: rockchip: dw-hdmi: fix ports description

2022-04-08 Thread Sascha Hauer
Current port description doesn't cover all possible cases. It currently expects one single port with two endpoints. When the HDMI connector is described in the device tree there can be two ports, first one going to the VOP and the second one going to the connector. Also on SoCs which only have a

[PATCH v10 23/24] dt-bindings: display: rockchip: Add binding for VOP2

2022-04-08 Thread Sascha Hauer
The VOP2 is found on newer Rockchip SoCs like the rk3568 or the rk3566. The binding differs slightly from the existing VOP binding, so add a new binding file for it. Signed-off-by: Sascha Hauer Reviewed-by: Rob Herring --- Notes: Changes since v5: - Add Robs Reviewed-by: Change

[PATCH v10 00/24] drm/rockchip: RK356x VOP2 support

2022-04-08 Thread Sascha Hauer
This is v10 finally. The series is rebased to v5.18-rc1 which means that it no longer depends on the clk patches, but there are new dependencies: drm/rockchip: Refactor IOMMU initialisation (https://lists.freedesktop.org/archives/dri-devel/2022-April/349548.html) arm64: dts: rockchip: add basic d

[PATCH v10 07/24] drm/rockchip: dw_hdmi: add rk3568 support

2022-04-08 Thread Sascha Hauer
Add a new dw_hdmi_plat_data struct and new compatible for rk3568. Signed-off-by: Benjamin Gaignard Signed-off-by: Sascha Hauer --- drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c | 31 + 1 file changed, 31 insertions(+) diff --git a/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c

[PATCH v10 08/24] dt-bindings: display: rockchip: dw-hdmi: Add compatible for rk3568 HDMI

2022-04-08 Thread Sascha Hauer
From: Benjamin Gaignard Define a new compatible for rk3568 HDMI. This version of HDMI hardware block needs two new clocks hclk_vio and hclk to provide phy reference clocks. Signed-off-by: Benjamin Gaignard Reviewed-by: Rob Herring Signed-off-by: Sascha Hauer --- .../devicetree/bindings/displ

[PATCH v10 03/24] drm/rockchip: Add crtc_endpoint_id to rockchip_encoder

2022-04-08 Thread Sascha Hauer
The VOP2 has an interface mux which decides to which encoder(s) a CRTC is routed to. The encoders and CRTCs are connected via of_graphs in the device tree. When given an encoder the VOP2 driver needs to know to which internal register setting this encoder matches. For this the VOP2 binding offers d

[PATCH v10 20/24] arm64: dts: rockchip: enable vop2 and hdmi tx on rock-3a

2022-04-08 Thread Sascha Hauer
From: Michael Riesch Enable the RK356x Video Output Processor (VOP) 2 on the Radxa ROCK3 Model A. Signed-off-by: Michael Riesch Reported-by: kernel test robot Link: https://lore.kernel.org/r/20220310210352.451136-4-michael.rie...@wolfvision.net Signed-off-by: Sascha Hauer --- Notes: Cha

[PATCH v10 05/24] dt-bindings: display: rockchip: dw-hdmi: use "ref" as clock name

2022-04-08 Thread Sascha Hauer
"vpll" is a misnomer. A clock input to a device should be named after the usage in the device, not after the clock that drives it. On the rk3568 the same clock is driven by the HPLL. This patch adds "ref" as a new alternative clock name for "vpll" Signed-off-by: Sascha Hauer Acked-by: Rob Herring

[PATCH v10 15/24] dt-bindings: display: rockchip: dw-hdmi: Make unwedge pinctrl optional

2022-04-08 Thread Sascha Hauer
None of the upstream device tree files has a "unwedge" pinctrl specified. Make it optional. Signed-off-by: Sascha Hauer Acked-by: Rob Herring --- Notes: Changes since v4: - Add Robs Ack .../devicetree/bindings/display/rockchip/rockchip,dw-hdmi.yaml | 1 + 1 file changed, 1 insertion

[PATCH v10 06/24] arm64: dts: rockchip: rk3399: rename HDMI ref clock to 'ref'

2022-04-08 Thread Sascha Hauer
The reference clock for the HDMI controller has been renamed to 'ref', the previous 'vpll' name is only left for compatibility in the driver. Rename the clock to the new name. Signed-off-by: Sascha Hauer --- arch/arm64/boot/dts/rockchip/rk3399.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 delet

[PATCH v10 22/24] drm: rockchip: Add VOP2 driver

2022-04-08 Thread Sascha Hauer
From: Andy Yan The VOP2 unit is found on Rockchip SoCs beginning with rk3566/rk3568. It replaces the VOP unit found in the older Rockchip SoCs. This driver has been derived from the downstream Rockchip Kernel and heavily modified: - All nonstandard DRM properties have been removed - dropped str

[PATCH v10 14/24] drm/rockchip: dw_hdmi: add default 594Mhz clk for 4K@60hz

2022-04-08 Thread Sascha Hauer
From: Nickey Yang add 594Mhz configuration parameters in rockchip_phy_config Signed-off-by: Nickey Yang Signed-off-by: Sascha Hauer --- Notes: Changes since v3: - new patch drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/dr

[PATCH v10 11/24] drm/rockchip: dw_hdmi: Use auto-generated tables

2022-04-08 Thread Sascha Hauer
From: Douglas Anderson The previous tables for mpll_cfg and curr_ctrl were created using the 20-pages of example settings provided by the PHY vendor. Those example settings weren't particularly dense, so there were places where we were guessing what the settings would be for 10-bit and 12-bit (n

Re: [PATCH] drm/amdgpu: Fix NULL pointer dereference

2022-04-08 Thread Bas Nieuwenhuizen
On Fri, Apr 8, 2022 at 12:01 PM Simon Ser wrote: > > Is amdgpu_display_get_fb_info ever called with NULL tiling_flags/tmz_surface? > If not, there's no point in adding NULL checks. It isn't called with NULL anywhere, the NULL checks that already exist seem redundant.

Re: [PATCH 5/6] drm/vc4: kms: Warn if we have an incompatible muxing setup

2022-04-08 Thread Maxime Ripard
Hi Thomas, On Wed, Apr 06, 2022 at 11:07:53AM +0200, Thomas Zimmermann wrote: > Am 28.03.22 um 17:36 schrieb Maxime Ripard: > > The documentation explicitly states we must prevent the output > > 2 and 3 from feeding from the same HVS channel. > > > > Let's add a warning to make some noise if we e

Re: [PATCH 0/6] drm/vc4: Fixes for the writeback

2022-04-08 Thread Maxime Ripard
On Wed, Apr 06, 2022 at 11:10:12AM +0200, Thomas Zimmermann wrote: > Hi > > Am 28.03.22 um 17:36 schrieb Maxime Ripard: > > Hi, > > > > This series address multiple issues with the transposer support, and thus > > the > > writeback support. > > With my comments considered, feel free to add > >

Re: [PATCH] drm/amdgpu: Fix NULL pointer dereference

2022-04-08 Thread Simon Ser
On Friday, April 8th, 2022 at 13:28, Bas Nieuwenhuizen wrote: > On Fri, Apr 8, 2022 at 12:01 PM Simon Ser cont...@emersion.fr wrote: > > > Is amdgpu_display_get_fb_info ever called with NULL > > tiling_flags/tmz_surface? > > If not, there's no point in adding NULL checks. > > It isn't called wi

Re: [PATCH] drm/msm/dp: add fail safe mode outside of event_mutex context

2022-04-08 Thread Dmitry Baryshkov
On Thu, 7 Apr 2022 at 19:51, Kuogee Hsieh wrote: > > There is possible circular locking dependency detected on event_mutex. > To break this possible circular locking, this patch move setting fail > safe mode out of event_mutex scope. Please provide the lockdep trace here, it might help other peop

Re: [PATCH v9 00/23] drm/rockchip: RK356x VOP2 support

2022-04-08 Thread Sascha Hauer
On Fri, Apr 08, 2022 at 10:07:48AM +0200, Sascha Hauer wrote: > On Wed, Apr 06, 2022 at 11:47:22AM +0200, Piotr Oniszczuk wrote: > > > > > > > Wiadomość napisana przez Piotr Oniszczuk w > > > dniu 01.04.2022, o godz. 15:05: > > > Sascha > > > > > > Now works perfectly! > > > (hd playback with

Re: [PATCH v6 8/8] drm/msm/dp: Handle eDP mode_valid differently from dp

2022-04-08 Thread Dmitry Baryshkov
On Thu, 7 Apr 2022 at 17:05, Sankeerth Billakanti (QUIC) wrote: > > Hi Dmitry, > > > > > > > On Wed, 30 Mar 2022 at 19:04, Sankeerth Billakanti > > > > > > wrote: > > > > > > > > > > > > > > The panel-edp driver modes needs to be validated differently > > > > > > > from DP because the link capabi

Re: [PATCH v6 1/8] drm/msm/dp: Add eDP support via aux_bus

2022-04-08 Thread Dmitry Baryshkov
On Fri, 8 Apr 2022 at 03:28, Doug Anderson wrote: > > Hi, > > On Thu, Apr 7, 2022 at 4:36 PM Dmitry Baryshkov > wrote: > > > > The ps8640 driver looks 'working by coincidence'. It calls > > dp_aux_populate, then immediately after the function returns it checks > > for the panel. If panel-edp is b

Re: [PATCH v6 1/8] drm/msm/dp: Add eDP support via aux_bus

2022-04-08 Thread Dmitry Baryshkov
On Fri, 8 Apr 2022 at 03:26, Doug Anderson wrote: > > Hi, > > On Thu, Apr 7, 2022 at 4:46 PM Dmitry Baryshkov > wrote: > > > > > The way I'm arguing it should work is that: > > > > > > 1. A whole bunch of the DP init code should move to the DP driver's > > > probe function. This includes parsing

Re: [PATCH] drm/msm: Fix range size vs end confusion

2022-04-08 Thread Dmitry Baryshkov
On Thu, 7 Apr 2022 at 23:27, Rob Clark wrote: > > From: Rob Clark > > The fourth param is size, rather than range_end. > > Note that we could increase the address space size if we had a way to > prevent buffers from spanning a 4G split, mostly just to avoid fw bugs > with 64b math. > > Fixes: 84c

Re: [PATCH v2] drm/msm/mdp5: check the return of kzalloc()

2022-04-08 Thread Dmitry Baryshkov
On Thu, 7 Apr 2022 at 05:33, wrote: > > From: Xiaoke Wang > > kzalloc() is a memory allocation function which can return NULL when > some internal memory errors happen. So it is better to check it to > prevent potential wrong memory access. > > Besides, since mdp5_plane_reset() is void type, so w

Re: [PATCH v2] drm/msm/dp: enhance both connect and disconnect pending_timeout handle

2022-04-08 Thread Dmitry Baryshkov
On 07/04/2022 00:28, Kuogee Hsieh wrote: dp_hpd_plug_handle() is responsible for setting up main link and send uevent to notify user space framework to start video stream. Similarly, dp_hdp_unplug_handle is responsible to send uevent to notify user space framework to stop video stream and then te

Re: [PATCH 3/3] drm/amd/display: Move connector debugfs to drm

2022-04-08 Thread kernel test robot
Hi Bhanuprakash, Thank you for the patch! Yet something to improve: [auto build test ERROR on drm-intel/for-linux-next] [also build test ERROR on drm-tip/drm-tip next-20220408] [cannot apply to drm/drm-next v5.18-rc1] [If your patch is applied to the wrong git tree, kindly drop us a note. And

[PATCH] drm/msm: properly add and remove internal bridges

2022-04-08 Thread Dmitry Baryshkov
Add calls to drm_bridge_add()/drm_bridge_remove() DRM bridges created by the driver. This fixes the following warning. WARNING: CPU: 0 PID: 1 at kernel/locking/mutex.c:579 __mutex_lock+0x840/0x9f4 DEBUG_LOCKS_WARN_ON(lock->magic != lock) Modules linked in: CPU: 0 PID: 1 Comm: swapper/0 Not tainted

Re: [PATCH] drm/amdgpu: Fix NULL pointer dereference

2022-04-08 Thread Grigory Vasilyev
Simon Ser and Bas Nieuwenhuizen, do you understand that you are proposing to make the code less safe in the future? In the future, someone might rewrite the code and we'll get an error. пт, 8 апр. 2022 г. в 14:48, Simon Ser : > > On Friday, April 8th, 2022 at 13:28, Bas Nieuwenhuizen > wrote: >

Re: [PATCH] drm/amdgpu: Fix NULL pointer dereference

2022-04-08 Thread Simon Ser
On Friday, April 8th, 2022 at 15:21, Grigory Vasilyev wrote: > Simon Ser and Bas Nieuwenhuizen, do you understand that you are > proposing to make the code less safe in the future? In the future, > someone might rewrite the code and we'll get an error. I don't think we should blindly add NULL ch

  1   2   3   >