[PATCH] dt-bindings: display: panel: convert truly,nt35597.txt to dt-schema

2025-05-07 Thread Neil Armstrong
}; - }; - - port@1 { - reg = <1>; - panel1_in: endpoint { - remote-endpoint = <&dsi1_out>; - }; - }; - }; - }; - }; --- base-commit: 08710e696081d58163c8078e0e096be6d35c5fad change-id: 20250507-topic-misc-truly-nt35597-yaml-ba40899e4980 Best regards, -- Neil Armstrong

Re: [PATCH v2 30/34] drm/bridge: imx8qxp-pixel-combiner: convert to devm_drm_bridge_alloc() API

2025-05-07 Thread Liu Ying
On 05/07/2025, Luca Ceresoli wrote: > Hello Liu, > > On Wed, 7 May 2025 10:10:53 +0800 > Liu Ying wrote: > >> On 05/07/2025, Luca Ceresoli wrote: >>> Hello Liu, >> >> Hi Luca, >> >>> >>> thanks for your further feedback. >>> >>> On Tue, 6 May 2025 10:24:18 +0800 >>> Liu Ying wrote: >>> >>>

Re: [PATCH v1 05/11] mm: convert VM_PFNMAP tracking to pfnmap_track() + pfnmap_untrack()

2025-05-07 Thread Lorenzo Stoakes
On Wed, May 07, 2025 at 03:25:42PM +0200, David Hildenbrand wrote: > > > > > > Obviously my series will break this but should be _fairly_ trivial to > > > update. > > > > > > You will however have to make sure to update tools/testing/vma/* to handle > > > the new functions in userland testing (they

Re: [PATCH] drm/bridge/synopsys: avoid field overflow warning

2025-05-07 Thread Heiko Stübner
Am Donnerstag, 10. April 2025, 14:36:45 Mitteleuropäische Sommerzeit schrieb Arnd Bergmann: > On Wed, Apr 9, 2025, at 09:07, Arnd Bergmann wrote: > > On Tue, Apr 8, 2025, at 19:51, Arnd Bergmann wrote: > >> From: Arnd Bergmann > >> > >> clang-16 and earlier complain about what it thinks might be

[PATCH v2 0/3] media: fix incorrect use of dma_sync_sg_*() calls

2025-05-07 Thread Marek Szyprowski
Dear All, This patchset fixes the incorrect use of dma_sync_sg_*() calls in media and related drivers. They are replaced with much safer dma_sync_sgtable_*() variants, which take care of passing the proper number of elements for the sync operation. Best regards Marek Szyprowski, PhD Samsung R&D I

Re: [PATCH] drm/etnaviv: Fix flush sequence logic

2025-05-07 Thread Fabio Estevam
Hi Tomeu, On Wed, May 7, 2025 at 8:22 AM Tomeu Vizoso wrote: > > We should be comparing the last submitted sequence number with that of > the address space we may be switching to. > > And we should be using the latter as the last submitted sequence number > afterwards. > > Signed-off-by: Tomeu Vi

[PATCH v5 11/16] drm/vkms: Allow to attach connectors and encoders via configfs

2025-05-07 Thread José Expósito
From: Louis Chauvet Create a default subgroup at /config/vkms/connectors/connector/possible_encoders that will contain symbolic links to the possible encoders for the connector. Reviewed-by: Louis Chauvet Signed-off-by: Louis Chauvet Co-developed-by: José Expósito Signed-off-by: José Expósito

[PATCH v5 12/16] drm/vkms: Allow to configure the default device creation

2025-05-07 Thread José Expósito
Add a new module param to allow to create or not the default VKMS instance. Useful when combined with configfs to avoid having additional VKMS instances. Reviewed-by: Louis Chauvet Signed-off-by: José Expósito --- drivers/gpu/drm/vkms/vkms_drv.c | 7 +++ 1 file changed, 7 insertions(+) dif

[PATCH v3 10/19] gpu: nova-core: add DMA object struct

2025-05-07 Thread Alexandre Courbot
Since we will need to allocate lots of distinct memory chunks to be shared between GPU and CPU, introduce a type dedicated to that. It is a light wrapper around CoherentAllocation. Signed-off-by: Alexandre Courbot --- drivers/gpu/nova-core/dma.rs | 60 ++

[PATCH v5 04/16] drm/vkms: Allow to configure the plane type via configfs

2025-05-07 Thread José Expósito
From: Louis Chauvet When a plane is created, add a `type` file to allow to set the type: - 0 overlay - 1 primary - 2 cursor Reviewed-by: Louis Chauvet Signed-off-by: Louis Chauvet Co-developed-by: José Expósito Signed-off-by: José Expósito --- Documentation/gpu/vkms.rst | 5 +

Re: [PATCH v5 2/3] drm/panfrost: Add support for Mali on the MT8370 SoC

2025-05-07 Thread Steven Price
On 02/05/2025 13:17, Louis-Alexis Eyraud wrote: > Add a compatible for the MediaTek MT8370 SoC, with an integrated ARM > Mali G57 MC2 GPU (Valhall-JM, dual core), with new platform data for > its support in the panfrost driver. > It uses the same data as MT8186 for the power management features to

[PATCH v5 15/16] drm/vkms: Allow to update the connector status

2025-05-07 Thread José Expósito
Implement the drm_connector_funcs.detect() callback to update the connector status by returning the status stored in the configuration. Reviewed-by: Louis Chauvet Signed-off-by: José Expósito --- drivers/gpu/drm/vkms/vkms_connector.c | 28 +++ drivers/gpu/drm/vkms/vkms_c

Re: [PATCH 3/3] drm/panfrost: show device-wide list of DRM GEM objects over DebugFS

2025-05-07 Thread Adrián Larumbe
On 06.05.2025 09:04, Boris Brezillon wrote: > On Thu, 24 Apr 2025 03:21:32 +0100 > Adrián Larumbe wrote: > > > This change is essentially a Panfrost port of commit a3707f53eb3f > > ("drm/panthor: show device-wide list of DRM GEM objects over DebugFS"). > > > > The DebugFS file is almost the same a

[PATCH v5 08/16] drm/vkms: Allow to configure multiple encoders via configfs

2025-05-07 Thread José Expósito
From: Louis Chauvet Create a default subgroup at /config/vkms/encoders to allow to create as many encoders as required. Reviewed-by: Louis Chauvet Signed-off-by: Louis Chauvet Co-developed-by: José Expósito Signed-off-by: José Expósito --- Documentation/gpu/vkms.rst | 6 ++ drive

Re: [PATCH 1/3] drm/panfrost: Add BO labelling to Panfrost

2025-05-07 Thread Adrián Larumbe
On 07.05.2025 14:18, Steven Price wrote: > On 24/04/2025 03:21, Adrián Larumbe wrote: > > Unlike in Panthor, from where this change is based on, there is no need > > to support tagging of BO's other than UM-exposed ones, so all strings > > can be freed with kfree(). > > > > This commit is done in p

Re: [PATCH v1] drm/bridge: cdns-dsi: Replace deprecated UNIVERSAL_DEV_PM_OPS()

2025-05-07 Thread Vitor Soares
On Mon, 2025-05-05 at 21:03 +0300, Tomi Valkeinen wrote: Hello, > Hi, > > On 05/05/2025 20:47, Vitor Soares wrote: > > On Mon, 2025-05-05 at 18:30 +0300, Tomi Valkeinen wrote: > > > Hi, > > > > > > On 05/05/2025 17:45, Vitor Soares wrote: > > > > On Tue, 2025-04-29 at 09:32 +0300, Tomi Valkeinen

Re: [PATCH v2 30/34] drm/bridge: imx8qxp-pixel-combiner: convert to devm_drm_bridge_alloc() API

2025-05-07 Thread Luca Ceresoli
Hello Liu, On Wed, 7 May 2025 18:16:28 +0800 Liu Ying wrote: [...] > After looking into this patch and patch 31(though I've already provided > my A-b) > more closely, I think the imx8qxp_pc and imx8{qm,qxp}_ldb main structures > should have the same life time with the embed

Re: [PATCH bpf-next v3 2/5] bpf: Add dmabuf iterator

2025-05-07 Thread Alexei Starovoitov
On Tue, May 6, 2025 at 5:10 PM T.J. Mercier wrote: > > +/** > + * get_first_dmabuf - begin iteration through global list of DMA-bufs > + * > + * Returns the first buffer in the global list of DMA-bufs that's not in the > + * process of being destroyed. Increments that buffer's reference count to >

Re: [PATCH v2 17/21] rust: num: Add an upward alignment helper for usize

2025-05-07 Thread Alexandre Courbot
On Tue May 6, 2025 at 12:25 AM JST, Joel Fernandes wrote: >> Actually it may be a good idea to move this into its own patch/series so >> it gets more attention as this is starting to look like the `num` or >> `num_integer` crates and we might be well-advised to take more >> inspiration from them in

Re: [PATCH bpf-next v3 2/5] bpf: Add dmabuf iterator

2025-05-07 Thread T.J. Mercier
On Wed, May 7, 2025 at 1:15 AM Christian König wrote: > > On 5/7/25 02:10, T.J. Mercier wrote: > > The dmabuf iterator traverses the list of all DMA buffers. > > > > DMA buffers are refcounted through their associated struct file. A > > reference is taken on each buffer as the list is iterated to

Re: [PATCH bpf-next v3 2/5] bpf: Add dmabuf iterator

2025-05-07 Thread T.J. Mercier
On Wed, May 7, 2025 at 7:14 AM Alexei Starovoitov wrote: > > On Tue, May 6, 2025 at 5:10 PM T.J. Mercier wrote: > > > > +/** > > + * get_first_dmabuf - begin iteration through global list of DMA-bufs > > + * > > + * Returns the first buffer in the global list of DMA-bufs that's not in > > the >

Re: [rfc] drm/ttm/memcg: simplest initial memcg/ttm integration (v2)

2025-05-07 Thread Johannes Weiner
Hello Dave, On Fri, May 02, 2025 at 01:35:59PM +1000, Dave Airlie wrote: > Hey all, > > This is my second attempt at adding the initial simple memcg/ttm > integration. > > This varies from the first attempt in two major ways: > > 1. Instead of using __GFP_ACCOUNT and direct calling kmem charges

Re: [PATCH v8 10/10] ref_tracker: eliminate the ref_tracker_dir name field

2025-05-07 Thread Jakub Kicinski
On Wed, 07 May 2025 09:06:35 -0400 Jeff Layton wrote: > + * @quarantime_count: max number of entries to be tracked quarantime ^ :( -- pw-bot: cr

Re: [PATCH] dt-bindings: display/msm: hdmi: Fix constraints on additional 'port' properties

2025-05-07 Thread Dmitry Baryshkov
On Wed, May 07, 2025 at 04:59:12PM -0500, Rob Herring (Arm) wrote: > The MSM HDMI port nodes are missing any restrictions on additional > properties. The $ref should be to "/properties/port" rather than > "/$defs/port-base" as there are not additional properties in the nodes > to define. > > Signe

RE: [PATCH] drm/dp: Fix Write_Status_Update_Request AUX request format

2025-05-07 Thread Lin, Wayne
[AMD Official Use Only - AMD Internal Distribution Only] Thanks for the feedback! I'll adjust it and give v2 soon. Thanks, Wayne > -Original Message- > From: Limonciello, Mario > Sent: Tuesday, May 6, 2025 4:22 AM > To: Lin, Wayne ; dri-devel@lists.freedesktop.org > Cc: ville.syrj...@li

[PATCH v3 3/3] drm/udl: use DRM_GEM_SHMEM_SIMPLE_DRIVER_OPS

2025-05-07 Thread oushixiong1025
From: Shixiong Ou Import dmabuf without mapping its sg_table to avoid issues likes: udl 2-1.4:1.0: swiotlb buffer is full (sz: 2097152 bytes), total 65536 (slots), used 1 (slots) Signed-off-by: Shixiong Ou --- drivers/gpu/drm/udl/udl_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH v3 2/3] drm/ast: use DRM_GEM_SHMEM_SIMPLE_DRIVER_OPS

2025-05-07 Thread oushixiong1025
From: Shixiong Ou Import dmabuf without mapping its sg_table to avoid issues likes: ast :07:00.0: swiotlb buffer is full (sz: 3145728 bytes), total 32768 (slots), used 0 (slots) Signed-off-by: Shixiong Ou --- drivers/gpu/drm/ast/ast_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deleti

[PATCH v3 1/3] drm/shmem-helper: Import dmabuf without mapping its sg_table

2025-05-07 Thread oushixiong1025
From: Shixiong Ou [WHY] 1. Drivers using DRM_GEM_SHADOW_PLANE_HELPER_FUNCS and DRM_GEM_SHMEM_DRIVER_OPS (e.g., udl, ast) do not require sg_table import. They only need dma_buf_vmap() to access the shared buffer's kernel virtual address. 2. On certain Aspeed-based boards, a dma_mask o

[PATCH] drm/etnaviv: Fix flush sequence logic

2025-05-07 Thread Tomeu Vizoso
We should be comparing the last submitted sequence number with that of the address space we may be switching to. And we should be using the latter as the last submitted sequence number afterwards. Signed-off-by: Tomeu Vizoso --- drivers/gpu/drm/etnaviv/etnaviv_buffer.c | 3 ++- 1 file changed,

Re: [PATCH v2 09/21] gpu: nova-core: move Firmware to firmware module

2025-05-07 Thread Alexandre Courbot
On Sat May 3, 2025 at 6:14 AM JST, Timur Tabi wrote: > On Thu, 2025-05-01 at 21:58 +0900, Alexandre Courbot wrote: >> +pub(crate) const FIRMWARE_VERSION: &str = "535.113.01"; > > This needs to change to 570.144. You can find images to use here: > > https://github.com/NVIDIA/linux-firmware/commits/

Re: [PATCH] drm/virtio: Fix NULL pointer deref in virtgpu_dma_buf_free_obj()

2025-05-07 Thread Dmitry Osipenko
On 5/7/25 05:51, Kasireddy, Vivek wrote: > Hi Dmitry, > >> Subject: Re: [PATCH] drm/virtio: Fix NULL pointer deref in >> virtgpu_dma_buf_free_obj() >> >> On 5/2/25 02:24, Vivek Kasireddy wrote: >>> There is a chance that obj->dma_buf would be NULL by the time >>> virtgpu_dma_buf_free_obj() is call

[PATCH v3 01/19] rust: dma: expose the count and size of CoherentAllocation

2025-05-07 Thread Alexandre Courbot
These properties are very useful to have and should be accessible. Signed-off-by: Alexandre Courbot --- rust/kernel/dma.rs | 14 ++ 1 file changed, 14 insertions(+) diff --git a/rust/kernel/dma.rs b/rust/kernel/dma.rs index 605e01e35715667f93297fd9ec49d8e7032e0910..18602d771054fceb

[PATCH v3 02/19] gpu: nova-core: derive useful traits for Chipset

2025-05-07 Thread Alexandre Courbot
We will commonly need to compare chipset versions, so derive the ordering traits to make that possible. Also derive Copy and Clone since passing Chipset by value will be more efficient than by reference. Signed-off-by: Alexandre Courbot --- drivers/gpu/nova-core/gpu.rs | 2 +- 1 file changed, 1

[PATCH v3 00/19] nova-core: run FWSEC-FRTS to perform first stage of GSP initialization

2025-05-07 Thread Alexandre Courbot
Hi everyone, I have respinned this series due to the introduction of `Devres::access()` in nova-next - this addition seriously simplifies the way we access the BAR and justifies sending a new version, although there are also more minor changes. While this series is still far from bringing the GPU

[PATCH v3 04/19] gpu: nova-core: take bound device in Gpu::new

2025-05-07 Thread Alexandre Courbot
We will need to perform things like allocating DMA memory during device creation, so make sure to take the device context that will allow us to perform these actions. This also allows us to use Devres::access to obtain the BAR without holding a RCU lock. Signed-off-by: Alexandre Courbot --- driv

[PATCH v3 17/19] gpu: nova-core: compute layout of the FRTS region

2025-05-07 Thread Alexandre Courbot
FWSEC-FRTS is run with the desired address of the FRTS region as parameter, which we need to compute depending on some hardware parameters. Do this in a `FbLayout` structure, that will be later extended to describe more memory regions used to boot the GSP. Signed-off-by: Alexandre Courbot --- d

[PATCH v3 13/19] gpu: nova-core: add falcon register definitions and base code

2025-05-07 Thread Alexandre Courbot
Add the common Falcon code and HAL for Ampere GPUs, and instantiate the GSP and SEC2 Falcons that will be required to boot the GSP. Signed-off-by: Alexandre Courbot --- drivers/gpu/nova-core/falcon.rs | 540 ++ drivers/gpu/nova-core/falcon/gsp.rs | 22

[PATCH v3 19/19] gpu: nova-core: load and run FWSEC-FRTS

2025-05-07 Thread Alexandre Courbot
With all the required pieces in place, load FWSEC-FRTS onto the GSP falcon, run it, and check that it successfully carved out the WPR2 region out of framebuffer memory. Signed-off-by: Alexandre Courbot --- drivers/gpu/nova-core/falcon.rs | 3 --- drivers/gpu/nova-core/gpu.rs| 57 +++

[PATCH v3 15/19] rust: num: Add an upward alignment helper for usize

2025-05-07 Thread Alexandre Courbot
From: Joel Fernandes This will be used in the nova-core driver where we need to upward-align the image size to get to the next image in the VBIOS ROM. [acour...@nvidia.com: handled conflicts due to removal of patch creating num.rs] Signed-off-by: Joel Fernandes Signed-off-by: Alexandre Courbot

[PATCH v3 08/19] rust: make ETIMEDOUT error available

2025-05-07 Thread Alexandre Courbot
We will use this error in the nova-core driver. Signed-off-by: Alexandre Courbot --- rust/kernel/error.rs | 1 + 1 file changed, 1 insertion(+) diff --git a/rust/kernel/error.rs b/rust/kernel/error.rs index 3dee3139fcd4379b94748c0ba1965f4e1865b633..083c7b068cf4e185100de96e520c54437898ee72 100

[PATCH v3 18/19] gpu: nova-core: extract FWSEC from BIOS and patch it to run FWSEC-FRTS

2025-05-07 Thread Alexandre Courbot
The FWSEC firmware needs to be extracted from the VBIOS and patched with the desired command, as well as the right signature. Do this so we are ready to load and run this firmware into the GSP falcon and create the FRTS region. [joelagn...@nvidia.com: give better names to FalconAppifHdrV1's fields

[PATCH v3 05/19] gpu: nova-core: define registers layout using helper macro

2025-05-07 Thread Alexandre Courbot
Add the register!() macro, which defines a given register's layout and provide bit-field accessors with a way to convert them to a given type. This macro will allow us to make clear definitions of the registers and manipulate their fields safely. The long-term goal is to eventually move it to the

[PATCH v3 07/19] gpu: nova-core: move Firmware to firmware module

2025-05-07 Thread Alexandre Courbot
We will extend the firmware methods, so move it to its own module instead to keep gpu.rs focused. Signed-off-by: Alexandre Courbot --- drivers/gpu/nova-core/firmware.rs | 46 +-- drivers/gpu/nova-core/gpu.rs | 35 +++-- 2 files cha

[PATCH v3 09/19] gpu: nova-core: wait for GFW_BOOT completion

2025-05-07 Thread Alexandre Courbot
Upon reset, the GPU executes the GFW_BOOT firmware in order to initialize its base parameters such as clocks. The driver must ensure that this step is completed before using the hardware. Signed-off-by: Alexandre Courbot --- drivers/gpu/nova-core/devinit.rs | 38 +++

[PATCH v3 11/19] gpu: nova-core: register sysmem flush page

2025-05-07 Thread Alexandre Courbot
A page of system memory is reserved so sysmembar can perform a read on it if a system write occurred since the last flush. Do this early as it can be required to e.g. reset the GPU falcons. Signed-off-by: Alexandre Courbot --- drivers/gpu/nova-core/gpu.rs | 45 ++

[PATCH v3 12/19] gpu: nova-core: add helper function to wait on condition

2025-05-07 Thread Alexandre Courbot
While programming the hardware, we frequently need to busy-wait until a condition (like a given bit of a register to switch value) happens. Add a basic `wait_on` helper function to wait on such conditions expressed as a closure, with a timeout argument. This is temporary as we will switch to `rea

[PATCH v3 06/19] gpu: nova-core: fix layout of NV_PMC_BOOT_0

2025-05-07 Thread Alexandre Courbot
The layout of NV_PMC_BOOT_0 has two small issues: - The "chipset" field, while useful to identify a chip, is actually an aggregate of two distinct fields named "architecture" and "implementation". - The "architecture" field is split, with its MSB being at a different location than the rest o

[PATCH v3 14/19] gpu: nova-core: firmware: add ucode descriptor used by FWSEC-FRTS

2025-05-07 Thread Alexandre Courbot
FWSEC-FRTS is the first firmware we need to run on the GSP falcon in order to initiate the GSP boot process. Introduce the structure that describes it. Signed-off-by: Alexandre Courbot --- drivers/gpu/nova-core/firmware.rs | 43 +++ 1 file changed, 43 insertio

[PATCH v5 13/16] drm/vkms: Remove completed task from the TODO list

2025-05-07 Thread José Expósito
Remove the configfs related TODO items from the "Runtime Configuration" section. Reviewed-by: Louis Chauvet Signed-off-by: José Expósito --- Documentation/gpu/vkms.rst | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/Documentation/gpu/vkms.rst b/Documentation/gpu/vkms

[PATCH v5 02/16] drm/vkms: Add and remove VKMS instances via configfs

2025-05-07 Thread José Expósito
From: Louis Chauvet Allow to create, enable, disable and destroy VKMS instances using configfs. For the moment, it is not possible to add pipeline items, so trying to enable the device will fail printing an informative error to the log. Reviewed-by: Louis Chauvet Signed-off-by: Louis Chauvet

[PATCH v5 01/16] drm/vkms: Expose device creation and destruction

2025-05-07 Thread José Expósito
In preparation for configfs support, expose vkms_create() and vkms_destroy(). Reviewed-by: Louis Chauvet Co-developed-by: Louis Chauvet Signed-off-by: Louis Chauvet Signed-off-by: José Expósito --- drivers/gpu/drm/vkms/vkms_drv.c | 4 ++-- drivers/gpu/drm/vkms/vkms_drv.h | 20 +++

[PATCH v5 05/16] drm/vkms: Allow to configure multiple CRTCs via configfs

2025-05-07 Thread José Expósito
From: Louis Chauvet Create a default subgroup at /config/vkms/crtcs to allow to create as many CRTCs as required. Reviewed-by: Louis Chauvet Signed-off-by: Louis Chauvet Co-developed-by: José Expósito Signed-off-by: José Expósito --- Documentation/gpu/vkms.rst | 6 ++ drivers/gpu

[PATCH v5 07/16] drm/vkms: Allow to attach planes and CRTCs via configfs

2025-05-07 Thread José Expósito
From: Louis Chauvet Create a default subgroup at /config/vkms/planes/plane/possible_crtcs that will contain symbolic links to the possible CRTCs for the plane. Reviewed-by: Louis Chauvet Signed-off-by: Louis Chauvet Co-developed-by: José Expósito Signed-off-by: José Expósito --- Documentati

[PATCH v5 00/16] drm/vkms: Add configfs support

2025-05-07 Thread José Expósito
Hi everyone, This series allow to configure one or more VKMS instances without having to reload the driver using configfs. The series is structured in 3 blocks: - Patches 1..11: Basic device configuration. For simplicity, I kept the available options as minimal as possible. - Patches 12

[PATCH v5 10/16] drm/vkms: Allow to configure multiple connectors via configfs

2025-05-07 Thread José Expósito
From: Louis Chauvet Create a default subgroup at /config/vkms/connectors to allow to create as many connectors as required. Reviewed-by: Louis Chauvet Signed-off-by: Louis Chauvet Co-developed-by: José Expósito Signed-off-by: José Expósito --- Documentation/gpu/vkms.rst | 6 ++ d

[PATCH v5 03/16] drm/vkms: Allow to configure multiple planes via configfs

2025-05-07 Thread José Expósito
From: Louis Chauvet Create a default subgroup at /config/vkms/planes to allow to create as many planes as required. Reviewed-by: Louis Chauvet Signed-off-by: Louis Chauvet Co-developed-by: José Expósito Signed-off-by: José Expósito --- Documentation/gpu/vkms.rst | 16 - driver

[PATCH v5 06/16] drm/vkms: Allow to configure CRTC writeback support via configfs

2025-05-07 Thread José Expósito
From: Louis Chauvet When a CRTC is created, add a `writeback` file to allow to enable or disable writeback connector support Reviewed-by: Louis Chauvet Signed-off-by: Louis Chauvet Co-developed-by: José Expósito Signed-off-by: José Expósito --- Documentation/gpu/vkms.rst | 4 +++

[PATCH v5 16/16] drm/vkms: Allow to configure connector status via configfs

2025-05-07 Thread José Expósito
When a connector is created, add a `status` file to allow to update the connector status to: - 1 connector_status_connected - 2 connector_status_disconnected - 3 connector_status_unknown If the device is enabled, updating the status hot-plug or unplugs the connector. Reviewed-by: Louis Chauve

[PATCH v5 09/16] drm/vkms: Allow to attach encoders and CRTCs via configfs

2025-05-07 Thread José Expósito
From: Louis Chauvet Create a default subgroup at /config/vkms/encoders/encoder/possible_crtcs that will contain symbolic links to the possible CRTCs for the encoder. Reviewed-by: Louis Chauvet Signed-off-by: Louis Chauvet Co-developed-by: José Expósito Signed-off-by: José Expósito --- Docum

[PATCH v5 14/16] drm/vkms: Allow to configure connector status

2025-05-07 Thread José Expósito
Allow to store the connector status in vkms_config_connector and add a getter and a setter functions as well a KUnit test. This change only adds the configuration, the connector status is not used yet. Reviewed-by: Louis Chauvet Signed-off-by: José Expósito --- drivers/gpu/drm/vkms/tests/vkms_

Re: [PATCH] drm/etnaviv: Fix flush sequence logic

2025-05-07 Thread Lucas Stach
Am Mittwoch, dem 07.05.2025 um 13:21 +0200 schrieb Tomeu Vizoso: > We should be comparing the last submitted sequence number with that of > the address space we may be switching to. > > And we should be using the latter as the last submitted sequence number > afterwards. > > Signed-off-by: Tomeu

Re: [PATCH V9 35/43] drm/amd/display: add 3x4 matrix colorop

2025-05-07 Thread kernel test robot
Hi Alex, kernel test robot noticed the following build warnings: [auto build test WARNING on drm-exynos/exynos-drm-next] [also build test WARNING on next-20250507] [cannot apply to linus/master drm/drm-next drm-intel/for-linux-next drm-intel/for-linux-next-fixes drm-misc/drm-misc-next drm-tip

Re: [PATCH 06/13] drm/dp: Modify drm_edp_probe_state

2025-05-07 Thread kernel test robot
se: next-20250411 patch link: https://lore.kernel.org/r/20250414041637.128039-7-suraj.kandpal%40intel.com patch subject: [PATCH 06/13] drm/dp: Modify drm_edp_probe_state config: i386-randconfig-141-20250414 (https://download.01.org/0day-ci/archive/20250507/202505071939.iq3zwvvx-...@intel.

Re: [PATCH] dt-bindings: gpu: mali-bifrost: Add compatible for RZ/V2N SoC

2025-05-07 Thread Rob Herring (Arm)
On Fri, 02 May 2025 17:25:40 +0100, Prabhakar wrote: > From: Lad Prabhakar > > Add a compatible string for the Renesas RZ/V2N SoC variants that include a > Mali-G31 GPU. These variants share the same restrictions on interrupts, > clocks, and power domains as the RZ/G2L SoC, so extend the existi

[PATCH V1] accel/amdxdna: Support submit commands without arguments

2025-05-07 Thread Lizhi Hou
The latest userspace runtime allows generating commands which do not have any argument. Remove the corresponding check in driver IOCTL to enable this use case. Signed-off-by: Lizhi Hou --- drivers/accel/amdxdna/amdxdna_ctx.c | 22 -- 1 file changed, 12 insertions(+), 10 delet

Re: [PATCH RESEND] drm/tegra: fix a possible null pointer dereference

2025-05-07 Thread Thierry Reding
On Wed, Feb 12, 2025 at 09:42:45AM +0800, Qiu-ji Chen wrote: > In tegra_crtc_reset(), new memory is allocated with kzalloc(), but > no check is performed. Before calling __drm_atomic_helper_crtc_reset, > state should be checked to prevent possible null pointer dereference. > > Fixes: b7e0b04ae450

Re: [PATCH -next v2] gpu: host1x: Use for_each_available_child_of_node_scoped()

2025-05-07 Thread Thierry Reding
On Fri, Aug 30, 2024 at 03:38:24PM +0800, Jinjie Ruan wrote: > Avoids the need for manual cleanup of_node_put() in early exits > from the loop. > > Signed-off-by: Jinjie Ruan > --- > v2: > - Split out from the patch set. > --- > drivers/gpu/host1x/bus.c | 11 +++ > 1 file changed, 3 inse

RE: [PATCH] drm/tegra: rgb: Fix the unbound reference count

2025-05-07 Thread Biju Das
Hi Thierry, > -Original Message- > From: Thierry Reding > Sent: 07 May 2025 17:00 > To: Biju Das > Subject: Re: [PATCH] drm/tegra: rgb: Fix the unbound reference count > > On Wed, Feb 05, 2025 at 11:21:35AM +, Biju Das wrote: > > The of_get_child_by_name() increments the refcount in

RE: [PATCH v24 0/5] drm/xe/xe_vm: Implement xe_vm_get_property_ioctl

2025-05-07 Thread Cavitt, Jonathan
Ignore this one: I misspelt the name of a recipient -Jonathan Cavitt -Original Message- From: Cavitt, Jonathan Sent: Wednesday, May 7, 2025 8:57 AM To: intel...@lists.freedesktop.org Cc: Gupta, saurabhg ; Zuo, Alex ; Cavitt, Jonathan ; joonas.lahti...@linux.intel.com; Brost, Matthew ;

[PATCH v3 0/3] media: fix incorrect use of dma_sync_sg_*() calls

2025-05-07 Thread Marek Szyprowski
Dear All, This patchset fixes the incorrect use of dma_sync_sg_*() calls in media and related drivers. They are replaced with much safer dma_sync_sgtable_*() variants, which take care of passing the proper number of elements for the sync operation. Best regards Marek Szyprowski, PhD Samsung R&D I

[PATCH v3 2/3] udmabuf: use sgtable-based scatterlist wrappers

2025-05-07 Thread Marek Szyprowski
Use common wrappers operating directly on the struct sg_table objects to fix incorrect use of scatterlists sync calls. dma_sync_sg_for_*() functions have to be called with the number of elements originally passed to dma_map_sg_*() function, not the one returned in sgtable's nents. Fixes: 1ffe09590

[PATCH v2] drm/bridge: cdns-dsi: Replace deprecated UNIVERSAL_DEV_PM_OPS()

2025-05-07 Thread Vitor Soares
From: Vitor Soares The deprecated UNIVERSAL_DEV_PM_OPS() macro uses the provided callbacks for both runtime PM and system sleep. This causes the DSI clocks to be disabled twice: once during runtime suspend and again during system suspend, resulting in a WARN message from the clock framework when

Re: [PATCH V1] accel/amdxdna: Support submit commands without arguments

2025-05-07 Thread Mario Limonciello
On 5/7/2025 11:15 AM, Lizhi Hou wrote: The latest userspace runtime allows generating commands which do not have any argument. Remove the corresponding check in driver IOCTL to enable this use case. Signed-off-by: Lizhi Hou Can the userspace handle discovery of the difference? Or does this n

Re: [PATCH v8 07/14] tee: refactor params_from_user()

2025-05-07 Thread Sumit Garg
On Fri, May 02, 2025 at 11:59:21AM +0200, Jens Wiklander wrote: > Break out the memref handling into a separate helper function. > No change in behavior. > > Signed-off-by: Jens Wiklander > --- > drivers/tee/tee_core.c | 94 -- > 1 file changed, 54 inserti

Re: [PATCH 1/8] drm/sched: Allow drivers to skip the reset and keep on running

2025-05-07 Thread Tvrtko Ursulin
On 07/05/2025 13:33, Maíra Canal wrote: Hi Tvrtko, Thanks for the review! On 06/05/25 08:32, Tvrtko Ursulin wrote: On 03/05/2025 21:59, Maíra Canal wrote: When the DRM scheduler times out, it's possible that the GPU isn't hung; instead, a job may still be running, and there may be no valid

Re: [RFC 0/4] Some (drm_sched_|dma_)fence lifetime issues

2025-05-07 Thread Christian König
On 5/7/25 14:28, Tvrtko Ursulin wrote: > > On 28/04/2025 14:15, Christian König wrote: >> On 4/24/25 09:07, Tvrtko Ursulin wrote: >>> >>> On 23/04/2025 14:12, Christian König wrote: On 4/18/25 18:42, Tvrtko Ursulin wrote: > Hi all, > > Recently I mentioned to Danilo about some fen

Re: [PATCH v6 07/11] arm64: dts: qcom: sa8775p-ride: add anx7625 DSI to DP bridge nodes

2025-05-07 Thread Ayushi Makhija
On 5/6/2025 5:58 PM, Dmitry Baryshkov wrote: > On Tue, May 06, 2025 at 05:42:50PM +0530, Ayushi Makhija wrote: >> Hi Dmitry, >> >> On 5/5/2025 3:32 PM, Dmitry Baryshkov wrote: >>> On Mon, May 05, 2025 at 03:12:41PM +0530, Ayushi Makhija wrote: Add anx7625 DSI to DP bridge device nodes. >>

Re: [PATCH v3 1/3] drm/shmem-helper: Import dmabuf without mapping its sg_table

2025-05-07 Thread Christian König
On 5/7/25 11:47, oushixiong1...@163.com wrote: > From: Shixiong Ou > > [WHY] > 1. Drivers using DRM_GEM_SHADOW_PLANE_HELPER_FUNCS and >DRM_GEM_SHMEM_DRIVER_OPS (e.g., udl, ast) do not require >sg_table import. >They only need dma_buf_vmap() to access the shared buffer's >kernel vi

Re: [PATCH 1/3] drm/panfrost: Add BO labelling to Panfrost

2025-05-07 Thread Adrián Larumbe
On 06.05.2025 08:54, Boris Brezillon wrote: > On Thu, 24 Apr 2025 03:21:30 +0100 > Adrián Larumbe wrote: > > > Unlike in Panthor, from where this change is based on, there is no need > > to support tagging of BO's other than UM-exposed ones, so all strings > > can be freed with kfree(). > > > > Th

Re: [PATCH] drm/i915/gt: Remove const from struct i915_wa list allocation

2025-05-07 Thread Andi Shyti
Hi, On Wed, Apr 30, 2025 at 01:56:51PM -0700, Kees Cook wrote: > On Mon, Apr 28, 2025 at 02:40:16PM +0300, Jani Nikula wrote: > > On Fri, 25 Apr 2025, Kees Cook wrote: > > > In preparation for making the kmalloc family of allocators type aware, > > > we need to make sure that the returned type fr

[PATCH v8 10/10] ref_tracker: eliminate the ref_tracker_dir name field

2025-05-07 Thread Jeff Layton
Now that we have dentries and the ability to create meaningful symlinks to them, don't keep a name string in each tracker. Switch the output format to print "class@address", and drop the name field. Also, add a kerneldoc header for ref_tracker_dir_init(). Signed-off-by: Jeff Layton --- drivers/

[PATCH v8 08/10] net: add symlinks to ref_tracker_dir for netns

2025-05-07 Thread Jeff Layton
After assigning the inode number to the namespace, use it to create a unique name for each netns refcount tracker with the ns.inum and net_cookie values in it, and register a symlink to the debugfs file for it. init_net is registered before the ref_tracker dir is created, so add a late_initcall()

[PATCH v8 01/10] ref_tracker: don't use %pK in pr_ostream() output

2025-05-07 Thread Jeff Layton
As Thomas Weißschuh points out [1], it is now preferable to use %p instead of hashed pointers with printk(), since raw pointers should no longer be leaked into the kernel log. Change the ref_tracker infrastructure to use %p instead of %pK in its formats. [1]: https://lore.kernel.org/netdev/202504

[PATCH v8 04/10] ref_tracker: add a static classname string to each ref_tracker_dir

2025-05-07 Thread Jeff Layton
A later patch in the series will be adding debugfs files for each ref_tracker that get created in ref_tracker_dir_init(). The format will be "class@%px". The current "name" string can vary between ref_tracker_dir objects of the same type, so it's not suitable for this purpose. Add a new "class" st

[PATCH v8 05/10] ref_tracker: allow pr_ostream() to print directly to a seq_file

2025-05-07 Thread Jeff Layton
Allow pr_ostream to also output directly to a seq_file without an intermediate buffer. The first caller of +ref_tracker_dir_seq_print() will come in a later patch, so mark that __maybe_unused for now. That designation will be removed once it is used. Reviewed-by: Andrew Lunn Signed-off-by: Jeff L

[PATCH v8 06/10] ref_tracker: automatically register a file in debugfs for a ref_tracker_dir

2025-05-07 Thread Jeff Layton
Currently, there is no convenient way to see the info that the ref_tracking infrastructure collects. Attempt to create a file in debugfs when called from ref_tracker_dir_init(). The file is given the name "class@%px", as having the unmodified address is helpful for debugging. This should be safe s

[PATCH v8 07/10] ref_tracker: add a way to create a symlink to the ref_tracker_dir debugfs file

2025-05-07 Thread Jeff Layton
Add the ability for a subsystem to add a user-friendly symlink that points to a ref_tracker_dir's debugfs file. Signed-off-by: Jeff Layton --- include/linux/ref_tracker.h | 13 + lib/ref_tracker.c | 28 2 files changed, 41 insertions(+) diff --

[PATCH v8 09/10] i915: add ref_tracker_dir symlinks for each tracker

2025-05-07 Thread Jeff Layton
Now that there is the ability to create a symlink for each tracker, do so for the i915 entries. Acked-by: Jani Nikula Signed-off-by: Jeff Layton --- drivers/gpu/drm/i915/intel_runtime_pm.c | 1 + drivers/gpu/drm/i915/intel_wakeref.c| 1 + 2 files changed, 2 insertions(+) diff --git a/drive

[PATCH v8 00/10] ref_tracker: add ability to register a debugfs file for a ref_tracker_dir

2025-05-07 Thread Jeff Layton
Minor update to fix up the build and some compiler warnings with more esoteric Kconfigs. Signed-off-by: Jeff Layton --- Changes in v8: - fix up compiler warnings that the KTR warned about - ensure builds with CONFIG_DEBUG_FS=n and CONFIG_REF_TRACKER=y work - Link to v7: https://lore.kernel.org/r

[PATCH v8 03/10] ref_tracker: have callers pass output function to pr_ostream()

2025-05-07 Thread Jeff Layton
In a later patch, we'll be adding a 3rd mechanism for outputting ref_tracker info via seq_file. Instead of a conditional, have the caller set a pointer to an output function in struct ostream. As part of this, the log prefix must be explicitly passed in, as it's too late for the pr_fmt macro. Revi

Re: [PATCH v8 03/14] optee: account for direction while converting parameters

2025-05-07 Thread Jens Wiklander
Hi Sumit, On Wed, May 7, 2025 at 2:42 PM Sumit Garg wrote: > > Hi Jens, > > On Fri, May 02, 2025 at 11:59:17AM +0200, Jens Wiklander wrote: > > The OP-TEE backend driver has two internal function pointers to convert > > between the subsystem type struct tee_param and the OP-TEE type struct > > op

[PATCH v8 02/10] ref_tracker: add a top level debugfs directory for ref_tracker

2025-05-07 Thread Jeff Layton
Add a new "ref_tracker" directory in debugfs. Each individual refcount tracker can register files under there to display info about currently-held references. Reviewed-by: Andrew Lunn Signed-off-by: Jeff Layton --- lib/ref_tracker.c | 16 1 file changed, 16 insertions(+) diff

Re: [PATCH] drm/etnaviv: Fix flush sequence logic

2025-05-07 Thread Christian Gmeiner
> Hi Tomeu, > > On Wed, May 7, 2025 at 8:22 AM Tomeu Vizoso wrote: > > > > We should be comparing the last submitted sequence number with that of > > the address space we may be switching to. > > > > And we should be using the latter as the last submitted sequence number > > afterwards. > > > > Si

Re: [PATCH 1/3] drm/gpusvm, drm/pagemap: Move migration functionality to drm_pagemap

2025-05-07 Thread kernel test robot
ig: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20250507/202505072016.4hmsizt8-...@intel.com/config) compiler: clang version 20.1.2 (https://github.com/llvm/llvm-project 58df0ef89dd64126512e4ee27b4ac3fd8ddf6247) reproduce (this is a W=1 build): (https://download.01.org/0day-

Re: [PATCH 1/3] drm/panfrost: Add BO labelling to Panfrost

2025-05-07 Thread Steven Price
On 24/04/2025 03:21, Adrián Larumbe wrote: > Unlike in Panthor, from where this change is based on, there is no need > to support tagging of BO's other than UM-exposed ones, so all strings > can be freed with kfree(). > > This commit is done in preparation of a following one that will allow > UM t

Re: [PATCH v1 05/11] mm: convert VM_PFNMAP tracking to pfnmap_track() + pfnmap_untrack()

2025-05-07 Thread David Hildenbrand
Obviously my series will break this but should be _fairly_ trivial to update. You will however have to make sure to update tools/testing/vma/* to handle the new functions in userland testing (they need to be stubbed otu). Hmm, seems to compile. I guess, because we won't have "__HAVE_PFNMAP_TRA

Re: [PATCH 2/3] drm/panfrost: Add driver IOCTL for setting BO labels

2025-05-07 Thread Steven Price
On 24/04/2025 03:21, Adrián Larumbe wrote: > Allow UM to label a BO for which it possesses a DRM handle. > > Signed-off-by: Adrián Larumbe Minor comments below, but otherwise: Reviewed-by: Steven Price > --- > drivers/gpu/drm/panfrost/panfrost_drv.c | 44 - > drivers/

Re: [PATCH] drm/tegra: Remove unneeded include

2025-05-07 Thread Thierry Reding
On Mon, Apr 28, 2025 at 04:34:35PM +0100, Jon Hunter wrote: > The header file 'tegra_drm.h' is included in gem.c, but this file is > already include 'drm.h'. Although there is no harm in including this > file again, it is also not necessary. Furthermore, the header file is > located under 'include/

[PATCH v3 03/19] gpu: nova-core: add missing GA100 definition

2025-05-07 Thread Alexandre Courbot
linux-firmware contains a directory for GA100, and it is a defined chipset in Nouveau. Signed-off-by: Alexandre Courbot --- drivers/gpu/nova-core/gpu.rs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/nova-core/gpu.rs b/drivers/gpu/nova-core/gpu.rs index 4de67a

[PATCH v3 16/19] nova-core: Add support for VBIOS ucode extraction for boot

2025-05-07 Thread Alexandre Courbot
From: Joel Fernandes Add support for navigating and setting up vBIOS ucode data required for GSP to boot. The main data extracted from the vBIOS is the FWSEC-FRTS firmware which runs on the GSP processor. This firmware runs in high secure mode, and sets up the WPR2 (Write protected region) before

  1   2   3   >