Re: [PATCH v2 2/5] staging: lustre: fixed 80-char rule

From: Sudip Mukherjee
Date: Tue Jul 28 2015 - 07:32:26 EST


On Mon, Jul 27, 2015 at 09:20:22PM +0200, Mario Bambagini wrote:
> This patch fixes 2 lines longer than 80 chars
>
> The first one is a function whose argument has been moved to next line.
> The second one is a comment split on two lines
>
> Signed-off-by: Mario Bambagini <mario.bambagini@xxxxxxxxx>
> ---
<snip>
>
> -static inline void update_capa_timer(struct obd_capa *ocapa, unsigned long expiry)
> +static inline void update_capa_timer(struct obd_capa *ocapa,
> + unsigned long expiry)
You are adding a new checkpatch warning here:
"Alignment should match open parenthesis"

regards
sudip
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/