public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: maxime.ripard@free-electrons.com (Maxime Ripard)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/2] ARM: sunxi: Enable syscon for the system controller
Date: Mon, 5 May 2014 17:55:17 -0500	[thread overview]
Message-ID: <20140505225517.GE9464@lukather> (raw)
In-Reply-To: <1399212160-26934-1-git-send-email-carlo@caione.org>

On Sun, May 04, 2014 at 04:02:38PM +0200, Carlo Caione wrote:
> The so called "system controller" in Allwinner A20 and A31 SoCs is
> multi-purpose controller that tries to add misc functionality to one
> memory region.
> In these SoCs it controls the internal SRAM partitioning but it also
> includes registers for chip versioning and NMI control.
> This patch adds the proper nodes in the DTS files and enable the syscon
> in the defconfig files.
> 
> Even though the system controller includes also register for managing the
> NMI controller, these register are not mapped in the syscon since they
> are directly used and mapped by the NMI controller itself.

Hmmm, what exactly do you want to achieve with this?

The NMI controller won't be able to use it, since it's initialized
much earlier than syscon and regmap.

Moreover, the A31 doesn't seem to have this system controller, or at
least this overlap.

And since on the A20, registers seem to have one usage only, so I
guess we can just split this IP into several nodes, just like we did
with the NMI.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140505/94cab0bb/attachment.sig>

  parent reply	other threads:[~2014-05-05 22:55 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-04 14:02 [PATCH 0/2] ARM: sunxi: Enable syscon for the system controller Carlo Caione
2014-05-04 14:02 ` [PATCH 1/2] ARM: sunxi: dts: Enable syscon in DTSI Carlo Caione
2014-05-04 14:02 ` [PATCH 2/2] ARM: sunxi: Add syscon support in defconfig Carlo Caione
2014-05-05 22:55 ` Maxime Ripard [this message]
2014-05-06  6:36   ` [linux-sunxi] Re: [PATCH 0/2] ARM: sunxi: Enable syscon for the system controller Chen-Yu Tsai
2014-05-06  8:03     ` Carlo Caione
2014-05-06  8:27       ` Chen-Yu Tsai
2014-05-06  8:42         ` Carlo Caione
2014-05-07  3:25       ` Maxime Ripard
2014-05-07  8:19         ` Carlo Caione
2014-05-08  3:04           ` Maxime Ripard
2014-05-08  6:14             ` Carlo Caione

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=20140505225517.GE9464@lukather \
    --to=maxime.ripard@free-electrons.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