Re: [RFC 00/11] THP support for zone device pages

2025-03-06 Thread Balbir Singh
On 3/7/25 10:08, Matthew Brost wrote: > On Thu, Mar 06, 2025 at 03:42:28PM +1100, Balbir Singh wrote: > > This is an exciting series to see. As of today, we have just merged this > series into the DRM subsystem / Xe [2], which adds very basic SVM > support. One of the performance bottlenecks we qu

[git pull] drm fixes for 6.14-rc6

2025-03-06 Thread Dave Airlie
Hey Linus, Looks like the cyclone is taking its time getting here, so I can at least get the drm fixes tree out. Fixes across the board, mostly xe and imagination with some amd and misc others. The xe fixes are mostly hmm related, though there are some others in there as well, nothing really stan

Re: [PATCH v8 0/7] drm/msm: make use of the HDMI connector infrastructure

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov On Wed, 26 Feb 2025 10:59:23 +0200, Dmitry Baryshkov wrote: > This patchset sits on top Maxime's HDMI connector patchset ([1]). > > Currently this is an RFC exploring the interface between HDMI bridges > and HDMI connector code. This has been lightly verified on the Qualc

Re: [PATCH v3 00/16] Introduce and use generic parity16/32/64 helper

2025-03-06 Thread H. Peter Anvin
On March 6, 2025 8:25:25 AM PST, Kuan-Wei Chiu wrote: >Several parts of the kernel contain redundant implementations of parity >calculations for 16/32/64-bit values. Introduces generic >parity16/32/64() helpers in bitops.h, providing a standardized >and optimized implementation. > >Subsequent pat

Re: [PATCH 1/2] drm/bridge: imx8qxp-ldb: cleanup return value

2025-03-06 Thread Liu Ying
On 03/07/2025, Luca Ceresoli wrote: > 'ret' can only be 0 at this point, being preceded by a 'if (ret) return > ret;'. So return 0 for clarity. > > Signed-off-by: Luca Ceresoli > --- > drivers/gpu/drm/bridge/imx/imx8qxp-ldb.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --gi

Re: [PATCH v3 01/16] bitops: Change parity8() return type to bool

2025-03-06 Thread Jiri Slaby
On 06. 03. 25, 17:25, Kuan-Wei Chiu wrote: Change return type to bool for better clarity. Update the kernel doc comment accordingly, including fixing "@value" to "@val" and adjusting examples. Also mark the function with __attribute_const__ to allow potential compiler optimizations. Co-developed

Re: [PATCH -next] drm/msm/dpu: Remove duplicate dpu_hw_cwb.h header

2025-03-06 Thread Dmitry Baryshkov
On Fri, Mar 07, 2025 at 09:50:30AM +0800, Jiapeng Chong wrote: > ./drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c: dpu_hw_cwb.h is included more > than once. > > Reported-by: Abaci Robot > Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=19239 > Signed-off-by: Jiapeng Chong > --- > drivers/

Re: [PATCH v3 00/16] Introduce and use generic parity16/32/64 helper

2025-03-06 Thread Jiri Slaby
On 06. 03. 25, 17:25, Kuan-Wei Chiu wrote: Several parts of the kernel contain redundant implementations of parity calculations for 16/32/64-bit values. Introduces generic parity16/32/64() helpers in bitops.h, providing a standardized and optimized implementation. Subsequent patches refactor var

Re: [RFC PATCH 0/3] gpu: nova-core: add basic timer subdevice implementation

2025-03-06 Thread Jason Gunthorpe
On Thu, Mar 06, 2025 at 11:42:38AM +0100, Simona Vetter wrote: > > Further, I just remembered, (Danilo please notice!) there is another > > related issue here that DMA mappings *may not* outlive remove() > > either. netdev had a bug related to this recently and it was all > > agreed that it is not

Re: [PATCH v6 00/14] drm/msm/dpu: Add Concurrent Writeback Support for DPU 10.x+

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov On Fri, 14 Feb 2025 16:14:23 -0800, Jessica Zhang wrote: > DPU supports a single writeback session running concurrently with primary > display when the CWB mux is configured properly. This series enables > clone mode for DPU driver and adds support for programming the CWB

Re:Re: [PATCH 0/6] Add support for RK3588 DisplayPort Controller

2025-03-06 Thread Andy Yan
Hi Piotr, 在 2025-03-06 22:28:08,"Piotr Oniszczuk" 写道: > > >> Wiadomość napisana przez Piotr Oniszczuk w dniu >> 6 mar 2025, o godz. 15:08: >> >> >> >>> Wiadomość napisana przez Andy Yan w dniu 6 mar 2025, o >>> godz. 13:15: >>> >>> Hi Piotr, >>> >>> >>> >>> Then when you DP cable plugi

[PATCH 03/11] dt-bindings: power: qcom,kpss-acc-v2: Add MSM8916 compatible

