Re: [PATCH v4] x86/boot: Don't return encryption mask from __startup_64()

[Khalid Ali <khaliidcaliy@gmail.com>]

> Sigh... this comment is broken, since there's no '*' before the last '/'.

>Here's how I find it:
>
>So I went to see why this patch caused early boot failure, since the code
>change doesn't seem wrong to me.
>
>After staring at the code for half hour and yet unable to see any issue, I
>went to disassemble the kernel image, then I found the above two lines of
>code wasn't there at all.
>
>Then looking at this again, it's obvious that the reason is the change to
>the comment is broken, leading the above two lines of code being commented
>out.

So what i was wringling all along was the comment. I don't know why i couldn't spot it
if those two lines where commented out then even boot was supposed to fail on my side too.
I guess toolchain difference. Probably this is the reason since i compiled with gcc not clang.

Thanks though. :)

>And this patch has other style issues too like the broken indent of function
>parameters after changing returning value from 'unsigned int' to 'void', and
Could please clarify more as checkpatch.pl didn't notice it.

>So, please, before posting patches, test them, and run
>./scripts/checkpatch.pl against them.

I used it, well the tool seems broken and giving me false positive. I fixed all your suggestions and sent 
v5, as my best i could, i fixed all issues and that patch seems good. 

Could you please check it?