From: Tony Lindgren <tony@atomide.com>
To: Janusz Krzysztofik <jkrzyszt@tis.icnet.pl>
Cc: Paul Walmsley <paul@pwsan.com>,
linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2a/5 v2] ARM: OMAP1: select clock rate by CPU type
Date: Thu, 1 Dec 2011 10:22:54 -0800 [thread overview]
Message-ID: <20111201182254.GU31337@atomide.com> (raw)
In-Reply-To: <201112011110.11476.jkrzyszt@tis.icnet.pl>
* Janusz Krzysztofik <jkrzyszt@tis.icnet.pl> [111201 01:35]:
> On Wednesday 30 of November 2011 at 23:28:38, Tony Lindgren wrote:
> >
> > We should also now be able to remove all the CONFIG_OMAP_ARM_XXXMHZ options
> > too, right?
>
> Right, but then, perhaps the initial version of patch 2a/5, which
> already started removing them, from omap1_defconfig for now, then going
> into the right direction while unblocking another regression fix (3/5),
> _is_ a good candidate for an rc fix?
But we did not allow dpll1 reprogramming earlier either, so we should
not need to make all these changes during the -rc cycle. I'm suspecting
that we've had this same behaviour for a really long time, and we just
have not seen it as omap1_defconfig had OMAP_CLOCKS_SET_BY_BOOTLOADER
option set.
So I'm baffled how your board would be booting at a different rate
compared to v3.1, it seems that the logic has not changed there. Or
else we have some simple bug somewhere.
Care to try to verify at what point your system started booting at
60MHz rate?
Regards,
Tony
next prev parent reply other threads:[~2011-12-01 18:22 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-27 3:32 [PATCH 0/5] ARM: OMAP1: Fix dpll1 reprogramming related issues Janusz Krzysztofik
2011-11-27 3:32 ` [PATCH 1/5] ARM: OMAP1: Fix dpll1 default rate reprogramming method Janusz Krzysztofik
2011-11-28 17:45 ` Tony Lindgren
2011-11-28 22:00 ` [PATCH 2a/5] Remove unsafe clock values from omap1_defconfig Janusz Krzysztofik
2011-11-30 22:32 ` Tony Lindgren
2011-11-30 20:57 ` [PATCH 2a/5 v2] ARM: OMAP1: select clock rate by CPU type Janusz Krzysztofik
2011-11-30 22:28 ` Tony Lindgren
2011-12-01 10:10 ` Janusz Krzysztofik
2011-12-01 18:22 ` Tony Lindgren [this message]
2011-12-01 18:54 ` Janusz Krzysztofik
2011-12-01 19:06 ` Tony Lindgren
2011-12-09 1:51 ` [PATCH] ARM: OMAP1: Set the omap1623 sram size to 16K Tony Lindgren
[not found] ` <201112010310.43890.jkrzyszt@tis.icnet.pl>
[not found] ` <20111201022750.GY13928@atomide.com>
2011-12-01 9:54 ` [PATCH 2a/5] Remove unsafe clock values from omap1_defconfig Janusz Krzysztofik
2011-12-01 10:21 ` Janusz Krzysztofik
2011-12-01 17:17 ` Tony Lindgren
2011-12-01 18:38 ` Janusz Krzysztofik
2011-12-01 19:04 ` Tony Lindgren
2011-12-01 19:23 ` Janusz Krzysztofik
2011-12-01 19:46 ` Tony Lindgren
2011-12-01 20:30 ` [PATCH] ARM: OMAP1: " Janusz Krzysztofik
2011-12-01 20:13 ` [PATCH 0/2 v2] ARM: OMAP1: Fix dpll1 reprogramming related issues Janusz Krzysztofik
2011-12-01 21:16 ` [PATCH v2] ARM: OMAP1: Update dpll1 default rate reprogramming method Janusz Krzysztofik
2011-12-02 2:09 ` [PATCH 0/2 v2] ARM: OMAP1: Fix dpll1 reprogramming related issues Tony Lindgren
2011-12-02 17:02 ` Janusz Krzysztofik
2011-12-01 20:13 ` [PATCH 1/2 v2] ARM: OMAP1: Fix ckctl value used for dpll1 defualt rate Janusz Krzysztofik
2011-12-01 20:13 ` [PATCH 2/2 v2] ARM: OMAP1: recalculate loops per jiffy after dpll1 reprogram Janusz Krzysztofik
2011-11-27 3:32 ` [PATCH 2/5] ARM: OMAP1: Fix ckctl value used for dpll1 defualt rate Janusz Krzysztofik
2011-11-27 3:32 ` [PATCH 3/5] ARM: OMAP1: Fix dpll1 reprogramming not actually allowed Janusz Krzysztofik
2011-11-28 17:46 ` Tony Lindgren
2011-11-27 3:32 ` [PATCH 4/5] init/calibrate.c: allow for recalibration of loops per jiffy Janusz Krzysztofik
2011-11-28 15:39 ` Russell King - ARM Linux
2011-11-28 22:30 ` Janusz Krzysztofik
2011-11-27 3:32 ` [PATCH 5/5] ARM: OMAP1: recalibrate loops per jiffy after dpll1 reprogram Janusz Krzysztofik
2011-11-29 0:25 ` [PATCH 5/5 v2] ARM: OMAP1: recalculate " Janusz Krzysztofik
2011-12-09 8:42 ` Russell King - ARM Linux
2011-12-09 10:00 ` Janusz Krzysztofik
2011-12-09 10:09 ` Janusz Krzysztofik
2011-12-10 0:25 ` Russell King - ARM Linux
2011-12-10 12:04 ` Janusz Krzysztofik
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=20111201182254.GU31337@atomide.com \
--to=tony@atomide.com \
--cc=jkrzyszt@tis.icnet.pl \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=paul@pwsan.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).