From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Tero Kristo <t-kristo@ti.com>,
linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org
Subject: Re: [PATCH 7/8] ARM: dts: set 'ti,set-rate-parent' for dpll4_m4 path
Date: Fri, 14 Feb 2014 16:01:48 +0200 [thread overview]
Message-ID: <52FE21CC.1010200@ti.com> (raw)
In-Reply-To: <52FE2020.2010601@ti.com>
[-- Attachment #1: Type: text/plain, Size: 641 bytes --]
On 14/02/14 15:54, Tero Kristo wrote:
> On 02/13/2014 12:04 PM, Tomi Valkeinen wrote:
>> Set 'ti,set-rate-parent' property for clocks in the dpll4_m4 clock
>> path, which is used for DSS functional clock. This fixes DSS driver's
>> clock rate configuration, which needs the rate to be propagated properly
>> to the divider node (dpll4_m4_ck).
>
>
> ... squash to #6?
I got no problem with that, but I think it's cleaner to separate these
two things. #6 converts the clock to use a different driver, and by
having these separate, it's easier to see that #6 converts the node 1:1
without other functional change.
Tomi
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 901 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: tomi.valkeinen@ti.com (Tomi Valkeinen)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 7/8] ARM: dts: set 'ti,set-rate-parent' for dpll4_m4 path
Date: Fri, 14 Feb 2014 16:01:48 +0200 [thread overview]
Message-ID: <52FE21CC.1010200@ti.com> (raw)
In-Reply-To: <52FE2020.2010601@ti.com>
On 14/02/14 15:54, Tero Kristo wrote:
> On 02/13/2014 12:04 PM, Tomi Valkeinen wrote:
>> Set 'ti,set-rate-parent' property for clocks in the dpll4_m4 clock
>> path, which is used for DSS functional clock. This fixes DSS driver's
>> clock rate configuration, which needs the rate to be propagated properly
>> to the divider node (dpll4_m4_ck).
>
>
> ... squash to #6?
I got no problem with that, but I think it's cleaner to separate these
two things. #6 converts the clock to use a different driver, and by
having these separate, it's easier to see that #6 converts the node 1:1
without other functional change.
Tomi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 901 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140214/74d12cda/attachment.sig>
next prev parent reply other threads:[~2014-02-14 14:02 UTC|newest]
Thread overview: 54+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-13 10:03 [PATCH 0/8] OMAP: OMAP3 DSS related clock patches Tomi Valkeinen
2014-02-13 10:03 ` Tomi Valkeinen
2014-02-13 10:03 ` [PATCH 1/8] clk: divider: fix rate calculation for fractional rates Tomi Valkeinen
2014-02-13 10:03 ` Tomi Valkeinen
2014-02-14 13:45 ` Tero Kristo
2014-02-14 13:45 ` Tero Kristo
2014-02-27 2:25 ` Mike Turquette
2014-02-27 2:25 ` Mike Turquette
2014-02-28 8:49 ` Tomi Valkeinen
2014-02-28 8:49 ` Tomi Valkeinen
2014-03-17 12:53 ` Tomi Valkeinen
2014-03-17 12:53 ` Tomi Valkeinen
2014-03-19 4:26 ` Mike Turquette
2014-03-19 4:26 ` Mike Turquette
2014-02-13 10:04 ` [PATCH 2/8] clk: ti/divider: " Tomi Valkeinen
2014-02-13 10:04 ` Tomi Valkeinen
2014-02-13 10:04 ` [PATCH 3/8] ARM: OMAP2+: clock: fix clkoutx2 with CLK_SET_RATE_PARENT Tomi Valkeinen
2014-02-13 10:04 ` Tomi Valkeinen
2014-02-13 10:45 ` Tomi Valkeinen
2014-02-13 10:45 ` Tomi Valkeinen
2014-02-13 10:04 ` [PATCH 4/8] ARM: dts: fix omap3 dss clock handle names Tomi Valkeinen
2014-02-13 10:04 ` Tomi Valkeinen
2014-02-14 13:48 ` Tero Kristo
2014-02-14 13:48 ` Tero Kristo
2014-02-13 10:04 ` [PATCH 5/8] ARM: dts: fix DPLL4 x2 clkouts on 3630 Tomi Valkeinen
2014-02-13 10:04 ` Tomi Valkeinen
2014-02-14 13:52 ` Tero Kristo
2014-02-14 13:52 ` Tero Kristo
2014-02-13 10:04 ` [PATCH 6/8] ARM: dts: use ti,fixed-factor-clock for dpll4_m4x2_mul_ck Tomi Valkeinen
2014-02-13 10:04 ` [PATCH 6/8] ARM: dts: use ti, fixed-factor-clock " Tomi Valkeinen
2014-02-14 13:54 ` [PATCH 6/8] ARM: dts: use ti,fixed-factor-clock " Tero Kristo
2014-02-14 13:54 ` [PATCH 6/8] ARM: dts: use ti, fixed-factor-clock " Tero Kristo
2014-02-14 14:05 ` [PATCH 6/8] ARM: dts: use ti,fixed-factor-clock " Tomi Valkeinen
2014-02-14 14:05 ` [PATCH 6/8] ARM: dts: use ti, fixed-factor-clock " Tomi Valkeinen
2014-02-13 10:04 ` [PATCH 7/8] ARM: dts: set 'ti,set-rate-parent' for dpll4_m4 path Tomi Valkeinen
2014-02-13 10:04 ` Tomi Valkeinen
2014-02-14 13:54 ` Tero Kristo
2014-02-14 13:54 ` Tero Kristo
2014-02-14 14:01 ` Tomi Valkeinen [this message]
2014-02-14 14:01 ` Tomi Valkeinen
2014-02-13 10:04 ` [PATCH 8/8] OMAPDSS: fix rounding when calculating fclk rate Tomi Valkeinen
2014-02-13 10:04 ` Tomi Valkeinen
2014-02-13 13:07 ` [PATCH 0/8] OMAP: OMAP3 DSS related clock patches Belisko Marek
2014-02-13 13:07 ` Belisko Marek
2014-02-27 17:23 ` Florian Vaussard
2014-02-27 17:23 ` Florian Vaussard
2014-03-05 8:35 ` Tomi Valkeinen
2014-03-05 8:35 ` Tomi Valkeinen
2014-03-05 10:12 ` Tero Kristo
2014-03-05 10:12 ` Tero Kristo
2014-03-05 11:25 ` Tomi Valkeinen
2014-03-05 11:25 ` Tomi Valkeinen
2014-03-05 17:19 ` Tony Lindgren
2014-03-05 17:19 ` Tony Lindgren
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=52FE21CC.1010200@ti.com \
--to=tomi.valkeinen@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=t-kristo@ti.com \
/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.