On Tue, Feb 3, 2026 at 11:12 PM Lazar, Lijo <[email protected]> wrote:
>
>
>
> On 04-Feb-26 8:36 AM, Pratik Vishwakarma wrote:
> > Signed-off-by: Pratik Vishwakarma <[email protected]>
> > ---
> > drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c | 5 ++++-
> > 1 file changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
> > b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
> > index 9f52b7b24198..fc68dffdea98 100644
> > --- a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
> > +++ b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
> > @@ -1909,7 +1909,10 @@ static int smu_hw_init(struct amdgpu_ip_block
> > *ip_block)
> > smu->pm_enabled = false;
> > return 0;
> > }
> > -
> > + if (adev->firmware.load_type != AMDGPU_FW_LOAD_PSP) {
> > + dev_info(adev->dev, "Do minimal SMU init for backdoor loading
> > case\n");
> > + return 0;
> > + }
>
> This doesn't hold good for all SOCs.
Yeah, I would drop this one.
Alex
>
> Thanks,
> Lijo
>
> > ret = smu_start_smc_engine(smu);
> > if (ret) {
> > dev_err(adev->dev, "SMC engine is not correctly up!\n");
>