Thread (2 messages) 2 messages, 2 authors, 2008-03-06

Re: [PATCH 1/8] pxa: un-nest pxafb_parse_options() to cleanup the coding style issue

From: Andrew Morton <akpm@linux-foundation.org>
Date: 2008-03-06 07:14:23

On Wed, 27 Feb 2008 10:33:10 +0800 "eric miao" [off-list ref] wrote:
quoted
From e946f577d3f869abcc587629e2fdd5f1563ce1df Mon Sep 17 00:00:00 2001
From: eric miao <redacted>
Date: Fri, 22 Feb 2008 11:29:08 +0800
Subject: [PATCH] pxa: un-nest pxafb_parse_options() to cleanup the
coding style issue

pxafb_parse_options() has very long lines exceeding far beyond 80
characters, which makes the function looks bad.  Un-nest it into
smaller functions and use a temporary string for only what has been
overridden instead of the whole dev_info() message to reduce the
line a bit more.
These patches were all badly wordwrapped.  Please refresh them against the
latest mainline kernel, fix your email client and then resend them, being
sure to cc myself.

Thanks.


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help