Re: [PATCH] drm/amd/display: Add null check for dm_state in create_validate_stream_for_sink

2024-07-16 Thread Hamza Mahfooz
WRITEBACK) Fixes: fa7041d9d2fc ("drm/amd/display: Fix ineffective setting of max bpc property") Reported-by: Dan Carpenter Cc: Tom Chung Cc: Rodrigo Siqueira Cc: Roman Li Cc: Hersen Wu Cc: Alex Hung Cc: Aurabindo Pillai Cc: Harry Wentland Cc: Hamza Mahfooz Signed-off-by: Sriniv

Re: [PATCH AUTOSEL 6.9 11/22] drm/amd/display: Reset freesync config before update new state

2024-07-16 Thread Hamza Mahfooz
Hi Sasha, On 7/16/24 10:24, Sasha Levin wrote: From: Tom Chung [ Upstream commit 6b8487cdf9fc7bae707519ac5b5daeca18d1e85b ] [Why] Sometimes the new_crtc_state->vrr_infopacket did not sync up with the current state. It will affect the update_freesync_state_on_stream() does not update the

Re: [PATCH 2/2] drm/amd/display: use drm_crtc_set_vblank_offdelay()

2024-07-10 Thread Hamza Mahfooz
On 7/10/24 04:43, Daniel Vetter wrote: On Tue, Jul 09, 2024 at 10:02:08AM -0400, Hamza Mahfooz wrote: On 7/9/24 06:09, Daniel Vetter wrote: On Tue, Jul 09, 2024 at 11:32:11AM +0200, Daniel Vetter wrote: On Mon, Jul 08, 2024 at 04:29:07PM -0400, Hamza Mahfooz wrote: Hook up

Re: [PATCH v4 0/2] Add support for 'power saving policy' property

2024-07-10 Thread Hamza Mahfooz
On 7/3/24 01:17, Mario Limonciello wrote: During the Display Next hackfest 2024 one of the topics discussed was the need for compositor to be able to relay intention to drivers that color fidelity is preferred over power savings. To accomplish this a new optional DRM property is being

Re: [PATCH 2/2] drm/amd/display: use drm_crtc_set_vblank_offdelay()

2024-07-09 Thread Hamza Mahfooz
On 7/9/24 06:09, Daniel Vetter wrote: On Tue, Jul 09, 2024 at 11:32:11AM +0200, Daniel Vetter wrote: On Mon, Jul 08, 2024 at 04:29:07PM -0400, Hamza Mahfooz wrote: Hook up drm_crtc_set_vblank_offdelay() in amdgpu_dm, so that we can enable PSR more quickly for displays that support it. Signed

[PATCH 2/2] drm/amd/display: use drm_crtc_set_vblank_offdelay()

2024-07-08 Thread Hamza Mahfooz
Hook up drm_crtc_set_vblank_offdelay() in amdgpu_dm, so that we can enable PSR more quickly for displays that support it. Signed-off-by: Hamza Mahfooz --- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 30 ++- 1 file changed, 22 insertions(+), 8 deletions(-) diff --git

[PATCH 1/2] drm/vblank: allow dynamic per-crtc vblank off delay

2024-07-08 Thread Hamza Mahfooz
-by: Hamza Mahfooz --- drivers/gpu/drm/drm_vblank.c | 31 ++- include/drm/drm_vblank.h | 7 +++ 2 files changed, 33 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_vblank.c b/drivers/gpu/drm/drm_vblank.c index 702a12bc93bd..4f475131a092 100644

[PATCH] MAINTAINERS: add an entry for AMD DC DML

2024-07-03 Thread Hamza Mahfooz
We want all DML changes to be reviewed by Chaitanya or Jun. So, add an entry for DML to MAINTAINERS. Suggested-by: Rodrigo Siqueira Signed-off-by: Hamza Mahfooz --- MAINTAINERS | 7 +++ 1 file changed, 7 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 7fec8ddb8d5b..d2fb60fee7e8

Re: [PATCH] drm/amd/display: Fix Makefile copyright notices

2024-06-24 Thread Hamza Mahfooz
On 6/24/24 09:58, Alex Deucher wrote: Leftover copy pasta from original code. Signed-off-by: Alex Deucher Cc: harry.wentl...@amd.com Reviewed-by: Hamza Mahfooz --- drivers/gpu/drm/amd/display/dc/dcn301/Makefile | 11 ++- drivers/gpu/drm/amd/display/dc/dcn31/Makefile | 10

[PATCH 35/36] drm/amd/display: [FW Promotion] Release 0.0.222.0

2024-06-11 Thread Hamza Mahfooz
From: Anthony Koo - Add new condition for PSR exit due to ESD recovery - Add new VB scaling feature for ABM by interpolating between existing VB parameters, allowing driver to have fine grain scaled VB levels between 0 - 250 Acked-by: Hamza Mahfooz Signed-off-by: Anthony Koo

[PATCH 25/36] drm/amd/display: Fix warning caused by an attempt to configure a non-otg master

2024-06-11 Thread Hamza Mahfooz
inting to an OTG master before operate in the old OTG master reference. If it's not, the function must set the old OTG reference to NULL and avoid calling resource_get_opp_heads_for_otg_master before the context is updated. Reviewed-by: Wenjing Liu Acked-by: Hamza Mahfooz Co-developed-by: Wenji

[PATCH 29/36] drm/amd/display: Remove unused value set from 'min_hratio_fact' in dml

2024-06-11 Thread Hamza Mahfooz
From: Ivan Lipski These portions of code are flagged as 'UNUSED_VALUE' by the Coverity analysis since the assigned values of these vars are never used in the code. Reviewed-by: Alex Hung Reviewed-by: Aurabindo Pillai Acked-by: Hamza Mahfooz Signed-off-by: Ivan Lipski --- .../drm/amd

