lkml.org 
[lkml]   [2015]   [Jan]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [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



\
 
 \ /
  Last update: 2015-01-20 11:41    [W:0.055 / U:0.184 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site