- 19 Feb, 2018 40 commits
-
-
Martin Tsai authored
Signed-off-by:
Martin Tsai <martin.tsai@amd.com> Reviewed-by:
Tony Cheng <Tony.Cheng@amd.com> Acked-by:
Harry Wentland <harry.wentland@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Charlene Liu authored
Signed-off-by:
Charlene Liu <charlene.liu@amd.com> Reviewed-by:
-
Harry Wentland authored
Signed-off-by:
-
Christian König authored
It also needs to be initialized. Signed-off-by:
Christian König <christian.koenig@amd.com> Reviewed-by:
Chunming Zhou <david1.zhou@amd.com> Signed-off-by:
-
Christian König authored
Without it we run into a memory leak. Signed-off-by:
-
Alex Deucher authored
No longer used since we changed the MC programming sequence. Reviewed-by:
Michel Dänzer <michel.daenzer@amd.com> Signed-off-by:
-
Alex Deucher authored
Using the wrong mask. Reviewed-by:
Hans de Ruiter <hans@keasigmadelta.com> Signed-off-by:
-
Arnd Bergmann authored
Building the amd display driver with link-time optimizations revealed a bug that caused dal_cmd_tbl_helper_dce80_get_table() and dal_cmd_tbl_helper_dce110_get_table() get called with an incompatible return type between the two callers in command_table_helper.c and command_table_helper2.c: drivers/gpu/drm/amd/amdgpu/../display/dc/bios/dce80/command_table_helper_dce80.h:31: error: type of 'dal_cmd_tbl_helper_dce80_get_table' does not match original declaration [-Werror=lto-type-mismatch] const struct command_table_helper *dal_cmd_tbl_helper_dce80_get_table(void); drivers/gpu/drm/amd/amdgpu/../display/dc/bios/dce80/command_table_helper_dce80.c:351: note: 'dal_cmd_tbl_helper_dce80_get_table' was previously declared here const struct command_table_helper *dal_cmd_tbl_helper_dce80_get_table(void) drivers/gpu/drm/amd/amdgpu/../display/dc/bios/dce110/command_table_helper_dce110.h:32: error: type of 'dal_cmd_tbl_helper_dce110_get_table' does not match original declaration [-Werror=lto-type-mismatch] const struct command_table_helper *dal_cmd_tbl_helper_dce110_get_table(void); drivers/gpu/drm/amd/amdgpu/../display/dc/bios/dce110/command_table_helper_dce110.c:361: note: 'dal_cmd_tbl_helper_dce110_get_table' was previously declared here const struct command_table_helper *dal_cmd_tbl_helper_dce110_get_table(void) The two versions of the structure are obviously derived from the same one, but have diverged over time, before they got added to the kernel. This moves the structure to a new shared header file and uses the superset of the members, to ensure the interfaces are all compatible. Fixes: ae79c310 ("drm/amd/display: Add DCE12 bios parser support") Signed-off-by:
Arnd Bergmann <arnd@arndb.de> Signed-off-by:
-
Christian König authored
Beneficial when a lot of processes are waiting for VMIDs. Signed-off-by:
-
Christian König authored
Now that we have the different cases for grabbing a VMID in separate functions, restructure the top level function to only have one place where VMIDs are assigned to jobs. Signed-off-by:
-
Christian König authored
Let's try this once more. Signed-off-by:
-
Christian König authored
Drop the "_locked" from the name, cleanup and simplify the logic a bit. Add missing comments. Signed-off-by:
-
Christian König authored
We do this later on when we flush the VMID anyway. Signed-off-by:
-
Christian König authored
The variable is protected by the VMID mutex anyway. v2: grab the mutex while resetting the VMID as well Signed-off-by:
-
Christian König authored
No functional change, but makes it easier to maintain the code. Signed-off-by:
-
Christian König authored
Similar to finding an idle one before reuse. This guarantees fairness between processes. Otherwise process with a reserved VMID have an unfair advantage while scheduling jobs. v2: improve commit message Signed-off-by:
-
Bas Nieuwenhuizen authored
If these bos are evicted and are in the validated list things blow up, so do not put them in there. Notably, that tries to add the bo to the LRU twice, which results in a BUG_ON in ttm_bo.c. While for the bo_list an alternative would be to not allow always valid bos in there, that does not work for the user fence. v2: Fixed whitespace issue pointed out by checkpatch.pl Signed-off-by:
Bas Nieuwenhuizen <basni@chromium.org> Reviewed-by:
-
Andrey Grodzovsky authored
This reverts commit 75737cb4. Fixes compute rings test failure on bare metal during full GPU reset. RCA: the ring buffer has to be filled with valid packets (such as NOPs) first before submitting MAP_QUEUEs packet into KIQ. Once a compute engine is mapped, it will immediately execute the ring buffer if the RTPR is not equal to the WTPR from the MQD. It could lead to engine hang if the ring buffer filled with random data. Signed-off-by:
Andrey Grodzovsky <andrey.grodzovsky@amd.com> Reviewed-by:
-
Christian König authored
Keep that at a common place instead of spread over all engines. Signed-off-by:
Felix Kuehling <felix.kuehling@amd.com> Reviewed-by:
-
Christian König authored
Add emit_reg_wait implementation for VCN v1. v2: cleanup the existing code as well Signed-off-by:
-
Christian König authored
Add emit_reg_wait implementation for VCE v4. v2: call new function directly from existing code Signed-off-by:
-
Christian König authored
Add emit_reg_wait implementation for UVD v7. v2: call new function directly from the existing code Signed-off-by:
-
Christian König authored
Add emit_reg_wait implementation for SDMA v4. Signed-off-by:
-
Christian König authored
Implement emit_reg_wait for gfx v9. Signed-off-by:
-
Christian König authored
Allows us to wait for a register value/mask on a ring. Signed-off-by:
-
Christian König authored
Forgot to update that during recent changes. Signed-off-by:
-
Rex Zhu authored
when echo "01">pp_dpm_pcie the pcie dpm will fix in highest link speed. But user should expect auto speed between level 0 and level1 Reviewed-by:
Rex Zhu <Rex.Zhu@amd.com> Signed-off-by:
-
Rex Zhu authored
Only user enter manual performance mode, driver allow user configure the sclk/mclk dpm parameters through sysfs pp_power_profile_mode. Reviewed-by:
-
Rex Zhu authored
Only when user set manual performance mode, driver enable pp_dpm_force_clock_level. so check the mode in pp_dpm_force_clock_level, and delete the same logic in callback functions. Reviewed-by:
-
Rex Zhu authored
Reviewed-by:
-
Christian König authored
Older VCE FW versions are buggy and can't work with 48bit address spaces. RFC: Should we limit the address space or just reject loading the older VCE firmware? Signed-off-by:
-
Luis de Bethencourt authored
The trailing semicolon is an empty statement that does no operation. Removing it since it doesn't do anything. Signed-off-by:
Luis de Bethencourt <luisbg@kernel.org> Signed-off-by:
-
Luis de Bethencourt authored
The trailing semicolon is an empty statement that does no operation. Removing the two instances of them since they don't do anything. Signed-off-by:
-
Alex Deucher authored
The atomic debugfs stuff gets created in drm_dev_alloc() but this gets called before we've enumerated all of our IPs, so move the DRIVER_ATOMIC flag setting to fix that. Since DRIVER_ATOMIC is a driver flag it's currently global to the driver so setting it affects all GPUs driven by the driver. Unfortunately, not all GPUs support atomic. Warn the user if that is the case. This is the same as our current behavior, but at least the atomic debugfs stuff gets created now. Acked-by:
-
Alex Deucher authored
It seems to be working now. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=102372Reviewed-by:
Mike Lothian <mike@fireburn.co.uk> Reviewed-by:
-
Tom St Denis authored
Remove redundant store of return code. Signed-off-by:
Tom St Denis <tom.stdenis@amd.com> Reviewed-by:
-
Tom St Denis authored
Add missing {} braces. Signed-off-by: -
Tom St Denis authored
Add missing {} braces. Signed-off-by: -
Tom St Denis authored
Hoist the comparison of the ret to -EDEADLK above the two code paths to simplify the function. Signed-off-by:
-
Tom St Denis authored
The dual ret/retval was more complex than need be. Now we drop the retval variable and assign the appropriate VM codes to ret instead. Signed-off-by:
-
