arm: Add basic support for MediaTek MT8127 & MT8135 SoCs
Subsystems (5)
v4 cover arm: Add basic support for MediaTek MT8127 & MT8135 SoCs
This is v4 of adding basic device tree support for MT8127 & MT8135. Since the last series didn't make it into v3.18-rc1, I updated the series and send it again. This series is based on v3.18-rc1
Besides changing patch subjects to make them look more consistence, I include config option patch original posted with mt8135 pinctrl support[1]. It seems more natural in this series and our works on clk & pmic will also need it.
We are planning to upstream drivers for MT8127 & MT8135 SoCs. MT8135 is a tablet soc featuring 2 CA7 + 2 CA15. MT8127 is a tablet soc with 4 CA7. They share many similar IP components with MT65xx series.
Changes in v4:
- Include config options for different mediatek SoCs.
Changes in v3:
- Changing earlyprintk order and dt-binding based on Matthias' suggestion.
v2:
- Change timer compatible string.
- Build dtb for mt8127 & mt8135
- Adding earlyprintk support for mt8127 & mt8135
[1] http://lists.infradead.org/pipermail/linux-arm-kernel/2014-September/289285.html
Joe.C (8):
ARM: mediatek: Add basic support for mt8127
ARM: mediatek: add dts for 8127 Moose board
ARM: mediatek: Add basic support for mt8135
ARM: mediatek: add dts for MT8135 evaluation board.
dt-bindings: add more chips in documentation for Mediatek SoC
ARM: dts: Build dtb for mt8127 & mt8135
ARM: mediatek: Add earlyprintk support for mt8127 & mt8135
ARM: mediatek: Add config option for mediatek SoCs.
Documentation/devicetree/bindings/arm/mediatek.txt | 15 ++- arch/arm/Kconfig.debug | 20 ++++ arch/arm/boot/dts/Makefile | 4 +- arch/arm/boot/dts/mt8127-moose.dts | 24 +++++ arch/arm/boot/dts/mt8127.dtsi | 93 +++++++++++++++++ arch/arm/boot/dts/mt8135-evbp1.dts | 24 +++++ arch/arm/boot/dts/mt8135.dtsi | 115 +++++++++++++++++++++ arch/arm/mach-mediatek/Kconfig | 22 +++- arch/arm/mach-mediatek/mediatek.c | 2 + 9 files changed, 312 insertions(+), 7 deletions(-) create mode 100644 arch/arm/boot/dts/mt8127-moose.dts create mode 100644 arch/arm/boot/dts/mt8127.dtsi create mode 100644 arch/arm/boot/dts/mt8135-evbp1.dts create mode 100644 arch/arm/boot/dts/mt8135.dtsi
PATCH v4 1/8 ARM: mediatek: Add basic support for mt8127
From: "Joe.C" <[email protected]>
This adds a basic dtsi for MT8127 SoC.
--- arch/arm/boot/dts/mt8127.dtsi | 93 +++++++++++++++++++++++++++++++++++++++ arch/arm/mach-mediatek/mediatek.c | 1 + 2 files changed, 94 insertions(+) create mode 100644 arch/arm/boot/dts/mt8127.dtsi
arch/arm/boot/dts/mt8127.dtsi new file +93 -0
@@ -0,0 +1,93 @@ +/* + * Copyright (c) 2014 MediaTek Inc. + * Author: Joe.C <[email protected]> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/interrupt-controller/arm-gic.h> +#include "skeleton64.dtsi" + +/ { + compatible = "mediatek,mt8127"; + interrupt-parent = <&gic>; + + cpus { + #address-cells = <1>; + #size-cells = <0>; + + cpu at 0 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x0>; + }; + cpu at 1 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x1>; + }; + cpu at 2 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x2>; + }; + cpu at 3 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x3>; + }; + + }; + + clocks { + #address-cells = <2>; + #size-cells = <2>; + compatible = "simple-bus"; + ranges; + + system_clk: dummy13m { + compatible = "fixed-clock"; + clock-frequency = <13000000>; + #clock-cells = <0>; + }; + + rtc_clk: dummy32k { + compatible = "fixed-clock"; + clock-frequency = <32000>; + #clock-cells = <0>; + }; + }; + + soc { + #address-cells = <2>; + #size-cells = <2>; + compatible = "simple-bus"; + ranges; + + timer: timer at 10008000 { + compatible = "mediatek,mt8127-timer", "mediatek,mt6577-timer"; + reg = <0 0x10008000 0 0x80>; + interrupts = <GIC_SPI 112 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&system_clk>, <&rtc_clk>; + clock-names = "system-clk", "rtc-clk"; + }; + + gic: interrupt-controller at 10211000 { + compatible = "arm,cortex-a7-gic"; + interrupt-controller; + #interrupt-cells = <3>; + reg = <0 0x10211000 0 0x1000>, + <0 0x10212000 0 0x1000>, + <0 0x10214000 0 0x2000>, + <0 0x10216000 0 0x2000>; + }; + }; +};
arch/arm/mach-mediatek/mediatek.c +1 -0
@@ -19,6 +19,7 @@static const char *const mediatek_board_dt_compat[] = {"mediatek,mt6589" , +"mediatek,mt8127" ,NULL , };
Signature · 1 line
1.8.1.1.dirty
Matthias Brugger
maintainerRe: [PATCH v4 1/8] ARM: mediatek: Add basic support for mt8127
2014-10-22 12:29 GMT+02:00 Joe.C <[email protected]>:
Quoted 24 lines
From: "Joe.C" <[email protected]>
This adds a basic dtsi for MT8127 SoC.
Signed-off-by Joe.C <[email protected]>--- arch/arm/boot/dts/mt8127.dtsi | 93 +++++++++++++++++++++++++++++++++++++++ arch/arm/mach-mediatek/mediatek.c | 1 + 2 files changed, 94 insertions(+) create mode 100644 arch/arm/boot/dts/mt8127.dtsiarch/arm/boot/dts/mt8127.dtsi new file +7
-0@@ -0,0 +1,93 @@ +/* + * Copyright (c) 2014 MediaTek Inc. + * Author: Joe.C <[email protected]> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation.
arch/arm/boot/dts/mt8127.dtsi line 17
Quoted 10 lines
arch/arm/boot/dts/mt8127.dtsi new file +10
-0+ * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/interrupt-controller/arm-gic.h> +#include "skeleton64.dtsi"
Cortex a7 is 32 bits, right? So why do you use skeleton64.dtsi?
Quoted 50 lines
arch/arm/boot/dts/mt8127.dtsi new file +50
-0+ +/ { + compatible = "mediatek,mt8127"; + interrupt-parent = <&gic>; + + cpus { + #address-cells = <1>; + #size-cells = <0>; + + cpu at 0 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x0>; + }; + cpu at 1 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x1>; + }; + cpu at 2 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x2>; + }; + cpu at 3 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x3>; + }; + + }; + + clocks { + #address-cells = <2>; + #size-cells = <2>; + compatible = "simple-bus"; + ranges; + + system_clk: dummy13m { + compatible = "fixed-clock"; + clock-frequency = <13000000>; + #clock-cells = <0>; + }; + + rtc_clk: dummy32k { + compatible = "fixed-clock"; + clock-frequency = <32000>; + #clock-cells = <0>; + }; + };
arch/arm/boot/dts/mt8127.dtsi line 77
Quoted 10 lines
arch/arm/boot/dts/mt8127.dtsi new file +10
-0+ + soc { + #address-cells = <2>; + #size-cells = <2>; + compatible = "simple-bus"; + ranges; + + timer: timer at 10008000 { + compatible = "mediatek,mt8127-timer", "mediatek,mt6577-timer"; + reg = <0 0x10008000 0 0x80>;
This should be "reg = <0x10008000 0x80>;" right?
Same in mt8135.dtsi
Quoted 31 lines
arch/arm/boot/dts/mt8127.dtsi new file +16
-0+ interrupts = <GIC_SPI 112 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&system_clk>, <&rtc_clk>; + clock-names = "system-clk", "rtc-clk"; + }; + + gic: interrupt-controller at 10211000 { + compatible = "arm,cortex-a7-gic"; + interrupt-controller; + #interrupt-cells = <3>; + reg = <0 0x10211000 0 0x1000>, + <0 0x10212000 0 0x1000>, + <0 0x10214000 0 0x2000>, + <0 0x10216000 0 0x2000>; + }; + }; +};arch/arm/mach-mediatek/mediatek.c +1
-1@@ -19,6 +19,7 @@static const char *const mediatek_board_dt_compat[] = {"mediatek,mt6589" , +"mediatek,mt8127" ,NULL , }; --1.8.1.1.dirty
Signature · 1 line
motzblog.wordpress.com
PATCH v4 1/8 ARM: mediatek: Add basic support for mt8127
Quoted 29 lines · Matthias Brugger
2014-10-22 12:29 GMT+02:00 Joe.C <[email protected]>:
Quoted 28 lines
From: "Joe.C" <[email protected]>
This adds a basic dtsi for MT8127 SoC.
Signed-off-by Joe.C <[email protected]>--- arch/arm/boot/dts/mt8127.dtsi | 93 +++++++++++++++++++++++++++++++++++++++ arch/arm/mach-mediatek/mediatek.c | 1 + 2 files changed, 94 insertions(+) create mode 100644 arch/arm/boot/dts/mt8127.dtsiarch/arm/boot/dts/mt8127.dtsi new file +11
-0@@ -0,0 +1,93 @@ +/* + * Copyright (c) 2014 MediaTek Inc. + * Author: Joe.C <[email protected]> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
Quoted 8 lines
arch/arm/boot/dts/mt8127.dtsi line 17
Quoted 6 lines
arch/arm/boot/dts/mt8127.dtsi new file +6
-0+ * GNU General Public License for more details. + */ + +#include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/interrupt-controller/arm-gic.h> +#include "skeleton64.dtsi"Cortex a7 is 32 bits, right? So why do you use
skeleton64.dtsi?
Cortex-A7 is 32-bit, but that doesn't mean it can only have 32-bit physical address. With LPAE enabled, we can have physical address more than 32 bits.
The main difference between "skeleton64.dtsi" and "skeleton.dtsi" is "#address-cells" property set to 2. Although there are few sources using "skeleton64.dtsi", some of them write "#address-cells = <2>" directly in order to have 64-bit address space. ARM's TC2 reference platform (vexpress-v2p-ca15_a7.dts) is an example.
Some of MediaTek ARMv7 SoCs support address space larger than 4GB. It will be convenient to share the sources if we all use 64-bit device tree.
Quoted 99 lines
Quoted 51 lines
arch/arm/boot/dts/mt8127.dtsi new file +50
-0+ +/ { + compatible = "mediatek,mt8127"; + interrupt-parent = <&gic>; + + cpus { + #address-cells = <1>; + #size-cells = <0>; + + cpu at 0 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x0>; + }; + cpu at 1 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x1>; + }; + cpu at 2 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x2>; + }; + cpu at 3 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x3>; + }; + + }; + + clocks { + #address-cells = <2>; + #size-cells = <2>; + compatible = "simple-bus"; + ranges; + + system_clk: dummy13m { + compatible = "fixed-clock"; + clock-frequency = <13000000>; + #clock-cells = <0>; + }; + + rtc_clk: dummy32k { + compatible = "fixed-clock"; + clock-frequency = <32000>; + #clock-cells = <0>; + }; + };arch/arm/boot/dts/mt8127.dtsi line 77
Quoted 10 lines
arch/arm/boot/dts/mt8127.dtsi new file +10
-0+ + soc { + #address-cells = <2>; + #size-cells = <2>; + compatible = "simple-bus"; + ranges; + + timer: timer at 10008000 { + compatible = "mediatek,mt8127-timer", "mediatek,mt6577-timer"; + reg = <0 0x10008000 0 0x80>;This should be "reg = <
0x100080000x80>;" right?
Same inmt8135.dtsiQuoted 31 lines
arch/arm/boot/dts/mt8127.dtsi new file +16
-0+ interrupts = <GIC_SPI 112 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&system_clk>, <&rtc_clk>; + clock-names = "system-clk", "rtc-clk"; + }; + + gic: interrupt-controller at 10211000 { + compatible = "arm,cortex-a7-gic"; + interrupt-controller; + #interrupt-cells = <3>; + reg = <0 0x10211000 0 0x1000>, + <0 0x10212000 0 0x1000>, + <0 0x10214000 0 0x2000>, + <0 0x10216000 0 0x2000>; + }; + }; +};arch/arm/mach-mediatek/mediatek.c +1
-1@@ -19,6 +19,7 @@static const char *const mediatek_board_dt_compat[] = {"mediatek,mt6589" , +"mediatek,mt8127" ,NULL , }; --1.8.1.1.dirty
Arnd Bergmann
maintainerRe: [PATCH v4 1/8] ARM: mediatek: Add basic support for mt8127
Quoted 17 lines · On Tuesday 04 November 2014 14:36:45 HC Yen wrote:
Quoted 6 lines
arch/arm/boot/dts/mt8127.dtsi line 17
Quoted 4 lines
arch/arm/boot/dts/mt8127.dtsi new file +4
-0+ +#include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/interrupt-controller/arm-gic.h> +#include "skeleton64.dtsi"Cortex a7 is 32 bits, right? So why do you use
skeleton64.dtsi?Cortex-A7 is 32-bit, but that doesn't mean it can only have 32-bit physical address. With LPAE enabled, we can have physical address more than 32 bits.
The main difference between "
skeleton64.dtsi" and "skeleton.dtsi" is "#address-cells" property set to 2. Although there are few sources using "skeleton64.dtsi", some of them write "#address-cells = <2>" directly in order to have 64-bit address space. ARM's TC2 reference platform (vexpress-v2p-ca15_a7.dts) is an example.
Quoted 3 lines
Some of MediaTek ARMv7 SoCs support address space larger than 4GB. It will be convenient to share the sources if we all use 64-bit device tree.
Right, in general, I'd use #address-cells=<2> for Cortex-A7/A15/A17.
Arnd
Matthias Brugger
maintainerRe: [PATCH v4 1/8] ARM: mediatek: Add basic support for mt8127
2014-11-04 8:39 GMT+01:00 Arnd Bergmann <[email protected]>:
Quoted 18 lines · On Tuesday 04 November 2014 14:36:45 HC Yen wrote:
Quoted 17 lines
Quoted 6 lines
arch/arm/boot/dts/mt8127.dtsi line 17
Quoted 4 lines
arch/arm/boot/dts/mt8127.dtsi new file +4
-0+ +#include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/interrupt-controller/arm-gic.h> +#include "skeleton64.dtsi"Cortex a7 is 32 bits, right? So why do you use
skeleton64.dtsi?Cortex-A7 is 32-bit, but that doesn't mean it can only have 32-bit physical address. With LPAE enabled, we can have physical address more than 32 bits.
The main difference between "
skeleton64.dtsi" and "skeleton.dtsi" is "#address-cells" property set to 2. Although there are few sources using "skeleton64.dtsi", some of them write "#address-cells = <2>" directly in order to have 64-bit address space. ARM's TC2 reference platform (vexpress-v2p-ca15_a7.dts) is an example.
Quoted 5 lines
Quoted 3 lines
Some of MediaTek ARMv7 SoCs support address space larger than 4GB. It will be convenient to share the sources if we all use 64-bit device tree.
Right, in general, I'd use #address-cells=<2> for Cortex-A7/A15/A17.
Alright, thanks for clarification. So we should use skeleton64.dtsi for mt6589 as well, right?
Quoted 2 lines
Arnd
Signature · 1 line
motzblog.wordpress.com
Arnd Bergmann
maintainerRe: [PATCH v4 1/8] ARM: mediatek: Add basic support for mt8127
Quoted 19 lines · On Tuesday 04 November 2014 09:36:39 Matthias Brugger wrote:
2014-11-04 8:39 GMT+01:00 Arnd Bergmann <[email protected]>:
Quoted 18 lines · On Tuesday 04 November 2014 14:36:45 HC Yen wrote:
Quoted 17 lines
Quoted 6 lines
arch/arm/boot/dts/mt8127.dtsi line 17
Quoted 4 lines
arch/arm/boot/dts/mt8127.dtsi new file +4
-0+ +#include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/interrupt-controller/arm-gic.h> +#include "skeleton64.dtsi"Cortex a7 is 32 bits, right? So why do you use
skeleton64.dtsi?Cortex-A7 is 32-bit, but that doesn't mean it can only have 32-bit physical address. With LPAE enabled, we can have physical address more than 32 bits.
The main difference between "
skeleton64.dtsi" and "skeleton.dtsi" is "#address-cells" property set to 2. Although there are few sources using "skeleton64.dtsi", some of them write "#address-cells = <2>" directly in order to have 64-bit address space. ARM's TC2 reference platform (vexpress-v2p-ca15_a7.dts) is an example.
Quoted 8 lines
Quoted 5 lines
Quoted 3 lines
Some of MediaTek ARMv7 SoCs support address space larger than 4GB. It will be convenient to share the sources if we all use 64-bit device tree.
Right, in general, I'd use #address-cells=<2> for Cortex-A7/A15/A17.
Alright, thanks for clarification. So we should use
skeleton64.dtsifor mt6589 as well, right?
If the chip is capable of accessing memory or registers higher than 4GB physical address, then you should, yes.
If the SoC has limitations that mean you can't have higher addresses anyway, then either way is fine, no need to change it.
Arnd
PATCH v4 2/8 ARM: mediatek: add dts for 8127 Moose board
In reply to Joe.CFrom: "Joe.C" <[email protected]>
Moose is a tablet evalutation board based on MT8127 SoC.
--- arch/arm/boot/dts/mt8127-moose.dts | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 arch/arm/boot/dts/mt8127-moose.dts
arch/arm/boot/dts/mt8127-moose.dts new file +24 -0
@@ -0,0 +1,24 @@ +/* + * Copyright (c) 2014 MediaTek Inc. + * Author: Joe.C <[email protected]> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +/dts-v1/; +#include "mt8127.dtsi" + +/ { + model = "mediatek,mt8127-moose", "mediatek,mt8127"; + + memory { + reg = <0 0x80000000 0 0x40000000>; + }; +};
Signature · 1 line
1.8.1.1.dirty
Matthias Brugger
maintainerRe: [PATCH v4 2/8] ARM: mediatek: add dts for 8127 Moose board
2014-10-22 12:29 GMT+02:00 Joe.C <[email protected]>:
Quoted 25 lines
From: "Joe.C" <[email protected]>
Moose is a tablet evalutation board based on MT8127 SoC.
Signed-off-by Joe.C <[email protected]>--- arch/arm/boot/dts/mt8127-moose.dts | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 arch/arm/boot/dts/mt8127-moose.dtsarch/arm/boot/dts/mt8127-moose.dts new file +9
-0@@ -0,0 +1,24 @@ +/* + * Copyright (c) 2014 MediaTek Inc. + * Author: Joe.C <[email protected]> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful,
arch/arm/boot/dts/mt8127-moose.dts line 19
Quoted 10 lines
arch/arm/boot/dts/mt8127-moose.dts new file +10
-0+ * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +/dts-v1/; +#include "mt8127.dtsi" + +/ { + model = "mediatek,mt8127-moose", "mediatek,mt8127";
The model should be a string that identifies the board, e.g. "Moose". Please add the compatible property.
arch/arm/boot/dts/mt8127-moose.dts line 22
Quoted 3 lines
arch/arm/boot/dts/mt8127-moose.dts new file +3
-0+ + memory { + reg = <0 0x80000000 0 0x40000000>;
See comment on mt8127 register definitions.
Quoted 5 lines
arch/arm/boot/dts/mt8127-moose.dts new file +2
-1+ }; +}; --1.8.1.1.dirty
Signature · 1 line
motzblog.wordpress.com
Re: [PATCH v4 2/8] ARM: mediatek: add dts for 8127 Moose board
Quoted 31 lines · Matthias Brugger
2014-10-22 12:29 GMT+02:00 Joe.C <[email protected]>:
Quoted 30 lines
From: "Joe.C" <[email protected]>
Moose is a tablet evalutation board based on MT8127 SoC.
Signed-off-by Joe.C <[email protected]>--- arch/arm/boot/dts/mt8127-moose.dts | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 arch/arm/boot/dts/mt8127-moose.dtsarch/arm/boot/dts/mt8127-moose.dts new file +14
-0@@ -0,0 +1,24 @@ +/* + * Copyright (c) 2014 MediaTek Inc. + * Author: Joe.C <[email protected]> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ +
Quoted 8 lines
arch/arm/boot/dts/mt8127-moose.dts line 19
Quoted 5 lines
arch/arm/boot/dts/mt8127-moose.dts new file +5
-0+/dts-v1/; +#include "mt8127.dtsi" + +/ { + model = "mediatek,mt8127-moose", "mediatek,mt8127";The model should be a string that identifies the board, e.g. "Moose". Please add the compatible property.
Thanks for catching this. I just note we have warning on this line. Will fix in next version.
Joe.C
PATCH v4 3/8 ARM: mediatek: Add basic support for mt8135
In reply to Joe.CFrom: "Joe.C" <[email protected]>
This adds a basic dtsi for MT8135 based SoC.
--- arch/arm/boot/dts/mt8135.dtsi | 115 ++++++++++++++++++++++++++++++++++++++ arch/arm/mach-mediatek/mediatek.c | 1 + 2 files changed, 116 insertions(+) create mode 100644 arch/arm/boot/dts/mt8135.dtsi
arch/arm/boot/dts/mt8135.dtsi new file +115 -0
@@ -0,0 +1,115 @@ +/* + * Copyright (c) 2014 MediaTek Inc. + * Author: Joe.C <[email protected]> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/interrupt-controller/arm-gic.h> +#include "skeleton64.dtsi" + +/ { + compatible = "mediatek,mt8135"; + interrupt-parent = <&gic>; + + cpu-map { + cluster0 { + core0 { + cpu = <&cpu0>; + }; + core1 { + cpu = <&cpu1>; + }; + }; + + cluster1 { + core0 { + cpu = <&cpu2>; + }; + core1 { + cpu = <&cpu3>; + }; + }; + }; + + cpus { + #address-cells = <1>; + #size-cells = <0>; + + cpu0: cpu at 0 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x000>; + }; + + cpu1: cpu at 1 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x001>; + }; + + cpu2: cpu at 100 { + device_type = "cpu"; + compatible = "arm,cortex-a15"; + reg = <0x100>; + }; + + cpu3: cpu at 101 { + device_type = "cpu"; + compatible = "arm,cortex-a15"; + reg = <0x101>; + }; + }; + + clocks { + #address-cells = <2>; + #size-cells = <2>; + compatible = "simple-bus"; + ranges; + + system_clk: dummy13m { + compatible = "fixed-clock"; + clock-frequency = <13000000>; + #clock-cells = <0>; + }; + + rtc_clk: dummy32k { + compatible = "fixed-clock"; + clock-frequency = <32000>; + #clock-cells = <0>; + }; + }; + + soc { + #address-cells = <2>; + #size-cells = <2>; + compatible = "simple-bus"; + ranges; + + timer: timer at 10008000 { + compatible = "mediatek,mt8135-timer", "mediatek,mt6577-timer"; + reg = <0 0x10008000 0 0x80>; + interrupts = <GIC_SPI 113 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&system_clk>, <&rtc_clk>; + clock-names = "system-clk", "rtc-clk"; + }; + + gic: interrupt-controller at 10211000 { + compatible = "arm,cortex-a15-gic"; + interrupt-controller; + #interrupt-cells = <3>; + reg = <0 0x10211000 0 0x1000>, + <0 0x10212000 0 0x1000>, + <0 0x10214000 0 0x2000>, + <0 0x10216000 0 0x2000>; + }; + }; +};
arch/arm/mach-mediatek/mediatek.c +1 -0
@@ -20,6 +20,7 @@static const char *const mediatek_board_dt_compat[] = {"mediatek,mt6589" ,"mediatek,mt8127" , +"mediatek,mt8135" ,NULL , };
Signature · 1 line
1.8.1.1.dirty
Matthias Brugger
maintainerRe: [PATCH v4 3/8] ARM: mediatek: Add basic support for mt8135
2014-10-22 12:29 GMT+02:00 Joe.C <[email protected]>:
Quoted 24 lines
From: "Joe.C" <[email protected]>
This adds a basic dtsi for MT8135 based SoC.
Signed-off-by Joe.C <[email protected]>--- arch/arm/boot/dts/mt8135.dtsi | 115 ++++++++++++++++++++++++++++++++++++++ arch/arm/mach-mediatek/mediatek.c | 1 + 2 files changed, 116 insertions(+) create mode 100644 arch/arm/boot/dts/mt8135.dtsiarch/arm/boot/dts/mt8135.dtsi new file +7
-0@@ -0,0 +1,115 @@ +/* + * Copyright (c) 2014 MediaTek Inc. + * Author: Joe.C <[email protected]> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation.
arch/arm/boot/dts/mt8135.dtsi line 17
Quoted 10 lines
arch/arm/boot/dts/mt8135.dtsi new file +10
-0+ * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/interrupt-controller/arm-gic.h> +#include "skeleton64.dtsi"
Same here A15 is 32 bits.
Quoted 113 lines
arch/arm/boot/dts/mt8135.dtsi new file +98
-0+ +/ { + compatible = "mediatek,mt8135"; + interrupt-parent = <&gic>; + + cpu-map { + cluster0 { + core0 { + cpu = <&cpu0>; + }; + core1 { + cpu = <&cpu1>; + }; + }; + + cluster1 { + core0 { + cpu = <&cpu2>; + }; + core1 { + cpu = <&cpu3>; + }; + }; + }; + + cpus { + #address-cells = <1>; + #size-cells = <0>; + + cpu0: cpu at 0 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x000>; + }; + + cpu1: cpu at 1 { + device_type = "cpu"; + compatible = "arm,cortex-a7"; + reg = <0x001>; + }; + + cpu2: cpu at 100 { + device_type = "cpu"; + compatible = "arm,cortex-a15"; + reg = <0x100>; + }; + + cpu3: cpu at 101 { + device_type = "cpu"; + compatible = "arm,cortex-a15"; + reg = <0x101>; + }; + }; + + clocks { + #address-cells = <2>; + #size-cells = <2>; + compatible = "simple-bus"; + ranges; + + system_clk: dummy13m { + compatible = "fixed-clock"; + clock-frequency = <13000000>; + #clock-cells = <0>; + }; + + rtc_clk: dummy32k { + compatible = "fixed-clock"; + clock-frequency = <32000>; + #clock-cells = <0>; + }; + }; + + soc { + #address-cells = <2>; + #size-cells = <2>; + compatible = "simple-bus"; + ranges; + + timer: timer at 10008000 { + compatible = "mediatek,mt8135-timer", "mediatek,mt6577-timer"; + reg = <0 0x10008000 0 0x80>; + interrupts = <GIC_SPI 113 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&system_clk>, <&rtc_clk>; + clock-names = "system-clk", "rtc-clk"; + }; + + gic: interrupt-controller at 10211000 { + compatible = "arm,cortex-a15-gic"; + interrupt-controller; + #interrupt-cells = <3>; + reg = <0 0x10211000 0 0x1000>, + <0 0x10212000 0 0x1000>, + <0 0x10214000 0 0x2000>, + <0 0x10216000 0 0x2000>; + }; + }; +};arch/arm/mach-mediatek/mediatek.c +1
-1@@ -20,6 +20,7 @@static const char *const mediatek_board_dt_compat[] = {"mediatek,mt6589" ,"mediatek,mt8127" , +"mediatek,mt8135" ,NULL , }; --1.8.1.1.dirty
Signature · 1 line
motzblog.wordpress.com
PATCH v4 4/8 ARM: mediatek: add dts for MT8135 evaluation board.
In reply to Joe.CFrom: "Joe.C" <[email protected]>
Evbp1 is a tablet evaluation board based on MT8135 SoC.
--- arch/arm/boot/dts/mt8135-evbp1.dts | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 arch/arm/boot/dts/mt8135-evbp1.dts
arch/arm/boot/dts/mt8135-evbp1.dts new file +24 -0
@@ -0,0 +1,24 @@ +/* + * Copyright (c) 2014 MediaTek Inc. + * Author: Joe.C <[email protected]> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +/dts-v1/; +#include "mt8135.dtsi" + +/ { + model = "mediatek,mt8135-evbp1", "mediatek,mt8135"; + + memory { + reg = <0 0x80000000 0 0x40000000>; + }; +};
Signature · 1 line
1.8.1.1.dirty
Matthias Brugger
maintainerRe: [PATCH v4 4/8] ARM: mediatek: add dts for MT8135 evaluation board.
2014-10-22 12:29 GMT+02:00 Joe.C <[email protected]>:
Quoted 25 lines
From: "Joe.C" <[email protected]>
Evbp1 is a tablet evaluation board based on MT8135 SoC.
Signed-off-by Joe.C <[email protected]>--- arch/arm/boot/dts/mt8135-evbp1.dts | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 arch/arm/boot/dts/mt8135-evbp1.dtsarch/arm/boot/dts/mt8135-evbp1.dts new file +9
-0@@ -0,0 +1,24 @@ +/* + * Copyright (c) 2014 MediaTek Inc. + * Author: Joe.C <[email protected]> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful,
arch/arm/boot/dts/mt8135-evbp1.dts line 19
Quoted 10 lines
arch/arm/boot/dts/mt8135-evbp1.dts new file +10
-0+ * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +/dts-v1/; +#include "mt8135.dtsi" + +/ { + model = "mediatek,mt8135-evbp1", "mediatek,mt8135";
Same here, you switched model and compatible property. Please provide both of them.
Quoted 8 lines
arch/arm/boot/dts/mt8135-evbp1.dts new file +5
-1+ + memory { + reg = <0 0x80000000 0 0x40000000>; + }; +}; --1.8.1.1.dirty
Signature · 1 line
motzblog.wordpress.com
PATCH v4 5/8 dt-bindings: add more chips in documentation for Mediatek SoC
In reply to Joe.CFrom: "Joe.C" <[email protected]>
Add MT8127 & MT8135 from Mediatek.
--- Documentation/devicetree/bindings/arm/mediatek.txt | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-)
Documentation/devicetree/bindings/arm/mediatek.txt +12 -3
@@ -1,10 +1,13 @@ -Mediatek MT6589 Platforms Device Tree Bindings +Mediatek Platforms Device Tree Bindings -Boards with a SoC of the Mediatek MT6589 shall have the following property: +Boards with a SoC of the Mediatek shall have the following property: Required root node property: -compatible: must contain "mediatek,mt6589" +compatible: Must contain one of + "mediatek,mt6589" + "mediatek,mt8127" + "mediatek,mt8135" Supported boards:
@@ -12,3 +15,9 @@ Supported boards: - bq Aquaris5 smart phone: Required root node properties: - compatible = "mundoreader,bq-aquaris5", "mediatek,mt6589"; +- MTK mt8127 tablet moose EVB: + Required root node properties: + - compatible = "mediatek,mt8127-moose", "mediatek,mt8127"; +- MTK mt8135 tablet EVB: + Required root node properties: + - compatible = "mediatek,mt8135-evbp1", "mediatek,mt8135";
Signature · 1 line
1.8.1.1.dirty
PATCH v4 6/8 ARM: dts: Build dtb for mt8127 & mt8135
In reply to Joe.CFrom: "Joe.C" <[email protected]>
This allows the "make dtbs" to build the moose and mt8135-evbp1 for MediaTek SoC
--- arch/arm/boot/dts/Makefile | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
arch/arm/boot/dts/Makefile +3 -1
@@ -515,7 +515,9 @@ dtb-$(CONFIG_MACH_DOVE) += dove-cm-a510.dtb \ dove-d2plug.dtb \ dove-d3plug.dtb \ dove-dove-db.dtb -dtb-$(CONFIG_ARCH_MEDIATEK) += mt6589-aquaris5.dtb +dtb-$(CONFIG_ARCH_MEDIATEK) += mt6589-aquaris5.dtb \ + mt8127-moose.dtb \ + mt8135-evbp1.dtb targets += dtbs dtbs_install targets += $(dtb-y)
Signature · 1 line
1.8.1.1.dirty
PATCH v4 7/8 ARM: mediatek: Add earlyprintk support for mt8127 & mt8135
In reply to Joe.CFrom: "Joe.C" <[email protected]>
Enable low-level debug for Mediatek mt8127 & mt8135 SoC.
--- arch/arm/Kconfig.debug | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+)
arch/arm/Kconfig.debug +20 -0
@@ -866,6 +866,22 @@ choice Say Y here if you want kernel low-level debugging support for Mediatek mt6589 based platforms on UART0. + config DEBUG_MT8127_UART0 + bool "Mediatek mt8127 UART0" + depends on ARCH_MEDIATEK + select DEBUG_UART_8250 + help + Say Y here if you want kernel low-level debugging support + for Mediatek mt8127 based platforms on UART0. + + config DEBUG_MT8135_UART3 + bool "Mediatek mt8135 UART3" + depends on ARCH_MEDIATEK + select DEBUG_UART_8250 + help + Say Y here if you want kernel low-level debugging support + for Mediatek mt8135 based platforms on UART3. + config DEBUG_VEXPRESS_UART0_DETECT bool "Autodetect UART0 on Versatile Express Cortex-A core tiles" depends on ARCH_VEXPRESS && CPU_CP15_MMU
@@ -1113,7 +1129,9 @@ config DEBUG_UART_PHYS default 0x10126000 if DEBUG_RK3X_UART1 default 0x101f1000 if ARCH_VERSATILE default 0x101fb000 if DEBUG_NOMADIK_UART + default 0x11002000 if DEBUG_MT8127_UART0 default 0x11006000 if DEBUG_MT6589_UART0 + default 0x11009000 if DEBUG_MT8135_UART3 default 0x16000000 if ARCH_INTEGRATOR default 0x18000300 if DEBUG_BCM_5301X default 0x1c090000 if DEBUG_VEXPRESS_UART0_RS1
@@ -1181,7 +1199,9 @@ config DEBUG_UART_VIRT default 0xf01fb000 if DEBUG_NOMADIK_UART default 0xf0201000 if DEBUG_BCM2835 default 0xf1000300 if DEBUG_BCM_5301X + default 0xf1002000 if DEBUG_MT8127_UART0 default 0xf1006000 if DEBUG_MT6589_UART0 + default 0xf1009000 if DEBUG_MT8135_UART3 default 0xf11f1000 if ARCH_VERSATILE default 0xf1600000 if ARCH_INTEGRATOR default 0xf1c28000 if DEBUG_SUNXI_UART0
Signature · 1 line
1.8.1.1.dirty
PATCH v4 8/8 ARM: mediatek: Add config option for mediatek SoCs.
In reply to Joe.CFrom: "Joe.C" <[email protected]>
Add config options for ARCH_MT6589, ARCH_MT8127, ARCH_MT8135. The upcoming pinctrl, clk drivers need to isolate SoCs support base on these options.
--- arch/arm/mach-mediatek/Kconfig | 22 +++++++++++++++++++--- 1 file changed, 19 insertions(+), 3 deletions(-)
arch/arm/mach-mediatek/Kconfig +19 -3
@@ -1,6 +1,22 @@ -config ARCH_MEDIATEK - bool "Mediatek MT6589 SoC" if ARCH_MULTI_V7 +menuconfig ARCH_MEDIATEK + bool "Mediatek MT65xx & MT81xx SoC" if ARCH_MULTI_V7 select ARM_GIC select MTK_TIMER help - Support for Mediatek Cortex-A7 Quad-Core-SoC MT6589. + Support for Mediatek MT65xx & MT81xx SoCs + +if ARCH_MEDIATEK + +config MACH_MT6589 + bool "MediaTek MT6589 SoCs support" + default ARCH_MEDIATEK + +config MACH_MT8127 + bool "MediaTek MT8127 SoCs support" + default ARCH_MEDIATEK + +config MACH_MT8135 + bool "MediaTek MT8135 SoCs support" + default ARCH_MEDIATEK + +endif
Signature · 1 line
1.8.1.1.dirty
Matthias Brugger
maintainerRe: [PATCH v4 8/8] ARM: mediatek: Add config option for mediatek SoCs.
2014-10-22 12:29 GMT+02:00 Joe.C <[email protected]>:
Quoted 31 lines
From: "Joe.C" <[email protected]>
Add config options for
ARCH_MT6589,ARCH_MT8127,ARCH_MT8135. The upcoming pinctrl, clk drivers need to isolate SoCs support base on these options.Signed-off-by Joe.C <[email protected]>--- arch/arm/mach-mediatek/Kconfig | 22 +++++++++++++++++++--- 1 file changed, 19 insertions(+), 3 deletions(-)arch/arm/mach-mediatek/Kconfig +9
-3@@ -1,6 +1,22 @@ -config ARCH_MEDIATEK - bool "Mediatek MT6589 SoC" if ARCH_MULTI_V7 +menuconfig ARCH_MEDIATEK + bool "Mediatek MT65xx & MT81xx SoC" if ARCH_MULTI_V7 select ARM_GIC select MTK_TIMER help - Support for Mediatek Cortex-A7 Quad-Core-SoC MT6589. + Support for Mediatek MT65xx & MT81xx SoCs + +if ARCH_MEDIATEK + +config MACH_MT6589 + bool "MediaTek MT6589 SoCs support" + default ARCH_MEDIATEK
arch/arm/mach-mediatek/Kconfig line 22
Quoted 10 lines
arch/arm/mach-mediatek/Kconfig +10
-0+ +config MACH_MT8127 + bool "MediaTek MT8127 SoCs support" + default ARCH_MEDIATEK + +config MACH_MT8135 + bool "MediaTek MT8135 SoCs support" + default ARCH_MEDIATEK + +endif
MACH_MTxxxx are not used nowhere, please remove them.
Quoted 3 lines
--
1.8.1.1.dirty
Signature · 1 line
motzblog.wordpress.com
Re: [PATCH v4 8/8] ARM: mediatek: Add config option for mediatek SoCs.
Hi Matthias,
Quoted 11 lines · Matthias Brugger
2014-10-22 12:29 GMT+02:00 Joe.C <[email protected]>:
Quoted 10 lines
arch/arm/mach-mediatek/Kconfig +10
-0+ +if ARCH_MEDIATEK + +config MACH_MT6589 + bool "MediaTek MT6589 SoCs support" + default ARCH_MEDIATEK + +config MACH_MT8127 + bool "MediaTek MT8127 SoCs support" + default ARCH_MEDIATEK
Quoted 8 lines
arch/arm/mach-mediatek/Kconfig line 22
Quoted 6 lines
arch/arm/mach-mediatek/Kconfig +6
-0+ +config MACH_MT8135 + bool "MediaTek MT8135 SoCs support" + default ARCH_MEDIATEK + +endif
MACH_MTxxxxare not used nowhere, please remove them.
Yes, they are not used in this series yet.
MACH_MT8135 was used in mt8135 pinctrl driver [1] and will be used in mt8135 clk driver we are about to send. I don't want to create false dependency between these 2 series so I put it here.
Do you prefer add them when they are first used?
Joe.C
[1]
http://lists.infradead.org/pipermail/linux-arm-kernel/2014-September/289288.html
Matthias Brugger
maintainerRe: [PATCH v4 8/8] ARM: mediatek: Add config option for mediatek SoCs.
Hi Joe,
2014-10-22 15:31 GMT+02:00 Yingjoe Chen <[email protected]>:
Quoted 24 lines
Hi Matthias,
Quoted 11 lines · Matthias Brugger
2014-10-22 12:29 GMT+02:00 Joe.C <[email protected]>:
Quoted 10 lines
arch/arm/mach-mediatek/Kconfig +10
-0+ +if ARCH_MEDIATEK + +config MACH_MT6589 + bool "MediaTek MT6589 SoCs support" + default ARCH_MEDIATEK + +config MACH_MT8127 + bool "MediaTek MT8127 SoCs support" + default ARCH_MEDIATEKQuoted 8 lines
arch/arm/mach-mediatek/Kconfig line 22
Quoted 6 lines
arch/arm/mach-mediatek/Kconfig +6
-0+ +config MACH_MT8135 + bool "MediaTek MT8135 SoCs support" + default ARCH_MEDIATEK + +endif
MACH_MTxxxxare not used nowhere, please remove them.
Quoted 6 lines
Yes, they are not used in this series yet.
MACH_MT8135was used in mt8135 pinctrl driver [1] and will be used in mt8135 clk driver we are about to send. I don't want to create false dependency between these 2 series so I put it here.
Do you prefer add them when they are first used?
Yes please add them, when they are first used.
Today I had a look on the pinctrl patches you mentioned. It looks to me as if the config is not needed in the series.
Thanks,
Matthias
[0] http://thread.gmane.org/gmane.linux.ports.arm.kernel/358505
Quoted 7 lines
Joe.C
[1]
http://lists.infradead.org/pipermail/linux-arm-kernel/2014-September/289288.html
Signature · 1 line
motzblog.wordpress.com
Re: [PATCH v4 8/8] ARM: mediatek: Add config option for mediatek SoCs.
Quoted 29 lines · Matthias Brugger
Hi Joe,
2014-10-22 15:31 GMT+02:00 Yingjoe Chen <[email protected]>:
Quoted 26 lines
Hi Matthias,
Quoted 11 lines · Matthias Brugger
2014-10-22 12:29 GMT+02:00 Joe.C <[email protected]>:
Quoted 10 lines
arch/arm/mach-mediatek/Kconfig +10
-0+ +if ARCH_MEDIATEK + +config MACH_MT6589 + bool "MediaTek MT6589 SoCs support" + default ARCH_MEDIATEK + +config MACH_MT8127 + bool "MediaTek MT8127 SoCs support" + default ARCH_MEDIATEKQuoted 8 lines
arch/arm/mach-mediatek/Kconfig line 22
Quoted 6 lines
arch/arm/mach-mediatek/Kconfig +6
-0+ +config MACH_MT8135 + bool "MediaTek MT8135 SoCs support" + default ARCH_MEDIATEK + +endif
MACH_MTxxxxare not used nowhere, please remove them.Yes, they are not used in this series yet.
Quoted 8 lines
Quoted 4 lines
MACH_MT8135was used in mt8135 pinctrl driver [1] and will be used in mt8135 clk driver we are about to send. I don't want to create false dependency between these 2 series so I put it here.
Do you prefer add them when they are first used?Yes please add them, when they are first used.
Today I had a look on the pinctrl patches you mentioned. It looks to me as if the config is not needed in the series.
OK, I'll remove these in next version.
Joe.C
Quoted 16 lines
Thanks,
Matthias[0] http://thread.gmane.org/gmane.linux.ports.arm.kernel/358505
Quoted 7 lines
Joe.C
[1]
http://lists.infradead.org/pipermail/linux-arm-kernel/2014-September/289288.html
PATCH v4 8/8 ARM: mediatek: Fix description for mediatek SoCs
In reply to Matthias BruggerFrom: "Joe.C" <[email protected]>
We support more MediaTek SoCs now, update the description.
--- arch/arm/mach-mediatek/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
(quoted diff) +0 -0
b/arch/arm/mach-mediatek/Kconfig
index 2c043a2..f73f588 100644
arch/arm/mach-mediatek/Kconfig +2 -2
@@ -1,6 +1,6 @@ config ARCH_MEDIATEK - bool "Mediatek MT6589 SoC" if ARCH_MULTI_V7 + bool "Mediatek MT65xx & MT81xx SoC" if ARCH_MULTI_V7 select ARM_GIC select MTK_TIMER help - Support for Mediatek Cortex-A7 Quad-Core-SoC MT6589. + Support for Mediatek MT65xx & MT81xx SoCs
Signature · 1 line
1.8.1.1.dirty
Re: [PATCH v4 8/8] ARM: mediatek: Fix description for mediatek SoCs
Hi Matthias,
Is this series + this new 8/8 patch OK?
Anything else that I should change?
Joe.C
Quoted 23 lines · Yingjoe Chen
From: "Joe.C" <[email protected]>
We support more MediaTek SoCs now, update the description.
Signed-off-by Joe.C <[email protected]>--- arch/arm/mach-mediatek/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)arch/arm/mach-mediatek/Kconfig +2
-2@@ -1,6 +1,6 @@ config ARCH_MEDIATEK - bool "Mediatek MT6589 SoC" if ARCH_MULTI_V7 + bool "Mediatek MT65xx & MT81xx SoC" if ARCH_MULTI_V7 select ARM_GIC select MTK_TIMER help - Support for Mediatek Cortex-A7 Quad-Core-SoC MT6589. + Support for Mediatek MT65xx & MT81xx SoCs
Matthias Brugger
maintainerRe: [PATCH v4 8/8] ARM: mediatek: Fix description for mediatek SoCs
Hi Joe,
2014-10-31 2:37 GMT+01:00 Yingjoe Chen <[email protected]>:
Quoted 5 lines
Hi Matthias,
Is this series + this new 8/8 patch OK?
Anything else that I should change?
I checked the patches on my mt8135 board and they work for me. So from my site everything fine.
I added the patches to my tree [0] branches v3.19-next/soc and v3.19-next/dts I made a small change to the commit message of patch 8 and I will try to send the pull request asap.
Thanks,
Matthias
[0] https://github.com/mbgg/linux-mediatek
Quoted 29 lines
Joe.C
Quoted 23 lines · Yingjoe Chen
From: "Joe.C" <[email protected]>
We support more MediaTek SoCs now, update the description.
Signed-off-by Joe.C <[email protected]>--- arch/arm/mach-mediatek/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)arch/arm/mach-mediatek/Kconfig +2
-2@@ -1,6 +1,6 @@ config ARCH_MEDIATEK - bool "Mediatek MT6589 SoC" if ARCH_MULTI_V7 + bool "Mediatek MT65xx & MT81xx SoC" if ARCH_MULTI_V7 select ARM_GIC select MTK_TIMER help - Support for Mediatek Cortex-A7 Quad-Core-SoC MT6589. + Support for Mediatek MT65xx & MT81xx SoCs
Signature · 1 line
motzblog.wordpress.com
Matthias Brugger
maintainerRe: [PATCH v4 8/8] ARM: mediatek: Fix description for mediatek SoCs
2014-10-31 15:53 GMT+01:00 Matthias Brugger <[email protected]>:
Quoted 11 lines
Hi Joe,
2014-10-31 2:37 GMT+01:00 Yingjoe Chen <[email protected]>:
Quoted 5 lines
Hi Matthias,
Is this series + this new 8/8 patch OK?
Anything else that I should change?I checked the patches on my mt8135 board and they work for me. So from my site everything fine.
I just had another look on them and I think they are not correct, please see my comments in the other mail.
Quoted 44 lines
I added the patches to my tree [0] branches v3.19-next/soc and v3.19-next/dts I made a small change to the commit message of patch 8 and I will try to send the pull request asap.
Thanks,
Matthias[0] https://github.com/mbgg/linux-mediatek
Quoted 12 lines
Joe.C
Quoted 9 lines · Yingjoe Chen
From: "Joe.C" <[email protected]>
We support more MediaTek SoCs now, update the description.
Signed-off-by Joe.C <[email protected]>--- arch/arm/mach-mediatek/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)Quoted 17 lines
Quoted 6 lines
arch/arm/mach-mediatek/Kconfig +0
-0arch/arm/mach-mediatek/Kconfig line 6
Quoted 8 lines
arch/arm/mach-mediatek/Kconfig +2
-2@@ -1,6 +1,6 @@ config ARCH_MEDIATEK - bool "Mediatek MT6589 SoC" if ARCH_MULTI_V7 + bool "Mediatek MT65xx & MT81xx SoC" if ARCH_MULTI_V7 select ARM_GIC select MTK_TIMER help - Support for Mediatek Cortex-A7 Quad-Core-SoC MT6589. + Support for Mediatek MT65xx & MT81xx SoCs--
motzblog.wordpress.com
Signature · 1 line
motzblog.wordpress.com