[PATCH 1/1] register refresh to add mmGC_CAC_INDEX_AUTO_INCR_EN

2021-01-25 Thread raykwok1150
From: Guo Lei sync form drm-next Signed-off-by: Guo Lei --- src/lib/ip/gfx90_bits.i | 7 ++- src/lib/ip/gfx90_regs.i | 2 +- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/src/lib/ip/gfx90_bits.i b/src/lib/ip/gfx90_bits.i index 6741947..8aabb8a 100644 ---

Re: [PATCH 2/2] drm/amd/display: Fix HDMI deep color output for DCE 6-11.

2021-01-25 Thread Mario Kleiner
Thanks Alex and Nicholas! Brings quite a bit of extra shiny to those older asics :) Nicholas, any thoughts on my cover-letter wrt. why a similar patch (that I wrote and tested to no good or bad effect) not seem to be needed on DCN, and probably not DCE-11.2+ either? Is what is left in DC for

Re: [PATCH 2/2] drm/amd/display: Fix HDMI deep color output for DCE 6-11.

2021-01-25 Thread Kazlauskas, Nicholas
On 2021-01-25 12:57 p.m., Alex Deucher wrote: On Thu, Jan 21, 2021 at 1:17 AM Mario Kleiner wrote: This fixes corrupted display output in HDMI deep color 10/12 bpc mode at least as observed on AMD Mullins, DCE-8.3. It will hopefully also provide fixes for other DCE's up to DCE-11, assuming

Re: [PATCH 2/2] drm/amd/display: Fix HDMI deep color output for DCE 6-11.

2021-01-25 Thread Alex Deucher
On Thu, Jan 21, 2021 at 1:17 AM Mario Kleiner wrote: > > This fixes corrupted display output in HDMI deep color > 10/12 bpc mode at least as observed on AMD Mullins, DCE-8.3. > > It will hopefully also provide fixes for other DCE's up to > DCE-11, assuming those will need similar fixes, but i

Re: [PATCH] amdgpu: fix clang build warning

2021-01-25 Thread Alex Deucher
On Mon, Jan 25, 2021 at 7:24 AM Arnd Bergmann wrote: > > From: Arnd Bergmann > > clang warns about the -mhard-float command line arguments > on architectures that do not support this: > > clang: error: argument unused during compilation: '-mhard-float' > [-Werror,-Wunused-command-line-argument]

Re: [PATCH] drm/amdgpu: fix bitwise vs logical negate

2021-01-25 Thread Alex Deucher
On Mon, Jan 25, 2021 at 3:46 AM Dan Carpenter wrote: > > There was a mixup between logical and bitwise negate so it just sets > "data1" and "data2" to zero. > > Fixes: 3c9a7b7d6e75 ("drm/amdgpu: update mmhub mgcg for mmhub_v2_3") > Signed-off-by: Dan Carpenter Someone else sent the same fix

Re: [PATCH] drm/amd/display: Fix a potential NULL dereference

