Re: [PATCH v6 13/14] drm/mediatek: Support CRC in OVL

2024-03-27 Thread 胡俊光

Re: [PATCH 09/22] gpio: virtio: drop owner assignment

2024-03-27 Thread Viresh Kumar
On 27-03-24, 13:41, Krzysztof Kozlowski wrote: > virtio core already sets the .owner, so driver does not need to. > > Signed-off-by: Krzysztof Kozlowski > > --- > > Depends on the first patch. > --- > drivers/gpio/gpio-virtio.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git

Re: [PATCH] drm/amdgpu: add IP's FW information to devcoredump

2024-03-27 Thread Khatri, Sunil
On 3/28/2024 8:38 AM, Alex Deucher wrote: On Tue, Mar 26, 2024 at 1:31 PM Sunil Khatri wrote: Add FW information of all the IP's in the devcoredump. Signed-off-by: Sunil Khatri Might want to include the vbios version info as well, e.g., atom_context->name atom_context->vbios_pn

Re: [PATCH 09/11] drm/dp_mst: Add drm_dp_mst_aux_for_parent()

2024-03-27 Thread Nautiyal, Ankit K
On 3/27/2024 7:55 PM, Imre Deak wrote: On Wed, Mar 27, 2024 at 02:30:53PM +0530, Nautiyal, Ankit K wrote: On 3/21/2024 1:41 AM, Imre Deak wrote: Add a function to get the AUX device of the parent of an MST port, used by a follow-up i915 patch in the patchset. Cc: Lyude Paul Cc:

Re: [PATCH v6 13/14] drm/mediatek: Support CRC in OVL

2024-03-27 Thread 宋孝謙

Re: [PATCH] drm/amdgpu: add IP's FW information to devcoredump

2024-03-27 Thread Alex Deucher
On Tue, Mar 26, 2024 at 1:31 PM Sunil Khatri wrote: > > Add FW information of all the IP's in the devcoredump. > > Signed-off-by: Sunil Khatri Might want to include the vbios version info as well, e.g., atom_context->name atom_context->vbios_pn atom_context->vbios_ver_str atom_context->date

Re: [PATCH v6 11/14] drm/mediatek: Support "None" alpha blending in Mixer

2024-03-27 Thread 宋孝謙

[pull] amdgpu, amdkfd drm-fixes-6.9

2024-03-27 Thread Alex Deucher
Hi Dave, Sima, Fixes for 6.9. Mostly fixes for new IPs added in 6.9. The following changes since commit 4cece764965020c22cff7665b18a012006359095: Linux 6.9-rc1 (2024-03-24 14:10:05 -0700) are available in the Git repository at: https://gitlab.freedesktop.org/agd5f/linux.git

Re: [PATCH v6 07/14] drm/mediatek: Support alpha blending in display driver

2024-03-27 Thread 宋孝謙

Re: [PATCH v6 04/14] drm/mediatek: Add DRM_MODE_ROTATE_0 to rotation property

2024-03-27 Thread 宋孝謙

[PATCH] nouveau/uvmm: fix addr/range calcs for remap operations

2024-03-27 Thread Dave Airlie
From: Dave Airlie dEQP-VK.sparse_resources.image_rebind.2d_array.r64i.128_128_8 was causing a remap operation like the below. op_remap: prev: 003fffed 000f a5abd18a op_remap: next: op_remap: unmap: 003fffed 0010 0 op_map: map:

Re: [PATCH 3/5] drm/panel: st7703: Add GameForce Chi Panel Support

2024-03-27 Thread Jessica Zhang
On 3/25/2024 6:49 AM, Chris Morgan wrote: From: Chris Morgan The GameForce Chi is a handheld device with a 3.5" 640x480 ST7703 based display panel. Hi Chris, Acked-by: Jessica Zhang Thanks, Jessica Zhang Signed-off-by: Chris Morgan ---

Re: [PATCH] drm/prime: Unbreak virtgpu dma-buf export

2024-03-27 Thread Dominik Behr
It is mapped, via iommu interface (vfio-iommu) when map_dma is called. -- Dominik On Mon, Mar 25, 2024 at 7:38 PM Rob Clark wrote: > > This is actually a bit concerning.. importing a host page backed > buffer without guest mapping into a passthru device probably doesn't > work and should be

