* [PATCH] arm64: dts: meson-gxl: add support for phicomm n1
@ 2018-10-13 9:45 yangxuan8282 at gmail.com
2018-10-28 11:32 ` Martin Blumenstingl
0 siblings, 1 reply; 3+ messages in thread
From: yangxuan8282 at gmail.com @ 2018-10-13 9:45 UTC (permalink / raw)
To: linus-amlogic
From: He Yangxuan <yangxuan8282@gmail.com>
This patch adds support for the Phicomm N1. This device based on P230 reference design.
The phy is RTL8211F, need to disable Energy Efficient Ethernet (EEE) to make it stable.
And this box doesn't have cvbs, so disable related section in device tree.
Signed-off-by: He Yangxuan <yangxuan8282@gmail.com>
---
arch/arm64/boot/dts/amlogic/Makefile | 1 +
.../dts/amlogic/meson-gxl-s905d-phicomm-n1.dts | 32 ++++++++++++++++++++++
2 files changed, 33 insertions(+)
create mode 100644 arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dts
diff --git a/arch/arm64/boot/dts/amlogic/Makefile b/arch/arm64/boot/dts/amlogic/Makefile
index a97c0e2d7..ebd96da2c 100644
--- a/arch/arm64/boot/dts/amlogic/Makefile
+++ b/arch/arm64/boot/dts/amlogic/Makefile
@@ -17,6 +17,7 @@ dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905x-nexbox-a95x.dtb
dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905x-p212.dtb
dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905d-p230.dtb
dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905d-p231.dtb
+dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905d-phicomm-n1.dtb
dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s805x-p241.dtb
dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905w-p281.dtb
dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905w-tx3-mini.dtb
diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dts b/arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dts
new file mode 100644
index 000000000..1daf58df0
--- /dev/null
+++ b/arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dts
@@ -0,0 +1,32 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * Copyright (c) 2018 He Yangxuan
+ */
+
+/dts-v1/;
+
+#include "meson-gxl-s905d-p230.dts"
+
+/ {
+ compatible = "phicomm,n1", "amlogic,s905d", "amlogic,meson-gxl";
+ model = "Phicomm N1";
+
+ cvbs-connector {
+ status = "disabled";
+ };
+};
+
+&cvbs_vdac_port {
+ status = "disabled";
+};
+
+&external_mdio {
+ external_phy: ethernet-phy at 0 {
+ compatible = "ethernet-phy-id001c.c916", "ethernet-phy-ieee802.3-c22";
+ reg = <0>;
+ eee-broken-1000t;
+ max-speed = <1000>;
+ interrupt-parent = <&gpio_intc>;
+ interrupts = <29 IRQ_TYPE_LEVEL_LOW>;
+ };
+};
--
2.11.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] arm64: dts: meson-gxl: add support for phicomm n1
2018-10-13 9:45 [PATCH] arm64: dts: meson-gxl: add support for phicomm n1 yangxuan8282 at gmail.com
@ 2018-10-28 11:32 ` Martin Blumenstingl
2018-11-01 6:28 ` He Yangxuan
0 siblings, 1 reply; 3+ messages in thread
From: Martin Blumenstingl @ 2018-10-28 11:32 UTC (permalink / raw)
To: linus-amlogic
Hi,
many thanks for your patch!
On Sat, Oct 13, 2018 at 11:45 AM <yangxuan8282@gmail.com> wrote:
>
> From: He Yangxuan <yangxuan8282@gmail.com>
>
> This patch adds support for the Phicomm N1. This device based on P230 reference design.
> The phy is RTL8211F, need to disable Energy Efficient Ethernet (EEE) to make it stable.
> And this box doesn't have cvbs, so disable related section in device tree.
do you have any hint how to connect the serial console on the PCB?
> Signed-off-by: He Yangxuan <yangxuan8282@gmail.com>
> ---
> arch/arm64/boot/dts/amlogic/Makefile | 1 +
> .../dts/amlogic/meson-gxl-s905d-phicomm-n1.dts | 32 ++++++++++++++++++++++
> 2 files changed, 33 insertions(+)
> create mode 100644 arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dts
>
> diff --git a/arch/arm64/boot/dts/amlogic/Makefile b/arch/arm64/boot/dts/amlogic/Makefile
> index a97c0e2d7..ebd96da2c 100644
> --- a/arch/arm64/boot/dts/amlogic/Makefile
> +++ b/arch/arm64/boot/dts/amlogic/Makefile
> @@ -17,6 +17,7 @@ dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905x-nexbox-a95x.dtb
> dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905x-p212.dtb
> dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905d-p230.dtb
> dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905d-p231.dtb
> +dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905d-phicomm-n1.dtb
> dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s805x-p241.dtb
> dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905w-p281.dtb
> dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905w-tx3-mini.dtb
> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dts b/arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dts
> new file mode 100644
> index 000000000..1daf58df0
> --- /dev/null
> +++ b/arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dts
> @@ -0,0 +1,32 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> +/*
> + * Copyright (c) 2018 He Yangxuan
> + */
> +
> +/dts-v1/;
> +
> +#include "meson-gxl-s905d-p230.dts"
> +
> +/ {
> + compatible = "phicomm,n1", "amlogic,s905d", "amlogic,meson-gxl";
the "phicomm" vendor-prefix should be documented in
devicetree/bindings/vendor-prefixes.txt (in a separate pach)
additionally "phicomm,n1" should be documented in
Documentation/devicetree/bindings/arm/amlogic.txt (in a separate patch
as well)
> + model = "Phicomm N1";
> +
> + cvbs-connector {
> + status = "disabled";
> + };
> +};
> +
> +&cvbs_vdac_port {
> + status = "disabled";
> +};
> +
> +&external_mdio {
> + external_phy: ethernet-phy at 0 {
> + compatible = "ethernet-phy-id001c.c916", "ethernet-phy-ieee802.3-c22";
please don't use a "ethernet-phy-id..." compatible
see commit 7630ea4bda18df "Documentation: net: phy: improve
explanation when to specify the PHY ID" for the reason why (when the
documentation was updated we already used "ethernet-phy-id..." for
some boards)
> + reg = <0>;
> + eee-broken-1000t;
> + max-speed = <1000>;
> + interrupt-parent = <&gpio_intc>;
please add a comment here stating to which GPIO IRQ 29 belongs to (see
meson-gxm-khadas-vim2.dts for an example)
> + interrupts = <29 IRQ_TYPE_LEVEL_LOW>;
> + };
> +};
> --
> 2.11.0
>
>
> _______________________________________________
> linux-amlogic mailing list
> linux-amlogic at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-amlogic
Regards
Martin
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] arm64: dts: meson-gxl: add support for phicomm n1
2018-10-28 11:32 ` Martin Blumenstingl
@ 2018-11-01 6:28 ` He Yangxuan
0 siblings, 0 replies; 3+ messages in thread
From: He Yangxuan @ 2018-11-01 6:28 UTC (permalink / raw)
To: linus-amlogic
On Sun, Oct 28, 2018 at 12:32:14PM +0100, Martin Blumenstingl wrote:
> Hi,
>
> many thanks for your patch!
>
> On Sat, Oct 13, 2018 at 11:45 AM <yangxuan8282@gmail.com> wrote:
> >
> > From: He Yangxuan <yangxuan8282@gmail.com>
> >
> > This patch adds support for the Phicomm N1. This device based on P230 reference design.
> > The phy is RTL8211F, need to disable Energy Efficient Ethernet (EEE) to make it stable.
> > And this box doesn't have cvbs, so disable related section in device tree.
> do you have any hint how to connect the serial console on the PCB?
I did not connect to serial my self, but other user say it's possible to connect it, details here: http://www.right.com.cn/forum/thread-324996-1-1.html
> > Signed-off-by: He Yangxuan <yangxuan8282@gmail.com>
> > ---
> > arch/arm64/boot/dts/amlogic/Makefile | 1 +
> > .../dts/amlogic/meson-gxl-s905d-phicomm-n1.dts | 32 ++++++++++++++++++++++
> > 2 files changed, 33 insertions(+)
> > create mode 100644 arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dts
> >
> > diff --git a/arch/arm64/boot/dts/amlogic/Makefile b/arch/arm64/boot/dts/amlogic/Makefile
> > index a97c0e2d7..ebd96da2c 100644
> > --- a/arch/arm64/boot/dts/amlogic/Makefile
> > +++ b/arch/arm64/boot/dts/amlogic/Makefile
> > @@ -17,6 +17,7 @@ dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905x-nexbox-a95x.dtb
> > dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905x-p212.dtb
> > dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905d-p230.dtb
> > dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905d-p231.dtb
> > +dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905d-phicomm-n1.dtb
> > dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s805x-p241.dtb
> > dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905w-p281.dtb
> > dtb-$(CONFIG_ARCH_MESON) += meson-gxl-s905w-tx3-mini.dtb
> > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dts b/arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dts
> > new file mode 100644
> > index 000000000..1daf58df0
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/amlogic/meson-gxl-s905d-phicomm-n1.dts
> > @@ -0,0 +1,32 @@
> > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> > +/*
> > + * Copyright (c) 2018 He Yangxuan
> > + */
> > +
> > +/dts-v1/;
> > +
> > +#include "meson-gxl-s905d-p230.dts"
> > +
> > +/ {
> > + compatible = "phicomm,n1", "amlogic,s905d", "amlogic,meson-gxl";
> the "phicomm" vendor-prefix should be documented in
> devicetree/bindings/vendor-prefixes.txt (in a separate pach)
> additionally "phicomm,n1" should be documented in
> Documentation/devicetree/bindings/arm/amlogic.txt (in a separate patch
> as well)
I will submit separate patch later for vendor.
> > + model = "Phicomm N1";
> > +
> > + cvbs-connector {
> > + status = "disabled";
> > + };
> > +};
> > +
> > +&cvbs_vdac_port {
> > + status = "disabled";
> > +};
> > +
> > +&external_mdio {
> > + external_phy: ethernet-phy at 0 {
> > + compatible = "ethernet-phy-id001c.c916", "ethernet-phy-ieee802.3-c22";
> please don't use a "ethernet-phy-id..." compatible
> see commit 7630ea4bda18df "Documentation: net: phy: improve
> explanation when to specify the PHY ID" for the reason why (when the
> documentation was updated we already used "ethernet-phy-id..." for
> some boards)
>
> > + reg = <0>;
> > + eee-broken-1000t;
> > + max-speed = <1000>;
> > + interrupt-parent = <&gpio_intc>;
> please add a comment here stating to which GPIO IRQ 29 belongs to (see
> meson-gxm-khadas-vim2.dts for an example)
>
> > + interrupts = <29 IRQ_TYPE_LEVEL_LOW>;
> > + };
> > +};
> > --
> > 2.11.0
> >
> >
> > _______________________________________________
> > linux-amlogic mailing list
> > linux-amlogic at lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-amlogic
>
>
> Regards
> Martin
Neil Armstrong suggest me in IRC just keep different part from meson-gxl-s905d-p230.dts, so the external phy part should simple like this:
&external_mdio {
external_phy: ethernet-phy at 0 {
/* Realtek RTL8211F (0x001cc916) */
eee-broken-1000t;
};
};
Yangxuan
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-11-01 6:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-10-13 9:45 [PATCH] arm64: dts: meson-gxl: add support for phicomm n1 yangxuan8282 at gmail.com
2018-10-28 11:32 ` Martin Blumenstingl
2018-11-01 6:28 ` He Yangxuan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox