Messages in this thread | | | Date | Wed, 9 Dec 2015 20:09:04 +0100 | From | Borislav Petkov <> | Subject | Re: [PATCH 02/12] selftests/x86: Add check_initial_reg_state |
| |
On Wed, Dec 09, 2015 at 10:56:36AM -0800, Andy Lutomirski wrote: > That was intentional -- everything lines up with the top-level "asm(". > checkpatch is confused because it understands "\t " at the front of a > line but not just a space. > > That being said, I could easily be convinced to switch to tabs there.
I'd say kernel coding style is tabs of 8 chars...
You could do
asm(".pushsection .text\n\t" ".type real_start, @function\n\t" ".global real_start\n\t" "real_start:\n\t" #ifdef __x86_64__ "mov %rax, ax\n\t" "mov %rbx, bx\n\t" ...
and align the opening "asm(" to the first tab...
Bah, whatever, I'm not that finicky to care enough.
-- Regards/Gruss, Boris.
ECO tip #101: Trim your mails when you reply.
| |