- 17 Jan, 2023 21 commits
-
-
Brandon Syu authored
[Why] There is an issue mapping non-allocated location of memory. It would allocate gpio registers from an array out of bounds. [How] Patch correct numbers of bounds for using. Tested-by:
Daniel Wheeler <Daniel.Wheeler@amd.com> Reviewed-by:
Martin Leung <Martin.Leung@amd.com> Acked-by:
Rodrigo Siqueira <Rodrigo.Siqueira@amd.com> Signed-off-by:
Brandon Syu <Brandon.Syu@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Tony Tascioglu authored
[Why] When an eDP panel is powered externally from a different GPU, we can avoid waiting for hardware sequencing delays when switching the backlight on/off as the display backlight is no longer powered by the original source. [How] This commit extends the 'link_powered_externally' variable to allow bypassing hardware delays for additional backlight commands and force the backlight on/off when a link is powered by another GPU. Tested-by:
Felipe Clark <felipe.clark@amd.com> Acked-by:
Tony Tascioglu <tony.tascioglu@amd.com> Signed-off-by:
-
Charlene Liu authored
keep dig_fifo enable for test pattern generation. Tested-by:
Chris Park <Chris.Park@amd.com> Acked-by:
Charlene Liu <Charlene.Liu@amd.com> Signed-off-by:
-
Dillon Varone authored
[WHY?] For dcn32, SMU handles DISPCLK dentist programming. [HOW?] Only use calls to SMU to set DISPCLK. Tested-by:
Aric Cyr <Aric.Cyr@amd.com> Acked-by:
Dillon Varone <Dillon.Varone@amd.com> Signed-off-by:
-
Saaem Rizvi authored
[Why] DCC meta was found to be detached from usable pixel data. Due to this DCC meta and the end of the fetched part of the frame will be on not be on the same mblk. Furthermore if the meta is not aligned to the mblk size, then we require an extra mblk in MALL to account for this. [How] Always add an additional mblk when DCC is enabled for detachment and misalignment. Tested-by:
Jun Lei <Jun.Lei@amd.com> Acked-by:
Saaem Rizvi <SyedSaaem.Rizvi@amd.com> Signed-off-by:
-
Dillon Varone authored
When using freesync on desktop, need to set freesync active bit for AMD VSDBv3 infopacket. Tested-by:
-
Rodrigo Siqueira authored
Remove some code that is never used from freesync file. Tested-by:
-
Tony Tascioglu authored
[Why] When an eDP panel is powered externally by a different GPU, we don't need to wait for hardware sequencing delays when powering down a link, as the display is not dependent on the GPU being powered down. [How] This commit adds a variable 'link_powered_externally' to indicate when a link is being powered by another GPU. Tested-by:
-
Aurabindo Pillai authored
Checking for disabled master pipe on a timing synchronized pipe is incorrect in the case of ODM combine. This case is acceptable as long as the disabled master pipe is part of the ODM tree. Skip printing error message if this condition holds true. Tested-by:
Aurabindo Pillai <aurabindo.pillai@amd.com> Signed-off-by:
-
Aurabindo Pillai authored
This reverts commit e81b6a44. Reverting to put in a better solution which does not involve checking DCN version. Tested-by:
-
Wenjing Liu authored
Tested-by:
Wesley Chalmers <Wesley.Chalmers@amd.com> Acked-by:
Wenjing Liu <wenjing.liu@amd.com> Signed-off-by:
-
Wenjing Liu authored
Tested-by:
-
Wenjing Liu authored
[why] Extract dp link training logic out to their own files. link_dp_training - high level training sequence and helper functions. link_dp_training_8b_10b - dp1.x training link_dp_training_auxless - aux-less training link_dp_traininig_dpia - dpia training link_dp_training_fixed_vs_pe_retimer - fixed vs pe retimer training link_dp_training_128b_132b - dp2.1 training Tested-by:
-
Saaem Rizvi authored
Current implementation of SubVP does not support cases where stream timing matched neither the destination rect nor the source rect. Will need to further debug to see how we can support these cases. Tested-by:
Alvin Lee <Alvin.Lee2@amd.com> Acked-by:
-
Bhawanpreet Lakha authored
[why] HDCP 1.4 failed on SL8800 SW w/a test driver use. [how] slower down the HW i2c speed when used by HW i2c. Tested-by:
Bhawanpreet Lakha <Bhawanpreet.Lakha@amd.com> Signed-off-by:
-
Dillon Varone authored
Remove array size on array passed to CalculateDETSwathFillLatencyHiding. Tested-by:
-
Dillon Varone authored
MALL SS and Subvp use the same calculations for determining the size of the required allocation for a given surface, which is already done in DML. Add an interface to extract this information from VBA variables and use in their respective helper functions. Also refactor existing code to remove stale workarounds. Tested-by:
-
Dillon Varone authored
DML does not explicitly consider support for space in MALL required for subvp phantom pipes. This adds a check to make sure portion of phantom surface can fit in MALL. Tested-by:
-
Dillon Varone authored
DML incorrectly uses surface width for determining DCC meta size in MALL allocation calculations. Meta pitch should be used instead. Tested-by:
-
Dillon Varone authored
Two issues existed: 1) Configs that support DRR, but have it disabled will fail subvp+vblank validation incorrectly. Use subvp+vblank path for this case. 2) Configs that support DRR and have it enabled can use higher voltage level than required if they also support subvp+vblank. Use lowest supported voltage level for this case. Tested-by:
-
Dan Carpenter authored
This tab was deleted accidentally and triggers a Smatch warning: drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1006 gfx_v8_0_init_microcode() warn: inconsistent indenting Add it back. Fixes: 0aaafb73 ("drm/amd: Use `amdgpu_ucode_*` helpers for GFX8") Signed-off-by:Dan Carpenter <error27@gmail.com> Signed-off-by:
-
- 13 Jan, 2023 19 commits
-
-
Charlene Liu authored
Required by display init, also update get_dig_mode Tested-by:
Hansen Dsouza <hansen.dsouza@amd.com> Reviewed-by:
Duncan Ma <duncan.ma@amd.com> Acked-by:
-
Mustapha Ghaddar authored
[WHY] Last PR missed name of a struct to match in DMUB [HOW] Update the logic in dmub_cmh.h header Tested-by:
Meenakshikumar Somasundaram <Meenakshikumar.Somasundaram@amd.com> Acked-by:
Mustapha Ghaddar <mghaddar@amd.com> Signed-off-by:
-
Yifan Zha authored
[Why] SDMA0_CNTL and MMHUB system aperture related registers are blocked by L1 Policy. Therefore, they cannot be accessed by VF and loged in violation. [How] For MMHUB registers, they will be programmed by PF. So VF will skip to program them in mmhubv3_0. For SDMA0_CNTL which is a PF_only register, VF don't need to program it in sdma_v6_0. Signed-off-by:
Yifan Zha <Yifan.Zha@amd.com> Acked-by:
-
Candice Li authored
Support RAS fatal error mode1 reset on smu v13_0_0 and v13_0_10. Signed-off-by:
Candice Li <candice.li@amd.com> Reviewed-by:
Evan Quan <evan.quan@amd.com> Reviewed-by:
Hawking Zhang <Hawking.Zhang@amd.com> Signed-off-by:
-
jie1zhan authored
From smu firmware,the value of power is transferred in units of watts. Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/2321 Fixes: 137aac26 ("drm/amdgpu/smu12: fix power reporting on renoir") Acked-by:
Jesse Zhang <Jesse.Zhang@amd.com> Reviewed-by:
Aaron Liu <aaron.liu@amd.com> Signed-off-by:
-
Mario Limonciello authored
Some dead code was introduced as part of utilizing the `amdgpu_ucode_*` helpers. Adjust the control flow to make sure that firmware is released in the appropriate error flows. Reported-by:
coverity-bot <keescook+coverity-bot@chromium.org> Addresses-Coverity-ID: 1530548 ("Control flow issues") Fixes: ec787deb ("drm/amd: Use `amdgpu_ucode_*` helpers for GFX9") Signed-off-by:
Mario Limonciello <mario.limonciello@amd.com> Reviewed-by:
Kees Cook <keescook@chromium.org> Reviewed-by:
-
Wenjing Liu authored
Tested-by:
George Shen <George.Shen@amd.com> Acked-by:
-
Wenjing Liu authored
[why] Moving dpcd logic from dc_link_dpcd to link_dpcd as part of link file restructure Tested-by:
-
Wenjing Liu authored
[why] 1. Move dd_link_ddc functions to link_ddc. 2. Move link ddc functions declaration exposed in dc to link.h 3. Move link ddc functions declaration exposed in dm to dc_link.h 4. Remove i2caux_interface.h file Tested-by:
-
Wenjing Liu authored
[why] Factor out hpd handling logic from generic dc link file. Tested-by:
-
Dillon Varone authored
[WHY?] FIFO enable sequence is incomplete as it is currently implemented in FW, and requires reset to prevent the FIFO to be enabled in an invalid state. This cannot be done until DIG FE is connected to the BE. [HOW?] Add FIFO enable sequence in driver for dcn32 with reset after DIG FE is connected to BE. Tested-by:
-
Cruise Hung authored
[Why] The DPIA link encoder (DIG) was not released when a Display XDR 6K monitor was disconnected. That's because the DPIA link encoder logic checked the BW when it removed the link encoder for it. And the timing BW was less than DP link BW. So, it failed to release the link encoder. And that caused the DIG was occupied and can't be used. The reason it checked the BW is it wants to identify whether this link was using DIG or HPO. It skips if it's not DIG. [How] Not to check the BW when it removes the link encoder because it's already in the assignment table, it must be the DIG. Tested-by:
Cruise Hung <Cruise.Hung@amd.com> Signed-off-by:
-
Dmytro Laktyushkin authored
A previous fix attempted to correct mismatch between DM display targets and dc panel instancing by only counting connected panels. This behaviour breaks a feature, thus this is an alternative solution that allows mapping display targets to dc links during mod_power_create. Tested-by:
Nicholas Kazlauskas <Nicholas.Kazlauskas@amd.com> Acked-by:
Dmytro Laktyushkin <Dmytro.Laktyushkin@amd.com> Signed-off-by:
-
Martin Leung authored
This reverts commit 1b5d0e7e. It caused corruption in some media players. Tested-by:
Nevenko Stupar <Nevenko.Stupar@amd.com> Acked-by:
-
Mustapha Ghaddar authored
[WHY] After introducing new DPIA NOTIFICATION we will need to update the exiting BW allocation logic [HOW] Updated the BW alloc source and header files Tested-by:
-
Konstantin Meskhidze authored
This commit fixes memory leakage in dc_construct_ctx() function. Signed-off-by:
Konstantin Meskhidze <konstantin.meskhidze@huawei.com> Signed-off-by:
-
Deepak R Varma authored
A logical evaluation already results in bool. There is no need for using a ternary operator based evaluation and bool conversion of the outcome. Issue identified using boolconv.cocci Coccinelle semantic patch. This was also reported by the Kernel Test Robot. Hence Fixes: 473683a0 ("drm/amd/display: Create a file dedicated for CRTC") Reported-by:
kernel test robot <lkp@intel.com> Signed-off-by:
Deepak R Varma <drv@mailo.com> Signed-off-by:
-
Jiapeng Chong authored
The assignment of the else and if branches is the same, so the if else here is redundant, so we remove it. ./drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c:1951:2-4: WARNING: possible condition with no effect (if == else). Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=3719Reported-by:
Abaci Robot <abaci@linux.alibaba.com> Signed-off-by:
Jiapeng Chong <jiapeng.chong@linux.alibaba.com> Signed-off-by:
-
Mario Limonciello authored
On DCN314 when resuming from s0i3 an ASSERT is shown indicating that `VBIOSSMC_MSG_SetHardMinDcfclkByFreq` returned `VBIOSSMC_Result_Failed`. This isn't a driver bug; it's a BIOS/configuration bug. To make this easier to triage, add an explicit warning when this issue happens. This matches the behavior utilized for failures with `VBIOSSMC_MSG_TransferTableDram2Smu` configuration. Signed-off-by:
Harry Wentland <harry.wentland@amd.com> Signed-off-by:
-