[PATCH 33/36] drm/amd/display: Check UnboundedRequestEnabled's value

2024-06-11 Thread Hamza Mahfooz
rry Wentland Acked-by: Hamza Mahfooz Signed-off-by: Alex Hung --- drivers/gpu/drm/amd/display/dc/dml2/display_mode_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display/dc/dml2/display_mode_core.c b/drivers/gpu/drm/amd/display/dc/d

[PATCH 27/36] drm/amd/display: Remove redundant checks for res_pool->dccg

2024-06-11 Thread Hamza Mahfooz
From: Alex Hung The null checks for res_pool->dccg are redundant as it was already dereferenced previously, as reported by Coverity; therefore the null checks are removed. This fixes 6 REVERSE_INULL issues reported by Coverity. Reviewed-by: Harry Wentland Acked-by: Hamza Mahfooz Signed-

[PATCH 34/36] drm/amd/display: Remove redundant null checks

2024-06-11 Thread Hamza Mahfooz
ked-by: Hamza Mahfooz Signed-off-by: Alex Hung --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 6 +++--- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c | 2 +- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 3 --- 3 files changed, 4 insertions(+), 7 deleti

[PATCH 36/36] drm/amd/display: 3.2.289

2024-06-11 Thread Hamza Mahfooz
From: Aric Cyr This version brings along the following: - DCN401 fixes - DPIA fixes - DML21 fixes - Misc Coverity fixes Acked-by: Hamza Mahfooz Signed-off-by: Aric Cyr --- drivers/gpu/drm/amd/display/dc/dc.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm

[PATCH 30/36] drm/amd/display: Remove redundant null checks

2024-06-11 Thread Hamza Mahfooz
From: Alex Hung The null checks are redundant as they were already dereferenced previously, as reported by Coverity; therefore the null checks are removed. This fixes 7 REVERSE_INULL issues reported by Coverity. Reviewed-by: Harry Wentland Acked-by: Hamza Mahfooz Signed-off-by: Alex Hung

[PATCH 32/36] drm/amd/display: Remove redundant checks for context

2024-06-11 Thread Hamza Mahfooz
From: Alex Hung The null checks for context are redundant as it was already dereferenced previously, as reported by Coverity; therefore the null checks are removed. This fixes 2 REVERSE_INULL issues reported by Coverity. Reviewed-by: Harry Wentland Acked-by: Hamza Mahfooz Signed-off-by: Alex

[PATCH 21/36] drm/amd/display: Make sure to reprogram ODM when resync fifo

2024-06-11 Thread Hamza Mahfooz
From: Alvin Lee Need to reconfigure ODM when resyncing FIFO because on OTG disable we clear all ODM programming Reviewed-by: Nicholas Kazlauskas Acked-by: Hamza Mahfooz Signed-off-by: Alvin Lee --- .../amd/display/dc/hwss/dcn314/dcn314_hwseq.c | 19 ++- .../amd/display/dc

[PATCH 22/36] drm/amd/display: Check dc_stream_state before it is used

2024-06-11 Thread Hamza Mahfooz
From: Alex Hung dc_state_get_stream_status dc_state_get_paired_subvp_stream and other functions can return null, and therefore null must be checked before status can be used. This fixes 21 NULL_RETURNS issues reported by Coverity. Reviewed-by: Harry Wentland Acked-by: Hamza Mahfooz Signed

[PATCH 31/36] drm/amd/display: Remove redundant checks for opp

2024-06-11 Thread Hamza Mahfooz
From: Alex Hung The null checks for opp are redundant as they were already dereferenced previously, as reported by Coverity; therefore the null checks are removed. This fixes 2 REVERSE_INULL issues reported by Coverity. Reviewed-by: Harry Wentland Acked-by: Hamza Mahfooz Signed-off-by: Alex

[PATCH 20/36] drm/amd/display: Remove duplicate HWSS interfaces

2024-06-11 Thread Hamza Mahfooz
- update_plane_addr should only be public, as it's used outside HWSS Reviewed-by: Aric Cyr Acked-by: Hamza Mahfooz Signed-off-by: Joshua Aberback --- .../gpu/drm/amd/display/dc/hwss/dce110/dce110_hwseq.c | 1 - .../gpu/drm/amd/display/dc/hwss/dcn10/dcn10_hwseq.c| 10 +- drivers/gpu/drm

[PATCH 05/36] drm/amd/display: Remove redundant condition with DEADCODE

2024-06-11 Thread Hamza Mahfooz
From: Ivan Lipski [WHY] Coverity analysis flagged this condition as DEADCODE since the variable 'req128_c' is always false, thus the condition is never true. [HOW] Remove the condition. Reviewed-by: Aurabindo Pillai Acked-by: Hamza Mahfooz Signed-off-by: Ivan Lipski --- .../gpu/drm/amd

[PATCH 17/36] drm/amd/display: Add null checker before access structs

2024-06-11 Thread Hamza Mahfooz
From: Alex Hung Checks null pointer before accessing various structs. This fixes 5 NULL_RETURNS issues reported by Coverity. Reviewed-by: Harry Wentland Acked-by: Hamza Mahfooz Signed-off-by: Alex Hung --- drivers/gpu/drm/amd/display/dc/dml2/dml21/dml21_utils.c | 2 ++ .../display/dc

[PATCH 24/36] drm/amd/display: Covert integers to double before divisions

2024-06-11 Thread Hamza Mahfooz
-by: Harry Wentland Acked-by: Hamza Mahfooz Signed-off-by: Alex Hung --- .../drm/amd/display/dc/dml/calcs/dcn_calcs.c | 2 +- .../drm/amd/display/dc/dml/dcn20/dcn20_fpu.c | 20 ++--- .../dc/dml/dcn20/display_mode_vba_20.c| 16 +-- .../dc/dml/dcn20/display_mode_vba_20v2.c

[PATCH 28/36] drm/amd/display: Remove redundant checks for ctx->dc_bios

2024-06-11 Thread Hamza Mahfooz
From: Alex Hung The null checks for ctx->dc_bios are redundant as it was already dereferenced previously, as reported by Coverity; therefore the null checks are removed. This fixes 7 REVERSE_INULL issues reported by Coverity. Reviewed-by: Harry Wentland Acked-by: Hamza Mahfooz Signed-

[PATCH 23/36] drm/amd/display: Check pipe_ctx before it is used

2024-06-11 Thread Hamza Mahfooz
From: Alex Hung resource_get_odm_slice_count and resource_get_otg_master_for_stream can return null, and their returns must be checked before used. This fixes 4 NULL_RETURNS issues reported by Coverity. Reviewed-by: Harry Wentland Acked-by: Hamza Mahfooz Signed-off-by: Alex Hung

[PATCH 26/36] drm/amd/display: Improve warning log for get OPP for OTG master

2024-06-11 Thread Hamza Mahfooz
to describe why the warning was triggered to make the debugging more straightforward. Reviewed-by: Wenjing Liu Acked-by: Hamza Mahfooz Signed-off-by: Rodrigo Siqueira --- drivers/gpu/drm/amd/display/dc/core/dc_resource.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/gpu/drm/amd

[PATCH 18/36] drm/amd/display: mirror case cleanup for cursors

2024-06-11 Thread Hamza Mahfooz
From: Sridevi Arvindekar Mirror case unsupported for cursors. So, remove code for mirror case with cursors. Reviewed-by: Nevenko Stupar Acked-by: Hamza Mahfooz Signed-off-by: Sridevi Arvindekar --- .../amd/display/dc/hwss/dcn401/dcn401_hwseq.c | 30 +-- 1 file changed, 1

[PATCH 16/36] drm/amd/display: Skip wbscl_set_scaler_filter if filter is null

2024-06-11 Thread Hamza Mahfooz
From: Alex Hung Callers can pass null in filter (i.e. from returned from the function wbscl_get_filter_coeffs_16p) and a null check is added to ensure that is not the case. This fixes 4 NULL_RETURNS issues reported by Coverity. Reviewed-by: Harry Wentland Acked-by: Hamza Mahfooz Signed-off

[PATCH 19/36] drm/amd/display: Fix NULL pointer dereference for DTN log in DCN401

2024-06-11 Thread Hamza Mahfooz
the gamut_remap callback in case the specific ASIC did not implement this function. Reviewed-by: Aurabindo Pillai Acked-by: Hamza Mahfooz Signed-off-by: Rodrigo Siqueira --- .../amd/display/dc/hwss/dcn10/dcn10_hwseq.c | 49 ++- 1 file changed, 27 insertions(+), 22 deletions(-) diff --git

[PATCH 15/36] drm/amd/display: Check BIOS images before it is used

2024-06-11 Thread Hamza Mahfooz
From: Alex Hung BIOS images may fail to load and null checks are added before they are used. This fixes 6 NULL_RETURNS issues reported by Coverity. Reviewed-by: Harry Wentland Acked-by: Hamza Mahfooz Signed-off-by: Alex Hung --- drivers/gpu/drm/amd/display/dc/bios/bios_parser.c | 14

[PATCH 12/36] drm/amd/display: Send message to notify the DPIA host router bandwidth

2024-06-11 Thread Hamza Mahfooz
From: Sung Joon Kim [why] Tell the system about the current host router bandwidth to be used to measure and calculate the right voltage to be used. [how] Send SMU message of each DPIA host router bandwidth. Reviewed-by: Nicholas Kazlauskas Acked-by: Hamza Mahfooz Signed-off-by: Sung Joon Kim

[PATCH 11/36] drm/amd/display: Add null check to dml21_find_dc_pipes_for_plane

2024-06-11 Thread Hamza Mahfooz
From: Dillon Varone When a phantom stream is in the process of being deconstructed, there could be pipes with no associated planes. In that case, ignore the phantom stream entirely when searching for associated pipes. Cc: sta...@vger.kernel.org Reviewed-by: Alvin Lee Acked-by: Hamza Mahfooz

[PATCH 14/36] drm/amd/display: Add null checker before passing variables

2024-06-11 Thread Hamza Mahfooz
From: Alex Hung Checks null pointer before passing variables to functions. This fixes 3 NULL_RETURNS issues reported by Coverity. Reviewed-by: Harry Wentland Acked-by: Hamza Mahfooz Signed-off-by: Alex Hung --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 9 ++--- 1 file changed

[PATCH 07/36] drm/amd/display: fix minor coding errors where dml21 phase 5 uses wrong variables

2024-06-11 Thread Hamza Mahfooz
From: Wenjing Liu There is a coding error which causes incorrect variables to be assigned in DML21 phase 5. Reviewed-by: Dillon Varone Acked-by: Hamza Mahfooz Signed-off-by: Wenjing Liu --- .../gpu/drm/amd/display/dc/dml2/dml21/src/dml2_top/dml_top.c | 4 ++-- 1 file changed, 2 insertions

[PATCH 13/36] drm/amd/display: Explicitly extend unsigned 16 bit to 64 bit

2024-06-11 Thread Hamza Mahfooz
pper bits of the result will all be 1. Cast it to unsigned long to avoid possible overflow. This fixes 4 SIGN_EXTENSION issues reported by Coverity. Reviewed-by: Harry Wentland Acked-by: Hamza Mahfooz Signed-off-by: Alex Hung --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 4 ++-

[PATCH 10/36] drm/amd/display: Attempt to avoid empty TUs when endpoint is DPIA

2024-06-11 Thread Hamza Mahfooz
is low enough that we will never require 2:1 ODM combine. Cc: sta...@vger.kernel.org # 6.6+ Reviewed-by: Wenjing Liu Acked-by: Hamza Mahfooz Signed-off-by: Michael Strauss --- .../amd/display/dc/hwss/dcn35/dcn35_hwseq.c | 72 +++ .../amd/display/dc/hwss/dcn35/dcn35_hwseq.h

[PATCH 04/36] Revert "drm/amd/display: workaround for oled eDP not lighting up on DCN401"

2024-06-11 Thread Hamza Mahfooz
From: Joshua Aberback This reverts commit e296c84e7d0561ed626591e1cf9b71a7ba0133f4. A proper fix for this issue has been implemented in DMUB FW. So, no need to keep the workaround. Reviewed-by: Wenjing Liu Acked-by: Hamza Mahfooz Signed-off-by: Joshua Aberback --- drivers/gpu/drm/amd

[PATCH 09/36] drm/amd/display: Refactor DCN3X into component folder

2024-06-11 Thread Hamza Mahfooz
From: Mounika Adhuri [why] Move DCN3X files to unique component folder. [how] Create respective component folder in dc, move the DCN3X files into corresponding new folders and made appropriate changes for compilation in Makefiles. Reviewed-by: Martin Leung Acked-by: Hamza Mahfooz Signed-off

[PATCH 06/36] drm/amd/display: Remove redundant condition in VBA 314 func

2024-06-11 Thread Hamza Mahfooz
-by: Aurabindo Pillai Reviewed-by: Alex Hung Acked-by: Hamza Mahfooz Signed-off-by: Ivan Lipski --- .../drm/amd/display/dc/dml/dcn314/display_mode_vba_314.c | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/dml/dcn314/display_mode_vba_314.c b/drivers/gpu/drm

[PATCH 02/36] drm/amd/display: Enable DCN401 idle optimizations by default

2024-06-11 Thread Hamza Mahfooz
From: Dillon Varone [WHY] Re-enable idle optimizations by default. Reviewed-by: Alvin Lee Acked-by: Hamza Mahfooz Signed-off-by: Dillon Varone --- drivers/gpu/drm/amd/display/dc/resource/dcn401/dcn401_resource.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display

[PATCH 01/36] drm/amd/display: DCN401 full power down in HW init if any link enabled

2024-06-11 Thread Hamza Mahfooz
WSS function for DCN401 - remove debug bit to disable boot optimizations for DCN401 Reviewed-by: Wenjing Liu Acked-by: Hamza Mahfooz Signed-off-by: Joshua Aberback --- drivers/gpu/drm/amd/display/dc/hwss/dcn401/dcn401_init.c | 1 + drivers/gpu/drm/amd/display/dc/resource/dcn

[PATCH 08/36] drm/amd/display: On clock init, maintain DISPCLK freq

2024-06-11 Thread Hamza Mahfooz
to ensure the timing is valid and unchanged. Reviewed-by: Alvin Lee Acked-by: Hamza Mahfooz Signed-off-by: Chris Park --- .../display/dc/clk_mgr/dcn401/dcn401_clk_mgr.c | 17 + .../amd/display/dc/hwss/dcn401/dcn401_hwseq.c | 11 ++- 2 files changed, 27 insertions

[PATCH 00/36] DC Patches June 11, 2024

2024-06-11 Thread Hamza Mahfooz
Cc: Daniel Wheeler Alex Hung (15): drm/amd/display: Explicitly extend unsigned 16 bit to 64 bit drm/amd/display: Add null checker before passing variables drm/amd/display: Check BIOS images before it is used drm/amd/display: Skip wbscl_set_scaler_filter if filter is null

[PATCH 03/36] drm/amd/display: Add dcn401 DIG fifo enable/disable

2024-06-11 Thread Hamza Mahfooz
Acked-by: Hamza Mahfooz Signed-off-by: Relja Vojvodic --- .../gpu/drm/amd/display/dc/dcn35/dcn35_dio_stream_encoder.c | 4 ++-- .../gpu/drm/amd/display/dc/dcn35/dcn35_dio_stream_encoder.h | 6 ++ .../drm/amd/display/dc/dcn401/dcn401_dio_stream_encoder.c | 4 +++- 3 files changed, 11 insertions

Re: [PATCH] Revert "drm/amd/display: avoid large on-stack structures"

2024-06-04 Thread Hamza Mahfooz
On 6/4/24 13:45, Aurabindo Pillai wrote: This reverts commit 44069f0f9b1fe577c5d4f05fa9eb02db8c618adc since the code path is called from FPU context, and triggers error like: [ 26.924055] BUG: sleeping function called from invalid context at include/linux/sched/mm.h:306 [ 26.924060]

[PATCH] drm/amd/display: prevent register access while in IPS

2024-06-03 Thread Hamza Mahfooz
We can't read/write to DCN registers while in IPS. Since, that can cause the system to hang. So, before proceeding with the access in that scenario, force the system out of IPS. Cc: sta...@vger.kernel.org # 6.6+ Signed-off-by: Hamza Mahfooz --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c

Re: [PATCH] drm/amd/display: Convert some legacy DRM debug macros into appropriate categories

2024-05-30 Thread Hamza Mahfooz
On 5/28/24 08:57, Tvrtko Ursulin wrote: From: Tvrtko Ursulin Currently when one enables driver debugging dmesg gets spammed, at I suspect vblank rate, with messages like: [drm:amdgpu_dm_atomic_check [amdgpu]] MPO enablement requested on crtc:[f073c3bb] Fix if by converting some

[PATCH 28/28] drm/amd/display: 3.2.280

2024-04-03 Thread Hamza Mahfooz
From: Aric Cyr This version brings along the following: - DCN10 fixes - DCN316 fixes - DML2 fixes - DWB fixes - Expanded FAMS support - Misc code style fixes - ODM fixes - VSC SDP fixes Acked-by: Hamza Mahfooz Signed-off-by: Aric Cyr --- drivers/gpu/drm/amd/display/dc/dc.h | 2 +- 1 file

[PATCH 21/28] drm/amd/display: Update resource capabilities and debug struct for DCN201

2024-04-03 Thread Hamza Mahfooz
From: Rodrigo Siqueira Some of the resource capabilities for DCN201 and the debug default option are outdated. This commit just set some of the missing configurations for DCN201. Acked-by: Hamza Mahfooz Signed-off-by: Rodrigo Siqueira --- .../gpu/drm/amd/display/dc/resource/dcn201

[PATCH 22/28] drm/amd/display: Disable P010 Support of DCN 1.0

2024-04-03 Thread Hamza Mahfooz
From: Rodrigo Siqueira [WHY] DCN 1.0 is not ready for the P010 support. [HOW] 1. Set the P010 plan_cap of DCN 1.0 to be false. 2. Let the DM do the plan cap initialization of DCN 1.0. Reviewed-by: Rodrigo Siqueira Signed-off-by: Figo Wang Signed-off-by: Rodrigo Siqueira ---

[PATCH 24/28] drm/amd/display: Add fallback configuration when set DRR

2024-04-03 Thread Hamza Mahfooz
From: Rodrigo Siqueira Set OTG/OPTC parameter to 0 if failed to set DRR. Acked-by: Hamza Mahfooz Signed-off-by: Rodrigo Siqueira --- .../gpu/drm/amd/display/dc/optc/dcn32/dcn32_optc.c| 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd

[PATCH 19/28] drm/amd/display: Adjust some includes used by display

2024-04-03 Thread Hamza Mahfooz
From: Rodrigo Siqueira Some of the includes used in the DC can be removed and others need to be update. This commit adjusts some of those headers in the display code. Acked-by: Hamza Mahfooz Signed-off-by: Rodrigo Siqueira --- drivers/gpu/drm/amd/display/dc/clk_mgr/dcn10/rv1_clk_mgr.c

[PATCH 23/28] drm/amd/display: Update DCN10 resource

2024-04-03 Thread Hamza Mahfooz
From: Rodrigo Siqueira Update DCN10 to use legacy fast update and ensure that the MPCC count is the same as the pipe_count. Acked-by: Hamza Mahfooz Signed-off-by: Rodrigo Siqueira --- drivers/gpu/drm/amd/display/dc/resource/dcn10/dcn10_resource.c | 2 ++ 1 file changed, 2 insertions(+) diff

[PATCH 25/28] drm/amd/display: Add fallback configuration for set DRR in DCN10

2024-04-03 Thread Hamza Mahfooz
From: Rodrigo Siqueira Set OTG/OPTC parameters to 0 if something goes wrong on DCN10. Acked-by: Hamza Mahfooz Signed-off-by: Rodrigo Siqueira --- .../drm/amd/display/dc/optc/dcn10/dcn10_optc.c| 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/drivers/gpu

[PATCH 27/28] drm/amd/display: Reorganize dwb header

2024-04-03 Thread Hamza Mahfooz
From: Rodrigo Siqueira This commit makes some small adjustments in the dwb header. Acked-by: Hamza Mahfooz Signed-off-by: Rodrigo Siqueira --- drivers/gpu/drm/amd/display/dc/inc/hw/dwb.h | 30 + 1 file changed, 19 insertions(+), 11 deletions(-) diff --git a/drivers/gpu

[PATCH 26/28] drm/amd/display: Change DPCD address range

2024-04-03 Thread Hamza Mahfooz
From: Rodrigo Siqueira Change DP_PHY_REPEATER_EXTENDED_WAIT_TIMEOUT with DP_PHY_REPEATER_128B132B_RATES. Acked-by: Hamza Mahfooz Signed-off-by: Rodrigo Siqueira --- drivers/gpu/drm/amd/display/dc/link/protocols/link_dpcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 12/28] drm/amd/display: Set VSC SDP Colorimetry same way for MST and SST

2024-04-03 Thread Hamza Mahfooz
From: Harry Wentland The previous check for the is_vsc_sdp_colorimetry_supported flag for MST sink signals did nothing. Simplify the code and use the same check for MST and SST. Cc: sta...@vger.kernel.org Reviewed-by: Agustin Gutierrez Acked-by: Hamza Mahfooz Signed-off-by: Harry Wentland

[PATCH 18/28] drm/amd/display: Add comments to improve the code readability

2024-04-03 Thread Hamza Mahfooz
From: Rodrigo Siqueira This commit just introduce some basic comments that helps to understand the overall behavior of some structs. Acked-by: Hamza Mahfooz Signed-off-by: Rodrigo Siqueira --- drivers/gpu/drm/amd/display/dc/inc/core_types.h | 2 ++ drivers/gpu/drm/amd/display

[PATCH 20/28] drm/amd/display: Drop legacy code

2024-04-03 Thread Hamza Mahfooz
From: Rodrigo Siqueira This commit removes code that are not used by display anymore. Acked-by: Hamza Mahfooz Signed-off-by: Rodrigo Siqueira --- .../amd/display/dc/inc/hw/stream_encoder.h| 4 --- drivers/gpu/drm/amd/display/dc/inc/resource.h | 7 .../amd/display/dc/optc/dcn20

[PATCH 08/28] drm/amd/display: expand the non standard link rate for testing

2024-04-03 Thread Hamza Mahfooz
From: Allen Pan [Why] 6.75 Gbps link rate training for DP_TEST_LINK_RATE_RATE_8 Reviewed-by: Charlene Liu Acked-by: Hamza Mahfooz Signed-off-by: Allen Pan --- drivers/gpu/drm/amd/display/dc/dc_dp_types.h | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/amd/display/dc

