[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20150304052217.GA4336@sudip-PC>
Date: Wed, 4 Mar 2015 10:52:17 +0530
From: Sudip Mukherjee <sudipm.mukherjee@...il.com>
To: Cláudio Maia <crrm@...p.ipp.pt>
Cc: thomas.petazzoni@...e-electrons.com, noralf@...nnes.org,
gregkh@...uxfoundation.org, devel@...verdev.osuosl.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] Staging: fbtft: fb_pcd8544: Fixed coding style
errors and warnings
On Tue, Mar 03, 2015 at 11:47:18PM +0000, Cláudio Maia wrote:
> Fixed two warnings occurring in lines that are over 80 characters and which are not comments.
commit message should not be more than 72 characters.
>
> Signed-off-by: Cláudio Maia <crrm@...p.ipp.pt>
> ---
<snip>
> static void set_addr_win(struct fbtft_par *par, int xs, int ys, int xe, int ye)
> {
> - fbtft_par_dbg(DEBUG_SET_ADDR_WIN, par, "%s(xs=%d, ys=%d, xe=%d, ye=%d)\n", __func__, xs, ys, xe, ye);
> + fbtft_par_dbg(DEBUG_SET_ADDR_WIN, par, "%s(xs=%d, ys=%d, xe=%d, ye=%d)\n",
> + __func__, xs, ys, xe, ye);
Alignment should match open parenthesis
regards
sudip
>
> /* H=0 Set X address of RAM */
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists