* [PATCH v2 0/4] Add Basic SoC support for MT7622 @ 2017-05-29 12:56 sean.wang at mediatek.com 2017-05-29 12:56 ` [PATCH v2 1/4] dt-bindings: mediatek: add bindings for MediaTek MT7622 SoC sean.wang at mediatek.com ` (3 more replies) 0 siblings, 4 replies; 10+ messages in thread From: sean.wang at mediatek.com @ 2017-05-29 12:56 UTC (permalink / raw) To: linux-arm-kernel From: Sean Wang <sean.wang@mediatek.com> Changes since v1: - update SPDX-License-Identifier - remove next-level-cache property since cache geometry detection was removed since 4.12 This patch set adds basic SoC support for MediaTek MT7622 SoC based on 4.12-rc1. Sean Wang (4): dt-bindings: mediatek: add bindings for MediaTek MT7622 SoC arm64: dts: mt7622: add basic nodes to the mt7622.dtsi file arm64: dts: mt7622: add dts file for MT7622 reference board variant 1 arm64: dts: mt7622: group clock DT nodes into separate DT file Documentation/devicetree/bindings/arm/mediatek.txt | 4 + .../interrupt-controller/mediatek,sysirq.txt | 3 +- .../devicetree/bindings/serial/mtk-uart.txt | 1 + arch/arm64/boot/dts/mediatek/Makefile | 1 + arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi | 20 +++++ arch/arm64/boot/dts/mediatek/mt7622-rfb1.dts | 27 ++++++ arch/arm64/boot/dts/mediatek/mt7622.dtsi | 98 ++++++++++++++++++++++ 7 files changed, 153 insertions(+), 1 deletion(-) create mode 100644 arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi create mode 100644 arch/arm64/boot/dts/mediatek/mt7622-rfb1.dts create mode 100644 arch/arm64/boot/dts/mediatek/mt7622.dtsi -- 1.9.1 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 1/4] dt-bindings: mediatek: add bindings for MediaTek MT7622 SoC 2017-05-29 12:56 [PATCH v2 0/4] Add Basic SoC support for MT7622 sean.wang at mediatek.com @ 2017-05-29 12:56 ` sean.wang at mediatek.com 2017-05-29 15:11 ` Matthias Brugger 2017-05-29 12:56 ` [PATCH v2 2/4] arm64: dts: mt7622: add basic nodes to the mt7622.dtsi file sean.wang at mediatek.com ` (2 subsequent siblings) 3 siblings, 1 reply; 10+ messages in thread From: sean.wang at mediatek.com @ 2017-05-29 12:56 UTC (permalink / raw) To: linux-arm-kernel From: Sean Wang <sean.wang@mediatek.com> This adds dt-binding documentation for MediaTek MT7622 SoC which currently only includes basic items such as ARM CPU, MediaTek SYSIRQ and UART. Signed-off-by: Sean Wang <sean.wang@mediatek.com> --- Documentation/devicetree/bindings/arm/mediatek.txt | 4 ++++ .../devicetree/bindings/interrupt-controller/mediatek,sysirq.txt | 3 ++- Documentation/devicetree/bindings/serial/mtk-uart.txt | 1 + 3 files changed, 7 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/arm/mediatek.txt b/Documentation/devicetree/bindings/arm/mediatek.txt index c860b24..fc68570 100644 --- a/Documentation/devicetree/bindings/arm/mediatek.txt +++ b/Documentation/devicetree/bindings/arm/mediatek.txt @@ -12,6 +12,7 @@ compatible: Must contain one of "mediatek,mt6592" "mediatek,mt6755" "mediatek,mt6795" + "mediatek,mt7622" "mediatek,mt7623" "mediatek,mt8127" "mediatek,mt8135" @@ -38,6 +39,9 @@ Supported boards: - Evaluation board for MT6795(Helio X10): Required root node properties: - compatible = "mediatek,mt6795-evb", "mediatek,mt6795"; +- Reference board variant 1 for MT7622: + Required root node properties: + - compatible = "mediatek,mt7622-rfb1", "mediatek,mt7622"; - Evaluation board for MT7623: Required root node properties: - compatible = "mediatek,mt7623-evb", "mediatek,mt7623"; diff --git a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt index a89c03b..eda1183 100644 --- a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt +++ b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt @@ -1,4 +1,4 @@ -+Mediatek 65xx/67xx/81xx sysirq ++Mediatek MT65xx/MT67xx/MT762x/MT81xx sysirq Mediatek SOCs sysirq support controllable irq inverter for each GIC SPI interrupt. @@ -15,6 +15,7 @@ Required properties: "mediatek,mt6582-sysirq" "mediatek,mt6580-sysirq" "mediatek,mt6577-sysirq" + "mediatek,mt7622-sysirq" "mediatek,mt2701-sysirq" - interrupt-controller : Identifies the node as an interrupt controller - #interrupt-cells : Use the same format as specified by GIC in arm,gic.txt. diff --git a/Documentation/devicetree/bindings/serial/mtk-uart.txt b/Documentation/devicetree/bindings/serial/mtk-uart.txt index 0015c72..daa1eef 100644 --- a/Documentation/devicetree/bindings/serial/mtk-uart.txt +++ b/Documentation/devicetree/bindings/serial/mtk-uart.txt @@ -8,6 +8,7 @@ Required properties: * "mediatek,mt6589-uart" for MT6589 compatible UARTS * "mediatek,mt6755-uart" for MT6755 compatible UARTS * "mediatek,mt6795-uart" for MT6795 compatible UARTS + * "mediatek,mt7622-uart" for MT7622 compatible UARTS * "mediatek,mt7623-uart" for MT7623 compatible UARTS * "mediatek,mt8127-uart" for MT8127 compatible UARTS * "mediatek,mt8135-uart" for MT8135 compatible UARTS -- 1.9.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v2 1/4] dt-bindings: mediatek: add bindings for MediaTek MT7622 SoC 2017-05-29 12:56 ` [PATCH v2 1/4] dt-bindings: mediatek: add bindings for MediaTek MT7622 SoC sean.wang at mediatek.com @ 2017-05-29 15:11 ` Matthias Brugger 2017-05-29 15:11 ` Matthias Brugger 0 siblings, 1 reply; 10+ messages in thread From: Matthias Brugger @ 2017-05-29 15:11 UTC (permalink / raw) To: linux-arm-kernel On 29/05/17 14:56, sean.wang at mediatek.com wrote: > From: Sean Wang <sean.wang@mediatek.com> > > This adds dt-binding documentation for MediaTek MT7622 SoC > which currently only includes basic items such as ARM CPU, > MediaTek SYSIRQ and UART. > > Signed-off-by: Sean Wang <sean.wang@mediatek.com> > --- > Documentation/devicetree/bindings/arm/mediatek.txt | 4 ++++ > .../devicetree/bindings/interrupt-controller/mediatek,sysirq.txt | 3 ++- > Documentation/devicetree/bindings/serial/mtk-uart.txt | 1 + > 3 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/arm/mediatek.txt b/Documentation/devicetree/bindings/arm/mediatek.txt > index c860b24..fc68570 100644 > --- a/Documentation/devicetree/bindings/arm/mediatek.txt > +++ b/Documentation/devicetree/bindings/arm/mediatek.txt > @@ -12,6 +12,7 @@ compatible: Must contain one of > "mediatek,mt6592" > "mediatek,mt6755" > "mediatek,mt6795" > + "mediatek,mt7622" > "mediatek,mt7623" > "mediatek,mt8127" > "mediatek,mt8135" > @@ -38,6 +39,9 @@ Supported boards: > - Evaluation board for MT6795(Helio X10): > Required root node properties: > - compatible = "mediatek,mt6795-evb", "mediatek,mt6795"; > +- Reference board variant 1 for MT7622: > + Required root node properties: > + - compatible = "mediatek,mt7622-rfb1", "mediatek,mt7622"; > - Evaluation board for MT7623: > Required root node properties: > - compatible = "mediatek,mt7623-evb", "mediatek,mt7623"; > diff --git a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt > index a89c03b..eda1183 100644 > --- a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt > +++ b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt > @@ -1,4 +1,4 @@ > -+Mediatek 65xx/67xx/81xx sysirq > ++Mediatek MT65xx/MT67xx/MT762x/MT81xx sysirq > > Mediatek SOCs sysirq support controllable irq inverter for each GIC SPI > interrupt. > @@ -15,6 +15,7 @@ Required properties: > "mediatek,mt6582-sysirq" > "mediatek,mt6580-sysirq" > "mediatek,mt6577-sysirq" > + "mediatek,mt7622-sysirq" > "mediatek,mt2701-sysirq" This file is a mess and I posted some cleanups on the mailinglist. No need for you to resubmit. If the DT maintainers are fine with my cleanups, I cann change the line accordingly. Regards, Matthias > - interrupt-controller : Identifies the node as an interrupt controller > - #interrupt-cells : Use the same format as specified by GIC in arm,gic.txt. > diff --git a/Documentation/devicetree/bindings/serial/mtk-uart.txt b/Documentation/devicetree/bindings/serial/mtk-uart.txt > index 0015c72..daa1eef 100644 > --- a/Documentation/devicetree/bindings/serial/mtk-uart.txt > +++ b/Documentation/devicetree/bindings/serial/mtk-uart.txt > @@ -8,6 +8,7 @@ Required properties: > * "mediatek,mt6589-uart" for MT6589 compatible UARTS > * "mediatek,mt6755-uart" for MT6755 compatible UARTS > * "mediatek,mt6795-uart" for MT6795 compatible UARTS > + * "mediatek,mt7622-uart" for MT7622 compatible UARTS > * "mediatek,mt7623-uart" for MT7623 compatible UARTS > * "mediatek,mt8127-uart" for MT8127 compatible UARTS > * "mediatek,mt8135-uart" for MT8135 compatible UARTS > ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 1/4] dt-bindings: mediatek: add bindings for MediaTek MT7622 SoC 2017-05-29 15:11 ` Matthias Brugger @ 2017-05-29 15:11 ` Matthias Brugger 0 siblings, 0 replies; 10+ messages in thread From: Matthias Brugger @ 2017-05-29 15:11 UTC (permalink / raw) To: linux-arm-kernel On 29/05/17 17:11, Matthias Brugger wrote: > > > On 29/05/17 14:56, sean.wang at mediatek.com wrote: >> From: Sean Wang <sean.wang@mediatek.com> >> >> This adds dt-binding documentation for MediaTek MT7622 SoC >> which currently only includes basic items such as ARM CPU, >> MediaTek SYSIRQ and UART. >> >> Signed-off-by: Sean Wang <sean.wang@mediatek.com> >> --- >> >> Documentation/devicetree/bindings/arm/mediatek.txt >> | 4 ++++ >> >> .../devicetree/bindings/interrupt-controller/mediatek,sysirq.txt >> | 3 ++- >> >> Documentation/devicetree/bindings/serial/mtk-uart.txt >> | 1 + >> 3 files changed, 7 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/arm/mediatek.txt >> b/Documentation/devicetree/bindings/arm/mediatek.txt >> index c860b24..fc68570 100644 >> --- a/Documentation/devicetree/bindings/arm/mediatek.txt >> +++ b/Documentation/devicetree/bindings/arm/mediatek.txt >> @@ -12,6 +12,7 @@ compatible: Must contain one of >> "mediatek,mt6592" >> "mediatek,mt6755" >> "mediatek,mt6795" >> + "mediatek,mt7622" >> "mediatek,mt7623" >> "mediatek,mt8127" >> "mediatek,mt8135" >> @@ -38,6 +39,9 @@ Supported boards: >> - Evaluation board for MT6795(Helio X10): >> Required root node properties: >> - compatible = "mediatek,mt6795-evb", "mediatek,mt6795"; >> +- Reference board variant 1 for MT7622: >> + Required root node properties: >> + - compatible = "mediatek,mt7622-rfb1", "mediatek,mt7622"; >> - Evaluation board for MT7623: >> Required root node properties: >> - compatible = "mediatek,mt7623-evb", "mediatek,mt7623"; >> diff --git >> a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt >> b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt >> >> index a89c03b..eda1183 100644 >> --- >> a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt >> >> +++ >> b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt >> >> @@ -1,4 +1,4 @@ >> -+Mediatek 65xx/67xx/81xx sysirq >> ++Mediatek MT65xx/MT67xx/MT762x/MT81xx sysirq >> Mediatek SOCs sysirq support controllable irq inverter for each GIC SPI >> interrupt. >> @@ -15,6 +15,7 @@ Required properties: >> "mediatek,mt6582-sysirq" >> "mediatek,mt6580-sysirq" >> "mediatek,mt6577-sysirq" >> + "mediatek,mt7622-sysirq" >> "mediatek,mt2701-sysirq" > > This file is a mess and I posted some cleanups on the mailinglist. No > need for you to resubmit. If the DT maintainers are fine with my > cleanups, I cann change the line accordingly. > I forgot: Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com> > Regards, > Matthias > >> - interrupt-controller : Identifies the node as an interrupt controller >> - #interrupt-cells : Use the same format as specified by GIC in >> arm,gic.txt. >> diff --git a/Documentation/devicetree/bindings/serial/mtk-uart.txt >> b/Documentation/devicetree/bindings/serial/mtk-uart.txt >> index 0015c72..daa1eef 100644 >> --- a/Documentation/devicetree/bindings/serial/mtk-uart.txt >> +++ b/Documentation/devicetree/bindings/serial/mtk-uart.txt >> @@ -8,6 +8,7 @@ Required properties: >> * "mediatek,mt6589-uart" for MT6589 compatible UARTS >> * "mediatek,mt6755-uart" for MT6755 compatible UARTS >> * "mediatek,mt6795-uart" for MT6795 compatible UARTS >> + * "mediatek,mt7622-uart" for MT7622 compatible UARTS >> * "mediatek,mt7623-uart" for MT7623 compatible UARTS >> * "mediatek,mt8127-uart" for MT8127 compatible UARTS >> * "mediatek,mt8135-uart" for MT8135 compatible UARTS >> ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 2/4] arm64: dts: mt7622: add basic nodes to the mt7622.dtsi file 2017-05-29 12:56 [PATCH v2 0/4] Add Basic SoC support for MT7622 sean.wang at mediatek.com 2017-05-29 12:56 ` [PATCH v2 1/4] dt-bindings: mediatek: add bindings for MediaTek MT7622 SoC sean.wang at mediatek.com @ 2017-05-29 12:56 ` sean.wang at mediatek.com 2017-05-29 12:56 ` [PATCH v2 3/4] arm64: dts: mt7622: add dts file for MT7622 reference board variant 1 sean.wang at mediatek.com 2017-05-29 12:56 ` [PATCH v2 4/4] arm64: dts: mt7622: group clock DT nodes into separate DT file sean.wang at mediatek.com 3 siblings, 0 replies; 10+ messages in thread From: sean.wang at mediatek.com @ 2017-05-29 12:56 UTC (permalink / raw) To: linux-arm-kernel From: Sean Wang <sean.wang@mediatek.com> add basic nodes into the mt7622.dtsi for the system bring-up which includes ARM CPU, GIC, timer, MediaTek UART, SYSIRQ and one reserved memory region for ATF. Signed-off-by: Sean Wang <sean.wang@mediatek.com> --- arch/arm64/boot/dts/mediatek/mt7622.dtsi | 97 ++++++++++++++++++++++++++++++++ 1 file changed, 97 insertions(+) create mode 100644 arch/arm64/boot/dts/mediatek/mt7622.dtsi diff --git a/arch/arm64/boot/dts/mediatek/mt7622.dtsi b/arch/arm64/boot/dts/mediatek/mt7622.dtsi new file mode 100644 index 0000000..3a729d9 --- /dev/null +++ b/arch/arm64/boot/dts/mediatek/mt7622.dtsi @@ -0,0 +1,97 @@ +/* + * Copyright (c) 2017 MediaTek Inc. + * Author: Ming Huang <ming.huang@mediatek.com> + * Sean Wang <sean.wang@mediatek.com> + * + * SPDX-License-Identifier: (GPL-2.0 OR MIT) + */ + +#include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/interrupt-controller/arm-gic.h> + +/ { + compatible = "mediatek,mt7622"; + interrupt-parent = <&sysirq>; + #address-cells = <2>; + #size-cells = <2>; + + cpus { + #address-cells = <2>; + #size-cells = <0>; + + cpu0: cpu at 0 { + device_type = "cpu"; + compatible = "arm,cortex-a53", "arm,armv8"; + reg = <0x0 0x0>; + enable-method = "psci"; + clock-frequency = <1300000000>; + }; + + cpu1: cpu at 1 { + device_type = "cpu"; + compatible = "arm,cortex-a53", "arm,armv8"; + reg = <0x0 0x1>; + enable-method = "psci"; + clock-frequency = <1300000000>; + }; + }; + + psci { + compatible = "arm,psci-0.2"; + method = "smc"; + }; + + reserved-memory { + #address-cells = <2>; + #size-cells = <2>; + ranges; + + /* 192 KiB reserved for ARM Trusted Firmware (BL31) */ + secmon_reserved: secmon at 43000000 { + reg = <0 0x43000000 0 0x30000>; + no-map; + }; + }; + + timer { + compatible = "arm,armv8-timer"; + interrupt-parent = <&gic>; + interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | + IRQ_TYPE_LEVEL_HIGH)>, + <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | + IRQ_TYPE_LEVEL_HIGH)>, + <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | + IRQ_TYPE_LEVEL_HIGH)>, + <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | + IRQ_TYPE_LEVEL_HIGH)>; + }; + + sysirq: interrupt-controller at 10200620 { + compatible = "mediatek,mt7622-sysirq", + "mediatek,mt6577-sysirq"; + interrupt-controller; + #interrupt-cells = <3>; + interrupt-parent = <&gic>; + reg = <0 0x10200620 0 0x20>; + }; + + gic: interrupt-controller at 10300000 { + compatible = "arm,gic-400"; + interrupt-controller; + #interrupt-cells = <3>; + interrupt-parent = <&gic>; + reg = <0 0x10310000 0 0x1000>, + <0 0x10320000 0 0x1000>, + <0 0x10340000 0 0x2000>, + <0 0x10360000 0 0x2000>; + }; + + uart0: serial at 11002000 { + compatible = "mediatek,mt7622-uart", + "mediatek,mt6577-uart"; + reg = <0 0x11002000 0 0x400>; + interrupts = <GIC_SPI 91 IRQ_TYPE_LEVEL_LOW>; + clocks = <&uart_clk>; + status = "disabled"; + }; +}; -- 1.9.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v2 3/4] arm64: dts: mt7622: add dts file for MT7622 reference board variant 1 2017-05-29 12:56 [PATCH v2 0/4] Add Basic SoC support for MT7622 sean.wang at mediatek.com 2017-05-29 12:56 ` [PATCH v2 1/4] dt-bindings: mediatek: add bindings for MediaTek MT7622 SoC sean.wang at mediatek.com 2017-05-29 12:56 ` [PATCH v2 2/4] arm64: dts: mt7622: add basic nodes to the mt7622.dtsi file sean.wang at mediatek.com @ 2017-05-29 12:56 ` sean.wang at mediatek.com 2017-05-29 19:09 ` kbuild test robot 2017-05-29 12:56 ` [PATCH v2 4/4] arm64: dts: mt7622: group clock DT nodes into separate DT file sean.wang at mediatek.com 3 siblings, 1 reply; 10+ messages in thread From: sean.wang at mediatek.com @ 2017-05-29 12:56 UTC (permalink / raw) To: linux-arm-kernel From: Sean Wang <sean.wang@mediatek.com> Add the support for the MT7622 reference board variant 1 from MediaTek. Signed-off-by: Sean Wang <sean.wang@mediatek.com> --- arch/arm64/boot/dts/mediatek/Makefile | 1 + arch/arm64/boot/dts/mediatek/mt7622-rfb1.dts | 27 +++++++++++++++++++++++++++ 2 files changed, 28 insertions(+) create mode 100644 arch/arm64/boot/dts/mediatek/mt7622-rfb1.dts diff --git a/arch/arm64/boot/dts/mediatek/Makefile b/arch/arm64/boot/dts/mediatek/Makefile index 9fbfd32..e71456a2 100644 --- a/arch/arm64/boot/dts/mediatek/Makefile +++ b/arch/arm64/boot/dts/mediatek/Makefile @@ -1,5 +1,6 @@ dtb-$(CONFIG_ARCH_MEDIATEK) += mt6755-evb.dtb dtb-$(CONFIG_ARCH_MEDIATEK) += mt6795-evb.dtb +dtb-$(CONFIG_ARCH_MEDIATEK) += mt7622-rfb1.dtb dtb-$(CONFIG_ARCH_MEDIATEK) += mt8173-evb.dtb always := $(dtb-y) diff --git a/arch/arm64/boot/dts/mediatek/mt7622-rfb1.dts b/arch/arm64/boot/dts/mediatek/mt7622-rfb1.dts new file mode 100644 index 0000000..c08309d --- /dev/null +++ b/arch/arm64/boot/dts/mediatek/mt7622-rfb1.dts @@ -0,0 +1,27 @@ +/* + * Copyright (c) 2017 MediaTek Inc. + * Author: Ming Huang <ming.huang@mediatek.com> + * Sean Wang <sean.wang@mediatek.com> + * + * SPDX-License-Identifier: (GPL-2.0 OR MIT) + */ + +/dts-v1/; +#include "mt7622.dtsi" + +/ { + model = "MediaTek MT7622 RFB1 board"; + compatible = "mediatek,mt7622-rfb1", "mediatek,mt7622"; + + chosen { + bootargs = "console=ttyS0,115200n1"; + }; + + memory { + reg = <0 0x40000000 0 0x3F000000>; + }; +}; + +&uart0 { + status = "okay"; +}; -- 1.9.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v2 3/4] arm64: dts: mt7622: add dts file for MT7622 reference board variant 1 2017-05-29 12:56 ` [PATCH v2 3/4] arm64: dts: mt7622: add dts file for MT7622 reference board variant 1 sean.wang at mediatek.com @ 2017-05-29 19:09 ` kbuild test robot 0 siblings, 0 replies; 10+ messages in thread From: kbuild test robot @ 2017-05-29 19:09 UTC (permalink / raw) To: linux-arm-kernel Hi Sean, [auto build test ERROR on robh/for-next] [also build test ERROR on v4.12-rc3 next-20170529] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/sean-wang-mediatek-com/Add-Basic-SoC-support-for-MT7622/20170529-210756 base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next config: arm64-defconfig (attached as .config) compiler: aarch64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705 reproduce: wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree make.cross ARCH=arm64 Note: the linux-review/sean-wang-mediatek-com/Add-Basic-SoC-support-for-MT7622/20170529-210756 HEAD c1f5996a1eaf50f9b321fe8c6cddbf24d4969703 builds fine. It only hurts bisectibility. All errors (new ones prefixed by >>): >> ERROR: Input tree has errors, aborting (use -f to force output) --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation -------------- next part -------------- A non-text attachment was scrubbed... Name: .config.gz Type: application/gzip Size: 34622 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170530/8d04e537/attachment-0001.gz> ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 4/4] arm64: dts: mt7622: group clock DT nodes into separate DT file 2017-05-29 12:56 [PATCH v2 0/4] Add Basic SoC support for MT7622 sean.wang at mediatek.com ` (2 preceding siblings ...) 2017-05-29 12:56 ` [PATCH v2 3/4] arm64: dts: mt7622: add dts file for MT7622 reference board variant 1 sean.wang at mediatek.com @ 2017-05-29 12:56 ` sean.wang at mediatek.com 2017-05-29 15:09 ` Matthias Brugger 3 siblings, 1 reply; 10+ messages in thread From: sean.wang at mediatek.com @ 2017-05-29 12:56 UTC (permalink / raw) To: linux-arm-kernel From: Sean Wang <sean.wang@mediatek.com> For more readability and maintenance, all the clock related DT nodes for mt7622 SoC are grouped into a separate DT file. And currently mt7622-clock.dtsi only includes the fixed clock for uart0 and the oscillator which would be extended after MT7622 clock driver is introduced. Signed-off-by: Sean Wang <sean.wang@mediatek.com> --- arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi | 20 ++++++++++++++++++++ arch/arm64/boot/dts/mediatek/mt7622.dtsi | 1 + 2 files changed, 21 insertions(+) create mode 100644 arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi diff --git a/arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi b/arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi new file mode 100644 index 0000000..264421d --- /dev/null +++ b/arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi @@ -0,0 +1,20 @@ +/* + * Copyright (c) 2017 MediaTek Inc. + * Author: Sean Wang <sean.wang@mediatek.com> + * + * SPDX-License-Identifier: (GPL-2.0 OR MIT) + */ +/ { + clk25m: oscillator at 0 { + compatible = "fixed-clock"; + #clock-cells = <0>; + clock-frequency = <25000000>; + clock-output-names = "clkxtal"; + }; + + uart_clk: dummy26m { + compatible = "fixed-clock"; + #clock-cells = <0>; + clock-frequency = <25000000>; + }; +}; diff --git a/arch/arm64/boot/dts/mediatek/mt7622.dtsi b/arch/arm64/boot/dts/mediatek/mt7622.dtsi index 3a729d9..3768c28 100644 --- a/arch/arm64/boot/dts/mediatek/mt7622.dtsi +++ b/arch/arm64/boot/dts/mediatek/mt7622.dtsi @@ -8,6 +8,7 @@ #include <dt-bindings/interrupt-controller/irq.h> #include <dt-bindings/interrupt-controller/arm-gic.h> +#include "mt7622-clock.dtsi" / { compatible = "mediatek,mt7622"; -- 1.9.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v2 4/4] arm64: dts: mt7622: group clock DT nodes into separate DT file 2017-05-29 12:56 ` [PATCH v2 4/4] arm64: dts: mt7622: group clock DT nodes into separate DT file sean.wang at mediatek.com @ 2017-05-29 15:09 ` Matthias Brugger 2017-05-30 1:16 ` Sean Wang 0 siblings, 1 reply; 10+ messages in thread From: Matthias Brugger @ 2017-05-29 15:09 UTC (permalink / raw) To: linux-arm-kernel On 29/05/17 14:56, sean.wang at mediatek.com wrote: > From: Sean Wang <sean.wang@mediatek.com> > > For more readability and maintenance, all the clock related DT > nodes for mt7622 SoC are grouped into a separate DT file. And > currently mt7622-clock.dtsi only includes the fixed clock for > uart0 and the oscillator which would be extended after MT7622 > clock driver is introduced. > We've not done this for any other Mediatek SoC at the moment. Do you expect the m7622 to have more clock nodes then for example mt7623? Apart from that this clock nodes are just dummy clocks which should disappear as soon as the clock driver got merged. Last point, you break bisectability if add a clock node in a patch *after* the consumer was added. So please add the clock nodes to the dtsi as we did up to now. If in the future we see that mt7622 has a whole bunch of clock nodes, we can still abstract them in an external dtsi file. Thanks, Matthias > Signed-off-by: Sean Wang <sean.wang@mediatek.com> > --- > arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi | 20 ++++++++++++++++++++ > arch/arm64/boot/dts/mediatek/mt7622.dtsi | 1 + > 2 files changed, 21 insertions(+) > create mode 100644 arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi > > diff --git a/arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi b/arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi > new file mode 100644 > index 0000000..264421d > --- /dev/null > +++ b/arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi > @@ -0,0 +1,20 @@ > +/* > + * Copyright (c) 2017 MediaTek Inc. > + * Author: Sean Wang <sean.wang@mediatek.com> > + * > + * SPDX-License-Identifier: (GPL-2.0 OR MIT) > + */ > +/ { > + clk25m: oscillator at 0 { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <25000000>; > + clock-output-names = "clkxtal"; > + }; > + > + uart_clk: dummy26m { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <25000000>; > + }; > +}; > diff --git a/arch/arm64/boot/dts/mediatek/mt7622.dtsi b/arch/arm64/boot/dts/mediatek/mt7622.dtsi > index 3a729d9..3768c28 100644 > --- a/arch/arm64/boot/dts/mediatek/mt7622.dtsi > +++ b/arch/arm64/boot/dts/mediatek/mt7622.dtsi > @@ -8,6 +8,7 @@ > > #include <dt-bindings/interrupt-controller/irq.h> > #include <dt-bindings/interrupt-controller/arm-gic.h> > +#include "mt7622-clock.dtsi" > > / { > compatible = "mediatek,mt7622"; > ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 4/4] arm64: dts: mt7622: group clock DT nodes into separate DT file 2017-05-29 15:09 ` Matthias Brugger @ 2017-05-30 1:16 ` Sean Wang 0 siblings, 0 replies; 10+ messages in thread From: Sean Wang @ 2017-05-30 1:16 UTC (permalink / raw) To: linux-arm-kernel On Mon, 2017-05-29 at 17:09 +0200, Matthias Brugger wrote: > > On 29/05/17 14:56, sean.wang at mediatek.com wrote: > > From: Sean Wang <sean.wang@mediatek.com> > > > > For more readability and maintenance, all the clock related DT > > nodes for mt7622 SoC are grouped into a separate DT file. And > > currently mt7622-clock.dtsi only includes the fixed clock for > > uart0 and the oscillator which would be extended after MT7622 > > clock driver is introduced. > > > > We've not done this for any other Mediatek SoC at the moment. > Do you expect the m7622 to have more clock nodes then for example mt7623? clocks on mt7622 has fewer nodes than mt7623 > Apart from that this clock nodes are just dummy clocks which should > disappear as soon as the clock driver got merged. > Last point, you break bisectability if add a clock node in a patch > *after* the consumer was added. > > So please add the clock nodes to the dtsi as we did up to now. If in the > future we see that mt7622 has a whole bunch of clock nodes, we can still > abstract them in an external dtsi file. > Okay, I will merge back them into the main dtsi files for those reasons in the next version. Appreciate all your reviewing and suggestions Sean > Thanks, > Matthias > > > Signed-off-by: Sean Wang <sean.wang@mediatek.com> > > --- > > arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi | 20 ++++++++++++++++++++ > > arch/arm64/boot/dts/mediatek/mt7622.dtsi | 1 + > > 2 files changed, 21 insertions(+) > > create mode 100644 arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi > > > > diff --git a/arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi b/arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi > > new file mode 100644 > > index 0000000..264421d > > --- /dev/null > > +++ b/arch/arm64/boot/dts/mediatek/mt7622-clock.dtsi > > @@ -0,0 +1,20 @@ > > +/* > > + * Copyright (c) 2017 MediaTek Inc. > > + * Author: Sean Wang <sean.wang@mediatek.com> > > + * > > + * SPDX-License-Identifier: (GPL-2.0 OR MIT) > > + */ > > +/ { > > + clk25m: oscillator at 0 { > > + compatible = "fixed-clock"; > > + #clock-cells = <0>; > > + clock-frequency = <25000000>; > > + clock-output-names = "clkxtal"; > > + }; > > + > > + uart_clk: dummy26m { > > + compatible = "fixed-clock"; > > + #clock-cells = <0>; > > + clock-frequency = <25000000>; > > + }; > > +}; > > diff --git a/arch/arm64/boot/dts/mediatek/mt7622.dtsi b/arch/arm64/boot/dts/mediatek/mt7622.dtsi > > index 3a729d9..3768c28 100644 > > --- a/arch/arm64/boot/dts/mediatek/mt7622.dtsi > > +++ b/arch/arm64/boot/dts/mediatek/mt7622.dtsi > > @@ -8,6 +8,7 @@ > > > > #include <dt-bindings/interrupt-controller/irq.h> > > #include <dt-bindings/interrupt-controller/arm-gic.h> > > +#include "mt7622-clock.dtsi" > > > > / { > > compatible = "mediatek,mt7622"; > > ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2017-05-30 1:16 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-05-29 12:56 [PATCH v2 0/4] Add Basic SoC support for MT7622 sean.wang at mediatek.com 2017-05-29 12:56 ` [PATCH v2 1/4] dt-bindings: mediatek: add bindings for MediaTek MT7622 SoC sean.wang at mediatek.com 2017-05-29 15:11 ` Matthias Brugger 2017-05-29 15:11 ` Matthias Brugger 2017-05-29 12:56 ` [PATCH v2 2/4] arm64: dts: mt7622: add basic nodes to the mt7622.dtsi file sean.wang at mediatek.com 2017-05-29 12:56 ` [PATCH v2 3/4] arm64: dts: mt7622: add dts file for MT7622 reference board variant 1 sean.wang at mediatek.com 2017-05-29 19:09 ` kbuild test robot 2017-05-29 12:56 ` [PATCH v2 4/4] arm64: dts: mt7622: group clock DT nodes into separate DT file sean.wang at mediatek.com 2017-05-29 15:09 ` Matthias Brugger 2017-05-30 1:16 ` Sean Wang
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).