All of lore.kernel.org
 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 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.