* [PATCH v4 0/3] arm64: Add initial support for Odroid-N2 @ 2019-05-27 14:02 Neil Armstrong 2019-05-27 14:02 ` [PATCH v4 1/3] dt-bindings: arm: amlogic: add G12B bindings Neil Armstrong ` (2 more replies) 0 siblings, 3 replies; 8+ messages in thread From: Neil Armstrong @ 2019-05-27 14:02 UTC (permalink / raw) To: khilman; +Cc: Neil Armstrong, linux-amlogic, linux-arm-kernel, linux-kernel This patchset adds basic support for : - Amlogic G12B, which is very similar to G12A - The HardKernel Odroid-N2 based on the S922X SoC The Amlogic G12B SoC is very similar with the G12A SoC, sharing most of the features and architecture, but with these differences : - The first CPU cluster only has 2xCortex-A53 instead of 4 - G12B has a second cluster of 4xCortex-A73 - Both cluster can achieve 2GHz instead of 1,8GHz for G12A - CPU Clock architecture is difference, thus needing a different compatible to handle this slight difference - Supports a MIPI CSI input - Embeds a Mali-G52 instead of a Mali-G31, but integration is the same Actual support is done in the same way as for the GXM support, including the G12A dtsi and redefining the CPU clusters. Unlike GXM, the first cluster is different, thus needing to remove the last 2 cpu nodes of the first cluster. Dependencies : - Patch 1, 2 : YAML rewrite of amlogic.txt bindings at [7] - Patch 3 : None since clock + g12b bindings has been acked Changes since v3 at [8]: - Dropped arm,armv8 - Dropped eee disable, not needed in further tests - Add comments about where are connected the regulators pins - Moved the phy0 regulator to the usb vbus regulator Changes since v2 at [5]: - sent the clk patches in standalone at [6] - rewrote the bindings on top of the YAML rewrite at [7] - Added MMC, SDCard and Network support to Odroid-N2 Changes since v1 at [3]: - Renamed the g12b cpu clocks like g12a counterparts - Rebased clock patches on top of Guillaume's Temperature sensor Clock patches at [4] - Added AO-CEC-B support to N2 DTS Changes since RFC at [1]: - Added bindings review tags - Moved the fclk_div3 CRITICAL flags to the gate - Removed invalid CRITICAL flags on the cpu clocks [1] https://lkml.kernel.org/r/20190327103308.25058-1-narmstrong@baylibre.com [2] https://lkml.kernel.org/r/20190325145914.32391-1-narmstrong@baylibre.com [3] https://lkml.kernel.org/r/20190404150518.30337-1-narmstrong@baylibre.com [4] https://lkml.kernel.org/r/20190412100221.26740-1-glaroque@baylibre.com [5] https://lkml.kernel.org/r/20190423091503.10847-1-narmstrong@baylibre.com [6] https://lkml.kernel.org/r/20190521150130.31684-1-narmstrong@baylibre.com [7] https://lkml.kernel.org/r/20190517152723.28518-2-robh@kernel.org [8] https://lkml.kernel.org/r/20190521151952.2779-1-narmstrong@baylibre.com Neil Armstrong (3): dt-bindings: arm: amlogic: add G12B bindings dt-bindings: arm: amlogic: add Odroid-N2 binding arm64: dts: meson: Add minimal support for Odroid-N2 .../devicetree/bindings/arm/amlogic.yaml | 6 + arch/arm64/boot/dts/amlogic/Makefile | 1 + .../boot/dts/amlogic/meson-g12b-odroid-n2.dts | 288 ++++++++++++++++++ arch/arm64/boot/dts/amlogic/meson-g12b.dtsi | 82 +++++ 4 files changed, 377 insertions(+) create mode 100644 arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts create mode 100644 arch/arm64/boot/dts/amlogic/meson-g12b.dtsi -- 2.21.0 ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v4 1/3] dt-bindings: arm: amlogic: add G12B bindings 2019-05-27 14:02 [PATCH v4 0/3] arm64: Add initial support for Odroid-N2 Neil Armstrong @ 2019-05-27 14:02 ` Neil Armstrong 2019-05-27 14:02 ` [PATCH v4 2/3] dt-bindings: arm: amlogic: add Odroid-N2 binding Neil Armstrong 2019-05-27 14:02 ` [PATCH v4 3/3] arm64: dts: meson: Add minimal support for Odroid-N2 Neil Armstrong 2 siblings, 0 replies; 8+ messages in thread From: Neil Armstrong @ 2019-05-27 14:02 UTC (permalink / raw) To: khilman, devicetree Cc: Neil Armstrong, linux-amlogic, linux-arm-kernel, linux-kernel, Rob Herring, Martin Blumenstingl Add compatible for the Amlogic G12B SoC, sharing most of the features and architecture with the G12A SoC. Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> Reviewed-by: Rob Herring <robh@kernel.org> Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> --- Documentation/devicetree/bindings/arm/amlogic.yaml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/amlogic.yaml b/Documentation/devicetree/bindings/arm/amlogic.yaml index 6d5bb493db03..28115dd49f96 100644 --- a/Documentation/devicetree/bindings/arm/amlogic.yaml +++ b/Documentation/devicetree/bindings/arm/amlogic.yaml @@ -137,4 +137,8 @@ properties: - amlogic,u200 - const: amlogic,g12a + - description: Boards with the Amlogic Meson G12B S922X SoC + items: + - const: amlogic,g12b + ... -- 2.21.0 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v4 2/3] dt-bindings: arm: amlogic: add Odroid-N2 binding 2019-05-27 14:02 [PATCH v4 0/3] arm64: Add initial support for Odroid-N2 Neil Armstrong 2019-05-27 14:02 ` [PATCH v4 1/3] dt-bindings: arm: amlogic: add G12B bindings Neil Armstrong @ 2019-05-27 14:02 ` Neil Armstrong 2019-05-27 14:02 ` [PATCH v4 3/3] arm64: dts: meson: Add minimal support for Odroid-N2 Neil Armstrong 2 siblings, 0 replies; 8+ messages in thread From: Neil Armstrong @ 2019-05-27 14:02 UTC (permalink / raw) To: khilman, devicetree Cc: Neil Armstrong, linux-amlogic, linux-arm-kernel, linux-kernel, Rob Herring, Martin Blumenstingl Add compatible for the Amlogic G12B (S922X) SoC based Odroid-N2 SBC from HardKernel. Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> Reviewed-by: Rob Herring <robh@kernel.org> Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> --- Documentation/devicetree/bindings/arm/amlogic.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/amlogic.yaml b/Documentation/devicetree/bindings/arm/amlogic.yaml index 28115dd49f96..f75df4471c0a 100644 --- a/Documentation/devicetree/bindings/arm/amlogic.yaml +++ b/Documentation/devicetree/bindings/arm/amlogic.yaml @@ -139,6 +139,8 @@ properties: - description: Boards with the Amlogic Meson G12B S922X SoC items: + - enum: + - hardkernel,odroid-n2 - const: amlogic,g12b ... -- 2.21.0 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v4 3/3] arm64: dts: meson: Add minimal support for Odroid-N2 2019-05-27 14:02 [PATCH v4 0/3] arm64: Add initial support for Odroid-N2 Neil Armstrong 2019-05-27 14:02 ` [PATCH v4 1/3] dt-bindings: arm: amlogic: add G12B bindings Neil Armstrong 2019-05-27 14:02 ` [PATCH v4 2/3] dt-bindings: arm: amlogic: add Odroid-N2 binding Neil Armstrong @ 2019-05-27 14:02 ` Neil Armstrong 2019-05-27 18:12 ` Martin Blumenstingl 2019-05-29 10:08 ` Neil Armstrong 2 siblings, 2 replies; 8+ messages in thread From: Neil Armstrong @ 2019-05-27 14:02 UTC (permalink / raw) To: khilman; +Cc: Neil Armstrong, linux-amlogic, linux-arm-kernel, linux-kernel This patch adds basic support for : - Amlogic G12B, which is very similar to G12A - The HardKernel Odroid-N2 based on the S922X SoC The Amlogic G12B SoC is very similar with the G12A SoC, sharing most of the features and architecture, but with these differences : - The first CPU cluster only has 2xCortex-A53 instead of 4 - G12B has a second cluster of 4xCortex-A73 - Both cluster can achieve 2GHz instead of 1,8GHz for G12A - CPU Clock architecture is difference, thus needing a different compatible to handle this slight difference - Supports a MIPI CSI input - Embeds a Mali-G52 instead of a Mali-G31, but integration is the same Actual support is done in the same way as for the GXM support, including the G12A dtsi and redefining the CPU clusters. Unlike GXM, the first cluster is different, thus needing to remove the last 2 cpu nodes of the first cluster. Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> --- arch/arm64/boot/dts/amlogic/Makefile | 1 + .../boot/dts/amlogic/meson-g12b-odroid-n2.dts | 288 ++++++++++++++++++ arch/arm64/boot/dts/amlogic/meson-g12b.dtsi | 82 +++++ 3 files changed, 371 insertions(+) create mode 100644 arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts create mode 100644 arch/arm64/boot/dts/amlogic/meson-g12b.dtsi diff --git a/arch/arm64/boot/dts/amlogic/Makefile b/arch/arm64/boot/dts/amlogic/Makefile index e129c03ced14..07b861fe5fa5 100644 --- a/arch/arm64/boot/dts/amlogic/Makefile +++ b/arch/arm64/boot/dts/amlogic/Makefile @@ -3,6 +3,7 @@ dtb-$(CONFIG_ARCH_MESON) += meson-axg-s400.dtb dtb-$(CONFIG_ARCH_MESON) += meson-g12a-sei510.dtb dtb-$(CONFIG_ARCH_MESON) += meson-g12a-u200.dtb dtb-$(CONFIG_ARCH_MESON) += meson-g12a-x96-max.dtb +dtb-$(CONFIG_ARCH_MESON) += meson-g12b-odroid-n2.dtb dtb-$(CONFIG_ARCH_MESON) += meson-gxbb-nanopi-k2.dtb dtb-$(CONFIG_ARCH_MESON) += meson-gxbb-nexbox-a95x.dtb dtb-$(CONFIG_ARCH_MESON) += meson-gxbb-odroidc2.dtb diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts new file mode 100644 index 000000000000..48783ead8dfb --- /dev/null +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts @@ -0,0 +1,288 @@ +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) +/* + * Copyright (c) 2019 BayLibre, SAS + * Author: Neil Armstrong <narmstrong@baylibre.com> + */ + +/dts-v1/; + +#include "meson-g12b.dtsi" +#include <dt-bindings/input/input.h> +#include <dt-bindings/gpio/meson-g12a-gpio.h> + +/ { + compatible = "hardkernel,odroid-n2", "amlogic,g12b"; + model = "Hardkernel ODROID-N2"; + + aliases { + serial0 = &uart_AO; + ethernet0 = ðmac; + }; + + chosen { + stdout-path = "serial0:115200n8"; + }; + + memory@0 { + device_type = "memory"; + reg = <0x0 0x0 0x0 0x40000000>; + }; + + emmc_pwrseq: emmc-pwrseq { + compatible = "mmc-pwrseq-emmc"; + reset-gpios = <&gpio BOOT_12 GPIO_ACTIVE_LOW>; + }; + + leds { + compatible = "gpio-leds"; + + blue { + label = "n2:blue"; + gpios = <&gpio_ao GPIOAO_11 GPIO_ACTIVE_HIGH>; + linux,default-trigger = "heartbeat"; + }; + }; + + tflash_vdd: regulator-tflash_vdd { + compatible = "regulator-fixed"; + + regulator-name = "TFLASH_VDD"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + + gpio = <&gpio_ao GPIOAO_8 GPIO_ACTIVE_HIGH>; + enable-active-high; + }; + + tf_io: gpio-regulator-tf_io { + compatible = "regulator-gpio"; + + regulator-name = "TF_IO"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <3300000>; + + gpios = <&gpio_ao GPIOAO_9 GPIO_ACTIVE_HIGH>; + gpios-states = <0>; + + states = <3300000 0 + 1800000 1>; + }; + + flash_1v8: regulator-flash_1v8 { + compatible = "regulator-fixed"; + regulator-name = "FLASH_1V8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + vin-supply = <&vcc_3v3>; + regulator-always-on; + }; + + main_12v: regulator-main_12v { + compatible = "regulator-fixed"; + regulator-name = "12V"; + regulator-min-microvolt = <12000000>; + regulator-max-microvolt = <12000000>; + regulator-always-on; + }; + + vcc_5v: regulator-vcc_5v { + compatible = "regulator-fixed"; + regulator-name = "5V"; + regulator-min-microvolt = <5000000>; + regulator-max-microvolt = <5000000>; + regulator-always-on; + }; + + vcc_1v8: regulator-vcc_1v8 { + compatible = "regulator-fixed"; + regulator-name = "VCC_1V8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + vin-supply = <&vcc_3v3>; + regulator-always-on; + }; + + vcc_3v3: regulator-vcc_3v3 { + compatible = "regulator-fixed"; + regulator-name = "VCC_3V3"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + vin-supply = <&vddao_3v3>; + regulator-always-on; + /* FIXME: actually controlled by VDDCPU_B_EN */ + }; + + hub_5v: regulator-hub_5v { + compatible = "regulator-fixed"; + regulator-name = "HUB_5V"; + regulator-min-microvolt = <5000000>; + regulator-max-microvolt = <5000000>; + vin-supply = <&vcc_5v>; + + /* Connected to the Hub CHIPENABLE, LOW sets low power state */ + gpio = <&gpio GPIOH_5 GPIO_ACTIVE_HIGH>; + enable-active-high; + }; + + usb_pwr_en: regulator-usb_pwr_en { + compatible = "regulator-fixed"; + regulator-name = "USB_PWR_EN"; + regulator-min-microvolt = <5000000>; + regulator-max-microvolt = <5000000>; + vin-supply = <&hub_5v>; + + /* Connected to the microUSB port power enable */ + gpio = <&gpio GPIOH_6 GPIO_ACTIVE_HIGH>; + enable-active-high; + }; + + vddao_1v8: regulator-vddao_1v8 { + compatible = "regulator-fixed"; + regulator-name = "VDDAO_1V8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + vin-supply = <&vddao_3v3>; + regulator-always-on; + }; + + vddao_3v3: regulator-vddao_3v3 { + compatible = "regulator-fixed"; + regulator-name = "VDDAO_3V3"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + vin-supply = <&main_12v>; + regulator-always-on; + }; + + hdmi-connector { + compatible = "hdmi-connector"; + type = "a"; + + port { + hdmi_connector_in: endpoint { + remote-endpoint = <&hdmi_tx_tmds_out>; + }; + }; + }; +}; + +&cec_AO { + pinctrl-0 = <&cec_ao_a_h_pins>; + pinctrl-names = "default"; + status = "disabled"; + hdmi-phandle = <&hdmi_tx>; +}; + +&cecb_AO { + pinctrl-0 = <&cec_ao_b_h_pins>; + pinctrl-names = "default"; + status = "okay"; + hdmi-phandle = <&hdmi_tx>; +}; + +&ext_mdio { + external_phy: ethernet-phy@0 { + /* Realtek RTL8211F (0x001cc916) */ + reg = <0>; + max-speed = <1000>; + }; +}; + +ðmac { + pinctrl-0 = <ð_pins>, <ð_rgmii_pins>; + pinctrl-names = "default"; + status = "okay"; + phy-mode = "rgmii"; + phy-handle = <&external_phy>; + amlogic,tx-delay-ns = <2>; +}; + +&gpio { + /* + * WARNING: The USB Hub on the Odroid-N2 needs a reset signal + * to be turned high in order to be detected by the USB Controller + * This signal should be handled by a USB specific power sequence + * in order to reset the Hub when USB bus is powered down. + */ + usb-hub { + gpio-hog; + gpios = <GPIOH_4 GPIO_ACTIVE_HIGH>; + output-high; + line-name = "usb-hub-reset"; + }; +}; + +&hdmi_tx { + status = "okay"; + pinctrl-0 = <&hdmitx_hpd_pins>, <&hdmitx_ddc_pins>; + pinctrl-names = "default"; + hdmi-supply = <&vcc_5v>; +}; + +&hdmi_tx_tmds_port { + hdmi_tx_tmds_out: endpoint { + remote-endpoint = <&hdmi_connector_in>; + }; +}; + +&ir { + status = "okay"; + pinctrl-0 = <&remote_input_ao_pins>; + pinctrl-names = "default"; +}; + +/* SD card */ +&sd_emmc_b { + status = "okay"; + pinctrl-0 = <&sdcard_c_pins>; + pinctrl-1 = <&sdcard_clk_gate_c_pins>; + pinctrl-names = "default", "clk-gate"; + + bus-width = <4>; + cap-sd-highspeed; + max-frequency = <50000000>; + disable-wp; + + cd-gpios = <&gpio GPIOC_6 GPIO_ACTIVE_LOW>; + vmmc-supply = <&tflash_vdd>; + vqmmc-supply = <&tf_io>; + +}; + +/* eMMC */ +&sd_emmc_c { + status = "okay"; + pinctrl-0 = <&emmc_pins>, <&emmc_ds_pins>; + pinctrl-1 = <&emmc_clk_gate_pins>; + pinctrl-names = "default", "clk-gate"; + + bus-width = <8>; + cap-mmc-highspeed; + mmc-ddr-1_8v; + mmc-hs200-1_8v; + max-frequency = <200000000>; + disable-wp; + + mmc-pwrseq = <&emmc_pwrseq>; + vmmc-supply = <&vcc_3v3>; + vqmmc-supply = <&flash_1v8>; +}; + +&uart_AO { + status = "okay"; + pinctrl-0 = <&uart_ao_a_pins>; + pinctrl-names = "default"; +}; + +&usb { + status = "okay"; + vbus-supply = <&usb_pwr_en>; +}; + +&usb2_phy0 { + phy-supply = <&vcc_5v>; +}; + +&usb2_phy1 { + phy-supply = <&vcc_5v>; +}; diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12b.dtsi new file mode 100644 index 000000000000..9e88e513b22d --- /dev/null +++ b/arch/arm64/boot/dts/amlogic/meson-g12b.dtsi @@ -0,0 +1,82 @@ +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) +/* + * Copyright (c) 2019 BayLibre, SAS + * Author: Neil Armstrong <narmstrong@baylibre.com> + */ + +#include "meson-g12a.dtsi" + +/ { + compatible = "amlogic,g12b"; + + cpus { + cpu-map { + cluster0 { + core0 { + cpu = <&cpu0>; + }; + + core1 { + cpu = <&cpu1>; + }; + }; + + cluster1 { + core0 { + cpu = <&cpu100>; + }; + + core1 { + cpu = <&cpu101>; + }; + + core2 { + cpu = <&cpu102>; + }; + + core3 { + cpu = <&cpu103>; + }; + }; + }; + + /delete-node/ cpu@2; + /delete-node/ cpu@3; + + cpu100: cpu@100 { + device_type = "cpu"; + compatible = "arm,cortex-a73"; + reg = <0x0 0x100>; + enable-method = "psci"; + next-level-cache = <&l2>; + }; + + cpu101: cpu@101 { + device_type = "cpu"; + compatible = "arm,cortex-a73"; + reg = <0x0 0x101>; + enable-method = "psci"; + next-level-cache = <&l2>; + }; + + cpu102: cpu@102 { + device_type = "cpu"; + compatible = "arm,cortex-a73"; + reg = <0x0 0x102>; + enable-method = "psci"; + next-level-cache = <&l2>; + }; + + cpu103: cpu@103 { + device_type = "cpu"; + compatible = "arm,cortex-a73"; + reg = <0x0 0x103>; + enable-method = "psci"; + next-level-cache = <&l2>; + }; + }; +}; + +&clkc { + compatible = "amlogic,g12b-clkc"; +}; -- 2.21.0 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v4 3/3] arm64: dts: meson: Add minimal support for Odroid-N2 2019-05-27 14:02 ` [PATCH v4 3/3] arm64: dts: meson: Add minimal support for Odroid-N2 Neil Armstrong @ 2019-05-27 18:12 ` Martin Blumenstingl 2019-05-29 10:08 ` Neil Armstrong 1 sibling, 0 replies; 8+ messages in thread From: Martin Blumenstingl @ 2019-05-27 18:12 UTC (permalink / raw) To: Neil Armstrong; +Cc: khilman, linux-amlogic, linux-kernel, linux-arm-kernel On Mon, May 27, 2019 at 4:03 PM Neil Armstrong <narmstrong@baylibre.com> wrote: > > This patch adds basic support for : > - Amlogic G12B, which is very similar to G12A > - The HardKernel Odroid-N2 based on the S922X SoC > > The Amlogic G12B SoC is very similar with the G12A SoC, sharing > most of the features and architecture, but with these differences : > - The first CPU cluster only has 2xCortex-A53 instead of 4 > - G12B has a second cluster of 4xCortex-A73 > - Both cluster can achieve 2GHz instead of 1,8GHz for G12A > - CPU Clock architecture is difference, thus needing a different > compatible to handle this slight difference > - Supports a MIPI CSI input > - Embeds a Mali-G52 instead of a Mali-G31, but integration is the same > > Actual support is done in the same way as for the GXM support, including > the G12A dtsi and redefining the CPU clusters. > Unlike GXM, the first cluster is different, thus needing to remove > the last 2 cpu nodes of the first cluster. > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 3/3] arm64: dts: meson: Add minimal support for Odroid-N2 2019-05-27 14:02 ` [PATCH v4 3/3] arm64: dts: meson: Add minimal support for Odroid-N2 Neil Armstrong 2019-05-27 18:12 ` Martin Blumenstingl @ 2019-05-29 10:08 ` Neil Armstrong 2019-05-29 18:13 ` Martin Blumenstingl 1 sibling, 1 reply; 8+ messages in thread From: Neil Armstrong @ 2019-05-29 10:08 UTC (permalink / raw) To: khilman; +Cc: linux-amlogic, linux-arm-kernel, linux-kernel On 27/05/2019 16:02, Neil Armstrong wrote: > This patch adds basic support for : > - Amlogic G12B, which is very similar to G12A > - The HardKernel Odroid-N2 based on the S922X SoC > > The Amlogic G12B SoC is very similar with the G12A SoC, sharing > most of the features and architecture, but with these differences : > - The first CPU cluster only has 2xCortex-A53 instead of 4 > - G12B has a second cluster of 4xCortex-A73 > - Both cluster can achieve 2GHz instead of 1,8GHz for G12A > - CPU Clock architecture is difference, thus needing a different > compatible to handle this slight difference > - Supports a MIPI CSI input > - Embeds a Mali-G52 instead of a Mali-G31, but integration is the same > > Actual support is done in the same way as for the GXM support, including > the G12A dtsi and redefining the CPU clusters. > Unlike GXM, the first cluster is different, thus needing to remove > the last 2 cpu nodes of the first cluster. > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> > --- > arch/arm64/boot/dts/amlogic/Makefile | 1 + > .../boot/dts/amlogic/meson-g12b-odroid-n2.dts | 288 ++++++++++++++++++ > arch/arm64/boot/dts/amlogic/meson-g12b.dtsi | 82 +++++ > 3 files changed, 371 insertions(+) > create mode 100644 arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts > create mode 100644 arch/arm64/boot/dts/amlogic/meson-g12b.dtsi > > diff --git a/arch/arm64/boot/dts/amlogic/Makefile b/arch/arm64/boot/dts/amlogic/Makefile > index e129c03ced14..07b861fe5fa5 100644 > --- a/arch/arm64/boot/dts/amlogic/Makefile > +++ b/arch/arm64/boot/dts/amlogic/Makefile > @@ -3,6 +3,7 @@ dtb-$(CONFIG_ARCH_MESON) += meson-axg-s400.dtb > dtb-$(CONFIG_ARCH_MESON) += meson-g12a-sei510.dtb > dtb-$(CONFIG_ARCH_MESON) += meson-g12a-u200.dtb > dtb-$(CONFIG_ARCH_MESON) += meson-g12a-x96-max.dtb > +dtb-$(CONFIG_ARCH_MESON) += meson-g12b-odroid-n2.dtb > dtb-$(CONFIG_ARCH_MESON) += meson-gxbb-nanopi-k2.dtb > dtb-$(CONFIG_ARCH_MESON) += meson-gxbb-nexbox-a95x.dtb > dtb-$(CONFIG_ARCH_MESON) += meson-gxbb-odroidc2.dtb > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts > new file mode 100644 > index 000000000000..48783ead8dfb > --- /dev/null > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts > @@ -0,0 +1,288 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Copyright (c) 2019 BayLibre, SAS > + * Author: Neil Armstrong <narmstrong@baylibre.com> > + */ > + > +/dts-v1/; > + [...] > + > + hub_5v: regulator-hub_5v { > + compatible = "regulator-fixed"; > + regulator-name = "HUB_5V"; > + regulator-min-microvolt = <5000000>; > + regulator-max-microvolt = <5000000>; > + vin-supply = <&vcc_5v>; > + > + /* Connected to the Hub CHIPENABLE, LOW sets low power state */ > + gpio = <&gpio GPIOH_5 GPIO_ACTIVE_HIGH>; > + enable-active-high; > + }; > + > + usb_pwr_en: regulator-usb_pwr_en { > + compatible = "regulator-fixed"; > + regulator-name = "USB_PWR_EN"; > + regulator-min-microvolt = <5000000>; > + regulator-max-microvolt = <5000000>; > + vin-supply = <&hub_5v>; > + > + /* Connected to the microUSB port power enable */ > + gpio = <&gpio GPIOH_6 GPIO_ACTIVE_HIGH>; > + enable-active-high; > + }; > + [...] > + > +&usb { > + status = "okay"; > + vbus-supply = <&usb_pwr_en>; > +}; > + > +&usb2_phy0 { > + phy-supply = <&vcc_5v>; > +}; > + > +&usb2_phy1 { > + phy-supply = <&vcc_5v>; > +}; In fact, I need to fixup here : usb2_phy1 needs &hub_5v and regulator-usb_pwr_en depends on &vcc_5v instead... @Martin, can I still keep your reviewed-by for v5 ? Neil > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b.dtsi b/arch/arm64/boot/dts/amlogic/meson-g12b.dtsi > new file mode 100644 > index 000000000000..9e88e513b22d > --- /dev/null > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b.dtsi > @@ -0,0 +1,82 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) [...] > + }; > + }; > +}; > + > +&clkc { > + compatible = "amlogic,g12b-clkc"; > +}; > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 3/3] arm64: dts: meson: Add minimal support for Odroid-N2 2019-05-29 10:08 ` Neil Armstrong @ 2019-05-29 18:13 ` Martin Blumenstingl 2019-05-31 9:21 ` Neil Armstrong 0 siblings, 1 reply; 8+ messages in thread From: Martin Blumenstingl @ 2019-05-29 18:13 UTC (permalink / raw) To: Neil Armstrong; +Cc: khilman, linux-amlogic, linux-kernel, linux-arm-kernel Hi Neil, On Wed, May 29, 2019 at 12:09 PM Neil Armstrong <narmstrong@baylibre.com> wrote: > > On 27/05/2019 16:02, Neil Armstrong wrote: > > This patch adds basic support for : > > - Amlogic G12B, which is very similar to G12A > > - The HardKernel Odroid-N2 based on the S922X SoC > > > > The Amlogic G12B SoC is very similar with the G12A SoC, sharing > > most of the features and architecture, but with these differences : > > - The first CPU cluster only has 2xCortex-A53 instead of 4 > > - G12B has a second cluster of 4xCortex-A73 > > - Both cluster can achieve 2GHz instead of 1,8GHz for G12A > > - CPU Clock architecture is difference, thus needing a different > > compatible to handle this slight difference > > - Supports a MIPI CSI input > > - Embeds a Mali-G52 instead of a Mali-G31, but integration is the same > > > > Actual support is done in the same way as for the GXM support, including > > the G12A dtsi and redefining the CPU clusters. > > Unlike GXM, the first cluster is different, thus needing to remove > > the last 2 cpu nodes of the first cluster. > > > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> > > --- > > arch/arm64/boot/dts/amlogic/Makefile | 1 + > > .../boot/dts/amlogic/meson-g12b-odroid-n2.dts | 288 ++++++++++++++++++ > > arch/arm64/boot/dts/amlogic/meson-g12b.dtsi | 82 +++++ > > 3 files changed, 371 insertions(+) > > create mode 100644 arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts > > create mode 100644 arch/arm64/boot/dts/amlogic/meson-g12b.dtsi > > > > diff --git a/arch/arm64/boot/dts/amlogic/Makefile b/arch/arm64/boot/dts/amlogic/Makefile > > index e129c03ced14..07b861fe5fa5 100644 > > --- a/arch/arm64/boot/dts/amlogic/Makefile > > +++ b/arch/arm64/boot/dts/amlogic/Makefile > > @@ -3,6 +3,7 @@ dtb-$(CONFIG_ARCH_MESON) += meson-axg-s400.dtb > > dtb-$(CONFIG_ARCH_MESON) += meson-g12a-sei510.dtb > > dtb-$(CONFIG_ARCH_MESON) += meson-g12a-u200.dtb > > dtb-$(CONFIG_ARCH_MESON) += meson-g12a-x96-max.dtb > > +dtb-$(CONFIG_ARCH_MESON) += meson-g12b-odroid-n2.dtb > > dtb-$(CONFIG_ARCH_MESON) += meson-gxbb-nanopi-k2.dtb > > dtb-$(CONFIG_ARCH_MESON) += meson-gxbb-nexbox-a95x.dtb > > dtb-$(CONFIG_ARCH_MESON) += meson-gxbb-odroidc2.dtb > > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts > > new file mode 100644 > > index 000000000000..48783ead8dfb > > --- /dev/null > > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts > > @@ -0,0 +1,288 @@ > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > +/* > > + * Copyright (c) 2019 BayLibre, SAS > > + * Author: Neil Armstrong <narmstrong@baylibre.com> > > + */ > > + > > +/dts-v1/; > > + > > [...] > > > + > > + hub_5v: regulator-hub_5v { > > + compatible = "regulator-fixed"; > > + regulator-name = "HUB_5V"; > > + regulator-min-microvolt = <5000000>; > > + regulator-max-microvolt = <5000000>; > > + vin-supply = <&vcc_5v>; > > + > > + /* Connected to the Hub CHIPENABLE, LOW sets low power state */ > > + gpio = <&gpio GPIOH_5 GPIO_ACTIVE_HIGH>; > > + enable-active-high; > > + }; > > + > > + usb_pwr_en: regulator-usb_pwr_en { > > + compatible = "regulator-fixed"; > > + regulator-name = "USB_PWR_EN"; > > + regulator-min-microvolt = <5000000>; > > + regulator-max-microvolt = <5000000>; > > + vin-supply = <&hub_5v>; > > + > > + /* Connected to the microUSB port power enable */ > > + gpio = <&gpio GPIOH_6 GPIO_ACTIVE_HIGH>; > > + enable-active-high; > > + }; > > + > > [...] > > > + > > +&usb { > > + status = "okay"; > > + vbus-supply = <&usb_pwr_en>; > > +}; > > + > > +&usb2_phy0 { > > + phy-supply = <&vcc_5v>; > > +}; > > + > > +&usb2_phy1 { > > + phy-supply = <&vcc_5v>; > > +}; > > In fact, I need to fixup here : > > usb2_phy1 needs &hub_5v and regulator-usb_pwr_en depends on &vcc_5v instead... sounds fine for me because I don't see a better way for now > @Martin, can I still keep your reviewed-by for v5 ? yes, you can keep it when you re-send it: can you please add a comment to the phy-supply in usb2_phy1? I have this in mind: "enable the hub which is connected to this port" (imho this is a valuable hint together with the "CHIPENABLE" comment that you already have inside &hub_5v and it helps to make it easier to understand without having to test it on physical hardware) Martin ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 3/3] arm64: dts: meson: Add minimal support for Odroid-N2 2019-05-29 18:13 ` Martin Blumenstingl @ 2019-05-31 9:21 ` Neil Armstrong 0 siblings, 0 replies; 8+ messages in thread From: Neil Armstrong @ 2019-05-31 9:21 UTC (permalink / raw) To: Martin Blumenstingl Cc: khilman, linux-amlogic, linux-kernel, linux-arm-kernel On 29/05/2019 20:13, Martin Blumenstingl wrote: > Hi Neil, > > On Wed, May 29, 2019 at 12:09 PM Neil Armstrong <narmstrong@baylibre.com> wrote: >> >> On 27/05/2019 16:02, Neil Armstrong wrote: >>> This patch adds basic support for : >>> - Amlogic G12B, which is very similar to G12A >>> - The HardKernel Odroid-N2 based on the S922X SoC >>> >>> The Amlogic G12B SoC is very similar with the G12A SoC, sharing >>> most of the features and architecture, but with these differences : >>> - The first CPU cluster only has 2xCortex-A53 instead of 4 >>> - G12B has a second cluster of 4xCortex-A73 >>> - Both cluster can achieve 2GHz instead of 1,8GHz for G12A >>> - CPU Clock architecture is difference, thus needing a different >>> compatible to handle this slight difference >>> - Supports a MIPI CSI input >>> - Embeds a Mali-G52 instead of a Mali-G31, but integration is the same >>> >>> Actual support is done in the same way as for the GXM support, including >>> the G12A dtsi and redefining the CPU clusters. >>> Unlike GXM, the first cluster is different, thus needing to remove >>> the last 2 cpu nodes of the first cluster. >>> >>> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> >>> --- >>> arch/arm64/boot/dts/amlogic/Makefile | 1 + >>> .../boot/dts/amlogic/meson-g12b-odroid-n2.dts | 288 ++++++++++++++++++ >>> arch/arm64/boot/dts/amlogic/meson-g12b.dtsi | 82 +++++ >>> 3 files changed, 371 insertions(+) >>> create mode 100644 arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts >>> create mode 100644 arch/arm64/boot/dts/amlogic/meson-g12b.dtsi >>> >>> diff --git a/arch/arm64/boot/dts/amlogic/Makefile b/arch/arm64/boot/dts/amlogic/Makefile >>> index e129c03ced14..07b861fe5fa5 100644 >>> --- a/arch/arm64/boot/dts/amlogic/Makefile >>> +++ b/arch/arm64/boot/dts/amlogic/Makefile >>> @@ -3,6 +3,7 @@ dtb-$(CONFIG_ARCH_MESON) += meson-axg-s400.dtb >>> dtb-$(CONFIG_ARCH_MESON) += meson-g12a-sei510.dtb >>> dtb-$(CONFIG_ARCH_MESON) += meson-g12a-u200.dtb >>> dtb-$(CONFIG_ARCH_MESON) += meson-g12a-x96-max.dtb >>> +dtb-$(CONFIG_ARCH_MESON) += meson-g12b-odroid-n2.dtb >>> dtb-$(CONFIG_ARCH_MESON) += meson-gxbb-nanopi-k2.dtb >>> dtb-$(CONFIG_ARCH_MESON) += meson-gxbb-nexbox-a95x.dtb >>> dtb-$(CONFIG_ARCH_MESON) += meson-gxbb-odroidc2.dtb >>> diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts >>> new file mode 100644 >>> index 000000000000..48783ead8dfb >>> --- /dev/null >>> +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-odroid-n2.dts >>> @@ -0,0 +1,288 @@ >>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) >>> +/* >>> + * Copyright (c) 2019 BayLibre, SAS >>> + * Author: Neil Armstrong <narmstrong@baylibre.com> >>> + */ >>> + >>> +/dts-v1/; >>> + >> >> [...] >> >>> + >>> + hub_5v: regulator-hub_5v { >>> + compatible = "regulator-fixed"; >>> + regulator-name = "HUB_5V"; >>> + regulator-min-microvolt = <5000000>; >>> + regulator-max-microvolt = <5000000>; >>> + vin-supply = <&vcc_5v>; >>> + >>> + /* Connected to the Hub CHIPENABLE, LOW sets low power state */ >>> + gpio = <&gpio GPIOH_5 GPIO_ACTIVE_HIGH>; >>> + enable-active-high; >>> + }; >>> + >>> + usb_pwr_en: regulator-usb_pwr_en { >>> + compatible = "regulator-fixed"; >>> + regulator-name = "USB_PWR_EN"; >>> + regulator-min-microvolt = <5000000>; >>> + regulator-max-microvolt = <5000000>; >>> + vin-supply = <&hub_5v>; >>> + >>> + /* Connected to the microUSB port power enable */ >>> + gpio = <&gpio GPIOH_6 GPIO_ACTIVE_HIGH>; >>> + enable-active-high; >>> + }; >>> + >> >> [...] >> >>> + >>> +&usb { >>> + status = "okay"; >>> + vbus-supply = <&usb_pwr_en>; >>> +}; >>> + >>> +&usb2_phy0 { >>> + phy-supply = <&vcc_5v>; >>> +}; >>> + >>> +&usb2_phy1 { >>> + phy-supply = <&vcc_5v>; >>> +}; >> >> In fact, I need to fixup here : >> >> usb2_phy1 needs &hub_5v and regulator-usb_pwr_en depends on &vcc_5v instead... > sounds fine for me because I don't see a better way for now > >> @Martin, can I still keep your reviewed-by for v5 ? > yes, you can keep it > > when you re-send it: can you please add a comment to the phy-supply in > usb2_phy1? > I have this in mind: "enable the hub which is connected to this port" > (imho this is a valuable hint together with the "CHIPENABLE" comment > that you already have inside &hub_5v and it helps to make it easier to > understand without having to test it on physical hardware) Sure, thanks ! Neil > > > Martin > ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-05-31 9:21 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-05-27 14:02 [PATCH v4 0/3] arm64: Add initial support for Odroid-N2 Neil Armstrong 2019-05-27 14:02 ` [PATCH v4 1/3] dt-bindings: arm: amlogic: add G12B bindings Neil Armstrong 2019-05-27 14:02 ` [PATCH v4 2/3] dt-bindings: arm: amlogic: add Odroid-N2 binding Neil Armstrong 2019-05-27 14:02 ` [PATCH v4 3/3] arm64: dts: meson: Add minimal support for Odroid-N2 Neil Armstrong 2019-05-27 18:12 ` Martin Blumenstingl 2019-05-29 10:08 ` Neil Armstrong 2019-05-29 18:13 ` Martin Blumenstingl 2019-05-31 9:21 ` Neil Armstrong
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).