On Tue, Mar 07, 2017 at 10:05:00AM -0600, Tom Lendacky wrote: > > And then you need to correct the function signature in the > > !CONFIG_AMD_MEM_ENCRYPT case, at the end of this file, too: > > > > unsigned long __init sme_enable(struct boot_params *bp) { return 0; } > > Yup, missed that. I'll make it match. Or, you can do this: unsigned long __init sme_enable(void *boot_data) { #ifdef CONFIG_AMD_MEM_ENCRYPT struct boot_params *bp = boot_data; unsigned int eax, ebx, ecx, edx; unsigned long cmdline_ptr; ... out: #endif /* CONFIG_AMD_MEM_ENCRYPT */ return sme_me_mask; } and never worry for function headers going out of whack. -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply.