[PATCH v2] drm/ingenic: Kconfig: select REGMAP and REGMAP_MMIO

2023-06-07 Thread Sui Jingfeng
Otherwise its failed to pass basic compile test on platform without REGMAP_MMIO selected by defconfig make -j$(nproc) ARCH=mips CROSS_COMPILE=mips64el-linux-gnuabi64- SYNCinclude/config/auto.conf.cmd Checking missing-syscalls for N32 CALLscripts/checksyscalls.sh Checking missing-s

Re: [PATCH 1/1] drm/bridge: Silence error messages upon probe deferral

2023-06-07 Thread Alexander Stein
Hi Laurent, Am Dienstag, 6. Juni 2023, 17:12:29 CEST schrieb Laurent Pinchart: > Hi Alexander, > > Thank you for the patch. > > On Tue, Jun 06, 2023 at 04:48:33PM +0200, Alexander Stein wrote: > > When -EPROBE_DEFER is returned do not raise an error, but silently return > > this error instead. F

Re: [PATCH 00/30] fbdev: Make userspace interfaces optional

2023-06-07 Thread Markus Elfring
> Modern Linux distrobutions have adopted DRM drivers for graphics > output and use fbdev only for the kernel's framebuffer console. Would you like to avoid a typo in subsequent cover letters? Regards, Markus

Re: [PATCH] drm: gem: add an option for supporting the dma-coherent hardware.

2023-06-07 Thread Paul Cercueil
Hi Sui, Le mercredi 07 juin 2023 à 18:30 +0800, Sui Jingfeng a écrit : > Hi, > > > On 2023/6/7 17:36, Paul Cercueil wrote: > > Hi Sui, > > > > Le mercredi 07 juin 2023 à 13:30 +0800, Sui Jingfeng a écrit : > > > The single map_noncoherent member of struct drm_gem_dma_object > > > may > > > not

Re: [PATCH] drm: gem: add an option for supporting the dma-coherent hardware.

2023-06-07 Thread Maxime Ripard
On Wed, Jun 07, 2023 at 06:30:01PM +0800, Sui Jingfeng wrote: > On 2023/6/7 17:36, Paul Cercueil wrote: > > Hi Sui, > > > > Le mercredi 07 juin 2023 à 13:30 +0800, Sui Jingfeng a écrit : > > > The single map_noncoherent member of struct drm_gem_dma_object may > > > not > > > sufficient for describ

Re: [PATCH 00/30] fbdev: Make userspace interfaces optional

2023-06-07 Thread Thomas Zimmermann
Am 07.06.23 um 14:06 schrieb Markus Elfring: Modern Linux distrobutions have adopted DRM drivers for graphics output and use fbdev only for the kernel's framebuffer console. Would you like to avoid a typo in subsequent cover letters? Ha! It says 'distrobutions'. Regards, Markus -- Thom

Re: [RESEND PATCH v2 3/6] ARM: dts: stm32: support display on stm32f746-disco board

2023-06-07 Thread Dario Binacchi
Hi Jagan, On Wed, Jun 7, 2023 at 8:43 AM Jagan Teki wrote: > > On Wed, Jun 7, 2023 at 12:01 PM Dario Binacchi > wrote: > > > > Add support to Rocktech RK043FN48H display on stm32f746-disco board. > > > > Signed-off-by: Dario Binacchi > > --- > > > > (no changes since v1) > > > > arch/arm/boot/

Re: [PATCH 2/2] drm: bridge: tc358767: give VSDELAY some positive value

2023-06-07 Thread Lucas Stach
Am Freitag, dem 02.06.2023 um 23:34 +0200 schrieb Marek Vasut: > On 6/2/23 21:15, Lucas Stach wrote: > > From: David Jander > > > > The documentation is not clear about how this delay works. > > Empirical tests have shown that with a VSDELAY of 0, the first > > scanline is not properly formatted

[PATCH] drm/etnaviv: disable MLCG and pulse eater on GPU reset

2023-06-07 Thread Lucas Stach
Module level clock gating and the pulse eater might interfere with the GPU reset, as they both have the potential to stop the clock and thus reset propagation to parts of the GPU. Signed-off-by: Lucas Stach --- I'm not aware of any cases where this would have been an issue, but it is what the dow

[PATCH] drm/etnaviv: slow down FE idle polling

2023-06-07 Thread Lucas Stach
Currently the FE is spinning way too fast when polling for new work in the FE idleloop. As each poll fetches 16 bytes from memory, a GPU running at 1GHz with the current setting of 200 wait cycle between fetches causes 80 MB/s of memory traffic just to check for new work when the GPU is otherwise i

[PATCH] drm/etnaviv: add HWDB entry for GC520 r5341 c204

2023-06-07 Thread Lucas Stach
This is the 2D GPU found on the i.MX8MP SoC. Feature bits taken from the downstream kernel driver 6.4.3.p4.4. Signed-off-by: Lucas Stach --- drivers/gpu/drm/etnaviv/etnaviv_hwdb.c | 31 ++ 1 file changed, 31 insertions(+) diff --git a/drivers/gpu/drm/etnaviv/etnaviv_hwdb

[PATCH 2/8] drm/etnaviv: free events the usual way in recover worker

2023-06-07 Thread Lucas Stach
Clearing the whole bitmap at once is only a minor optimization in a path that should be extremely cold. Free the events by calling event_free() instead of directly manipulating the completion count and event bitmap. Signed-off-by: Lucas Stach --- drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 3 +-- 1

[PATCH 5/8] drm/etnaviv: avoid runtime PM usage in etnaviv_gpu_bind

