Reviewed-by: Xiangliang Yu <[email protected]>
> -----Original Message----- > From: amd-gfx [mailto:[email protected]] On Behalf > Of Alex Deucher > Sent: Tuesday, December 19, 2017 11:07 PM > To: [email protected] > Cc: Deucher, Alexander <[email protected]> > Subject: [PATCH 2/2] drm/amdgpu: use defines for > mmBIF_IOV_FUNC_IDENTIFIER fields > > Rather than magic numbers. > > Signed-off-by: Alex Deucher <[email protected]> > --- > drivers/gpu/drm/amd/amdgpu/vi.c | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/vi.c > b/drivers/gpu/drm/amd/amdgpu/vi.c index c59b37365e4d..da2b99c2d95f > 100644 > --- a/drivers/gpu/drm/amd/amdgpu/vi.c > +++ b/drivers/gpu/drm/amd/amdgpu/vi.c > @@ -455,11 +455,10 @@ static void vi_detect_hw_virtualization(struct > amdgpu_device *adev) > adev->asic_type == CHIP_FIJI) { > reg = RREG32(mmBIF_IOV_FUNC_IDENTIFIER); > /* bit0: 0 means pf and 1 means vf */ > - /* bit31: 0 means disable IOV and 1 means enable */ > - if (reg & 1) > + if (REG_GET_FIELD(reg, BIF_IOV_FUNC_IDENTIFIER, > +FUNC_IDENTIFIER)) > adev->virt.caps |= AMDGPU_SRIOV_CAPS_IS_VF; > - > - if (reg & 0x80000000) > + /* bit31: 0 means disable IOV and 1 means enable */ > + if (REG_GET_FIELD(reg, BIF_IOV_FUNC_IDENTIFIER, > IOV_ENABLE)) > adev->virt.caps |= AMDGPU_SRIOV_CAPS_ENABLE_IOV; > } > > -- > 2.13.6 > > _______________________________________________ > 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
