[Public]

smu_get_power_profile_mode actually calls smu->ppt_funcs->get_power_profile_mode

So shouldn't it be sufficient to do this?

        } else if (DEVICE_ATTR_IS(pp_power_profile_mode)) {
                if (!adev->smu.ppt_funcs->get_power_profile_mode)
                        *states = ATTR_STATE_UNSUPPORTED;
        }


From: Lazar, Lijo <[email protected]>
Sent: Saturday, October 30, 2021 10:06
To: Alex Deucher <[email protected]>
Cc: Limonciello, Mario <[email protected]>; 
[email protected]; Deucher, Alexander <[email protected]>
Subject: Re: [PATCH] drm/amdgpu/pm: Don't show pp_power_profile_mode for YC and 
later APUs


[Public]

For swsmu, we could add a helper macro like smu_is_supported(x) that checks if 
ppt func is not NULL.

Thanks,
Lijo

Reply via email to