All of lore.kernel.org
 help / color / mirror / Atom feed
From: s.hauer@pengutronix.de (Sascha Hauer)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/8] soc: Add MediaTek pericfg controller support
Date: Wed, 21 Jan 2015 09:17:48 +0100	[thread overview]
Message-ID: <20150121081748.GC12209@pengutronix.de> (raw)
In-Reply-To: <20150120161555.GH15924@leverpostej>

On Tue, Jan 20, 2015 at 04:15:55PM +0000, Mark Rutland wrote:
> > +
> > +	pericfg: pericfg at 10003000 {
> > +		#reset-cells = <1>;
> > +		#clock-cells = <1>;
> > +		compatible = "mediatek,mt8135-pericfg";
> > +		reg = <0 0x10003000 0 0x1000>;
> > +	};
> 
> This looks identical to the infracfg (same address too). Is that
> deliberate?

Both controllers indeed control clocks and resets, but with a different
register layout, so the nodes really look identical. I'll change the
address to the real address of the MT8135 pericfg though to make the
example more realistic.

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

WARNING: multiple messages have this Message-ID (diff)
From: Sascha Hauer <s.hauer@pengutronix.de>
To: Mark Rutland <mark.rutland@arm.com>
Cc: Olof Johansson <olof@lixom.net>, Arnd Bergmann <arnd@arndb.de>,
	Samuel Ortiz <sameo@linux.intel.com>,
	Mark Brown <broonie@kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	Eddie Huang <eddie.huang@mediatek.com>,
	Lee Jones <lee.jones@linaro.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 2/8] soc: Add MediaTek pericfg controller support
Date: Wed, 21 Jan 2015 09:17:48 +0100	[thread overview]
Message-ID: <20150121081748.GC12209@pengutronix.de> (raw)
In-Reply-To: <20150120161555.GH15924@leverpostej>

On Tue, Jan 20, 2015 at 04:15:55PM +0000, Mark Rutland wrote:
> > +
> > +	pericfg: pericfg@10003000 {
> > +		#reset-cells = <1>;
> > +		#clock-cells = <1>;
> > +		compatible = "mediatek,mt8135-pericfg";
> > +		reg = <0 0x10003000 0 0x1000>;
> > +	};
> 
> This looks identical to the infracfg (same address too). Is that
> deliberate?

Both controllers indeed control clocks and resets, but with a different
register layout, so the nodes really look identical. I'll change the
address to the real address of the MT8135 pericfg though to make the
example more realistic.

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

  reply	other threads:[~2015-01-21  8:17 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-20  9:47 MediaTek PMIC support Sascha Hauer
2015-01-20  9:47 ` Sascha Hauer
2015-01-20  9:47 ` [PATCH 1/8] soc: Add MediaTek infracfg controller support Sascha Hauer
2015-01-20  9:47   ` Sascha Hauer
2015-01-20 16:14   ` Mark Rutland
2015-01-20 16:14     ` Mark Rutland
2015-01-21  8:14     ` Sascha Hauer
2015-01-21  8:14       ` Sascha Hauer
2015-01-20  9:47 ` [PATCH 2/8] soc: Add MediaTek pericfg " Sascha Hauer
2015-01-20  9:47   ` Sascha Hauer
2015-01-20 13:18   ` Yingjoe Chen
2015-01-20 13:18     ` Yingjoe Chen
2015-01-20 13:27     ` Sascha Hauer
2015-01-20 13:27       ` Sascha Hauer
2015-01-20 16:15   ` Mark Rutland
2015-01-20 16:15     ` Mark Rutland
2015-01-21  8:17     ` Sascha Hauer [this message]
2015-01-21  8:17       ` Sascha Hauer
2015-01-20  9:47 ` [PATCH 3/8] dt: bindings: Add MediaTek MT8135/MT8173 reset controller defines Sascha Hauer
2015-01-20  9:47   ` Sascha Hauer
2015-01-20  9:47 ` [PATCH 4/8] soc: mediatek: Add PMIC wrapper for MT8135 and MT6397 SoC Sascha Hauer
2015-01-20  9:47   ` Sascha Hauer
2015-01-20  9:47 ` [PATCH 5/8] ARM: dts: mt8135: Add pericfg, infracfg and pmic wrapper nodes Sascha Hauer
2015-01-20  9:47   ` Sascha Hauer
2015-01-20  9:47 ` [PATCH 6/8] ARM: dts: mt8135-evbp1: Add PMIC support Sascha Hauer
2015-01-20  9:47   ` Sascha Hauer
2015-01-20  9:47 ` [PATCH 7/8] mfd: Add support for the MediaTek MT6397 PMIC Sascha Hauer
2015-01-20  9:47   ` Sascha Hauer
2015-01-20  9:47 ` [PATCH 8/8] regulator: MT6397: Add support for MT6397 regulator Sascha Hauer
2015-01-20  9:47   ` Sascha Hauer
2015-01-20 10:07   ` Paul Bolle
2015-01-20 10:07     ` Paul Bolle
2015-01-20 10:54     ` Sascha Hauer
2015-01-20 10:54       ` Sascha Hauer
2015-01-21  8:26   ` Sascha Hauer
2015-01-21  8:26     ` Sascha Hauer
  -- strict thread matches above, loose matches on Subject: below --
2015-02-05 15:25 [PATCH v3] MediaTek PMIC support Sascha Hauer
2015-02-05 15:25 ` [PATCH 2/8] soc: Add MediaTek pericfg controller support Sascha Hauer
2015-02-05 15:25   ` Sascha Hauer

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=20150121081748.GC12209@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --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.