On Tue, Jul 26, 2016 at 11:38 AM, Tom St Denis <tstdeni...@gmail.com> wrote: > Enables GFX PG and UVD static PG on Carrizo class > asics. > > Signed-off-by: Tom St Denis <tom.stde...@amd.com> > --- > drivers/gpu/drm/amd/amdgpu/vi.c | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/drivers/gpu/drm/amd/amdgpu/vi.c b/drivers/gpu/drm/amd/amdgpu/vi.c > index 9ba64989f092..289a4c4ac9e7 100644 > --- a/drivers/gpu/drm/amd/amdgpu/vi.c > +++ b/drivers/gpu/drm/amd/amdgpu/vi.c > @@ -1578,7 +1578,16 @@ static int vi_common_early_init(void *handle) > AMD_CG_SUPPORT_HDP_LS | > AMD_CG_SUPPORT_SDMA_MGCG | > AMD_CG_SUPPORT_SDMA_LS; > + /* rev0 hardware doesn't support PG */
It's probably worth correcting this comment. A0 theoretically supports PG, but probably requires additional workarounds. Something like "Don't enable PG on rev0 hw due to hw issues" or something like that. > adev->pg_flags = 0; > + if (adev->rev_id != 0x00) { > + adev->pg_flags |= AMD_PG_SUPPORT_GFX_PG | > + AMD_PG_SUPPORT_GFX_SMG | > + AMD_PG_SUPPORT_GFX_PIPELINE; > + /* powerplay UVD PG doesn't work yet */ > + if (!amdgpu_powerplay) > + adev->pg_flags |= AMD_PG_SUPPORT_UVD; It's probably worth testing UVD PG on rev0 hw as well so see if we can enable this on them as well. OTOH, there isn't likely much rev0 hw in the wild, so it may not matter too much. Alex > + } > adev->external_rev_id = adev->rev_id + 0x1; > break; > case CHIP_STONEY: > -- > 2.9.2 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx