Messages in this thread | | | Date | Tue, 20 Jan 2015 11:29:43 +0100 (CET) | From | Thomas Gleixner <> | Subject | Re: [PATCH 0/2] x86: earlyprintk cleanup |
| |
On Mon, 19 Jan 2015, Alexander Kuleshov wrote:
> The following two patches remove unused macro definitions from > early_serial_console and early_printk x86 code.
Sigh. These mechanical 'follow the output of some code analyzer' changes are really horrible.
Why not doing the obvious: Move the duplicated defines into a common header file and be done with it?
And if you look deeper, then you'll notice that those defines are used in two other files as well.
What's worse is that the whole set of defines is readily available in include/uapi/linux/serial_reg.h already (just with different symbols).
So what's the point of these removal patches?
Thanks,
tglx
| |