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 <Prike.Liang@xxxxxxx> Thanks, Prike > -----Original Message----- > From: Wang, Kevin(Yang) <Kevin1.Wang@xxxxxxx> > Sent: Friday, September 27, 2019 2:58 PM > To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Huang, Ray <Ray.Huang@xxxxxxx>; Feng, Kenneth > <Kenneth.Feng@xxxxxxx>; Liang, Prike <Prike.Liang@xxxxxxx>; Wang, > Kevin(Yang) <Kevin1.Wang@xxxxxxx> > 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 <kevin1.wang@xxxxxxx> > --- > 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 amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx
