From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Mike Turquette <mturquette@linaro.org>,
Tero Kristo <t-kristo@ti.com>,
linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org
Subject: Re: [PATCH 1/8] clk: divider: fix rate calculation for fractional rates
Date: Fri, 28 Feb 2014 10:49:26 +0200 [thread overview]
Message-ID: <53104D96.5040704@ti.com> (raw)
In-Reply-To: <20140227022524.12081.32368@quantum>
[-- Attachment #1: Type: text/plain, Size: 689 bytes --]
On 27/02/14 04:25, Mike Turquette wrote:
>> Basically the patch looks good to me, but it might be good to have a
>> testing round of sort with this. It can potentially cause regressions on
>> multiple boards if the drivers happen to rely on the "broken" clock
>> rates. Same for patch #2 which is a copy paste of this one, but only
>> impacts TI boards.
>
> Agreed. I've taken patches #1 & #2 into clk-next. Let's let them stew in
> -next for a while and see if anyone's board catches on fire.
Thanks. I understand there's a possibility this could break some other
driver. I still wish we could get this to 3.14, as this would fix OMAP
display subsystem.
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 1/8] clk: divider: fix rate calculation for fractional rates
Date: Fri, 28 Feb 2014 10:49:26 +0200 [thread overview]
Message-ID: <53104D96.5040704@ti.com> (raw)
In-Reply-To: <20140227022524.12081.32368@quantum>
On 27/02/14 04:25, Mike Turquette wrote:
>> Basically the patch looks good to me, but it might be good to have a
>> testing round of sort with this. It can potentially cause regressions on
>> multiple boards if the drivers happen to rely on the "broken" clock
>> rates. Same for patch #2 which is a copy paste of this one, but only
>> impacts TI boards.
>
> Agreed. I've taken patches #1 & #2 into clk-next. Let's let them stew in
> -next for a while and see if anyone's board catches on fire.
Thanks. I understand there's a possibility this could break some other
driver. I still wish we could get this to 3.14, as this would fix OMAP
display subsystem.
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/20140228/ffb9e296/attachment.sig>
next prev parent reply other threads:[~2014-02-28 8:49 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 [this message]
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
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=53104D96.5040704@ti.com \
--to=tomi.valkeinen@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=mturquette@linaro.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.