- 21 Oct, 2017 40 commits
-
-
Tom St Denis authored
The cast of dc_link is redundant. Signed-off-by:
Tom St Denis <tom.stdenis@amd.com> Acked-by:
Andrey Grodzovsky <andey.grodzovsky@amd.com> Reviewed-by:
Harry Wentland <harry.wentland@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Tom St Denis authored
Signed-off-by:
-
Tom St Denis authored
Move WARN_ON higher up and in doing so fix brace style. Signed-off-by:
-
Tom St Denis authored
Signed-off-by:
-
Tom St Denis authored
Replace inlined strncpy with library call. Signed-off-by:
-
Tom St Denis authored
Signed-off-by:
-
Tom St Denis authored
There is a local reference to the dc_link that wasn't being used so we shorten references throughout the function. Signed-off-by:
-
Tom St Denis authored
Signed-off-by:
-
Tom St Denis authored
Signed-off-by:
-
Tom St Denis authored
Signed-off-by:
-
Tom St Denis authored
Signed-off-by:
-
Tom St Denis authored
Signed-off-by:
-
Dave Airlie authored
turns out to be a win to inline this. Signed-off-by:
Dave Airlie <airlied@redhat.com> Reviewed-by:
-
Dave Airlie authored
This is unused code. Signed-off-by:
-
Harry Wentland authored
All files should have MIT headers. Signed-off-by:
-
Harry Wentland authored
This reduces the code size. This is basically a redo of Dave's change with the same name on top of the latest DML. Signed-off-by:
-
Harry Wentland authored
Having one struct on the stack was somewhat confusing. Also mention that the whole function should eventually go. Signed-off-by:
Andrey Grodzovsky <andrey.grodzovsky@amd.com> Signed-off-by:
-
Dmytro Laktyushkin authored
Signed-off-by:
Dmytro Laktyushkin <Dmytro.Laktyushkin@amd.com> Reviewed-by:
Tony Cheng <Tony.Cheng@amd.com> Acked-by:
-
Dmytro Laktyushkin authored
Signed-off-by:
-
Tony Cheng authored
1080p idle, stutter efficiency goes up from 95.8% to 97.8% result in 5mW saving from APU and 8mW saving from DDR4 Signed-off-by:
Yongqiang Sun <yongqiang.sun@amd.com> Acked-by:
-
Harry Wentland authored
Found by 0-day https://lists.freedesktop.org/archives/dri-devel/2017-October/154698.htmlSigned-off-by:
-
Tony Cheng authored
- regression fixes in dal 3.1.06 - more linux upstream backport Signed-off-by:
-
Harry Wentland authored
This function likes to blow 1024 stack size when something is added to the addr struct. For now just dynamically allocate. Signed-off-by:
-
Harry Wentland authored
One more thing that happened due to uneven applying of patches. This mirrors what Darren Salt fixed. Signed-off-by:
-
Harry Wentland authored
Some of this stuff is not really dm_services Signed-off-by:
-
Hersen Wu authored
Signed-off-by:
Hersen Wu <hersenxs.wu@amd.com> Reviewed-by:
-
Tony Cheng authored
Raven bug fixes - RS3 MPO stability improvement - SLS fixes - AM4 fixes DAL 3.1 - merge in upstream patches from DRM maintainer Dave Arilie - hubp, dpp HW objects Signed-off-by:
-
Yue Hin Lau authored
Signed-off-by:
Yue Hin Lau <Yuehin.Lau@amd.com> Reviewed-by:
-
Yongqiang Sun authored
Three monitor connected and playing a video will occupy all 4 pipes, if hot plug forth display, commit streams will be failed due to no free pipe can be found. Work around: When forth monitor connected, mark video plane as a fake plane, remove it in dc, keep it in dm and report address to OS, until OS turn off MPO. Signed-off-by:
-
Andrew Jiang authored
One of the HDMI ports map to DIGD, which we did not map to DP3 ext HDMI settings. Add the DIGD case so that we can get proper retimer settings from SBIOS for that port. Signed-off-by:
Andrew Jiang <Andrew.Jiang@amd.com> Reviewed-by:
Charlene Liu <Charlene.Liu@amd.com> Reviewed-by:
-
Andrew Jiang authored
There were cases where the chip caps held additional bits that led to the retimer/redriver check failing; use the proper mask to ensure that we have only the bits that we care about for the check in question. Signed-off-by:
-
Andrew Jiang authored
A previous patch set the addressable timing as active + border, when in fact, the VESA standard specifies active as equal to addressable + border. This patch makes the fix more correct and in line with the standard. Signed-off-by:
-
Eric Bernstein authored
Signed-off-by:
Eric Bernstein <eric.bernstein@amd.com> Reviewed-by:
-
Dave Airlie authored
This code isn't used at all in the kernel tree, perhaps it can wait to be imported when it is. It also does a lot of floating point calcs, so probably good to drop it until it's needed and we can ensure proper fpu accessors. Signed-off-by:
-
Harry Wentland authored
Use combo of tabs and spaces Signed-off-by:
-
Harry Wentland authored
Signed-off-by:
Bhawanpreet Lakha <Bhawanpreet.Lakha@amd.com> Acked-by:
-
Andrew Jiang authored
This actually happens quite a bit, and having it as an error causes false positive messages when running tests. Signed-off-by:
-
kbuild test robot authored
drivers/gpu/drm/amd/amdgpu/../display/dc/gpio/gpio_service.c:134:3-8: WARNING: NULL check before freeing functions like kfree, debugfs_remove, debugfs_remove_recursive or usb_free_urb is not needed. Maybe consider reorganizing relevant code to avoid passing NULL values. drivers/gpu/drm/amd/amdgpu/../display/dc/gpio/gpio_service.c:175:4-9: WARNING: NULL check before freeing functions like kfree, debugfs_remove, debugfs_remove_recursive or usb_free_urb is not needed. Maybe consider reorganizing relevant code to avoid passing NULL values. NULL check before some freeing functions is not needed. Based on checkpatch warning "kfree(NULL) is safe this check is probably not required" and kfreeaddr.cocci by Julia Lawall. Generated by: scripts/coccinelle/free/ifnullfree.cocci Fixes: f09cd1f46388 ("drm/amd/display: Use kernel alloc/free") Signed-off-by:Fengguang Wu <fengguang.wu@intel.com> Reviewed-by:
-
kbuild test robot authored
drivers/gpu/drm/amd/amdgpu/../display/dc/gpio/gpio_service.c:134:2-3: Unneeded semicolon Remove unneeded semicolon. Generated by: scripts/coccinelle/misc/semicolon.cocci Fixes: 80be23c57868 ("drm/amd/dc: Add dc display driver (v2)") CC: Harry Wentland <harry.wentland@amd.com> Signed-off-by: -
Harry Wentland authored
v2: convert value to bool using !! Signed-off-by:
-
