Re: [PATCH v3 07/13] drm: Make drivers depends on DRM_DW_HDMI

2024-04-22 Thread Mark Brown
On Tue, Apr 02, 2024 at 04:43:46PM +0100, Mark Brown wrote: > On Wed, Mar 27, 2024 at 11:57:02AM +0100, Maxime Ripard wrote: > > > DRM_DW_HDMI has a number of dependencies that might not be enabled. > > However, drivers were used to selecting it while not enforcing the > > DRM_DW_HDMI

Re: [PATCH v2 3/4] drm/xe: Extract xe_gt_idle() helper

2024-04-22 Thread Ghimiray, Himal Prasad
On 19-04-2024 14:28, Aravind Iddamsetty wrote: This would be used in other places outside of gt_reset path. v2: 1. Add kernel doc for xe_gt_idle (Michal) 2. fix return as no actual error is reported by xe_uc_stop (Himal) Cc: Lucas De Marchi Cc: Michal Wajdeczko Cc: Himal Prasad Ghimiray

[PATCH v1 4/4] drm/ci: add tests on vkms

2024-04-22 Thread Vignesh Raman
Add job that runs igt on top of vkms. Acked-by: Maíra Canal Acked-by: Helen Koike Signed-off-by: Vignesh Raman Acked-by: Jessica Zhang Tested-by: Jessica Zhang Acked-by: Maxime Ripard Signed-off-by: Helen Koike --- MAINTAINERS | 1 +

[PATCH v1 1/4] drm/ci: uprev mesa version

2024-04-22 Thread Vignesh Raman
zlib.net is not allowing tarball download anymore and results in below error in kernel+rootfs_arm32 container build, urllib.error.HTTPError: HTTP Error 403: Forbidden urllib.error.HTTPError: HTTP Error 415: Unsupported Media Type Uprev mesa to latest version which includes a fix for this issue.

[PATCH v1 2/4] drm/ci: build virtual GPU driver as module

2024-04-22 Thread Vignesh Raman
With latest IGT, the tests tries to load the module and it fails. So build the virtual GPU driver for virtio as module. Signed-off-by: Vignesh Raman --- drivers/gpu/drm/ci/build.sh | 1 - drivers/gpu/drm/ci/igt_runner.sh | 6 +++---

[PATCH v1 0/4] Uprev mesa and IGT

2024-04-22 Thread Vignesh Raman
Uprev mesa and IGT to the latest version. Stop vendoring the testlist into the kernel. Instead, use the testlist from the IGT build to ensure we do not miss renamed or newly added tests. Update the xfails with the latest testlist run. Also build virtual GPU driver for virtio as module. This

Re: [PATCH] drm/panel-edp: Add panel CSOT MNB601LS1-1

2024-04-22 Thread Xuxin Xiong
Yes, I read the edid from the panels, one is CSO and the other is CSW. The details are as follows, please help check. Thank you! 1. MNC207QS1-1 edid-decode (hex): 00 ff ff ff ff ff ff 00 0e 6f 00 12 e7 00 00 00 1e 21 01 04 a5 1b 12 78 03 8a d5 9c 5e 59 90 25 1b 50 54 00 00 00 01 01 01 01 01 01

Re: [PATCH v2 2/2] drm/i915: Fix gt reset with GuC submission is disabled

2024-04-22 Thread John Harrison
On 4/22/2024 13:19, Nirmoy Das wrote: Currently intel_gt_reset() kills the GuC and then resets requested engines. This is problematic because there is a dedicated CSB FIFO which only GuC can access and if that FIFO fills up, the hardware will block on the next context switch until there is space

Re: [PATCH v2 1/2] drm/i915: Refactor confusing __intel_gt_reset()

2024-04-22 Thread John Harrison
On 4/22/2024 13:19, Nirmoy Das wrote: __intel_gt_reset() is really for resetting engines though the name might suggest something else. So add a helper function to remove confusions with no functional changes. v2: Move intel_gt_reset_all_engines() next to intel_gt_reset_engine() to make

[PATCH AUTOSEL 5.4 6/9] gpu: host1x: Do not setup DMA for virtual devices

2024-04-22 Thread Sasha Levin
From: Thierry Reding [ Upstream commit 8ab58f6841b19423231c5db3378691ec80c778f8 ] The host1x devices are virtual compound devices and do not perform DMA accesses themselves, so they do not need to be set up for DMA. Ideally we would also not need to set up DMA masks for the virtual devices,

[PATCH AUTOSEL 5.10 7/9] gpu: host1x: Do not setup DMA for virtual devices

2024-04-22 Thread Sasha Levin
From: Thierry Reding [ Upstream commit 8ab58f6841b19423231c5db3378691ec80c778f8 ] The host1x devices are virtual compound devices and do not perform DMA accesses themselves, so they do not need to be set up for DMA. Ideally we would also not need to set up DMA masks for the virtual devices,

[PATCH AUTOSEL 5.15 07/10] gpu: host1x: Do not setup DMA for virtual devices

2024-04-22 Thread Sasha Levin
From: Thierry Reding [ Upstream commit 8ab58f6841b19423231c5db3378691ec80c778f8 ] The host1x devices are virtual compound devices and do not perform DMA accesses themselves, so they do not need to be set up for DMA. Ideally we would also not need to set up DMA masks for the virtual devices,

[PATCH AUTOSEL 6.1 14/19] drm/amdgpu: Refine IB schedule error logging

