drm/i915/huc: drop intel_huc_is_authenticated

The function name is confusing, because it doesn't check the actual auth
status in HW but the SW status. Given that there is only one user (the
huc_auth function itself), just get rid of it and use the FW status
checker directly.
Signed-off-by: default avatarDaniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Reviewed-by: default avatarAlan Previn <alan.previn.teres.alexis@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220504204816.2082588-2-daniele.ceraolospurio@intel.com
parent a5c89f7c
...@@ -96,7 +96,7 @@ int intel_huc_auth(struct intel_huc *huc) ...@@ -96,7 +96,7 @@ int intel_huc_auth(struct intel_huc *huc)
struct intel_guc *guc = &gt->uc.guc; struct intel_guc *guc = &gt->uc.guc;
int ret; int ret;
GEM_BUG_ON(intel_huc_is_authenticated(huc)); GEM_BUG_ON(intel_uc_fw_is_running(&huc->fw));
if (!intel_uc_fw_is_loaded(&huc->fw)) if (!intel_uc_fw_is_loaded(&huc->fw))
return -ENOEXEC; return -ENOEXEC;
......
...@@ -50,11 +50,6 @@ static inline bool intel_huc_is_used(struct intel_huc *huc) ...@@ -50,11 +50,6 @@ static inline bool intel_huc_is_used(struct intel_huc *huc)
return intel_uc_fw_is_available(&huc->fw); return intel_uc_fw_is_available(&huc->fw);
} }
static inline bool intel_huc_is_authenticated(struct intel_huc *huc)
{
return intel_uc_fw_is_running(&huc->fw);
}
void intel_huc_load_status(struct intel_huc *huc, struct drm_printer *p); void intel_huc_load_status(struct intel_huc *huc, struct drm_printer *p);
#endif #endif
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment