[PULL] drm-misc-next-fixes

2021-11-10 Thread Maxime Ripard
Hi Dave, Daniel, Here's this week PR for drm-misc-next-fixes. Maxime drm-misc-next-fixes-2021-11-10: Removed the TTM Huge Page functionnality to address a crash, a timeout fix for udl, CONFIG_FB dependency improvements, a fix for a circular locking depency in imx, a NULL pointer dereference fix

Re: [RFC v2 02/22] drm: Add Enhanced Gamma and color lut range attributes

2021-11-10 Thread Pekka Paalanen
On Wed, 10 Nov 2021 00:02:16 +0200 Ville Syrjälä wrote: > On Tue, Nov 09, 2021 at 03:47:58PM -0500, Harry Wentland wrote: > > On 2021-11-08 04:54, Pekka Paalanen wrote: > > > On Thu, 4 Nov 2021 12:27:56 -0400 > > > Harry Wentland wrote: > > > > > >> On 2021-11-04 04:38, Pekka Paalanen wrote

[PATCH] drm/i915/ttm: Fix illegal addition to shrinker list

2021-11-10 Thread Thomas Hellström
There's a small window of opportunity during which the adjust_lru() function can be called with a GEM refcount of zero from the TTM eviction code. This results in a kernel BUG(). Ensure that we don't attempt to modify the GEM shrinker lists unless we have a GEM refcount. Fixes: ebd4a8ec7799 ("drm

Re: [PATCH] drm/bridge: cdns-dsi: Make sure to to create proper aliases for dt

2021-11-10 Thread Laurent Pinchart
Hi Nishanth, Thank you for the patch, and sorry for the late reply. On Tue, Nov 09, 2021 at 08:55:38AM -0600, Nishanth Menon wrote: > On 12:40-20210921, Nishanth Menon wrote: > > Add MODULE_DEVICE_TABLE to the device tree table to create required > > aliases needed for module to be loaded with de

Re: printk deadlock due to double lock attempt on current CPU's runqueue

2021-11-10 Thread Vincent Guittot
On Tue, 9 Nov 2021 at 22:38, Peter Zijlstra wrote: > > On Tue, Nov 09, 2021 at 12:06:48PM -0800, Sultan Alsawaf wrote: > > Hi, > > > > I encountered a printk deadlock on 5.13 which appears to still affect the > > latest > > kernel. The deadlock occurs due to printk being used while having the >

Re: [PATCH v2 3/9] drm/format-helper: Add destination-buffer pitch to drm_fb_swab()

2021-11-10 Thread Thomas Zimmermann
Am 08.11.21 um 19:57 schrieb Amanoel Dawod: Hi, On Mon, Nov 1, 2021, at 10:15 AM, Thomas Zimmermann wrote: Add destination-buffer pitch as argument to drm_fb_swab(). Done for consistency with the rest of the interface. v2: * update documentation (Noralf) Signed-off-by: Thomas Zimmer

Re: [PATCH] drm/i915: Use per device iommu check

2021-11-10 Thread Tvrtko Ursulin
On 10/11/2021 07:12, Lu Baolu wrote: Hi Tvrtko, On 2021/11/9 20:17, Tvrtko Ursulin wrote: From: Tvrtko Ursulin On igfx + dgfx setups, it appears that intel_iommu=igfx_off option only disables the igfx iommu. Stop relying on global intel_iommu_gfx_mapped and probe presence of iommu domain per

Re: [Intel-gfx] [PATCH] drm/i915: Use per device iommu check

2021-11-10 Thread Tvrtko Ursulin
On 10/11/2021 07:25, Lu Baolu wrote: On 2021/11/10 1:35, Tvrtko Ursulin wrote: On 09/11/2021 17:19, Lucas De Marchi wrote: On Tue, Nov 09, 2021 at 12:17:59PM +, Tvrtko Ursulin wrote: From: Tvrtko Ursulin On igfx + dgfx setups, it appears that intel_iommu=igfx_off option only disables

Re: printk deadlock due to double lock attempt on current CPU's runqueue

2021-11-10 Thread Daniel Vetter
On Tue, Nov 09, 2021 at 10:38:47PM +0100, Peter Zijlstra wrote: > On Tue, Nov 09, 2021 at 12:06:48PM -0800, Sultan Alsawaf wrote: > > Hi, > > > > I encountered a printk deadlock on 5.13 which appears to still affect the > > latest > > kernel. The deadlock occurs due to printk being used while hav

printk deadlock due to double lock attempt on current CPU's runqueue

2021-11-10 Thread Sultan Alsawaf
Hi, I encountered a printk deadlock on 5.13 which appears to still affect the latest kernel. The deadlock occurs due to printk being used while having the current CPU's runqueue locked, and the underlying framebuffer console attempting to lock the same runqueue when printk tries to flush the log b

Re: [Intel-gfx] [PATCH] drm/i915: Use per device iommu check

2021-11-10 Thread Lu Baolu
On 2021/11/10 1:35, Tvrtko Ursulin wrote: On 09/11/2021 17:19, Lucas De Marchi wrote: On Tue, Nov 09, 2021 at 12:17:59PM +, Tvrtko Ursulin wrote: From: Tvrtko Ursulin On igfx + dgfx setups, it appears that intel_iommu=igfx_off option only disables the igfx iommu. Stop relying on global i

Re: [PATCH] drm/i915: Use per device iommu check

2021-11-10 Thread Lu Baolu
Hi Tvrtko, On 2021/11/9 20:17, Tvrtko Ursulin wrote: From: Tvrtko Ursulin On igfx + dgfx setups, it appears that intel_iommu=igfx_off option only disables the igfx iommu. Stop relying on global intel_iommu_gfx_mapped and probe presence of iommu domain per device to accurately reflect its status

Re: [PATCH 2/2] drm/sched: serialize job_timeout and scheduler

2021-11-10 Thread Daniel Vetter
On Tue, Nov 09, 2021 at 08:17:01AM -0800, Rob Clark wrote: > On Tue, Nov 9, 2021 at 1:07 AM Daniel Vetter wrote: > > > > On Mon, Nov 08, 2021 at 03:39:17PM -0800, Rob Clark wrote: > > > I stumbled across this thread when I ran into the same issue, while > > > working out how to move drm/msm to use

Re: [PATCH 2/2] drm/sched: serialize job_timeout and scheduler

2021-11-10 Thread Christian König
Am 10.11.21 um 10:50 schrieb Daniel Vetter: On Tue, Nov 09, 2021 at 08:17:01AM -0800, Rob Clark wrote: On Tue, Nov 9, 2021 at 1:07 AM Daniel Vetter wrote: On Mon, Nov 08, 2021 at 03:39:17PM -0800, Rob Clark wrote: I stumbled across this thread when I ran into the same issue, while working out

Re: printk deadlock due to double lock attempt on current CPU's runqueue

2021-11-10 Thread John Ogness
On 2021-11-10, Daniel Vetter wrote: > I'm a bit out of the loop but from lwn articles my understanding is > that part of upstreaming from -rt we no longer have the explicit "I'm > a safe console for direct printing" opt-in. Which I get from a > backwards compat pov, but I still think for at least

Re: Questions about KMS flip

2021-11-10 Thread Christian König
Am 08.11.21 um 15:59 schrieb Daniel Vetter: On Mon, Nov 08, 2021 at 08:44:24AM +0100, Christian König wrote: Am 05.11.21 um 19:13 schrieb Daniel Vetter: On Thu, Nov 04, 2021 at 12:44:34PM -0400, Harry Wentland wrote: +Nick It looks to be the old drm_plane_state->fb holds that reference. See

Re: [PATCH] drm/omap: increase DSS5 max tv pclk to 192MHz

2021-11-10 Thread Tomi Valkeinen
On 09/11/2021 15:33, Neil Armstrong wrote: Hi Tomi, On 12/10/2021 15:39, Neil Armstrong wrote: From: Tomi Valkeinen DSS5's maximum tv pclk rate (i.e. HDMI) is set to 186MHz, which comes from the TRM (DPLL_HDMI_CLK1 frequency must be lower than 186 MHz). To support DRA76's wide screen HDMI fea

[PATCH v1 1/1] drm: Replace kernel.h with the necessary inclusions

2021-11-10 Thread Andy Shevchenko
When kernel.h is used in the headers it adds a lot into dependency hell, especially when there are circular dependencies are involved. Replace kernel.h inclusion with the list of what is really being used. Signed-off-by: Andy Shevchenko --- include/drm/drm_gem_ttm_helper.h | 2 +- include/drm/

[PATCH v1 1/1] drm/i915: Replace kernel.h with the necessary inclusions

2021-11-10 Thread Andy Shevchenko
When kernel.h is used in the headers it adds a lot into dependency hell, especially when there are circular dependencies are involved. Replace kernel.h inclusion with the list of what is really being used. Signed-off-by: Andy Shevchenko --- include/drm/intel-gtt.h | 5 - 1 file changed, 4 i

Re: [PATCH] drm/i915/ttm: Fix illegal addition to shrinker list

2021-11-10 Thread Matthew Auld
On 10/11/2021 08:55, Thomas Hellström wrote: There's a small window of opportunity during which the adjust_lru() function can be called with a GEM refcount of zero from the TTM eviction code. This results in a kernel BUG(). Ensure that we don't attempt to modify the GEM shrinker lists unless we

Re: [PATCH] drm/i915/ttm: Fix illegal addition to shrinker list

2021-11-10 Thread Thomas Hellström
On 11/10/21 11:29, Matthew Auld wrote: On 10/11/2021 08:55, Thomas Hellström wrote: There's a small window of opportunity during which the adjust_lru() function can be called with a GEM refcount of zero from the TTM eviction code. This results in a kernel BUG(). Ensure that we don't attempt t

[PATCH v3 0/9] drm/simpledrm: Enable damage clips and virtual screens

2021-11-10 Thread Thomas Zimmermann
Enable FB_DAMAGE_CLIPS with simpledrm for improved performance and/or less overhead. With this in place, add support for virtual screens (i.e., framebuffers that are larger than the display resolution). This also enables fbdev panning and page flipping. After the discussion and bug fixing wrt to f

[PATCH v3 2/9] drm/format-helper: Rework format-helper memcpy functions

2021-11-10 Thread Thomas Zimmermann
Move destination-buffer clipping from all format-helper memcpy function into callers. Support destination-buffer pitch. Only distinguish between system and I/O memory, but use same logic everywhere. Signed-off-by: Thomas Zimmermann Tested-by: Noralf Trønnes Reviewed-by: Noralf Trønnes --- driv

[PATCH v3 1/9] drm/format-helper: Export drm_fb_clip_offset()

2021-11-10 Thread Thomas Zimmermann
Provide a function that computes the offset into a blit destination buffer. This will allow to move destination-buffer clipping into the format-helper callers. v2: * provide documentation (Sam) * return 'unsigned int' (Sam, Noralf) Signed-off-by: Thomas Zimmermann Reviewed-by: No

[PATCH v3 5/9] drm/format-helper: Streamline blit-helper interface

2021-11-10 Thread Thomas Zimmermann
Move destination-buffer clipping from format-helper blit function into caller. Rename drm_fb_blit_rect_dstclip() to drm_fb_blit_toio(). Done for consistency with the rest of the interface. Remove drm_fb_blit_dstclip(), which isn't required. Signed-off-by: Thomas Zimmermann Reviewed-by: Noralf Trø

[PATCH v3 9/9] drm: Clarify semantics of struct drm_mode_config.{min, max}_{width, height}

2021-11-10 Thread Thomas Zimmermann
Add additional information on the semantics of the size fields in struct drm_mode_config. Also add a TODO to review all driver for correct usage of these fields. Signed-off-by: Thomas Zimmermann Acked-by: Noralf Trønnes --- Documentation/gpu/todo.rst| 15 +++ include/drm/drm_mod

[PATCH v3 3/9] drm/format-helper: Add destination-buffer pitch to drm_fb_swab()

2021-11-10 Thread Thomas Zimmermann
Add destination-buffer pitch as argument to drm_fb_swab(). Done for consistency with the rest of the interface. v2: * update documentation (Noralf) Signed-off-by: Thomas Zimmermann Tested-by: Noralf Trønnes Reviewed-by: Noralf Trønnes --- drivers/gpu/drm/drm_format_helper.c | 21 +

[PATCH v3 4/9] drm/format-helper: Rework format-helper conversion functions

2021-11-10 Thread Thomas Zimmermann
Move destination-buffer clipping from all format-helper conversion functions into callers. Support destination-buffer pitch. Only distinguish between system and I/O memory, but use same logic everywhere. Simply harmonize the interface and semantics of the existing code. Not all conversion helpers

[PATCH v3 8/9] drm/simpledrm: Support virtual screen sizes

2021-11-10 Thread Thomas Zimmermann
Add constants for the maximum size of the shadow-plane surface size. Useful for shadow planes with virtual screen sizes. The current sizes are 4096 scanlines with 4096 pixels each. This seems reasonable for current hardware, but can be increased as necessary. In simpledrm, set the maximum framebuf

[PATCH v3 6/9] drm/fb-helper: Allocate shadow buffer of surface height

2021-11-10 Thread Thomas Zimmermann
Allocating a shadow buffer of the height of the buffer object does not support fbdev overallocation. Use surface height instead. Signed-off-by: Thomas Zimmermann Reviewed-by: Noralf Trønnes --- drivers/gpu/drm/drm_fb_helper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/d

[PATCH v3 7/9] drm/simpledrm: Enable FB_DAMAGE_CLIPS property

2021-11-10 Thread Thomas Zimmermann
Enable the FB_DAMAGE_CLIPS property to reduce display-update overhead. Also fixes a warning in the kernel log. simple-framebuffer simple-framebuffer.0: [drm] drm_plane_enable_fb_damage_clips() not called Fix the computation of the blit rectangle. This wasn't an issue so far, as simpledrm alway

Re: printk deadlock due to double lock attempt on current CPU's runqueue

2021-11-10 Thread Daniel Vetter
On Wed, Nov 10, 2021 at 11:13:37AM +0106, John Ogness wrote: > On 2021-11-10, Daniel Vetter wrote: > > I'm a bit out of the loop but from lwn articles my understanding is > > that part of upstreaming from -rt we no longer have the explicit "I'm > > a safe console for direct printing" opt-in. Which

Re: [PATCH 3/3] drm: Update documentation and TODO of gem_prime_mmap hook

2021-11-10 Thread Daniel Vetter
On Mon, Nov 08, 2021 at 11:28:46AM +0100, Thomas Zimmermann wrote: > The hook gem_prime_mmap in struct drm_driver is deprecated. Document > the new requirements. > > Signed-off-by: Thomas Zimmermann Reviewed-by: Daniel Vetter > --- > Documentation/gpu/todo.rst | 11 --- > include/drm/

Re: [PATCH] drm/i915: Skip error capture when wedged on init

2021-11-10 Thread Matthew Auld
On Tue, 9 Nov 2021 at 12:20, Tvrtko Ursulin wrote: > > From: Tvrtko Ursulin > > Trying to capture uninitialised engines when we wedged on init ends in > tears. Skip that together with uC capture, since failure to initialise the > latter can actually be one of the reasons for wedging on init. > >

[PATCH v4] drm/msm: use compatible lists to find mdp node

2021-11-10 Thread Krishna Manikandan
In the current implementation, substring comparison using device node name is used to find mdp node during driver probe. Use compatible string list instead of node name to get mdp node from the parent mdss node. Signed-off-by: Krishna Manikandan Reviewed-by: Stephen Boyd Changes in v2: - Use

Re: printk deadlock due to double lock attempt on current CPU's runqueue

2021-11-10 Thread Peter Zijlstra
On Wed, Nov 10, 2021 at 11:50:38AM +0100, Petr Mladek wrote: > On Tue 2021-11-09 12:06:48, Sultan Alsawaf wrote: > > Hi, > > > > I encountered a printk deadlock on 5.13 which appears to still affect the > > latest > > kernel. The deadlock occurs due to printk being used while having the > > curr

Re: [PATCH] drm/i915: Skip error capture when wedged on init

2021-11-10 Thread Tvrtko Ursulin
On 10/11/2021 10:48, Matthew Auld wrote: On Tue, 9 Nov 2021 at 12:20, Tvrtko Ursulin wrote: From: Tvrtko Ursulin Trying to capture uninitialised engines when we wedged on init ends in tears. Skip that together with uC capture, since failure to initialise the latter can actually be one of t

[PATCH v2] drm/i915: Skip error capture when wedged on init

2021-11-10 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Trying to capture uninitialised engines when we wedged on init ends in tears. Skip that together with uC capture, since failure to initialise the latter can actually be one of the reasons for wedging on init. v2: * Use i915_disable_error_state when wedging on init/fini. Si

Re: [Intel-gfx] [PATCH 1/1] drm/i915/rpm: Enable runtime pm autosuspend by default

2021-11-10 Thread Ville Syrjälä
On Wed, Nov 10, 2021 at 10:59:26AM +0530, Tilak Tangudu wrote: > Enable runtime pm autosuspend by default for gen12 and > later versions. > > Signed-off-by: Tilak Tangudu > --- > drivers/gpu/drm/i915/intel_runtime_pm.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm

Re: [RFC v2 02/22] drm: Add Enhanced Gamma and color lut range attributes

2021-11-10 Thread Ville Syrjälä
On Wed, Nov 10, 2021 at 10:49:24AM +0200, Pekka Paalanen wrote: > On Wed, 10 Nov 2021 00:02:16 +0200 > Ville Syrjälä wrote: > > > On Tue, Nov 09, 2021 at 03:47:58PM -0500, Harry Wentland wrote: > > > On 2021-11-08 04:54, Pekka Paalanen wrote: > > > > On Thu, 4 Nov 2021 12:27:56 -0400 > > > > Ha

Re: [PATCH] drm/i915: Use per device iommu check

2021-11-10 Thread Lu Baolu
On 2021/11/10 17:30, Tvrtko Ursulin wrote: On 10/11/2021 07:12, Lu Baolu wrote: Hi Tvrtko, On 2021/11/9 20:17, Tvrtko Ursulin wrote: From: Tvrtko Ursulin On igfx + dgfx setups, it appears that intel_iommu=igfx_off option only disables the igfx iommu. Stop relying on global intel_iommu_gfx_ma

[PATCH linux-next] drm/msm/dp: remove unneeded variable

2021-11-10 Thread cgel . zte
From: Changcheng Deng Fix the following coccicheck review: ./drivers/gpu/drm/msm/dp/dp_debug.c: Unneeded variable Remove unneeded variable used to store return value. Reported-by: Zeal Robot Signed-off-by: Changcheng Deng --- drivers/gpu/drm/msm/dp/dp_debug.c | 3 +-- 1 file changed, 1 inser

Re: [PATCH] drm/i915: Use per device iommu check

2021-11-10 Thread Tvrtko Ursulin
On 10/11/2021 12:04, Lu Baolu wrote: On 2021/11/10 17:30, Tvrtko Ursulin wrote: On 10/11/2021 07:12, Lu Baolu wrote: Hi Tvrtko, On 2021/11/9 20:17, Tvrtko Ursulin wrote: From: Tvrtko Ursulin On igfx + dgfx setups, it appears that intel_iommu=igfx_off option only disables the igfx iommu. S

Re: [Intel-gfx] [PATCH] drm/i915: Use per device iommu check

2021-11-10 Thread Robin Murphy
On 2021-11-10 09:35, Tvrtko Ursulin wrote: On 10/11/2021 07:25, Lu Baolu wrote: On 2021/11/10 1:35, Tvrtko Ursulin wrote: On 09/11/2021 17:19, Lucas De Marchi wrote: On Tue, Nov 09, 2021 at 12:17:59PM +, Tvrtko Ursulin wrote: From: Tvrtko Ursulin On igfx + dgfx setups, it appears that

Re: [Intel-gfx] [PATCH] drm/i915: Use per device iommu check

2021-11-10 Thread Lu Baolu
On 2021/11/10 17:35, Tvrtko Ursulin wrote: On 10/11/2021 07:25, Lu Baolu wrote: On 2021/11/10 1:35, Tvrtko Ursulin wrote: On 09/11/2021 17:19, Lucas De Marchi wrote: On Tue, Nov 09, 2021 at 12:17:59PM +, Tvrtko Ursulin wrote: From: Tvrtko Ursulin On igfx + dgfx setups, it appears that

Re: [PATCH v3 7/9] drm/simpledrm: Enable FB_DAMAGE_CLIPS property

2021-11-10 Thread Noralf Trønnes
Den 10.11.2021 11.37, skrev Thomas Zimmermann: > Enable the FB_DAMAGE_CLIPS property to reduce display-update > overhead. Also fixes a warning in the kernel log. > > simple-framebuffer simple-framebuffer.0: [drm] > drm_plane_enable_fb_damage_clips() not called > > Fix the computation of the

Re: [PATCH] drm/i915: Use per device iommu check

2021-11-10 Thread Lu Baolu
On 2021/11/10 20:08, Tvrtko Ursulin wrote: On 10/11/2021 12:04, Lu Baolu wrote: On 2021/11/10 17:30, Tvrtko Ursulin wrote: On 10/11/2021 07:12, Lu Baolu wrote: Hi Tvrtko, On 2021/11/9 20:17, Tvrtko Ursulin wrote: From: Tvrtko Ursulin On igfx + dgfx setups, it appears that intel_iommu=igfx

Re: [PATCH v2 1/3] drm/bridge: parade-ps8640: Don't try to enable VDO if poweron fails

2021-11-10 Thread Dafna Hirschfeld
On 02.11.21 11:36, AngeloGioacchino Del Regno wrote: If the bridge cannot get powered on, there's no reason to try to communicate with it: change the ps8640_bridge_poweron function to return an error value to the caller, so that we can avoid calling ps8640_bridge_vdo_control() in ps8640_pre_en

Re: [PATCH v2 2/3] drm/bridge: parade-ps8640: Move real poweroff action to new function

2021-11-10 Thread Dafna Hirschfeld
On 02.11.21 11:36, AngeloGioacchino Del Regno wrote: In preparation for varying the poweron error handling in function ps8640_bridge_poweron(), move function ps8640_bridge_poweroff() up and also move the actual logic to power off the chip to a new __ps8640_bridge_poweroff() function. Signed-o

Re: [PATCH v2 2/3] drm/bridge: parade-ps8640: Move real poweroff action to new function

2021-11-10 Thread AngeloGioacchino Del Regno
Il 10/11/21 13:44, Dafna Hirschfeld ha scritto: On 02.11.21 11:36, AngeloGioacchino Del Regno wrote: In preparation for varying the poweron error handling in function ps8640_bridge_poweron(), move function ps8640_bridge_poweroff() up and also move the actual logic to power off the chip to a ne

Re: [PATCH v3 7/9] drm/simpledrm: Enable FB_DAMAGE_CLIPS property

2021-11-10 Thread Thomas Zimmermann
Hi Am 10.11.21 um 13:34 schrieb Noralf Trønnes: Den 10.11.2021 11.37, skrev Thomas Zimmermann: Enable the FB_DAMAGE_CLIPS property to reduce display-update overhead. Also fixes a warning in the kernel log. simple-framebuffer simple-framebuffer.0: [drm] drm_plane_enable_fb_damage_clips()

Re: [PATCH v2 3/3] drm/bridge: parade-ps8640: Perform full poweroff if poweron fails

2021-11-10 Thread Dafna Hirschfeld
On 02.11.21 11:36, AngeloGioacchino Del Regno wrote: In function ps8640_bridge_poweron(), in case of a failure not relative to the regulators enablement, the code was disabling the regulators but the gpio changes that happened during the poweron sequence were not being reverted back to a clean

Re: [PATCH 2/2] drm/sched: serialize job_timeout and scheduler

2021-11-10 Thread Andrey Grodzovsky
On 2021-11-10 5:09 a.m., Christian König wrote: Am 10.11.21 um 10:50 schrieb Daniel Vetter: On Tue, Nov 09, 2021 at 08:17:01AM -0800, Rob Clark wrote: On Tue, Nov 9, 2021 at 1:07 AM Daniel Vetter wrote: On Mon, Nov 08, 2021 at 03:39:17PM -0800, Rob Clark wrote: I stumbled across this threa

Re: [PATCH 3/3] drm: Update documentation and TODO of gem_prime_mmap hook

2021-11-10 Thread Thomas Zimmermann
Hi Am 10.11.21 um 11:46 schrieb Daniel Vetter: On Mon, Nov 08, 2021 at 11:28:46AM +0100, Thomas Zimmermann wrote: The hook gem_prime_mmap in struct drm_driver is deprecated. Document the new requirements. Signed-off-by: Thomas Zimmermann Reviewed-by: Daniel Vetter Thanks a lot! With the

Re: [PATCH] drm/bridge: parade-ps8640: Assign drm_device to dp aux channel

2021-11-10 Thread Dafna Hirschfeld
On 04.11.21 12:20, AngeloGioacchino Del Regno wrote: As it was done with other bridge drivers and to solve a warning coming from drm_dp_aux_register(), add a backpointer to drm_device in the drm_dp_aux that we're registering. it is common to dump the exact warning in the commit log so peopl

[PATCH v6 1/7] dt-bindings: mediatek,dpi: Add DP_INTF compatible

2021-11-10 Thread Guillaume Ranquet
From: Markus Schneider-Pargmann DP_INTF is similar to DPI but does not have the exact same feature set or register layouts. DP_INTF is the sink of the display pipeline that is connected to the DisplayPort controller and encoder unit. It takes the same clocks as DPI. Signed-off-by: Markus Schnei

[PATCH v6 2/7] dt-bindings: mediatek,dp: Add Display Port binding

2021-11-10 Thread Guillaume Ranquet
From: Markus Schneider-Pargmann This controller is present on several mediatek hardware. Currently mt8195 and mt8395 have this controller without a functional difference, so only one compatible field is added. The controller can have two forms, as a normal display port and as an embedded display

[PATCH v6 4/7] video/hdmi: Add audio_infoframe packing for DP

2021-11-10 Thread Guillaume Ranquet
From: Markus Schneider-Pargmann Similar to HDMI, DP uses audio infoframes as well which are structured very similar to the HDMI ones. This patch adds a helper function to pack the HDMI audio infoframe for DP, called hdmi_audio_infoframe_pack_for_dp(). hdmi_audio_infoframe_pack_only() is split in

[PATCH v6 3/7] drm/edid: Add cea_sad helpers for freq/length

2021-11-10 Thread Guillaume Ranquet
From: Markus Schneider-Pargmann This patch adds two helper functions that extract the frequency and word length from a struct cea_sad. For these helper functions new defines are added that help translate the 'freq' and 'byte2' fields into real numbers. Signed-off-by: Markus Schneider-Pargmann

[PATCH v6 5/7] drm/mediatek: dpi: Add dpintf support

2021-11-10 Thread Guillaume Ranquet
From: Markus Schneider-Pargmann dpintf is the displayport interface hardware unit. This unit is similar to dpi and can reuse most of the code. This patch adds support for mt8195-dpintf to this dpi driver. Main differences are: - Some features/functional components are not available for dpintf

[PATCH v6 6/7] phy: phy-mtk-dp: Add driver for DP phy

2021-11-10 Thread Guillaume Ranquet
From: Markus Schneider-Pargmann This is a new driver that supports the integrated DisplayPort phy for mediatek SoCs, especially the mt8195. The phy is integrated into the DisplayPort controller and will be created by the mtk-dp driver. This driver expects a struct regmap to be able to work on the

[PATCH v6 7/7] drm/mediatek: Add mt8195 DisplayPort driver

2021-11-10 Thread Guillaume Ranquet
From: Markus Schneider-Pargmann This patch adds a DisplayPort driver for the Mediatek mt8195 SoC and a according phy driver mediatek-dp-phy. It supports both functional units on the mt8195, the embedded DisplayPort as well as the external DisplayPort units. It offers hot-plug-detection, audio up

Re: printk deadlock due to double lock attempt on current CPU's runqueue

2021-11-10 Thread Daniel Vetter
On Wed, Nov 10, 2021 at 12:20:05PM +0100, Peter Zijlstra wrote: > On Wed, Nov 10, 2021 at 11:50:38AM +0100, Petr Mladek wrote: > > On Tue 2021-11-09 12:06:48, Sultan Alsawaf wrote: > > > Hi, > > > > > > I encountered a printk deadlock on 5.13 which appears to still affect the > > > latest > > > k

Re: [PATCH 2/2] drm/sched: serialize job_timeout and scheduler

2021-11-10 Thread Daniel Vetter
On Wed, Nov 10, 2021 at 11:09:50AM +0100, Christian König wrote: > Am 10.11.21 um 10:50 schrieb Daniel Vetter: > > On Tue, Nov 09, 2021 at 08:17:01AM -0800, Rob Clark wrote: > > > On Tue, Nov 9, 2021 at 1:07 AM Daniel Vetter wrote: > > > > On Mon, Nov 08, 2021 at 03:39:17PM -0800, Rob Clark wrote:

Re: Questions about KMS flip

2021-11-10 Thread Daniel Vetter
On Wed, Nov 10, 2021 at 11:18:02AM +0100, Christian König wrote: > Am 08.11.21 um 15:59 schrieb Daniel Vetter: > > On Mon, Nov 08, 2021 at 08:44:24AM +0100, Christian König wrote: > > > Am 05.11.21 um 19:13 schrieb Daniel Vetter: > > > > On Thu, Nov 04, 2021 at 12:44:34PM -0400, Harry Wentland wrot

[PATCH] MAINTAINERS: update information for nouveau

2021-11-10 Thread Karol Herbst
Some side notes on this. Atm we do want to use gitlab for bug tracking and merge requests. But due to the nature of the current linux kernel development, we can only do so for nouveau internal changes. Everything else still needs to be sent as emails and this is also includes changes to UAPI etc.

Re: refactor the i915 GVT support and move to the modern mdev API v2

2021-11-10 Thread Joonas Lahtinen
Quoting Christoph Hellwig (2021-11-09 09:59:57) > On Thu, Nov 04, 2021 at 02:59:18PM +0200, Joonas Lahtinen wrote: > > The minimal we should do is to eliminate the double underscore > > prefixed functions. But I would prefer to have the symbol exports by > > default so that we can enable the functi

Re: [PATCH v1 1/1] drm: Replace kernel.h with the necessary inclusions

2021-11-10 Thread Thomas Zimmermann
Hi Am 10.11.21 um 11:24 schrieb Andy Shevchenko: When kernel.h is used in the headers it adds a lot into dependency hell, especially when there are circular dependencies are involved. Replace kernel.h inclusion with the list of what is really being used. Signed-off-by: Andy Shevchenko --- i

Re: [PATCH] drm/i915: Use per device iommu check

2021-11-10 Thread Tvrtko Ursulin
On 10/11/2021 12:35, Lu Baolu wrote: On 2021/11/10 20:08, Tvrtko Ursulin wrote: On 10/11/2021 12:04, Lu Baolu wrote: On 2021/11/10 17:30, Tvrtko Ursulin wrote: On 10/11/2021 07:12, Lu Baolu wrote: Hi Tvrtko, On 2021/11/9 20:17, Tvrtko Ursulin wrote: From: Tvrtko Ursulin On igfx + dgfx

Re: [PATCH] drm/i915: Use per device iommu check

2021-11-10 Thread Robin Murphy
On 2021-11-10 14:11, Tvrtko Ursulin wrote: On 10/11/2021 12:35, Lu Baolu wrote: On 2021/11/10 20:08, Tvrtko Ursulin wrote: On 10/11/2021 12:04, Lu Baolu wrote: On 2021/11/10 17:30, Tvrtko Ursulin wrote: On 10/11/2021 07:12, Lu Baolu wrote: Hi Tvrtko, On 2021/11/9 20:17, Tvrtko Ursulin wr

[PATCH v4] drm/ttm: Clarify that the TTM_PL_SYSTEM is under TTMs control

2021-11-10 Thread Zack Rusin
TTM takes full control over TTM_PL_SYSTEM placed buffers. This makes driver internal usage of TTM_PL_SYSTEM prone to errors because it requires the drivers to manually handle all interactions between TTM which can swap out those buffers whenever it thinks it's the right thing to do and driver. CPU

[PATCH v3 4/5] dt-bindings: media: Update maintainers for st, stm32-hwspinlock.yaml

2021-11-10 Thread patrice.chotard
From: Patrice Chotard Benjamin has left the company, remove his name from maintainers. Signed-off-by: Patrice Chotard --- .../devicetree/bindings/hwlock/st,stm32-hwspinlock.yaml | 1 - 1 file changed, 1 deletion(-) diff --git a/Documentation/devicetree/bindings/hwlock/st,stm32-hwspin

[PATCH v3 2/5] dt-bindings: mfd: timers: Update maintainers for st, stm32-timers

2021-11-10 Thread patrice.chotard
From: Patrice Chotard Benjamin has left the company, remove his name from maintainers. Signed-off-by: Patrice Chotard --- Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml | 1 - 1 file changed, 1 deletion(-) diff --git a/Documentation/devicetree/bindings/mfd/st,stm32-timers.yaml b/

[PATCH v3 5/5] dt-bindings: treewide: Update @st.com email address to @foss.st.com

2021-11-10 Thread patrice.chotard
From: Patrice Chotard Not all @st.com email address are concerned, only people who have a specific @foss.st.com email will see their entry updated. For some people, who left the company, remove their email. Cc: Alexandre Torgue Cc: Arnaud Pouliquen Cc: Fabien Dessenne Cc: Christophe Roullier

[PATCH v3 3/5] dt-bindings: media: Update maintainers for st, stm32-cec.yaml

2021-11-10 Thread patrice.chotard
From: Patrice Chotard Benjamin has left the company, remove his name from maintainers. Signed-off-by: Patrice Chotard --- Documentation/devicetree/bindings/media/st,stm32-cec.yaml | 1 - 1 file changed, 1 deletion(-) diff --git a/Documentation/devicetree/bindings/media/st,stm32-cec.yaml b/Do

[PATCH v3 0/5] Update STMicroelectronics maintainers email

2021-11-10 Thread patrice.chotard
From: Patrice Chotard Update maintainers name for some yaml files. Update @st.com email address to @foss.st.com as @foss.st.com email address is dedicated for upstream activities. Changes in v3: _ fix typo in patch 2/3/4 commit message _ resend to missing mailing list Patrice Chotard (5):

[PATCH v3 1/5] dt-bindings: timer: Update maintainers for st, stm32-timer

2021-11-10 Thread patrice.chotard
From: Patrice Chotard Benjamin has left the company, add Fabrice and myself as maintainers. Signed-off-by: Patrice Chotard --- Documentation/devicetree/bindings/timer/st,stm32-timer.yaml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/tim

Re: [RFC v2 02/22] drm: Add Enhanced Gamma and color lut range attributes

2021-11-10 Thread Harry Wentland
On 2021-11-10 06:55, Ville Syrjälä wrote: > On Wed, Nov 10, 2021 at 10:49:24AM +0200, Pekka Paalanen wrote: >> On Wed, 10 Nov 2021 00:02:16 +0200 >> Ville Syrjälä wrote: >> >>> On Tue, Nov 09, 2021 at 03:47:58PM -0500, Harry Wentland wrote: On 2021-11-08 04:54, Pekka Paalanen wrote:

Re: [PATCH v4 07/13] drm/msm: Track "seqno" fences by idr

2021-11-10 Thread Akhil P Oommen
On 7/28/2021 6:36 AM, Rob Clark wrote: From: Rob Clark Previously the (non-fd) fence returned from submit ioctl was a raw seqno, which is scoped to the ring. But from UABI standpoint, the ioctls related to seqno fences all specify a submitqueue. We can take advantage of that to replace the se

Re: [PATCH v2] drm/dp: Fix aux->transfer NULL pointer dereference on drm_dp_dpcd_access

2021-11-10 Thread Harry Wentland
On 2021-11-05 03:35, Yuan, Perry wrote: > [AMD Official Use Only] > > Hi Jani: > > >> -Original Message- >> From: Jani Nikula >> Sent: Wednesday, November 3, 2021 7:31 PM >> To: Yuan, Perry ; Maarten Lankhorst >> ; Maxime Ripard >> ; Thomas Zimmermann ; >> David Airlie ; Daniel Vetter

[PATCH] drm/nouveau: hdmigv100.c: fix corrupted HDMI Vendor InfoFrame

2021-11-10 Thread Hans Verkuil
gv100_hdmi_ctrl() writes vendor_infoframe.subpack0_high to 0x6f0110, and then overwrites it with 0. Just drop the overwrite with 0, that's clearly a mistake. Because of this issue the HDMI VIC is 0 instead of 1 in the HDMI Vendor InfoFrame when transmitting 4kp30. Signed-off-by: Hans Verkuil Fix

[PATCH] drm/nouveau: set RGB quantization range to FULL

2021-11-10 Thread Hans Verkuil
The nouveau driver outputs full range RGB, but the AVI InfoFrame just says 'Default' instead of 'Full'. Call drm_hdmi_avi_infoframe_quant_range to fill in the quantization field of the AVI InfoFrame correctly. Now displays that advertise RGB Selectable Quantization Range in their EDID will underst

Re: [PATCH v1 1/1] drm: Replace kernel.h with the necessary inclusions

2021-11-10 Thread Andy Shevchenko
On Wed, Nov 10, 2021 at 3:55 PM Thomas Zimmermann wrote: > Am 10.11.21 um 11:24 schrieb Andy Shevchenko: > > When kernel.h is used in the headers it adds a lot into dependency hell, > > especially when there are circular dependencies are involved. > > > > Replace kernel.h inclusion with the list o

Re: [PATCH] drm/radeon: correct indentation

2021-11-10 Thread Alex Deucher
Applied. Thanks! Alex On Wed, Nov 10, 2021 at 11:08 AM Xu Wang wrote: > > Correct indentation in radeon_driver_load_kms. > > Signed-off-by: Xu Wang > --- > drivers/gpu/drm/radeon/radeon_kms.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/radeon/radeo

Re: [PATCH] drm/amdgpu: remove unneeded variable

2021-11-10 Thread Alex Deucher
Applied. Thanks! Alex On Wed, Nov 10, 2021 at 12:53 AM wrote: > > From: ran jianping > > Fix the following coccicheck review: > ./drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c:1174:14-18 > :Unneeded variable > > Remove unneeded variable used to store return value. > > Reported-by: Ze

Re: [PATCH v1 1/1] drm: Replace kernel.h with the necessary inclusions

2021-11-10 Thread Thomas Zimmermann
Hi Am 10.11.21 um 17:34 schrieb Andy Shevchenko: On Wed, Nov 10, 2021 at 3:55 PM Thomas Zimmermann wrote: Am 10.11.21 um 11:24 schrieb Andy Shevchenko: When kernel.h is used in the headers it adds a lot into dependency hell, especially when there are circular dependencies are involved. Repla

Re: [PATCH v4 07/13] drm/msm: Track "seqno" fences by idr

2021-11-10 Thread Rob Clark
On Wed, Nov 10, 2021 at 7:28 AM Akhil P Oommen wrote: > > On 7/28/2021 6:36 AM, Rob Clark wrote: > > From: Rob Clark > > > > Previously the (non-fd) fence returned from submit ioctl was a raw > > seqno, which is scoped to the ring. But from UABI standpoint, the > > ioctls related to seqno fences

Re: [PATCH v2 6/8] drm: vkms: Refactor the plane composer to accept new formats

2021-11-10 Thread Igor Torrente
On Tue, Nov 9, 2021 at 8:40 AM Pekka Paalanen wrote: > > Hi Igor, > > again, that is a really nice speed-up. Unfortunately, I find the code > rather messy and hard to follow. I hope my comments below help with > re-designing it to be easier to understand. > > > On Tue, 26 Oct 2021 08:34:06 -0300 >

Re: [PATCH 2/2] MAINTAINERS: Mark VMware mailing list entries as private

2021-11-10 Thread Srivatsa S. Bhat
On Tue, Nov 09, 2021 at 01:57:31PM -0800, Joe Perches wrote: > On Tue, 2021-11-09 at 00:58 +, Nadav Amit wrote: > > > On Nov 8, 2021, at 4:37 PM, Joe Perches wrote: > > > On Mon, 2021-11-08 at 16:22 -0800, Srivatsa S. Bhat wrote: > > > > > > So it's an exploder not an actual maintainer and it

Re: [PATCH v2 0/8] Add new formats support to vkms

2021-11-10 Thread Igor Torrente
Hi Pekka, On Tue, Nov 9, 2021 at 6:32 AM Pekka Paalanen wrote: > > On Tue, 26 Oct 2021 08:34:00 -0300 > Igor Torrente wrote: > > > Summary > > === > > This series of patches refactor some vkms components in order to introduce > > new formats to the planes and writeback connector. > > > > Now

Re: [PATCH] MAINTAINERS: update information for nouveau

2021-11-10 Thread Daniel Vetter
On Wed, Nov 10, 2021 at 02:31:57PM +0100, Karol Herbst wrote: > Some side notes on this. Atm we do want to use gitlab for bug tracking and > merge requests. But due to the nature of the current linux kernel > development, we can only do so for nouveau internal changes. > > Everything else still ne

Re: [PATCH 2/2] MAINTAINERS: Mark VMware mailing list entries as private

2021-11-10 Thread Nadav Amit
> On Nov 10, 2021, at 9:20 AM, Srivatsa S. Bhat wrote: > > On Tue, Nov 09, 2021 at 01:57:31PM -0800, Joe Perches wrote: >> On Tue, 2021-11-09 at 00:58 +, Nadav Amit wrote: On Nov 8, 2021, at 4:37 PM, Joe Perches wrote: On Mon, 2021-11-08 at 16:22 -0800, Srivatsa S. Bhat wrote: >>

Re: [RFC] arm64: dts: imx8mm: Add MIPI and LCDIF nodes

2021-11-10 Thread Jagan Teki
On Wed, Nov 10, 2021 at 2:24 AM Tim Harvey wrote: > > On Tue, Nov 9, 2021 at 12:39 PM Marek Vasut wrote: > > > > On 11/9/21 8:35 PM, Adam Ford wrote: > > > > [...] > > > > >> diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > >> b/arch/arm64/boot/dts/freescale/imx8mm.dtsi > > >> index 20

Re: [PATCH] [PATCH v2] drm/sun4i: fix unmet dependency on RESET_CONTROLLER for PHY_SUN6I_MIPI_DPHY

2021-11-10 Thread Jernej Škrabec
Hi Julian, Dne torek, 09. november 2021 ob 04:23:51 CET je Julian Braha napisal(a): > When PHY_SUN6I_MIPI_DPHY is selected, and RESET_CONTROLLER > is not selected, Kbuild gives the following warning: > > WARNING: unmet direct dependencies detected for PHY_SUN6I_MIPI_DPHY > Depends on [n]: (ARCH

Re: [PATCH 2/2] MAINTAINERS: Mark VMware mailing list entries as private

2021-11-10 Thread Srivatsa S. Bhat
On Wed, Nov 10, 2021 at 05:40:09PM +, Nadav Amit wrote: > > > > On Nov 10, 2021, at 9:20 AM, Srivatsa S. Bhat > > wrote: > > > > On Tue, Nov 09, 2021 at 01:57:31PM -0800, Joe Perches wrote: > >> On Tue, 2021-11-09 at 00:58 +, Nadav Amit wrote: > On Nov 8, 2021, at 4:37 PM, Joe Per

Re: [PATCH 2/2] drm/sched: serialize job_timeout and scheduler

2021-11-10 Thread Rob Clark
On Wed, Nov 10, 2021 at 1:50 AM Daniel Vetter wrote: > > On Tue, Nov 09, 2021 at 08:17:01AM -0800, Rob Clark wrote: > > On Tue, Nov 9, 2021 at 1:07 AM Daniel Vetter wrote: > > > > > > On Mon, Nov 08, 2021 at 03:39:17PM -0800, Rob Clark wrote: > > > > I stumbled across this thread when I ran into

Re: [PATCH] drm/bridge: dw-hdmi: handle ELD when DRM_BRIDGE_ATTACH_NO_CONNECTOR

2021-11-10 Thread Jernej Škrabec
Hi Neil, sorry for late response. Dne petek, 29. oktober 2021 ob 15:59:47 CET je Neil Armstrong napisal(a): > The current ELD handling takes the internal connector ELD buffer and > shares it to the I2S and AHB sub-driver. > > But with DRM_BRIDGE_ATTACH_NO_CONNECTOR, the connector is created > el

[PATCH] drm/msm: Fix mmap to include VM_IO and VM_DONTDUMP

2021-11-10 Thread Douglas Anderson
In commit 510410bfc034 ("drm/msm: Implement mmap as GEM object function") we switched to a new/cleaner method of doing things. That's good, but we missed a little bit. Before that commit, we used to _first_ run through the drm_gem_mmap_obj() case where `obj->funcs->mmap()` was NULL. That meant tha

[PATCH v6 2/8] drm/ingenic: Add support for JZ4780 and HDMI output

2021-11-10 Thread H. Nikolaus Schaller
From: Paul Boddie Add support for the LCD controller present on JZ4780 SoCs. This SoC uses 8-byte descriptors which extend the current 4-byte descriptors used for other Ingenic SoCs. Tested on MIPS Creator CI20 board. Signed-off-by: Paul Boddie Signed-off-by: Ezequiel Garcia Signed-off-by: H.

[PATCH v6 1/8] drm/ingenic: prepare ingenic drm for later addition of JZ4780

2021-11-10 Thread H. Nikolaus Schaller
This changes the way the regmap is allocated to prepare for the later addition of the JZ4780 which has more registers and bits than the others. Therefore we make the regmap as big as the reg property in the device tree tells. Suggested-by: Paul Cercueil Signed-off-by: H. Nikolaus Schaller ---

  1   2   >