Re: [PATCH v2 19/23] clk: sunxi-ng: ccu: add Display Engine 3.3 (DE33) support

2024-07-05 Thread Stephen Boyd
Quoting Ryan Walklin (2024-07-05 01:39:15) > Hi Stephen, thanks for reviewing. > > On Thu, 4 Jul 2024, at 11:02 AM, Stephen Boyd wrote: > > Quoting Ryan Walklin (2024-07-03 03:51:09) > >> +#include > > > > What is this include for? > > > for writel, however have confirmed this should instead be #

Re: [PATCH v7 8/8] drm/ttm: Balance ttm_resource_cursor_init() and ttm_resource_cursor_fini()

2024-07-05 Thread kernel test robot
Hi Thomas, kernel test robot noticed the following build warnings: [auto build test WARNING on drm-xe/drm-xe-next] [also build test WARNING on drm/drm-next drm-exynos/exynos-drm-next drm-intel/for-linux-next drm-tip/drm-tip next-20240703] [cannot apply to drm-intel/for-linux-next-fixes drm-misc/

[PATCH v4 11/11] drm/amd/display: remove dc_edid handler from dm_helpers_parse_edid_caps

2024-07-05 Thread Melissa Wen
We can parse edid caps from drm_edid and drm_eld and any calls of dm_helpers_parse_edid_caps is made in a state that we have drm_edid set to amdgpu connector. Signed-off-by: Melissa Wen --- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 6 ++-- .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 2

[PATCH v4 10/11] drm/amd/display: get SADB from drm_eld when parsing EDID caps

2024-07-05 Thread Melissa Wen
instead of parsing struct edid. Signed-off-by: Melissa Wen --- .../drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 15 +++ 1 file changed, 3 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c b/drivers/gpu/drm/amd/display/amdgpu_dm

[PATCH v4 09/11] drm/amd/display: get SAD from drm_eld when parsing EDID caps

2024-07-05 Thread Melissa Wen
instead of parsing struct edid. Signed-off-by: Melissa Wen --- .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c b/drivers/gpu/drm/amd/display/amdgpu_dm/

[PATCH v4 08/11] drm/amd/display: parse display name from drm_eld

2024-07-05 Thread Melissa Wen
We don't need to parse dc_edid to get the display name since it's already set in drm_eld which in turn had it values updated when updating connector with the opaque drm_edid. Signed-off-by: Melissa Wen --- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 10 ++ 1 file changed, 6

[PATCH v4 07/11] drm/amd/display: don't give initial values for sad/b_count

2024-07-05 Thread Melissa Wen
Signed-off-by: Melissa Wen --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c index 85704fd75ee4.

[PATCH v4 06/11] drm/amd/display: use drm_edid_product_id for parsing EDID product info

2024-07-05 Thread Melissa Wen
Since [1], we can use drm_edid_product_id to get debug info from drm_edid instead of directly parsing EDID. [1] https://lore.kernel.org/dri-devel/cover.1712655867.git.jani.nik...@intel.com/ Signed-off-by: Melissa Wen --- .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 37 ++- 1

[PATCH v4 05/11] drm/amd/display: remove redundant freesync parser for DP

2024-07-05 Thread Melissa Wen
When updating connector under drm_edid infrastructure, many calculations and validations are already done and become redundant inside AMD driver. Remove those driver-specific code in favor of the DRM common code. Signed-off-by: Melissa Wen --- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 88 +

[PATCH v4 04/11] drm/amd/display: always call connector_update when parsing freesync_caps

2024-07-05 Thread Melissa Wen
Update connector caps with drm_edid data before parsing info for freesync. Signed-off-by: Melissa Wen --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm

[PATCH v4 03/11] drm/amd/display: switch to setting physical address directly

2024-07-05 Thread Melissa Wen
Connectors have source physical address available in display info. Use drm_dp_cec_attach() to use it instead of parsing the EDID again. Signed-off-by: Melissa Wen --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/driver

[PATCH v4 02/11] drm/amd/display: switch amdgpu_dm_connector to use struct drm_edid

2024-07-05 Thread Melissa Wen
Replace raw edid handling (struct edid) with the opaque EDID type (struct drm_edid) on amdgpu_dm_connector for consistency. It may also prevent mismatch of approaches in different parts of the driver code. Working in progress. It was only exercised with IGT tests. v2: use const to fix warnings (Al

[PATCH v4 01/11] drm/amd/display: clean unused variables for hdmi freesync parser

2024-07-05 Thread Melissa Wen
Signed-off-by: Melissa Wen --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c index 98cf523a629e..1dfa7ec9af35 100644 --- a/drivers/gpu/d

[PATCH v4 00/11] drm/amd/display: switch amdgpu_dm_connector to use struct drm_edid

2024-07-05 Thread Melissa Wen
Hi, This series is an updated version of the previous attempt to make AMD display driver use opaque drm_edid instead of open struct edid[1]. Here I address Jani's last review and remove the raw edid handling when parsing edid caps by using drm_eld info and drm_edid_product_id from Jani's series[2]

[PATCH v8 2/2] drm/amdgpu: Enable async flip on overlay planes

2024-07-05 Thread André Almeida
amdgpu can handle async flips on overlay planes, so allow it for atomic async checks. Signed-off-by: André Almeida --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_plane.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_pla

[PATCH v8 1/2] drm/atomic: Let drivers decide which planes to async flip

2024-07-05 Thread André Almeida
Currently, DRM atomic uAPI allows only primary planes to be flipped asynchronously. However, each driver might be able to perform async flips in other different plane types. To enable drivers to set their own restrictions on which type of plane they can or cannot flip, use the existing atomic_async

[PATCH v8 0/1] drm/atomic: Ease async flip restrictions

2024-07-05 Thread André Almeida
Hi, As per my previous patchsets, the goal of this work is to find a nice way to allow amdgpu to perform async page flips in the overlay plane as well, not only on the primary one. Currently, when using the atomic uAPI, this is the only type of plane allowed to do async flips, and every driver acc

Re: [PATCH] drm: lcdif: Use adjusted_mode .clock instead of .crtc_clock

2024-07-05 Thread Marek Vasut
On 6/24/24 11:19 AM, Alexander Stein wrote: Am Freitag, 31. Mai 2024, 22:27:21 CEST schrieb Marek Vasut: In case an upstream bridge modified the required clock frequency in its .atomic_check callback by setting adjusted_mode.clock , make sure that clock frequency is generated by the LCDIFv3 bloc

Re: [PATCH v16 3/9] mm/gup: Introduce memfd_pin_folios() for pinning memfd folios

2024-07-05 Thread Andrew Morton
On Fri, 5 Jul 2024 22:11:14 + "Kasireddy, Vivek" wrote: > Hi Andrew and SJ, > > > > > > > > > I didn't look deep into the patch, so unsure if that's a valid fix, > > > though. > > > May I ask your thoughts? > > > > Perhaps we should propagate the errno which was returned by > > try_grab

RE: [PATCH v16 3/9] mm/gup: Introduce memfd_pin_folios() for pinning memfd folios

2024-07-05 Thread Kasireddy, Vivek
Hi Andrew and SJ, > > On Fri, 5 Jul 2024 13:48:25 -0700 SeongJae Park wrote: > > > > + * memfd_pin_folios() - pin folios associated with a memfd > > [...] > > > + for (i = 0; i < nr_found; i++) { > > > + /* > > > + * As there ca

Re: [PATCH v16 3/9] mm/gup: Introduce memfd_pin_folios() for pinning memfd folios

2024-07-05 Thread Andrew Morton
On Fri, 5 Jul 2024 13:48:25 -0700 SeongJae Park wrote: > > + * memfd_pin_folios() - pin folios associated with a memfd > [...] > > + for (i = 0; i < nr_found; i++) { > > + /* > > +* As there can be multiple entries for a > >

Re: [PATCH 09/11] usb: dwc2: Skip clock gating on Broadcom SoCs

2024-07-05 Thread Lukas Wunner
On Fri, Jul 05, 2024 at 12:16:14PM -0500, Jeremy Linton wrote: > > Am 05.07.24 um 17:03 schrieb Lukas Wunner: > > > Careful there, the patch vaguely says... > > > > > > With that added and identified as "BCM2848", > > > an id in use by other OSs for this device, the dw2 > > > controlle

Re: [PATCH v16 3/9] mm/gup: Introduce memfd_pin_folios() for pinning memfd folios

2024-07-05 Thread SeongJae Park
Hello Vivek and Andrew, On Sun, 23 Jun 2024 23:36:11 -0700 Vivek Kasireddy wrote: > For drivers that would like to longterm-pin the folios associated > with a memfd, the memfd_pin_folios() API provides an option to > not only pin the folios via FOLL_PIN but also to check and migrate > them if t

[PATCH v2 5/5] drm/etnaviv: take current primitive into account when checking for hung GPU

2024-07-05 Thread Lucas Stach
Large draws can make the GPU appear to be stuck to the current hangcheck logic as the FE address will not move until the draw is finished. However, the FE has a debug register, which records the current primitive ID within a draw. Using this debug register we can extend the timeout as long as the d

[PATCH v2 1/5] drm/etnaviv: hold GPU lock across perfmon sampling

2024-07-05 Thread Lucas Stach
The perfmon sampling mutates shared GPU state (e.g. VIVS_HI_CLOCK_CONTROL to select the pipe for the perf counter reads). To avoid clashing with other functions mutating the same state (e.g. etnaviv_gpu_update_clock) the perfmon sampling needs to hold the GPU lock. Fixes: 68dc0b295dcb ("drm/etnavi

[PATCH v2 3/5] drm/etnaviv: unconditionally enable debug registers

2024-07-05 Thread Lucas Stach
A later change will use the FE debug registers to improve GPU progress monitoring. Instead of having to keep track of the usage state of the debug registers and lock access to the VIVS_HI_CLOCK_CONTROL register, statically enable debug register access during GPU init. The Vivante downstream driver

[PATCH v2 4/5] drm/etnaviv: update hardware headers from rnndb

2024-07-05 Thread Lucas Stach
Update state_hi.xml.h header from etna_viv commit 8f43a34fd9cd ("rndb: document FE current primitve debug reg") Signed-off-by: Lucas Stach --- v2: no changes --- drivers/gpu/drm/etnaviv/state_hi.xml.h | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/d

[PATCH v2 2/5] drm/etnaviv: assert GPU lock held in perfmon pipe_*_read functions

2024-07-05 Thread Lucas Stach
The perf counter read functions don't just read registers, but they also mutate state to direct the reads towards the correct pipe and engine. Assert that the GPU mutex is held at this point, so that those state changes don't interfere with others. Signed-off-by: Lucas Stach --- v2: new patch ---

Re: [PULL] drm-fixes for v6.10

2024-07-05 Thread pr-tracker-bot
The pull request you sent on Fri, 5 Jul 2024 17:14:38 +0200: > https://gitlab.freedesktop.org/drm/kernel.git tags/drm-fixes-2024-07-05 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/dd9d7390b2de008448eb3328d4a0504c76c74572 Thank you! -- Deet-doot-dot, I am a bot. ht

Re: Time for drm-ci-next?

2024-07-05 Thread Rob Clark
On Fri, Jul 5, 2024 at 3:36 AM Daniel Vetter wrote: > > On Thu, Jul 04, 2024 at 08:40:26AM -0700, Rob Clark wrote: > > On Thu, Jul 4, 2024 at 7:08 AM Daniel Vetter wrote: > > > > > > On Tue, Jul 02, 2024 at 05:32:39AM -0700, Rob Clark wrote: > > > > On Fri, Jun 28, 2024 at 10:44 AM Daniel Vetter

Re: [PATCH 10/10] MAINTAINERS: Add maintainer for i.MX8qxp Display Controller

2024-07-05 Thread Frank Li
On Fri, Jul 05, 2024 at 05:09:32PM +0800, Liu Ying wrote: > Add myself as the maintainer of i.MX8qxp Display Controller. > > Signed-off-by: Liu Ying > --- > MAINTAINERS | 19 +++ > 1 file changed, 19 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 35db18d26c11..

Re: vc4: Increased CMA usage after replace drm_gem_dma_object for drm_gem_object in vc4_exec_info

2024-07-05 Thread Stefan Wahren
Hi, Am 05.07.24 um 17:19 schrieb Daniel Vetter: Adding dri-devel On Fri, 5 Jul 2024 at 17:14, Stefan Wahren wrote: Hi, last year my Raspberry Pi 3B Plus died, so i didn't noticed this sooner. If I ran Raspberry Pi OS Bullseye with X11 and a recent Mainline Kernel (arm/

[Bug 219007] opening and closing /dev/dri/card0 in a QEMU KVM instance will shutdown system, 6.10.0-rc6+

2024-07-05 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=219007 --- Comment #1 from Colin Ian King (colin.i.k...@gmail.com) --- Note this also can reproduce when running *without* root privileged too, so this is a user space DoS attack vector. -- You may reply to this email to add a comment. You are receivi

[Bug 219007] opening and closing /dev/dri/card0 in a QEMU KVM instance will shutdown system, 6.10.0-rc6+

2024-07-05 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=219007 Colin Ian King (colin.i.k...@gmail.com) changed: What|Removed |Added Hardware|All |Intel

[Bug 219007] New: opening and closing /dev/dri/card0 in a QEMU KVM instance will shutdown system, 6.10.0-rc6+

2024-07-05 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=219007 Bug ID: 219007 Summary: opening and closing /dev/dri/card0 in a QEMU KVM instance will shutdown system, 6.10.0-rc6+ Product: Drivers Version: 2.5 Hardware: All

Re: [PATCH 0/8] dma-buf: heaps: Support carved-out heaps and ECC related-flags

2024-07-05 Thread Daniel Vetter
Just figured I'll jump in on one detail here. On Fri, Jul 05, 2024 at 04:31:34PM +0200, Thierry Reding wrote: > On Thu, Jul 04, 2024 at 02:24:49PM GMT, Maxime Ripard wrote: > > On Fri, Jun 28, 2024 at 04:42:35PM GMT, Thierry Reding wrote: > > > On Fri, Jun 28, 2024 at 03:08:46PM GMT, Maxime Ripard

[PATCH v7 8/8] drm/ttm: Balance ttm_resource_cursor_init() and ttm_resource_cursor_fini()

2024-07-05 Thread Thomas Hellström
Make the interface more symmetric by providing and using a ttm_resource_cursor_init(). Signed-off-by: Thomas Hellström --- drivers/gpu/drm/ttm/ttm_bo.c | 3 ++- drivers/gpu/drm/ttm/ttm_bo_util.c | 4 +++- drivers/gpu/drm/ttm/ttm_resource.c | 34 +- include/dr

[PATCH v7 7/8] drm/ttm: Use the LRU walker for eviction

2024-07-05 Thread Thomas Hellström
Use the LRU walker for eviction. This helps removing a lot of code with weird locking semantics. The functionality is slightly changed so that when trylocked buffer objects are exhausted, we continue to interleave walks with ticket-locks while there is still progress made. The list walks are not r

[PATCH v7 6/8] drm/ttm: Use the LRU walker helper for swapping

2024-07-05 Thread Thomas Hellström
Rework the TTM swapping to use the LRU walker helper. This helps fixing up the ttm_bo_swapout() interface to be consistent about not requiring any locking. For now mimic the current behaviour of using trylock only. We could be using ticket-locks here but defer that until it's deemed necessary. The

[PATCH v7 2/8] drm/ttm: Slightly clean up LRU list iteration

2024-07-05 Thread Thomas Hellström
To make the transition to using lru hitches easier, simplify the ttm_resource_manager_next() interface to only take the cursor and reuse ttm_resource_manager_next() functionality from ttm_resource_manager_first(). Cc: Christian König Cc: Somalapuram Amaranath Cc: Matthew Brost Cc: Signed-off-b

[PATCH v7 5/8] drm/ttm: Provide a generic LRU walker helper

2024-07-05 Thread Thomas Hellström
Provide a generic LRU walker in TTM, in the spirit of drm_gem_lru_scan() but building on the restartable TTM LRU functionality. The LRU walker optionally supports locking objects as part of a ww mutex locking transaction, to mimic to some extent the current functionality in ttm. However any -EDEAD

[PATCH v7 3/8] drm/ttm: Use LRU hitches

2024-07-05 Thread Thomas Hellström
Have iterators insert themselves into the list they are iterating over using hitch list nodes. Since only the iterator owner can remove these list nodes from the list, it's safe to unlock the list and when continuing, use them as a starting point. Due to the way LRU bumping works in TTM, newly adde

[PATCH v7 4/8] drm/ttm, drm/amdgpu, drm/xe: Consider hitch moves within bulk sublist moves

2024-07-05 Thread Thomas Hellström
To address the problem with hitches moving when bulk move sublists are lru-bumped, register the list cursors with the ttm_lru_bulk_move structure when traversing its list, and when lru-bumping the list, move the cursor hitch to the tail. This also means it's mandatory for drivers to call ttm_lru_bu

[PATCH v7 1/8] drm/ttm: Allow TTM LRU list nodes of different types

2024-07-05 Thread Thomas Hellström
To be able to handle list unlocking while traversing the LRU list, we want the iterators not only to point to the next position of the list traversal, but to insert themselves as list nodes at that point to work around the fact that the next node might otherwise disappear from the list while the it

[PATCH v7 0/8] TTM LRU-walk cherry-picks

2024-07-05 Thread Thomas Hellström
These are cherry-picks from the xe shrinker series here: https://patchwork.freedesktop.org/series/131815/ extracted to speed up review progress and inclusion. The series provides a restartable LRU walk and makes it possible resume the walk after dropping the lock to evict or swap out. Patch 1-4

Re: [PATCH 09/11] usb: dwc2: Skip clock gating on Broadcom SoCs

2024-07-05 Thread Stefan Wahren
Hi Jeremy, Am 05.07.24 um 17:03 schrieb Lukas Wunner: On Fri, Jul 05, 2024 at 12:22:33PM +0200, Stefan Wahren wrote: Am 05.07.24 um 10:48 schrieb Lukas Wunner: The real question is whether BCM2848 platforms likewise cannot disable the clock of the dwc2 controller or whether this is specific to

Re: vc4: Increased CMA usage after replace drm_gem_dma_object for drm_gem_object in vc4_exec_info

2024-07-05 Thread Daniel Vetter
Adding dri-devel On Fri, 5 Jul 2024 at 17:14, Stefan Wahren wrote: > Hi, > last year my Raspberry Pi 3B Plus died, so i didn't noticed this sooner. > If I ran Raspberry Pi OS Bullseye with X11 and a recent Mainline Kernel > (arm/multi_v7_defconfig) on my new Raspberry Pi 3 B+, i'm getting the >

Re: [PATCH] MAINTAINERS: drm/sti: mark it as Odd Fixes

2024-07-05 Thread Alain Volmat
Hi Krzysztof, Daniel, and Maxime, On Fri, Jul 05, 2024 at 03:41:26PM +0200, Daniel Vetter wrote: > On Fri, Jul 05, 2024 at 01:33:38PM +0200, Krzysztof Kozlowski wrote: > > On 05/07/2024 13:22, Maxime Ripard wrote: > > > On Fri, Jul 05, 2024 at 01:19:50PM GMT, Daniel Vetter wrote: > > >> On Fri, Ju

[PULL] drm-fixes for v6.10

2024-07-05 Thread Daniel Vetter
Hi Linus, Dave is on vacations, should be back next week. Just small fixes all over here, all quiet as it should. Cheers, Sima drm-fixes-2024-07-05: drm-fixes for v6.10-rc7 drivers: - amd: mostly amdgpu display fixes + radeon vm NULL deref fix - xe: migration error handling + typoed register na

Re: [PATCH 09/11] usb: dwc2: Skip clock gating on Broadcom SoCs

2024-07-05 Thread Lukas Wunner
On Fri, Jul 05, 2024 at 12:22:33PM +0200, Stefan Wahren wrote: > Am 05.07.24 um 10:48 schrieb Lukas Wunner: > > A similar issue was reported for Agilex platforms back in 2021: > > > > https://lore.kernel.org/all/5e8cbce0-3260-2971-484f-fc73a3b2b...@synopsys.com/ > > > > It was fixed by commit 3d8

Re: [PATCH v2] drm/msm: add msm8998 hdmi phy/pll support

2024-07-05 Thread Dmitry Baryshkov
On Thu, Jul 04, 2024 at 06:45:36PM GMT, Marc Gonzalez wrote: > From: Arnaud Vrac > > Ported from the downstream driver. Please write some sensible commit message. > > Signed-off-by: Arnaud Vrac > Signed-off-by: Marc Gonzalez > --- > drivers/gpu/drm/msm/Makefile | 1 + >

Re: [PATCH 0/8] dma-buf: heaps: Support carved-out heaps and ECC related-flags

2024-07-05 Thread Thierry Reding
On Thu, Jul 04, 2024 at 02:24:49PM GMT, Maxime Ripard wrote: > On Fri, Jun 28, 2024 at 04:42:35PM GMT, Thierry Reding wrote: > > On Fri, Jun 28, 2024 at 03:08:46PM GMT, Maxime Ripard wrote: > > > Hi, > > > > > > On Fri, Jun 28, 2024 at 01:29:17PM GMT, Thierry Reding wrote: > > > > On Tue, May 21,

Re: [PATCH] MAINTAINERS: drm/sti: mark it as Odd Fixes

2024-07-05 Thread Daniel Vetter
On Fri, Jul 05, 2024 at 01:33:38PM +0200, Krzysztof Kozlowski wrote: > On 05/07/2024 13:22, Maxime Ripard wrote: > > On Fri, Jul 05, 2024 at 01:19:50PM GMT, Daniel Vetter wrote: > >> On Fri, Jul 05, 2024 at 12:03:56PM +0200, Krzysztof Kozlowski wrote: > >>> Patches to STI DRM are not being picked u

Re: [PATCH 2/5] drm/exec: don't immediately add the prelocked obj

2024-07-05 Thread Thomas Hellström
On Fri, 2024-07-05 at 14:41 +0200, Christian König wrote: > Am 03.07.24 um 17:51 schrieb Thomas Hellström: > > On Wed, 2024-07-03 at 15:25 +0200, Christian König wrote: > > > Some contended objects might never be locked again in the case of > > > eviction > > > handling for example. > > > > > > Ma

[PATCH v3] drm/xe: Use write-back caching mode for system memory on DGFX

2024-07-05 Thread Thomas Hellström
The caching mode for buffer objects with VRAM as a possible placement was forced to write-combined, regardless of placement. However, write-combined system memory is expensive to allocate and even though it is pooled, the pool is expensive to shrink, since it involves global CPU TLB flushes. More

Re: [PATCH 00/10] Add Freescale i.MX8qxp Display Controller support

2024-07-05 Thread Francesco Dolcini
Hello Liu, On Fri, Jul 05, 2024 at 05:09:22PM +0800, Liu Ying wrote: > This patch series aims to add Freescale i.MX8qxp Display Controller support. I really appreciate your work here, I am looking forward for a better support in mainline Linux for both i.MX8QXP and i.MX8QP. With that said, would

Re: [PATCH 2/5] drm/exec: don't immediately add the prelocked obj

2024-07-05 Thread Christian König
Am 03.07.24 um 17:51 schrieb Thomas Hellström: On Wed, 2024-07-03 at 15:25 +0200, Christian König wrote: Some contended objects might never be locked again in the case of eviction handling for example. Make sure that those doesn't show up in the list of locked objects until they are explicitely

[PATCH v4 7/7] drm/mgag200: Implement struct drm_crtc_funcs.get_vblank_timestamp

2024-07-05 Thread Thomas Zimmermann
Implement struct drm_crtc_funcs.get_vblank_timestamp with the DRM helper drm_crtc_vblank_helper_get_vblank_timestamp() with its helper get_scanout_position. Read the scanout position from the MGAREG_VCOUNT register. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/mgag200/mgag200_drv.h | 10

[PATCH v4 6/7] drm/mgag200: Add vblank support

2024-07-05 Thread Thomas Zimmermann
There's no VBLANK interrupt on Matrox chipsets. The workaround that is being used here and in other free Matrox drivers is to program to the value of and enable the VLINE interrupt. This triggers an interrupt at the time when VBLANK begins. VLINE uses separate registers for enabling and clearing

[PATCH v4 5/7] drm/mgag200: Add dedicted variable for field

2024-07-05 Thread Thomas Zimmermann
Replace the line-compare value of with a dedicated variable . Improves readability and prepares for vblank support. Signed-off-by: Thomas Zimmermann Acked-by: Sam Ravnborg --- drivers/gpu/drm/mgag200/mgag200_mode.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a

[PATCH v4 2/7] drm/mgag200: Align register field names with documentation

2024-07-05 Thread Thomas Zimmermann
In mgag200_set_mode_regs(), align variable names with the field names given in the Matrox programming manuals. Makes the code and docs grep- able. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/mgag200/mgag200_mode.c | 56 +- 1 file changed, 28 insertions(+), 28 del

[PATCH v4 4/7] drm/mgag200: Add dedicated variables for blanking fields

2024-07-05 Thread Thomas Zimmermann
Represent fields for horizontal and vertical blanking with , , and . Aligns the code with the Matrox programming manuals. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/mgag200/mgag200_mode.c | 29 ++ 1 file changed, 16 insertions(+), 13 deletions(-) diff --git a/

[PATCH v4 3/7] drm/mgag200: Use adjusted mode values for CRTCs

2024-07-05 Thread Thomas Zimmermann
Use the values with the crtc_ prefix from struct drm_display_mode to program hardware. The DRM core adjusted these values to the requirements of CRTC hardware. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/mgag200/mgag200_mode.c | 16 1 file changed, 8 insertions(+), 8 de

[PATCH v4 1/7] drm/mgag200: Use hexadecimal register indeces

2024-07-05 Thread Thomas Zimmermann
In mgag200_set_mode_regs(), use hexadecimal indices for accessing registers. Aligns the code with the register indices in the Matrox programming manuals. Also convert to lower-case hexadecimal values. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/mgag200/mgag200_mode.c | 50 +-

[PATCH v4 0/7] drm/mgag200: Implement VBLANK support

2024-07-05 Thread Thomas Zimmermann
(was: Rate-limit shadow-FB-to-console-update to screen refresh) Implement support for VBLANK events in mgag200. Patches 1 to 5 prepare mgag200's modesetting code by renaming or adding variables for various hardware fields. This makes the code more readable and aligns it with the programming manua

Re: [PATCH 5/6] drm/sti: hdmi: drop driver owner assignment

2024-07-05 Thread Krzysztof Kozlowski
On 05/07/2024 13:14, Maxime Ripard wrote: > +Alain > > On Tue, Jun 04, 2024 at 03:18:55PM GMT, Krzysztof Kozlowski wrote: >> On 27/05/2024 16:47, Alain Volmat wrote: >>> Hi Krzysztof, >>> >>> thanks for your patch, sorry for the delay. >>> ... >>> >>> Acked-by: Alain Volmat >> >> What does thi

Re: [PATCH] MAINTAINERS: drm/sti: mark it as Odd Fixes

2024-07-05 Thread Krzysztof Kozlowski
On 05/07/2024 13:22, Maxime Ripard wrote: > On Fri, Jul 05, 2024 at 01:19:50PM GMT, Daniel Vetter wrote: >> On Fri, Jul 05, 2024 at 12:03:56PM +0200, Krzysztof Kozlowski wrote: >>> Patches to STI DRM are not being picked up, so even though there is >>> maintainer activity, it seems that these drive

Re: [PATCH v6] drm/display: split DSC helpers from DP helpers

2024-07-05 Thread Maxime Ripard
On Thu, 4 Jul 2024 22:17:08 +0300, Dmitry Baryshkov wrote: > Currently the DRM DSC functions are selected by the > DRM_DISPLAY_DP_HELPER Kconfig symbol. This is not optimal, since the DSI > code (both panel and host drivers) end up selecting the seemingly > irrelevant DP helpers. Split the DSC code

Re: [PATCH] MAINTAINERS: drm/sti: mark it as Odd Fixes

2024-07-05 Thread Maxime Ripard
On Fri, Jul 05, 2024 at 01:19:50PM GMT, Daniel Vetter wrote: > On Fri, Jul 05, 2024 at 12:03:56PM +0200, Krzysztof Kozlowski wrote: > > Patches to STI DRM are not being picked up, so even though there is > > maintainer activity, it seems that these drivers are not being actively > > looked at. Ref

Re: [PATCH 1/4] drm/sti: dvo: drop driver owner assignment

2024-07-05 Thread Maxime Ripard
On Sat, 30 Mar 2024 21:38:28 +0100, Krzysztof Kozlowski wrote: > Core in platform_driver_register() already sets the .owner, so driver > does not need to. Whatever is set here will be anyway overwritten by > main driver calling platform_driver_register(). > > Applied to misc/kernel.git (drm-mis

Re: [PATCH] MAINTAINERS: drm/sti: mark it as Odd Fixes

2024-07-05 Thread Daniel Vetter
On Fri, Jul 05, 2024 at 12:03:56PM +0200, Krzysztof Kozlowski wrote: > Patches to STI DRM are not being picked up, so even though there is > maintainer activity, it seems that these drivers are not being actively > looked at. Reflect this in maintainer status. Note that since the driver is in drm

Re: [PATCH 5/6] drm/sti: hdmi: drop driver owner assignment

2024-07-05 Thread Maxime Ripard
+Alain On Tue, Jun 04, 2024 at 03:18:55PM GMT, Krzysztof Kozlowski wrote: > On 27/05/2024 16:47, Alain Volmat wrote: > > Hi Krzysztof, > > > > thanks for your patch, sorry for the delay. > > > > On Sat, Mar 30, 2024 at 09:57:21PM +0100, Krzysztof Kozlowski wrote: > >> Core in platform_driver_reg

Re: [PATCH v3 9/9] drm/tests: Add test for drm_framebuffer_free()

2024-07-05 Thread kernel test robot
d test for drm_framebuffer_free() config: arc-randconfig-001-20240705 (https://download.01.org/0day-ci/archive/20240705/202407051851.5p18lnvw-...@intel.com/config) compiler: arceb-elf-gcc (GCC) 13.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240705/202407051851.5p18lnvw-...

Re: [PATCH 5/6] drm/sti: hdmi: drop driver owner assignment

2024-07-05 Thread Krzysztof Kozlowski
On 05/07/2024 12:37, Dmitry Baryshkov wrote: > Acked-by: Alain Volmat What does this ack mean? You are the maintainer, so what is supposed to happen now? If maintainer does not take patches, then the DRM STI looks orphaned. >>> >>> +Cc Neil, Dmitry, >>> >>> I pinged and pin

Re: [pull] amdgpu, radeon drm-fixes-6.10

2024-07-05 Thread Daniel Vetter
On Wed, Jul 03, 2024 at 02:47:23PM -0400, Alex Deucher wrote: > Hi Dave, Sima, > > Fixes for 6.10. > > The following changes since commit 22a40d14b572deb80c0648557f4bd502d7e83826: > > Linux 6.10-rc6 (2024-06-30 14:40:44 -0700) > > are available in the Git repository at: > > https://gitlab.

Re: [pull] drm/msm: drm-msm-next-2024-07-04 for v6.11

2024-07-05 Thread Daniel Vetter
On Thu, Jul 04, 2024 at 08:48:18AM -0700, Rob Clark wrote: > Hi Dave, Sima, > > This is the main pull for v6.11. It includes a merge of the immutable > tag qcom/20240430-a750-raytracing-v3-2-7f57c5ac0...@gmail.com to pick > up dependencies for raytracing and SMEM speedbin. > > Further descriptio

Re: [PULL] drm-misc-next

2024-07-05 Thread Daniel Vetter
On Thu, Jul 04, 2024 at 03:17:09PM +0200, Maxime Ripard wrote: > Hi, > > Here's this week drm-misc-next PR, and the last PR for the 6.11 release cycle. > > Thanks! > Maxime > > drm-misc-next-2024-07-04: > drm-misc-next for $kernel-version: > > UAPI Changes: > > Cross-subsystem Changes: > > Co

Re: [PATCH 5/6] drm/sti: hdmi: drop driver owner assignment

2024-07-05 Thread Dmitry Baryshkov
On Fri, 5 Jul 2024 at 13:04, Neil Armstrong wrote: > > On 05/07/2024 11:59, Krzysztof Kozlowski wrote: > > On 04/06/2024 15:18, Krzysztof Kozlowski wrote: > >> On 27/05/2024 16:47, Alain Volmat wrote: > >>> Hi Krzysztof, > >>> > >>> thanks for your patch, sorry for the delay. > >>> > >>> On Sat, M

Re: Time for drm-ci-next?

2024-07-05 Thread Daniel Vetter
On Thu, Jul 04, 2024 at 08:40:26AM -0700, Rob Clark wrote: > On Thu, Jul 4, 2024 at 7:08 AM Daniel Vetter wrote: > > > > On Tue, Jul 02, 2024 at 05:32:39AM -0700, Rob Clark wrote: > > > On Fri, Jun 28, 2024 at 10:44 AM Daniel Vetter wrote: > > > > > > > > On Thu, Jun 27, 2024 at 11:51:37AM -0700,

Re: [PATCH v2] backlight: l4f00242t03: Add check for spi_setup

2024-07-05 Thread Daniel Thompson
On Fri, Jul 05, 2024 at 05:28:00PM +0800, Chen Ni wrote: > Add check for the return value of spi_setup() and return the error > if it fails in order to catch the error. > > Signed-off-by: Chen Ni Reviewed-by: Daniel Thompson Daniel.

Re: [PATCH 09/11] usb: dwc2: Skip clock gating on Broadcom SoCs

2024-07-05 Thread Stefan Wahren
Am 05.07.24 um 10:48 schrieb Lukas Wunner: On Thu, Jul 04, 2024 at 03:14:50PM +0100, Florian Fainelli wrote: On 6/30/2024 4:36 PM, Stefan Wahren wrote: On resume of the Raspberry Pi the dwc2 driver fails to enable HCD_FLAG_HW_ACCESSIBLE before re-enabling the interrupts. This causes a situation

Re: [PULL] drm-intel-gt-next

2024-07-05 Thread Daniel Vetter
On Thu, Jul 04, 2024 at 07:31:54AM +, Tvrtko Ursulin wrote: > > Hi Dave, Sima, > > The final pull for 6.11 is quite small and only contains a handful of > fixes in areas such as stolen memory probing on ATS-M, GuC priority > handling, out of memory reporting noise downgrade and fence register

Re: [PATCH 5/6] drm/sti: hdmi: drop driver owner assignment

2024-07-05 Thread Neil Armstrong
On 05/07/2024 11:59, Krzysztof Kozlowski wrote: On 04/06/2024 15:18, Krzysztof Kozlowski wrote: On 27/05/2024 16:47, Alain Volmat wrote: Hi Krzysztof, thanks for your patch, sorry for the delay. On Sat, Mar 30, 2024 at 09:57:21PM +0100, Krzysztof Kozlowski wrote: Core in platform_driver_regi

[PATCH] MAINTAINERS: drm/sti: mark it as Odd Fixes

2024-07-05 Thread Krzysztof Kozlowski
Patches to STI DRM are not being picked up, so even though there is maintainer activity, it seems that these drivers are not being actively looked at. Reflect this in maintainer status. Link: https://lore.kernel.org/all/77b4e4ad-2b1e-4b6d-bc3b-0c7b339bc...@linaro.org/ Link: https://lore.kernel.

Re: [pull] amdgpu, radeon drm-next-6.11

2024-07-05 Thread Daniel Vetter
On Wed, Jul 03, 2024 at 05:13:13PM -0400, Alex Deucher wrote: > Hi Dave, Sima, > > More new stuff for 6.11. There will be a few additional patches next > week for new IPs that were added in this cycle just to get them tied off, > but this should be it for general changes. > > The following chang

Re: [pull] amdgpu, amdkfd drm-next-6.11

2024-07-05 Thread Daniel Vetter
On Fri, Jun 28, 2024 at 05:31:35PM -0400, Alex Deucher wrote: > Hi Dave, Sima, > > More stuff for 6.11. > > The following changes since commit a78313bb206e0c456a989f380c4cbd8af8af7c76: > > Merge tag 'drm-intel-gt-next-2024-06-12' of > https://gitlab.freedesktop.org/drm/i915/kernel into drm-ne

Re: [PATCH 5/6] drm/sti: hdmi: drop driver owner assignment

2024-07-05 Thread Krzysztof Kozlowski
On 04/06/2024 15:18, Krzysztof Kozlowski wrote: > On 27/05/2024 16:47, Alain Volmat wrote: >> Hi Krzysztof, >> >> thanks for your patch, sorry for the delay. >> >> On Sat, Mar 30, 2024 at 09:57:21PM +0100, Krzysztof Kozlowski wrote: >>> Core in platform_driver_register() already sets the .owner, so

Re: [PATCH 2/5] drm/mediatek: Support "None" blending in Mixer

2024-07-05 Thread 胡俊光

[PATCH v2] backlight: l4f00242t03: Add check for spi_setup

2024-07-05 Thread Chen Ni
Add check for the return value of spi_setup() and return the error if it fails in order to catch the error. Signed-off-by: Chen Ni --- Changelog: v1 -> v2: 1. Use dev_err_probe() to match other error paths --- drivers/video/backlight/l4f00242t03.c | 5 - 1 file changed, 4 insertions(+), 1

Re: [GIT PULL] mediatek drm next for 6.11

2024-07-05 Thread Daniel Vetter
On Fri, Jun 28, 2024 at 01:46:32PM +, Chun-Kuang Hu wrote: > Hi, Dave & Daniel: > > This includes: > > 1. Convert to platform remove callback returning void > 2. Drop chain_mode_fixup call in mode_valid() > 3. Fixes the errors of MediaTek display driver found by IGT. > 4. Add display support

Re: [PATCH 2/5] drm/mediatek: Support "None" blending in Mixer

2024-07-05 Thread 胡俊光

Re: [GIT PULL] etnaviv-next for 6.11

2024-07-05 Thread Daniel Vetter
On Fri, Jun 28, 2024 at 12:39:33PM +0200, Lucas Stach wrote: > Hi Dave, hi Sima, > > please pull the following changes for the next merge window. Mostly > fixes, but as they concern new hardware support there's no need to rush > them into the current -rc. > > Regards, > Lucas > > The following c

Re: [PATCH v5 2/3] dt-bindings: arm: mediatek: mmsys: Add OF graph support for board path

2024-07-05 Thread 胡俊光

Re: [GIT PULL] exynos-drm-next

2024-07-05 Thread Daniel Vetter
On Wed, Jul 03, 2024 at 04:59:12PM +0900, Inki Dae wrote: > Hi Dave and Daniel, > >Just three cleanup patches to Exynos Virtual Display driver >for improved management. > > Please kindly let me know if there is any problem. > > Thanks, > Inki Dae > > > The following changes since co

Re: [PATCH v2] drm/msm: Fix incorrect file name output in adreno_request_fw()

2024-07-05 Thread Dmitry Baryshkov
On Fri, 5 Jul 2024 at 12:15, Aleksandr Mishin wrote: > > In adreno_request_fw() when debugging information is printed to the log > after firmware load, an incorrect filename is printed. 'newname' is used > instead of 'fwname', so prefix "qcom/" is being added to filename. > Looks like "copy-paste"

[PATCH v2] drm/msm: Fix incorrect file name output in adreno_request_fw()

2024-07-05 Thread Aleksandr Mishin
In adreno_request_fw() when debugging information is printed to the log after firmware load, an incorrect filename is printed. 'newname' is used instead of 'fwname', so prefix "qcom/" is being added to filename. Looks like "copy-paste" mistake. Fix this mistake by replacing 'newname' with 'fwname'

[PATCH 5/5] drm/i915/display: Load the lut values and enable sharpness

2024-07-05 Thread Nemesa Garg
Load the lut values during pipe enable. v2: Add the display version check Signed-off-by: Nemesa Garg --- drivers/gpu/drm/i915/display/intel_crtc.c| 3 +++ drivers/gpu/drm/i915/display/intel_display.c | 6 ++ drivers/gpu/drm/i915/display/skl_scaler.c| 13 - 3 files chan

[PATCH 2/5] drm/i915/display: Compute the scaler filter coefficients

2024-07-05 Thread Nemesa Garg
The sharpness property requires the use of one of the scaler so need to set the sharpness scaler coefficient values. These values are based on experiments and vary for different tap value/win size. These values are normalized by taking the sum of all values and then dividing each value with a sum.

[PATCH 4/5] drm/i915/display: Add registers and compute the strength

2024-07-05 Thread Nemesa Garg
Add new registers and related bits. Compute the strength value and tap value based on display mode. Signed-off-by: Nemesa Garg --- drivers/gpu/drm/i915/display/intel_display.c | 5 +- .../drm/i915/display/intel_display_types.h| 1 + .../drm/i915/display/intel_sharpen_filter.c | 105 ++

[PATCH 3/5] drm/i915/display: Enable the second scaler for sharpness

2024-07-05 Thread Nemesa Garg
As only second scaler can be used for sharpness check if it is available and if panel fitting is also not enabled, the set the sharpness. Panel fitting will have the preference over sharpness property. v2: Added the panel fitting check before enabling sharpness Signed-off-by: Nemesa Garg --- dr

  1   2   >