2024-04-22 Thread Sasha Levin
From: Lijo Lazar [ Upstream commit 4b18a91faf1752f9bd69a4ed3aed2c8f6e5b0528 ] Downgrade to debug information when IBs are skipped. Also, use dev_* to identify the device. Signed-off-by: Lijo Lazar Reviewed-by: Christian König Reviewed-by: Asad Kamal Signed-off-by: Alex Deucher

[PATCH AUTOSEL 6.1 11/19] gpu: host1x: Do not setup DMA for virtual devices

2024-04-22 Thread Sasha Levin
From: Thierry Reding [ Upstream commit 8ab58f6841b19423231c5db3378691ec80c778f8 ] The host1x devices are virtual compound devices and do not perform DMA accesses themselves, so they do not need to be set up for DMA. Ideally we would also not need to set up DMA masks for the virtual devices,

[PATCH AUTOSEL 6.6 21/29] amd/amdkfd: sync all devices to wait all processes being evicted

2024-04-22 Thread Sasha Levin
From: Zhigang Luo [ Upstream commit d06af584be5a769d124b7302b32a033e9559761d ] If there are more than one device doing reset in parallel, the first device will call kfd_suspend_all_processes() to evict all processes on all devices, this call takes time to finish. other device will start reset

[PATCH AUTOSEL 6.6 20/29] drm/amdgpu: Fix VCN allocation in CPX partition

2024-04-22 Thread Sasha Levin
From: Lijo Lazar [ Upstream commit f7e232de51bb1b45646e5b7dc4ebcf13510f2630 ] VCN need not be shared in CPX mode always for all GFX 9.4.3 SOC SKUs. In certain configs, VCN instance can be exclusively allocated to a partition even under CPX mode. Signed-off-by: Lijo Lazar Reviewed-by: James

[PATCH AUTOSEL 6.6 19/29] drm/amd/display: Skip on writeback when it's not applicable

2024-04-22 Thread Sasha Levin
From: Alex Hung [ Upstream commit ecedd99a9369fb5cde601ae9abd58bca2739f1ae ] [WHY] dynamic memory safety error detector (KASAN) catches and generates error messages "BUG: KASAN: slab-out-of-bounds" as writeback connector does not support certain features which are not initialized. [HOW] Skip

[PATCH AUTOSEL 6.6 18/29] drm/amdgpu: implement IRQ_STATE_ENABLE for SDMA v4.4.2

2024-04-22 Thread Sasha Levin
From: Tao Zhou [ Upstream commit f886b49feaae30acd599e37d4284836024b0f3ed ] SDMA_CNTL is not set in some cases, driver configures it by itself. v2: simplify code Signed-off-by: Tao Zhou Reviewed-by: Hawking Zhang Signed-off-by: Alex Deucher Signed-off-by: Sasha Levin ---

[PATCH AUTOSEL 6.6 17/29] drm/amdgpu: Refine IB schedule error logging

2024-04-22 Thread Sasha Levin
From: Lijo Lazar [ Upstream commit 4b18a91faf1752f9bd69a4ed3aed2c8f6e5b0528 ] Downgrade to debug information when IBs are skipped. Also, use dev_* to identify the device. Signed-off-by: Lijo Lazar Reviewed-by: Christian König Reviewed-by: Asad Kamal Signed-off-by: Alex Deucher

[PATCH AUTOSEL 6.6 14/29] gpu: host1x: Do not setup DMA for virtual devices

2024-04-22 Thread Sasha Levin
From: Thierry Reding [ Upstream commit 8ab58f6841b19423231c5db3378691ec80c778f8 ] The host1x devices are virtual compound devices and do not perform DMA accesses themselves, so they do not need to be set up for DMA. Ideally we would also not need to set up DMA masks for the virtual devices,

[PATCH AUTOSEL 6.8 41/43] drm/xe: Label RING_CONTEXT_CONTROL as masked

2024-04-22 Thread Sasha Levin
From: Ashutosh Dixit [ Upstream commit f76646c83f028c62853c23dac49204232e903597 ] RING_CONTEXT_CONTROL is a masked register. v2: Also clean up setting register value (Lucas) Reviewed-by: Matt Roper Reviewed-by: Lucas De Marchi Signed-off-by: Ashutosh Dixit Link:

[PATCH AUTOSEL 6.8 40/43] drm/xe/xe_migrate: Cast to output precision before multiplying operands

2024-04-22 Thread Sasha Levin
From: Himal Prasad Ghimiray [ Upstream commit 9cb46b31f3d08ed3fce86349e8c12f96d7c88717 ] Addressing potential overflow in result of multiplication of two lower precision (u32) operands before widening it to higher precision (u64). -v2 Fix commit message and description. (Rodrigo) Cc: Rodrigo

[PATCH AUTOSEL 6.8 32/43] drm/amdgpu: Fix VCN allocation in CPX partition

2024-04-22 Thread Sasha Levin
From: Lijo Lazar [ Upstream commit f7e232de51bb1b45646e5b7dc4ebcf13510f2630 ] VCN need not be shared in CPX mode always for all GFX 9.4.3 SOC SKUs. In certain configs, VCN instance can be exclusively allocated to a partition even under CPX mode. Signed-off-by: Lijo Lazar Reviewed-by: James

[PATCH AUTOSEL 6.8 33/43] amd/amdkfd: sync all devices to wait all processes being evicted

2024-04-22 Thread Sasha Levin
From: Zhigang Luo [ Upstream commit d06af584be5a769d124b7302b32a033e9559761d ] If there are more than one device doing reset in parallel, the first device will call kfd_suspend_all_processes() to evict all processes on all devices, this call takes time to finish. other device will start reset

[PATCH AUTOSEL 6.8 31/43] drm/amd/pm: fix the high voltage issue after unload

2024-04-22 Thread Sasha Levin
From: Kenneth Feng [ Upstream commit 3818708e9c9712e2ba4006bc23502ee7b031bd3f ] fix the high voltage issue after unload on smu 13.0.10 Signed-off-by: Kenneth Feng Reviewed-by: Hawking Zhang Signed-off-by: Alex Deucher Signed-off-by: Sasha Levin ---

[PATCH AUTOSEL 6.8 30/43] drm/amd/display: Skip on writeback when it's not applicable

2024-04-22 Thread Sasha Levin
From: Alex Hung [ Upstream commit ecedd99a9369fb5cde601ae9abd58bca2739f1ae ] [WHY] dynamic memory safety error detector (KASAN) catches and generates error messages "BUG: KASAN: slab-out-of-bounds" as writeback connector does not support certain features which are not initialized. [HOW] Skip

[PATCH AUTOSEL 6.8 29/43] drm/amdgpu: implement IRQ_STATE_ENABLE for SDMA v4.4.2

2024-04-22 Thread Sasha Levin
From: Tao Zhou [ Upstream commit f886b49feaae30acd599e37d4284836024b0f3ed ] SDMA_CNTL is not set in some cases, driver configures it by itself. v2: simplify code Signed-off-by: Tao Zhou Reviewed-by: Hawking Zhang Signed-off-by: Alex Deucher Signed-off-by: Sasha Levin ---

[PATCH AUTOSEL 6.8 28/43] drm/amdgpu: add smu 14.0.1 discovery support

2024-04-22 Thread Sasha Levin
From: Yifan Zhang [ Upstream commit 533eefb9be76c3b23d220ee18edfda8eb56cefff ] This patch to add smu 14.0.1 support Reviewed-by: Alex Deucher Signed-off-by: Yifan Zhang Signed-off-by: Alex Deucher Signed-off-by: Sasha Levin --- drivers/gpu/drm/amd/amdgpu/amdgpu_discovery.c | 1 + 1 file

[PATCH AUTOSEL 6.8 27/43] drm/amd/display: add DCN 351 version for microcode load

2024-04-22 Thread Sasha Levin
From: Li Ma [ Upstream commit d4396924c3d44f34d0643f650e70892e07f3677f ] There is a new DCN veriosn 3.5.1 need to load Signed-off-by: Li Ma Reviewed-by: Yifan Zhang Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher Signed-off-by: Sasha Levin ---

[PATCH AUTOSEL 6.8 26/43] drm/amdgpu: Refine IB schedule error logging

2024-04-22 Thread Sasha Levin
From: Lijo Lazar [ Upstream commit 4b18a91faf1752f9bd69a4ed3aed2c8f6e5b0528 ] Downgrade to debug information when IBs are skipped. Also, use dev_* to identify the device. Signed-off-by: Lijo Lazar Reviewed-by: Christian König Reviewed-by: Asad Kamal Signed-off-by: Alex Deucher

[PATCH AUTOSEL 6.8 21/43] gpu: host1x: Do not setup DMA for virtual devices

2024-04-22 Thread Sasha Levin
From: Thierry Reding [ Upstream commit 8ab58f6841b19423231c5db3378691ec80c778f8 ] The host1x devices are virtual compound devices and do not perform DMA accesses themselves, so they do not need to be set up for DMA. Ideally we would also not need to set up DMA masks for the virtual devices,

[PATCH AUTOSEL 6.8 18/43] accel/ivpu: Improve clarity of MMU error messages

2024-04-22 Thread Sasha Levin
From: "Wachowski, Karol" [ Upstream commit 3556f922612caf4c9b97cf7337626f8342b3dea3 ] This patch improves readability and clarity of MMU error messages. Previously, the error strings were somewhat confusing and could lead to ambiguous interpretations, making it difficult to diagnose issues.

[PATCH AUTOSEL 6.8 19/43] accel/ivpu: Fix missed error message after VPU rename

2024-04-22 Thread Sasha Levin
From: Jacek Lawrynowicz [ Upstream commit 0d298e23292b7a5b58c5589fe33b96e95363214f ] Change "VPU" to "NPU" in ivpu_suspend() so it matches all other error messages. Signed-off-by: Jacek Lawrynowicz Reviewed-by: Jeffrey Hugo Link:

[PATCH AUTOSEL 6.8 17/43] accel/ivpu: Remove d3hot_after_power_off WA

2024-04-22 Thread Sasha Levin
From: Jacek Lawrynowicz [ Upstream commit e3caadf1f9dfc9d62b5ffc3bd73ebac0c8f26b3f ] Always enter D3hot after entering D0i3 an all platforms. This minimizes power usage. Signed-off-by: Jacek Lawrynowicz Reviewed-by: Jeffrey Hugo Link:

[PATCH AUTOSEL 6.8 13/43] nouveau/gsp: Avoid addressing beyond end of rpc->entries

2024-04-22 Thread Sasha Levin
From: Kees Cook [ Upstream commit 838ae9f45c4e43b4633d8b0ad1fbedff9ecf177d ] Using the end of rpc->entries[] for addressing runs into both compile-time and run-time detection of accessing beyond the end of the array. Use the base pointer instead, since was allocated with the additional bytes

Re: [PATCH v3 14/15] riscv: Add support for suppressing warning backtraces

2024-04-22 Thread Charlie Jenkins
On Wed, Apr 03, 2024 at 06:19:35AM -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

[PATCH v8 7/7] x86/vmware: Add TDX hypercall support

2024-04-22 Thread Alexey Makhalov
VMware hypercalls use I/O port, VMCALL or VMMCALL instructions. Add __tdx_hypercall path to support TDX guests. No change in high bandwidth hypercalls, as only low bandwidth ones are supported for TDX guests. Co-developed-by: Tim Merrifield Signed-off-by: Tim Merrifield Signed-off-by: Alexey

[PATCH v8 5/7] drm/vmwgfx: Use VMware hypercall API

2024-04-22 Thread Alexey Makhalov
Switch from VMWARE_HYPERCALL macro to vmware_hypercall API. Eliminate arch specific code. drivers/gpu/drm/vmwgfx/vmwgfx_msg_arm64.h: implement arm64 variant of vmware_hypercall. And keep it here until introduction of ARM64 VMWare hypervisor interface. Signed-off-by: Alexey Makhalov Reviewed-by:

[PATCH v8 6/7] x86/vmware: Undefine VMWARE_HYPERCALL

2024-04-22 Thread Alexey Makhalov
No more direct use of VMWARE_HYPERCALL macro should be allowed. Signed-off-by: Alexey Makhalov --- arch/x86/include/asm/vmware.h | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/x86/include/asm/vmware.h b/arch/x86/include/asm/vmware.h index 2ac87068184a..84a31f579a30 100644 ---

[PATCH v8 4/7] input/vmmouse: Use VMware hypercall API

2024-04-22 Thread Alexey Makhalov
Switch from VMWARE_HYPERCALL macro to vmware_hypercall API. Eliminate arch specific code. No functional changes intended. Signed-off-by: Alexey Makhalov Reviewed-by: Nadav Amit Reviewed-by: Zack Rusin Acked-by: Dmitry Torokhov --- drivers/input/mouse/vmmouse.c | 78

[PATCH v8 2/7] x86/vmware: Introduce VMware hypercall API

2024-04-22 Thread Alexey Makhalov
Introduce vmware_hypercall family of functions. It is a common implementation to be used by the VMware guest code and virtual device drivers in architecture independent manner. The API consists of vmware_hypercallX and vmware_hypercall_hb_{out,in} set of functions by analogy with KVM hypercall

[PATCH v8 3/7] ptp/vmware: Use VMware hypercall API

2024-04-22 Thread Alexey Makhalov
Switch from VMWARE_HYPERCALL macro to vmware_hypercall API. Eliminate arch specific code. No functional changes intended. Signed-off-by: Alexey Makhalov Reviewed-by: Nadav Amit Reviewed-by: Jeff Sipek --- drivers/ptp/ptp_vmw.c | 14 +++--- 1 file changed, 3 insertions(+), 11

[PATCH v8 1/7] x86/vmware: Move common macros to vmware.h

2024-04-22 Thread Alexey Makhalov
Move VMware hypercall macros to vmware.h. This is a prerequisite for the introduction of vmware_hypercall API. No functional changes besides exporting vmware_hypercall_mode symbol. Signed-off-by: Alexey Makhalov Reviewed-by: Nadav Amit --- arch/x86/include/asm/vmware.h | 72

[PATCH v8 0/7] VMware hypercalls enhancements

2024-04-22 Thread Alexey Makhalov
No functional changes from version 7. Peter please consider reviewing patch 7 where we addressed your comments from version 6. Thanks! VMware hypercalls invocations were all spread out across the kernel implementing same ABI as in-place asm-inline. With encrypted memory and confidential computing

[PATCH 2/2] drm/msm/dsi: Remove dsi_phy_write_[un]delay()

2024-04-22 Thread Konrad Dybcio
These are dummy wrappers that do literally nothing interesting. Remove them. Signed-off-by: Konrad Dybcio --- drivers/gpu/drm/msm/dsi/phy/dsi_phy.h | 3 -- drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c | 3 +- drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm.c | 81 +++--- 3

[PATCH 0/2] Remove more useless wrappers

2024-04-22 Thread Konrad Dybcio
Shaving off some cruft obj files seem to be identical pre and post cleanup which is always a good sign Signed-off-by: Konrad Dybcio --- Konrad Dybcio (2): drm/msm/dsi: Remove dsi_phy_read/write() drm/msm/dsi: Remove dsi_phy_write_[un]delay() drivers/gpu/drm/msm/dsi/phy/dsi_phy.h

[PATCH 1/2] drm/msm/dsi: Remove dsi_phy_read/write()

2024-04-22 Thread Konrad Dybcio
These are dummy wrappers that do literally nothing interesting. Remove them. Signed-off-by: Konrad Dybcio --- drivers/gpu/drm/msm/dsi/phy/dsi_phy.h | 2 - drivers/gpu/drm/msm/dsi/phy/dsi_phy_10nm.c | 273 +--- drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c |

Re: [PATCH v2 3/7] arm64: defconfig: Enable HIMAX_HX83102 panel

2024-04-22 Thread Doug Anderson
Hi, On Mon, Apr 22, 2024 at 2:03 AM Cong Yang wrote: > > DRM_PANEL_HIMAX_HX83102 is being split out from DRM_PANEL_BOE_TV101WUM_NL6. > Since the arm64 defconfig had the BOE panel driver enabled, let's also > enable the himax driver. > > Signed-off-by: Cong Yang > --- >

Re: [PATCH v2 2/7] drm/panel: himax-hx83102: Break out as separate driver

2024-04-22 Thread Doug Anderson
Hi, On Mon, Apr 22, 2024 at 2:03 AM Cong Yang wrote: > > The Starry HX83102 based mipi panel should never have been part of the boe > tv101wum driver. Discussion with Doug and Linus in V1 [1], we need a > separate driver to enable the hx83102 controller. > > In hx83102 driver, add DSI commands

