From: Stefan Wahren <stefan.wahren@i2se.com> To: Sascha Hauer <s.hauer@pengutronix.de>, broonie@kernel.org Cc: fabio.estevam@freescale.com, mark.rutland@arm.com, marex@denx.de, pawel.moll@arm.com, ijc+devicetree@hellion.org.uk, dbaryshkov@gmail.com, viresh.kumar@linaro.org, linux-pm@vger.kernel.org, rjw@rjwysocki.net, lgirdwood@gmail.com, sre@kernel.org, devicetree@vger.kernel.org, robh+dt@kernel.org, kernel@pengutronix.de, galak@codeaurora.org, sebastien.szymanski@armadeus.com, shawn.guo@linaro.org, dwmw2@infradead.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 5/7] ARM: dts: enable regulator support for i.MX23/i.MX28 Date: Mon, 23 Mar 2015 18:54:34 +0100 (CET) [thread overview] Message-ID: <1469391177.549604.1427133274978.JavaMail.open-xchange@oxbaltgw02.schlund.de> (raw) In-Reply-To: <20150323060706.GB4946@pengutronix.de> Hi Sascha, > Sascha Hauer <s.hauer@pengutronix.de> hat am 23. März 2015 um 07:07 > geschrieben: > > > On Sun, Mar 22, 2015 at 12:30:01AM +0000, Stefan Wahren wrote: > > This patch enables the on-chip regulator support for all i.MX23 and > > i.MX28 boards. > > > > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> > > --- > > arch/arm/boot/dts/imx23.dtsi | 67 +++++++++++++++++++++++++++++++++++++++++- > > arch/arm/boot/dts/imx28.dtsi | 67 +++++++++++++++++++++++++++++++++++++++++- > > 2 files changed, 132 insertions(+), 2 deletions(-) > > > > diff --git a/arch/arm/boot/dts/imx23.dtsi b/arch/arm/boot/dts/imx23.dtsi > > index bbcfb5a..be0aee8 100644 > > --- a/arch/arm/boot/dts/imx23.dtsi > > +++ b/arch/arm/boot/dts/imx23.dtsi > > @@ -404,8 +404,73 @@ > > }; > > > > power@80044000 { > > + compatible = "fsl,imx23-power"; > > + #address-cells = <1>; > > + #size-cells = <1>; > > reg = <0x80044000 0x2000>; > > - status = "disabled"; > > + interrupts = <3>; > > + ranges; > > + > > + dcdc: regulator@80044010 { > > + reg = <0x80044010 0x10>, > > + <0x80044090 0x10>, > > + <0x800440c0 0x10>; > > + reg-names = "base-address", > > + "misc-address", > > + "status-address"; > > + compatible = "fsl,imx23-dcdc"; > > + regulator-name = "dcdc"; > > + regulator-boot-on; > > + regulator-always-on; > > + }; > > It is very unusual to describe the regulators of a device on a register > level like this and to iomemap each register individually. I think you > should drop the registers here and put this knowledge into the driver > like (nearly?) all others do. do mean dropping the base address of the regulator, too? How would you implement it (bare register address or regmap or syscon), since there are overlapping register regions for different driver? @Mark: What's your opinion? Stefan > > Sascha > > -- > 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 | _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: stefan.wahren@i2se.com (Stefan Wahren) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 5/7] ARM: dts: enable regulator support for i.MX23/i.MX28 Date: Mon, 23 Mar 2015 18:54:34 +0100 (CET) [thread overview] Message-ID: <1469391177.549604.1427133274978.JavaMail.open-xchange@oxbaltgw02.schlund.de> (raw) In-Reply-To: <20150323060706.GB4946@pengutronix.de> Hi Sascha, > Sascha Hauer <s.hauer@pengutronix.de> hat am 23. M?rz 2015 um 07:07 > geschrieben: > > > On Sun, Mar 22, 2015 at 12:30:01AM +0000, Stefan Wahren wrote: > > This patch enables the on-chip regulator support for all i.MX23 and > > i.MX28 boards. > > > > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> > > --- > > arch/arm/boot/dts/imx23.dtsi | 67 +++++++++++++++++++++++++++++++++++++++++- > > arch/arm/boot/dts/imx28.dtsi | 67 +++++++++++++++++++++++++++++++++++++++++- > > 2 files changed, 132 insertions(+), 2 deletions(-) > > > > diff --git a/arch/arm/boot/dts/imx23.dtsi b/arch/arm/boot/dts/imx23.dtsi > > index bbcfb5a..be0aee8 100644 > > --- a/arch/arm/boot/dts/imx23.dtsi > > +++ b/arch/arm/boot/dts/imx23.dtsi > > @@ -404,8 +404,73 @@ > > }; > > > > power at 80044000 { > > + compatible = "fsl,imx23-power"; > > + #address-cells = <1>; > > + #size-cells = <1>; > > reg = <0x80044000 0x2000>; > > - status = "disabled"; > > + interrupts = <3>; > > + ranges; > > + > > + dcdc: regulator at 80044010 { > > + reg = <0x80044010 0x10>, > > + <0x80044090 0x10>, > > + <0x800440c0 0x10>; > > + reg-names = "base-address", > > + "misc-address", > > + "status-address"; > > + compatible = "fsl,imx23-dcdc"; > > + regulator-name = "dcdc"; > > + regulator-boot-on; > > + regulator-always-on; > > + }; > > It is very unusual to describe the regulators of a device on a register > level like this and to iomemap each register individually. I think you > should drop the registers here and put this knowledge into the driver > like (nearly?) all others do. do mean dropping the base address of the regulator, too? How would you implement it (bare register address or regmap or syscon), since there are overlapping register regions for different driver? @Mark: What's your opinion? Stefan > > Sascha > > -- > 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 |
next prev parent reply other threads:[~2015-03-23 17:54 UTC|newest] Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-03-22 0:29 [PATCH 0/7] power: enable cpufreq-dt support for i.MX23/i.MX28 Stefan Wahren 2015-03-22 0:29 ` Stefan Wahren 2015-03-22 0:29 ` [PATCH 1/7] DT: add binding for mxs power subsystem Stefan Wahren 2015-03-22 0:29 ` Stefan Wahren 2015-03-22 0:29 ` [PATCH 2/7] power: mxs_power: add driver " Stefan Wahren 2015-03-22 0:29 ` Stefan Wahren 2015-03-22 10:40 ` Sebastian Reichel 2015-03-22 10:40 ` Sebastian Reichel [not found] ` <1426984203-9133-1-git-send-email-stefan.wahren-eS4NqCHxEME@public.gmane.org> 2015-03-22 0:29 ` [PATCH 3/7] DT: add binding for MXS regulator Stefan Wahren 2015-03-22 0:29 ` Stefan Wahren 2015-03-22 0:30 ` [PATCH 4/7] regulator: add mxs regulator driver Stefan Wahren 2015-03-22 0:30 ` Stefan Wahren 2015-03-22 16:14 ` Mark Brown 2015-03-22 16:14 ` Mark Brown 2015-03-23 17:45 ` Stefan Wahren 2015-03-23 17:45 ` Stefan Wahren 2015-03-23 18:39 ` Mark Brown 2015-03-23 18:39 ` Mark Brown 2015-03-22 0:30 ` [PATCH 5/7] ARM: dts: enable regulator support for i.MX23/i.MX28 Stefan Wahren 2015-03-22 0:30 ` Stefan Wahren 2015-03-23 6:07 ` Sascha Hauer 2015-03-23 6:07 ` Sascha Hauer 2015-03-23 17:54 ` Stefan Wahren [this message] 2015-03-23 17:54 ` Stefan Wahren 2015-03-23 18:37 ` Mark Brown 2015-03-23 18:37 ` Mark Brown [not found] ` <20150323183740.GI14954-GFdadSzt00ze9xe1eoZjHA@public.gmane.org> 2015-03-24 6:45 ` Stefan Wahren 2015-03-24 6:45 ` Stefan Wahren 2015-03-24 17:01 ` Mark Brown 2015-03-24 17:01 ` Mark Brown 2015-03-24 6:12 ` Sascha Hauer 2015-03-24 6:12 ` Sascha Hauer 2015-03-22 0:30 ` [PATCH 6/7] ARM: mxs: register cpufreq-dt in pm init Stefan Wahren 2015-03-22 0:30 ` Stefan Wahren 2015-03-22 0:30 ` [PATCH 7/7] ARM: dts: add OPPs for i.MX23/i.MX28 Stefan Wahren 2015-03-22 0:30 ` Stefan Wahren 2015-03-24 20:45 ` Juergen Borleis 2015-03-24 20:45 ` Juergen Borleis [not found] ` <201503242145.26370.juergen-vozXLyro3r7AVMDmWPUVSw@public.gmane.org> 2015-03-25 20:20 ` Stefan Wahren 2015-03-25 20:20 ` Stefan Wahren
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=1469391177.549604.1427133274978.JavaMail.open-xchange@oxbaltgw02.schlund.de \ --to=stefan.wahren@i2se.com \ --cc=broonie@kernel.org \ --cc=dbaryshkov@gmail.com \ --cc=devicetree@vger.kernel.org \ --cc=dwmw2@infradead.org \ --cc=fabio.estevam@freescale.com \ --cc=galak@codeaurora.org \ --cc=ijc+devicetree@hellion.org.uk \ --cc=kernel@pengutronix.de \ --cc=lgirdwood@gmail.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-pm@vger.kernel.org \ --cc=marex@denx.de \ --cc=mark.rutland@arm.com \ --cc=pawel.moll@arm.com \ --cc=rjw@rjwysocki.net \ --cc=robh+dt@kernel.org \ --cc=s.hauer@pengutronix.de \ --cc=sebastien.szymanski@armadeus.com \ --cc=shawn.guo@linaro.org \ --cc=sre@kernel.org \ --cc=viresh.kumar@linaro.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: linkBe 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.