> static void append_badrange_entry(struct badrange *badrange, > - struct badrange_entry *bre, u64 addr, u64 length) > + struct badrange_entry *bre, u64 addr, u64 > length)
Please stop sending this kind of crap. Two tabs are a very common style used in a lot of the kernel, and some people actually prefer it. Instead of arguing what is better just stick to what the surrounding code does. Or in other words: Feel free to be a codingstyle nazi for your code (I am for some of mine), but leave others peoples code alone with "cleanup" patches.