Am 18.02.2017 um 00:08 schrieb Alex Deucher:
On Fri, Feb 17, 2017 at 3:38 PM, Alex Deucher <[email protected]> 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?

Mhm, unfortunately not really. VCE has some interactions with the display path for frame grabbing, but as far as I know we haven't implemented any of that.

Maybe patch #9 has some relation with that. Does it work when you only leave the VRAM at the location programmed by the BIOS but keep the rest?

Anyway in the meantime patch #1 is Reviewed-by: Christian König <[email protected]> and patches #10 and #11 are Acked-by: Christian König <[email protected]>.

Regards,
Christian.


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

_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx


_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to