Re: [PATCH v2 1/7] dt-bindings: display: panel: Add himax hx83102 panel bindings

2024-04-22 Thread Doug Anderson
Hi, On Mon, Apr 22, 2024 at 8:14 AM Rob Herring wrote: > > > +additionalProperties: false > > Perhaps 'unevaluatedProperties' instead. Don't you want to support > standard properties such as width-mm/height-mm? For at least those two properties, it looks like the answer is "no". >From reading

Re: [PATCH 2/2] dt-bindings: panel-simple-dsi: Add generic panel-dsi

2024-04-22 Thread Krzysztof Kozlowski
On 22/04/2024 16:34, Johan Adolfsson wrote: > > > -Original Message- > From: Krzysztof Kozlowski > Sent: den 19 april 2024 01:05 > To: Johan Adolfsson ; Neil Armstrong > ; Jessica Zhang ; Sam > Ravnborg ; Maarten Lankhorst > ; Maxime Ripard ; > Thomas Zimmermann ; David Airlie ; >

Re: [PATCH v3 1/4] drm: add devm release action

2024-04-22 Thread Rodrigo Vivi
On Mon, Apr 22, 2024 at 12:27:53PM +0530, Aravind Iddamsetty wrote: > In scenarios where drm_dev_put is directly called by driver we want to > release devm_drm_dev_init_release action associated with struct > drm_device. > > v2: Directly expose the original function, instead of introducing a >

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Arnd Bergmann
On Mon, Apr 22, 2024, at 21:42, Masahiro Yamada wrote: > On Tue, Apr 23, 2024 at 3:24 AM Arnd Bergmann wrote: >> On Mon, Apr 22, 2024, at 18:58, Geert Uytterhoeven wrote: >> > On Mon, Apr 22, 2024 at 3:55 PM Arnd Bergmann wrote: >> >> On Mon, Apr 22, 2024, at 15:28, Jani Nikula wrote: >> >> I

[PATCH v2 1/2] drm/i915: Refactor confusing __intel_gt_reset()

2024-04-22 Thread Nirmoy Das
__intel_gt_reset() is really for resetting engines though the name might suggest something else. So add a helper function to remove confusions with no functional changes. v2: Move intel_gt_reset_all_engines() next to intel_gt_reset_engine() to make diff simple(John) Cc: John Harrison

[PATCH v2 2/2] drm/i915: Fix gt reset with GuC submission is disabled

2024-04-22 Thread Nirmoy Das
Currently intel_gt_reset() kills the GuC and then resets requested engines. This is problematic because there is a dedicated CSB FIFO which only GuC can access and if that FIFO fills up, the hardware will block on the next context switch until there is space that means the system is effectively

Re: [PATCH v4 8/9] drm/bridge: tfp410: Use fwnode API to acquire device properties

2024-04-22 Thread Dmitry Baryshkov
On Tue, Apr 23, 2024 at 03:19:02AM +0800, Sui Jingfeng wrote: > Make this driver DT-independent by calling the freshly created helpers, > which reduce boilerplate and open the door for otherwise use cases. No > functional changes for DT based systems. > > Signed-off-by: Sui Jingfeng > --- >

Re: [PATCH v4 7/9] drm-bridge: it66121: Use fwnode API to acquire device properties

2024-04-22 Thread Dmitry Baryshkov
On Tue, Apr 23, 2024 at 03:19:01AM +0800, Sui Jingfeng wrote: > Make this driver DT-independent by calling the freshly created helpers, > which reduce boilerplate and open the door for otherwise use cases. No > functional changes for DT based systems. > > Signed-off-by: Sui Jingfeng > --- >

Re: [PATCH v4 6/9] drm-bridge: sii902x: Use fwnode API to acquire device properties

2024-04-22 Thread Dmitry Baryshkov
On Tue, Apr 23, 2024 at 03:19:00AM +0800, Sui Jingfeng wrote: > Make this driver less DT-dependent by calling the freshly created helper > functions, which reduce boilerplate. Should be no functional changes for > DT based systems. > > Signed-off-by: Sui Jingfeng > --- >

Re: [PATCH v4 5/9] drm/bridge: display-connector: Add platform module alias

2024-04-22 Thread Dmitry Baryshkov
On Tue, Apr 23, 2024 at 03:18:59AM +0800, Sui Jingfeng wrote: > Otherwise when compiled as module, this driver will not be probed on > non-DT environment. This is a fundamential step to make this driver > truely OF-independent. > > Signed-off-by: Sui Jingfeng > --- >

Re: [PATCH v4 4/9] drm-bridge: display-connector: Use fwnode API to acquire device properties

2024-04-22 Thread Dmitry Baryshkov
On Tue, Apr 23, 2024 at 03:18:58AM +0800, Sui Jingfeng wrote: > Make this driver less DT-dependent by using the fwnode helper functions, > should be no functional changes for DT based systems. Do the necessary > works before it can be truely DT-independent, this patch do no harms even > though

Re: [PATCH v4 3/9] drm/bridge: simple-bridge: Add platform module alias

2024-04-22 Thread Dmitry Baryshkov
On Tue, Apr 23, 2024 at 03:18:57AM +0800, Sui Jingfeng wrote: > Otherwise when compiled as module, this driver will not be probed on > non-DT environment. This is a fundamential step to make this driver > truely OF-independent. > > Signed-off-by: Sui Jingfeng > --- >

Re: [PATCH v4 2/9] drm/bridge: simple-bridge: Use fwnode API to acquire device properties