2025-03-06 Thread Konrad Dybcio
From: Konrad Dybcio MSM8916 seems to reuse the same hardware as MSM8974 and friends (for whom this binding document was created). Add a new compatible for it. Signed-off-by: Konrad Dybcio --- Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml | 4 +++- 1 file changed, 3 insertions(+

Re: [PATCH v2 0/4] drm/msm/dpu: follow rules for drm_atomic_helper_check_modeset()

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov On Thu, 23 Jan 2025 14:43:32 +0200, Dmitry Baryshkov wrote: > As pointed out by Simona, the drm_atomic_helper_check_modeset() and > drm_atomic_helper_check() require the former function is rerun if the > driver's callbacks modify crtc_state->mode_changed. MSM is one of the

Re: [PATCH v2] dt-bindings: display/msm: qcom, sa8775p-mdss: Add missing eDP phy

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov On Fri, 21 Feb 2025 16:13:11 +0100, Krzysztof Kozlowski wrote: > The Qualcomm SA8775p MDSS display block comes with eDP phy, already used > in DTS and already documented in phy/qcom,edp-phy.yaml binding. Add the > missing device node in the binding and extend example to s

Re: [PATCH next] drm/msm/dpu: fix error pointer dereference in msm_kms_init_aspace()

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov On Tue, 25 Feb 2025 10:30:26 +0300, Dan Carpenter wrote: > If msm_gem_address_space_create() fails, then return right away. > Otherwise it leads to a Oops when we dereference "aspace" on the next > line. > > Applied, thanks! [1/1] drm/msm/dpu: fix error pointer derefer

Re: [PATCH 04/11] arm64: dts: qcom: msm8916: Fix KPSS ACC compatible

2025-03-06 Thread Dmitry Baryshkov
On Thu, Mar 06, 2025 at 07:11:16PM +0100, Konrad Dybcio wrote: > From: Konrad Dybcio > > The current compatible has been used with no corresponding > documentation. Replace it with one that has been documented. qcom,msm8916-kpss-acc is also not documented. Most likely you meant qcom,kpss-acc-v2

Re: [PATCH 10/11] arm64: dts: qcom: x1e80100-romulus: Drop clock-names from PS8830

2025-03-06 Thread Dmitry Baryshkov
On Thu, Mar 06, 2025 at 07:11:22PM +0100, Konrad Dybcio wrote: > From: Konrad Dybcio > > The preemptively-merged node contains a property absent from the final > bindings. Remove it. > > Signed-off-by: Konrad Dybcio > --- > arch/arm64/boot/dts/qcom/x1e80100-microsoft-romulus.dtsi | 2 -- > 1 f

[PATCH v5 2/2] drm/msm/dp: reuse generic HDMI codec implementation

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov The MSM DisplayPort driver implements several HDMI codec functions in the driver, e.g. it manually manages HDMI codec device registration, returning ELD and plugged_cb support. In order to reduce code duplication reuse drm_hdmi_audio_* helpers and drm_bridge_connector integ

Re: [PATCH 11/11] arm64: dts: qcom: x1e001de-devkit: Drop clock-names from PS8830

2025-03-06 Thread Dmitry Baryshkov
On Thu, Mar 06, 2025 at 07:11:23PM +0100, Konrad Dybcio wrote: > From: Konrad Dybcio > > The preemptively-merged node contains a property absent from the final > bindings. Remove it. > > Signed-off-by: Konrad Dybcio > --- > arch/arm64/boot/dts/qcom/x1e001de-devkit.dts | 3 --- > 1 file changed

Re: [PATCH 03/11] dt-bindings: power: qcom,kpss-acc-v2: Add MSM8916 compatible

2025-03-06 Thread Dmitry Baryshkov
On Thu, Mar 06, 2025 at 07:11:15PM +0100, Konrad Dybcio wrote: > From: Konrad Dybcio > > MSM8916 seems to reuse the same hardware as MSM8974 and friends (for > whom this binding document was created). Add a new compatible for it. Ok, I should have read dt-bindigns before sending a comment. But t

[PATCH v5 1/2] drm/bridge: split HDMI Audio from DRM_BRIDGE_OP_HDMI

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov As pointed out by Laurent, OP bits are supposed to describe operations. Split DRM_BRIDGE_OP_HDMI_AUDIO from DRM_BRIDGE_OP_HDMI instead of overloading DRM_BRIDGE_OP_HDMI. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/bridge/lontium-lt9611.c| 2 +- drivers/g

Re: [PATCH 0/6] Add support for RK3588 DisplayPort Controller

2025-03-06 Thread Piotr Oniszczuk
> Wiadomość napisana przez Andy Yan w dniu 6 mar 2025, o > godz. 01:59: > > > > > Both of the two config options should be enabled. > andy@Pro480:~/WorkSpace/linux-next$ rg DW_DP .config > 4044:CONFIG_ROCKCHIP_DW_DP=y here i’m a bit lost…. greping on full kernel sources (with applied htt

Re: [PATCH 1/2] dt-bindings: drm/bridge: ti-sn65dsi83: add h/vsync-disable bindings

2025-03-06 Thread Dmitry Baryshkov
On Thu, Mar 06, 2025 at 10:11:32AM +0100, A. Zini wrote: > From: Alessandro Zini > > Add hsync- and vsync-disable bindings, used to disable the generation of > h/vsync signals. Please describe, why this is necessary at all, instead of desribing the contents of the patch. > > Signed-off-by: Ale

Re: [PATCH drm-next 1/2] vmalloc: Add atomic_vmap

2025-03-06 Thread Jocelyn Falempe
On 06/03/2025 16:52, Simona Vetter wrote: On Thu, Mar 06, 2025 at 02:24:51PM +0100, Jocelyn Falempe wrote: On 06/03/2025 05:52, Matthew Wilcox wrote: On Thu, Mar 06, 2025 at 12:25:53AM +0900, Ryosuke Yasuoka wrote: Some drivers can use vmap in drm_panic, however, vmap is sleepable and takes lo

Re: [PATCH v2 0/2] Fix native cursors with vmwgfx

2025-03-06 Thread Maaz Mombasawala
On 3/5/25 20:11, Zack Rusin wrote: > vmwgfx had a number of cursor issues that related both to our handling > of dumb buffers and general detection when a cursor has actually changed. > Fix those issues and bump the kernel module version to allow userspace > to recognize fixed versions of the drive

Re:Re:Re: [PATCH v5 04/16] drm/atomic: Introduce helper to lookup connector by encoder

2025-03-06 Thread Andy Yan
Hi All, At 2025-03-07 09:08:48, "Andy Yan" wrote: >Hi All, > >At 2025-03-06 23:41:24, "Simona Vetter" wrote: >>On Thu, Mar 06, 2025 at 08:10:16AM +0100, Maxime Ripard wrote: >>> On Thu, Mar 06, 2025 at 09:16:24AM +0800, Andy Yan wrote: >>> > >>> > Hi Maxime and Dmitry: >>> > >>> > At 2025-03-0

[PATCH v3 4/8] drm/msm/dpu: use single CTL if it is the only CTL returned by RM

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov On DPU >= 5.0 CTL blocks were reworked in order to support using a single CTL for all outputs. In preparation of reworking the RM code to return single CTL make sure that dpu_encoder can cope with that. Reviewed-by: Marijn Suijten Tested-by: Neil Armstrong # on SM8550-QR

Re: [PATCH] drm/ast: Support both SHMEM helper and VRAM helper

2025-03-06 Thread Huacai Chen
On Wed, Mar 5, 2025 at 5:04 PM Thomas Zimmermann wrote: > > Hi > > Am 04.03.25 um 12:55 schrieb Huacai Chen: > > On Tue, Mar 4, 2025 at 5:39 PM Thomas Zimmermann > > wrote: > >> Hi > >> > >> Am 04.03.25 um 10:19 schrieb Huacai Chen: > >>> On Tue, Mar 4, 2025 at 4:41 PM Thomas Zimmermann > >>>

Re:Re: [PATCH v5 04/16] drm/atomic: Introduce helper to lookup connector by encoder

2025-03-06 Thread Andy Yan
Hi All, At 2025-03-06 23:41:24, "Simona Vetter" wrote: >On Thu, Mar 06, 2025 at 08:10:16AM +0100, Maxime Ripard wrote: >> On Thu, Mar 06, 2025 at 09:16:24AM +0800, Andy Yan wrote: >> > >> > Hi Maxime and Dmitry: >> > >> > At 2025-03-06 04:13:53, "Dmitry Baryshkov" >> > wrote: >> > >On Wed, Ma

[PATCH RFC v3 3/7] drm/display: dp: use new DCPD access helpers

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov Switch drm_dp_helper.c to use new set of DPCD read / write helpers. Reviewed-by: Lyude Paul Acked-by: Jani Nikula Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/display/drm_dp_helper.c | 296 +--- 1 file changed, 116 insertions(+), 180

[PATCH RFC v3 0/7] drm/display: dp: add new DPCD access functions

2025-03-06 Thread Dmitry Baryshkov
Existing DPCD access functions return an error code or the number of bytes being read / write in case of partial access. However a lot of drivers either (incorrectly) ignore partial access or mishandle error codes. In other cases this results in a boilerplate code which compares returned value with

[PATCH RFC v3 1/7] drm/display: dp: implement new access helpers

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov Existing DPCD access functions return an error code or the number of bytes being read / write in case of partial access. However a lot of drivers either (incorrectly) ignore partial access or mishandle error codes. In other cases this results in a boilerplate code which com

[PATCH RFC v3 2/7] drm/display: dp: change drm_dp_dpcd_read_link_status() return value

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov drm_dp_dpcd_read_link_status() follows the "return error code or number of bytes read" protocol, with the code returning less bytes than requested in case of some errors. However most of the drivers interpreted that as "return error code in case of any error". Switch drm_dp

[PATCH RFC v3 7/7] drm/display: dp-tunnel: use new DCPD access helpers

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov Switch drm_dp_tunnel.c to use new set of DPCD read / write helpers. Reviewed-by: Lyude Paul Acked-by: Jani Nikula Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/display/drm_dp_tunnel.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) d

[PATCH RFC v3 5/7] drm/display: dp-cec: use new DCPD access helpers

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov Switch drm_dp_cec.c to use new set of DPCD read / write helpers. Reviewed-by: Lyude Paul Acked-by: Jani Nikula Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/display/drm_dp_cec.c | 37 ++-- 1 file changed, 18 insertions(+), 19 delet

[PATCH RFC v3 6/7] drm/display: dp-mst-topology: use new DCPD access helpers

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov Switch drm_dp_mst_topology.c to use new set of DPCD read / write helpers. Reviewed-by: Lyude Paul Acked-by: Jani Nikula Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/display/drm_dp_mst_topology.c | 105 +- 1 file changed, 51 insertions(+),

[PATCH v3 0/8] drm/msm/dpu: improve CTL handling on DPU >= 5.0 platforms

2025-03-06 Thread Dmitry Baryshkov
Since version 5.0 the DPU got an improved way of handling multi-output configurations. It is now possible to program all pending changes through a single CTL and flush everything at the same time. Implement corresponding changes in the DPU driver. Signed-off-by: Dmitry Baryshkov --- Changes in v

[PATCH v3 1/8] drm/msm/dpu: don't overwrite CTL_MERGE_3D_ACTIVE register

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov In case of complex pipelines (e.g. the forthcoming quad-pipe) the DPU might use more that one MERGE_3D block for a single output. Follow the pattern and extend the CTL_MERGE_3D_ACTIVE active register instead of simply writing new value there. Currently at most one MERGE_3D

[PATCH v3 2/8] drm/msm/dpu: program master INTF value

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov If several interfaces are being handled through a single CTL, a main ('master') INTF needs to be programmed into a separate register. Write corresponding value into that register. Co-developed-by: Marijn Suijten Signed-off-by: Marijn Suijten Reviewed-by: Marijn Suijten

[PATCH v3 6/8] drm/msm/dpu: allocate single CTL for DPU >= 5.0

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov Unlike previous generation, since DPU 5.0 it is possible to use just one CTL to handle all INTF and WB blocks for a single output. And one has to use single CTL to support bonded DSI config. Allocate single CTL for these DPU versions. Reviewed-by: Marijn Suijten Tested-by

[PATCH v3 3/8] drm/msm/dpu: pass master interface to CTL configuration

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov Active controls require setup of the master interface. Pass the selected interface to CTL configuration. Reviewed-by: Marijn Suijten Tested-by: Neil Armstrong # on SM8550-QRD Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c | 2 +

[PATCH v3 8/8] drm/msm/dpu: drop now-unused condition for has_legacy_ctls

2025-03-06 Thread Dmitry Baryshkov
From: Dmitry Baryshkov Now as we have dropped the DPU_CTL_SPLIT_DISPLAY from DPU >= 5.0 configuration, drop the rm->has_legacy_ctl condition which short-cutted the check for those platforms. Suggested-by: Marijn Suijten Reviewed-by: Marijn Suijten Tested-by: Neil Armstrong # on SM8550-QRD Sig

Re: [PATCH 2/2] drm/bridge: fsl-ldb: make warning message more informative

2025-03-06 Thread Liu Ying
On 03/07/2025, Luca Ceresoli wrote: > This warning notifies a clock was set to an inaccurate value. Modify the > string to also show the clock name. > > While doing that also rewrap the entire function call. > > Signed-off-by: Luca Ceresoli > --- > drivers/gpu/drm/bridge/fsl-ldb.c | 6 +++--- >

Re: [PATCH 2/2] drm/bridge: fsl-ldb: make warning message more informative

2025-03-06 Thread Liu Ying
On 03/07/2025, Frank Li wrote: > On Thu, Mar 06, 2025 at 06:28:41PM +0100, Luca Ceresoli wrote: >> This warning notifies a clock was set to an inaccurate value. Modify the >> string to also show the clock name. >> >> While doing that also rewrap the entire function call. >> >> Signed-off-by: Luca C

[PATCH v3 3/3] vfio/pci: Allow MMIO regions to be exported through dma-buf

2025-03-06 Thread Vivek Kasireddy
>From Jason Gunthorpe: "dma-buf has become a way to safely acquire a handle to non-struct page memory that can still have lifetime controlled by the exporter. Notably RDMA can now import dma-buf FDs and build them into MRs which allows for PCI P2P operations. Extend this to allow vfio-pci to export

Re: [PATCH -next] drm/msm/dpu: Remove duplicate dpu_hw_cwb.h header

2025-03-06 Thread Dmitry Baryshkov
On Fri, Mar 07, 2025 at 09:50:30AM +0800, Jiapeng Chong wrote: > ./drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c: dpu_hw_cwb.h is included more > than once. > > Reported-by: Abaci Robot > Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=19239 > Signed-off-by: Jiapeng Chong > --- > drivers/

[PATCH v3 0/3] vfio/pci: Allow MMIO regions to be exported through dma-buf

2025-03-06 Thread Vivek Kasireddy
This is an attempt to revive the patches posted by Jason Gunthorpe at: https://patchwork.kernel.org/project/linux-media/cover/0-v2-472615b3877e+28f7-vfio_dma_buf_...@nvidia.com/ Here is the cover letter text from Jason's original series: "dma-buf has become a way to safely acquire a handle to non-

[PATCH v3 1/3] vfio: Export vfio device get and put registration helpers

2025-03-06 Thread Vivek Kasireddy
These helpers are useful for managing additional references taken on the device from other associated VFIO modules. Original-patch-by: Jason Gunthorpe Signed-off-by: Vivek Kasireddy --- drivers/vfio/vfio_main.c | 2 ++ include/linux/vfio.h | 2 ++ 2 files changed, 4 insertions(+) diff --gi

[PATCH v3 2/3] vfio/pci: Share the core device pointer while invoking feature functions

2025-03-06 Thread Vivek Kasireddy
There is no need to share the main device pointer (struct vfio_device *) with all the feature functions as they only need the core device pointer. Therefore, extract the core device pointer once in the caller (vfio_pci_core_ioctl_feature) and share it instead. Signed-off-by: Vivek Kasireddy ---

Re: [PATCH] drm/imx: legacy-bridge: fix inconsistent indenting warning

2025-03-06 Thread Liu Ying
On 03/05/2025, Charles Han wrote: > Fix below inconsistent indenting smatch warning. > smatch warnings: > drivers/gpu/drm/bridge/imx/imx-legacy-bridge.c:79 > devm_imx_drm_legacy_bridge() warn: inconsistent indenting > > Signed-off-by: Charles Han > --- > drivers/gpu/drm/bridge/imx/imx-legacy-br

Re: [PATCH v3 00/16] Introduce and use generic parity16/32/64 helper

2025-03-06 Thread H. Peter Anvin
On March 6, 2025 8:25:25 AM PST, Kuan-Wei Chiu wrote: >Several parts of the kernel contain redundant implementations of parity >calculations for 16/32/64-bit values. Introduces generic >parity16/32/64() helpers in bitops.h, providing a standardized >and optimized implementation. > >Subsequent pat

Re: [git pull] drm fixes for 6.14-rc6

2025-03-06 Thread pr-tracker-bot
The pull request you sent on Fri, 7 Mar 2025 10:13:09 +1000: > https://gitlab.freedesktop.org/drm/kernel.git tags/drm-fixes-2025-03-07 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/28f587adb69957125241a8df359b68b134f3c4a1 Thank you! -- Deet-doot-dot, I am a bot. ht

Re: [PATCH v5 2/7] bits: introduce fixed-type genmasks

2025-03-06 Thread Vincent Mailhol
On 06/03/2025 à 22:08, Andy Shevchenko wrote: > On Thu, Mar 06, 2025 at 08:29:53PM +0900, Vincent Mailhol via B4 Relay wrote: >> From: Yury Norov >> >> Add GENMASK_TYPE() which generalizes __GENMASK() to support different >> types, and implement fixed-types versions of GENMASK() based on it. >> Th

Re: [PATCH 3/5] dt-bindings: display: simple-bridge: Document DPI color encoder

2025-03-06 Thread Liu Ying
On 03/07/2025, Rob Herring wrote: > On Thu, Mar 06, 2025 at 12:35:49PM +0100, Maxime Ripard wrote: >> On Thu, Mar 06, 2025 at 03:02:41PM +0800, Liu Ying wrote: >>> On 03/06/2025, Rob Herring wrote: On Wed, Mar 05, 2025 at 10:35:26AM +0100, Alexander Stein wrote: > Hi, > > Am Dienst

Re: [PATCH 2/2] drm/bridge: ti-sn65dsi83: add h/vsync-disable support

2025-03-06 Thread Dmitry Baryshkov
On Thu, Mar 06, 2025 at 10:11:33AM +0100, A. Zini wrote: > From: Alessandro Zini > > The h/vsync-disable properties are used to control whether to use or > not h/vsync signals, by configuring their pulse width to zero. > > This is required on some panels which are driven in DE-only mode but do >

[PATCH 09/11] arm64: dts: qcom: x1e80100-dell-xps13-9345: Drop clock-names from PS8830

2025-03-06 Thread Konrad Dybcio
From: Konrad Dybcio The preemptively-merged node contains a property absent from the final bindings. Remove it. Signed-off-by: Konrad Dybcio --- arch/arm64/boot/dts/qcom/x1e80100-dell-xps13-9345.dts | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/arm64/boot/dts/qcom/x1e80100-dell-xps

Re: [PATCH drm-next 2/2] drm/virtio: Use atomic_vmap to work drm_panic in GUI

2025-03-06 Thread kernel test robot
patch link: https://lore.kernel.org/r/20250305152555.318159-3-ryasuoka%40redhat.com patch subject: [PATCH drm-next 2/2] drm/virtio: Use atomic_vmap to work drm_panic in GUI config: i386-buildonly-randconfig-003-20250306 (https://download.01.org/0day-ci/archive/20250307/202503071022.q1pg7suf

Re: [PATCH] drm/panel: novatek-nt36523: transition to mipi_dsi wrapped functions

2025-03-06 Thread Dmitry Baryshkov
On Thu, Mar 06, 2025 at 11:08:18PM +0530, Tejas Vipin wrote: > > > On 3/6/25 10:58 PM, Doug Anderson wrote: > > Hi, > > > > On Thu, Mar 6, 2025 at 6:05 AM wrote: > >> > >> On 06/03/2025 14:43, Tejas Vipin wrote: > >>> Changes the novatek-nt36523 panel to use multi style functions for > >>> impr

[PATCH] drm/gma500: Add NULL check for pci_gfx_root in mid_get_vbt_data()

2025-03-06 Thread Ivan Abramov
Since pci_get_domain_bus_and_slot() can return NULL, add NULL check for pci_gfx_root in the mid_get_vbt_data(). This change is similar to the checks implemented in mid_get_fuse_settings() and mid_get_pci_revID(), which were introduced by commit 0cecdd818cd7 ("gma500: Final enables for Oaktrail") a

Re: [PATCH v5 10/16] drm/bridge: ti-sn65dsi83: Switch to drm_bridge_helper_reset_crtc

2025-03-06 Thread Herve Codina
Hi Maxime, On Tue, 04 Mar 2025 12:10:53 +0100 Maxime Ripard wrote: > Now that we have a helper for bridge drivers to call to reset the output > pipeline, let's use it. > > Reviewed-by: Dmitry Baryshkov > Signed-off-by: Maxime Ripard > --- > drivers/gpu/drm/bridge/ti-sn65dsi83.c | 28

Re: [PATCH] drm/gma500: Remove unused mrst_helper_funcs

2025-03-06 Thread Patrik Jakobsson
On Sat, Feb 1, 2025 at 2:14 AM wrote: > > From: "Dr. David Alan Gilbert" > > The mrst_helper_funcs const was added in 2013 by > commit ac6113ebb70d ("drm/gma500/mrst: Add SDVO clock calculation") > and commented as 'Not used yet'. > > It's not been used since, so remove it. You talk about mrst_h

Re: [PATCH v4 3/8] bits: introduce fixed-type genmasks

2025-03-06 Thread Andy Shevchenko
On Thu, Mar 06, 2025 at 06:22:33PM +0900, Vincent Mailhol wrote: > On 06/03/2025 at 04:45, Andy Shevchenko wrote: > >>> But GENMASK_U128() becomes a special case now. > >>> The 128-bit GENMASK is unsued, but it's exported in uapi. Is there any > >>> simple way to end up with a common implementation

Re: [PATCH 2/2] mailmap: remap all addresses to kernel.org alias

2025-03-06 Thread neil . armstrong
On 05/03/2025 23:44, Dmitry Baryshkov wrote: Remap all historical and non-historical entries to my kernel.org email. Signed-off-by: Dmitry Baryshkov --- .mailmap | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/.mailmap b/.mailmap index 01145c078838bf9348e8d0e5e4

Re: [PATCH v4 4/8] bits: introduce fixed-type BIT

2025-03-06 Thread Jani Nikula
On Wed, 05 Mar 2025, David Laight wrote: > So it is even questionable whether BIT8() and BIT16() should even > exist at all. If nothing else, they do provide compile time checks for the bit being 0..7 and 0..15, respectively. BR, Jani. -- Jani Nikula, Intel

Re: [PATCH v5 04/16] drm/atomic: Introduce helper to lookup connector by encoder

2025-03-06 Thread Herve Codina
Hi Maxime, Hi Maxime, On Tue, 04 Mar 2025 12:10:47 +0100 Maxime Ripard wrote: > With the bridges switching over to drm_bridge_connector, the direct > association between a bridge driver and its connector was lost. > > This is mitigated for atomic bridge drivers by the fact you can access > the

[PATCH 1/2] drm/msm/dpu: correct dpu_crtc_check_mode_changed docs

2025-03-06 Thread Dmitry Baryshkov
Correct commit 20972609d12c ("drm/msm/dpu: Require modeset if clone mode status changes") and describe old_crtc_state and new_crtc_state params instead of the single previously used parameter crtc_state. Fixes: 20972609d12c ("drm/msm/dpu: Require modeset if clone mode status changes") Signed-off-

[PATCH 2/2] drm/msm/dpu: correct struct dpu_encoder_virt docs

2025-03-06 Thread Dmitry Baryshkov
Fix a typo in struct dpu_encoder_virt kerneldoc, which made it ignore description of the cwb_mask field. Fixes: dd331404ac7c ("drm/msm/dpu: Configure CWB in writeback encoder") Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 2 +- 1 file changed, 1 insertion(+),

[PATCH 0/2] drm/msm/dpu: two fixes for kerneldocs

2025-03-06 Thread Dmitry Baryshkov
, 3 insertions(+), 2 deletions(-) --- base-commit: 6d3175a72cc07e90f81fb35841048a8a9b5134cb change-id: 20250306-dpu-fix-docs-3700642b33ea Best regards, -- Dmitry Baryshkov

Re: RESEND Re: [PATCH v16 2/7] drm/ttm/pool, drm/ttm/tt: Provide a helper to shrink pages

2025-03-06 Thread Thomas Hellström
On Wed, 2025-03-05 at 13:01 +1000, Dave Airlie wrote: > I've looked over the two patches mentioned here, I think they have > seen enough time and we need to unblock, > > Please add and merge them: > Acked-by: Dave Airlie > > Dave. Thanks for unblocking, Dave. Pushed to drm-misc-next yesterday.

Re: [PATCH 2/6] drm/bridge: synopsys: Add DW DPTX Controller support library

2025-03-06 Thread Yubing Zhang
Hi Dmitry, On 2025/3/2 2:14, Dmitry Baryshkov wrote: On Sun, Feb 23, 2025 at 07:30:25PM +0800, Andy Yan wrote: From: Andy Yan The DW DP TX Controller is compliant with the DisplayPort Specification Version 1.4 with the following features: * DisplayPort 1.4a * Main Link: 1/2/4 lanes * Main Li

Re: [PATCH v4 4/8] bits: introduce fixed-type BIT

2025-03-06 Thread Andy Shevchenko
On Wed, Mar 05, 2025 at 09:50:27PM +, David Laight wrote: > On Wed, 5 Mar 2025 21:56:22 +0200 > Andy Shevchenko wrote: > > On Thu, Mar 06, 2025 at 02:17:18AM +0900, Vincent Mailhol wrote: > > > On 06/03/2025 at 00:48, Andy Shevchenko wrote: > > > > On Wed, Mar 05, 2025 at 11:48:10PM +0900, V

Re: [PATCH 1/2] MAINTAINERS: use kernel.org alias

2025-03-06 Thread neil . armstrong
On 05/03/2025 23:44, Dmitry Baryshkov wrote: My Linaro email will stop working soon. Use @kernel.org email instead. Signed-off-by: Dmitry Baryshkov --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 29e1a423eee5bcf9df7938aaffe

Re: [PATCH] drm/gma500: Replace deprecated strncpy() with strscpy()

2025-03-06 Thread Patrik Jakobsson
On Tue, Feb 25, 2025 at 9:39 PM Thorsten Blum wrote: > > strncpy() is deprecated for NUL-terminated destination buffers. Use > strscpy() instead and remove the manual NUL-termination. > > Compile-tested only. > > Link: https://github.com/KSPP/linux/issues/90 > Cc: linux-harden...@vger.kernel.org >

Re: [PATCH v4 4/8] bits: introduce fixed-type BIT

2025-03-06 Thread Vincent Mailhol
On 06/03/2025 at 18:12, Andy Shevchenko wrote: > On Wed, Mar 05, 2025 at 09:50:27PM +, David Laight wrote: >> On Wed, 5 Mar 2025 21:56:22 +0200 >> Andy Shevchenko wrote: >>> On Thu, Mar 06, 2025 at 02:17:18AM +0900, Vincent Mailhol wrote: On 06/03/2025 at 00:48, Andy Shevchenko wrote: >

Re: [PATCH] drm/i915: implement vmap/vunmap GEM object functions

2025-03-06 Thread Andi Shyti
Hi Asbjorn, On Sat, Jun 29, 2024 at 06:25:06PM +, Asbjørn Sloth Tønnesen wrote: > Implement i915_gem_vmap_object() and i915_gem_vunmap_object(), > based on i915_gem_dmabuf_vmap() and i915_gem_dmabuf_vunmap(). > > This enables a drm_client to use drm_client_buffer_vmap() and > drm_client_buffe

Re: [PATCH] drm/gma500: fix inconsistent indenting warning

2025-03-06 Thread Patrik Jakobsson
On Wed, Mar 5, 2025 at 9:49 AM Charles Han wrote: > > Fix below inconsistent indenting smatch warning. > smatch warnings: > drivers/gpu/drm/gma500/cdv_device.c:218 cdv_errata() warn: inconsistent > indenting > > Signed-off-by: Charles Han Thanks for the patch. Applied to drm-misc-next -Patrik

Re: [PATCH] drm/gma500: Add NULL check for pci_gfx_root in mid_get_vbt_data()

2025-03-06 Thread Patrik Jakobsson
On Wed, Mar 5, 2025 at 12:20 PM Ivan Abramov wrote: > > Since pci_get_domain_bus_and_slot() can return NULL, add NULL check for > pci_gfx_root in the mid_get_vbt_data(). > > This change is similar to the checks implemented in mid_get_fuse_settings() > and mid_get_pci_revID(), which were introduced

Re: [PATCH v3 1/4] drm/i915: Skip harmful unregister steps if not registered

2025-03-06 Thread Jani Nikula
On Wed, 05 Mar 2025, Janusz Krzysztofik wrote: > Starting with commit ec3e00b4ee27 ("drm/i915: stop registering if > drm_dev_register() fails"), we return from i915_driver_register() > immediately if drm_dev_register() fails, skipping remaining registration > steps. However, the _unregister() co

Re: [PATCH v4 3/8] bits: introduce fixed-type genmasks

2025-03-06 Thread Vincent Mailhol
On 06/03/2025 at 04:45, Andy Shevchenko wrote: >>> But GENMASK_U128() becomes a special case now. >>> The 128-bit GENMASK is unsued, but it's exported in uapi. Is there any >>> simple way to end up with a common implementation for all fixed-type >>> GENMASKs? >> >> What bothers me is that the 128 b

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

2025-03-06 Thread José Expósito
Hi Louis, On Tue, Mar 04, 2025 at 07:17:53PM +0100, Louis Chauvet wrote: > > > Le 04/03/2025 à 17:23, José Expósito a écrit : > > On Tue, Mar 04, 2025 at 04:35:50PM +0100, Louis Chauvet wrote: > > > > > > > > > Le 04/03/2025 à 15:54, José Expósito a écrit : > > > > Hi Louis, > > > > > > > > O

Re: [RFC PATCH 0/3] gpu: nova-core: add basic timer subdevice implementation

2025-03-06 Thread Simona Vetter
On Wed, Mar 05, 2025 at 11:10:12AM -0400, Jason Gunthorpe wrote: > On Wed, Mar 05, 2025 at 08:30:34AM +0100, Simona Vetter wrote: > > - developers who want to quickly test new driver versions without full > > reboot. They're often preferring convenience over correctness, like with > > the remov

Re: [PATCH v3 0/4] drm/i915: Fix harmfull driver register/unregister assymetry

2025-03-06 Thread Krzysztof Karas
Hi Janusz, throughout the series you modify the code right after introducing it. How about changing the order of things a bit: 1) order the functions in a symmetrical way between register/unregister steps and group them as you see necessary, (At that point you would not be fixing the issue ye

Re: [PATCH v4 3/8] mailbox: mtk-cmdq: Add driver data to support for MT8196

2025-03-06 Thread 林睿祥

Re: [PATCH] drm/i915/gt/uc: Fix typo in a comment

2025-03-06 Thread Andi Shyti
Hi Yuichiro, On Mon, Feb 24, 2025 at 05:56:37PM +0900, Yuichiro Tsuji wrote: > Fix typo in a comment. > > explaination -> explanation > > Signed-off-by: Yuichiro Tsuji reviewed and merged, thanks. Andi

Re: [PATCH 0/6] Add support for RK3588 DisplayPort Controller

2025-03-06 Thread Piotr Oniszczuk
> Wiadomość napisana przez Piotr Oniszczuk w dniu 6 > mar 2025, o godz. 15:08: > > > >> Wiadomość napisana przez Andy Yan w dniu 6 mar 2025, o >> godz. 13:15: >> >> Hi Piotr, >> >> >> >> Then when you DP cable plugin, you can run command as bellow to see if the >> driver detects the

Re: [PATCH v5 04/16] drm/atomic: Introduce helper to lookup connector by encoder

2025-03-06 Thread Simona Vetter
On Thu, Mar 06, 2025 at 08:10:16AM +0100, Maxime Ripard wrote: > On Thu, Mar 06, 2025 at 09:16:24AM +0800, Andy Yan wrote: > > > > Hi Maxime and Dmitry: > > > > At 2025-03-06 04:13:53, "Dmitry Baryshkov" > > wrote: > > >On Wed, Mar 05, 2025 at 02:19:36PM +0100, Maxime Ripard wrote: > > >> Hi An

Re: [PATCH v5 04/16] drm/atomic: Introduce helper to lookup connector by encoder

2025-03-06 Thread Simona Vetter
On Tue, Mar 04, 2025 at 12:10:47PM +0100, Maxime Ripard wrote: > With the bridges switching over to drm_bridge_connector, the direct > association between a bridge driver and its connector was lost. > > This is mitigated for atomic bridge drivers by the fact you can access > the encoder, and then

[PATCH v5 3/7] bits: introduce fixed-type BIT_U*()

2025-03-06 Thread Vincent Mailhol via B4 Relay
From: Lucas De Marchi Implement fixed-type BIT_U*() to help drivers add stricter checks, like was done for GENMASK_U*(). Signed-off-by: Lucas De Marchi Acked-by: Jani Nikula Co-developed-by: Vincent Mailhol Signed-off-by: Vincent Mailhol --- Changelog: v4 -> v5: - Rename GENMASK_t()

[PATCH v5 5/7] test_bits: add tests for __GENMASK() and __GENMASK_ULL()

2025-03-06 Thread Vincent Mailhol via B4 Relay
From: Vincent Mailhol The definitions of GENMASK() and GENMASK_ULL() do not depend any more on __GENMASK() and __GENMASK_ULL(). Duplicate the existing unit tests so that __GENMASK{,ULL}() is still covered. Signed-off-by: Vincent Mailhol --- lib/test_bits.c | 18 ++ 1 file chang

[PATCH v5 2/7] bits: introduce fixed-type genmasks

2025-03-06 Thread Vincent Mailhol via B4 Relay
From: Yury Norov Add GENMASK_TYPE() which generalizes __GENMASK() to support different types, and implement fixed-types versions of GENMASK() based on it. The fixed-type version allows more strict checks to the min/max values accepted, which is useful for defining registers like implemented by i9

[PATCH v5 4/7] drm/i915: Convert REG_GENMASK*() to fixed-width GENMASK_U*()

2025-03-06 Thread Vincent Mailhol via B4 Relay
From: Lucas De Marchi Now that include/linux/bits.h implements fixed-width GENMASK_U*(), use them to implement the i915/xe specific macros. Converting each driver to use the generic macros are left for later, when/if other driver-specific macros are also generalized. Signed-off-by: Lucas De Marc

Re: [PATCH v5 1/7] bits: split the definition of the asm and non-asm GENMASK()

2025-03-06 Thread Andy Shevchenko
On Thu, Mar 06, 2025 at 08:29:52PM +0900, Vincent Mailhol via B4 Relay wrote: > From: Vincent Mailhol > > In an upcoming change, GENMASK() and its friends will indirectly > depend on sizeof() which is not available in asm. > > Instead of adding further complexity to __GENMASK() to make it work >

[PULL] drm-misc-next

2025-03-06 Thread Thomas Zimmermann
Hi Dave, Sima, here's the PR for drm-misc-next. It includes two new drivers for Apple Touch Bar displays. There are new helpers for TTM's shrinker, which xe now uses. There are a number of changes to the rockchip driver. There are also cross-subsystem changes for drivers/base and fbdev. The latte

Re: [PATCH v4 3/8] mailbox: mtk-cmdq: Add driver data to support for MT8196

2025-03-06 Thread AngeloGioacchino Del Regno
Il 06/03/25 12:00, Jason-JH Lin (林睿祥) ha scritto: [snip] CPR_GSIZE is the setting for allocating the CPR SRAM size to each VM. Would be awesome if you could then clarify the comment that you have later in the code here, from... /* config cpr size for host vm */ to /* Set the amount of CPR

Re: [PATCH v5 2/7] bits: introduce fixed-type genmasks

2025-03-06 Thread Andy Shevchenko
On Thu, Mar 06, 2025 at 08:29:53PM +0900, Vincent Mailhol via B4 Relay wrote: > From: Yury Norov > > Add GENMASK_TYPE() which generalizes __GENMASK() to support different > types, and implement fixed-types versions of GENMASK() based on it. > The fixed-type version allows more strict checks to th

Re: [PATCH v5 0/7] bits: Fixed-type GENMASK()/BIT()

2025-03-06 Thread Andy Shevchenko
On Thu, Mar 06, 2025 at 08:29:51PM +0900, Vincent Mailhol via B4 Relay wrote: > Introduce some fixed width variant of the GENMASK() and the BIT() > macros in bits.h. Note that the main goal is not to get the correct > type, but rather to enforce more checks at compile time. For example: > > GENM

Re: [PATCH v5 1/7] bits: split the definition of the asm and non-asm GENMASK()

2025-03-06 Thread Lucas De Marchi
On Thu, Mar 06, 2025 at 08:29:52PM +0900, Vincent Mailhol via B4 Relay wrote: From: Vincent Mailhol In an upcoming change, GENMASK() and its friends will indirectly depend on sizeof() which is not available in asm. Instead of adding further complexity to __GENMASK() to make it work for both as

[PATCH v3 08/11] backlight: lcd: Replace fb events with a dedicated function call

2025-03-06 Thread Thomas Zimmermann
Remove support for fb events from the lcd subsystem. Provide the helper lcd_notify_blank_all() instead. In fbdev, call lcd_notify_blank_all() to inform the lcd subsystem of changes to a display's blank state. Fbdev maintains a list of all installed notifiers. Instead of fbdev notifiers, maintain a

Re: [PATCH v4 drm-dp 3/8] drm/hisilicon/hibmc: Add dp serdes cfg in dp process

2025-03-06 Thread Yongbang Shi
On Wed, Mar 05, 2025 at 07:26:42PM +0800, Yongbang Shi wrote: From: Baihan Li Add dp serdes cfg in link training process, and related adapting and modificating. Change some init values about training, because we want completely to negotiation process, so we start with the maximum rate and the e

Re: [PATCH v4 drm-dp 7/8] drm/hisilicon/hibmc: Enable this hot plug detect of irq feature

2025-03-06 Thread Yongbang Shi
On Wed, Mar 05, 2025 at 07:26:46PM +0800, Yongbang Shi wrote: From: Baihan Li Add HPD interrupt enable functions in drm framework. Add link reset process to reset link status when a new connector pulgged in. Because the connected VGA connector would make driver can't get the userspace call,

  1   2   3   >