From: Keerthy <a0393675-l0cyMroinI0@public.gmane.org>
To: Mugunthan V N <mugunthanvnm-l0cyMroinI0@public.gmane.org>,
linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>,
Keerthy <j-keerthy-l0cyMroinI0@public.gmane.org>,
Tero Kristo <t-kristo-l0cyMroinI0@public.gmane.org>,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 1/1] arm: dts: am43xx-clock: Add qspi clock node.
Date: Fri, 26 Jun 2015 11:50:30 +0530 [thread overview]
Message-ID: <558CEF2E.3050202@ti.com> (raw)
In-Reply-To: <1435167933-5249-1-git-send-email-mugunthanvnm-l0cyMroinI0@public.gmane.org>
On Wednesday 24 June 2015 11:15 PM, Mugunthan V N wrote:
> From: Sourav Poddar <sourav.poddar-l0cyMroinI0@public.gmane.org>
>
> Add clock node for qspi controller driver on am437x.
>
> Cc: Keerthy <j-keerthy-l0cyMroinI0@public.gmane.org>
> Cc: Tero Kristo <t-kristo-l0cyMroinI0@public.gmane.org>
> Signed-off-by: Sourav Poddar <sourav.poddar-l0cyMroinI0@public.gmane.org>
> Signed-off-by: Mugunthan V N <mugunthanvnm-l0cyMroinI0@public.gmane.org>
> ---
> arch/arm/boot/dts/am4372.dtsi | 2 ++
> arch/arm/boot/dts/am43xx-clocks.dtsi | 8 ++++++++
> 2 files changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi
> index 649e7b9..8ab7831 100644
> --- a/arch/arm/boot/dts/am4372.dtsi
> +++ b/arch/arm/boot/dts/am4372.dtsi
> @@ -903,6 +903,8 @@
> #address-cells = <1>;
> #size-cells = <0>;
> ti,hwmods = "qspi";
> + clocks = <&qspi_fclk>;
> + clock-names = "fck";
> interrupts = <0 138 0x4>;
> num-cs = <4>;
> status = "disabled";
> diff --git a/arch/arm/boot/dts/am43xx-clocks.dtsi b/arch/arm/boot/dts/am43xx-clocks.dtsi
> index cc88728..1506585 100644
> --- a/arch/arm/boot/dts/am43xx-clocks.dtsi
> +++ b/arch/arm/boot/dts/am43xx-clocks.dtsi
> @@ -763,4 +763,12 @@
> ti,bit-shift = <8>;
> reg = <0x8a68>;
> };
> +
> + qspi_fclk: qspi_fclk {
> + #clock-cells = <0>;
> + compatible = "ti,fixed-factor-clock";
> + clocks = <&dpll_per_m2_div4_ck>;
From the clock tree this does not seem to be sourced from dpll_per_m2.
I guess this needs to be changed.
-Keerthy
> + clock-mult = <1>;
> + clock-div = <1>;
> + };
> };
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: a0393675@ti.com (Keerthy)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/1] arm: dts: am43xx-clock: Add qspi clock node.
Date: Fri, 26 Jun 2015 11:50:30 +0530 [thread overview]
Message-ID: <558CEF2E.3050202@ti.com> (raw)
In-Reply-To: <1435167933-5249-1-git-send-email-mugunthanvnm@ti.com>
On Wednesday 24 June 2015 11:15 PM, Mugunthan V N wrote:
> From: Sourav Poddar <sourav.poddar@ti.com>
>
> Add clock node for qspi controller driver on am437x.
>
> Cc: Keerthy <j-keerthy@ti.com>
> Cc: Tero Kristo <t-kristo@ti.com>
> Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
> Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com>
> ---
> arch/arm/boot/dts/am4372.dtsi | 2 ++
> arch/arm/boot/dts/am43xx-clocks.dtsi | 8 ++++++++
> 2 files changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi
> index 649e7b9..8ab7831 100644
> --- a/arch/arm/boot/dts/am4372.dtsi
> +++ b/arch/arm/boot/dts/am4372.dtsi
> @@ -903,6 +903,8 @@
> #address-cells = <1>;
> #size-cells = <0>;
> ti,hwmods = "qspi";
> + clocks = <&qspi_fclk>;
> + clock-names = "fck";
> interrupts = <0 138 0x4>;
> num-cs = <4>;
> status = "disabled";
> diff --git a/arch/arm/boot/dts/am43xx-clocks.dtsi b/arch/arm/boot/dts/am43xx-clocks.dtsi
> index cc88728..1506585 100644
> --- a/arch/arm/boot/dts/am43xx-clocks.dtsi
> +++ b/arch/arm/boot/dts/am43xx-clocks.dtsi
> @@ -763,4 +763,12 @@
> ti,bit-shift = <8>;
> reg = <0x8a68>;
> };
> +
> + qspi_fclk: qspi_fclk {
> + #clock-cells = <0>;
> + compatible = "ti,fixed-factor-clock";
> + clocks = <&dpll_per_m2_div4_ck>;
From the clock tree this does not seem to be sourced from dpll_per_m2.
I guess this needs to be changed.
-Keerthy
> + clock-mult = <1>;
> + clock-div = <1>;
> + };
> };
>
WARNING: multiple messages have this Message-ID (diff)
From: Keerthy <a0393675@ti.com>
To: Mugunthan V N <mugunthanvnm@ti.com>, <linux-omap@vger.kernel.org>
Cc: Tony Lindgren <tony@atomide.com>, Keerthy <j-keerthy@ti.com>,
Tero Kristo <t-kristo@ti.com>, <devicetree@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/1] arm: dts: am43xx-clock: Add qspi clock node.
Date: Fri, 26 Jun 2015 11:50:30 +0530 [thread overview]
Message-ID: <558CEF2E.3050202@ti.com> (raw)
In-Reply-To: <1435167933-5249-1-git-send-email-mugunthanvnm@ti.com>
On Wednesday 24 June 2015 11:15 PM, Mugunthan V N wrote:
> From: Sourav Poddar <sourav.poddar@ti.com>
>
> Add clock node for qspi controller driver on am437x.
>
> Cc: Keerthy <j-keerthy@ti.com>
> Cc: Tero Kristo <t-kristo@ti.com>
> Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
> Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com>
> ---
> arch/arm/boot/dts/am4372.dtsi | 2 ++
> arch/arm/boot/dts/am43xx-clocks.dtsi | 8 ++++++++
> 2 files changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi
> index 649e7b9..8ab7831 100644
> --- a/arch/arm/boot/dts/am4372.dtsi
> +++ b/arch/arm/boot/dts/am4372.dtsi
> @@ -903,6 +903,8 @@
> #address-cells = <1>;
> #size-cells = <0>;
> ti,hwmods = "qspi";
> + clocks = <&qspi_fclk>;
> + clock-names = "fck";
> interrupts = <0 138 0x4>;
> num-cs = <4>;
> status = "disabled";
> diff --git a/arch/arm/boot/dts/am43xx-clocks.dtsi b/arch/arm/boot/dts/am43xx-clocks.dtsi
> index cc88728..1506585 100644
> --- a/arch/arm/boot/dts/am43xx-clocks.dtsi
> +++ b/arch/arm/boot/dts/am43xx-clocks.dtsi
> @@ -763,4 +763,12 @@
> ti,bit-shift = <8>;
> reg = <0x8a68>;
> };
> +
> + qspi_fclk: qspi_fclk {
> + #clock-cells = <0>;
> + compatible = "ti,fixed-factor-clock";
> + clocks = <&dpll_per_m2_div4_ck>;
From the clock tree this does not seem to be sourced from dpll_per_m2.
I guess this needs to be changed.
-Keerthy
> + clock-mult = <1>;
> + clock-div = <1>;
> + };
> };
>
next prev parent reply other threads:[~2015-06-26 6:20 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-24 17:45 [PATCH 1/1] arm: dts: am43xx-clock: Add qspi clock node Mugunthan V N
2015-06-24 17:45 ` Mugunthan V N
2015-06-24 17:45 ` Mugunthan V N
[not found] ` <1435167933-5249-1-git-send-email-mugunthanvnm-l0cyMroinI0@public.gmane.org>
2015-06-26 6:20 ` Keerthy [this message]
2015-06-26 6:20 ` Keerthy
2015-06-26 6:20 ` Keerthy
2015-06-26 7:33 ` Roger Quadros
2015-06-26 7:33 ` Roger Quadros
2015-06-26 7:33 ` Roger Quadros
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=558CEF2E.3050202@ti.com \
--to=a0393675-l0cymroini0@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=j-keerthy-l0cyMroinI0@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=mugunthanvnm-l0cyMroinI0@public.gmane.org \
--cc=t-kristo-l0cyMroinI0@public.gmane.org \
--cc=tony-4v6yS6AI5VpBDgjK7y7TUQ@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.