2024-04-22 Thread Dmitry Baryshkov
On Tue, Apr 23, 2024 at 03:18:56AM +0800, Sui Jingfeng wrote: > Make this driver less DT-dependent by calling the freshly created helpers, > should be no functional changes for DT based systems. But open the door for > otherwise use cases. Even though there is no user emerged yet, this still > do

Re: [PATCH v4 1/9] drm/bridge: Allow using fwnode API to get the next bridge

2024-04-22 Thread Dmitry Baryshkov
On Tue, Apr 23, 2024 at 03:18:55AM +0800, Sui Jingfeng wrote: > Currently, the various display bridge drivers rely on OF infrastructures > to works very well, yet there are platforms and/or devices absence of 'OF' > support. Such as virtual display drivers, USB display apapters and ACPI > based

Re: [PATCH V2 2/2] drm/bridge: samsung-dsim: Fix porch calcalcuation rounding

2024-04-22 Thread Marek Vasut
On 4/22/24 3:04 PM, Adam Ford wrote: On Mon, Apr 22, 2024 at 8:01 AM Marek Vasut wrote: On 4/22/24 2:09 PM, Adam Ford wrote: On Sun, Apr 21, 2024 at 9:36 AM Marek Vasut wrote: On 2/12/24 12:09 AM, Adam Ford wrote: When using video sync pulses, the HFP, HBP, and HSA are divided between

Re: [PATCH 2/3] drm/msm/mdp4: don't destroy mdp4_kms in mdp4_kms_init error path

2024-04-22 Thread Dmitry Baryshkov
On Mon, Apr 22, 2024 at 11:17:15AM -0700, Abhinav Kumar wrote: > > > On 4/19/2024 7:33 PM, Dmitry Baryshkov wrote: > > Since commit 3c74682637e6 ("drm/msm/mdp4: move resource allocation to > > the _probe function") the mdp4_kms data is allocated during probe. It is > > an error to destroy it

Re: [PATCH 1/3] drm/msm: don't clean up priv->kms prematurely

2024-04-22 Thread Dmitry Baryshkov
On Mon, Apr 22, 2024 at 09:12:20AM -0700, Abhinav Kumar wrote: > > > On 4/21/2024 3:35 PM, Dmitry Baryshkov wrote: > > On Sat, Apr 20, 2024 at 04:02:00PM -0700, Abhinav Kumar wrote: > > > > > > > > > On 4/19/2024 7:33 PM, Dmitry Baryshkov wrote: > > > > MSM display drivers provide kms

Re: [PATCH 1/3] drm/msm: don't clean up priv->kms prematurely

2024-04-22 Thread Dmitry Baryshkov
On Mon, Apr 22, 2024 at 10:23:18AM -0700, Abhinav Kumar wrote: > > > On 4/21/2024 3:35 PM, Dmitry Baryshkov wrote: > > On Sat, Apr 20, 2024 at 04:02:00PM -0700, Abhinav Kumar wrote: > > > > > > > > > On 4/19/2024 7:33 PM, Dmitry Baryshkov wrote: > > > > MSM display drivers provide kms

Re: [PATCH 1/2] drm/panel: panel-simple: Add generic panel-dsi driver

2024-04-22 Thread Dmitry Baryshkov
On Mon, Apr 22, 2024 at 02:05:01PM +, Johan Adolfsson wrote: > Hi, > Sorry for delayed response, still investigating why these mails didn't reach > my inbox as expected.. > > -Original Message- > From: Krzysztof Kozlowski > Sent: den 19 april 2024 01:06 > To: Johan Adolfsson ; Neil

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Masahiro Yamada
On Tue, Apr 23, 2024 at 3:24 AM Arnd Bergmann wrote: > > On Mon, Apr 22, 2024, at 18:58, Geert Uytterhoeven wrote: > > On Mon, Apr 22, 2024 at 3:55 PM Arnd Bergmann wrote: > >> On Mon, Apr 22, 2024, at 15:28, Jani Nikula wrote: > >> Whereas this one is broken: > >> > >> config FEATURE_A > >>

Re: [PATCH 1/1] video: Handle HAS_IOPORT dependencies

2024-04-22 Thread Arnd Bergmann
On Mon, Apr 22, 2024, at 10:34, Niklas Schnelle wrote: > On Thu, 2024-04-11 at 16:00 +0200, Helge Deller wrote: >> * Niklas Schnelle : >> > In a future patch HAS_IOPORT=n will disable inb()/outb() and friends at >> > compile time. We thus need to #ifdef functions and their callsites which >> >

[PATCH v4 9/9] drm/bridge: tfp410: Add platform module alias

2024-04-22 Thread Sui Jingfeng
Otherwise when compiled as module, this driver will not be probed on non-DT environment. This is a fundamential step to make this driver truely OF-independent. Signed-off-by: Sui Jingfeng --- drivers/gpu/drm/bridge/ti-tfp410.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH v4 8/9] drm/bridge: tfp410: Use fwnode API to acquire device properties

2024-04-22 Thread Sui Jingfeng
Make this driver DT-independent by calling the freshly created helpers, which reduce boilerplate and open the door for otherwise use cases. No functional changes for DT based systems. Signed-off-by: Sui Jingfeng --- drivers/gpu/drm/bridge/ti-tfp410.c | 41 +++--- 1 file

[PATCH v4 7/9] drm-bridge: it66121: Use fwnode API to acquire device properties

2024-04-22 Thread Sui Jingfeng
Make this driver DT-independent by calling the freshly created helpers, which reduce boilerplate and open the door for otherwise use cases. No functional changes for DT based systems. Signed-off-by: Sui Jingfeng --- drivers/gpu/drm/bridge/ite-it66121.c | 57 +--- 1 file