[PATCH 14/28] drm/amd/display: Rebuild test pattern params for DP_TEST_PATTERN_VIDEO_MODE

2024-04-03 Thread Hamza Mahfooz
of the previously enabled test pattern. [How] Rebuild test pattern params even when test pattern is video mode, allowing the pipe to have updated test_pattern_params values. Reviewed-by: Nevenko Stupar Reviewed-by: Chaitanya Dhere Acked-by: Hamza Mahfooz Signed-off-by: George Shen --- drivers/gpu/drm

[PATCH 16/28] drm/amd/display: [FW Promotion] Release 0.0.212.0

2024-04-03 Thread Hamza Mahfooz
From: Anthony Koo - Add boot option to change the ONO powerup flow, impacting the order of power domains to power up or down first Acked-by: Hamza Mahfooz Signed-off-by: Anthony Koo --- drivers/gpu/drm/amd/display/dmub/inc/dmub_cmd.h | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 17/28] drm/amd/display: Add V_TOTAL_REGS to dcn10

2024-04-03 Thread Hamza Mahfooz
From: Rodrigo Siqueira DCN10 OPTC is used by other DCNs, and in some cases it might be useful to have V_TOTAL_REGS available. This commit add V_TOTAL_REGS as part of the TG field. Acked-by: Hamza Mahfooz Signed-off-by: Rodrigo Siqueira --- drivers/gpu/drm/amd/display/dc/optc/dcn10

[PATCH 07/28] drm/amd/display: fix disable otg wa logic in DCN316

2024-04-03 Thread Hamza Mahfooz
From: Fudongwang [Why] Wrong logic cause screen corruption. [How] Port logic from DCN35/314. Cc: sta...@vger.kernel.org Reviewed-by: Nicholas Kazlauskas Acked-by: Hamza Mahfooz Signed-off-by: Fudongwang --- .../dc/clk_mgr/dcn316/dcn316_clk_mgr.c| 19 --- 1 file

[PATCH 15/28] Revert "drm/amd/display: Enabling urgent latency adjustment for DCN35"

2024-04-03 Thread Hamza Mahfooz
From: Qili Lu This reverts commit 5145de86407c2d28cfed007b5014c0d0b40d5913. It causes a dead loop in dml_prefetch_check. Reviewed-by: Nicholas Kazlauskas Acked-by: Hamza Mahfooz Signed-off-by: Qili Lu --- drivers/gpu/drm/amd/display/dc/dml/dcn35/dcn35_fpu.c | 4 ++-- 1 file changed, 2

[PATCH 11/28] drm/amd/display: Program VSC SDP colorimetry for all DP sinks >= 1.4

2024-04-03 Thread Hamza Mahfooz
: Xaver Hugl Cc: Melissa Wen Cc: Agustin Gutierrez Reviewed-by: Agustin Gutierrez Acked-by: Hamza Mahfooz Signed-off-by: Harry Wentland --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/display

[PATCH 13/28] drm/amd/display: Adjust dprefclk by down spread percentage.

2024-04-03 Thread Hamza Mahfooz
From: Zhongwei [Why] OLED panels show no display for large vtotal timings. [How] Check if ss is enabled and read from lut for spread spectrum percentage. Adjust dprefclk as required. DP_DTO adjustment is for edp only. Cc: sta...@vger.kernel.org Reviewed-by: Nicholas Kazlauskas Acked-by: Hamza

[PATCH 06/28] drm/amd/display: Do not recursively call manual trigger programming

2024-04-03 Thread Hamza Mahfooz
From: Dillon Varone [WHY] We should not be recursively calling the manual trigger programming function when FAMS is not in use. Cc: sta...@vger.kernel.org Reviewed-by: Alvin Lee Acked-by: Hamza Mahfooz Signed-off-by: Dillon Varone --- drivers/gpu/drm/amd/display/dc/optc/dcn32/dcn32_optc.c

[PATCH 10/28] drm/amd/display: add dwb support to dml2

2024-04-03 Thread Hamza Mahfooz
up Reviewed-by: Chris Park Acked-by: Hamza Mahfooz Signed-off-by: Charlene Liu --- .../amd/display/dc/dml2/display_mode_core.c | 1 + .../amd/display/dc/dml2/display_mode_core.h | 1 + .../dc/dml2/display_mode_lib_defines.h| 2 + .../display/dc/dml2/dml2_translation_helper.c | 45

[PATCH 09/28] drm/amd/display: refactor vpg.h

2024-04-03 Thread Hamza Mahfooz
From: Parandhaman K why and how: as part of cleanup, need to refactor vpg. It was improperly referenced as a dcn specfic part of link. the dcn agnostic code needed was ripped out and put into vpg.h, now in dc/inc/hw. Reviewed-by: Alvin Lee Acked-by: Hamza Mahfooz Signed-off-by: Parandhaman K

[PATCH 00/28] DC Patches April 3, 2024

2024-04-03 Thread Hamza Mahfooz
Cc: Daniel Wheeler Alex Hung (1): drm/amd/display: Return max resolution supported by DWB Allen Pan (1): drm/amd/display: expand the non standard link rate for testing Anthony Koo (1): drm/amd/display: [FW Promotion] Release 0.0.212.0 Aric Cyr (1): drm/amd/display: 3.2.280 Charlene

[PATCH 04/28] drm/amd/display: fix an incorrect ODM policy assigned for subvp

2024-04-03 Thread Hamza Mahfooz
ODM policy logic. So it doesn't matter whether if main pipe's ODM policy is set, phantom pipe will always have the same policy because it running the same calcualtion to derive ODM policy. Cc: sta...@vger.kernel.org Reviewed-by: Alvin Lee Acked-by: Hamza Mahfooz Signed-off-by: Wenjing Liu

[PATCH 05/28] drm/amd/display: Add driver support for future FAMS versions

