On Fri, Feb 17, 2017 at 3:38 PM, Alex Deucher <alexdeucher at gmail.com> wrote: > Since evergreen, the vbios has programmed the FB_LOCATION to the proper size > during asic_init, so there is no need to reprogram them in the driver. > We can safely leave the location as set by the vbios. This simplifies the > driver significantly. Tested this on CI and VI. Works fine with DAL/DC. With the legacy display path, VCE fails to initialize (ECPU won't come out of reset). Any ideas? Alex > > Alex Deucher (11): > drm/amdgpu/vce2: fix vce bar programming > drm/amdgpu: put gtt at 0 in the internal address space > drm/amdgpu/gmc8: use the vram location programmed by the vbios > drm/amdgpu/gmc7: use the vram location programmed by the vbios > drm/amdgpu/gmc6: use the vram location programmed by the vbios > drm/amdgpu/gmc8: drop fb location programming > drm/amdgpu/gmc7: drop fb location programming > drm/amdgpu/gmc6: drop fb location programming > drm/amdgpu: drop set_vga_render_state from display funcs (v2) > drm/amdgpu: remove *_mc_access from display funcs (v2) > drm/amd/dc/dm: remove redundant display structs > > drivers/gpu/drm/amd/amdgpu/amdgpu.h | 4 - > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 8 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h | 12 -- > drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 135 +--------------------- > drivers/gpu/drm/amd/amdgpu/dce_v10_0.h | 6 - > drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 80 +------------ > drivers/gpu/drm/amd/amdgpu/dce_v11_0.h | 6 - > drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | 115 ------------------ > drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 82 +------------ > drivers/gpu/drm/amd/amdgpu/dce_v8_0.h | 6 - > drivers/gpu/drm/amd/amdgpu/dce_virtual.c | 97 +++++++--------- > drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c | 38 ++---- > drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c | 35 ++---- > drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c | 34 ++---- > drivers/gpu/drm/amd/amdgpu/vce_v2_0.c | 17 +-- > drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 75 +----------- > 16 files changed, 83 insertions(+), 667 deletions(-) > > -- > 2.5.5 >