[PATCH v4 6/9] drm-bridge: sii902x: Use fwnode API to acquire device properties

2024-04-22 Thread Sui Jingfeng
Make this driver less DT-dependent by calling the freshly created helper functions, which reduce boilerplate. Should be no functional changes for DT based systems. Signed-off-by: Sui Jingfeng --- drivers/gpu/drm/bridge/sii902x.c | 43 +++- 1 file changed, 14

[PATCH v4 5/9] drm/bridge: display-connector: Add platform module alias

2024-04-22 Thread Sui Jingfeng
Otherwise when compiled as module, this driver will not be probed on non-DT environment. This is a fundamential step to make this driver truely OF-independent. Signed-off-by: Sui Jingfeng --- drivers/gpu/drm/bridge/display-connector.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH v4 4/9] drm-bridge: display-connector: Use fwnode API to acquire device properties

2024-04-22 Thread Sui Jingfeng
Make this driver less DT-dependent by using the fwnode helper functions, should be no functional changes for DT based systems. Do the necessary works before it can be truely DT-independent, this patch do no harms even though there is no user yet. Signed-off-by: Sui Jingfeng ---

[PATCH v4 3/9] drm/bridge: simple-bridge: Add platform module alias

2024-04-22 Thread Sui Jingfeng
Otherwise when compiled as module, this driver will not be probed on non-DT environment. This is a fundamential step to make this driver truely OF-independent. Signed-off-by: Sui Jingfeng --- drivers/gpu/drm/bridge/simple-bridge.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH v4 2/9] drm/bridge: simple-bridge: Use fwnode API to acquire device properties

2024-04-22 Thread Sui Jingfeng
Make this driver less DT-dependent by calling the freshly created helpers, should be no functional changes for DT based systems. But open the door for otherwise use cases. Even though there is no user emerged yet, this still do no harms. Signed-off-by: Sui Jingfeng ---

[PATCH v4 1/9] drm/bridge: Allow using fwnode API to get the next bridge

2024-04-22 Thread Sui Jingfeng
Currently, the various display bridge drivers rely on OF infrastructures to works very well, yet there are platforms and/or devices absence of 'OF' support. Such as virtual display drivers, USB display apapters and ACPI based systems etc. Add fwnode based helpers to fill the niche, this allows

[PATCH v4 0/9] drm/bridge: Allow using fwnode API to get the next bridge

2024-04-22 Thread Sui Jingfeng
Currently, the various display bridge drivers rely on OF infrastructures to works very well, yet there are platforms and/or devices absence of 'OF' support. Such as virtual display drivers, USB display apapters and ACPI based systems etc. Add fwnode based helpers to fill the niche, this allows

[linux-next:master] BUILD REGRESSION f529a6d274b3b8c75899e949649d231298f30a32

2024-04-22 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master branch HEAD: f529a6d274b3b8c75899e949649d231298f30a32 Add linux-next specific files for 20240422 Error/Warning reports: https://lore.kernel.org/oe-kbuild-all/202404221830.cjqlhldl-...@intel.com Error

Re: [PATCH v1 1/1] drm: fixed: Don't use "proxy" headers

2024-04-22 Thread Jani Nikula
On Mon, 22 Apr 2024, Andy Shevchenko wrote: > Update header inclusions to follow IWYU (Include What You Use) > principle. > > Signed-off-by: Andy Shevchenko Assuming it builds, and nothing depends on other stuff from kernel.h via drm_fixed.h, Reviewed-by: Jani Nikula > --- >

Re: [PATCH v3 00/13] drm: zynqmp_dp: IRQ cleanups and debugfs support

2024-04-22 Thread Sean Anderson
On 4/22/24 14:45, Sean Anderson wrote: > This series cleans up the zyqnmp_dp IRQ and locking situation. Once > that's done, it adds debugfs support. The intent is to enable compliance > testing or to help debug signal-integrity issues. > > Last time I discussed converting the HPD work(s) to a

[PATCH v3 12/13] drm: zynqmp_dp: Take dp->lock in zynqmp_dp_hpd_work_func

2024-04-22 Thread Sean Anderson
Add a non-locking version of zynqmp_dp_bridge_detect and use it in zynqmp_dp_hpd_work_func so we can take the lock explicitly. This will make it easier to check for hpd_ignore when we add debugfs support. Signed-off-by: Sean Anderson --- Changes in v3: - New drivers/gpu/drm/xlnx/zynqmp_dp.c |

[PATCH v3 06/13] drm: zynqmp_dp: Don't delay work

2024-04-22 Thread Sean Anderson
We always call scheduled_delayed_work with no delay, so just use a non-delayed work_struct instead. Signed-off-by: Sean Anderson --- Changes in v3: - New drivers/gpu/drm/xlnx/zynqmp_dp.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git

[PATCH v3 10/13] drm: zynqmp_dp: Use AUX IRQs instead of polling

2024-04-22 Thread Sean Anderson
Instead of polling the status register for the AUX status, just enable the IRQs and signal a completion. Signed-off-by: Sean Anderson --- Changes in v3: - New drivers/gpu/drm/xlnx/zynqmp_dp.c | 35 +++- 1 file changed, 25 insertions(+), 10 deletions(-) diff --git

[PATCH v3 13/13] drm: zynqmp_dp: Add debugfs interface for compliance testing

