From: Eddie Huang <[email protected]>
The MT8173 eval board contains a MT6397 PMIC. This adds the
corresponding device node to the dts file.
Signed-off-by: Henry Chen <[email protected]>
Signed-off-by: Eddie Huang <[email protected]>
---
Base on 4.1-rc1 and commit as following,
ARM64: dts: Mediatek: MT8173 updates [1] and
clk: Add common clock support for Mediatek MT8135 and MT8173 [2] patch
[1] http://lists.infradead.org/pipermail/linux-arm-kernel/2015-May/344751.html
[2] http://lists.infradead.org/pipermail/linux-arm-kernel/2015-April/338763.html
---
arch/arm64/boot/dts/mediatek/mt8173-evb.dts | 174 ++++++++++++++++++++++++++++
1 file changed, 174 insertions(+)
diff --git a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
index 43d5401..b967914 100644
--- a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
+++ b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
@@ -33,6 +33,180 @@
chosen { };
};
+&pwrap {
+ pmic: mt6397 {
+ compatible = "mediatek,mt6397";
+ interrupt-parent = <&pio>;
+ interrupts = <11 IRQ_TYPE_LEVEL_HIGH>;
+ interrupt-controller;
+ #interrupt-cells = <2>;
+
+ mt6397regulator: mt6397regulator {
+ compatible = "mediatek,mt6397-regulator";
+ mt6397_vpca15_reg: buck_vpca15 {
+ regulator-compatible = "buck_vpca15";
+ regulator-name = "vpca15";
+ regulator-min-microvolt = < 700000>;
+ regulator-max-microvolt = <1350000>;
+ regulator-ramp-delay = <12500>;
+ regulator-always-on;
+ };
+ mt6397_vpca7_reg: buck_vpca7 {
+ regulator-compatible = "buck_vpca7";
+ regulator-name = "vpca7";
+ regulator-min-microvolt = < 700000>;
+ regulator-max-microvolt = <1350000>;
+ regulator-ramp-delay = <12500>;
+ regulator-enable-ramp-delay = <115>;
+ };
+ mt6397_vsramca15_reg: buck_vsramca15 {
+ regulator-compatible = "buck_vsramca15";
+ regulator-name = "vsramca15";
+ regulator-min-microvolt = < 700000>;
+ regulator-max-microvolt = <1350000>;
+ regulator-ramp-delay = <12500>;
+ regulator-always-on;
+ };
+ mt6397_vsramca7_reg: buck_vsramca7 {
+ regulator-compatible = "buck_vsramca7";
+ regulator-name = "vsramca7";
+ regulator-min-microvolt = < 700000>;
+ regulator-max-microvolt = <1350000>;
+ regulator-ramp-delay = <12500>;
+ regulator-always-on;
+ };
+ mt6397_vcore_reg: buck_vcore {
+ regulator-compatible = "buck_vcore";
+ regulator-name = "vcore";
+ regulator-min-microvolt = < 700000>;
+ regulator-max-microvolt = <1350000>;
+ regulator-ramp-delay = <12500>;
+ regulator-always-on;
+ };
+ mt6397_vgpu_reg: buck_vgpu {
+ regulator-compatible = "buck_vgpu";
+ regulator-name = "vgpu";
+ regulator-min-microvolt = < 700000>;
+ regulator-max-microvolt = <1350000>;
+ regulator-ramp-delay = <12500>;
+ regulator-enable-ramp-delay = <115>;
+ };
+ mt6397_vdrm_reg: buck_vdrm {
+ regulator-compatible = "buck_vdrm";
+ regulator-name = "vdrm";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <1400000>;
+ regulator-ramp-delay = <12500>;
+ regulator-always-on;
+ };
+ mt6397_vio18_reg: buck_vio18 {
+ regulator-compatible = "buck_vio18";
+ regulator-name = "vio18";
+ regulator-min-microvolt = <1620000>;
+ regulator-max-microvolt = <1980000>;
+ regulator-ramp-delay = <12500>;
+ regulator-always-on;
+ };
+ mt6397_vtcxo_reg: ldo_vtcxo {
+ regulator-compatible = "ldo_vtcxo";
+ regulator-name = "vtcxo";
+ regulator-always-on;
+ };
+ mt6397_va28_reg: ldo_va28 {
+ regulator-compatible = "ldo_va28";
+ regulator-name = "va28";
+ regulator-always-on;
+ };
+ mt6397_vcama_reg: ldo_vcama {
+ regulator-compatible = "ldo_vcama";
+ regulator-name = "vcama";
+ regulator-min-microvolt = <1500000>;
+ regulator-max-microvolt = <2800000>;
+ regulator-enable-ramp-delay = <218>;
+ };
+ mt6397_vio28_reg: ldo_vio28 {
+ regulator-compatible = "ldo_vio28";
+ regulator-name = "vio28";
+ regulator-always-on;
+ };
+ mt6397_vusb_reg: ldo_vusb {
+ regulator-compatible = "ldo_vusb";
+ regulator-name = "vusb";
+ };
+ mt6397_vmc_reg: ldo_vmc {
+ regulator-compatible = "ldo_vmc";
+ regulator-name = "vmc";
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-enable-ramp-delay = <218>;
+ };
+ mt6397_vmch_reg: ldo_vmch {
+ regulator-compatible = "ldo_vmch";
+ regulator-name = "vmch";
+ regulator-min-microvolt = <3000000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-enable-ramp-delay = <218>;
+ };
+ mt6397_vemc_3v3_reg: ldo_vemc3v3 {
+ regulator-compatible = "ldo_vemc3v3";
+ regulator-name = "vemc_3v3";
+ regulator-min-microvolt = <3000000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-enable-ramp-delay = <218>;
+ };
+ mt6397_vgp1_reg: ldo_vgp1 {
+ regulator-compatible = "ldo_vgp1";
+ regulator-name = "vcamd";
+ regulator-min-microvolt = <1220000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-enable-ramp-delay = <240>;
+ };
+ mt6397_vgp2_reg: ldo_vgp2 {
+ regulator-compatible = "ldo_vgp2";
+ regulator-name = "vcamio";
+ regulator-min-microvolt = <1000000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-enable-ramp-delay = <218>;
+ };
+ mt6397_vgp3_reg: ldo_vgp3 {
+ regulator-compatible = "ldo_vgp3";
+ regulator-name = "vcamaf";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-enable-ramp-delay = <218>;
+ };
+ mt6397_vgp4_reg: ldo_vgp4 {
+ regulator-compatible = "ldo_vgp4";
+ regulator-name = "vgp4";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-enable-ramp-delay = <218>;
+ };
+ mt6397_vgp5_reg: ldo_vgp5 {
+ regulator-compatible = "ldo_vgp5";
+ regulator-name = "vgp5";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <3000000>;
+ regulator-enable-ramp-delay = <218>;
+ };
+ mt6397_vgp6_reg: ldo_vgp6 {
+ regulator-compatible = "ldo_vgp6";
+ regulator-name = "vgp6";
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-enable-ramp-delay = <218>;
+ };
+ mt6397_vibr_reg: ldo_vibr {
+ regulator-compatible = "ldo_vibr";
+ regulator-name = "vibr";
+ regulator-min-microvolt = <1300000>;
+ regulator-max-microvolt = <3300000>;
+ regulator-enable-ramp-delay = <218>;
+ };
+ };
+ };
+};
+
&uart0 {
status = "okay";
};
--
1.8.1.1.dirty
Hi Matthias,
On Mon, 2015-06-22 at 21:19 +0800, Henry Chen wrote:
> From: Eddie Huang <[email protected]>
>
> The MT8173 eval board contains a MT6397 PMIC. This adds the
> corresponding device node to the dts file.
>
> Signed-off-by: Henry Chen <[email protected]>
> Signed-off-by: Eddie Huang <[email protected]>
> ---
> Base on 4.1-rc1 and commit as following,
> ARM64: dts: Mediatek: MT8173 updates [1] and
> clk: Add common clock support for Mediatek MT8135 and MT8173 [2] patch
>
> [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2015-May/344751.html
> [2] http://lists.infradead.org/pipermail/linux-arm-kernel/2015-April/338763.html
> ---
> arch/arm64/boot/dts/mediatek/mt8173-evb.dts | 174 ++++++++++++++++++++++++++++
> 1 file changed, 174 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
> index 43d5401..b967914 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
> @@ -33,6 +33,180 @@
> chosen { };
> };
>
> +&pwrap {
> + pmic: mt6397 {
> + compatible = "mediatek,mt6397";
> + interrupt-parent = <&pio>;
> + interrupts = <11 IRQ_TYPE_LEVEL_HIGH>;
> + interrupt-controller;
> + #interrupt-cells = <2>;
> +
> + mt6397regulator: mt6397regulator {
> + compatible = "mediatek,mt6397-regulator";
> + mt6397_vpca15_reg: buck_vpca15 {
> + regulator-compatible = "buck_vpca15";
> + regulator-name = "vpca15";
> + regulator-min-microvolt = < 700000>;
> + regulator-max-microvolt = <1350000>;
> + regulator-ramp-delay = <12500>;
> + regulator-always-on;
I see you pull da9211 regulator dts patch, but not this one. How about
this patch ?
Thanks
Eddie
On Monday, July 06, 2015 03:43:57 PM Eddie Huang wrote:
> Hi Matthias,
>
> On Mon, 2015-06-22 at 21:19 +0800, Henry Chen wrote:
> > From: Eddie Huang <[email protected]>
> >
> > The MT8173 eval board contains a MT6397 PMIC. This adds the
> > corresponding device node to the dts file.
> >
> > Signed-off-by: Henry Chen <[email protected]>
> > Signed-off-by: Eddie Huang <[email protected]>
> > ---
> > Base on 4.1-rc1 and commit as following,
> > ARM64: dts: Mediatek: MT8173 updates [1] and
> > clk: Add common clock support for Mediatek MT8135 and MT8173 [2] patch
> >
> > [1]
> > http://lists.infradead.org/pipermail/linux-arm-kernel/2015-May/344751.htm
> > l [2]
> > http://lists.infradead.org/pipermail/linux-arm-kernel/2015-April/338763.h
> > tml ---
> >
> > arch/arm64/boot/dts/mediatek/mt8173-evb.dts | 174
> > ++++++++++++++++++++++++++++ 1 file changed, 174 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
> > b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts index 43d5401..b967914
> > 100644
> > --- a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
> > +++ b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
> > @@ -33,6 +33,180 @@
> >
> > chosen { };
> >
> > };
> >
> > +&pwrap {
> > + pmic: mt6397 {
> > + compatible = "mediatek,mt6397";
> > + interrupt-parent = <&pio>;
> > + interrupts = <11 IRQ_TYPE_LEVEL_HIGH>;
> > + interrupt-controller;
> > + #interrupt-cells = <2>;
> > +
> > + mt6397regulator: mt6397regulator {
> > + compatible = "mediatek,mt6397-regulator";
> > + mt6397_vpca15_reg: buck_vpca15 {
> > + regulator-compatible = "buck_vpca15";
> > + regulator-name = "vpca15";
> > + regulator-min-microvolt = < 700000>;
> > + regulator-max-microvolt = <1350000>;
> > + regulator-ramp-delay = <12500>;
> > + regulator-always-on;
>
> I see you pull da9211 regulator dts patch, but not this one. How about
> this patch ?
I inserted some new-lines and added it to v4.2-next/arm64
Thanks,
Matthias
On Mon, 2015-07-06 at 18:39 +0200, Matthias Brugger wrote:
> On Monday, July 06, 2015 03:43:57 PM Eddie Huang wrote:
> > On Mon, 2015-06-22 at 21:19 +0800, Henry Chen wrote:
> > > From: Eddie Huang <[email protected]>
> > >
[...]
> > > diff --git a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
> > > b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts index 43d5401..b967914
> > > 100644
> > > --- a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
> > > +++ b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
> > > @@ -33,6 +33,180 @@
> > >
> > > chosen { };
> > >
> > > };
> > >
> > > +&pwrap {
> > > + pmic: mt6397 {
> > > + compatible = "mediatek,mt6397";
> > > + interrupt-parent = <&pio>;
> > > + interrupts = <11 IRQ_TYPE_LEVEL_HIGH>;
> > > + interrupt-controller;
> > > + #interrupt-cells = <2>;
> > > +
> > > + mt6397regulator: mt6397regulator {
> > > + compatible = "mediatek,mt6397-regulator";
> > > + mt6397_vpca15_reg: buck_vpca15 {
> > > + regulator-compatible = "buck_vpca15";
> > > + regulator-name = "vpca15";
> > > + regulator-min-microvolt = < 700000>;
> > > + regulator-max-microvolt = <1350000>;
> > > + regulator-ramp-delay = <12500>;
> > > + regulator-always-on;
> >
> > I see you pull da9211 regulator dts patch, but not this one. How about
> > this patch ?
>
> I inserted some new-lines and added it to v4.2-next/arm64
>
Thanks the pull. Normally we put alias node in alphabetical order, make
later merge more easily. I think you can do the modification and squeeze
into the same commit.
Thanks
Eddie
On Tuesday, July 07, 2015 11:09:50 AM Eddie Huang wrote:
> On Mon, 2015-07-06 at 18:39 +0200, Matthias Brugger wrote:
> > On Monday, July 06, 2015 03:43:57 PM Eddie Huang wrote:
> > > On Mon, 2015-06-22 at 21:19 +0800, Henry Chen wrote:
> > > > From: Eddie Huang <[email protected]>
>
> [...]
>
> > > > diff --git a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
> > > > b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts index 43d5401..b967914
> > > > 100644
> > > > --- a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
> > > > +++ b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
> > > > @@ -33,6 +33,180 @@
> > > >
> > > > chosen { };
> > > >
> > > > };
> > > >
> > > > +&pwrap {
> > > > + pmic: mt6397 {
> > > > + compatible = "mediatek,mt6397";
> > > > + interrupt-parent = <&pio>;
> > > > + interrupts = <11 IRQ_TYPE_LEVEL_HIGH>;
> > > > + interrupt-controller;
> > > > + #interrupt-cells = <2>;
> > > > +
> > > > + mt6397regulator: mt6397regulator {
> > > > + compatible = "mediatek,mt6397-regulator";
> > > > + mt6397_vpca15_reg: buck_vpca15 {
> > > > + regulator-compatible = "buck_vpca15";
> > > > + regulator-name = "vpca15";
> > > > + regulator-min-microvolt = < 700000>;
> > > > + regulator-max-microvolt = <1350000>;
> > > > + regulator-ramp-delay = <12500>;
> > > > + regulator-always-on;
> > >
> > > I see you pull da9211 regulator dts patch, but not this one. How about
> > > this patch ?
> >
> > I inserted some new-lines and added it to v4.2-next/arm64
>
> Thanks the pull. Normally we put alias node in alphabetical order, make
> later merge more easily. I think you can do the modification and squeeze
> into the same commit.
>
I used the order of the dtsi file, but I think your approach is more straight
forward. I will change that.
Thanks for your comment.
Matthias