2024-04-03 Thread Hamza Mahfooz
From: Dillon Varone [WHY] Changes to support future versions of FAMS. Reviewed-by: Alvin Lee Acked-by: Hamza Mahfooz Signed-off-by: Dillon Varone --- .../gpu/drm/amd/display/dc/bios/bios_parser.c | 2 -- drivers/gpu/drm/amd/display/dc/core/dc.c | 5 - .../drm/amd/display/dc/core

[PATCH 03/28] drm/amd/display: always reset ODM mode in context when adding first plane

2024-04-03 Thread Hamza Mahfooz
removed. This has caused an issue where we kept ODM combine when it should have been remove when a plane is added. The change is to reset ODM mode when adding the first plane. Cc: sta...@vger.kernel.org Reviewed-by: Alvin Lee Acked-by: Hamza Mahfooz Signed-off-by: Wenjing Liu --- drivers/gpu/drm

[PATCH 01/28] drm/amd/display: Add option to configure mapping policy for edp0 on dp1

2024-04-03 Thread Hamza Mahfooz
From: Lewis Huang [Why] We want flexibility to choose how pwrseq instance is mapped to eDP panel [How] Add configuration option to choose the pwrseq mapping policy. When enabled, allow fixed mapping between DIG inst to pwrseq inst. Reviewed-by: Anthony Koo Acked-by: Hamza Mahfooz Signed-off

[PATCH 02/28] drm/amd/display: Return max resolution supported by DWB

2024-04-03 Thread Hamza Mahfooz
From: Alex Hung mode_config's max width x height is 4096x2160 and is higher than DWB's max resolution 3840x2160 which is returned instead. Cc: sta...@vger.kernel.org Reviewed-by: Harry Wentland Acked-by: Hamza Mahfooz Signed-off-by: Alex Hung --- drivers/gpu/drm/amd/display/amdgpu_dm

Re: [PATCH 01/12] kbuild: make -Woverride-init warnings more consistent

2024-03-26 Thread Hamza Mahfooz
: 2cd3271b7a31 ("kbuild: avoid duplicate warning options") Signed-off-by: Arnd Bergmann Acked-by: Hamza Mahfooz For the amdgpu changes. --- drivers/gpu/drm/amd/display/dc/dce110/Makefile | 2 +- drivers/gpu/drm/amd/display/dc/dce112/Makefile | 2 +- drivers/gpu/drm/amd/display

[PATCH] drm/amd/display: fix IPX enablement

2024-03-22 Thread Hamza Mahfooz
hecks for exiting idle in DC") Tested-by: Mark Broadworth Signed-off-by: Hamza Mahfooz --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_psr.c | 8 +--- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_psr.h | 2 +- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/a

Re: [PATCH] drm/amd/display: Get min/max vfreq from display_info

2024-03-13 Thread Hamza Mahfooz
On 3/12/24 09:47, Harry Wentland wrote: We need the min/max vfreq on the amdgpu_dm_connector in order to program VRR. Fixes: db3e4f1cbb84 ("drm/amd/display: Use freesync when `DRM_EDID_FEATURE_CONTINUOUS_FREQ` found") Signed-off-by: Harry Wentland Acked-by: Hamza Mahfooz ---

[PATCH 6/6] drm/amd/display: add amdgpu_dm support for DCN351

2024-02-28 Thread Hamza Mahfooz
Add Display Manager specific changes for DCN3.5.1. Signed-off-by: Hamza Mahfooz --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 9 + drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_hdcp.c | 1 + 2 files changed, 10 insertions(+) diff --git a/drivers/gpu/drm/amd/display

[PATCH 5/6] drm/amd/display: add DC changes for DCN351

2024-02-28 Thread Hamza Mahfooz
Add DC support for DCN 3.5.1. Signed-off-by: Hamza Mahfooz --- .../display/dc/bios/command_table_helper2.c |1 + .../gpu/drm/amd/display/dc/core/dc_resource.c |6 + .../dc/dcn35/dcn35_dio_stream_encoder.h |1 + drivers/gpu/drm/amd/display/dc/dml/Makefile |3 + .../amd

[PATCH 4/6] drm/amd/display: add DCN351 IRQ changes

2024-02-28 Thread Hamza Mahfooz
Add DCN3.5.1 interrupt support. Signed-off-by: Hamza Mahfooz --- drivers/gpu/drm/amd/display/dc/irq/Makefile | 11 +- .../dc/irq/dcn351/irq_service_dcn351.c| 409 ++ .../dc/irq/dcn351/irq_service_dcn351.h| 12 + 3 files changed, 431 insertions(+), 1 deletion

[PATCH 3/6] drm/amd/display: add DMUB source files and changes for DCN351

2024-02-28 Thread Hamza Mahfooz
DMUB support is required to light-up displays. Signed-off-by: Hamza Mahfooz --- drivers/gpu/drm/amd/display/dmub/src/Makefile | 1 + .../drm/amd/display/dmub/src/dmub_dcn351.c| 34 +++ .../drm/amd/display/dmub/src/dmub_dcn351.h| 13 +++ .../gpu/drm/amd/display/dmub

[PATCH 1/6] drm/amd/display: add DCN351 version identifiers

2024-02-28 Thread Hamza Mahfooz
Add DCN3.5.1 ASIC identifiers. Signed-off-by: Hamza Mahfooz --- drivers/gpu/drm/amd/display/dmub/dmub_srv.h | 1 + drivers/gpu/drm/amd/display/include/dal_asic_id.h | 2 ++ 2 files changed, 3 insertions(+) diff --git a/drivers/gpu/drm/amd/display/dmub/dmub_srv.h b/drivers/gpu/drm/amd

Re: [PATCH] drm/amd/display: Add monitor patch for specific eDP