2024-04-22 Thread Sean Anderson
Add a debugfs interface for exercising the various test modes supported by the DisplayPort controller. This allows performing compliance testing, or performing signal integrity measurements on a failing link. At the moment, we do not support sink-driven link quality testing, although such support

[PATCH v3 11/13] drm: zynqmp_dp: Split off several helper functions

2024-04-22 Thread Sean Anderson
In preparation for supporting compliance testing, split off several helper functions. No functional change intended. Signed-off-by: Sean Anderson Reviewed-by: Laurent Pinchart Reviewed-by: Tomi Valkeinen --- (no changes since v1) drivers/gpu/drm/xlnx/zynqmp_dp.c | 49

[PATCH v3 07/13] drm: zynqmp_dp: Add locking

2024-04-22 Thread Sean Anderson
Add some locking to prevent the IRQ/workers/bridge API calls from stepping on each other's toes. This lock protects: - Non-atomic registers configuring the link. That is, everything but the IRQ registers (since these are accessed in an atomic fashion), and the DP AUX registers (since these

[PATCH v3 05/13] drm: zynqmp_dp: Rearrange zynqmp_dp for better padding

2024-04-22 Thread Sean Anderson
Sort the members of struct zynqmp_dp to reduce padding necessary for alignment. Signed-off-by: Sean Anderson --- (no changes since v2) Changes in v2: - New drivers/gpu/drm/xlnx/zynqmp_dp.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git

[PATCH v3 09/13] drm: zynqmp_dp: Convert to a hard IRQ

2024-04-22 Thread Sean Anderson
Now that all of the sleeping work is done outside of the IRQ, we can convert it to a hard IRQ. Signed-off-by: Sean Anderson --- Changes in v3: - New drivers/gpu/drm/xlnx/zynqmp_dp.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/xlnx/zynqmp_dp.c

[PATCH v3 04/13] drm: zynqmp_dp: Adjust training values per-lane

2024-04-22 Thread Sean Anderson
The feedback we get from the DPRX is per-lane. Make changes using this information, instead of picking the maximum values from all lanes. This results in more-consistent training on marginal links. Signed-off-by: Sean Anderson Reviewed-by: Tomi Valkeinen --- (no changes since v1)

[PATCH v3 08/13] drm: zynqmp_dp: Don't retrain the link in our IRQ

2024-04-22 Thread Sean Anderson
Retraining the link can take a while, and might involve waiting for DPCD reads/writes to complete. In preparation for unthreading the IRQ handler, move this into its own work function. Signed-off-by: Sean Anderson --- (no changes since v2) Changes in v2: - Document hpd_irq_work - Split this

[PATCH v3 03/13] drm: zynqmp_dp: Downgrade log level for aux retries message

2024-04-22 Thread Sean Anderson
Enable this message for verbose debugging only as it is otherwise printed after every AUX message, quickly filling the log buffer. Signed-off-by: Sean Anderson Reviewed-by: Laurent Pinchart Reviewed-by: Tomi Valkeinen --- (no changes since v1) drivers/gpu/drm/xlnx/zynqmp_dp.c | 2 +- 1 file

[PATCH v3 00/13] drm: zynqmp_dp: IRQ cleanups and debugfs support

2024-04-22 Thread Sean Anderson
This series cleans up the zyqnmp_dp IRQ and locking situation. Once that's done, it adds debugfs support. The intent is to enable compliance testing or to help debug signal-integrity issues. Last time I discussed converting the HPD work(s) to a threaded IRQ. I did not end up doing that for this

[PATCH v3 02/13] drm: xlnx: Fix kerneldoc

2024-04-22 Thread Sean Anderson
Fix a few errors in the kerneldoc. Mostly this addresses missing/renamed members. Signed-off-by: Sean Anderson --- Changes in v3: - Split off documentation for base pointers to previous commit Changes in v2: - New drivers/gpu/drm/xlnx/zynqmp_dpsub.h | 1 + drivers/gpu/drm/xlnx/zynqmp_kms.h

[PATCH v3 01/13] drm: xlnx: Store base pointers in zynqmp_disp directly

2024-04-22 Thread Sean Anderson
The blend, avbuf, and audio members of zynqmp_disp are anonymous structs with only one member each. This is rather pointless, so move the members up a level. Signed-off-by: Sean Anderson --- Changes in v3: - New drivers/gpu/drm/xlnx/zynqmp_disp.c | 44 +- 1 file

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Arnd Bergmann
On Mon, Apr 22, 2024, at 18:58, Geert Uytterhoeven wrote: > On Mon, Apr 22, 2024 at 3:55 PM Arnd Bergmann wrote: >> On Mon, Apr 22, 2024, at 15:28, Jani Nikula wrote: >> Whereas this one is broken: >> >> config FEATURE_A >>tristate "user visible if I2C is enabled" >>depends on I2C

Re: [PATCH 2/3] drm/msm/mdp4: don't destroy mdp4_kms in mdp4_kms_init error path

2024-04-22 Thread Abhinav Kumar
On 4/19/2024 7:33 PM, Dmitry Baryshkov wrote: Since commit 3c74682637e6 ("drm/msm/mdp4: move resource allocation to the _probe function") the mdp4_kms data is allocated during probe. It is an error to destroy it during mdp4_kms_init(), as the data is still referenced by the drivers's data and

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Geert Uytterhoeven
Hi Jani, CC kbuild On Mon, Apr 22, 2024 at 7:00 PM Jani Nikula wrote: > On Mon, 22 Apr 2024, "Arnd Bergmann" wrote: > > I'm not sure where this misunderstanding comes from, as you > > seem to be repeating the same incorrect assumption about > > how select works that Maxime wrote in his

  1   2   3   >