From: Maxime Ripard <maxime.ripard@free-electrons.com> To: Chen-Yu Tsai <wens@csie.org> Cc: linux-arm-kernel <linux-arm-kernel@lists.infradead.org>, Hans de Goede <hdegoede@redhat.com>, linux-kernel <linux-kernel@vger.kernel.org>, linux-sunxi <linux-sunxi@googlegroups.com> Subject: Re: [PATCH v3 5/5] ARM: sun5i: Add C.H.I.P DTS Date: Sun, 11 Oct 2015 19:20:05 +0200 [thread overview] Message-ID: <20151011172005.GV2278@lukather> (raw) In-Reply-To: <CAGb2v66JnJwDLYQ40ckYp_OTrLmprYKPhXrV1ME66fEeF=Vewg@mail.gmail.com> [-- Attachment #1: Type: text/plain, Size: 3453 bytes --] On Fri, Oct 09, 2015 at 11:22:23PM +0800, Chen-Yu Tsai wrote: > On Fri, Oct 9, 2015 at 4:42 PM, Maxime Ripard > <maxime.ripard@free-electrons.com> wrote: > > The C.H.I.P. is a small SBC with an Allwinner R8, 8GB of NAND, 512MB of > > RAM, USB host and OTG, a wifi / bluetooth combo chip, an audio/video jack > > and two connectors to plug additional boards on top of it. > > > > Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> > > Reviewed-by: Hans de Goede <hdegoede@redhat.com> > > --- > > arch/arm/boot/dts/Makefile | 3 +- > > arch/arm/boot/dts/sun5i-r8-chip.dts | 213 ++++++++++++++++++++++++++++++++++++ > > 2 files changed, 215 insertions(+), 1 deletion(-) > > create mode 100644 arch/arm/boot/dts/sun5i-r8-chip.dts > > > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > > index 342ab3116feb..bf165ed4e7fa 100644 > > --- a/arch/arm/boot/dts/Makefile > > +++ b/arch/arm/boot/dts/Makefile > > @@ -600,7 +600,8 @@ dtb-$(CONFIG_MACH_SUN5I) += \ > > sun5i-a13-olinuxino.dtb \ > > sun5i-a13-olinuxino-micro.dtb \ > > sun5i-a13-q8-tablet.dtb \ > > - sun5i-a13-utoo-p66.dtb > > + sun5i-a13-utoo-p66.dtb \ > > + sun5i-r8-chip.dtb > > dtb-$(CONFIG_MACH_SUN6I) += \ > > sun6i-a31-app4-evb1.dtb \ > > sun6i-a31-colombus.dtb \ > > diff --git a/arch/arm/boot/dts/sun5i-r8-chip.dts b/arch/arm/boot/dts/sun5i-r8-chip.dts > > new file mode 100644 > > index 000000000000..0d450a828372 > > --- /dev/null > > +++ b/arch/arm/boot/dts/sun5i-r8-chip.dts > > snip > > > +®_dcdc2 { > > + regulator-min-microvolt = <1000000>; > > + regulator-max-microvolt = <1400000>; > > + regulator-name = "cpuvdd"; > > Other boards seem to follow the power pin names on the SoC and call > this "vdd-cpu". > > > + regulator-always-on; > > +}; > > + > > +®_dcdc3 { > > + regulator-min-microvolt = <1000000>; > > + regulator-max-microvolt = <1300000>; > > + regulator-name = "corevdd"; > > And this was named "vdd-int" or "vdd-int-dll" (for SoCs with separate > VDD_DLL pins). > > > + regulator-always-on; > > +}; > > + > > +®_ldo1 { > > + regulator-name = "rtcvdd"; > > And this one was "vdd-rtc". > > I know you followed the names set in the design doc. Just wondering if there > should be some convention on these. I think if we have a document that clearly reference them with some other name, we should just stick with the name used there, especially if it's only cosmetic, which is the case here. > > +}; > > + > > +®_ldo2 { > > + regulator-min-microvolt = <2700000>; > > + regulator-max-microvolt = <3300000>; > > + regulator-name = "avcc"; > > + regulator-always-on; > > +}; > > + > > +®_ldo5 { > > + regulator-min-microvolt = <1800000>; > > + regulator-max-microvolt = <1800000>; > > + regulator-name = "vcc-1v8"; > > +}; > > + > > +®_usb0_vbus { > > + pinctrl-0 = <&chip_vbus_pin>; > > + vin-supply = <®_vcc5v0>; > > + gpio = <&pio 1 10 GPIO_ACTIVE_HIGH>; /* PB10 */ > > status = "okay"; ? Ah, yes, indeed. > The rest looks good. Is that an Ack from you if I add the status ? Thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: maxime.ripard@free-electrons.com (Maxime Ripard) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v3 5/5] ARM: sun5i: Add C.H.I.P DTS Date: Sun, 11 Oct 2015 19:20:05 +0200 [thread overview] Message-ID: <20151011172005.GV2278@lukather> (raw) In-Reply-To: <CAGb2v66JnJwDLYQ40ckYp_OTrLmprYKPhXrV1ME66fEeF=Vewg@mail.gmail.com> On Fri, Oct 09, 2015 at 11:22:23PM +0800, Chen-Yu Tsai wrote: > On Fri, Oct 9, 2015 at 4:42 PM, Maxime Ripard > <maxime.ripard@free-electrons.com> wrote: > > The C.H.I.P. is a small SBC with an Allwinner R8, 8GB of NAND, 512MB of > > RAM, USB host and OTG, a wifi / bluetooth combo chip, an audio/video jack > > and two connectors to plug additional boards on top of it. > > > > Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> > > Reviewed-by: Hans de Goede <hdegoede@redhat.com> > > --- > > arch/arm/boot/dts/Makefile | 3 +- > > arch/arm/boot/dts/sun5i-r8-chip.dts | 213 ++++++++++++++++++++++++++++++++++++ > > 2 files changed, 215 insertions(+), 1 deletion(-) > > create mode 100644 arch/arm/boot/dts/sun5i-r8-chip.dts > > > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > > index 342ab3116feb..bf165ed4e7fa 100644 > > --- a/arch/arm/boot/dts/Makefile > > +++ b/arch/arm/boot/dts/Makefile > > @@ -600,7 +600,8 @@ dtb-$(CONFIG_MACH_SUN5I) += \ > > sun5i-a13-olinuxino.dtb \ > > sun5i-a13-olinuxino-micro.dtb \ > > sun5i-a13-q8-tablet.dtb \ > > - sun5i-a13-utoo-p66.dtb > > + sun5i-a13-utoo-p66.dtb \ > > + sun5i-r8-chip.dtb > > dtb-$(CONFIG_MACH_SUN6I) += \ > > sun6i-a31-app4-evb1.dtb \ > > sun6i-a31-colombus.dtb \ > > diff --git a/arch/arm/boot/dts/sun5i-r8-chip.dts b/arch/arm/boot/dts/sun5i-r8-chip.dts > > new file mode 100644 > > index 000000000000..0d450a828372 > > --- /dev/null > > +++ b/arch/arm/boot/dts/sun5i-r8-chip.dts > > snip > > > +®_dcdc2 { > > + regulator-min-microvolt = <1000000>; > > + regulator-max-microvolt = <1400000>; > > + regulator-name = "cpuvdd"; > > Other boards seem to follow the power pin names on the SoC and call > this "vdd-cpu". > > > + regulator-always-on; > > +}; > > + > > +®_dcdc3 { > > + regulator-min-microvolt = <1000000>; > > + regulator-max-microvolt = <1300000>; > > + regulator-name = "corevdd"; > > And this was named "vdd-int" or "vdd-int-dll" (for SoCs with separate > VDD_DLL pins). > > > + regulator-always-on; > > +}; > > + > > +®_ldo1 { > > + regulator-name = "rtcvdd"; > > And this one was "vdd-rtc". > > I know you followed the names set in the design doc. Just wondering if there > should be some convention on these. I think if we have a document that clearly reference them with some other name, we should just stick with the name used there, especially if it's only cosmetic, which is the case here. > > +}; > > + > > +®_ldo2 { > > + regulator-min-microvolt = <2700000>; > > + regulator-max-microvolt = <3300000>; > > + regulator-name = "avcc"; > > + regulator-always-on; > > +}; > > + > > +®_ldo5 { > > + regulator-min-microvolt = <1800000>; > > + regulator-max-microvolt = <1800000>; > > + regulator-name = "vcc-1v8"; > > +}; > > + > > +®_usb0_vbus { > > + pinctrl-0 = <&chip_vbus_pin>; > > + vin-supply = <®_vcc5v0>; > > + gpio = <&pio 1 10 GPIO_ACTIVE_HIGH>; /* PB10 */ > > status = "okay"; ? Ah, yes, indeed. > The rest looks good. Is that an Ack from you if I add the status ? Thanks! 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: 819 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20151011/f9f0c737/attachment-0001.sig>
next prev parent reply other threads:[~2015-10-11 17:20 UTC|newest] Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-10-09 8:42 [PATCH v3 0/5] ARM: sunxi: Introduce CHIP support Maxime Ripard 2015-10-09 8:42 ` Maxime Ripard 2015-10-09 8:42 ` [PATCH v3 1/5] ARM: sunxi: Add R8 support Maxime Ripard 2015-10-09 8:42 ` Maxime Ripard 2015-10-09 14:20 ` Chen-Yu Tsai 2015-10-09 14:20 ` Chen-Yu Tsai 2015-10-11 17:08 ` Maxime Ripard 2015-10-11 17:08 ` Maxime Ripard 2015-10-09 8:42 ` [PATCH v3 2/5] ARM: sun5i: Add R8 DTSI Maxime Ripard 2015-10-09 8:42 ` Maxime Ripard 2015-10-09 8:42 ` [PATCH v3 3/5] ARM: sun5i: dt: Move uart3 pinctrl node to common DTSI Maxime Ripard 2015-10-09 8:42 ` Maxime Ripard 2015-10-09 14:22 ` Chen-Yu Tsai 2015-10-09 14:22 ` Chen-Yu Tsai 2015-10-11 17:09 ` Maxime Ripard 2015-10-11 17:09 ` Maxime Ripard 2015-10-09 8:42 ` [PATCH v3 4/5] ARM: sun5i: dt: Add UART3 CTS and RTS pins Maxime Ripard 2015-10-09 8:42 ` Maxime Ripard 2015-10-09 14:24 ` Chen-Yu Tsai 2015-10-09 14:24 ` Chen-Yu Tsai 2015-10-11 17:10 ` Maxime Ripard 2015-10-11 17:10 ` Maxime Ripard 2015-10-09 8:42 ` [PATCH v3 5/5] ARM: sun5i: Add C.H.I.P DTS Maxime Ripard 2015-10-09 8:42 ` Maxime Ripard 2015-10-09 15:22 ` Chen-Yu Tsai 2015-10-09 15:22 ` Chen-Yu Tsai 2015-10-11 17:20 ` Maxime Ripard [this message] 2015-10-11 17:20 ` Maxime Ripard 2015-10-12 2:20 ` Chen-Yu Tsai 2015-10-12 2:20 ` Chen-Yu Tsai 2015-10-12 9:32 ` Maxime Ripard 2015-10-12 9:32 ` Maxime Ripard
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=20151011172005.GV2278@lukather \ --to=maxime.ripard@free-electrons.com \ --cc=hdegoede@redhat.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-sunxi@googlegroups.com \ --cc=wens@csie.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.