just a couple of typos in the patch title: drm\amdgpu: [DALI] Dali Varient Detection It should be: drm/amdgpu: [DALI] Dali Variant Detection With that fixed, patch is: Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx> On Wed, Feb 5, 2020 at 9:59 AM Tawfik, Aly <Aly.Tawfik@xxxxxxx> wrote: > > Hi, > > > > Dali is a raven2 based asic that drives at a lower (6W) TDP than other raven 2 chips. Currently the fused internal id is the same on all raven 2 boards, this means that the detection process must be done through PCIE REV ID. > > Unfortunately PCIE REV ID is not defined inside the scope of display. I created a patch to alter the fused value for internal rev_id if the chip is detected as dali through PCIE REV ID. So that detection of the chip will be possible inside of Display core. > > > > Can you kindly provide feedback on this workaround. > > > > Best Regards, > > Aly Tawfik > > > > _______________________________________________ > amd-gfx mailing list > amd-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx