Re: [PATCH 0/2] ARM: mvebu: Add support for SPI controller in Armada 370/XP

All of lore.kernel.org
 help / color / mirror / Atom feed

From: Ezequiel Garcia <ezequiel.garcia-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
To: Jason Cooper <jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org>
Cc: Thomas Petazzoni
	<thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	Lior Amsalem <alior-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org>,
	Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>,
	devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
	Gregory Clement
	<gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
	Florian Fainelli
	<florian-p3rKhJxN3npAfugRpC6u6w@public.gmane.org>,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH 0/2] ARM: mvebu: Add support for SPI controller in Armada 370/XP
Date: Mon, 4 Feb 2013 16:29:15 -0300	[thread overview]
Message-ID: <20130204192914.GA2456@localhost> (raw)
In-Reply-To: <20130204183733.GM14746-u4khhh1J0LxI1Ri9qeTfzeTW4wlIGRCZ@public.gmane.org>

Jason,

On Mon, Feb 04, 2013 at 01:37:33PM -0500, Jason Cooper wrote:
> This series looks good.  just a few comments:
> 
> On Mon, Feb 04, 2013 at 01:51:20PM -0300, Ezequiel Garcia wrote:
> > Hi,
> > 
> > This patchset adds support for the SPI controller
> > available in Armada 370 and Armada XP SoC.
> > 
> > The patches are based in Jason Cooper's mvebu/dt branch.
> 
> It probably doesn't matter for this series (I haven't tried to apply it
> yet), but please base future work off of a mainline tag (eg v3.8-rc6).
> It makes merge conflicts easier to resolve.
> 
> Feel free to test-merge against the mvebu/ branches and let me know of
> any conflicts.
> 

Ah, I see.

I was mistakenly assuming you wanted against that branch;
I'll base the patches on mainline in the future.

Thanks for the advice.

> Also, your patch series appears to be out of order, did you use git
> send-email once per file?
> 

I always send the cover letter manually. This time I did it after the patches.
I'm in the process of automating this in my workflow,
but feel free to make a suggestion.

Thanks,

-- 
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com

------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_jan
_______________________________________________
spi-devel-general mailing list
spi-devel-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/spi-devel-general

WARNING: multiple messages have this Message-ID (diff)

From: ezequiel.garcia@free-electrons.com (Ezequiel Garcia)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/2] ARM: mvebu: Add support for SPI controller in Armada 370/XP
Date: Mon, 4 Feb 2013 16:29:15 -0300	[thread overview]
Message-ID: <20130204192914.GA2456@localhost> (raw)
In-Reply-To: <20130204183733.GM14746@titan.lakedaemon.net>

Jason,

On Mon, Feb 04, 2013 at 01:37:33PM -0500, Jason Cooper wrote:
> This series looks good.  just a few comments:
> 
> On Mon, Feb 04, 2013 at 01:51:20PM -0300, Ezequiel Garcia wrote:
> > Hi,
> > 
> > This patchset adds support for the SPI controller
> > available in Armada 370 and Armada XP SoC.
> > 
> > The patches are based in Jason Cooper's mvebu/dt branch.
> 
> It probably doesn't matter for this series (I haven't tried to apply it
> yet), but please base future work off of a mainline tag (eg v3.8-rc6).
> It makes merge conflicts easier to resolve.
> 
> Feel free to test-merge against the mvebu/ branches and let me know of
> any conflicts.
> 

Ah, I see.

I was mistakenly assuming you wanted against that branch;
I'll base the patches on mainline in the future.

Thanks for the advice.

> Also, your patch series appears to be out of order, did you use git
> send-email once per file?
> 

I always send the cover letter manually. This time I did it after the patches.
I'm in the process of automating this in my workflow,
but feel free to make a suggestion.

Thanks,

-- 
Ezequiel Garc?a, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com

next prev parent reply	other threads:[~2013-02-04 19:29 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-04 16:38 [PATCH 1/2] ARM: mvebu: Add support for SPI controller in Armada 370/XP Ezequiel Garcia
2013-02-04 16:38 ` Ezequiel Garcia
2013-02-04 16:38 ` Ezequiel Garcia
     [not found] ` <1359995888-2385-1-git-send-email-ezequiel.garcia-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2013-02-04 16:38   ` [PATCH 2/2] ARM: mvebu: Update defconfig to select SPI support Ezequiel Garcia
2013-02-04 16:38     ` Ezequiel Garcia
2013-02-04 16:38     ` Ezequiel Garcia
     [not found]     ` <1359995888-2385-2-git-send-email-ezequiel.garcia-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2013-02-05 15:40       ` Gregory CLEMENT
2013-02-05 15:40         ` Gregory CLEMENT
2013-02-04 16:51   ` [PATCH 0/2] ARM: mvebu: Add support for SPI controller in Armada 370/XP Ezequiel Garcia
2013-02-04 16:51     ` Ezequiel Garcia
2013-02-04 18:37     ` Jason Cooper
2013-02-04 18:37       ` Jason Cooper
     [not found]       ` <20130204183733.GM14746-u4khhh1J0LxI1Ri9qeTfzeTW4wlIGRCZ@public.gmane.org>
2013-02-04 19:29         ` Ezequiel Garcia [this message]
2013-02-04 19:29           ` Ezequiel Garcia
2013-02-04 19:47           ` Jason Cooper
2013-02-04 19:47             ` Jason Cooper
     [not found]             ` <20130204194714.GN14746-u4khhh1J0LxI1Ri9qeTfzeTW4wlIGRCZ@public.gmane.org>
2013-02-04 20:01               ` Ezequiel Garcia
2013-02-04 20:01                 ` Ezequiel Garcia
2013-02-04 19:03     ` Andrew Lunn
2013-02-04 19:03       ` Andrew Lunn
     [not found]       ` <20130204190326.GA20212-g2DYL2Zd6BY@public.gmane.org>
2013-02-04 19:33         ` Ezequiel Garcia
2013-02-04 19:33           ` Ezequiel Garcia
2013-02-05 13:57   ` [PATCH 1/2] " Gregory CLEMENT
2013-02-05 13:57     ` Gregory CLEMENT

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=20130204192914.GA2456@localhost \
    --to=ezequiel.garcia-wi1+55scjutkeb57/3fjtnbpr1lh4cv8@public.gmane.org \
    --cc=alior-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org \
    --cc=andrew-g2DYL2Zd6BY@public.gmane.org \
    --cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
    --cc=florian-p3rKhJxN3npAfugRpC6u6w@public.gmane.org \
    --cc=gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
    --cc=jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    --cc=thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.