[PATCH 4/6] drm/amdgpu/psp: Do not load asd for SRIOV

Yu, Xiangliang Xiangliang.Yu at amd.com
Fri May 5 07:04:31 UTC 2017


> -----Original Message-----
> From: Huang Rui [mailto:ray.huang at amd.com]
> Sent: Friday, May 05, 2017 2:47 PM
> To: Yu, Xiangliang <Xiangliang.Yu at amd.com>
> Cc: amd-gfx at lists.freedesktop.org; Wang, Daniel(Xiaowei)
> <Daniel.Wang2 at amd.com>
> Subject: Re: [PATCH 4/6] drm/amdgpu/psp: Do not load asd for SRIOV
> 
> On Thu, May 04, 2017 at 02:36:35PM +0800, Xiangliang Yu wrote:
> > If psp version doesn't match asd version, asd loading will be failed.
> > Add workaround to bypass it for sriov.
> >
> > Signed-off-by: Daniel Wang <Daniel.Wang2 at amd.com>
> > Signed-off-by: Xiangliang Yu <Xiangliang.Yu at amd.com>
> > ---
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 7 +++++++
> >  1 file changed, 7 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
> > b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
> > index ac5e92e..5041073 100644
> > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
> > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
> > @@ -230,6 +230,13 @@ static int psp_asd_load(struct psp_context *psp)
> >  	int ret;
> >  	struct psp_gfx_cmd_resp *cmd;
> >
> > +	/* If PSP version doesn't match ASD version, asd loading will be failed.
> > +	 * add workaround to bypass it for sriov now.
> > +	 * TODO: add version check to make it common
> > +	 */
> 
> Which psp version? Firmware or boot loader? Can you explain clear?

SRIOV doesn't care about ASD, I'll remove the comments later to avoid confusing.



More information about the amd-gfx mailing list