2023-06-07 Thread Lucas Stach
Nothing in this callpath actually touches the GPU, so there is no reason to get it out of suspend state here. Only if runtime PM isn't enabled at all we must make sure to enable the clocks, so the GPU init routine can access the GPU later on. This also removes the need to guard against the state w

[PATCH 3/8] drm/etnaviv: move runtime PM handling to events

2023-06-07 Thread Lucas Stach
Conceptually events are the right abstraction to handle the GPU runtime PM state: as long as any event is pending the GPU can not be idle. Events are also properly freed and reallocated when the GPU has been reset after a hang. Signed-off-by: Lucas Stach --- drivers/gpu/drm/etnaviv/etnaviv_gem.h

[PATCH 1/8] drm/etnaviv: move down etnaviv_gpu_recover_hang() in file

2023-06-07 Thread Lucas Stach
So it can use the event_free function without adding another forward declaration. No functional change. Signed-off-by: Lucas Stach --- drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 88 +-- 1 file changed, 44 insertions(+), 44 deletions(-) diff --git a/drivers/gpu/drm/etnaviv/e

[PATCH 6/8] drm/etnaviv: better track GPU state

2023-06-07 Thread Lucas Stach
Instead of only tracking if the FE is running, use a enum to better describe the various states the GPU can be in. This allows some additional validation to make sure that functions that expect a certain GPU state are only called when the GPU is actually in that state. Signed-off-by: Lucas Stach

[PATCH 7/8] drm/etnaviv: drop GPU initialized property

2023-06-07 Thread Lucas Stach
Now that it is only used to track the driver internal state of the MMU global and cmdbuf objects, we can get rid of this property by making the free/finit functions of those objects safe to call on an uninitialized object. Signed-off-by: Lucas Stach --- drivers/gpu/drm/etnaviv/etnaviv_cmdbuf.c |

[PATCH 8/8] drm/etnaviv: expedited MMU fault handling

2023-06-07 Thread Lucas Stach
The GPU is halted when it hits a MMU exception, so there is no point in waiting for the job timeout to expire or try to work out if the GPU is still making progress in the timeout handler, as we know that the GPU won't make any more progress. Signed-off-by: Lucas Stach --- drivers/gpu/drm/etnavi

[PATCH 4/8] drm/etnaviv: make clock handling symetric between runtime resume and suspend

2023-06-07 Thread Lucas Stach
Currently the clock is enabled in the runtime resume function, but are disabled a level further down in the callstack in the suspend function. Move the clock disable into the suspend function to make handling symmetrical between resume and suspend. Signed-off-by: Lucas Stach --- drivers/gpu/drm/

Re: [PATCH] drm/panel-edp: Add AUO B116XAB01.4 edp panel entry

2023-06-07 Thread AngeloGioacchino Del Regno
Il 07/06/23 12:21, Laura Nao ha scritto: Add a panel entry for the AUO B116XAB01.4 edp panel, found in the Acer Chromebook Spin 311 (CP311-3H) laptop. Signed-off-by: Laura Nao Reviewed-by: AngeloGioacchino Del Regno

Re: [PATCH V8 0/7] drm: bridge: samsung-dsim: Support variable clocking

2023-06-07 Thread Rasmus Villemoes
On 26/05/2023 05.05, Adam Ford wrote: > This series fixes the blanking pack size and the PMS calculation. It then > adds support to allows the DSIM to dynamically DPHY clocks, and support > non-burst mode while allowing the removal of the hard-coded clock values > for the PLL for imx8m mini/nano/p

Re: [PATCH V8 0/7] drm: bridge: samsung-dsim: Support variable clocking

2023-06-07 Thread Adam Ford
On Wed, Jun 7, 2023 at 8:15 AM Rasmus Villemoes wrote: > > On 26/05/2023 05.05, Adam Ford wrote: > > This series fixes the blanking pack size and the PMS calculation. It then > > adds support to allows the DSIM to dynamically DPHY clocks, and support > > non-burst mode while allowing the removal

