All of lore.kernel.org
 help / color / mirror / Atom feed
From: Maxime Coquelin <maxime.coquelin@st.com>
To: Peter Griffin <peter.griffin@linaro.org>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, patrice.chotard@st.com,
	srinivas.kandagatla@gmail.com, tj@kernel.org
Cc: lee.jones@linaro.org, linux-ide@vger.kernel.org,
	devicetree@vger.kernel.org
Subject: Re: [PATCH 0/3] Enable ahci_st for stih407 and fix hang
Date: Tue, 3 Mar 2015 13:23:09 +0100	[thread overview]
Message-ID: <54F5A7AD.9000906@st.com> (raw)
In-Reply-To: <1425300519-13747-1-git-send-email-peter.griffin@linaro.org>

Hi Peter,

On 03/02/2015 01:48 PM, Peter Griffin wrote:
> Hi,
>
> Now that miphy28lp phy is present upstream we can add sata DT nodes for
> stih407 family silicon. As part of testing this I found a hang
> with the current driver implmentation which does some register writes
> before enabling the IP clock. Presumably on stih416 devices the sata
> clock must have already been enabled by the targetpack/bootloader.
>
> This series also tidys up some inaccuracies in the DT documentation
> where the doc hasn't kept aligned with the driver. In addtion to these
> fixes I've also added an example for stih407 silicon.
>
> Peter Griffin (3):
>    ahci: st: Update the ahci_st DT documentation
>    ARM: DT: STi: STiH407: Add sata DT nodes.
>    ahci: st: st_configure_oob must be called after IP is clocked.
>
>   Documentation/devicetree/bindings/ata/ahci-st.txt | 45 ++++++++++++++++-------
>   arch/arm/boot/dts/stih407-family.dtsi             | 44 ++++++++++++++++++++++
>   drivers/ata/ahci_st.c                             |  6 ++-
>   3 files changed, 80 insertions(+), 15 deletions(-)
>

For the series, you can add my:
Acked-by: Maxime Coquelin <maxime.coquelin@st.com>

Thanks,
Maxime

WARNING: multiple messages have this Message-ID (diff)
From: maxime.coquelin@st.com (Maxime Coquelin)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/3] Enable ahci_st for stih407 and fix hang
Date: Tue, 3 Mar 2015 13:23:09 +0100	[thread overview]
Message-ID: <54F5A7AD.9000906@st.com> (raw)
In-Reply-To: <1425300519-13747-1-git-send-email-peter.griffin@linaro.org>

Hi Peter,

On 03/02/2015 01:48 PM, Peter Griffin wrote:
> Hi,
>
> Now that miphy28lp phy is present upstream we can add sata DT nodes for
> stih407 family silicon. As part of testing this I found a hang
> with the current driver implmentation which does some register writes
> before enabling the IP clock. Presumably on stih416 devices the sata
> clock must have already been enabled by the targetpack/bootloader.
>
> This series also tidys up some inaccuracies in the DT documentation
> where the doc hasn't kept aligned with the driver. In addtion to these
> fixes I've also added an example for stih407 silicon.
>
> Peter Griffin (3):
>    ahci: st: Update the ahci_st DT documentation
>    ARM: DT: STi: STiH407: Add sata DT nodes.
>    ahci: st: st_configure_oob must be called after IP is clocked.
>
>   Documentation/devicetree/bindings/ata/ahci-st.txt | 45 ++++++++++++++++-------
>   arch/arm/boot/dts/stih407-family.dtsi             | 44 ++++++++++++++++++++++
>   drivers/ata/ahci_st.c                             |  6 ++-
>   3 files changed, 80 insertions(+), 15 deletions(-)
>

For the series, you can add my:
Acked-by: Maxime Coquelin <maxime.coquelin@st.com>

Thanks,
Maxime

WARNING: multiple messages have this Message-ID (diff)
From: Maxime Coquelin <maxime.coquelin@st.com>
To: Peter Griffin <peter.griffin@linaro.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>, <patrice.chotard@st.com>,
	<srinivas.kandagatla@gmail.com>, <tj@kernel.org>
Cc: <lee.jones@linaro.org>, <linux-ide@vger.kernel.org>,
	<devicetree@vger.kernel.org>
Subject: Re: [PATCH 0/3] Enable ahci_st for stih407 and fix hang
Date: Tue, 3 Mar 2015 13:23:09 +0100	[thread overview]
Message-ID: <54F5A7AD.9000906@st.com> (raw)
In-Reply-To: <1425300519-13747-1-git-send-email-peter.griffin@linaro.org>

Hi Peter,

On 03/02/2015 01:48 PM, Peter Griffin wrote:
> Hi,
>
> Now that miphy28lp phy is present upstream we can add sata DT nodes for
> stih407 family silicon. As part of testing this I found a hang
> with the current driver implmentation which does some register writes
> before enabling the IP clock. Presumably on stih416 devices the sata
> clock must have already been enabled by the targetpack/bootloader.
>
> This series also tidys up some inaccuracies in the DT documentation
> where the doc hasn't kept aligned with the driver. In addtion to these
> fixes I've also added an example for stih407 silicon.
>
> Peter Griffin (3):
>    ahci: st: Update the ahci_st DT documentation
>    ARM: DT: STi: STiH407: Add sata DT nodes.
>    ahci: st: st_configure_oob must be called after IP is clocked.
>
>   Documentation/devicetree/bindings/ata/ahci-st.txt | 45 ++++++++++++++++-------
>   arch/arm/boot/dts/stih407-family.dtsi             | 44 ++++++++++++++++++++++
>   drivers/ata/ahci_st.c                             |  6 ++-
>   3 files changed, 80 insertions(+), 15 deletions(-)
>

For the series, you can add my:
Acked-by: Maxime Coquelin <maxime.coquelin@st.com>

Thanks,
Maxime

  parent reply	other threads:[~2015-03-03 12:23 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-02 12:48 [PATCH 0/3] Enable ahci_st for stih407 and fix hang Peter Griffin
2015-03-02 12:48 ` Peter Griffin
2015-03-02 12:48 ` [PATCH 1/3] ahci: st: Update the ahci_st DT documentation Peter Griffin
2015-03-02 12:48   ` Peter Griffin
2015-03-02 16:59   ` Lee Jones
2015-03-02 16:59     ` Lee Jones
2015-03-02 12:48 ` [PATCH 2/3] ARM: DT: STi: STiH407: Add sata DT nodes Peter Griffin
2015-03-02 12:48   ` Peter Griffin
2015-03-02 17:00   ` Lee Jones
2015-03-02 17:00     ` Lee Jones
2015-03-02 12:48 ` [PATCH 3/3] ahci: st: st_configure_oob must be called after IP is clocked Peter Griffin
2015-03-02 12:48   ` Peter Griffin
2015-03-02 17:03   ` Lee Jones
2015-03-02 17:03     ` Lee Jones
2015-03-03 12:23 ` Maxime Coquelin [this message]
2015-03-03 12:23   ` [PATCH 0/3] Enable ahci_st for stih407 and fix hang Maxime Coquelin
2015-03-03 12:23   ` Maxime Coquelin

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=54F5A7AD.9000906@st.com \
    --to=maxime.coquelin@st.com \
    --cc=devicetree@vger.kernel.org \
    --cc=lee.jones@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=patrice.chotard@st.com \
    --cc=peter.griffin@linaro.org \
    --cc=srinivas.kandagatla@gmail.com \
    --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.