2024-02-28 Thread Hamza Mahfooz
: Hamza Mahfooz Cc: Tsung-hua Lin Cc: Chris Chi Cc: Harry Wentland Tested-by: Daniel Wheeler Reviewed-by: Sun peng Li Acked-by: Rodrigo Siqueira Signed-off-by: Ivan Lipski --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 6 ++ 1 file changed, 6 insertions(+) diff --git

Re: [PATCH 1/2] Revert "drm/amd: Remove freesync video mode amdgpu parameter"

2024-02-27 Thread Hamza Mahfooz
/2985 Signed-off-by: Alex Deucher Series is: Acked-by: Hamza Mahfooz --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 + drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 27 + 2 files changed, 28 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu

Re: [PATCH v2] drm/amd/display: add panel_power_savings sysfs entry to eDP connectors

2024-02-16 Thread Hamza Mahfooz
On 2/16/24 03:19, Pekka Paalanen wrote: On Fri, 2 Feb 2024 10:28:35 -0500 Hamza Mahfooz wrote: We want programs besides the compositor to be able to enable or disable panel power saving features. Could you also explain why, in the commit message, please? It is unexpected for arbitrary

Re: [PATCH v2] drm/amd/display: add panel_power_savings sysfs entry to eDP connectors

2024-02-16 Thread Hamza Mahfooz
On 2/16/24 03:19, Pekka Paalanen wrote: On Fri, 2 Feb 2024 10:28:35 -0500 Hamza Mahfooz wrote: We want programs besides the compositor to be able to enable or disable panel power saving features. Could you also explain why, in the commit message, please? It is unexpected for arbitrary

Re: [PATCH 15/17] drm/amd/display: Drop unnecessary header

2024-02-14 Thread Hamza Mahfooz
those includes. Signed-off-by: Rodrigo Siqueira Reviewed-by: Hamza Mahfooz --- drivers/gpu/drm/amd/display/dc/basics/dce_calcs.c | 2 -- drivers/gpu/drm/amd/display/dc/clk_mgr/clk_mgr.c | 2 -- drivers/gpu/drm/amd/display/dc/clk_mgr/dcn10/rv1_clk_mgr.c | 2

Re: [PATCH 05/17] drm/amd/display: Remove redundant FPU guard

2024-02-14 Thread Hamza Mahfooz
On 2/14/24 13:38, Rodrigo Siqueira wrote: The function dcn32_build_wm_range_table call DC_FP_START/END. Drop the unnecessary FPU guard. Signed-off-by: Rodrigo Siqueira Reviewed-by: Hamza Mahfooz --- drivers/gpu/drm/amd/display/dc/clk_mgr/dcn32/dcn32_clk_mgr.c | 2 -- 1 file changed, 2

Re: [PATCH 03/17] drm/amd/display: Remove unused file

2024-02-14 Thread Hamza Mahfooz
On 2/14/24 13:38, Rodrigo Siqueira wrote: The file rv1_clk_mgr_clk.c is not used and for this reason useless. Drop the unnecessary file. Signed-off-by: Rodrigo Siqueira Reviewed-by: Hamza Mahfooz --- .../dc/clk_mgr/dcn10/rv1_clk_mgr_clk.c| 79 --- 1 file changed

Re: [PATCH 02/17] drm/amd/display: Initialize variable with default value

2024-02-14 Thread Hamza Mahfooz
On 2/14/24 13:38, Rodrigo Siqueira wrote: Set a default value for target_div. Signed-off-by: Rodrigo Siqueira Reviewed-by: Hamza Mahfooz --- drivers/gpu/drm/amd/display/dc/clk_mgr/dce100/dce_clk_mgr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm

Re: [PATCH 01/17] drm/amd/display: Remove break after return

2024-02-14 Thread Hamza Mahfooz
On 2/14/24 13:38, Rodrigo Siqueira wrote: Remove break after return since it will never be reached. Signed-off-by: Rodrigo Siqueira Reviewed-by: Hamza Mahfooz --- drivers/gpu/drm/amd/display/dc/clk_mgr/clk_mgr.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/amd

Re: [PATCH 04/17] drm/amd/display: Add SMU timeout check and retry

2024-02-14 Thread Hamza Mahfooz
On 2/14/24 13:38, Rodrigo Siqueira wrote: Instead of only asserting in the case of the SMU wait time is not what we expect, add the SMU timeout check and try again. Signed-off-by: Rodrigo Siqueira Acked-by: Hamza Mahfooz --- .../display/dc/clk_mgr/dcn21/rn_clk_mgr_vbios_smu.c | 11

[PATCH] drm/amdgpu: respect the abmlevel module parameter value if it is set

2024-02-09 Thread Hamza Mahfooz
from changing the ABM level if the module parameter is set to a non-auto value. Signed-off-by: Hamza Mahfooz --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 11 ++- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 15

Re: [PATCH] drm/amd/display: Fix && vs || typos

2024-02-09 Thread Hamza Mahfooz
On 2/9/24 08:02, Dan Carpenter wrote: These ANDs should be ORs or it will lead to a NULL dereference. Fixes: fb5a3d037082 ("drm/amd/display: Add NULL test for 'timing generator' in 'dcn21_set_pipe()'") Fixes: 886571d217d7 ("drm/amd/display: Fix 'panel_cntl' could be null in

[PATCH] drm/amdgpu: make damage clips support configurable

2024-02-08 Thread Hamza Mahfooz
Limonciello Signed-off-by: Hamza Mahfooz --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 + drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 13 + drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 7 +++ 3 files changed, 21 insertions(+) diff --git a/drivers/gpu/drm/amd

[PATCH] drm/amd/display: only call sysfs_remove_group() for eDP connectors

2024-02-05 Thread Hamza Mahfooz
ctors") Signed-off-by: Hamza Mahfooz --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c index adda423615a1..b3

  1   2   3   4   5   6   >