linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: robert.jarzmik@free.fr (Robert Jarzmik)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL RESEND] pxa for v3.20
Date: Sat, 24 Jan 2015 15:18:11 +0100	[thread overview]
Message-ID: <874mrgdzz0.fsf@free.fr> (raw)
In-Reply-To: <20150123215423.GD20896@quad.lixom.net> (Olof Johansson's message of "Fri, 23 Jan 2015 13:54:23 -0800")

Olof Johansson <olof@lixom.net> writes:

> Hi Robert,
>
> Branch looks good, I'll merge this into next/fixes-non-critical. One nit below
> though:
>
>> ----------------------------------------------------------------
>> Dmitry Eremin-Solenikov (3):
>>       ARM: pxa: add regulator_has_full_constraints to corgi board file
>>       ARM: pxa: add regulator_has_full_constraints to poodle board file
>>       ARM: pxa: add regulator_has_full_constraints to spitz board file
>> 
>> Martin Vajnar (1):
>>       hx4700: regulator: declare full constraints
>
> Please use consistent patch subjects in the future. We've ended up using ARM:
> <platform>: <foo> across all platforms these days.
>
> I'm not going to ask you to respin just for this, but please remember to fix
> future patches when you apply them if they're not already correct.
You're right, I missed that one.

In case you've not already pulled, I created :
   https://github.com/rjarzmik/linux.git tags/pxa-for-3.20-bis

I can resend a proper pull request if you wish, or we can leave it as is if you
already pulled. In any case I'll be more carefull next time, maybe I can
convince my bot to check that for me too.

Or even better checkpatch could check that any patch with a diffstat fully in
arch/arm/{mach,plat}-<XYZ> has a subject beginning with "ARM: XYZ: ", that would
be cool, and put the burden on others as well :)

Cheers.

-- 
Robert

  reply	other threads:[~2015-01-24 14:18 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-20 19:01 [GIT PULL RESEND] pxa for v3.20 Robert Jarzmik
2015-01-23 21:54 ` Olof Johansson
2015-01-24 14:18   ` Robert Jarzmik [this message]
2015-01-27  2:17     ` Olof Johansson
2015-01-27  4:42       ` Robert Jarzmik
2015-01-27  4:45         ` Olof Johansson

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=874mrgdzz0.fsf@free.fr \
    --to=robert.jarzmik@free.fr \
    --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).