linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: dinh.linux@gmail.com (Dinh Nguyen)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] clk: socfpga: add divider registers to the main pll outputs
Date: Wed, 16 Apr 2014 15:57:00 -0500	[thread overview]
Message-ID: <534EEE9C.6070806@gmail.com> (raw)
In-Reply-To: <20140416204950.GK16054@pengutronix.de>



On 04/16/2014 03:49 PM, Steffen Trumtrar wrote:
> Hi!
>
> On Wed, Apr 16, 2014 at 03:23:11PM -0500, Dinh Nguyen wrote:
>>
>>
>> On 04/16/2014 03:14 PM, dinguyen at altera.com wrote:
>>> From: Dinh Nguyen <dinguyen@altera.com>
>>>
>>> The C0(mpu_clk), C1(main_clk), and C2(dbg_base_clk) outputs from the main
>>> PLL go through a pre-divider before coming into the system. These registers
>>> were hidden for the CycloneV platform, but are not used for the ArriaV
>>
>> Sorry but this should be "but are now used"
>>
>
> ???
>
> I don't get it. Do we have these registers on the cyclone V AND arria V or do
> we only have them on the arria V ?

These registers are there for both CycloneV and ArriaV.They are 
configured by the preloader, so it was "hidden" as a fixed-divider. I 
could have designated these values as fixed for the ArriaV as well, but 
it would be better if I just read this "hidden" register to get the 
divider value.

Dinh
>
> IIRC I had made a patch that adds dividers to some place in the clocktree, but
> I can't remember if these are the same.
>
> Regards,
> Steffen
>
>>> platform.
>>>
>>> This patch updates the clock driver to read the div-reg property for the
>>> socfpga-periph-clk clocks. Also moves the div_mask define to clk.h for re-use.
>>>
>>> Signed-off-by: Dinh Nguyen <dinguyen@altera.com>
>

  reply	other threads:[~2014-04-16 20:57 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-16 20:14 [PATCH 1/2] clk: socfpga: add divider registers to the main pll outputs dinguyen at altera.com
2014-04-16 20:14 ` [PATCH 2/2] ARM: socfpga: dts: Add div-reg to the main_pll clocks dinguyen at altera.com
2014-04-16 20:23 ` [PATCH 1/2] clk: socfpga: add divider registers to the main pll outputs Dinh Nguyen
2014-04-16 20:49   ` Steffen Trumtrar
2014-04-16 20:57     ` Dinh Nguyen [this message]
2014-04-17  7:37       ` Steffen Trumtrar

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=534EEE9C.6070806@gmail.com \
    --to=dinh.linux@gmail.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).