linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: mroberto@cpti.cetuc.puc-rio.br (Marcelo Jimenez)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/2] ARM: pcmcia: fix checkpatch.pl issues in soc_common.c
Date: Fri, 26 Mar 2010 18:57:07 -0300	[thread overview]
Message-ID: <cecb6d8f1003261457x69c319c1w884ade1746f81e8b@mail.gmail.com> (raw)
In-Reply-To: <20100326181800.GB21682@n2100.arm.linux.org.uk>

On Fri, Mar 26, 2010 at 15:18, Russell King - ARM Linux
<linux@arm.linux.org.uk> wrote:
> On Thu, Mar 25, 2010 at 01:19:29PM +0300, Sergei Shtylyov wrote:
>>
>> ? I'd disagree in this case. Anybody will hardly grep for "mask:
>> %s%s%s%s%s%sflags: %s%s%s%s%s%s".
>
> Well, I disagree on a matter of principle with you. ?If you want to avoid
> the line going over 80 columns, move the entire string onto the next line.
> Then we can have the entire string intact _and_ we can have it fitting
> within 80 columns.

This is no big deal. Also, I have already submitted the new patch and
Valdis had a good point for egrep.

Regards,
Marcelo.

  reply	other threads:[~2010-03-26 21:57 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-24 23:04 [PATCH 1/2] ARM: pcmcia: Fix for building DEBUG with sa11xx_base.c as a module Marcelo Roberto Jimenez
2010-03-24 23:04 ` [PATCH 2/2] ARM: pcmcia: fix checkpatch.pl issues in soc_common.c Marcelo Roberto Jimenez
2010-03-24 23:07   ` Russell King - ARM Linux
2010-03-24 23:20     ` Marcelo Jimenez
2010-03-26 15:56       ` Valdis.Kletnieks at vt.edu
2010-03-25 10:19     ` Sergei Shtylyov
2010-03-25 18:26       ` Valdis.Kletnieks at vt.edu
2010-03-26 18:18       ` Russell King - ARM Linux
2010-03-26 21:57         ` Marcelo Jimenez [this message]
2010-03-26 23:06           ` Russell King - ARM Linux
2010-03-24 23:45   ` Marcelo Roberto Jimenez
2010-03-25 17:04     ` Dominik Brodowski
2010-03-25 17:47       ` Marcelo Jimenez
2010-03-28  9:44       ` Russell King - ARM Linux
2010-03-28  9:46         ` Dominik Brodowski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cecb6d8f1003261457x69c319c1w884ade1746f81e8b@mail.gmail.com \
    --to=mroberto@cpti.cetuc.puc-rio.br \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).