Re: [PATCH v2 13/17] drm/v3d: Create a CPU job extension for the timestamp query job

2023-11-27 Thread Iago Toral
El jue, 23-11-2023 a las 21:47 -0300, Maíra Canal escribió: (...) > +static void > +v3d_timestamp_query(struct v3d_cpu_job *job) > +{ > +   struct v3d_timestamp_query_info *timestamp_query = > >timestamp_query; > +   struct v3d_bo *bo = to_v3d_bo(job->base.bo[0]); I presume there is an

Re: [PATCH v3 0/2] Add DSS support for TI AM62A7 SoC

2023-11-27 Thread Tomi Valkeinen
Hi, On 08/11/2023 19:16, Aradhya Bhatia wrote: This patch series adds a new compatible for the Display SubSystem (DSS) controller on TI's AM62A7 SoC. It further adds the required support, for the same, in the tidss driver. The DSS controller is similar to the recently added AM625 DSS, with the

Re: [PATCH RFC v3 22/37] drm/rockchip: inno_hdmi: Remove useless output format

2023-11-27 Thread Maxime Ripard
Hi, On Sat, Nov 25, 2023 at 11:00:52AM +0100, Johan Jonker wrote: > In stead of further cripplingRockchip HDMI drivers one could also make > it functional based  on EDID info. I'm not crippling it, it was dead code. This has no functional impact whatsoever. > To start with the output could you

Re: [PATCH v3] drm/mediatek: Stop using iommu_present()

2023-11-27 Thread AngeloGioacchino Del Regno
Il 23/11/23 14:41, Robin Murphy ha scritto: Remove the pointless check. If an IOMMU is providing transparent DMA API ops for any device(s) we care about, the DT code will have enforced the appropriate probe ordering already. And if the IOMMU *is* entirely absent, then attempting to go ahead with

Re: [PATCH v3 101/108] drm/bridge: ti-sn65dsi86: Make use of devm_pwmchip_alloc() function

2023-11-27 Thread Uwe Kleine-König
Hello, On Thu, Nov 23, 2023 at 10:17:15AM +0100, Uwe Kleine-König wrote: > On Tue, Nov 21, 2023 at 08:14:14AM -0800, Doug Anderson wrote: > > On Tue, Nov 21, 2023 at 8:05 AM Uwe Kleine-König > > wrote: > > > On Tue, Nov 21, 2023 at 07:15:51AM -0800, Doug Anderson wrote: > > > > > @@ -1585,22

Re: (subset) [PATCH v9 00/12] drm/meson: add support for MIPI DSI Display

2023-11-27 Thread Neil Armstrong
Hi, On Fri, 24 Nov 2023 09:41:11 +0100, Neil Armstrong wrote: > The Amlogic G12A, G12B & SM1 SoCs embeds a Synopsys DW-MIPI-DSI transceiver > (ver 1.21a), > with a custom glue managing the IP resets, clock and data input similar to > the DW-HDMI > glue on the same Amlogic SoCs. > > This is a

Re: [PATCH v6 2/2] drm/i915/guc: Close deregister-context race against CT-loss

2023-11-27 Thread Rodrigo Vivi
On Tue, Nov 14, 2023 at 07:52:29AM -0800, Alan Previn wrote: > If we are at the end of suspend or very early in resume > its possible an async fence signal (via rcu_call) is triggered > to free_engines which could lead us to the execution of > the context destruction worker (after a prior worker

[PATCH] drm/msm/mdp4: flush vblank event on disable

2023-11-27 Thread Dmitry Baryshkov
Flush queued events when disabling the crtc. This avoids timeouts when we come back and wait for dependencies (like the previous frame's flip_done). Fixes: c8afe684c95c ("drm/msm: basic KMS driver for snapdragon") Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/mdp4/mdp4_crtc.c | 9

Re: [PATCH v2 0/2] drm/bridge: panel: Check device dependency before managing device link

2023-11-27 Thread Linus Walleij
On Mon, Nov 27, 2023 at 5:29 PM Maxime Ripard wrote: > On Mon, Nov 27, 2023 at 05:03:53PM +0100, Linus Walleij wrote: > > > Liu Ying (2): > > > driver core: Export device_is_dependent() to modules > > > drm/bridge: panel: Check device dependency before managing device link > > > > I just

Re: [PATCH v2 1/2] driver core: Export device_is_dependent() to modules

2023-11-27 Thread Linus Walleij
On Mon, Nov 27, 2023 at 7:20 PM Greg KH wrote: [Maxime] > > So, a committer just applied this to drm-misc-fixes without your > > approval. That was me, mea culpa. I learned a lesson. Or two. > Wait, why exactly is this needed? Nothing outside of the driver core > should be needing this

[PATCH 2/3] Revert "drm/bridge: panel: Check device dependency before managing device link"

2023-11-27 Thread Linus Walleij
This reverts commit 39d5b6a64ace77d0c11c398d272218df5f939abb. This patch was causing build errors by using an unexported function from the device core, which Greg questions the saneness in exporting. Link:

[PATCH 1/3] Revert "driver core: Export device_is_dependent() to modules"

2023-11-27 Thread Linus Walleij
This reverts commit 1d5e8f4bf06da86b71cc9169110d1a0e1e7af337. Greg says: "why exactly is this needed? Nothing outside of the driver core should be needing this function, it shouldn't be public at all (I missed that before.) So please, revert it for now, let's figure out why DRM thinks this is

[PATCH 0/3] Revert panel fixes and original buggy patch

2023-11-27 Thread Linus Walleij
1 - drivers/gpu/drm/bridge/panel.c | 26 -- 2 files changed, 27 deletions(-) --- base-commit: 95ba893c9f4feb836ddce627efd0bb6af6667031 change-id: 20231127-revert-panel-fix-e4da3e11e7a4 Best regards, -- Linus Walleij

[PATCH 3/3] Revert "drm/bridge: panel: Add a device link between drm device and panel device"

2023-11-27 Thread Linus Walleij
This reverts commit 199cf07ebd2b0d41185ac79b895547d45610b681. This patch creates bugs on devices where the DRM device is the ancestor of the panel devices. Attempts to fix this have failed because it leads to using device core functionality which is questionable. Reported-by: Linus Walleij

Re: [Nouveau] [PATCH 3/3] nouveau/gsp: add some basic registry entries.

2023-11-27 Thread Timur Tabi
On Tue, Oct 31, 2023 at 12:20 AM Dave Airlie wrote: > rpc->size = sizeof(*rpc); > - rpc->numEntries = 1; > - rpc->entries[0].nameOffset = offsetof(typeof(*rpc), entries[1]); > - rpc->entries[0].type = 1; > - rpc->entries[0].data = 0; > -

Re: [PATCH] drm/amd/pm: fix a memleak in aldebaran_tables_init

2023-11-27 Thread Alex Deucher
Applied. Thanks! On Thu, Nov 23, 2023 at 3:08 AM Dinghao Liu wrote: > > When kzalloc() for smu_table->ecc_table fails, we should free > the previously allocated resources to prevent memleak. > > Fixes: edd794208555 ("drm/amd/pm: add message smu to get ecc_table v2") > Signed-off-by: Dinghao Liu

Re: [PATCH 02/14] drm: Fix TODO list mentioning non-KMS drivers

2023-11-27 Thread Alex Deucher
On Wed, Nov 22, 2023 at 7:25 AM Thomas Zimmermann wrote: > > Non-KMS drivers have been removed from DRM. Update the TODO list > accordingly. > > Signed-off-by: Thomas Zimmermann > Fixes: a276afc19eec ("drm: Remove some obsolete drm pciids(tdfx, mga, i810, > savage, r128, sis, via)") > Cc: Cai

Re: [PATCH 01/14] arch/powerpc: Remove legacy DRM drivers from default configs

2023-11-27 Thread Alex Deucher
On Wed, Nov 22, 2023 at 7:25 AM Thomas Zimmermann wrote: > > DRM drivers for user-space modesetting have been removed. Do not > select the respective options in the default configs. > > Signed-off-by: Thomas Zimmermann > Fixes: a276afc19eec ("drm: Remove some obsolete drm pciids(tdfx, mga, i810,

Re: [PATCH 03/14] drm: Include

2023-11-27 Thread Alex Deucher
On Wed, Nov 22, 2023 at 7:25 AM Thomas Zimmermann wrote: > > One of the source files includes via , > which will be removed. Include drm_auth.h directly. > > Signed-off-by: Thomas Zimmermann Acked-by: Alex Deucher > --- > drivers/gpu/drm/drm_pci.c | 1 + > 1 file changed, 1 insertion(+) > >

Re: [PATCH 08/14] drm: Remove entry points for legacy ioctls

2023-11-27 Thread Alex Deucher
On Wed, Nov 22, 2023 at 7:25 AM Thomas Zimmermann wrote: > > DRM drivers with user-space mode setting have been removed in Linux > v6.3. [1] Now remove the ioctl entry points for these drivers. Invoking > any of the ioctl ops will unconditionally return -EINVAL to user space. > This has already

Re: [PATCH 05/14] accel: Include

2023-11-27 Thread Alex Deucher
On Wed, Nov 22, 2023 at 7:25 AM Thomas Zimmermann wrote: > > One of the source files includes via , > which will be removed. Include drm_auth.h directly. > > Signed-off-by: Thomas Zimmermann > Cc: Oded Gabbay Acked-by: Alex Deucher > --- > drivers/accel/drm_accel.c | 1 + > 1 file changed,

Re: [PATCH 14/14] drm: Remove Kconfig option for legacy support (CONFIG_DRM_LEGACY)

2023-11-27 Thread Alex Deucher
On Wed, Nov 22, 2023 at 7:25 AM Thomas Zimmermann wrote: > > Remove CONFIG_DRM_LEGACY from Kconfig. Nothing depends on the option. > > Signed-off-by: Thomas Zimmermann Acked-by: Alex Deucher > --- > drivers/gpu/drm/Kconfig | 21 - > 1 file changed, 21 deletions(-) > >

[PATCH] drm: lcdif: Switch to drmm_mode_config_init

2023-11-27 Thread Marek Vasut
Switch from deprecated unmanaged drm_mode_config_init() to managed drmm_mode_config_init(). No functional change. Signed-off-by: Marek Vasut --- Cc: Daniel Vetter Cc: David Airlie Cc: Fabio Estevam Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: NXP Linux Team Cc: Pengutronix Kernel Team Cc:

Re: [PATCH] drm/msm/mdp4: flush vblank event on disable

2023-11-27 Thread Abhinav Kumar
On 11/27/2023 1:54 PM, Dmitry Baryshkov wrote: Flush queued events when disabling the crtc. This avoids timeouts when we come back and wait for dependencies (like the previous frame's flip_done). Fixes: c8afe684c95c ("drm/msm: basic KMS driver for snapdragon") Signed-off-by: Dmitry Baryshkov

Re: [PATCH] drm/msm/mdp4: flush vblank event on disable

2023-11-27 Thread Dmitry Baryshkov
On Tue, 28 Nov 2023 at 00:00, Abhinav Kumar wrote: > > > > On 11/27/2023 1:54 PM, Dmitry Baryshkov wrote: > > Flush queued events when disabling the crtc. This avoids timeouts when > > we come back and wait for dependencies (like the previous frame's > > flip_done). > > > > Fixes: c8afe684c95c

Re: [PATCH 0/3] Revert panel fixes and original buggy patch

2023-11-27 Thread Linus Walleij
On Tue, Nov 28, 2023 at 12:25 AM Laurent Pinchart wrote: > On Tue, Nov 28, 2023 at 12:10:18AM +0100, Linus Walleij wrote: > > This series reverts the attempts to fix the bug that went > > into v6.7-rc1 in commit 199cf07ebd2b > > "drm/bridge: panel: Add a device link between drm device and panel

Re: [PATCH v2] Remove custom dumb_map_offset implementation in msm driver

2023-11-27 Thread Rob Clark
On Tue, Nov 21, 2023 at 5:14 AM Dmitry Baryshkov wrote: > > On Tue, 21 Nov 2023 at 04:26, Rob Clark wrote: > > > > On Wed, Nov 15, 2023 at 11:33 AM Dmitry Baryshkov > > wrote: > > > > > > On Wed, 15 Nov 2023 at 20:46, Dipam Turkar wrote: > > > > > > > > They are not outdated, my bad. I went

Re: [PATCH 07/14] drm/radeon: Do not include

2023-11-27 Thread Alex Deucher
On Wed, Nov 22, 2023 at 7:25 AM Thomas Zimmermann wrote: > > Including is not required by radeon. > > Signed-off-by: Thomas Zimmermann > Cc: Alex Deucher > Cc: "Christian König" > Cc: "Pan, Xinhui" > Cc: amd-...@lists.freedesktop.org Acked-by: Alex Deucher > --- >

Re: [PATCH 10/14] drm: Remove support for legacy drivers

2023-11-27 Thread Alex Deucher
On Wed, Nov 22, 2023 at 7:25 AM Thomas Zimmermann wrote: > > Remove all hooks and calls into code for user-space mode setting from > the DRM core. Without the drivers and ioctl entry points, none of this > is required any longer. > > Signed-off-by: Thomas Zimmermann Acked-by: Alex Deucher >

Re: [PATCH 09/14] drm: Remove the legacy DRM_IOCTL_MODESET_CTL ioctl

2023-11-27 Thread Alex Deucher
On Wed, Nov 22, 2023 at 7:25 AM Thomas Zimmermann wrote: > > DRM drivers with user-space mode setting have been removed in Linux > v6.3. [1] Now remove the ioctl entry points for these drivers. Invoking > any of the ioctl ops will unconditionally return -EINVAL to user space. > > Invoking

Re: [PATCH 11/14] drm: Remove locking for legacy ioctls and DRM_UNLOCKED

2023-11-27 Thread Alex Deucher
On Wed, Nov 22, 2023 at 7:25 AM Thomas Zimmermann wrote: > > Modern DRM drivers acquire ioctl locks by themselves. Legacy ioctls > for user-space mode setting used to acquire drm_global_mutex. After > removing the ioctl entry points, also remove the locking code. The only > legacy ioctl without

[PATCH v2] drm/mxsfb: Switch to drmm_mode_config_init

2023-11-27 Thread Marek Vasut
Switch from deprecated unmanaged drm_mode_config_init() to managed drmm_mode_config_init(). No functional change. Signed-off-by: Marek Vasut --- Cc: Daniel Vetter Cc: David Airlie Cc: Fabio Estevam Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: NXP Linux Team Cc: Pengutronix Kernel Team Cc:

Re: [PATCH v2] drm/amdgpu: Fix cat debugfs amdgpu_regs_didt causes kernel null pointer

2023-11-27 Thread Alex Deucher
Applied. Thanks! Alex On Thu, Nov 23, 2023 at 3:22 AM Christian König wrote: > > Am 23.11.23 um 02:22 schrieb Lu Yao: > > For 'AMDGPU_FAMILY_SI' family cards, in 'si_common_early_init' func, init > > 'didt_rreg' and 'didt_wreg' to 'NULL'. But in func > > 'amdgpu_debugfs_regs_didt_read/write',

Re: [PATCH 13/14] char/agp: Remove frontend code

2023-11-27 Thread Alex Deucher
On Wed, Nov 22, 2023 at 7:25 AM Thomas Zimmermann wrote: > > The AGP subsystem supports a user-space interface via /dev/agpgart. It > is only enabled with DRM support for mode setting in user space. (i.e., > CONFIG_DRM_LEGACY). All of that DRM code has been removed and the option > will go away.

Re: [PATCH v2] Remove custom dumb_map_offset implementation in msm driver

2023-11-27 Thread Dmitry Baryshkov
On Mon, 27 Nov 2023 at 22:52, Rob Clark wrote: > > On Tue, Nov 21, 2023 at 5:14 AM Dmitry Baryshkov > wrote: > > > > On Tue, 21 Nov 2023 at 04:26, Rob Clark wrote: > > > > > > On Wed, Nov 15, 2023 at 11:33 AM Dmitry Baryshkov > > > wrote: > > > > > > > > On Wed, 15 Nov 2023 at 20:46, Dipam

Re: [PATCH 12/14] drm: Remove source code for non-KMS drivers

2023-11-27 Thread Alex Deucher
On Wed, Nov 22, 2023 at 7:25 AM Thomas Zimmermann wrote: > > Remove all remaining source code for non-KMS drivers. These drivers > have been removed in v6.3 and won't comeback. > > Signed-off-by: Thomas Zimmermann Acked-by: Alex Deucher

Re: [Nouveau] [PATCH 3/3] nouveau/gsp: add some basic registry entries.

2023-11-27 Thread Dave Airlie
On Tue, 28 Nov 2023 at 06:48, Timur Tabi wrote: > > On Tue, Oct 31, 2023 at 12:20 AM Dave Airlie wrote: > > rpc->size = sizeof(*rpc); > > - rpc->numEntries = 1; > > - rpc->entries[0].nameOffset = offsetof(typeof(*rpc), entries[1]); > > - rpc->entries[0].type = 1; > > -

Re: [PATCH 0/3] Revert panel fixes and original buggy patch

2023-11-27 Thread Laurent Pinchart
, 27 deletions(-) > --- > base-commit: 95ba893c9f4feb836ddce627efd0bb6af6667031 > change-id: 20231127-revert-panel-fix-e4da3e11e7a4 -- Regards, Laurent Pinchart

Re: Radeon regression in 6.6 kernel

2023-11-27 Thread Phillip Susi
Alex Deucher writes: >> In that case those are the already known problems with the scheduler >> changes, aren't they? > > Yes. Those changes went into 6.7 though, not 6.6 AFAIK. Maybe I'm > misunderstanding what the original report was actually testing. If it > was 6.7, then try reverting: >

Re: [PATCH 0/3] Revert panel fixes and original buggy patch

2023-11-27 Thread Laurent Pinchart
On Tue, Nov 28, 2023 at 12:36:15AM +0100, Linus Walleij wrote: > On Tue, Nov 28, 2023 at 12:25 AM Laurent Pinchart wrote: > > On Tue, Nov 28, 2023 at 12:10:18AM +0100, Linus Walleij wrote: > > > This series reverts the attempts to fix the bug that went > > > into v6.7-rc1 in commit 199cf07ebd2b >

Re: [PATCH] drm/bridge: ti-sn65dsi86: Associate PWM device to auxiliary device

2023-11-27 Thread Doug Anderson
Hi, On Mon, Nov 27, 2023 at 2:15 AM Uwe Kleine-König wrote: > > It's the ti_sn65dsi86.pwm auxiliary driver that creates the pwmchip, so > let the auxiliary device be the parent of the pwm device. > > Note that getting a reference to the ti-sn65dsi86's pwm using pwm_get() > isn't affected by this

[PATCH] drm/msm/dpu: Capture dpu snapshot when frame_done_timer timeouts

2023-11-27 Thread Paloma Arellano
Trigger a devcoredump to dump dpu registers and capture the drm atomic state when the frame_done_timer timeouts. Signed-off-by: Paloma Arellano --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git

Re: [PATCH drm-misc-next 4/5] drm/gpuvm: fall back to drm_exec_lock_obj()

2023-11-27 Thread Danilo Krummrich
Hi, On 11/27/23 14:52, Christian König wrote: Am 25.11.23 um 00:40 schrieb Danilo Krummrich: Hi Christian, do you think it makes sense to handle this in drm_exec_prepare_obj() or even dma_resv_reserve_fences() even? IIRC for drm_exec the discussion has gone a bit back and forth between

[PATCH v3 05/17] drm/v3d: Don't allow two multisync extensions in the same job

2023-11-27 Thread Maíra Canal
Currently, two multisync extensions can be added to the same job and only the last multisync extension will be used. To avoid this vulnerability, don't allow two multisync extensions in the same job. Signed-off-by: Maíra Canal --- drivers/gpu/drm/v3d/v3d_submit.c | 5 + 1 file changed, 5

[PATCH v3 03/17] drm/v3d: Detach job submissions IOCTLs to a new specific file

2023-11-27 Thread Maíra Canal
From: Melissa Wen We will include a new job submission type, the CPU job submission. For readability and maintability, separate the job submission IOCTLs and related operations from v3d_gem.c. Minor fix in the CSD submission kernel doc: CSD (texture formatting) -> CSD (compute shader).

[PATCH v3 00/17] drm/v3d: Introduce CPU jobs

2023-11-27 Thread Maíra Canal
This patchset implements the basic infrastructure for a new type of V3D job, a CPU job. A CPU job is a job that requires CPU intervention. It would be nice to perform this operations on the kernel space as we can attach multiple in/out syncobjs to it. Why we want a CPU job on the kernel?

Re: [PATCH 2/2] drm/amdgpu: use GTT only as fallback for VRAM|GTT

2023-11-27 Thread Christian König
Am 27.11.23 um 17:47 schrieb Bhardwaj, Rajneesh: [AMD Official Use Only - General] -Original Message- From: amd-gfx On Behalf Of Hamza Mahfooz Sent: Monday, November 27, 2023 10:53 AM To: Christian König ; jani.nik...@linux.intel.com; kher...@redhat.com; d...@redhat.com;

[PATCH v3 01/17] drm/v3d: Remove unused function header

2023-11-27 Thread Maíra Canal
From: Melissa Wen v3d_mmu_get_offset header was added but the function was never defined. Just remove it. Signed-off-by: Melissa Wen Signed-off-by: Maíra Canal --- drivers/gpu/drm/v3d/v3d_drv.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/v3d/v3d_drv.h

[PATCH v3 02/17] drm/v3d: Move wait BO ioctl to the v3d_bo file

2023-11-27 Thread Maíra Canal
From: Melissa Wen IOCTLs related to BO operations reside on the file v3d_bo.c. The wait BO ioctl is the only IOCTL regarding BOs that is placed in a different file. So, move it to the v3d_bo.c file. Signed-off-by: Melissa Wen Signed-off-by: Maíra Canal --- drivers/gpu/drm/v3d/v3d_bo.c | 33

[PATCH v3 04/17] drm/v3d: Simplify job refcount handling

2023-11-27 Thread Maíra Canal
From: Melissa Wen Instead of checking if the job is NULL every time we call the function, check it inside the function. Signed-off-by: Melissa Wen Signed-off-by: Maíra Canal --- drivers/gpu/drm/v3d/v3d_submit.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git

[PATCH v3 06/17] drm/v3d: Decouple job allocation from job initiation

2023-11-27 Thread Maíra Canal
We want to allow the IOCTLs to allocate the job without initiating it. This will be useful for the CPU job submission IOCTL, as the CPU job has the need to use information from the user extensions. Currently, the user extensions are parsed before the job allocation, making it impossible to fill

[PATCH v3 07/17] drm/v3d: Add a CPU job submission

2023-11-27 Thread Maíra Canal
From: Melissa Wen Create a new type of job, a CPU job. A CPU job is a type of job that performs operations that requires CPU intervention. The overall idea is to use user extensions to enable different types of CPU job, allowing the CPU job to perform different operations according to the type

[PATCH v3 08/17] drm/v3d: Use v3d_get_extensions() to parse CPU job data

2023-11-27 Thread Maíra Canal
Currently, v3d_get_extensions() only parses multisync data and assigns it to the `struct v3d_submit_ext`. But, to implement the CPU job with user extensions, we want v3d_get_extensions() to be able to parse CPU job data and assign it to the `struct v3d_cpu_job`. Therefore, allow the function

[PATCH v3 12/17] drm/v3d: Create a CPU job extension for a indirect CSD job

2023-11-27 Thread Maíra Canal
A CPU job is a type of job that performs operations that requires CPU intervention. An indirect CSD job is a job that, when executed in the queue, will map the indirect buffer, read the dispatch parameters, and submit a regular dispatch. Therefore, it is a job that needs CPU intervention. So,

[PATCH v3 09/17] drm/v3d: Create tracepoints to track the CPU job

2023-11-27 Thread Maíra Canal
Create tracepoints to track the three major events of a CPU job lifetime: 1. Submission of a `v3d_submit_cpu` IOCTL 2. Beginning of the execution of a CPU job 3. Ending of the execution of a CPU job Signed-off-by: Maíra Canal --- drivers/gpu/drm/v3d/v3d_sched.c | 4 +++

[PATCH v3 11/17] drm/v3d: Enable BO mapping

2023-11-27 Thread Maíra Canal
For the indirect CSD CPU job, we will need to access the internal contents of the BO with the dispatch parameters. Therefore, create methods to allow the mapping and unmapping of the BO. Signed-off-by: Maíra Canal --- drivers/gpu/drm/v3d/v3d_bo.c | 18 ++

[PATCH v3 10/17] drm/v3d: Detach the CSD job BO setup

2023-11-27 Thread Maíra Canal
From: Melissa Wen Detach CSD job setup from CSD submission ioctl to reuse it in CPU submission ioctl for indirect CSD job. Signed-off-by: Melissa Wen Co-developed-by: Maíra Canal Signed-off-by: Maíra Canal --- drivers/gpu/drm/v3d/v3d_submit.c | 68 1 file

Re: [PATCH v5] Documentation/gpu: VM_BIND locking document

2023-11-27 Thread Danilo Krummrich
On 11/22/23 08:49, Thomas Hellström wrote: On 11/21/23 14:56, Boris Brezillon wrote: On Tue, 21 Nov 2023 11:40:46 +0100 Thomas Hellström wrote: Add the first version of the VM_BIND locking document which is intended to be part of the xe driver upstreaming agreement. The document describes

[PATCH v3 1/1] drm/i915/pxp: Add missing tag for Wa_14019159160

2023-11-27 Thread Alan Previn
Add missing tag for "Wa_14019159160 - Case 2" (for existing PXP code that ensures run alone mode bit is set to allow PxP-decryption. v3: - Check targeted platforms using IP_VAL. (John Harrison) v2: - Fix WA id number (John Harrison). - Improve comments and code to be specific for

Re: [Intel-gfx] [PATCH v1 1/1] drm/i915/gt: Dont wait forever when idling in suspend

2023-11-27 Thread Rodrigo Vivi
On Tue, Nov 14, 2023 at 08:22:27AM -0800, Alan Previn wrote: > When suspending, add a timeout when calling > intel_gt_pm_wait_for_idle else if we have a leaked > wakeref (which would be indicative of a bug elsewhere > in the driver), driver will at exit the suspend-resume > cycle, after the kernel

Re: [PATCH v2 1/2] driver core: Export device_is_dependent() to modules

2023-11-27 Thread Greg KH
On Mon, Nov 27, 2023 at 05:38:13PM +0100, Maxime Ripard wrote: > Greg, Rafael, > > On Mon, Nov 27, 2023 at 01:14:13PM +0800, Liu Ying wrote: > > Export device_is_dependent() since the drm_kms_helper module is starting > > to use it. > > > > Signed-off-by: Liu Ying > > --- > > v2: > > * Newly

Re: [PATCH 03/12] dt-bindings: display: msm: qcm2290-mdss: Allow 2 interconnects

2023-11-27 Thread Rob Herring
On Sat, 25 Nov 2023 15:17:31 +0100, Konrad Dybcio wrote: > In addition to MDP0, the cpu-cfg interconnect is also necessary. > Allow it. > > Signed-off-by: Konrad Dybcio > --- > Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml | 4 ++-- > 1 file changed, 2 insertions(+), 2

Re: [PATCH 04/12] dt-bindings: display: msm: sm8450-mdss: Allow 3 interconnects

2023-11-27 Thread Rob Herring
On Sat, 25 Nov 2023 15:17:32 +0100, Konrad Dybcio wrote: > In addition to MDP01, the cpu-cfg interconnect is also necessary. > Allow it. > > Signed-off-by: Konrad Dybcio > --- > Documentation/devicetree/bindings/display/msm/qcom,sm8450-mdss.yaml | 4 ++-- > 1 file changed, 2 insertions(+), 2

Re: [PATCH] Replace ioremap_cache() with memremap()

2023-11-27 Thread Nischala Yelchuri
Wei, this is one of the Hyper-V code improvement tasks that Michael Kelley identified. Using memremap() is the right thing to do here. Abhinav Singh (cc'ed) also tried to fix this earlier as there are sparse warnings with ioremap_cache(). On Sun, Nov 12, 2023 at 11:12:33PM +, Wei Liu wrote:

Re: [Nouveau] [PATCH] nouveau/gsp/r535: remove a stray unlock in r535_gsp_rpc_send()

2023-11-27 Thread Timur Tabi
On Mon, 2023-11-27 at 15:56 +0300, Dan Carpenter wrote: > This unlock doesn't belong here and it leads to a double unlock in > the caller, r535_gsp_rpc_push(). > > Fixes: 176fdcbddfd2 ("drm/nouveau/gsp/r535: add support for booting GSP- > RM") > Signed-off-by: Dan Carpenter Nice catch!

Re: [PATCH v2 1/1] drm/i915/pxp: Add missing tag for Wa_14019159160

2023-11-27 Thread Rodrigo Vivi
On Wed, Nov 22, 2023 at 12:30:03PM -0800, Alan Previn wrote: > Add missing tag for "Wa_14019159160 - Case 2" (for existing > PXP code that ensures run alone mode bit is set to allow > PxP-decryption. > > v2: - Fix WA id number (John Harrison). > - Improve comments and code to be specific >

Re: [PATCH] dt-bindings: display: mediatek: dsi: remove Xinlei's mail

2023-11-27 Thread Rob Herring
On Thu, 23 Nov 2023 14:49:27 +0100, Michael Walle wrote: > Xinlei Lee's mail is bouncing: > > : host mailgw02.mediatek.com[216.200.240.185] said: > 550 Relaying mail to xinlei@mediatek.com is not allowed (in reply to > RCPT TO command) > > Remove it. > > Signed-off-by: Michael

[PATCH v3 15/17] drm/v3d: Create a CPU job extension to copy timestamp query to a buffer

2023-11-27 Thread Maíra Canal
A CPU job is a type of job that performs operations that requires CPU intervention. A copy timestamp query job is a job that copy the complete or partial result of a query to a buffer. As V3D doesn't provide any mechanism to obtain a timestamp from the GPU, it is a job that needs CPU intervention.

[PATCH v3 16/17] drm/v3d: Create a CPU job extension for the reset performance query job

2023-11-27 Thread Maíra Canal
A CPU job is a type of job that performs operations that requires CPU intervention. A reset performance query job is a job that resets the performance queries by resetting the values of the perfmons. Moreover, we also reset the syncobjs related to the availability of the query. So, create a user

[PATCH v3 13/17] drm/v3d: Create a CPU job extension for the timestamp query job

2023-11-27 Thread Maíra Canal
A CPU job is a type of job that performs operations that requires CPU intervention. A timestamp query job is a job that calculates the query timestamp and updates the query availability by signaling a syncobj. As V3D doesn't provide any mechanism to obtain a timestamp from the GPU, it is a job

[PATCH v3 17/17] drm/v3d: Create a CPU job extension for the copy performance query job

2023-11-27 Thread Maíra Canal
A CPU job is a type of job that performs operations that requires CPU intervention. A copy performance query job is a job that copy the complete or partial result of a query to a buffer. In order to copy the result of a performance query to a buffer, we need to get the values from the performance

[PATCH v3 14/17] drm/v3d: Create a CPU job extension for the reset timestamp job

2023-11-27 Thread Maíra Canal
A CPU job is a type of job that performs operations that requires CPU intervention. A reset timestamp job is a job that resets the timestamp queries based on the value offset of the first query. As V3D doesn't provide any mechanism to obtain a timestamp from the GPU, it is a job that needs CPU

Re: [Intel-xe] [PATCH v5] Documentation/gpu: VM_BIND locking document

2023-11-27 Thread Rodrigo Vivi
On Tue, Nov 21, 2023 at 11:40:46AM +0100, Thomas Hellström wrote: > Add the first version of the VM_BIND locking document which is > intended to be part of the xe driver upstreaming agreement. > > The document describes and discuss the locking used during exec- > functions, evicton and for

[PATCH] drm/bridge: ti-sn65dsi86: Associate PWM device to auxiliary device

2023-11-27 Thread Uwe Kleine-König
It's the ti_sn65dsi86.pwm auxiliary driver that creates the pwmchip, so let the auxiliary device be the parent of the pwm device. Note that getting a reference to the ti-sn65dsi86's pwm using pwm_get() isn't affected by this change as ti_sn65dsi86_add_aux_device() sets the auxiliary device's

Re: [PATCH drm-misc-next 0/5] PowerVR VM fixes

2023-11-27 Thread Frank Binns
Hi, Thank you for the patches. On Sat, 2023-11-25 at 00:36 +0100, Danilo Krummrich wrote: > Hi, > > Some major GPUVM changes landed just before v8 of the PowerVR series. Since v8 > went in rather quickly (review process was finished otherwise) I haven't had > the > chance to review the

Re: [PATCH] drm/panfrost: Really power off GPU cores in panfrost_gpu_power_off()

2023-11-27 Thread Marek Szyprowski
a system crash: root@target:~# ./modetest -C trying to open device 'i915'...failed trying to open device 'amdgpu'...failed trying to open device 'radeon'...failed trying to open device 'nouveau'...failed trying to open device 'vmwgfx'...failed trying to open device 'omapdrm'...failed trying to open d

[PATCH] Revert "drm/sched: Qualify drm_sched_wakeup() by drm_sched_entity_is_ready()"

2023-11-27 Thread Bert Karwatzki
Commit f3123c25 (in combination with the use of work queues by the gpu scheduler) leads to random lock ups of the GUI [1,2]. This is not a complete revert of commit f3123c25 as drm_sched_wakeup still needs its entity argument to pass it to drm_sched_can_queue. [1]

Re: [PATCH v2 11/12] drm/rockchip: vop2: Add debugfs support

2023-11-27 Thread Andy Yan
Hi Sascha: thanks for you review. On 11/27/23 18:13, Sascha Hauer wrote: On Wed, Nov 22, 2023 at 08:56:01PM +0800, Andy Yan wrote: From: Andy Yan /sys/kernel/debug/dri/vop2/summary: dump vop display state /sys/kernel/debug/dri/vop2/regs: dump whole vop registers

Re: [PATCH -next] drm/imagination: Remove unneeded semicolon

2023-11-27 Thread Frank Binns
Hi, Thank you for the patch. On Mon, 2023-11-27 at 09:04 +0800, Yang Li wrote: > ./drivers/gpu/drm/imagination/pvr_free_list.c:258:2-3: Unneeded semicolon > > Reported-by: Abaci Robot > Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=7635 > Signed-off-by: Yang Li Reviewed-by: Frank

Re: [PATCH][next] drm/imagination: Fix a couple of spelling mistakes in literal strings

2023-11-27 Thread Frank Binns
Hi Colin, Thank you for the patch. On Fri, 2023-11-24 at 16:39 +, Colin Ian King wrote: > There are a couple of spelling mistakes in literal strings in the > stid_fmts array. Fix these. > > Signed-off-by: Colin Ian King Reviewed-by: Frank Binns > --- >

Re: [PATCH v2 10/12] drm/rockchip: vop2: Add support for rk3588

2023-11-27 Thread Sascha Hauer
Hi Andy, Looks good overall, two small things inside. On Wed, Nov 22, 2023 at 08:55:44PM +0800, Andy Yan wrote: > > +#define vop2_output_if_is_hdmi(x)(x == ROCKCHIP_VOP2_EP_HDMI0 || x == > ROCKCHIP_VOP2_EP_HDMI1) > +#define vop2_output_if_is_dp(x) (x == ROCKCHIP_VOP2_EP_DP0

Re: [PATCH 2/2] drm/panfrost: Fix incorrect updating of current device frequency

2023-11-27 Thread Steven Price
On 25/11/2023 20:52, Adrián Larumbe wrote: > It was noticed when setting the Panfrost's DVFS device to the performant > governor, GPU frequency as reported by fdinfo had dropped to 0 permamently. > > There are two separate issues causing this behaviour: > - Not initialising the device's

Re: [PATCH 1/2] drm/panfrost: Consider dma-buf imported objects as resident

2023-11-27 Thread Steven Price
On 25/11/2023 20:52, Adrián Larumbe wrote: > A GEM object constructed from a dma-buf imported sgtable should be regarded > as being memory resident, because the dma-buf API mandates backing storage > to be allocated when attachment succeeds. This obviously can cause a bit of double-counting

Re: [PATCH v1 1/2] dt-bindings: display: bridge: cdns: Add properties to support StarFive JH7110 SoC

2023-11-27 Thread Krzysztof Kozlowski
On 27/11/2023 12:34, Shengyang Chen wrote: > From: Keith Zhao > > Add properties in CDNS DSI yaml file to match with > CDNS DSI module in StarFive JH7110 SoC. > > Signed-off-by: Keith Zhao > --- > .../bindings/display/bridge/cdns,dsi.yaml | 38 ++- > 1 file changed, 36

Re: [PATCH v1 1/2] dt-bindings: display: bridge: cdns: Add properties to support StarFive JH7110 SoC

2023-11-27 Thread Krzysztof Kozlowski
On 27/11/2023 12:34, Shengyang Chen wrote: > From: Keith Zhao > > Add properties in CDNS DSI yaml file to match with > CDNS DSI module in StarFive JH7110 SoC. > > Signed-off-by: Keith Zhao > --- > .../bindings/display/bridge/cdns,dsi.yaml | 38 ++- > 1 file changed, 36

Re: [PATCH 1/6] dt-bindings: tc358775: Add support for tc358765

2023-11-27 Thread Michael Walle
Hi, > The tc358765 is similar to tc358775 except for the stdby-gpios. Bad timing (for me). I'm about to send a bigger patch series for the tc358775 which fixes the (completely) broken initialialization. And also contains some of your fixes. That being said, I intend to make the standby gpio

Re: [PATCH v2 11/12] drm/rockchip: vop2: Add debugfs support

2023-11-27 Thread Sascha Hauer
On Wed, Nov 22, 2023 at 08:56:01PM +0800, Andy Yan wrote: > From: Andy Yan > > /sys/kernel/debug/dri/vop2/summary: dump vop display state > /sys/kernel/debug/dri/vop2/regs: dump whole vop registers > /sys/kernel/debug/dri/vop2/active_regs: only dump the registers of > activated modules > >

Re: Implement per-key keyboard backlight as auxdisplay?

2023-11-27 Thread Werner Sembach
Hi Hans, Am 22.11.23 um 19:34 schrieb Hans de Goede: Hi Werner, [snip] Another idea I want to throw in the mix: Maybe the kernel is not the right place to implement this at all. RGB stuff is not at all standardized and every vendor is doing completely different interfaces, which does not

Re: [PATCH] drm/panfrost: Really power off GPU cores in panfrost_gpu_power_off()

2023-11-27 Thread AngeloGioacchino Del Regno
eobuf2_common videodev mc s5p_cec CPU: 0 PID: 0 Comm: swapper/0 Not tainted 6.7.0-rc2-next-20231127-00055-ge14abcb527d6 #7649 Hardware name: Samsung Exynos (Flattened Device Tree) PC is at panfrost_gpu_irq_handler+0x18/0xfc LR is at __handle_irq_event_percpu+0xcc/0x31c ... Process swapper/0 (pid: 0

Re: [PATCH v1 3/4] drm/rockchip: rk3066_hdmi: Remove useless output format

2023-11-27 Thread Heiko Stübner
Hi Johan, Am Donnerstag, 23. November 2023, 13:54:28 CET schrieb Johan Jonker: > > On 11/20/23 18:06, Heiko Stuebner wrote: > > Hi Johan, > > > > Am Donnerstag, 2. November 2023, 14:42:19 CET schrieb Johan Jonker: > >> The Rk3066 hdmi output format is hard coded to RGB. Remove > >> all useless

[PATCH v1 2/2] gpu: drm: bridge: cadence: Add a driver and platform ops for StarFive JH7110 SoC

2023-11-27 Thread Shengyang Chen
From: Keith Zhao In order to fit CDNS DSI module in StarFive JH7110 SoC, The mainly modification is followed: 1.Add driver for StarFive JH7110 SoC to drive its CDNS DSI module. 2.Add platform ops in cdns-dsi.c for StarFive JH7110 SoC probing. Signed-off-by: Keith Zhao --- MAINTAINERS

[PATCH v1 1/2] dt-bindings: display: bridge: cdns: Add properties to support StarFive JH7110 SoC

2023-11-27 Thread Shengyang Chen
From: Keith Zhao Add properties in CDNS DSI yaml file to match with CDNS DSI module in StarFive JH7110 SoC. Signed-off-by: Keith Zhao --- .../bindings/display/bridge/cdns,dsi.yaml | 38 ++- 1 file changed, 36 insertions(+), 2 deletions(-) diff --git

[PATCH v1 0/2] Add StarFive JH7110 SoC DSI support

2023-11-27 Thread Shengyang Chen
This series is the v1 series that attempts to support the CDNS DSI driver used to converts DPI to DSI. CDNS DSI is embedded in StarFive JH7110 SoC. The series has been tested on the VisionFive 2 board. Keith Zhao (2): dt-bindings: display: bridge: cdns: Add properties to support StarFive

Re: [PATCH v2 06/17] drm/v3d: Decouple job allocation from job initiation

2023-11-27 Thread Maira Canal
Hi Iago, On 11/27/23 05:12, Iago Toral wrote: El jue, 23-11-2023 a las 21:47 -0300, Maíra Canal escribió: We want to allow the IOCTLs to allocate the job without initiating it. This will be useful for the CPU job submission IOCTL, as the CPU job has the need to use information from the user

Re: Bug in the error handling in TTMs pool implementation

2023-11-27 Thread Karolina Stolarek
On 24.11.2023 16:53, Christian König wrote: @Karolina do you of hand know a way how we could exercise this in a TTM unit test? Basically we would need to redirect the alloc_pages_node() symbol to an unit test internal function and let it return an error (or something like that). Do I

Re: [Intel-gfx] [PATCH] drm/i915/display: Fix phys_base to be relative not absolute

2023-11-27 Thread Paz Zcharya
On Wed, Nov 22, 2023 at 02:26:55PM +0100, Andrzej Hajda wrote: > > > On 21.11.2023 13:06, Andrzej Hajda wrote: > > On 18.11.2023 00:01, Paz Zcharya wrote: > > > On Tue, Nov 14, 2023 at 10:13:59PM -0500, Rodrigo Vivi wrote: > > > > On Sun, Nov 05, 2023 at 05:27:03PM +, Paz Zcharya wrote: > >

[PATCH] PCI: qcom: Fix compile error

2023-11-27 Thread Vignesh Raman
Commit a2458d8f618a ("PCI/ASPM: pci_enable_link_state: Add argument to acquire bus lock") has added an argument to acquire bus lock in pci_enable_link_state, but qcom_pcie_enable_aspm calls it without this argument, resulting in below build error. drivers/pci/controller/dwc/pcie-qcom.c:973:9:

Re: [PATCH 1/3] drm/bridge: ti-sn65dsi86: Simplify using pm_runtime_resume_and_get()

2023-11-27 Thread Doug Anderson
Hi, On Thu, Nov 23, 2023 at 9:54 AM Uwe Kleine-König wrote: > > pm_runtime_resume_and_get() already drops the runtime PM usage counter > in the error case. So a call to pm_runtime_put_sync() can be dropped. > > Signed-off-by: Uwe Kleine-König > --- > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 6

Re: [PATCH] drm/msm/dpu: Capture dpu snapshot when frame_done_timer timeouts

2023-11-27 Thread Dmitry Baryshkov
On Tue, 28 Nov 2023 at 03:12, Paloma Arellano wrote: > > Trigger a devcoredump to dump dpu registers and capture the drm atomic > state when the frame_done_timer timeouts. > > Signed-off-by: Paloma Arellano > --- > drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 13 +++-- > 1 file

Re: [v3 3/3] drm/bridge: it6505: Add audio support

2023-11-27 Thread Chen-Yu Tsai
On Tue, Nov 21, 2023 at 8:54 PM AngeloGioacchino Del Regno wrote: > > Il 30/07/23 20:08, Jiaxin Yu ha scritto: > > Add audio support for it6505 > > > > 1. Bridge to hdmi-codec to support audio feature. At the same time, > > the function of automatically detecting audio is removed. > > 2. It

<    1   2   3   >