Re: [PATCH v2 2/2] drm/i915: Never return 0 if not all requests retired

2022-11-21 Thread Janusz Krzysztofik
Hi Nimroy, Thanks for looking at this. On Friday, 18 November 2022 20:56:50 CET Das, Nirmoy wrote: > > On 11/18/2022 11:42 AM, Janusz Krzysztofik wrote: > > Users of intel_gt_retire_requests_timeout() expect 0 return value on > > success. However, we have no protection from passing back 0 poten

Re: [PATCH v4 00/10] Initial support for Cadence MHDP(HDMI/DP) for i.MX8MQ

2022-11-21 Thread Alexander Stein
Hello Sandor, thanks for the updated series. Am Montag, 21. November 2022, 08:23:50 CET schrieb Sandor Yu: > The patch set initial support for Cadence MHDP(HDMI/DP) DRM bridge > drivers and Cadence HDP-TX PHY(HDMI/DP) drivers for iMX8MQ. > > The patch set compose of DRM bridge drivers and PHY dr

Re: [PATCH v2 1/2] drm/i915: Fix negative value passed as remaining time

2022-11-21 Thread Janusz Krzysztofik
On Friday, 18 November 2022 11:42:21 CET Janusz Krzysztofik wrote: > Commit b97060a99b01 ("drm/i915/guc: Update intel_gt_wait_for_idle to work > with GuC") extended the API of intel_gt_retire_requests_timeout() with an > extra argument 'remaining_timeout', intended for passing back unconsumed > por

Re: [Intel-gfx] [PATCH] drm/i915: Fix workarounds on Gen2-3

2022-11-21 Thread Tvrtko Ursulin
On 18/11/2022 17:14, Matt Roper wrote: On Fri, Nov 18, 2022 at 11:52:49AM +, Tvrtko Ursulin wrote: From: Tvrtko Ursulin In 3653727560d0 ("drm/i915: Simplify internal helper function signature") I broke the old platforms by not noticing engine workaround init does not initialize the list

[PATCH v4 0/3] add color management support for the crtc

2022-11-21 Thread Kalyan Thota
Add color management support for the crtc provided there are enough dspps that can be allocated from the catalog Kalyan Thota (3): drm/msm/disp/dpu1: pin 1 crtc to 1 encoder drm/msm/disp/dpu1: add helper to know if display is builtin drm/msm/disp/dpu1: add color management support for the cr

[PATCH v4 2/3] drm/msm/disp/dpu1: add helper to know if display is builtin

2022-11-21 Thread Kalyan Thota
Since DRM encoder type for few encoders can be similar (like eDP and DP), get the connector type for a given encoder to differentiate between builtin and pluggable displays. Changes in v1: - add connector type in the disp_info (Dmitry) - add helper functions to know encoder type - update commit te

[PATCH v4 3/3] drm/msm/disp/dpu1: add color management support for the crtc

2022-11-21 Thread Kalyan Thota
Add color management support for the crtc provided there are enough dspps that can be allocated from the catalog. Changes in v1: - cache color enabled state in the dpu crtc obj (Dmitry) - simplify dspp allocation while creating crtc (Dmitry) - register for color when crtc is created (Dmitry) Chan

[PATCH v4 1/3] drm/msm/disp/dpu1: pin 1 crtc to 1 encoder

2022-11-21 Thread Kalyan Thota
Pin each crtc with one encoder. This arrangement will disallow crtc switching between encoders and also will facilitate to advertise certain features on crtc based on encoder type. Changes in v1: - use drm_for_each_encoder macro while iterating through encoder list (Dmitry) Changes in v2: - mak

[PATCH 3/5] kobject: kset_uevent_ops: make filter() callback take a const *

2022-11-21 Thread Greg Kroah-Hartman
The filter() callback in struct kset_uevent_ops does not modify the kobject passed into it, so make the pointer const to enforce this restriction. When doing so, fix up all existing filter() callbacks to have the correct signature to preserve the build. Cc: "Rafael J. Wysocki" Cc: Sumit Semwal

Re: [Linaro-mm-sig] [PATCH 3/5] kobject: kset_uevent_ops: make filter() callback take a const *

2022-11-21 Thread Christian König
Am 21.11.22 um 10:46 schrieb Greg Kroah-Hartman: The filter() callback in struct kset_uevent_ops does not modify the kobject passed into it, so make the pointer const to enforce this restriction. When doing so, fix up all existing filter() callbacks to have the correct signature to preserve the

Re: [PATCH AUTOSEL 6.0 38/44] drm/amdgpu: Unlock bo_list_mutex after error handling

2022-11-21 Thread Michel Dänzer
On 11/19/22 03:11, Sasha Levin wrote: > From: Philip Yang > > [ Upstream commit 64f65135c41a75f933d3bca236417ad8e9eb75de ] > > Get below kernel WARNING backtrace when pressing ctrl-C to kill kfdtest > application. > > If amdgpu_cs_parser_bos returns error after taking bo_list_mutex, as > caller

RE: [PATCH 585/606] backlight: adp8860: Convert to i2c's .probe_new()

2022-11-21 Thread Hennerich, Michael
> -Original Message- > From: Uwe Kleine-König > Sent: Freitag, 18. November 2022 23:45 > To: Angel Iglesias ; Lee Jones > ; Grant Likely ; Wolfram > Sang ; Hennerich, Michael > ; Lee Jones ; Daniel > Thompson ; Jingoo Han > ; Helge Deller > Cc: linux-...@vger.kernel.org; ker...@pengutro

Re: DRM-managed resources / devm_drm_dev_alloc leaking resources

2022-11-21 Thread Stanislaw Gruszka
Hi On Sat, Nov 19, 2022 at 12:14:14PM +0100, Daniel Vetter wrote: > On Fri, Nov 18, 2022 at 05:15:58PM +0100, Maxime Ripard wrote: > > Hi, > > > > Javier and I looked into it some more today, and I think we have a > > better idea of what is going on. > > > > On Thu, Nov 17, 2022 at 05:53:11PM +0

Re: [PATCH 585/606] backlight: adp8860: Convert to i2c's .probe_new()

2022-11-21 Thread Daniel Thompson
On Fri, Nov 18, 2022 at 11:45:19PM +0100, Uwe Kleine-König wrote: > From: Uwe Kleine-König > > .probe_new() doesn't get the i2c_device_id * parameter, so determine > that explicitly in the probe function. > > Signed-off-by: Uwe Kleine-König Reviewed-by: Daniel Thompson Daniel.

Re: [PATCH 586/606] backlight: adp8870: Convert to i2c's .probe_new()

2022-11-21 Thread Daniel Thompson
On Fri, Nov 18, 2022 at 11:45:20PM +0100, Uwe Kleine-König wrote: > From: Uwe Kleine-König > > .probe_new() doesn't get the i2c_device_id * parameter, so determine > that explicitly in the probe function. > > Signed-off-by: Uwe Kleine-König Reviewed-by: Daniel Thompson Daniel.

Re: [PATCH v2 2/2] drm/i915: Never return 0 if not all requests retired

2022-11-21 Thread Andrzej Hajda
On 21.11.2022 09:30, Janusz Krzysztofik wrote: Hi Nimroy, Thanks for looking at this. On Friday, 18 November 2022 20:56:50 CET Das, Nirmoy wrote: On 11/18/2022 11:42 AM, Janusz Krzysztofik wrote: Users of intel_gt_retire_requests_timeout() expect 0 return value on success. However, we hav

Re: The state of Quantization Range handling

2022-11-21 Thread Pekka Paalanen
On Fri, 18 Nov 2022 15:53:29 + Dave Stevenson wrote: > Hi Pekka > > On Fri, 18 Nov 2022 at 10:15, Pekka Paalanen wrote: > > > > On Thu, 17 Nov 2022 22:13:26 +0100 > > Sebastian Wick wrote: > > > > > Hi Dave, > > > > > > I noticed that I didn't get the Broadcast RGB property thanks to you

Re: [PATCH 587/606] backlight: arcxcnn: Convert to i2c's .probe_new()

2022-11-21 Thread Daniel Thompson
On Fri, Nov 18, 2022 at 11:45:21PM +0100, Uwe Kleine-König wrote: > From: Uwe Kleine-König > > The probe function doesn't make use of the i2c_device_id * parameter so it > can be trivially converted. > > Signed-off-by: Uwe Kleine-König Reviewed-by: Daniel Thompson Daniel.

[PATCH 0/8] drm/mipi-dbi: Convert to shadow-plane helpers

2022-11-21 Thread Thomas Zimmermann
Convert the MIPI-DBI-based drivers to shadow-plane helpers. The drivers vmap/vunmap GEM buffer memory during the atomic commit. Shadow-plane helpers automate this process. Patches 1 to 4 prepare the MIPI code for the change and simplify the restof the patchset. Patches 5 to 7 rework the vmap code

[PATCH 4/8] drm/mipi-dbi: Initialize default driver functions with macro

2022-11-21 Thread Thomas Zimmermann
Introduce DRM_MIPI_DBI_SIMPLE_DISPLAY_PIPE_FUNCS to initialize MIPI-DBI helpers to default values and convert drivers. The prepare_fb function set by some drivers is called implicitly by simple-kms helpers, so leave it out. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/panel/panel-ilitek-

[PATCH 5/8] drm/mipi-dbi: Prepare framebuffer copy operation in pipe-update helpers

2022-11-21 Thread Thomas Zimmermann
Move the vmap/vunmap blocks from the inner fb_dirty helpers into the MIPI DBI update helpers. The function calls can result in waiting and/or processing overhead. Reduce the penalties by executing the functions once in the outer-most function of the pipe update. This change also prepares for MIPI

[PATCH 1/8] drm/simple-kms: Remove drm_gem_simple_display_pipe_prepare_fb()

2022-11-21 Thread Thomas Zimmermann
The helper drm_gem_simple_display_pipe_prepare_fb() is simple-KMS' default implementation for prepare_fb. Remove the call from drivers that set it explicitly. Then inline the helper into the only caller within simple-kms helpers and remove . No functional changes. Simple-KMS drivers that implement

[PATCH 7/8] drm/mipi-dbi: Use shadow-plane mappings

2022-11-21 Thread Thomas Zimmermann
Use the buffer mappings provided by shadow-plane helpers. As the mappings are established while the commit can still fail, errors are now reported correctly to callers. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/drm_mipi_dbi.c | 31 +++ drivers/gpu/drm/tiny/

[PATCH 8/8] drm/mipi-dbi: Move drm_dev_{enter, exit}() out from fb_dirty functions

2022-11-21 Thread Thomas Zimmermann
Call drm_dev_enter() and drm_dev_exit() in the outer-most callbacks of the modesetting pipeline. If drm_dev_enter() fails, the driver can thus avoid unnecessary work. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/drm_mipi_dbi.c | 13 +++-- drivers/gpu/drm/tiny/ili9225.c | 13 +

[PATCH 2/8] drm/ili9225: Call MIPI DBI mode_valid helper

2022-11-21 Thread Thomas Zimmermann
MIPI DBI drivers validate each mode against their native resolution. Add this test to ili9225. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/tiny/ili9225.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/tiny/ili9225.c b/drivers/gpu/drm/tiny/ili9225.c index 815bab28582

[PATCH 3/8] drm/st7586: Call MIPI DBI mode_valid helper