2021-01-25 Thread Alex Deucher
On Mon, Jan 25, 2021 at 3:47 AM Dan Carpenter wrote: > > The debug printk dereferences "link->link_enc" before we have ensured > that it is non-NULL. Fix this potential NULL derefence by moving the > printk after the check. > > Fixes: 1975b95ad4e7 ("drm/amd/display: Log link/connector info

Re: [PATCH] drm/amd/display: change license of color_table.c

2021-01-25 Thread Alex Deucher
On Mon, Jan 25, 2021 at 7:07 AM Jonathan Gray wrote: > > This still needs to be corrected. Applied. Thanks! Alex > > On Thu, Nov 19, 2020 at 01:30:41PM +1100, Jonathan Gray wrote: > > Change the license of color_table.c to match color_table.h granting > > permission to modify and distribute.

Re: [PATCH] drm/amd/display: fix unused variable warning

2021-01-25 Thread Harry Wentland
On 2021-01-25 7:48 a.m., Arnd Bergmann wrote: From: Arnd Bergmann After all users of the 'dm' warnings got hidden in an #ifdef, the compiler started warning about it being unused: drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:5380:33: error: unused variable 'dm'

Re: [RFC 0/7] Proposal for isolating FPU operation

2021-01-25 Thread Christian König
Hi Rodrigo, good to see this finally be tackled. The whole approach looks solid to me, just one things I've noted. +void dcn3x_populate_dml_writeback_from_context(struct dc *dc, + struct resource_context *res_ctx, display_e2e_pipe_params_st *pipes) +{ + DC_FP_START(); +

Re: [PATCH v4 01/14] drm/ttm: Remap all page faults to per process dummy page.

2021-01-25 Thread Andrey Grodzovsky
On 1/19/21 8:56 AM, Daniel Vetter wrote: On Mon, Jan 18, 2021 at 04:01:10PM -0500, Andrey Grodzovsky wrote: On device removal reroute all CPU mappings to dummy page. v3: Remove loop to find DRM file and instead access it by vma->vm_file->private_data. Move dummy page installation into a

Re: [PATCH][next] drm/amdgpu: Fix masking binary not operator on two mask operations

2021-01-25 Thread Alex Deucher
Applied. Thanks! Alex On Sun, Jan 24, 2021 at 11:36 PM Huang Rui wrote: > > On Fri, Jan 22, 2021 at 11:00:22PM +0800, Colin King wrote: > > From: Colin Ian King > > > > Currently the ! operator is incorrectly being used to flip bits on > > mask values. Fix this by using the bit-wise ~

[RFC 4/7] drm/amd/display: Move calculate_wm_and_dlg to FPU folder

2021-01-25 Thread Rodrigo Siqueira
The function dcn3x_calculate_wm_and_dlg and dcn30_calculate_wm_and_dlg_fp require access to FPU operation; for this reason, this commit moves this function to the fpu directory. Signed-off-by: Rodrigo Siqueira --- .../drm/amd/display/dc/dcn30/dcn30_resource.c | 181 +-

[RFC 7/7] drm/amd/display: Drop float flages from DCN30 files

2021-01-25 Thread Rodrigo Siqueira
All functions that require FPU access associated with DCN30 were moved to the specific file inside the FPU directory. For this reason, we don't need to use the `-mhard-float -msse` flags for any DCN30 file directly, which means that we can safely drop those flags from the Makefile and any other

[RFC 6/7] drm/amd/display: Move bounding box functions to FPU folder

2021-01-25 Thread Rodrigo Siqueira
This commit moves all operations and data structures related to the bounding box for DCN30 to the FPU folders. Signed-off-by: Rodrigo Siqueira --- .../drm/amd/display/dc/dcn30/dcn30_resource.c | 325 + .../drm/amd/display/dc/dcn30/dcn30_resource.h | 2 -

[RFC 5/7] drm/amd/display: Move patch bounding box to FPU folder

2021-01-25 Thread Rodrigo Siqueira
The function dcn20_patch_bounding_box is shared from DCN2 to DCN3 and uses FPU operations. For this reason, this commit moves this function to the fpu_commons file. Signed-off-by: Rodrigo Siqueira --- .../drm/amd/display/dc/dcn20/dcn20_resource.c | 36 +---

[RFC 3/7] drm/amd/display: Add FPU file for functions shared across ASICs

2021-01-25 Thread Rodrigo Siqueira
In our DC code, we have a couple of functions that use FPU and are shared between multiples ASICs; this situation complicates the work of centralizing FPU functions in a single place. This commit tries to alleviate this problem by creating a file named fpu_commons to centralize shared functions

[RFC 2/7] drm/amd/display: Moves dcn30_set_mcif_arb_params to FPU folder

2021-01-25 Thread Rodrigo Siqueira
The function dcn30_set_mcif_arb_params uses some double-precision operation; for this reason, this commit moves this function to the directory fpu_operation, where all FPU operations should be centralized. Signed-off-by: Rodrigo Siqueira --- .../drm/amd/display/dc/dcn30/dcn30_resource.c | 81

[RFC 1/7] drm/amd/display: Introduce FPU directory inside DC

2021-01-25 Thread Rodrigo Siqueira
The display core files rely on FPU operation, which requires to be compiled with special flags. Ideally, we don't want these FPU operations to get spread around the DC code; nevertheless, it happens in the current source. This commit introduces a new directory named fpu_operation that intends to

[RFC 0/7] Proposal for isolating FPU operation

2021-01-25 Thread Rodrigo Siqueira
Hi, In the display core, we utilize floats and doubles units for calculating modesetting parameters. One side effect of our approach to use double-precision is the fact that we spread multiple FPU access across our driver, which means that we can accidentally clobber user space FPU state. As an

Re: [PATCH] drm/amd/display: use div_s64() for 64-bit division

2021-01-25 Thread Arnd Bergmann
On Mon, Jan 25, 2021 at 1:51 PM Chen, Guchun wrote: > > [AMD Public Use] > > Hi Arnd Bergmann, > > Thanks for your patch. This link error during compile has been fixed by below > commit and been submitted to drm-next branch already. > > 5da047444e82 drm/amd/display: fix 64-bit division issue on

RE: [PATCH] drm/amd/display: use div_s64() for 64-bit division

2021-01-25 Thread Chen, Guchun
[AMD Public Use] Hi Arnd Bergmann, Thanks for your patch. This link error during compile has been fixed by below commit and been submitted to drm-next branch already. 5da047444e82 drm/amd/display: fix 64-bit division issue on 32-bit OS Regards, Guchun -Original Message- From: amd-gfx

[PATCH] drm/amd/display: fix unused variable warning

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann After all users of the 'dm' warnings got hidden in an #ifdef, the compiler started warning about it being unused: drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:5380:33: error: unused variable 'dm' [-Werror,-Wunused-variable] Add another such #ifdef. Fixes:

[PATCH] amdgpu: fix clang build warning

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann clang warns about the -mhard-float command line arguments on architectures that do not support this: clang: error: argument unused during compilation: '-mhard-float' [-Werror,-Wunused-command-line-argument] Move this into the gcc-specific arguments. Fixes: e77165bf7b02

Re: [PATCH] drm/amd/display: change license of color_table.c

2021-01-25 Thread Jonathan Gray
This still needs to be corrected. On Thu, Nov 19, 2020 at 01:30:41PM +1100, Jonathan Gray wrote: > Change the license of color_table.c to match color_table.h granting > permission to modify and distribute. > > Signed-off-by: Jonathan Gray > --- > .../amd/display/modules/color/color_table.c |

[PATCH] drm/amd/display: use div_s64() for 64-bit division

2021-01-25 Thread Arnd Bergmann
From: Arnd Bergmann The open-coded 64-bit division causes a link error on 32-bit machines: ERROR: modpost: "__udivdi3" [drivers/gpu/drm/amd/amdgpu/amdgpu.ko] undefined! ERROR: modpost: "__divdi3" [drivers/gpu/drm/amd/amdgpu/amdgpu.ko] undefined! Use the div_s64() to perform the division here.

[PATCH] drm/amd/display: Fix a potential NULL dereference

2021-01-25 Thread Dan Carpenter
The debug printk dereferences "link->link_enc" before we have ensured that it is non-NULL. Fix this potential NULL derefence by moving the printk after the check. Fixes: 1975b95ad4e7 ("drm/amd/display: Log link/connector info provided in BIOS object table") Signed-off-by: Dan Carpenter ---

[PATCH] drm/amdgpu: fix bitwise vs logical negate

2021-01-25 Thread Dan Carpenter
There was a mixup between logical and bitwise negate so it just sets "data1" and "data2" to zero. Fixes: 3c9a7b7d6e75 ("drm/amdgpu: update mmhub mgcg for mmhub_v2_3") Signed-off-by: Dan Carpenter --- drivers/gpu/drm/amd/amdgpu/mmhub_v2_3.c | 4 ++-- 1 file changed, 2 insertions(+), 2

RE: [PATCH] drm/amd/amdgpu: add error handling to amdgpu_virt_read_pf2vf_data

2021-01-25 Thread Liu, Monk
[AMD Official Use Only - Approved for External Use] Reviewed-by: Monk Liu Thanks -- Monk Liu | Cloud-GPU Core team -- -Original Message- From: Chen, JingWen Sent: Thursday, January 21, 2021 11:13 AM To: