From: gregory.clement@free-electrons.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/2] arm64: dts: marvell: fix clocksource for CP110 slave SPI0
Date: Tue, 20 Sep 2016 17:07:06 +0200 [thread overview]
Message-ID: <87pony625h.fsf@free-electrons.com> (raw)
In-Reply-To: <1473183672-12028-3-git-send-email-mw@semihalf.com> (Marcin Wojtas's message of "Tue, 6 Sep 2016 19:41:12 +0200")
Hi Olof and Arnd,
On mar., sept. 06 2016, Marcin Wojtas <mw@semihalf.com> wrote:
> I2C and SPI interfaces share common clock trees within the CP110 HW block.
> It occurred that SPI0 interface has wrong clock assignment in the device
> tree, which is fixed in this commit to a proper value.
>
> Fixes: c749b8d9de32 ("arm64: dts: marvell: add description for the ...")
> Signed-off-by: Marcin Wojtas <mw@semihalf.com>
Acked-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
Could you take this patch for 4.9? I didn't realized at first view it
was for 4.9, I though I could put it the PR for fixes for 4.8.
I can do a new PR for dt64 with this single patch if you prefer.
The last option will be to have it as a fix for v4.9-rc2.
Thanks,
Gregory
> ---
> arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
> index 37bff70..150675c 100644
> --- a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
> @@ -133,7 +133,7 @@
> #address-cells = <0x1>;
> #size-cells = <0x0>;
> cell-index = <1>;
> - clocks = <&cps_syscon0 0 3>;
> + clocks = <&cps_syscon0 1 21>;
> status = "disabled";
> };
>
> --
> 1.8.3.1
>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
WARNING: multiple messages have this Message-ID (diff)
From: Gregory CLEMENT <gregory.clement@free-electrons.com>
To: Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>,
Marcin Wojtas <mw@semihalf.com>
Cc: linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
sebastian.hesselbarth@gmail.com, andrew@lunn.ch,
jason@lakedaemon.net, thomas.petazzoni@free-electrons.com,
will.deacon@arm.com, robh+dt@kernel.org, mark.rutland@arm.com,
nadavh@marvell.com, alior@marvell.com, jaz@semihalf.com,
tn@semihalf.com
Subject: Re: [PATCH 2/2] arm64: dts: marvell: fix clocksource for CP110 slave SPI0
Date: Tue, 20 Sep 2016 17:07:06 +0200 [thread overview]
Message-ID: <87pony625h.fsf@free-electrons.com> (raw)
In-Reply-To: <1473183672-12028-3-git-send-email-mw@semihalf.com> (Marcin Wojtas's message of "Tue, 6 Sep 2016 19:41:12 +0200")
Hi Olof and Arnd,
On mar., sept. 06 2016, Marcin Wojtas <mw@semihalf.com> wrote:
> I2C and SPI interfaces share common clock trees within the CP110 HW block.
> It occurred that SPI0 interface has wrong clock assignment in the device
> tree, which is fixed in this commit to a proper value.
>
> Fixes: c749b8d9de32 ("arm64: dts: marvell: add description for the ...")
> Signed-off-by: Marcin Wojtas <mw@semihalf.com>
Acked-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
Could you take this patch for 4.9? I didn't realized at first view it
was for 4.9, I though I could put it the PR for fixes for 4.8.
I can do a new PR for dt64 with this single patch if you prefer.
The last option will be to have it as a fix for v4.9-rc2.
Thanks,
Gregory
> ---
> arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
> index 37bff70..150675c 100644
> --- a/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi
> @@ -133,7 +133,7 @@
> #address-cells = <0x1>;
> #size-cells = <0x0>;
> cell-index = <1>;
> - clocks = <&cps_syscon0 0 3>;
> + clocks = <&cps_syscon0 1 21>;
> status = "disabled";
> };
>
> --
> 1.8.3.1
>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2016-09-20 15:07 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-06 17:41 [PATCH 0/2] Armada 7k/8k SPI0 clock references fix Marcin Wojtas
2016-09-06 17:41 ` Marcin Wojtas
2016-09-06 17:41 ` [PATCH 1/2] arm64: dts: marvell: fix clocksource for CP110 master SPI0 Marcin Wojtas
2016-09-06 17:41 ` Marcin Wojtas
2016-09-20 14:56 ` Gregory CLEMENT
2016-09-20 14:56 ` Gregory CLEMENT
2016-09-06 17:41 ` [PATCH 2/2] arm64: dts: marvell: fix clocksource for CP110 slave SPI0 Marcin Wojtas
2016-09-06 17:41 ` Marcin Wojtas
2016-09-20 15:07 ` Gregory CLEMENT [this message]
2016-09-20 15:07 ` 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=87pony625h.fsf@free-electrons.com \
--to=gregory.clement@free-electrons.com \
--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 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.