x86/microcode/AMD: Have __apply_microcode_amd() return bool

This is the natural thing to do anyway.

No functional changes.

Signed-off-by: Borislav Petkov (AMD) <bp@alien8.de>

diff --git a/arch/x86/kernel/cpu/microcode/amd.c b/arch/x86/kernel/cpu/microcode/amd.c
index 9a5ebbbc65423a6e3d662c5d9f7a631b96bcfd93..ac3fd07e36ac45ec99432b9d679f86501c8e4851 100644 (file)
--- a/arch/x86/kernel/cpu/microcode/amd.c
+++ b/arch/x86/kernel/cpu/microcode/amd.c
@@ -484,7 +484,7 @@ static void scan_containers(u8 *ucode, size_t size, struct cont_desc *desc)
        }
 }
 
-static int __apply_microcode_amd(struct microcode_amd *mc, unsigned int psize)
+static bool __apply_microcode_amd(struct microcode_amd *mc, unsigned int psize)
 {
        unsigned long p_addr = (unsigned long)&mc->hdr.data_code;
        u32 rev, dummy;
@@ -508,9 +508,9 @@ static int __apply_microcode_amd(struct microcode_amd *mc, unsigned int psize)
        native_rdmsr(MSR_AMD64_PATCH_LEVEL, rev, dummy);
 
        if (rev != mc->hdr.patch_id)
-               return -1;
+               return false;
 
-       return 0;
+       return true;
 }
 
 /*
@@ -544,7 +544,7 @@ static bool early_apply_microcode(u32 old_rev, void *ucode, size_t size)
        if (old_rev > mc->hdr.patch_id)
                return ret;
 
-       return !__apply_microcode_amd(mc, desc.psize);
+       return __apply_microcode_amd(mc, desc.psize);
 }
 
 static bool get_builtin_microcode(struct cpio_data *cp)
@@ -763,7 +763,7 @@ void reload_ucode_amd(unsigned int cpu)
        rdmsr(MSR_AMD64_PATCH_LEVEL, rev, dummy);
 
        if (rev < mc->hdr.patch_id) {
-               if (!__apply_microcode_amd(mc, p->size))
+               if (__apply_microcode_amd(mc, p->size))
                        pr_info_once("reload revision: 0x%08x\n", mc->hdr.patch_id);
        }
 }
@@ -816,7 +816,7 @@ static enum ucode_state apply_microcode_amd(int cpu)
                goto out;
        }
 
-       if (__apply_microcode_amd(mc_amd, p->size)) {
+       if (!__apply_microcode_amd(mc_amd, p->size)) {
                pr_err("CPU%d: update failed for patch_level=0x%08x\n",
                        cpu, mc_amd->hdr.patch_id);
                return UCODE_ERROR;