Re: [PATCH 1/4] drm/msm/dsi: Drop redundant NULL-ifying of clocks on error paths

2025-01-06 Thread Abhinav Kumar
On 1/6/2025 12:49 AM, Krzysztof Kozlowski wrote: dsi_clk_init(), which gets the clocks, is called only through platform driver probe and its failure is a failure of the probe. Therefore NULL-ifying specific clocks is pointless and redundant - the PTR_ERR value stored there won't be used/deref

Re: [PATCH 2/4] drm/msm/dsi: Simplify with dev_err_probe()

2025-01-06 Thread Abhinav Kumar
On 1/6/2025 12:49 AM, Krzysztof Kozlowski wrote: dsi_clk_init() and msm_dsi_host_init() are called only from platform dsi_get_config() as well? OR you didnt want to explicitly mention that since its called from msm_dsi_host_init(). driver probe function, so using dev_err_probe is both ap

Re: [PATCH 3/4] drm/msm/dsi: Minor whitespace and style cleanup

2025-01-06 Thread Abhinav Kumar
On 1/6/2025 12:49 AM, Krzysztof Kozlowski wrote: Cleanup few obvious kernel coding style violations: missing or unnecessary braces in 'if-else', unnecessary break lines, incorrect breaking of long function declarations, unnecessary 'else' after a 'return'. No functional impact expected. Sign

Re: [PATCH 4/4] drm/msm/dsi: Drop unnecessary -ENOMEM message

2025-01-06 Thread Abhinav Kumar
On 1/6/2025 12:49 AM, Krzysztof Kozlowski wrote: Kernel core already prints detailed report about memory allocation failures, so drivers should not have their own error messages. Signed-off-by: Krzysztof Kozlowski --- drivers/gpu/drm/msm/dsi/dsi_host.c | 4 +--- 1 file changed, 1 insertio

Re: [PATCH 0/4] drm/msm/dpu: enable CDM for all supported platforms

2025-01-06 Thread Abhinav Kumar
On 12/23/2024 8:25 PM, Dmitry Baryshkov wrote: Enable CDM block on all the platforms where it is supposed to be present. Notably, from the platforms being supported by the DPU driver it is not enabled for SM6115 (DPU 6.3), QCM2290 (DPU 6.5) and SM6375 (DPU 6.9) Thanks for enabling it, but c

Re: [PATCH] drm/msm/dpu: Add writeback support for SM6150

2025-01-06 Thread Abhinav Kumar
f_count = ARRAY_SIZE(sdm845_vbif), --- base-commit: 8155b4ef3466f0e289e8fcc9e6e62f3f4dceeac2 change-id: 20250106-add-writeback-support-for-sm6150-ba7657196ea8 Best regards,

Re: [PATCH v3 0/6] Support for GPU ACD feature on Adreno X1-85

2025-01-06 Thread Akhil P Oommen
On Sun, Jan 05, 2025 at 04:55:42PM -0800, Rob Clark wrote: > fwiw, I did see some perf boost (was mainly looking at gfxbench aztec > ruins vk high/normal, and also a separate mesa MR that fixes some LRZ > issues with turnip, so I don't remember how much boost was related to > which offhand).. I've

Re: [PATCH v2] iommu/arm-smmu-qcom: Only enable stall on smmu-v2

2025-01-06 Thread Rob Clark
On Mon, Jan 6, 2025 at 12:11 PM Akhil P Oommen wrote: > > On 1/3/2025 1:00 AM, Akhil P Oommen wrote: > > On 1/3/2025 12:02 AM, Rob Clark wrote: > >> From: Rob Clark > >> > >> On mmu-500, stall-on-fault seems to stall all context banks, causing the > >> GMU to misbehave. So limit this feature to

Re: [PATCH v2] iommu/arm-smmu-qcom: Only enable stall on smmu-v2

2025-01-06 Thread Rob Clark
On Mon, Jan 6, 2025 at 12:11 PM Akhil P Oommen wrote: > > On 1/3/2025 1:00 AM, Akhil P Oommen wrote: > > On 1/3/2025 12:02 AM, Rob Clark wrote: > >> From: Rob Clark > >> > >> On mmu-500, stall-on-fault seems to stall all context banks, causing the > >> GMU to misbehave. So limit this feature to

[PATCH 1/4] drm/msm/dsi: Drop redundant NULL-ifying of clocks on error paths

2025-01-06 Thread Krzysztof Kozlowski
dsi_clk_init(), which gets the clocks, is called only through platform driver probe and its failure is a failure of the probe. Therefore NULL-ifying specific clocks is pointless and redundant - the PTR_ERR value stored there won't be used/dereferenced afterwards. What's more, variant-specific clo

[PATCH 3/4] drm/msm/dsi: Minor whitespace and style cleanup

2025-01-06 Thread Krzysztof Kozlowski
Cleanup few obvious kernel coding style violations: missing or unnecessary braces in 'if-else', unnecessary break lines, incorrect breaking of long function declarations, unnecessary 'else' after a 'return'. No functional impact expected. Signed-off-by: Krzysztof Kozlowski --- drivers/gpu/drm/m

[PATCH 2/4] drm/msm/dsi: Simplify with dev_err_probe()

2025-01-06 Thread Krzysztof Kozlowski
dsi_clk_init() and msm_dsi_host_init() are called only from platform driver probe function, so using dev_err_probe is both appropriate and beneficial: - Properly marks device deferred probe status, - Avoids dmesg flood on probe deferrals, - Already incorporates printing ERR value, - Shows devic

[PATCH 4/4] drm/msm/dsi: Drop unnecessary -ENOMEM message

2025-01-06 Thread Krzysztof Kozlowski
Kernel core already prints detailed report about memory allocation failures, so drivers should not have their own error messages. Signed-off-by: Krzysztof Kozlowski --- drivers/gpu/drm/msm/dsi/dsi_host.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/msm/d

[PATCH 0/4] drm/msm/dsi: Minor cleanups

2025-01-06 Thread Krzysztof Kozlowski
/dsi: Drop unnecessary -ENOMEM message drivers/gpu/drm/msm/dsi/dsi_host.c | 158 - 1 file changed, 68 insertions(+), 90 deletions(-) --- base-commit: 8155b4ef3466f0e289e8fcc9e6e62f3f4dceeac2 change-id: 20250106-drm-msm-cleanups-ddacf1fc3ba5 Best regards

Re: [PATCH v2] iommu/arm-smmu-qcom: Only enable stall on smmu-v2

2025-01-06 Thread Akhil P Oommen
On 1/3/2025 1:00 AM, Akhil P Oommen wrote: > On 1/3/2025 12:02 AM, Rob Clark wrote: >> From: Rob Clark >> >> On mmu-500, stall-on-fault seems to stall all context banks, causing the >> GMU to misbehave. So limit this feature to smmu-v2 for now. >> >> This fixes an issue with an older mesa bug tak

Re: [PATCH 44/45] arm64: dts: qcom: add mst support for pixel stream clk for DP0

2025-01-06 Thread Bjorn Andersson
On Thu, Dec 05, 2024 at 08:32:15PM -0800, Abhinav Kumar wrote: > From: Yongxing Mou > I'd expect "sa8775p" in the subject prefix. > Populate the pixel clock for stream 1 for DP0 for sa8775p DP controller. Please write your commit messages in the style expressed in https://docs.kernel.org/proce

Re: [PATCH 03/45] drm/msm/dp: fix the intf_type of MST interfaces

2025-01-06 Thread Bjorn Andersson
On Thu, Dec 05, 2024 at 08:31:34PM -0800, Abhinav Kumar wrote: > Interface type of MST interfaces is currently INTF_NONE. > Fix this to INTF_DP. > Wouldn't it make sense to introduce this later in the series, once the implementation would actually handle this case? Or could/should we have left th

Re: [PATCH 00/45] drm/msm/dp: Add MST support for MSM chipsets

2025-01-06 Thread Bjorn Andersson
On Thu, Dec 05, 2024 at 08:31:31PM -0800, Abhinav Kumar wrote: Please discuss with and reply to Dmitry's questions/feedback on the list, so that it's possible for others to join the discussion. Regards, Bjorn > Add support for Multi-stream transport for MSM chipsets that allow > a single instanc

Re: [PATCH 00/45] drm/msm/dp: Add MST support for MSM chipsets

2025-01-06 Thread Abhinav Kumar
Hi Bjorn / Dmitry Happy New Year ! On 1/6/2025 5:06 PM, Bjorn Andersson wrote: On Thu, Dec 05, 2024 at 08:31:31PM -0800, Abhinav Kumar wrote: Please discuss with and reply to Dmitry's questions/feedback on the list, so that it's possible for others to join the discussion. Regards, Bjorn Sor

Re: [PATCH v2] drm/msm/dpu: correct LM pairing for SM6150

2025-01-06 Thread Abhinav Kumar
On 12/17/2024 4:35 AM, Dmitry Baryshkov wrote: The SM6150 platform doesn't have 3DMux (MERGE_3D) block, so it can not split the screen between two LMs. Drop lm_pair fields as they don't make sense for this platform. Suggested-by: Abhinav Kumar Fixes: cb2f9144693b ("drm/msm/dpu: Add SM6150 su

Re: [PATCH v3 3/6] drm/msm/adreno: Add module param to disable ACD

2025-01-06 Thread Akhil P Oommen
On 12/31/2024 4:02 PM, Konrad Dybcio wrote: > On 30.12.2024 10:11 PM, Akhil P Oommen wrote: >> Add a module param to disable ACD which will help to quickly rule it >> out for any GPU issues. >> >> Signed-off-by: Akhil P Oommen >> --- > > Is that something useful during internal development, or do

Re: [PATCH] drm/msm/dpu: Add writeback support for SM6150

2025-01-06 Thread fange zhang
f_count = ARRAY_SIZE(sm6150_intf),   .intf = sm6150_intf,   .vbif_count = ARRAY_SIZE(sdm845_vbif), --- base-commit: 8155b4ef3466f0e289e8fcc9e6e62f3f4dceeac2 change-id: 20250106-add-writeback-support-for-sm6150-ba7657196ea8 Best regards,

Re: [PATCH v3 3/4] drm/i915/dp: Use the generic helper to control LTTPR transparent mode

2025-01-06 Thread Abel Vesa
On 25-01-03 20:09:42, Dmitry Baryshkov wrote: > On Fri, Jan 03, 2025 at 02:58:17PM +0200, Abel Vesa wrote: > > LTTPRs operating modes are defined by the DisplayPort standard and the > > generic framework now provides a helper to switch between them, which > > is handling the explicit disabling of n

Re: [PATCH v2 0/5] drm/connector: make mode_valid() callback accept const mode pointer

2025-01-06 Thread Harry Wentland
On 2025-01-06 04:45, Dmitry Baryshkov wrote: > On Mon, 6 Jan 2025 at 10:55, Maxime Ripard wrote: >> >> On Mon, Jan 06, 2025 at 12:47:07AM +0200, Dmitry Baryshkov wrote: >>> On Sat, Dec 14, 2024 at 03:37:04PM +0200, Dmitry Baryshkov wrote: While working on the generic mode_valid() implement

Re: [PATCH 2/4] drm/msm/dsi: Simplify with dev_err_probe()

2025-01-06 Thread Krzysztof Kozlowski
On 07/01/2025 03:56, Abhinav Kumar wrote: > > > On 1/6/2025 12:49 AM, Krzysztof Kozlowski wrote: >> dsi_clk_init() and msm_dsi_host_init() are called only from platform > > dsi_get_config() as well? OR you didnt want to explicitly mention that > since its called from msm_dsi_host_init(). > As

Re: [PATCH v2 0/5] drm/connector: make mode_valid() callback accept const mode pointer

2025-01-06 Thread Maxime Ripard
On Mon, Jan 06, 2025 at 12:47:07AM +0200, Dmitry Baryshkov wrote: > On Sat, Dec 14, 2024 at 03:37:04PM +0200, Dmitry Baryshkov wrote: > > While working on the generic mode_valid() implementation for the HDMI > > Connector framework I noticed that unlike other DRM objects > > drm_connector accepts n

Re: [PATCH v2 0/5] drm/connector: make mode_valid() callback accept const mode pointer

2025-01-06 Thread Thomas Zimmermann
Hi Am 05.01.25 um 23:47 schrieb Dmitry Baryshkov: On Sat, Dec 14, 2024 at 03:37:04PM +0200, Dmitry Baryshkov wrote: While working on the generic mode_valid() implementation for the HDMI Connector framework I noticed that unlike other DRM objects drm_connector accepts non-const pointer to struc

Re: [PATCH v2 0/5] drm/connector: make mode_valid() callback accept const mode pointer

2025-01-06 Thread Dmitry Baryshkov
On Mon, 6 Jan 2025 at 10:55, Maxime Ripard wrote: > > On Mon, Jan 06, 2025 at 12:47:07AM +0200, Dmitry Baryshkov wrote: > > On Sat, Dec 14, 2024 at 03:37:04PM +0200, Dmitry Baryshkov wrote: > > > While working on the generic mode_valid() implementation for the HDMI > > > Connector framework I noti

Re: [PATCH v3 15/15] drm/msm/dpu: Enable quad-pipe for DSC and dual-DSI case

2025-01-06 Thread Jun Nie
Dmitry Baryshkov 于2025年1月4日周六 01:51写道: > > On Fri, Jan 03, 2025 at 11:49:07PM +0800, Jun Nie wrote: > > Dmitry Baryshkov 于2024年12月20日周五 07:46写道: > > > > > > On Thu, Dec 19, 2024 at 03:49:33PM +0800, Jun Nie wrote: > > > > Request 4 mixers and 4 DSC for the case that both dual-DSI and DSC are > >