Re: [PATCH 4/4] drm/tiny: st7586: drop driver owner assignment

2024-03-27 Thread David Lechner
On 3/27/24 12:48 PM, Krzysztof Kozlowski wrote: > Core in spi_register_driver() already sets the .owner, so driver > does not need to. > > Signed-off-by: Krzysztof Kozlowski > --- Acked-by: David Lechner

Re: [PATCH 1/4] drm/tiny: ili9225: drop driver owner assignment

2024-03-27 Thread David Lechner
On 3/27/24 12:48 PM, Krzysztof Kozlowski wrote: > Core in spi_register_driver() already sets the .owner, so driver > does not need to. > > Signed-off-by: Krzysztof Kozlowski > --- Acked-by: David Lechner

Re: [PATCH v7 2/3] drm/i915/gt: Do not generate the command streamer for all the CCS

2024-03-27 Thread Matt Roper
On Wed, Mar 27, 2024 at 04:56:18PM +0100, Andi Shyti wrote: > We want a fixed load CCS balancing consisting in all slices > sharing one single user engine. For this reason do not create the > intel_engine_cs structure with its dedicated command streamer for > CCS slices beyond the first. > >

[RFC PATCH v3 6/6] drm/amd/display: remove redundant freesync parser for DP

2024-03-27 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 | 86

[RFC PATCH v3 4/6] drm/amd/display: switch to setting physical address directly

2024-03-27 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 | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[RFC PATCH v3 5/6] drm/amd/display: always call connector_update when parsing freesync_caps

2024-03-27 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

[RFC PATCH v3 3/6] drm/amd/display: switch amdgpu_dm_connector to use struct drm_edid

2024-03-27 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

[RFC PATCH v3 2/6] drm/amd/display: clean unused variables for hdmi freesync parser

2024-03-27 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 05ffe844b1fc..280562707cd0 100644 ---

[RFC PATCH v3 1/6] drm/amd/display: remove unused pixel_clock_mhz from amdgpu_dm_connector

2024-03-27 Thread Melissa Wen
We set pixel_clock_mhz when updating freesync caps, but it's not used anywhere. Remove it. Signed-off-by: Melissa Wen --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 3 --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.h | 1 - 2 files changed, 4 deletions(-) diff --git

[RFC PATCH v3 0/6] drm/amd/display: switch amdgpu_dm_connector to use struct drm_edid

2024-03-27 Thread Melissa Wen
Hi, After finding a null-pointer dereference when running igt@kms_connector_force_edid (fixed by [1]), I started migrating raw edid handles in amdgpu connectors to use opaque drm_edid and its helpers. However, I'm struggling to remove all drm_edid_raw() occurrences for different reasons and,

Re: [PATCH] drm/i915/gt: Limit the reserved VM space to only the platforms that need it

2024-03-27 Thread Andi Shyti
Hi, On Wed, Mar 27, 2024 at 09:05:46PM +0100, Andi Shyti wrote: > Commit 9bb66c179f50 ("drm/i915: Reserve some kernel space per > vm") reduces the available VM space of one page in order to apply > Wa_16018031267 and Wa_16018063123. > > This page was reserved indiscrimitely in all platforms even

[PATCH v2 1/3] arch: Select fbdev helpers with CONFIG_VIDEO

2024-03-27 Thread Thomas Zimmermann
Various Kconfig options selected the per-architecture helpers for fbdev. But none of the contained code depends on fbdev. Standardize on CONFIG_VIDEO, which will allow to add more general helpers for video functionality. CONFIG_VIDEO protects each architecture's video/ directory. This allows for

[PATCH v2 3/3] arch: Rename fbdev header and source files

2024-03-27 Thread Thomas Zimmermann
The per-architecture fbdev code has no dependencies on fbdev and can be used for any video-related subsystem. Rename the files to 'video'. Use video-sti.c on parisc as the source file depends on CONFIG_STI_CORE. Further update all includes statements, includ guards, and Makefiles. Also update a

[PATCH v2 2/3] arch: Remove struct fb_info from video helpers

2024-03-27 Thread Thomas Zimmermann
The per-architecture video helpers do not depend on struct fb_info or anything else from fbdev. Remove it from the interface and replace fb_is_primary_device() with video_is_primary_device(). The new helper is similar in functionality, but can operate on non-fbdev devices. Signed-off-by: Thomas

[PATCH v2 0/3] arch: Remove fbdev dependency from video helpers

2024-03-27 Thread Thomas Zimmermann
Make architecture helpers for display functionality depend on general video functionality instead of fbdev. This avoids the dependency on fbdev and makes the functionality available for non-fbdev code. Patch 1 replaces the variety of Kconfig options that control the Makefiles with CONFIG_VIDEO.

Re: [PATCH] drm/ast: Fix soft lockup

2024-03-27 Thread Thomas Zimmermann
Hi Am 27.03.24 um 09:53 schrieb Jocelyn Falempe: Hi, Thanks for your patch. I'm wondering how you can trigger this infinite loop ? Yeah, a bit more context for this bug would be welcome. It's hard to judge the fix without. Best regards Thomas Also this looks like a simple fix, that can

[PATCH] drm/i915/gt: Limit the reserved VM space to only the platforms that need it

2024-03-27 Thread Andi Shyti
Commit 9bb66c179f50 ("drm/i915: Reserve some kernel space per vm") reduces the available VM space of one page in order to apply Wa_16018031267 and Wa_16018063123. This page was reserved indiscrimitely in all platforms even when not needed. Limit it to DG2 onwards. Fixes: 9bb66c179f50 ("drm/i915:

Re: [PATCH v2 12/14] sh: Add support for suppressing warning backtraces

2024-03-27 Thread Simon Horman
On Wed, Mar 27, 2024 at 08:10:51AM -0700, Guenter Roeck wrote: > On 3/27/24 07:44, Simon Horman wrote: > > On Mon, Mar 25, 2024 at 10:52:46AM -0700, Guenter Roeck wrote: > > > Add name of functions triggering warning backtraces to the __bug_table > > > object section to enable support for

Re: [PATCH v9 3/6] iio: core: Add new DMABUF interface infrastructure

2024-03-27 Thread Paul Cercueil
Hi Andrew, Le mercredi 27 mars 2024 à 12:21 -0500, Andrew Davis a écrit : > On 3/10/24 7:48 AM, Paul Cercueil wrote: > > Add the necessary infrastructure to the IIO core to support a new > > optional DMABUF based interface. > > > > With this new interface, DMABUF objects (externally created) can

Re: [PATCH v9 3/6] iio: core: Add new DMABUF interface infrastructure

2024-03-27 Thread Andrew Davis
On 3/10/24 7:48 AM, Paul Cercueil wrote: Add the necessary infrastructure to the IIO core to support a new optional DMABUF based interface. With this new interface, DMABUF objects (externally created) can be attached to a IIO buffer, and subsequently used for data transfer. A userspace

Re: [PATCH] vmwgfx: Create debugfs ttm_resource_manager entry only if needed

2024-03-27 Thread Jocelyn Falempe
I just pushed it to drm-misc-fixes. Thanks for your reviews, -- Jocelyn On 14/03/2024 09:14, Jocelyn Falempe wrote: On 13/03/2024 18:57, Zack Rusin wrote: On Tue, Mar 12, 2024 at 5:36 AM Jocelyn Falempe wrote: [...] Thanks! That looks great. I can push it through drm-misc-fixes.

Re: [PATCH 1/2] drm/qxl: remove unused `count` variable from `qxl_surface_id_alloc()`

2024-03-27 Thread Miguel Ojeda
On Wed, Mar 27, 2024 at 6:56 PM Miguel Ojeda wrote: > > Closes: > https://lore.kernel.org/lkml/caniq72mjc5t4n25sqvysroehxxpxypz4ppznesjhenc3qap...@mail.gmail.com/ Should have a [1] at the end. > Signed-off-by: Miguel Ojeda > --- > Given there is a loop going on here, it would be good to

Re: drivers/gpu/drm/qxl/qxl_cmd.c:424:6: error: variable 'count' set but not used

2024-03-27 Thread Miguel Ojeda
On Wed, Mar 27, 2024 at 3:43 PM Miguel Ojeda wrote: > > Will do -- I found another one when running the CI with the above one fixed: > > drivers/gpu/drm/qxl/qxl_ioctl.c:148:14: error: variable > 'num_relocs' set but not used [-Werror,-Wunused-but-set-variable] > > I will send you one for that

[PATCH 2/2] drm/qxl: remove unused variable from `qxl_process_single_command()`

2024-03-27 Thread Miguel Ojeda
Clang 14 in an (essentially) defconfig loongarch64 build for next-20240327 reports [1]: drivers/gpu/drm/qxl/qxl_ioctl.c:148:14: error: variable 'num_relocs' set but not used [-Werror,-Wunused-but-set-variable] The variable was originally used in the `out_free_bos` label, but commit

[PATCH 1/2] drm/qxl: remove unused `count` variable from `qxl_surface_id_alloc()`

2024-03-27 Thread Miguel Ojeda
Clang 14 in an (essentially) defconfig loongarch64 build for next-20240326 reports [1]: drivers/gpu/drm/qxl/qxl_cmd.c:424:6: error: variable 'count' set but not used [-Werror,-Wunused-but-set-variable] The variable is already unused in the version that got into the tree. Thus remove the

[PATCH 4/4] drm/tiny: st7586: drop driver owner assignment

2024-03-27 Thread Krzysztof Kozlowski
Core in spi_register_driver() already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- drivers/gpu/drm/tiny/st7586.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/tiny/st7586.c b/drivers/gpu/drm/tiny/st7586.c index

[PATCH 3/4] drm/tiny: panel-mipi-dbi: drop driver owner assignment

2024-03-27 Thread Krzysztof Kozlowski
Core in spi_register_driver() already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- drivers/gpu/drm/tiny/panel-mipi-dbi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/tiny/panel-mipi-dbi.c b/drivers/gpu/drm/tiny/panel-mipi-dbi.c index

[PATCH 2/4] drm/tiny: mi0283qt: drop driver owner assignment

2024-03-27 Thread Krzysztof Kozlowski
Core in spi_register_driver() already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- drivers/gpu/drm/tiny/mi0283qt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/tiny/mi0283qt.c b/drivers/gpu/drm/tiny/mi0283qt.c index

[PATCH 1/4] drm/tiny: ili9225: drop driver owner assignment

2024-03-27 Thread Krzysztof Kozlowski
Core in spi_register_driver() already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- drivers/gpu/drm/tiny/ili9225.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/tiny/ili9225.c b/drivers/gpu/drm/tiny/ili9225.c index

[PATCH] video: backlight: otm3225a: drop driver owner assignment

2024-03-27 Thread Krzysztof Kozlowski
Core in spi_register_driver() already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- drivers/video/backlight/otm3225a.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/video/backlight/otm3225a.c b/drivers/video/backlight/otm3225a.c index

Re: [PATCH v2] dt-bindings: display: bridge: it6505: Add #sound-dai-cells

2024-03-27 Thread Rob Herring
On Wed, 27 Mar 2024 16:52:48 +0800, Chen-Yu Tsai wrote: > The ITE IT6505 display bridge can take one I2S input and transmit it > over the DisplayPort link. > > Add #sound-dai-cells (= 0) to the binding for it. > > Signed-off-by: Chen-Yu Tsai > --- > Changes since v1 [1]: > - Reference

[RFC PATCH v2 6/6] drm/amd/display: remove redundant freesync parser for DP

2024-03-27 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 | 86

[RFC PATCH v2 4/6] drm/amd/display: switch to setting physical address directly

2024-03-27 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 | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[RFC PATCH v2 5/6] drm/amd/display: always call connector_update when parsing freesync_caps

2024-03-27 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

[RFC PATCH v2 3/6] drm/amd/display: switch amdgpu_dm_connector to use struct drm_edid

2024-03-27 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

[RFC PATCH v2 2/6] drm/amd/display: clean unused variables for hdmi freesync parser

2024-03-27 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 05ffe844b1fc..280562707cd0 100644 ---

[RFC PATCH v2 0/6] switch amdgpu_dm_connector to use struct drm_edid

2024-03-27 Thread Melissa Wen
Hi, After finding a null-pointer dereference when running igt@kms_connector_force_edid (fixed by [1]), I started migrating raw edid handles in amdgpu connectors to use opaque drm_edid and its helpers. However, I'm struggling to remove all drm_edid_raw() occurrences for different reasons and,

[RFC PATCH v2 1/6] drm/amd/display: remove unused pixel_clock_mhz from amdgpu_dm_connector

2024-03-27 Thread Melissa Wen
We set pixel_clock_mhz when updating freesync caps, but it's not used anywhere. Remove it. Signed-off-by: Melissa Wen --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 3 --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.h | 1 - 2 files changed, 4 deletions(-) diff --git

Re: [RESEND v3 2/2] drm: Add CONFIG_DRM_WERROR

2024-03-27 Thread Nathan Chancellor
On Wed, Mar 27, 2024 at 09:59:01AM +0200, Jani Nikula wrote: > On Wed, 27 Mar 2024, Maxime Ripard wrote: > > On Tue, Mar 26, 2024 at 03:56:50PM -0700, Nathan Chancellor wrote: > >> On Tue, Mar 05, 2024 at 11:07:36AM +0200, Jani Nikula wrote: > >> > Add kconfig to enable -Werror subsystem wide.

Re: [PATCH 1/4] dt-bindings: display: bridge: add the Hot-plug MIPI DSI connector

2024-03-27 Thread Rob Herring
On Tue, Mar 26, 2024 at 05:28:11PM +0100, Luca Ceresoli wrote: > Add bindings for a physical, hot-pluggable connector allowing the far end > of a MIPI DSI bus to be connected and disconnected at runtime. > > Signed-off-by: Luca Ceresoli > --- > .../bridge/hotplug-video-connector-dsi.yaml

Re: [PATCH 4/4] drm/bridge: hotplug-bridge: add driver to support hot-pluggable DSI bridges

2024-03-27 Thread Luca Ceresoli
Hi Maxime, On Wed, 27 Mar 2024 13:42:40 +0100 Maxime Ripard wrote: > On Tue, Mar 26, 2024 at 05:28:14PM +0100, Luca Ceresoli wrote: > > This driver implements the point of a DRM pipeline where a connector allows > > removal of all the following bridges up to the panel. > > > > The DRM

[PATCH v7 3/3] drm/i915/gt: Enable only one CCS for compute workload

2024-03-27 Thread Andi Shyti
Enable only one CCS engine by default with all the compute sices allocated to it. While generating the list of UABI engines to be exposed to the user, exclude any additional CCS engines beyond the first instance. This change can be tested with igt i915_query. Fixes: d2eae8e98d59 ("drm/i915/dg2:

[PATCH v7 2/3] drm/i915/gt: Do not generate the command streamer for all the CCS

2024-03-27 Thread Andi Shyti
We want a fixed load CCS balancing consisting in all slices sharing one single user engine. For this reason do not create the intel_engine_cs structure with its dedicated command streamer for CCS slices beyond the first. Fixes: d2eae8e98d59 ("drm/i915/dg2: Drop force_probe requirement")

[PATCH v7 1/3] drm/i915/gt: Disable HW load balancing for CCS

2024-03-27 Thread Andi Shyti
The hardware should not dynamically balance the load between CCS engines. Wa_14019159160 recommends disabling it across all platforms. Fixes: d2eae8e98d59 ("drm/i915/dg2: Drop force_probe requirement") Signed-off-by: Andi Shyti Cc: Chris Wilson Cc: Joonas Lahtinen Cc: Matt Roper Cc: # v6.2+

[PATCH v7 0/3] Disable automatic load CCS load balancing

2024-03-27 Thread Andi Shyti
Hi, this series does basically two things: 1. Disables automatic load balancing as adviced by the hardware workaround. 2. Assigns all the CCS slices to one single user engine. The user will then be able to query only one CCS engine >From v5 I have created a new file,

Re: [PATCH 18/22] nvdimm: virtio_pmem: drop owner assignment

2024-03-27 Thread Dave Jiang
On 3/27/24 5:41 AM, Krzysztof Kozlowski wrote: > virtio core already sets the .owner, so driver does not need to. > > Signed-off-by: Krzysztof Kozlowski Acked-by: Dave Jiang > > --- > > Depends on the first patch. > --- > drivers/nvdimm/virtio_pmem.c | 1 - > 1 file changed, 1

[linux-next:master] BUILD REGRESSION 26074e1be23143b2388cacb36166766c235feb7c

2024-03-27 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master branch HEAD: 26074e1be23143b2388cacb36166766c235feb7c Add linux-next specific files for 20240327 Error/Warning reports: https://lore.kernel.org/oe-kbuild-all/202403271905.bybgjipi-...@intel.com https

Re: [PATCH v3 3/3] drm/panthor: Drop the dev_enter/exit() sections in _irq_suspend/resume()

2024-03-27 Thread Steven Price
On 26/03/2024 11:12, Boris Brezillon wrote: > There's no reason for _irq_suspend/resume() to be called after the > device has been unplugged, and keeping this dev_enter/exit() > section in _irq_suspend() is turns _irq_suspend() into a NOP > when called from the _unplug() functions, which we don't

Re: [PATCH v2 12/14] sh: Add support for suppressing warning backtraces

2024-03-27 Thread Guenter Roeck
On 3/27/24 07:44, Simon Horman wrote: On Mon, Mar 25, 2024 at 10:52:46AM -0700, Guenter Roeck wrote: Add name of functions triggering warning backtraces to the __bug_table object section to enable support for suppressing WARNING backtraces. To limit image size impact, the pointer to the

Re: [PATCH v5 04/16] drm/vkms: Add typedef and documentation for pixel_read and pixel_write functions

2024-03-27 Thread Maíra Canal
On 3/26/24 12:56, Louis Chauvet wrote: Le 25/03/24 - 10:56, Maíra Canal a écrit : On 3/13/24 14:44, Louis Chauvet wrote: Introduce two typedefs: pixel_read_t and pixel_write_t. It allows the compiler to check if the passed functions take the correct arguments. Such typedefs will help ensuring

Re: [PATCH v2] backlight: lp8788: Drop support for platform data

2024-03-27 Thread Lee Jones
On Thu, 14 Mar 2024, Uwe Kleine-König wrote: > The backlight driver supports getting passed platform data. However this > isn't used. This allows to remove quite some dead code from the driver > because bl->pdata is always NULL, and so bl->mode is always > LP8788_BL_REGISTER_ONLY. > >

Re: [PATCH 01/11] drm/i915/dp: Fix DSC line buffer depth programming

2024-03-27 Thread Imre Deak
On Tue, Mar 26, 2024 at 12:50:17PM -0700, Manasi Navare wrote: Hi, > Hi Imre, > > Thanks for the DSC fixes. > > Would the line buf depth calculation that was getting set to 0 impact > DSC on all platforms or was this issue only specific to MTL and was > getting set correctly with older

Re: [PATCH v2 12/14] sh: Add support for suppressing warning backtraces

2024-03-27 Thread Simon Horman
On Mon, Mar 25, 2024 at 10:52:46AM -0700, Guenter Roeck wrote: > Add name of functions triggering warning backtraces to the __bug_table > object section to enable support for suppressing WARNING backtraces. > > To limit image size impact, the pointer to the function name is only added > to the

Re: drivers/gpu/drm/qxl/qxl_cmd.c:424:6: error: variable 'count' set but not used

2024-03-27 Thread Miguel Ojeda
On Wed, Mar 27, 2024 at 8:59 AM Maxime Ripard wrote: > > It looks reasonable to me, can you send a formal patch? Will do -- I found another one when running the CI with the above one fixed: drivers/gpu/drm/qxl/qxl_ioctl.c:148:14: error: variable 'num_relocs' set but not used

Re: [PATCH v2 0/6] backlight: Remove struct backlight_properties.fb_blank

2024-03-27 Thread Lee Jones
On Wed, 27 Mar 2024, Lee Jones wrote: > On Tue, 19 Mar 2024 10:37:19 +0100, Thomas Zimmermann wrote: > > The field fb_blank in struct backlight_properties has been marked for > > removal. Remove it in favor of the power and state fields. > > > > Patches 1 to 5 prepare several backlight drivers.

Re: [PATCH v2 0/6] backlight: Remove struct backlight_properties.fb_blank

2024-03-27 Thread Lee Jones
On Tue, 19 Mar 2024 10:37:19 +0100, Thomas Zimmermann wrote: > The field fb_blank in struct backlight_properties has been marked for > removal. Remove it in favor of the power and state fields. > > Patches 1 to 5 prepare several backlight drivers. They remove fb_blank > or replace driver code

Re: [PATCH 09/11] drm/dp_mst: Add drm_dp_mst_aux_for_parent()

2024-03-27 Thread Imre Deak
On Wed, Mar 27, 2024 at 02:30:53PM +0530, Nautiyal, Ankit K wrote: > > On 3/21/2024 1:41 AM, Imre Deak wrote: > > Add a function to get the AUX device of the parent of an MST port, used > > by a follow-up i915 patch in the patchset. > > > > Cc: Lyude Paul > > Cc: dri-devel@lists.freedesktop.org

Re: [PATCH v5 11/16] drm/vkms: Add YUV support

2024-03-27 Thread Pekka Paalanen
On Wed, 13 Mar 2024 18:45:05 +0100 Louis Chauvet wrote: > From: Arthur Grillo > > Add support to the YUV formats bellow: > > - NV12/NV16/NV24 > - NV21/NV61/NV42 > - YUV420/YUV422/YUV444 > - YVU420/YVU422/YVU444 > > The conversion from yuv to rgb is done with fixed-point arithmetic, using >

Re: [PATCH 02/22] um: virt-pci: drop owner assignment

2024-03-27 Thread Johannes Berg
On Wed, 2024-03-27 at 13:40 +0100, Krzysztof Kozlowski wrote: > virtio core already sets the .owner, so driver does not need to. > All further patches depend on the first virtio patch, therefore please ack > and this should go via one tree: virtio? Sure. Though it's not really actually

Re: [PATCH 02/22] um: virt-pci: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
On 27/03/2024 14:34, Johannes Berg wrote: > On Wed, 2024-03-27 at 13:40 +0100, Krzysztof Kozlowski wrote: >> virtio core already sets the .owner, so driver does not need to. > >> All further patches depend on the first virtio patch, therefore please ack >> and this should go via one tree: virtio?

Re: [PATCH 1/1] drm/qxl: fixes qxl_fence_wait

2024-03-27 Thread Maxime Ripard
Hi, On Wed, Mar 20, 2024 at 04:25:48PM +0100, Linux regression tracking (Thorsten Leemhuis) wrote: > On 08.03.24 02:08, Alex Constantino wrote: > > Fix OOM scenario by doing multiple notifications to the OOM handler through > > a busy wait logic. > > Changes from commit 5a838e5d5825 ("drm/qxl:

Re: [PATCH] MAINTAINERS: Add myself as maintainer for Xilinx DRM drivers

2024-03-27 Thread Laurent Pinchart
Pinchart > +M: Tomi Valkeinen > L: dri-devel@lists.freedesktop.org > S: Maintained > T: git git://anongit.freedesktop.org/drm/drm-misc > > --- > base-commit: e8f897f4afef0031fe618a8e94127a0934896aba > change-id: 20240327-xilinx-maintainer-f6020f6cba4d -- Regards, Laurent Pinchart

Re: [PATCH 17/22] wireless: mac80211_hwsim: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
On 27/03/2024 13:55, Kalle Valo wrote: > Krzysztof Kozlowski writes: > >> virtio core already sets the .owner, so driver does not need to. >> >> Signed-off-by: Krzysztof Kozlowski > > We use "wifi:" in the title, not "wireless:". It would be nice if you > can fix this during commit. Sure.

[PATCH 22/22] sound: virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- sound/virtio/virtio_card.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/virtio/virtio_card.c b/sound/virtio/virtio_card.c index

[PATCH 21/22] fuse: virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- fs/fuse/virtio_fs.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/fuse/virtio_fs.c b/fs/fuse/virtio_fs.c index 322af827a232..ca7b64f9c3c7 100644 ---

[PATCH 20/22] scsi: virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/scsi/virtio_scsi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/scsi/virtio_scsi.c b/drivers/scsi/virtio_scsi.c index

[PATCH] MAINTAINERS: Add myself as maintainer for Xilinx DRM drivers

2024-03-27 Thread Tomi Valkeinen
:drivers/gpu/drm/xen/ DRM DRIVERS FOR XILINX M: Laurent Pinchart +M: Tomi Valkeinen L: dri-devel@lists.freedesktop.org S: Maintained T: git git://anongit.freedesktop.org/drm/drm-misc --- base-commit: e8f897f4afef0031fe618a8e94127a0934896aba change-id: 20240327

Re: [PATCH v5 11/16] drm/vkms: Add YUV support

2024-03-27 Thread Pekka Paalanen
On Tue, 26 Mar 2024 16:57:03 +0100 Louis Chauvet wrote: > Le 25/03/24 - 11:26, Maíra Canal a écrit : > > On 3/13/24 14:45, Louis Chauvet wrote: > > > From: Arthur Grillo > > > > > > Add support to the YUV formats bellow: > > > > > > - NV12/NV16/NV24 > > > - NV21/NV61/NV42 > > > -

Re: [PATCH 17/22] wireless: mac80211_hwsim: drop owner assignment

2024-03-27 Thread Kalle Valo
Krzysztof Kozlowski writes: > virtio core already sets the .owner, so driver does not need to. > > Signed-off-by: Krzysztof Kozlowski We use "wifi:" in the title, not "wireless:". It would be nice if you can fix this during commit. -- https://patchwork.kernel.org/project/linux-wireless/list/

[PATCH 19/22] rpmsg: virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/rpmsg/virtio_rpmsg_bus.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/rpmsg/virtio_rpmsg_bus.c b/drivers/rpmsg/virtio_rpmsg_bus.c index

[PATCH 18/22] nvdimm: virtio_pmem: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/nvdimm/virtio_pmem.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/nvdimm/virtio_pmem.c b/drivers/nvdimm/virtio_pmem.c index

[PATCH 17/22] wireless: mac80211_hwsim: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/net/wireless/virtual/mac80211_hwsim.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/wireless/virtual/mac80211_hwsim.c

[PATCH 16/22] net: vmw_vsock: virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- net/vmw_vsock/virtio_transport.c | 1 - 1 file changed, 1 deletion(-) diff --git a/net/vmw_vsock/virtio_transport.c b/net/vmw_vsock/virtio_transport.c index

[PATCH 15/22] net: 9p: virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- net/9p/trans_virtio.c | 1 - 1 file changed, 1 deletion(-) diff --git a/net/9p/trans_virtio.c b/net/9p/trans_virtio.c index e305071eb7b8..0b8086f58ad5

[PATCH 14/22] net: virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/net/virtio_net.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c index

[PATCH 13/22] net: caif: virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/net/caif/caif_virtio.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/caif/caif_virtio.c b/drivers/net/caif/caif_virtio.c index

[PATCH 12/22] misc: nsm: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/misc/nsm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/misc/nsm.c b/drivers/misc/nsm.c index 0eaa3b4484bd..ef7b32742340 100644 ---

[PATCH 11/22] iommu: virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/iommu/virtio-iommu.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/iommu/virtio-iommu.c b/drivers/iommu/virtio-iommu.c index

[PATCH 10/22] drm/virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/gpu/drm/virtio/virtgpu_drv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/virtio/virtgpu_drv.c

[PATCH 09/22] gpio: virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/gpio/gpio-virtio.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpio/gpio-virtio.c b/drivers/gpio/gpio-virtio.c index

[PATCH 08/22] firmware: arm_scmi: virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/firmware/arm_scmi/virtio.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/firmware/arm_scmi/virtio.c

[PATCH 07/22] crypto: virtio - drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/crypto/virtio/virtio_crypto_core.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/crypto/virtio/virtio_crypto_core.c

[PATCH 06/22] virtio_console: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/char/virtio_console.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/char/virtio_console.c b/drivers/char/virtio_console.c index

[PATCH 05/22] hwrng: virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/char/hw_random/virtio-rng.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/char/hw_random/virtio-rng.c

[PATCH 04/22] bluetooth: virtio: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/bluetooth/virtio_bt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/bluetooth/virtio_bt.c b/drivers/bluetooth/virtio_bt.c index

Re: [PATCH 4/4] drm/bridge: hotplug-bridge: add driver to support hot-pluggable DSI bridges

2024-03-27 Thread Maxime Ripard
On Tue, Mar 26, 2024 at 05:28:14PM +0100, Luca Ceresoli wrote: > This driver implements the point of a DRM pipeline where a connector allows > removal of all the following bridges up to the panel. > > The DRM subsystem currently allows hotplug of the monitor but not preceding > components.

[PATCH 03/22] virtio_blk: drop owner assignment

2024-03-27 Thread Krzysztof Kozlowski
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/block/virtio_blk.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/block/virtio_blk.c b/drivers/block/virtio_blk.c index

  1   2   3   4   5   >