diff options
author | 2012-03-08 19:12:11 -0500 | |
---|---|---|
committer | 2012-03-09 12:47:56 -0800 | |
commit | dfe42443ea1d30c98db59b7b1f914bc147d31336 (patch) | |
tree | 479554def3fb2d90592f229e47f06e354aac1e35 /mm/page_alloc.c | |
parent | serial: delete useless void casts in 8250.c (diff) | |
download | linux-dev-dfe42443ea1d30c98db59b7b1f914bc147d31336.tar.xz linux-dev-dfe42443ea1d30c98db59b7b1f914bc147d31336.zip |
serial: reduce number of indirections in 8250 code
The serial_8250_port struct contains within a serial_port struct
and many times one or the other, or both are in scope within
functions via a passed in arg, or via container_of.
However there are a lot of cases where we have access directly
to the port pointer, but yet go through the parent 8250_port
structure instead to get it. These should just use the port
struct directly.
Similarly there are cases where it makes sense (from a code
cleanliness point of view) to declare a local for the port
struct, so we aren't going through the parent 8250_port struct
repeatedly to get to it.
We get a small reduction in text size, but it appears that
gcc was smart enough to internally be doing most of this
already, so the readability improvement is the larger gain.
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'mm/page_alloc.c')
0 files changed, 0 insertions, 0 deletions