linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: Minimal support for dm814x
Date: Wed, 11 Nov 2015 09:40:29 -0800	[thread overview]
Message-ID: <20151111174028.GF3218@atomide.com> (raw)
In-Reply-To: <7933071D-84FD-4A85-8CD0-CC504CE1D9D1@audioscience.com>

* Delio Brignoli <dbrignoli@audioscience.com> [151110 02:24]:
> On 10 Nov 2015, at 09:50, Matthijs van Duin <matthijsvanduin@gmail.com> wrote:
> > On 9 November 2015 at 16:06, Tony Lindgren <tony@atomide.com> wrote:
> >> The PLL support is still missing, so it relies on the bootloader
> >> configured PLL values for now. I'm hoping to post PLL support patches over
> >> next few weeks and then we can have that and more devices working for v4.5.
> > 
> > Ah, yes, configuring a DPLL-LJ is fun.. figuring out how to write the
> > desired ratio as M/(M2*(1+N)) while simultaneously satisfying all
> > constraints on M, N, M2, refclk, and dco. :-)
> 
> Yes, indeed. We have the additional requirement of being able to adjust the frequency (by a relatively small amount) without loss of lock. Recalculating the DCO mode and M,N,M2 from scratch each time based on the target frequency, like was done in the 2.6.37 based tree from TI was not acceptable, so we try to change m2 first to see if we can reach the target frequency and fall back to recalculate parameters from scratch if that fails.

Well we do first try to set the rate using the divider only at least for
drivers/clk/ti/fapll.c used on dm816x. I'm thinking about doing a similar
driver for the dm814x adpll where we have a PLL and separate output clocks
in a single driver as the PLL and output control registers are all mixed
in.

> BTW, are you aware of section 2.1.2 of ?TMS320DM814x DaVinci Digital Media Processors Silicon Revisions 3.0, 2.1?? <http://www.ti.com/lit/er/sprz343c/sprz343c.pdf>

OK good to know :)

Tony

  parent reply	other threads:[~2015-11-11 17:40 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-05 13:08 Minimal support for dm814x Delio Brignoli
2015-11-09 15:06 ` Tony Lindgren
2015-11-10  8:50   ` Matthijs van Duin
2015-11-10 10:23     ` Delio Brignoli
2015-11-10 10:44       ` Matthijs van Duin
2015-11-11 17:40       ` Tony Lindgren [this message]
2015-11-12  9:20         ` Matthijs van Duin
2015-11-12 17:41           ` Tony Lindgren
2015-11-13  7:14             ` Matthijs van Duin
2015-11-13 10:59               ` Matthijs van Duin
2015-11-13 14:52                 ` Tony Lindgren
2015-11-13 14:57                   ` Tony Lindgren
2015-11-18  5:22                   ` Matthijs van Duin
2015-11-18  8:26                     ` Delio Brignoli
2015-11-18 10:01                       ` Matthijs van Duin
2015-11-18 11:09                         ` Delio Brignoli
2015-11-10 10:05   ` Delio Brignoli

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=20151111174028.GF3218@atomide.com \
    --to=tony@atomide.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 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).