Re: [PATCH v3 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-06-07 Thread Paulo Pavacic
Hello, uto, 6. lip 2023. u 16:45 Krzysztof Kozlowski napisao je: > > On 06/06/2023 16:07, Paulo Pavacic wrote: > > Fannal C3004 is a 480x800 display made by fannal that requires > > DCS initialization sequences. > > > > Signed-off-by: Paulo Pavacic > > --- > > v4 changelog: > > - formatting and

Re: [PATCH] drm/panel-edp: Add AUO B116XAB01.4 edp panel entry

2023-06-07 Thread Doug Anderson
Hi, On Wed, Jun 7, 2023 at 3:23 AM Laura Nao wrote: > > Add a panel entry for the AUO B116XAB01.4 edp panel, found in the Acer > Chromebook Spin 311 (CP311-3H) laptop. > > Signed-off-by: Laura Nao > --- > drivers/gpu/drm/panel/panel-edp.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a

Re: [PATCH v3 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-06-07 Thread Krzysztof Kozlowski
On 07/06/2023 15:38, Paulo Pavacic wrote: > Hello, > > uto, 6. lip 2023. u 16:45 Krzysztof Kozlowski > napisao je: >> >> On 06/06/2023 16:07, Paulo Pavacic wrote: >>> Fannal C3004 is a 480x800 display made by fannal that requires >>> DCS initialization sequences. >>> >>> Signed-off-by: Paulo Pava

Re: [Intel-gfx] [PATCH] drm/i915: Avoid circular locking dependency when flush delayed work on gt reset

2023-06-07 Thread Andrzej Hajda
On 06.06.2023 05:00, Zhanjun Dong wrote: This attemps to avoid circular locing dependency between flush delayed work and intel_gt_reset. WARNING: possible circular locking dependency detected 6.4.0-rc1-drmtip_1340-g31e3463b0edb+ #1 Not tainted ---

Re: [PATCH v2] drm/bridge: ti-sn65dsi86: Avoid possible buffer overflow

2023-06-07 Thread Doug Anderson
Hi, On Tue, Jun 6, 2023 at 6:25 PM Su Hui wrote: > > Smatch error:buffer overflow 'ti_sn_bridge_refclk_lut' 5 <= 5. > > Fixes: cea86c5bb442 ("drm/bridge: ti-sn65dsi86: Implement the pwm_chip") > Signed-off-by: Su Hui > --- > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +- > 1 file changed, 1 inse

Re: [PATCH 00/30] fbdev: Make userspace interfaces optional

2023-06-07 Thread Markus Elfring
> Ha! It says 'distrobutions'. Do you tend to prefer any distributions? Regards, Markus

Re: [PATCH V8 0/7] drm: bridge: samsung-dsim: Support variable clocking

2023-06-07 Thread Adam Ford
On Wed, Jun 7, 2023 at 8:27 AM Adam Ford wrote: > > On Wed, Jun 7, 2023 at 8:15 AM Rasmus Villemoes > wrote: > > > > On 26/05/2023 05.05, Adam Ford wrote: > > > This series fixes the blanking pack size and the PMS calculation. It then > > > adds support to allows the DSIM to dynamically DPHY clo

Re: [PATCH] drm: gem: add an option for supporting the dma-coherent hardware.

2023-06-07 Thread Sui Jingfeng
Hi,  welcome to discussion. I have limited skills in manipulating English. It may not express what I'm really means in the short time. Part of word in the sentence may not as accurate as your. Well, please don't misunderstand, I'm not doing the rude to you. I will explain it with more detail

Re: [PATCH] drm/msm/dpu: tidy up some error checking

2023-06-07 Thread Dan Carpenter
On Tue, Jun 06, 2023 at 10:23:46PM +0200, Marijn Suijten wrote: > > @@ -359,8 +359,8 @@ static void dpu_encoder_phys_cmd_tearcheck_config( > > * frequency divided by the no. of rows (lines) in the LCDpanel. > > */ > > vsync_hz = dpu_kms_get_clk_rate(dpu_kms, "vsync"); > > - if (vsyn

Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-07 Thread Andi Shyti
On Tue, Jun 06, 2023 at 12:00:42PM +0200, Andi Shyti wrote: > From: Fei Yang > > To comply with the design that buffer objects shall have immutable > cache setting through out their life cycle, {set, get}_caching ioctl's > are no longer supported from MTL onward. With that change caching > policy

[PATCH v2] drm/panel-edp: Add AUO B116XAB01.4 edp panel entry

2023-06-07 Thread Laura Nao
Add a panel entry for the AUO B116XAB01.4 edp panel, found in the Acer Chromebook Spin 311 (CP311-3H) laptop. Signed-off-by: Laura Nao --- Changes in v2: - Sorted by product ID drivers/gpu/drm/panel/panel-edp.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/panel/panel-edp

Re: [PATCH 2/2] drm: bridge: tc358767: give VSDELAY some positive value

2023-06-07 Thread Marek Vasut
On 6/7/23 14:53, Lucas Stach wrote: Am Freitag, dem 02.06.2023 um 23:34 +0200 schrieb Marek Vasut: On 6/2/23 21:15, Lucas Stach wrote: From: David Jander The documentation is not clear about how this delay works. Empirical tests have shown that with a VSDELAY of 0, the first scanline is not p

[PATCH v4 0/3] drm/panel: add fannal c3004 panel

2023-06-07 Thread Paulo Pavacic
Fannal C3004 is a 2 lane MIPI DSI 480x800 panel which requires initialization with DSI DCS commands. After few initialization commands delay is required. Paulo Pavacic (3): dt-bindings: add fannal vendor prefix dt-bindings: display: panel: add fannal,c3004 drm/panel-fannal-c3004: Add fannal

[PATCH v4 2/3] dt-bindings: display: panel: add fannal,c3004

2023-06-07 Thread Paulo Pavacic
Added fannal to vendor-prefixes and dt bindings for Fannal C3004. Fannal C3004 is a 480x800 MIPI DSI Panel which requires DCS initialization sequences with certain delays between certain commands. Signed-off-by: Paulo Pavacic Reviewed-by: Krzysztof Kozlowski --- v4 changelog: - removed driver'

[PATCH v4 3/3] drm/panel-fannal-c3004: Add fannal c3004 DSI panel

2023-06-07 Thread Paulo Pavacic
Fannal C3004 is a 480x800 display made by fannal that requires DCS initialization sequences. Signed-off-by: Paulo Pavacic --- v4 changelog: - removal of blank lines, code wrapping changes - reset pin initialization handling changes v3 changelog: - formatting and style changes - remove BIT_NON

[PATCH v4 1/3] dt-bindings: add fannal vendor prefix

2023-06-07 Thread Paulo Pavacic
Fannal is a one-stop-solution provider for touch displays in industrial application, committed to delivering a variety of professional high-quality products and technical services globally. Website: www.fannal.com Signed-off-by: Paulo Pavacic Acked-by: Krzysztof Kozlowski --- Documentation/dev

Re: [PATCH 30/30] fbdev: Make support for userspace interfaces configurable

2023-06-07 Thread Thomas Zimmermann
Hi Am 07.06.23 um 10:48 schrieb Geert Uytterhoeven: Hi Thomas, Thanks for your patch! On Mon, Jun 5, 2023 at 4:48 PM Thomas Zimmermann wrote: Add Kconfig option CONFIG_FB_DEVICE and make the virtual fbdev device optional. If the new option has not been selected, fbdev does not create a files

Re: [PATCH 30/30] fbdev: Make support for userspace interfaces configurable

2023-06-07 Thread Geert Uytterhoeven
Hi Thomas, On Wed, Jun 7, 2023 at 5:15 PM Thomas Zimmermann wrote: > Am 07.06.23 um 10:48 schrieb Geert Uytterhoeven: > > On Mon, Jun 5, 2023 at 4:48 PM Thomas Zimmermann > > wrote: > >> --- a/drivers/video/fbdev/Kconfig > >> +++ b/drivers/video/fbdev/Kconfig > >> @@ -57,6 +57,15 @@ config FIRM

[PATCH v6 00/13] Enable Colorspace connector property in amdgpu

2023-06-07 Thread Harry Wentland
This patchset is based on Joshua's previous patchset [1], as well as my previous patchset [2]. It is - enabling support for the colorspace property in amdgpu, as well as - allowing drivers to specify the supported set of colorspaces, and Colorspace, Infoframes, and YCbCr matrix --

[PATCH v6 01/13] drm/connector: Convert DRM_MODE_COLORIMETRY to enum

2023-06-07 Thread Harry Wentland
This allows us to use strongly typed arguments. v2: - Bring NO_DATA back - Provide explicit enum values v3: - Drop unnecessary '&' from kerneldoc (emersion) v4: - Fix Normal Colorimetry comment Signed-off-by: Harry Wentland Reviewed-by: Simon Ser Reviewed-by: Sebastian Wick Reviewed-by: Pe

[PATCH v6 02/13] drm/connector: Add enum documentation to drm_colorspace

2023-06-07 Thread Harry Wentland
From: Joshua Ashton To match the other enums, and add more information about these values. v2: - Specify where an enum entry comes from - Clarify DEFAULT and NO_DATA behavior - BT.2020 CYCC is "constant luminance" - correct type for BT.601 v4: - drop DP/HDMI clarifications that might create

[PATCH v6 05/13] drm/connector: Print connector colorspace in state debugfs

2023-06-07 Thread Harry Wentland
v3: Fix kerneldocs (kernel test robot) v4: Avoid returning NULL from drm_get_colorspace_name Signed-off-by: Harry Wentland Reviewed-by: Sebastian Wick Reviewed-by: Joshua Ashton Cc: Pekka Paalanen Cc: Sebastian Wick Cc: vitaly.pros...@amd.com Cc: Uma Shankar Cc: Ville Syrjälä Cc: Joshua A

[PATCH v6 06/13] drm/connector: Allow drivers to pass list of supported colorspaces

2023-06-07 Thread Harry Wentland
Drivers might not support all colorspaces defined in dp_colorspaces and hdmi_colorspaces. This results in undefined behavior when userspace is setting an unsupported colorspace. Allow drivers to pass the list of supported colorspaces when creating the colorspace property. v2: - Use 0 to indicate

[PATCH v6 07/13] drm/amd/display: Always pass connector_state to stream validation

2023-06-07 Thread Harry Wentland
We need the connector_state for colorspace and scaling information and can get it from connector->state. Signed-off-by: Harry Wentland Reviewed-by: Joshua Ashton Cc: Pekka Paalanen Cc: Sebastian Wick Cc: vitaly.pros...@amd.com Cc: Joshua Ashton Cc: Simon Ser Cc: Melissa Wen Cc: dri-devel@l

[PATCH v6 11/13] drm/amd/display: Always set crtcinfo from create_stream_for_sink

2023-06-07 Thread Harry Wentland
From: Joshua Ashton Given that we always pass dm_state into here now, this won't ever trigger anymore. This is needed for we will always fail mode validation with invalid clocks or link bandwidth errors. Signed-off-by: Joshua Ashton Signed-off-by: Harry Wentland Reviewed-by: Harry Wentland

[PATCH v6 09/13] drm/amd/display: Signal mode_changed if colorspace changed

2023-06-07 Thread Harry Wentland
We need to signal mode_changed to make sure we update the output colorspace. v2: No need to call drm_hdmi_avi_infoframe_colorimetry as DC does its own infoframe packing. Signed-off-by: Harry Wentland Reviewed-by: Leo Li Reviewed-by: Joshua Ashton Cc: Pekka Paalanen Cc: Sebastian Wick Cc

[PATCH v6 08/13] drm/amd/display: Register Colorspace property for DP and HDMI

2023-06-07 Thread Harry Wentland
We want compositors to be able to set the output colorspace on DP and HDMI outputs, based on the caps reported from the receiver via EDID. Signed-off-by: Harry Wentland Reviewed-by: Joshua Ashton Cc: Pekka Paalanen Cc: Sebastian Wick Cc: vitaly.pros...@amd.com Cc: Joshua Ashton Cc: Simon Ser

[PATCH v6 03/13] drm/connector: Pull out common create_colorspace_property code

2023-06-07 Thread Harry Wentland
Signed-off-by: Harry Wentland Reviewed-by: Sebastian Wick Reviewed-by: Joshua Ashton Cc: Pekka Paalanen Cc: Sebastian Wick Cc: vitaly.pros...@amd.com Cc: Uma Shankar Cc: Ville Syrjälä Cc: Joshua Ashton Cc: Jani Nikula Cc: Simon Ser Cc: Ville Syrjälä Cc: Melissa Wen Cc: dri-devel@lists.

[PATCH v6 04/13] drm/connector: Use common colorspace_names array

2023-06-07 Thread Harry Wentland
We an use bitfields to track the support ones for HDMI and DP. This allows us to print colorspaces in a consistent manner without needing to know whether we're dealing with DP or HDMI. v4: - Rename _MAX to _COUNT and leave comment to indicate it's not a valid value - Fix misplaced function doc

[PATCH v6 13/13] drm/amd/display: Refactor avi_info_frame colorimetry determination

2023-06-07 Thread Harry Wentland
From: Joshua Ashton Replace the messy two if-else chains here that were on the same value with a switch on the enum. Signed-off-by: Joshua Ashton Signed-off-by: Harry Wentland Reviewed-by: Harry Wentland Cc: Pekka Paalanen Cc: Sebastian Wick Cc: vitaly.pros...@amd.com Cc: Joshua Ashton Cc

[PATCH v6 10/13] drm/amd/display: Send correct DP colorspace infopacket

2023-06-07 Thread Harry Wentland
Look at connector->colorimetry to determine output colorspace. We don't want to impact current SDR behavior, so DRM_MODE_COLORIMETRY_DEFAULT preserves current behavior. Also add support to explicitly set BT601 and BT709. v4: - Roll support for BT709 and BT601 into this patch - Add default case t

[PATCH v6 12/13] drm/amd/display: Add debugfs for testing output colorspace

2023-06-07 Thread Harry Wentland
In order to IGT test colorspace we'll want to print the currently enabled colorspace on a stream. We add a new debugfs to do so, using the same scheme as current bpc reporting. This might also come in handy when debugging display issues. v4: - Fix function doc comment - Fix sRGB debug print Sign

Re: [PATCH v2] drm/panel-edp: Add AUO B116XAB01.4 edp panel entry

2023-06-07 Thread Doug Anderson
Hi, On Wed, Jun 7, 2023 at 8:06 AM Laura Nao wrote: > > Add a panel entry for the AUO B116XAB01.4 edp panel, found in the Acer > Chromebook Spin 311 (CP311-3H) laptop. > > Signed-off-by: Laura Nao > --- > > Changes in v2: > - Sorted by product ID > > drivers/gpu/drm/panel/panel-edp.c | 1 + > 1

Re: [PATCH drm-next v4 09/14] drm/nouveau: fence: separate fence alloc and emit

2023-06-07 Thread kernel test robot
Hi Danilo, kernel test robot noticed the following build errors: [auto build test ERROR on 33a86170888b7e4aa0cea94ebb9c67180139cea9] url: https://github.com/intel-lab-lkp/linux/commits/Danilo-Krummrich/drm-execution-context-for-GEM-buffers-v4/20230607-063442 base

Re: [PATCH v17 1/1] drm/i915: Allow user to set cache at BO creation

2023-06-07 Thread Andi Shyti
Hi Fei, On Tue, Jun 06, 2023 at 12:00:42PM +0200, Andi Shyti wrote: > From: Fei Yang > > To comply with the design that buffer objects shall have immutable > cache setting through out their life cycle, {set, get}_caching ioctl's > are no longer supported from MTL onward. With that change caching

Re: [PATCH drm-next v4 12/14] drm/nouveau: nvkm/vmm: implement raw ops to manage uvmm

2023-06-07 Thread kernel test robot
Hi Danilo, kernel test robot noticed the following build warnings: [auto build test WARNING on 33a86170888b7e4aa0cea94ebb9c67180139cea9] url: https://github.com/intel-lab-lkp/linux/commits/Danilo-Krummrich/drm-execution-context-for-GEM-buffers-v4/20230607-063442 base

Re: [PATCH v6 00/13] Enable Colorspace connector property in amdgpu

2023-06-07 Thread Simon Ser
Thanks! All of the core DRM patches (1-6) are Reviewed-by: Simon Ser

Re: [PATCH] drm: gem: add an option for supporting the dma-coherent hardware.

2023-06-07 Thread Paul Cercueil
Hi Sui, Le mercredi 07 juin 2023 à 22:38 +0800, Sui Jingfeng a écrit : > Hi,  welcome to discussion. > > > I have limited skills in manipulating English. > > It may not express what I'm really means in the short time. > > Part of word in the sentence may not as accurate as your. > > Well, ple

[linux-next:master] BUILD REGRESSION abbd8bb42915d9ed06df11b430bf4ecb3d8ac5ad

2023-06-07 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master branch HEAD: abbd8bb42915d9ed06df11b430bf4ecb3d8ac5ad Add linux-next specific files for 20230607 Error/Warning reports: https://lore.kernel.org/oe-kbuild-all/202305132244.dwzbucud-...@intel.com https

Re: [PATCH] drm: gem: add an option for supporting the dma-coherent hardware.

2023-06-07 Thread Sui Jingfeng
Hi, On 2023/6/8 00:12, Paul Cercueil wrote: Hi Sui, Le mercredi 07 juin 2023 à 22:38 +0800, Sui Jingfeng a écrit : Hi,  welcome to discussion. I have limited skills in manipulating English. It may not express what I'm really means in the short time. Part of word in the sentence may not as

Re: [Intel-gfx] [PATCH] drm/i915: Avoid circular locking dependency when flush delayed work on gt reset

2023-06-07 Thread John Harrison
On 6/6/2023 10:53, John Harrison wrote: On 6/5/2023 20:00, Zhanjun Dong wrote: This attemps to avoid circular locing dependency between flush delayed work and intel_gt_reset. locing -> locking WARNING: possible circular locking dependency detected 6.4.0-rc1-drmtip_1340-g31e3463b0edb+ #1 Not

Re: [PATCH v1 2/8] dt-bindings: display: panel: mipi-dbi-spi: add spi-3wire property

2023-06-07 Thread Conor Dooley
On Wed, Jun 07, 2023 at 01:55:01PM +0200, Leonard Göhrs wrote: > Some MIPI DBI panels support a three wire mode (clock, chip select, > bidirectional data) that can be used to ask the panel if it is already set > up by e.g. the bootloader and can thus skip the initialization. > This enables a flicke

Re: [PATCH v1 1/8] dt-bindings: display: panel: mipi-dbi-spi: add shineworld lh133k compatible

2023-06-07 Thread Conor Dooley
On Wed, Jun 07, 2023 at 01:55:00PM +0200, Leonard Göhrs wrote: > The Shineworld LH133K is a 1.3" 240x240px RGB LCD with a MIPI DBI > compatible SPI interface. > The initialization procedure is quite basic with the exception of > requiring inverted colors. > A basic mipi-dbi-cmd[1] script to get the

Re: PROBLEM: AMD Ryzen 9 7950X iGPU - Blinking Issue

2023-06-07 Thread Alex Deucher
On Wed, Jun 7, 2023 at 4:42 AM Felix Richter wrote: > > Hi Guys, > > so I checked, the kernel I am running has this commit > (https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > /commit/?id=08da182175db4c7f80850354849d95f2670e8cd9) applied already! > > https://github.com/ju6ge/lin

Re: [PATCH v6 04/12] dt-bindings: display/msm: Add SM6350 MDSS

2023-06-07 Thread Krzysztof Kozlowski
On 06/06/2023 14:43, Konrad Dybcio wrote: > Document the SM6350 MDSS. > > Signed-off-by: Konrad Dybcio > --- > .../bindings/display/msm/qcom,sm6350-mdss.yaml | 213 > + > 1 file changed, 213 insertions(+) > Reviewed-by: Krzysztof Kozlowski Best regards, Krzysztof

Re: [PATCH v6 05/12] dt-bindings: display/msm: Add SM6375 MDSS

2023-06-07 Thread Krzysztof Kozlowski
On 06/06/2023 14:43, Konrad Dybcio wrote: > Document the SM6375 MDSS. > > Signed-off-by: Konrad Dybcio > --- > .../bindings/display/msm/qcom,sm6375-mdss.yaml | 215 > + > 1 file changed, 215 insertions(+) > Reviewed-by: Krzysztof Kozlowski Best regards, Krzysztof

Re: [PATCH 1/1] drm/bridge: Silence error messages upon probe deferral

2023-06-07 Thread Laurent Pinchart
Hi Alexander, On Wed, Jun 07, 2023 at 01:26:03PM +0200, Alexander Stein wrote: > Am Dienstag, 6. Juni 2023, 17:12:29 CEST schrieb Laurent Pinchart: > > On Tue, Jun 06, 2023 at 04:48:33PM +0200, Alexander Stein wrote: > > > When -EPROBE_DEFER is returned do not raise an error, but silently return >

Re: [PATCH] mm: fix hugetlb page unmap count balance issue

2023-06-07 Thread Andrew Morton
On Tue, 16 May 2023 15:34:40 -0700 Mike Kravetz wrote: > On 05/15/23 10:04, Mike Kravetz wrote: > > On 05/12/23 16:29, Mike Kravetz wrote: > > > On 05/12/23 14:26, James Houghton wrote: > > > > On Fri, May 12, 2023 at 12:20 AM Junxiao Chang > > > > wrote: > > > > > > > > This alone doesn't fix

[PATCH] drm/i915: Avoid circular locking dependency when flush delayed work on gt reset

2023-06-07 Thread Zhanjun Dong
This attempts to avoid circular locking dependency between flush delayed work and intel_gt_reset. Switched from cancel_delayed_work_sync to cancel_delayed_work, the non-sync version for reset path, it is safe as the worker has the trylock code to handle the lock; Meanwhile keep the sync version

Re: [PATCH] drm: rcar-du: Replace DRM_INFO() with drm_info()

2023-06-07 Thread Kieran Bingham
Quoting Laurent Pinchart (2023-05-30 10:26:39) > drm_info() adds proper context to the kernel log message, as it receives > the drm_device pointer. It is thus preferred over DRM_INFO(). Replace > the latter with the former. > > Signed-off-by: Laurent Pinchart Reviewed-by: Kieran Bingham > ---

Re: [PATCH v2 1/5] drm/i915/gsc: fixes and updates for GSC memory allocation

2023-06-07 Thread Teres Alexis, Alan Previn
On Mon, 2023-06-05 at 19:23 -0700, Ceraolo Spurio, Daniele wrote: > A few fixes/updates are required around the GSC memory allocation and it > is easier to do them all at the same time. The changes are as follows: > > 1 - Switch the memory allocation to stolen memory. We need to avoid > accesses f

Re: [PATCH 24/30] fbdev/core: Pass Linux device to pm_vt_switch_*() functions

2023-06-07 Thread Sam Ravnborg
On Mon, Jun 05, 2023 at 04:48:06PM +0200, Thomas Zimmermann wrote: > Pass the Linux device to pm_vt_switch_*() instead of the virtual > fbdev device. Prepares fbdev for making struct fb_info.dev optional. > > The type of device that is passed to the PM functions does not matter > much. It is only

Re: [PATCH] mm: fix hugetlb page unmap count balance issue

2023-06-07 Thread David Hildenbrand
On 17.05.23 00:34, Mike Kravetz wrote: On 05/15/23 10:04, Mike Kravetz wrote: On 05/12/23 16:29, Mike Kravetz wrote: On 05/12/23 14:26, James Houghton wrote: On Fri, May 12, 2023 at 12:20 AM Junxiao Chang wrote: This alone doesn't fix mapcounting for PTE-mapped HugeTLB pages. You need someth

Re: [PATCH 25/30] fbdev/core: Move framebuffer and backlight helpers into separate files

2023-06-07 Thread Sam Ravnborg
Hi Thomas, On Mon, Jun 05, 2023 at 04:48:07PM +0200, Thomas Zimmermann wrote: > Move framebuffer and backlight helpers into separate files. Leave > fbsysfs.c to sysfs-related code. No functional changes. > > The framebuffer helpers are not in fbmem.c because they are under > GPL-2.0-or-later copy

Re: [PATCH 26/30] fbdev/core: Add fb_device_{create,destroy}()

2023-06-07 Thread Sam Ravnborg
Hi Thomas, On Mon, Jun 05, 2023 at 04:48:08PM +0200, Thomas Zimmermann wrote: > Move the logic to create and destroy fbdev devices into the new > helpers fb_device_create() and fb_device_destroy(). > > There was a call to fb_cleanup_device() in do_unregister_framebuffer() > that was too late. The

Re: [Freedreno] [PATCH v3 6/7] drm/msm/dsi: Add phy configuration for MSM8226

2023-06-07 Thread Jeykumar Sankaran
On 6/1/2023 10:00 AM, Luca Weiss wrote: MSM8226 uses a modified PLL lock sequence compared to MSM8974, which is based on the function dsi_pll_enable_seq_m in the msm-3.10 kernel. Worth noting that the msm-3.10 downstream kernel also will try other sequences in case this one doesn't work, but

Re: [Intel-gfx] [PATCH] drm/i915: Avoid circular locking dependency when flush delayed work on gt reset

2023-06-07 Thread John Harrison
On 6/7/2023 12:03, Zhanjun Dong wrote: This attempts to avoid circular locking dependency between flush delayed work and intel_gt_reset. Switched from cancel_delayed_work_sync to cancel_delayed_work, the non-sync version for reset path, it is safe as the worker has the trylock code to handle t

Re: [PATCH drm-next v4 04/14] drm: debugfs: provide infrastructure to dump a DRM GPU VA space

2023-06-07 Thread kernel test robot
Hi Danilo, kernel test robot noticed the following build warnings: [auto build test WARNING on 33a86170888b7e4aa0cea94ebb9c67180139cea9] url: https://github.com/intel-lab-lkp/linux/commits/Danilo-Krummrich/drm-execution-context-for-GEM-buffers-v4/20230607-063442 base

Re: [PATCH 27/30] fbdev/core: Move procfs code to separate file

2023-06-07 Thread Sam Ravnborg
Hi Thomas, On Mon, Jun 05, 2023 at 04:48:09PM +0200, Thomas Zimmermann wrote: > Move fbdev's procfs code into a separate file and contain it in > init and cleanup helpers. No functional changes. Maybe add: Delete the unused for_each_registered_fb while touching the code. The change to use proc_re

Re: [PATCH v1 2/8] dt-bindings: display: panel: mipi-dbi-spi: add spi-3wire property

2023-06-07 Thread Noralf Trønnes
On 6/7/23 13:55, Leonard Göhrs wrote: > Some MIPI DBI panels support a three wire mode (clock, chip select, > bidirectional data) that can be used to ask the panel if it is already set > up by e.g. the bootloader and can thus skip the initialization. > This enables a flicker-free boot. > > Sign

Re: [PATCH 28/30] fbdev/core: Move file-I/O code into separate file

2023-06-07 Thread Sam Ravnborg
Hi Thomas. On Mon, Jun 05, 2023 at 04:48:10PM +0200, Thomas Zimmermann wrote: > Move fbdev's file-I/O code into a separate file and contain it in > init and cleanup helpers. No functional changes. > > Signed-off-by: Thomas Zimmermann Consider moving the two helps as noted below. With or without

Re: [PATCH 29/30] fbdev/core: Rework fb init code

2023-06-07 Thread Sam Ravnborg
Hi Thomas. On Mon, Jun 05, 2023 at 04:48:11PM +0200, Thomas Zimmermann wrote: > Init the class "graphics" before the rest of fbdev. Later steps, such > as the sysfs code, depend on the class. Also arrange the module's exit > code in reverse order. > > Unexport the global variable fb_class, which

Re: [PATCH] mm: fix hugetlb page unmap count balance issue

2023-06-07 Thread Andrew Morton
On Wed, 7 Jun 2023 13:53:10 -0700 Mike Kravetz wrote: > > > > BUGs aren't good. Can we please find a way to push this along? > > > > Have we heard anything from any udmabuf people? > > > > I have not heard anything. When this issue popped up, it took me by surprise. > > udmabuf maintainer

Re: [Freedreno] [PATCH v3 6/7] drm/msm/dsi: Add phy configuration for MSM8226

2023-06-07 Thread Luca Weiss
On Mittwoch, 7. Juni 2023 21:46:31 CEST Jeykumar Sankaran wrote: > On 6/1/2023 10:00 AM, Luca Weiss wrote: > > MSM8226 uses a modified PLL lock sequence compared to MSM8974, which is > > based on the function dsi_pll_enable_seq_m in the msm-3.10 kernel. > > > > Worth noting that the msm-3.10 downs

Re: [RFC PATCH v2 12/13] drm/msm/dpu: add support for virtual planes

2023-06-07 Thread Abhinav Kumar
On 3/20/2023 6:18 PM, Dmitry Baryshkov wrote: Only several SSPP blocks support such features as YUV output or scaling, thus different DRM planes have different features. Properly utilizing all planes requires the attention of the compositor, who should prefer simpler planes to YUV-supporting

RE: [PATCH v3] drm/i915: Fix a VMA UAF for multi-gt platform

2023-06-07 Thread Venkatesh Reddy, Sushma
Thank you, Nirmoy. We have verified it on Chrome. Tested-by: Sushma Venkatesh Reddy -Original Message- From: Das, Nirmoy Sent: Wednesday, June 7, 2023 1:11 AM To: intel-...@lists.freedesktop.org Cc: dri-devel@lists.freedesktop.org; Das, Nirmoy ; Joonas Lahtinen ; Vivi, Rodrigo ; Tvrtk

Re: [PATCH v3] drm/i915/mtl/gsc: Add a gsc_info debugfs

2023-06-07 Thread Teres Alexis, Alan Previn
On Mon, 2023-06-05 at 21:32 -0700, Ceraolo Spurio, Daniele wrote: > Add a new debugfs to dump information about the GSC. This includes: > > - the FW path and SW tracking status; > - the release, security and compatibility versions; > - the HECI1 status registers. > > Note that those are the same

[pull] amdgpu, radeon drm-fixes-6.4

2023-06-07 Thread Alex Deucher
Hi Dave, Daniel, Fixes for 6.4. The following changes since commit 9561de3a55bed6bdd44a12820ba81ec416e705a7: Linux 6.4-rc5 (2023-06-04 14:04:27 -0400) are available in the Git repository at: https://gitlab.freedesktop.org/agd5f/linux.git tags/amd-drm-fixes-6.4-2023-06-07 for you to fetch

[PATCH v2 00/10] drm/panel and i2c-hid: Allow panels and touchscreens to power sequence together

2023-06-07 Thread Douglas Anderson
The big motivation for this patch series is mostly described in the patch ("drm/panel: Add a way for other devices to follow panel state"), but to quickly summarize here: for touchscreens that are connected to a panel we need the ability to power sequence the two device together. This is not a ne

[PATCH v2 01/10] dt-bindings: HID: i2c-hid: Add "panel" property to i2c-hid backed touchscreens

2023-06-07 Thread Douglas Anderson
As talked about in the patch ("drm/panel: Add a way for other devices to follow panel state"), touchscreens that are connected to panels are generally expected to be power sequenced together with the panel they're attached to. Today, nothing provides information allowing you to find out that a touc

[PATCH v2 02/10] drm/panel: Check for already prepared/enabled in drm_panel

2023-06-07 Thread Douglas Anderson
In a whole pile of panel drivers, we have code to make the prepare/unprepare/enable/disable callbacks behave as no-ops if they've already been called. It's silly to have this code duplicated everywhere. Add it to the core instead so that we can eventually delete it from all the drivers. Note: to ge

[PATCH v2 03/10] drm/panel: Add a way for other devices to follow panel state

2023-06-07 Thread Douglas Anderson
These days, it's fairly common to see panels that have touchscreens attached to them. The panel and the touchscreen can somewhat be thought of as totally separate devices and, historically, this is how Linux has treated them. However, treating them as separate isn't necessarily the best way to mode

[PATCH v2 04/10] of: property: fw_devlink: Add a devlink for panel followers

2023-06-07 Thread Douglas Anderson
Inform fw_devlink of the fact that a panel follower (like a touchscreen) is effectively a consumer of the panel from the purposes of fw_devlink. NOTE: this patch isn't required for correctness but instead optimizes probe order / helps avoid deferrals. Signed-off-by: Douglas Anderson --- Since th

[PATCH v2 05/10] HID: i2c-hid: Switch to SYSTEM_SLEEP_PM_OPS()

2023-06-07 Thread Douglas Anderson
The SYSTEM_SLEEP_PM_OPS() allows us to get rid of '#ifdef CONFIG_PM_SLEEP', as talked about in commit 1a3c7bb08826 ("PM: core: Add new *_PM_OPS macros, deprecate old ones"). This change is expected to have no functional effect. Signed-off-by: Douglas Anderson --- (no changes since v1) drivers

[PATCH v2 07/10] HID: i2c-hid: Make suspend and resume into helper functions

2023-06-07 Thread Douglas Anderson
In a future patch we'd like to be able to call the current i2c-hid suspend and resume functions from times other than system suspend. Move the functions higher up in the file and have them take a "struct i2c_hid" to make this simpler. We'll then add tiny wrappers of the functions for use with syste

[PATCH v2 06/10] HID: i2c-hid: Rearrange probe() to power things up later

2023-06-07 Thread Douglas Anderson
In a future patch, we want to change i2c-hid not to necessarily power up the touchscreen during probe. In preparation for that, rearrange the probe function so that we put as much stuff _before_ powering up the device as possible. This change is expected to have no functional effect. Signed-off-b

[PATCH v2 08/10] HID: i2c-hid: Support being a panel follower

2023-06-07 Thread Douglas Anderson
As talked about in the patch ("drm/panel: Add a way for other devices to follow panel state"), we really want to keep the power states of a touchscreen and the panel it's attached to in sync with each other. In that spirit, add support to i2c-hid to be a panel follower. This will let the i2c-hid dr

[PATCH v2 09/10] HID: i2c-hid: Do panel follower work on the system_wq

2023-06-07 Thread Douglas Anderson
Turning on an i2c-hid device can be a slow process. This is why i2c-hid devices use PROBE_PREFER_ASYNCHRONOUS. Unfortunately, when we're a panel follower the i2c-hid power up sequence now blocks the power on of the panel. Let's fix that by scheduling the work on the system_wq. Signed-off-by: Dougl

<    1   2   3   >