Does use the default value (false) for the Boolean variable of is_apu not enough for identifying DGPU? Anyway, initialize the is_apu during smu early initialize also fine and the patch is
Reviewed-by: Prike Liang <[email protected]> Thanks, Prike > -----Original Message----- > From: Wang, Kevin(Yang) <[email protected]> > Sent: Friday, September 27, 2019 2:58 PM > To: [email protected] > Cc: Huang, Ray <[email protected]>; Feng, Kenneth > <[email protected]>; Liang, Prike <[email protected]>; Wang, > Kevin(Yang) <[email protected]> > Subject: [PATCH] drm/amd/powerplay: initlialize smu->is_apu is false by > default > > the member of is_apu in smu_context need to initlialize by default. > > set default value is false (dGPU) > > for patch: > drm/amd/powerplay: bypass dpm_context null pointer check guard > for some smu series > > Signed-off-by: Kevin Wang <[email protected]> > --- > drivers/gpu/drm/amd/powerplay/amdgpu_smu.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c > b/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c > index 7b995b0834eb..6a64f765fcd4 100644 > --- a/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c > +++ b/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c > @@ -712,6 +712,7 @@ static int smu_early_init(void *handle) > > smu->adev = adev; > smu->pm_enabled = !!amdgpu_dpm; > + smu->is_apu = false; > mutex_init(&smu->mutex); > > return smu_set_funcs(adev); > -- > 2.17.1 _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