2022-11-21 Thread Thomas Zimmermann
MIPI DBI drivers validate each mode against their native resolution. Add this test to st7586. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/tiny/st7586.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/tiny/st7586.c b/drivers/gpu/drm/tiny/st7586.c index ce57fa9917e51..

[PATCH 6/8] drm/mipi-dbi: Support shadow-plane state

2022-11-21 Thread Thomas Zimmermann
Introduce struct drm_mipi_dbi_plane_state that contains state related to MIPI DBI. It currently only inherits from struct drm_shadow_plane_state, so that MIPI DBI drivers can use the vmap'ed GEM-buffer memory. Implement state helpers, the {begin,end}_fb_access helpers and wire up everything. With

Re: [PATCH v2 2/2] drm/i915: Never return 0 if not all requests retired

2022-11-21 Thread Janusz Krzysztofik
Hi Andrzej, Thanks for your comment. On Monday, 21 November 2022 11:17:42 CET Andrzej Hajda wrote: > > On 21.11.2022 09:30, Janusz Krzysztofik wrote: > > Hi Nimroy, > > > > Thanks for looking at this. > > > > On Friday, 18 November 2022 20:56:50 CET Das, Nirmoy wrote: > >> On 11/18/2022 11:42 AM

Re: [PATCH v3 0/6] dt-bindings: Add macros for video interface bus types

2022-11-21 Thread Sakari Ailus
Hi Laurent, On Sat, Nov 19, 2022 at 09:15:04PM +0200, Laurent Pinchart wrote: > Hello, > > On Fri, Nov 18, 2022 at 06:23:38PM +0900, Paul Elder wrote: > > Hi Sakari, > > > > Gentle ping. > > > > On Sun, Jul 17, 2022 at 06:54:00AM +, Sakari Ailus wrote: > > > Folks, > > > > > > > Laurent Pi

Re: [PATCH v2 2/2] drm/i915: Never return 0 if not all requests retired

2022-11-21 Thread Janusz Krzysztofik
On Monday, 21 November 2022 11:51:15 CET Janusz Krzysztofik wrote: > Hi Andrzej, > > Thanks for your comment. > > On Monday, 21 November 2022 11:17:42 CET Andrzej Hajda wrote: > > > > On 21.11.2022 09:30, Janusz Krzysztofik wrote: > > > Hi Nimroy, > > > > > > Thanks for looking at this. > > > >

Re: [PATCH 588/606] backlight: bd6107: Convert to i2c's .probe_new()

2022-11-21 Thread Daniel Thompson
On Fri, Nov 18, 2022 at 11:45:22PM +0100, Uwe Kleine-König wrote: > From: Uwe Kleine-König > > The probe function doesn't make use of the i2c_device_id * parameter so it > can be trivially converted. > > Signed-off-by: Uwe Kleine-König Reviewed-by: Daniel Thompson Daniel.

Re: [PATCH 589/606] backlight: lm3630a: Convert to i2c's .probe_new()

2022-11-21 Thread Daniel Thompson
On Fri, Nov 18, 2022 at 11:45:23PM +0100, Uwe Kleine-König wrote: > From: Uwe Kleine-König > > The probe function doesn't make use of the i2c_device_id * parameter so it > can be trivially converted. > > Signed-off-by: Uwe Kleine-König Reviewed-by: Daniel Thompson Daniel.

Re: [PATCH v3 0/6] dt-bindings: Add macros for video interface bus types

2022-11-21 Thread Laurent Pinchart
Hi Sakari, On Mon, Nov 21, 2022 at 10:54:01AM +, Sakari Ailus wrote: > On Sat, Nov 19, 2022 at 09:15:04PM +0200, Laurent Pinchart wrote: > > On Fri, Nov 18, 2022 at 06:23:38PM +0900, Paul Elder wrote: > > > On Sun, Jul 17, 2022 at 06:54:00AM +, Sakari Ailus wrote: > > > > Folks, > > > > >

Re: [PATCH AUTOSEL 6.0 38/44] drm/amdgpu: Unlock bo_list_mutex after error handling

2022-11-21 Thread Christian König
Am 21.11.22 um 10:57 schrieb Michel Dänzer: On 11/19/22 03:11, Sasha Levin wrote: From: Philip Yang [ Upstream commit 64f65135c41a75f933d3bca236417ad8e9eb75de ] Get below kernel WARNING backtrace when pressing ctrl-C to kill kfdtest application. If amdgpu_cs_parser_bos returns error after ta

Re: [PATCH 590/606] backlight: lm3639: Convert to i2c's .probe_new()

2022-11-21 Thread Daniel Thompson
On Fri, Nov 18, 2022 at 11:45:24PM +0100, Uwe Kleine-König wrote: > From: Uwe Kleine-König > > The probe function doesn't make use of the i2c_device_id * parameter so it > can be trivially converted. > > Signed-off-by: Uwe Kleine-König Reviewed-by: Daniel Thompson Daniel.

Re: [PATCH 591/606] backlight: lp855x: Convert to i2c's .probe_new()

2022-11-21 Thread Daniel Thompson
On Fri, Nov 18, 2022 at 11:45:25PM +0100, Uwe Kleine-König wrote: > From: Uwe Kleine-König > > .probe_new() doesn't get the i2c_device_id * parameter, so determine > that explicitly in the probe function. > > Signed-off-by: Uwe Kleine-König Reviewed-by: Daniel Thompson Daniel.

Re: [PATCH 592/606] backlight: lv5207lp: Convert to i2c's .probe_new()

