From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp-out-101.synserver.de ([212.40.185.101]:1070 "EHLO smtp-out-101.synserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932208AbaJWKuC (ORCPT ); Thu, 23 Oct 2014 06:50:02 -0400 Message-ID: <5448DD58.7030705@metafoo.de> Date: Thu, 23 Oct 2014 12:50:00 +0200 From: Lars-Peter Clausen MIME-Version: 1.0 To: Sudip Mukherjee , Brian Vandre CC: gregkh@linuxfoundation.org, jic23@kernel.org, alexandre.belloni@free-electrons.com, linux-iio@vger.kernel.org, marex@denx.de, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] Staging: iio: adc: fix line over 80 characters References: <20141022042115.GA2530@sudip-PC> In-Reply-To: <20141022042115.GA2530@sudip-PC> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 10/22/2014 06:21 AM, Sudip Mukherjee wrote: > On Tue, Oct 21, 2014 at 05:56:47PM -0500, Brian Vandre wrote: >> This fixes the 2 checkpatch.pl warnings: >> WARNING: line over 80 characters >> > please check your patch with --strict option of checkpatch.pl , and you will get : > "Alignment should match open parenthesis" . Those checkpatch warnings are suggestions, not hard requirements. The idea is to improve code legibility, but if the change has the adverse effect the warning can and should be ignored. Also when making a change you should keep the existing indention style of a file. - Lars