All of lore.kernel.org
 help / color / mirror / Atom feed
From: Shawn Guo <shawn.guo@freescale.com>
To: Tejun Heo <tj@kernel.org>
Cc: Russell King - ARM Linux <linux@arm.linux.org.uk>,
	linux-ide@vger.kernel.org, devicetree@vger.kernel.org,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	Kumar Gala <galak@codeaurora.org>,
	linux-arm-kernel@lists.infradead.org, linux-doc@vger.kernel.org,
	Mark Rutland <mark.rutland@arm.com>,
	Pawel Moll <pawel.moll@arm.com>,
	Randy Dunlap <rdunlap@infradead.org>,
	Rob Herring <robh+dt@kernel.org>,
	Sascha Hauer <kernel@pengutronix.de>
Subject: Re: [PATCH 0/8] iMX6 SATA configuration
Date: Wed, 25 Jun 2014 23:29:14 +0800	[thread overview]
Message-ID: <20140625152912.GI2224@dragon> (raw)
In-Reply-To: <20140624220958.GI14909@htj.dyndns.org>

On Tue, Jun 24, 2014 at 06:09:58PM -0400, Tejun Heo wrote:
> On Tue, Jun 24, 2014 at 11:19:01AM +0100, Russell King - ARM Linux wrote:
> > Another round of these patches.  I've integrated Shawn's patch to split
> > the documentation into this series so I can add the DT documentation
> > on top of it.
> > 
> >  .../devicetree/bindings/ata/ahci-platform.txt      |   6 -
> >  Documentation/devicetree/bindings/ata/imx-sata.txt |  36 ++++
> >  arch/arm/boot/dts/imx6q-cubox-i.dts                |   4 +
> >  drivers/ata/ahci_imx.c                             | 184 ++++++++++++++++++++-
> >  4 files changed, 216 insertions(+), 14 deletions(-)
> 
> Patches look good to me.  Shawn, can you please review the ahci_imx
> portion?

Reviewed-by: Shawn Guo <shawn.guo@freescale.com>

I also tested the series on imx6q-sabresd board and nothing breaks
regarding SATA support on the board.

> How should the patches be routed?  Should I pick only the
> ata parts or apply the whole thing?

I will apply the last two dts patches via IMX tree, and please apply
the first 6 patches through your tree.

Shawn

WARNING: multiple messages have this Message-ID (diff)
From: shawn.guo@freescale.com (Shawn Guo)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/8] iMX6 SATA configuration
Date: Wed, 25 Jun 2014 23:29:14 +0800	[thread overview]
Message-ID: <20140625152912.GI2224@dragon> (raw)
In-Reply-To: <20140624220958.GI14909@htj.dyndns.org>

On Tue, Jun 24, 2014 at 06:09:58PM -0400, Tejun Heo wrote:
> On Tue, Jun 24, 2014 at 11:19:01AM +0100, Russell King - ARM Linux wrote:
> > Another round of these patches.  I've integrated Shawn's patch to split
> > the documentation into this series so I can add the DT documentation
> > on top of it.
> > 
> >  .../devicetree/bindings/ata/ahci-platform.txt      |   6 -
> >  Documentation/devicetree/bindings/ata/imx-sata.txt |  36 ++++
> >  arch/arm/boot/dts/imx6q-cubox-i.dts                |   4 +
> >  drivers/ata/ahci_imx.c                             | 184 ++++++++++++++++++++-
> >  4 files changed, 216 insertions(+), 14 deletions(-)
> 
> Patches look good to me.  Shawn, can you please review the ahci_imx
> portion?

Reviewed-by: Shawn Guo <shawn.guo@freescale.com>

I also tested the series on imx6q-sabresd board and nothing breaks
regarding SATA support on the board.

> How should the patches be routed?  Should I pick only the
> ata parts or apply the whole thing?

I will apply the last two dts patches via IMX tree, and please apply
the first 6 patches through your tree.

Shawn

  reply	other threads:[~2014-06-25 15:29 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-24 10:19 [PATCH 0/8] iMX6 SATA configuration Russell King - ARM Linux
2014-06-24 10:19 ` Russell King - ARM Linux
2014-06-24 10:19 ` [PATCH v3 1/8] dt-bindings: ata: create bindings for imx sata controller Russell King
2014-06-24 10:19   ` Russell King
2014-06-24 10:19 ` [PATCH v3 2/8] ata: ahci_imx: warn when disabling ahci link Russell King
2014-06-24 10:19   ` Russell King
2014-06-24 22:05   ` Tejun Heo
2014-06-24 22:05     ` Tejun Heo
2014-06-24 10:19 ` [PATCH v3 3/8] ata: ahci_imx: allow hardware parameters to be specified in DT Russell King
2014-06-24 10:19   ` Russell King
2014-06-24 10:19 ` [PATCH v3 5/8] ata: ahci_imx: add disable for spread-spectrum Russell King
2014-06-24 10:19   ` Russell King
2014-06-24 10:19 ` [PATCH v3 6/8] dt-bindings: ata: document ability to disable spread-spectrum clock Russell King
2014-06-24 10:19   ` Russell King
     [not found] ` <20140624101901.GL32514-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2014-06-24 10:19   ` [PATCH v3 4/8] dt-bindings: ata: add ahci_imx electrical properties Russell King
2014-06-24 10:19     ` Russell King
2014-06-24 10:19   ` [PATCH v3 7/8] ARM: dts: cubox-i: add eSATA DT configuration Russell King
2014-06-24 10:19     ` Russell King
2014-06-25 15:32     ` Shawn Guo
2014-06-25 15:32       ` Shawn Guo
2014-06-24 10:20 ` [PATCH v3 8/8] ARM: dts: cubox-i: disable spread-spectrum for Cubox-i eSATA Russell King
2014-06-24 10:20   ` Russell King
2014-06-24 12:45 ` [PATCH 0/8] iMX6 SATA configuration Rob Herring
2014-06-24 12:45   ` Rob Herring
2014-06-24 22:09 ` Tejun Heo
2014-06-24 22:09   ` Tejun Heo
2014-06-25 15:29   ` Shawn Guo [this message]
2014-06-25 15:29     ` Shawn Guo
2014-07-01 21:25     ` Tejun Heo
2014-07-01 21:25       ` Tejun Heo

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=20140625152912.GI2224@dragon \
    --to=shawn.guo@freescale.com \
    --cc=devicetree@vger.kernel.org \
    --cc=galak@codeaurora.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=kernel@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=pawel.moll@arm.com \
    --cc=rdunlap@infradead.org \
    --cc=robh+dt@kernel.org \
    --cc=tj@kernel.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.