2022-11-21 Thread Daniel Thompson
On Fri, Nov 18, 2022 at 11:45:26PM +0100, Uwe Kleine-König wrote: > From: Uwe Kleine-König > > The probe function doesn't make use of the i2c_device_id * parameter so it > can be trivially converted. > > Signed-off-by: Uwe Kleine-König Reviewed-by: Daniel Thompson Daniel.

Re: [PATCH 593/606] backlight: tosa: Convert to i2c's .probe_new()

2022-11-21 Thread Daniel Thompson
On Fri, Nov 18, 2022 at 11:45:27PM +0100, Uwe Kleine-König wrote: > From: Uwe Kleine-König > > The probe function doesn't make use of the i2c_device_id * parameter so it > can be trivially converted. > > Signed-off-by: Uwe Kleine-König Reviewed-by: Daniel Thompson Daniel.

Re: [PATCH] drm/vc4: Fix NULL pointer access in vc4_platform_drm_probe()

2022-11-21 Thread Dave Stevenson
Hi Lino On Sun, 20 Nov 2022 at 15:26, Lino Sanfilippo wrote: > > From: Lino Sanfilippo > > In vc4_platform_drm_probe() function vc4_match_add_drivers() is called to > find component matches for the component drivers. If no such match is found > the passed variable "match" is still NULL after the

[PATCH] drm/i915/selftests: Fix error return code in live_parallel_switch()

2022-11-21 Thread Ziyang Xuan
In live_parallel_switch(), excute intel_context_create() failed, the error code should be set before return from error handling path. Fixes: 50d16d44cce4 ("drm/i915/selftests: Exercise context switching in parallel") Signed-off-by: Ziyang Xuan --- drivers/gpu/drm/i915/gem/selftests/i915_gem_con

git send-email friendly smtp provider anyone?

2022-11-21 Thread Noralf Trønnes
Hi A couple of years ago my email provider blocked me from using git send-email with their smtp server. So I switched to the one my ISP provides. Now my ISP have outsourced their email service so the first 3 emails gets through and the rest looks like it ends up in a tar pit or something, 18 hours

Re: [PATCH 3/5] kobject: kset_uevent_ops: make filter() callback take a const *

2022-11-21 Thread Rafael J. Wysocki
On Mon, Nov 21, 2022 at 10:47 AM Greg Kroah-Hartman wrote: > > The filter() callback in struct kset_uevent_ops does not modify the > kobject passed into it, so make the pointer const to enforce this > restriction. When doing so, fix up all existing filter() callbacks to > have the correct signatu

Re: [PATCH] i2c: qcom-geni: fix error return code in geni_i2c_gpi_xfer

2022-11-21 Thread Tommaso Merciai
Hi Wang, On Mon, Nov 21, 2022 at 06:17:52PM +0800, Wang Yufen wrote: > Fix to return a negative error code from the gi2c->err instead of > 0. > > Fixes: d8703554f4de ("i2c: qcom-geni: Add support for GPI DMA") > Signed-off-by: Wang Yufen > --- > drivers/i2c/busses/i2c-qcom-geni.c | 1 - > 1 fil

Re: [Intel-gfx] [PATCH v2 2/2] drm/i915: Never return 0 if not all requests retired

2022-11-21 Thread Andrzej Hajda
On 21.11.2022 11:59, Janusz Krzysztofik wrote: On Monday, 21 November 2022 11:51:15 CET Janusz Krzysztofik wrote: Hi Andrzej, Thanks for your comment. On Monday, 21 November 2022 11:17:42 CET Andrzej Hajda wrote: On 21.11.2022 09:30, Janusz Krzysztofik wrote: Hi Nimroy, Thanks for looking

Re: [PATCH 0/8] drm/mipi-dbi: Convert to shadow-plane helpers

2022-11-21 Thread Noralf Trønnes
Den 21.11.2022 11.45, skrev Thomas Zimmermann: > Convert the MIPI-DBI-based drivers to shadow-plane helpers. The > drivers vmap/vunmap GEM buffer memory during the atomic commit. > Shadow-plane helpers automate this process. > > Patches 1 to 4 prepare the MIPI code for the change and simplify >

Re: [PATCH 0/8] drm/mipi-dbi: Convert to shadow-plane helpers

2022-11-21 Thread Thomas Zimmermann
Hi Am 21.11.22 um 13:27 schrieb Noralf Trønnes: Den 21.11.2022 11.45, skrev Thomas Zimmermann: Convert the MIPI-DBI-based drivers to shadow-plane helpers. The drivers vmap/vunmap GEM buffer memory during the atomic commit. Shadow-plane helpers automate this process. Patches 1 to 4 prepare th

Re: [PATCH v3] drm/i915/mtl: Media GT and Render GT share common GGTT

2022-11-21 Thread Iddamsetty, Aravind
On 19-11-2022 04:22, Matt Roper wrote: > On Tue, Nov 15, 2022 at 08:34:54PM +0530, Aravind Iddamsetty wrote: >> On XE_LPM+ platforms the media engines are carved out into a separate >> GT but have a common GGTMMADR address range which essentially makes >> the GGTT address space to be shared betw

[PATCH] drm/amdgpu: fix unused-function error

2022-11-21 Thread Ren Zhijie
If CONFIG_DRM_AMDGPU=y and CONFIG_DRM_AMD_DC is not set, gcc complained about unused-function : drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c:1705:13: error: ‘amdgpu_discovery_set_sriov_display’ defined but not used [-Werror=unused-function] static void amdgpu_discovery_set_sriov_display(struct

Re: git send-email friendly smtp provider anyone?

2022-11-21 Thread Simon Ser
I think you can apply for a linux.dev mailbox [1]. [1]: https://korg.docs.kernel.org/linuxdev.html

[PATCH] gpu: vgpu: delete a semicolon

2022-11-21 Thread Li kunyu
Remove extra semicolons at the end of statements to make them look cleaner. Signed-off-by: Li kunyu --- drivers/gpu/drm/i915/gvt/vgpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/gvt/vgpu.c b/drivers/gpu/drm/i915/gvt/vgpu.c index 56c71474008a..5ccb04

[PATCH v2 0/2] Make ILI9486 driver working with 16-bits SPI controllers

2022-11-21 Thread Carlo Caione
This patchset is trying to fix problems seen on S905X boards when interfacing with an ILI9486 equipped SPI panel. To: Kamlesh Gurudasani To: David Airlie To: Daniel Vetter To: Mark Brown To: Neil Armstrong To: Kevin Hilman To: Jerome Brunet To: Martin Blumenstingl Cc: dri-devel@lists.freed

[PATCH v2 1/2] drm/tiny: rpi-lcd-35: Enable driver module autoloading

2022-11-21 Thread Carlo Caione
SPI devices use the spi_device_id for module autoloading even on systems using device tree. Add the spi_device_id entry to enable autoloading for the 3.5inch RPi Display (rpi-lcd-35). Signed-off-by: Carlo Caione --- drivers/gpu/drm/tiny/ili9486.c | 1 + 1 file changed, 1 insertion(+) diff --gi

[PATCH v2 2/2] drm/tiny: ili9486: Do not assume 8-bit only SPI controllers

2022-11-21 Thread Carlo Caione
The pixel data for the ILI9486 is always 16-bits wide and it must be sent over the SPI bus. When the controller is only able to deal with 8-bit transfers, this 16-bits data needs to be swapped before the sending to account for the big endian bus, this is on the contrary not needed when the SPI cont

[PATCH] i2c: qcom-geni: fix error return code in geni_i2c_gpi_xfer

2022-11-21 Thread Wang Yufen
Fix to return a negative error code from the gi2c->err instead of 0. Fixes: d8703554f4de ("i2c: qcom-geni: Add support for GPI DMA") Signed-off-by: Wang Yufen --- drivers/i2c/busses/i2c-qcom-geni.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/i2c/busses/i2c-qcom-geni.c b/drivers/i

[PATCH] drm/logicvc: Drop obsolete dependency on COMPILE_TEST

2022-11-21 Thread Jean Delvare
Since commit 0166dc11be91 ("of: make CONFIG_OF user selectable"), it is possible to test-build any driver which depends on OF on any architecture by explicitly selecting OF. Therefore depending on COMPILE_TEST as an alternative is no longer needed. It is actually better to always build such driver

Re: [PATCH v10 00/19] drm: Analog TV Improvements

2022-11-21 Thread Daniel Vetter
On Thu, Nov 17, 2022 at 10:28:43AM +0100, Maxime Ripard wrote: > Hi, > > Here's a series aiming at improving the command line named modes support, > and more importantly how we deal with all the analog TV variants. > > The named modes support were initially introduced to allow to specify the > an

[PATCH v3 1/2] drm/i915: Fix negative value passed as remaining time

2022-11-21 Thread Janusz Krzysztofik
Commit b97060a99b01 ("drm/i915/guc: Update intel_gt_wait_for_idle to work with GuC") extended the API of intel_gt_retire_requests_timeout() with an extra argument 'remaining_timeout', intended for passing back unconsumed portion of requested timeout when 0 (success) is returned. However, when requ

[PATCH v3 0/2] drm/i915: Fix timeout handling when retiring requests

2022-11-21 Thread Janusz Krzysztofik
Fixes for issues discovered via code review while working on https://gitlab.freedesktop.org/drm/intel/issues/7349. v3: PATCH 1: don't fail on any error passed back via remaining_timeout, PATCH 2: use conditional expression, more compact but also better reflecting intention standing behind

[PATCH v3 2/2] drm/i915: Never return 0 if not all requests retired

2022-11-21 Thread Janusz Krzysztofik
Users of intel_gt_retire_requests_timeout() expect 0 return value on success. However, we have no protection from passing back 0 potentially returned by a call to dma_fence_wait_timeout() when it succedes right after its timeout has expired. Replace 0 with -ETIME before potentially using the time

Re: [PATCH v4 0/4] new subsystem for compute accelerator devices

2022-11-21 Thread Thomas Zimmermann
Acked-by: Thomas Zimmermann Am 19.11.22 um 21:44 schrieb Oded Gabbay: This is the fourth (and hopefully last) version of the patch-set to add the new subsystem for compute accelerators. I removed the RFC headline as I believe it is now ready for merging. Compare to v3, this patch-set contains

Re: [PATCH v4 0/4] new subsystem for compute accelerator devices

2022-11-21 Thread Oded Gabbay
On Mon, Nov 21, 2022 at 8:26 AM Dave Airlie wrote: > > On Sun, 20 Nov 2022 at 06:44, Oded Gabbay wrote: > > > > This is the fourth (and hopefully last) version of the patch-set to add the > > new subsystem for compute accelerators. I removed the RFC headline as > > I believe it is now ready for m

Re: [PATCH v4 2/4] accel: add dedicated minor for accelerator devices

2022-11-21 Thread Oded Gabbay
On Sun, Nov 20, 2022 at 11:47 PM Jeffrey Hugo wrote: > > On 11/19/2022 1:44 PM, Oded Gabbay wrote: > > diff --git a/drivers/accel/drm_accel.c b/drivers/accel/drm_accel.c > > index fac6ad6ac28e..703d40c4ff45 100644 > > --- a/drivers/accel/drm_accel.c > > +++ b/drivers/accel/drm_accel.c > > @@ -8,14

Re: [PATCH 0/8] drm/mipi-dbi: Convert to shadow-plane helpers

2022-11-21 Thread Noralf Trønnes
Den 21.11.2022 13.41, skrev Thomas Zimmermann: > Hi > > Am 21.11.22 um 13:27 schrieb Noralf Trønnes: >> >> >> Den 21.11.2022 11.45, skrev Thomas Zimmermann: >>> Convert the MIPI-DBI-based drivers to shadow-plane helpers. The >>> drivers vmap/vunmap GEM buffer memory during the atomic commit. >>

Re: [PATCH v4 4/4] doc: add documentation for accel subsystem

2022-11-21 Thread Oded Gabbay
On Mon, Nov 21, 2022 at 12:02 AM Jeffrey Hugo wrote: > > On 11/19/2022 1:44 PM, Oded Gabbay wrote: > > Add an introduction section for the accel subsystem. Most of the > > relevant data is in the DRM documentation, so the introduction only > > presents the why of the new subsystem, how are the com

Re: git send-email friendly smtp provider anyone?

2022-11-21 Thread Maxime Ripard
On Mon, Nov 21, 2022 at 12:48:52PM +0100, Noralf Trønnes wrote: > A couple of years ago my email provider blocked me from using git > send-email with their smtp server. So I switched to the one my ISP > provides. Now my ISP have outsourced their email service so the first 3 > emails gets through an

Re: [PATCH 0/8] drm/mipi-dbi: Convert to shadow-plane helpers

2022-11-21 Thread Javier Martinez Canillas
On 11/21/22 13:27, Noralf Trønnes wrote: > > > Den 21.11.2022 11.45, skrev Thomas Zimmermann: >> Convert the MIPI-DBI-based drivers to shadow-plane helpers. The >> drivers vmap/vunmap GEM buffer memory during the atomic commit. >> Shadow-plane helpers automate this process. >> >> Patches 1 to 4 p

Re: [PATCH v4 4/4] doc: add documentation for accel subsystem

2022-11-21 Thread Jeffrey Hugo
On 11/21/2022 8:18 AM, Oded Gabbay wrote: On Mon, Nov 21, 2022 at 12:02 AM Jeffrey Hugo wrote: On 11/19/2022 1:44 PM, Oded Gabbay wrote: Add an introduction section for the accel subsystem. Most of the relevant data is in the DRM documentation, so the introduction only presents the why of the

Re: [PATCH v4] dma-buf: fix racing conflict of dma_heap_add()

2022-11-21 Thread Dawei Li
On Sat, Nov 05, 2022 at 12:05:36AM +0800, Dawei Li wrote: Hi Christian, May I have your opinion on this change? Thanks, Dawei > Racing conflict could be: > task A task B > list_for_each_entry > strcmp(h->name)) >list_for_each_entry >

[PATCH resend] drm/armada: Fix off-by-one error in armada_overlay_get_property()

2022-11-21 Thread Geert Uytterhoeven
As ffs() returns one more than the index of the first bit set (zero means no bits set), the color key mode value is shifted one position too much. Fix this by using FIELD_GET() instead. Fixes: c96103b6c49ff9a8 ("drm/armada: move colorkey properties into overlay plane state") Signed-off-by: Geert

Re: [PATCH v4 0/4] new subsystem for compute accelerator devices

2022-11-21 Thread Alex Deucher
On Sat, Nov 19, 2022 at 3:44 PM Oded Gabbay wrote: > > This is the fourth (and hopefully last) version of the patch-set to add the > new subsystem for compute accelerators. I removed the RFC headline as > I believe it is now ready for merging. > > Compare to v3, this patch-set contains one additio

Re: [PATCH v4 0/4] new subsystem for compute accelerator devices

2022-11-21 Thread Alex Deucher
On Mon, Nov 21, 2022 at 10:57 AM Alex Deucher wrote: > > On Sat, Nov 19, 2022 at 3:44 PM Oded Gabbay wrote: > > > > This is the fourth (and hopefully last) version of the patch-set to add the > > new subsystem for compute accelerators. I removed the RFC headline as > > I believe it is now ready f

[PATCH resend] drm: mxsfb: DRM_MXSFB should depend on ARCH_MXS || ARCH_MXC

