From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============7110053801299823618==" MIME-Version: 1.0 From: Alexandra Yates Subject: Re: [Powertop] [PATCH v2 00/14] pull: various changes, mostly build system related Date: Tue, 18 Nov 2014 10:08:47 -0800 Message-ID: <50055.10.24.5.249.1416334127.squirrel@linux.intel.com> In-Reply-To: CAG27Bk06CL1bLu+gze-t96xRTq3ZE+Hdguc7i3ZO3PeDZfc8mQ@mail.gmail.com To: powertop@lists.01.org List-ID: --===============7110053801299823618== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable > On 17 November 2014 04:45, Alexandra Yates > wrote: >> When I run checkpatch.pl I get white spaces and over 80 char line errors >> on your patches. > > Hi Alexandra, > > You are right, I missed in the change 0004 'csstoh.sh: check every > single return value' a white space error. That is now fixed in: > > https://github.com/kerolasa/powertop/commit/6d7e331488db1e142892557943124= 2b8fb1c6007 > > 80 char line errors are strange. I do not see them. Are we using the > same version of the checkpatch.pl? > > $ git describe > v3.18-rc5 > $ shasum scripts/checkpatch.pl > a13cad76323e71c97ba279838feef9730509fc59 scripts/checkpatch.pl > > -- > Sami Kerola > http://www.iki.fi/kerolasa/ > I'm using the tip of the branch from git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git Thank you, Alexandra. --===============7110053801299823618==--