2022-11-21 Thread Geert Uytterhoeven
Freescale/NXP i.MX LCDIF and eLCDIF LCD controllers are only present on Freescale/NXP i.MX SoCs. Hence add a dependency on ARCH_MXS || ARCH_MXC, to prevent asking the user about this driver when configuring a kernel without Freescale/NXP i.MX support. Fixes: 45d59d704080cc0c ("drm: Add new driver

[PATCH resend] drm: mxsfb: DRM_IMX_LCDIF should depend on ARCH_MXC

2022-11-21 Thread Geert Uytterhoeven
The Freescale/NXP i.MX LCDIFv3 LCD controller is only present on Freescale/NXP i.MX SoCs. Hence add a dependency on ARCH_MXC, to prevent asking the user about this driver when configuring a kernel without Freescale/NXP i.MX support. Fixes: 9db35bb349a0ef32 ("drm: lcdif: Add support for i.MX8MP LC

Re: [PATCH resend] drm/armada: Fix off-by-one error in armada_overlay_get_property()

2022-11-21 Thread Russell King (Oracle)
On Mon, Nov 21, 2022 at 04:43:14PM +0100, Geert Uytterhoeven wrote: > As ffs() returns one more than the index of the first bit set (zero > means no bits set), the color key mode value is shifted one position too > much. > > Fix this by using FIELD_GET() instead. Reviewed-by: Russell King (Oracle

[Bug 216716] [drm:psp_resume [amdgpu]] *ERROR* PSP resume failed on r9 7950x igpu

2022-11-21 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=216716 Artem S. Tashkinov (a...@gmx.com) changed: What|Removed |Added Status|NEW |RESOLVED Reso

Re: [PATCH] drm: Fix potential null-ptr-deref due to drmm_mode_config_init()

2022-11-21 Thread Thomas Zimmermann
Hi, merged into drm-misc-next. Thanks a lot. Best regards Thomas Am 18.11.22 um 03:16 schrieb Shang XiaoJing: drmm_mode_config_init() will call drm_mode_create_standard_properties() and won't check the ret value. When drm_mode_create_standard_properties() failed due to alloc, property will be

Re: [PATCH v2 3/3] drm/gem-shmem: When drm_gem_object_init failed, should release object

2022-11-21 Thread Thomas Zimmermann
Hi Am 19.11.22 um 07:41 schrieb ChunyouTang: when goto err_free, the object had init, so it should be release when fail. Signed-off-by: ChunyouTang Thanks a lot. I merged the patch into drm-misc-next. Best regards Thomas --- drivers/gpu/drm/drm_gem.c | 19 --

Re: git send-email friendly smtp provider anyone?

2022-11-21 Thread Noralf Trønnes
Den 21.11.2022 14.33, skrev Simon Ser: > I think you can apply for a linux.dev mailbox [1]. > Yeah you're right, I didn't know about that possibility. But it depends on whether or not I can just use their smtp server and keep my current email address. This looks like what's the problem with my

Re: git send-email friendly smtp provider anyone?

2022-11-21 Thread Simon Ser
On Monday, November 21st, 2022 at 17:52, Noralf Trønnes wrote: > Den 21.11.2022 14.33, skrev Simon Ser: > > > I think you can apply for a linux.dev mailbox 1. > > Yeah you're right, I didn't know about that possibility. > But it depends on whether or not I can just use their smtp server and >

Re: git send-email friendly smtp provider anyone?

2022-11-21 Thread Simon Ser
On Monday, November 21st, 2022 at 18:02, Simon Ser wrote: > On Monday, November 21st, 2022 at 17:52, Noralf Trønnes nor...@tronnes.org > wrote: > > > Den 21.11.2022 14.33, skrev Simon Ser: > > > > > I think you can apply for a linux.dev mailbox 1. > > > > Yeah you're right, I didn't know abou

Re: [PATCH] drm/amdgpu: fix unused-function error

2022-11-21 Thread Alex Deucher
Applied. Thanks! Alex On Mon, Nov 21, 2022 at 8:09 AM Ren Zhijie wrote: > > If CONFIG_DRM_AMDGPU=y and CONFIG_DRM_AMD_DC is not set, > gcc complained about unused-function : > > drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c:1705:13: error: > ‘amdgpu_discovery_set_sriov_display’ defined but not

[PATCH] drm/amdgpu: Partially revert "drm/amdgpu: update drm_display_info correctly when the edid is read"

2022-11-21 Thread Alex Deucher
This partially reverts 20543be93ca45968f344261c1a997177e51bd7e1. Calling drm_connector_update_edid_property() in amdgpu_connector_free_edid() causes a noticable pause in the system every 10 seconds on polled outputs so revert this part of the change. Bug: https://gitlab.freedesktop.org/drm/amd/-/

Re: [Intel-gfx] [PATCH v3 1/2] drm/i915: Fix negative value passed as remaining time

2022-11-21 Thread Andrzej Hajda
On 21.11.2022 15:56, Janusz Krzysztofik wrote: Commit b97060a99b01 ("drm/i915/guc: Update intel_gt_wait_for_idle to work with GuC") extended the API of intel_gt_retire_requests_timeout() with an extra argument 'remaining_timeout', intended for passing back unconsumed portion of requested timeout

Re: [PATCH] [next] drm/amdgpu: Replace remaining 1-element array with flex-array

2022-11-21 Thread Alex Deucher
Applied. Thanks! Alex On Mon, Nov 21, 2022 at 2:26 AM Paulo Miguel Almeida wrote: > > One-element arrays are deprecated, and we are replacing them with > flexible array members instead. So, replace one-element array with > flexible-array member in struct GOP_VBIOS_CONTENT and refactor the > res

Re: git send-email friendly smtp provider anyone?

2022-11-21 Thread Noralf Trønnes
Den 21.11.2022 18.06, skrev Simon Ser: > On Monday, November 21st, 2022 at 18:02, Simon Ser > wrote: > >> On Monday, November 21st, 2022 at 17:52, Noralf Trønnes nor...@tronnes.org >> wrote: >> >>> Den 21.11.2022 14.33, skrev Simon Ser: >>> I think you can apply for a linux.dev mailbox

Re: [PATCH v3] drm/i915/mtl: Enable Idle Messaging for GSC CS

2022-11-21 Thread Vivi, Rodrigo
On Fri, 2022-11-18 at 13:37 -0800, Dixit, Ashutosh wrote: > On Fri, 18 Nov 2022 10:37:37 -0800, Vivi, Rodrigo wrote: > > > > On Sat, 2022-11-19 at 00:03 +0530, Badal Nilawar wrote: > > > From: Vinay Belgaumkar > > > > > > By defaut idle messaging is disabled for GSC CS so to unblock RC6 > > > en

Re: [PATCH v3] drm/i915/mtl: Enable Idle Messaging for GSC CS

2022-11-21 Thread Vivi, Rodrigo
On Sat, 2022-11-19 at 09:02 +0530, Nilawar, Badal wrote: > > > On 19-11-2022 00:07, Vivi, Rodrigo wrote: > > On Sat, 2022-11-19 at 00:03 +0530, Badal Nilawar wrote: > > > From: Vinay Belgaumkar > > > > > > By defaut idle messaging is disabled for GSC CS so to unblock RC6 > > > entry on media ti

Re: git send-email friendly smtp provider anyone?

2022-11-21 Thread Noralf Trønnes
Den 21.11.2022 16.19, skrev Maxime Ripard: > On Mon, Nov 21, 2022 at 12:48:52PM +0100, Noralf Trønnes wrote: >> A couple of years ago my email provider blocked me from using git >> send-email with their smtp server. So I switched to the one my ISP >> provides. Now my ISP have outsourced their em

[PATCH] drm: rcar-du: Fix Kconfig dependency between DRM and RZG2L_MIPI_DSI

2022-11-21 Thread Biju Das
When CONFIG_DRM=m and CONFIG_DRM_RZG2L_MIPI_DSI=y, it results in a build failure. This patch fixes the build issue by adding dependency to DRM. Fixes: 7a043f978ed1 ("drm: rcar-du: Add RZ/G2L DSI driver") Reported-by: kernel test robot Signed-off-by: Biju Das --- Ref: * https://lore.kernel.org/

Re: [Intel-gfx] [PATCH v2 0/5] Add module oriented dmesg output

2022-11-21 Thread John Harrison
On 11/18/2022 02:52, Jani Nikula wrote: On Thu, 17 Nov 2022, john.c.harri...@intel.com wrote: From: John Harrison When trying to analyse bug reports from CI, customers, etc. it can be difficult to work out exactly what is happening on which GT in a multi-GT system. So add GT oriented debug/err

Re: [PATCH v4] dma-buf: fix racing conflict of dma_heap_add()

2022-11-21 Thread Christian König
Hi Dawei, from the technical description, coding style etc.. it looks clean to me, but I'm the completely wrong person to ask for a background check. I have a high level understanding of how dma-heaps work, but not a single line of this code is from me. Feel free to add my Acked-by, but Lau

Re: [PATCH v13,2/3] drm: mediatek: Set dpi format in mmsys

2022-11-21 Thread Matthias Brugger
On 24/10/2022 04:04, xinlei@mediatek.com wrote: From: Xinlei Lee Dpi output needs to adjust the output format to dual edge for MT8186. Co-developed-by: Jitao Shi Signed-off-by: Jitao Shi Signed-off-by: Xinlei Lee Reviewed-by: CK Hu Reviewed-by: AngeloGioacchino Del Regno Reviewed-

Re: [PATCH v10 18/19] drm/vc4: vec: Add support for more analog TV standards

2022-11-21 Thread Mateusz Kwiatkowski
Hi Mauro, As the author of the original version of this commit, and also a person who argued quite a bit on these descriptions and decisions, let me chip in a bit. W dniu 17.11.2022 o 16:49, Mauro Carvalho Chehab pisze: > On Thu, 17 Nov 2022 10:29:01 +0100 > Maxime Ripard wrote: > >> From: Mateu

Re: [PATCH v3 2/6] dt-bindings: Use new video interface bus type macros in examples

2022-11-21 Thread Sakari Ailus
Hi Laurent, On Sat, Jul 16, 2022 at 06:12:05PM +0300, Laurent Pinchart wrote: > Hi Sakari, > > On Sat, Jul 16, 2022 at 12:28:45PM +0300, Sakari Ailus wrote: > > On Thu, Jun 16, 2022 at 01:14:06AM +0300, Laurent Pinchart wrote: > > > Now that a header exists with macros for the media interface bus

Re: [PATCH] drm/amdgpu: Partially revert "drm/amdgpu: update drm_display_info correctly when the edid is read"

2022-11-21 Thread Luben Tuikov
Acked-by: Luben Tuikov Regards, Luben On 2022-11-21 12:38, Alex Deucher wrote: > This partially reverts 20543be93ca45968f344261c1a997177e51bd7e1. > > Calling drm_connector_update_edid_property() in > amdgpu_connector_free_edid() causes a noticable pause in > the system every 10 seconds on polle

[linux-next:master] BUILD REGRESSION e4cd8d3ff7f9efeb97330e5e9b99eeb2a68f5cf9

2022-11-21 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master branch HEAD: e4cd8d3ff7f9efeb97330e5e9b99eeb2a68f5cf9 Add linux-next specific files for 20221121 Error/Warning reports: https://lore.kernel.org/oe-kbuild-all/202211130053.np70vidn-...@intel.com https

[PATCH] drm/msm/dpu: Print interrupt index in addition to the mask

2022-11-21 Thread Marijn Suijten
The mask only describes the `irq_idx % 32` part, making it generally impossible to deduce what interrupt is being enabled/disabled. Since `debug/core_irq` in debugfs (and other prints) also include the full `DPU_IRQ_IDX()` value, print the same full value here for easier correlation instead of onl

[PATCH] drm/mediatek: Clean dangling pointer on bind error path

2022-11-21 Thread Nícolas F . R . A . Prado
mtk_drm_bind() can fail, in which case drm_dev_put() is called, destroying the drm_device object. However a pointer to it was still being held in the private object, and that pointer would be passed along to DRM in mtk_drm_sys_prepare() if a suspend were triggered at that point, resulting in a pani

Re: [PATCH v4 0/4] new subsystem for compute accelerator devices

2022-11-21 Thread Sonal Santan
On 11/19/22 12:44, Oded Gabbay wrote: > This is the fourth (and hopefully last) version of the patch-set to add the > new subsystem for compute accelerators. I removed the RFC headline as > I believe it is now ready for merging. > > Compare to v3, this patch-set contains one additional patch that

[PATCH 0/6] drm/i915: Add support for GSC FW loading

2022-11-21 Thread Daniele Ceraolo Spurio
Starting from MTL, the GSC FW is runtime loaded by the driver, instead of being stored in flash memory. Loading the GSC FW is required to allow the media GT to go into its C6 state and for content protection features (PXP, HDCP). The loading happens via a submission to the GSC engine. All subseque

  1   2   >