* [PATCH v2 0/2] Add Mediatek X20 Development Board support @ 2018-06-05 16:34 Manivannan Sadhasivam 2018-06-05 16:34 ` [PATCH v2 1/2] dt-bindings: arm: mediatek: Document Mediatek X20 Development Board Manivannan Sadhasivam 2018-06-05 16:34 ` [PATCH v2 2/2] arm64: dts: Add Mediatek X20 Development Board support Manivannan Sadhasivam 0 siblings, 2 replies; 10+ messages in thread From: Manivannan Sadhasivam @ 2018-06-05 16:34 UTC (permalink / raw) To: matthias.bgg, robh+dt Cc: linux-arm-kernel, linux-mediatek, devicetree, linux-kernel, taiten.peng, daniel.thompson, amit.kucheria, manivannanece23, Manivannan Sadhasivam Add devicetree support for Mediatek X20 Development Board by Archermind. This board is based on the Deca-Core MT6797 SoC from Mediatek and is one of the 96Boards Consumer Edition platform. With this devicetree change, board can boot into initramfs. More information about this board can be found in 96Boards product page: https://www.96boards.org/product/mediatek-x20/ Thanks, Mani Changes in v2: * Added documentation for board compatible * Dropped vendor documetation patch since it got applied Manivannan Sadhasivam (2): dt-bindings: arm: mediatek: Document Mediatek X20 Development Board arm64: dts: Add Mediatek X20 Development Board support .../devicetree/bindings/arm/mediatek.txt | 3 ++ arch/arm64/boot/dts/mediatek/Makefile | 1 + .../boot/dts/mediatek/mt6797-x20-dev.dts | 33 +++++++++++++++++++ 3 files changed, 37 insertions(+) create mode 100644 arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts -- 2.17.0 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 1/2] dt-bindings: arm: mediatek: Document Mediatek X20 Development Board 2018-06-05 16:34 [PATCH v2 0/2] Add Mediatek X20 Development Board support Manivannan Sadhasivam @ 2018-06-05 16:34 ` Manivannan Sadhasivam 2018-06-12 18:17 ` Rob Herring 2018-06-05 16:34 ` [PATCH v2 2/2] arm64: dts: Add Mediatek X20 Development Board support Manivannan Sadhasivam 1 sibling, 1 reply; 10+ messages in thread From: Manivannan Sadhasivam @ 2018-06-05 16:34 UTC (permalink / raw) To: matthias.bgg, robh+dt Cc: linux-arm-kernel, linux-mediatek, devicetree, linux-kernel, taiten.peng, daniel.thompson, amit.kucheria, manivannanece23, Manivannan Sadhasivam Document Mediatek X20 Development Board which is a 96Boards Consumer Edition platform based on MT6797 SoC. Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> --- Documentation/devicetree/bindings/arm/mediatek.txt | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/mediatek.txt b/Documentation/devicetree/bindings/arm/mediatek.txt index 7d21ab37c19c..356df1ee11b9 100644 --- a/Documentation/devicetree/bindings/arm/mediatek.txt +++ b/Documentation/devicetree/bindings/arm/mediatek.txt @@ -47,6 +47,9 @@ Supported boards: - Evaluation board for MT6797(Helio X20): Required root node properties: - compatible = "mediatek,mt6797-evb", "mediatek,mt6797"; +- Mediatek X20 Development Board: + Required root node properties: + - compatible = "archermind,mt6797-x20-dev", "mediatek,mt6797"; - Reference board variant 1 for MT7622: Required root node properties: - compatible = "mediatek,mt7622-rfb1", "mediatek,mt7622"; -- 2.17.0 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v2 1/2] dt-bindings: arm: mediatek: Document Mediatek X20 Development Board 2018-06-05 16:34 ` [PATCH v2 1/2] dt-bindings: arm: mediatek: Document Mediatek X20 Development Board Manivannan Sadhasivam @ 2018-06-12 18:17 ` Rob Herring 0 siblings, 0 replies; 10+ messages in thread From: Rob Herring @ 2018-06-12 18:17 UTC (permalink / raw) To: Manivannan Sadhasivam Cc: matthias.bgg, linux-arm-kernel, linux-mediatek, devicetree, linux-kernel, taiten.peng, daniel.thompson, amit.kucheria, manivannanece23 On Tue, Jun 05, 2018 at 10:04:41PM +0530, Manivannan Sadhasivam wrote: > Document Mediatek X20 Development Board which is a 96Boards Consumer > Edition platform based on MT6797 SoC. > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > --- > Documentation/devicetree/bindings/arm/mediatek.txt | 3 +++ > 1 file changed, 3 insertions(+) Reviewed-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 2/2] arm64: dts: Add Mediatek X20 Development Board support 2018-06-05 16:34 [PATCH v2 0/2] Add Mediatek X20 Development Board support Manivannan Sadhasivam 2018-06-05 16:34 ` [PATCH v2 1/2] dt-bindings: arm: mediatek: Document Mediatek X20 Development Board Manivannan Sadhasivam @ 2018-06-05 16:34 ` Manivannan Sadhasivam 2018-07-16 13:24 ` Matthias Brugger 1 sibling, 1 reply; 10+ messages in thread From: Manivannan Sadhasivam @ 2018-06-05 16:34 UTC (permalink / raw) To: matthias.bgg, robh+dt Cc: linux-arm-kernel, linux-mediatek, devicetree, linux-kernel, taiten.peng, daniel.thompson, amit.kucheria, manivannanece23, Manivannan Sadhasivam Add initial device tree support for Mediatek X20 Development Board based on MT6797 Deca core SoC. This board is one of the 96Boards Consumer Edition platform. Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> --- arch/arm64/boot/dts/mediatek/Makefile | 1 + .../boot/dts/mediatek/mt6797-x20-dev.dts | 33 +++++++++++++++++++ 2 files changed, 34 insertions(+) create mode 100644 arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts diff --git a/arch/arm64/boot/dts/mediatek/Makefile b/arch/arm64/boot/dts/mediatek/Makefile index ac17f60f998c..5b7fd6ad96e4 100644 --- a/arch/arm64/boot/dts/mediatek/Makefile +++ b/arch/arm64/boot/dts/mediatek/Makefile @@ -3,5 +3,6 @@ dtb-$(CONFIG_ARCH_MEDIATEK) += mt2712-evb.dtb dtb-$(CONFIG_ARCH_MEDIATEK) += mt6755-evb.dtb dtb-$(CONFIG_ARCH_MEDIATEK) += mt6795-evb.dtb dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-evb.dtb +dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-x20-dev.dtb dtb-$(CONFIG_ARCH_MEDIATEK) += mt7622-rfb1.dtb dtb-$(CONFIG_ARCH_MEDIATEK) += mt8173-evb.dtb diff --git a/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts new file mode 100644 index 000000000000..2c09ca95d9e2 --- /dev/null +++ b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts @@ -0,0 +1,33 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Device Tree Source for MediaTek X20 Development Board + * + * Copyright (C) 2018, Linaro Ltd. + * + */ + +/dts-v1/; + +#include "mt6797.dtsi" + +/ { + model = "Mediatek X20 Development Board"; + compatible = "archermind,mt6797-x20-dev", "mediatek,mt6797"; + + aliases { + serial0 = &uart1; + }; + + memory@40000000 { + device_type = "memory"; + reg = <0 0x40000000 0 0x1e605000>; + }; + + chosen { + stdout-path = "serial0:115200n8"; + }; +}; + +&uart1 { + status = "okay"; +}; -- 2.17.0 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: Add Mediatek X20 Development Board support 2018-06-05 16:34 ` [PATCH v2 2/2] arm64: dts: Add Mediatek X20 Development Board support Manivannan Sadhasivam @ 2018-07-16 13:24 ` Matthias Brugger 2018-07-17 14:04 ` Manivannan Sadhasivam 0 siblings, 1 reply; 10+ messages in thread From: Matthias Brugger @ 2018-07-16 13:24 UTC (permalink / raw) To: Manivannan Sadhasivam, robh+dt Cc: linux-arm-kernel, linux-mediatek, devicetree, linux-kernel, taiten.peng, daniel.thompson, amit.kucheria, manivannanece23 Hi Manivanna, On 05/06/18 18:34, Manivannan Sadhasivam wrote: > Add initial device tree support for Mediatek X20 Development Board > based on MT6797 Deca core SoC. This board is one of the 96Boards > Consumer Edition platform. > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > --- > arch/arm64/boot/dts/mediatek/Makefile | 1 + > .../boot/dts/mediatek/mt6797-x20-dev.dts | 33 +++++++++++++++++++ > 2 files changed, 34 insertions(+) > create mode 100644 arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts > > diff --git a/arch/arm64/boot/dts/mediatek/Makefile b/arch/arm64/boot/dts/mediatek/Makefile > index ac17f60f998c..5b7fd6ad96e4 100644 > --- a/arch/arm64/boot/dts/mediatek/Makefile > +++ b/arch/arm64/boot/dts/mediatek/Makefile > @@ -3,5 +3,6 @@ dtb-$(CONFIG_ARCH_MEDIATEK) += mt2712-evb.dtb > dtb-$(CONFIG_ARCH_MEDIATEK) += mt6755-evb.dtb > dtb-$(CONFIG_ARCH_MEDIATEK) += mt6795-evb.dtb > dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-evb.dtb > +dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-x20-dev.dtb > dtb-$(CONFIG_ARCH_MEDIATEK) += mt7622-rfb1.dtb > dtb-$(CONFIG_ARCH_MEDIATEK) += mt8173-evb.dtb > diff --git a/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts > new file mode 100644 > index 000000000000..2c09ca95d9e2 > --- /dev/null > +++ b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts > @@ -0,0 +1,33 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Device Tree Source for MediaTek X20 Development Board > + * > + * Copyright (C) 2018, Linaro Ltd. > + * > + */ > + > +/dts-v1/; > + > +#include "mt6797.dtsi" > + > +/ { > + model = "Mediatek X20 Development Board"; > + compatible = "archermind,mt6797-x20-dev", "mediatek,mt6797"; > + > + aliases { > + serial0 = &uart1; > + }; Sorry for the late answer. Why did you use uart1 instead of uart0? I know that uart0 is used by the bootloader, but because of their exotic flash procedure I find it quite useful to see this messages in my serial connection. Of course you can only use 921600 baudrate then. > + > + memory@40000000 { > + device_type = "memory"; > + reg = <0 0x40000000 0 0x1e605000>; How comes this strange number for the memory size? BTW do you know the differences between the EVB board and the x20 one? I used the evb dts for booting up to now. Best regards, Matthias > + }; > + > + chosen { > + stdout-path = "serial0:115200n8"; > + }; > +}; > + > +&uart1 { > + status = "okay"; > +}; > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: Add Mediatek X20 Development Board support 2018-07-16 13:24 ` Matthias Brugger @ 2018-07-17 14:04 ` Manivannan Sadhasivam 2018-07-18 15:07 ` Matthias Brugger 0 siblings, 1 reply; 10+ messages in thread From: Manivannan Sadhasivam @ 2018-07-17 14:04 UTC (permalink / raw) To: Matthias Brugger Cc: robh+dt, linux-arm-kernel, linux-mediatek, devicetree, linux-kernel, taiten.peng, daniel.thompson, amit.kucheria, manivannanece23 Hi Matthias, On Mon, Jul 16, 2018 at 03:24:44PM +0200, Matthias Brugger wrote: > Hi Manivanna, > > On 05/06/18 18:34, Manivannan Sadhasivam wrote: > > Add initial device tree support for Mediatek X20 Development Board > > based on MT6797 Deca core SoC. This board is one of the 96Boards > > Consumer Edition platform. > > > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > > --- > > arch/arm64/boot/dts/mediatek/Makefile | 1 + > > .../boot/dts/mediatek/mt6797-x20-dev.dts | 33 +++++++++++++++++++ > > 2 files changed, 34 insertions(+) > > create mode 100644 arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts > > > > diff --git a/arch/arm64/boot/dts/mediatek/Makefile b/arch/arm64/boot/dts/mediatek/Makefile > > index ac17f60f998c..5b7fd6ad96e4 100644 > > --- a/arch/arm64/boot/dts/mediatek/Makefile > > +++ b/arch/arm64/boot/dts/mediatek/Makefile > > @@ -3,5 +3,6 @@ dtb-$(CONFIG_ARCH_MEDIATEK) += mt2712-evb.dtb > > dtb-$(CONFIG_ARCH_MEDIATEK) += mt6755-evb.dtb > > dtb-$(CONFIG_ARCH_MEDIATEK) += mt6795-evb.dtb > > dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-evb.dtb > > +dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-x20-dev.dtb > > dtb-$(CONFIG_ARCH_MEDIATEK) += mt7622-rfb1.dtb > > dtb-$(CONFIG_ARCH_MEDIATEK) += mt8173-evb.dtb > > diff --git a/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts > > new file mode 100644 > > index 000000000000..2c09ca95d9e2 > > --- /dev/null > > +++ b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts > > @@ -0,0 +1,33 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* > > + * Device Tree Source for MediaTek X20 Development Board > > + * > > + * Copyright (C) 2018, Linaro Ltd. > > + * > > + */ > > + > > +/dts-v1/; > > + > > +#include "mt6797.dtsi" > > + > > +/ { > > + model = "Mediatek X20 Development Board"; > > + compatible = "archermind,mt6797-x20-dev", "mediatek,mt6797"; > > + > > + aliases { > > + serial0 = &uart1; > > + }; > > Sorry for the late answer. > Why did you use uart1 instead of uart0? > I know that uart0 is used by the bootloader, but because of their exotic flash > procedure I find it quite useful to see this messages in my serial connection. > Of course you can only use 921600 baudrate then. > The reason is, most of the Mezzanine boards use USB to UART bridge on the UART1 port available on the Low Speed Expansion header. Additionally, the convention followed by most of the Consumer Edition 96Boards is to use UART1 as the debug serial port. That's why eventhough the bootloader is using UART0, I forced the kernel to use UART1. > > + > > + memory@40000000 { > > + device_type = "memory"; > > + reg = <0 0x40000000 0 0x1e605000>; > > How comes this strange number for the memory size? > Actually, the memory size is extracted from the downstream kernel [1]. > BTW do you know the differences between the EVB board and the x20 one? > I used the evb dts for booting up to now. > Sorry, I don't know the difference between both boards! Thanks, Mani [1] https://github.com/helio-x20/linux/blob/helio-x20/arch/arm64/boot/dts/amt6797_64_open.dts#L8 > Best regards, > Matthias > > > + }; > > + > > + chosen { > > + stdout-path = "serial0:115200n8"; > > + }; > > +}; > > + > > +&uart1 { > > + status = "okay"; > > +}; > > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: Add Mediatek X20 Development Board support 2018-07-17 14:04 ` Manivannan Sadhasivam @ 2018-07-18 15:07 ` Matthias Brugger 2018-07-18 16:43 ` Manivannan Sadhasivam 0 siblings, 1 reply; 10+ messages in thread From: Matthias Brugger @ 2018-07-18 15:07 UTC (permalink / raw) To: Manivannan Sadhasivam Cc: robh+dt, linux-arm-kernel, linux-mediatek, devicetree, linux-kernel, taiten.peng, daniel.thompson, amit.kucheria, manivannanece23 On 17/07/18 16:04, Manivannan Sadhasivam wrote: > Hi Matthias, > > On Mon, Jul 16, 2018 at 03:24:44PM +0200, Matthias Brugger wrote: >> Hi Manivanna, >> >> On 05/06/18 18:34, Manivannan Sadhasivam wrote: >>> Add initial device tree support for Mediatek X20 Development Board >>> based on MT6797 Deca core SoC. This board is one of the 96Boards >>> Consumer Edition platform. >>> >>> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> >>> --- >>> arch/arm64/boot/dts/mediatek/Makefile | 1 + >>> .../boot/dts/mediatek/mt6797-x20-dev.dts | 33 +++++++++++++++++++ >>> 2 files changed, 34 insertions(+) >>> create mode 100644 arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts >>> >>> diff --git a/arch/arm64/boot/dts/mediatek/Makefile b/arch/arm64/boot/dts/mediatek/Makefile >>> index ac17f60f998c..5b7fd6ad96e4 100644 >>> --- a/arch/arm64/boot/dts/mediatek/Makefile >>> +++ b/arch/arm64/boot/dts/mediatek/Makefile >>> @@ -3,5 +3,6 @@ dtb-$(CONFIG_ARCH_MEDIATEK) += mt2712-evb.dtb >>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt6755-evb.dtb >>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt6795-evb.dtb >>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-evb.dtb >>> +dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-x20-dev.dtb >>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt7622-rfb1.dtb >>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt8173-evb.dtb >>> diff --git a/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts >>> new file mode 100644 >>> index 000000000000..2c09ca95d9e2 >>> --- /dev/null >>> +++ b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts >>> @@ -0,0 +1,33 @@ >>> +// SPDX-License-Identifier: GPL-2.0 >>> +/* >>> + * Device Tree Source for MediaTek X20 Development Board >>> + * >>> + * Copyright (C) 2018, Linaro Ltd. >>> + * >>> + */ >>> + >>> +/dts-v1/; >>> + >>> +#include "mt6797.dtsi" >>> + >>> +/ { >>> + model = "Mediatek X20 Development Board"; >>> + compatible = "archermind,mt6797-x20-dev", "mediatek,mt6797"; >>> + >>> + aliases { >>> + serial0 = &uart1; >>> + }; >> >> Sorry for the late answer. >> Why did you use uart1 instead of uart0? >> I know that uart0 is used by the bootloader, but because of their exotic flash >> procedure I find it quite useful to see this messages in my serial connection. >> Of course you can only use 921600 baudrate then. >> > > The reason is, most of the Mezzanine boards use USB to UART bridge on the > UART1 port available on the Low Speed Expansion header. Additionally, the > convention followed by most of the Consumer Edition 96Boards is to use UART1 > as the debug serial port. That's why eventhough the bootloader is using UART0, > I forced the kernel to use UART1. > Sounds reasonable, I don't have this 96 boards adapter and I forgot about it. >>> + >>> + memory@40000000 { >>> + device_type = "memory"; >>> + reg = <0 0x40000000 0 0x1e605000>; >> >> How comes this strange number for the memory size? >> > > Actually, the memory size is extracted from the downstream kernel [1]. > Hm, Archermind "Helio X20 Development Board Hardware Manual" talks about 2GB LPDDR3-SDRAM. Would you mind to check if that is actually true? >> BTW do you know the differences between the EVB board and the x20 one? >> I used the evb dts for booting up to now. >> > > Sorry, I don't know the difference between both boards! Sure, no problem :) Regards, Matthias > > Thanks, > Mani > > [1] https://github.com/helio-x20/linux/blob/helio-x20/arch/arm64/boot/dts/amt6797_64_open.dts#L8 > >> Best regards, >> Matthias >> >>> + }; >>> + >>> + chosen { >>> + stdout-path = "serial0:115200n8"; >>> + }; >>> +}; >>> + >>> +&uart1 { >>> + status = "okay"; >>> +}; >>> ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: Add Mediatek X20 Development Board support 2018-07-18 15:07 ` Matthias Brugger @ 2018-07-18 16:43 ` Manivannan Sadhasivam 2018-07-19 9:03 ` Matthias Brugger 0 siblings, 1 reply; 10+ messages in thread From: Manivannan Sadhasivam @ 2018-07-18 16:43 UTC (permalink / raw) To: Matthias Brugger Cc: robh+dt, linux-arm-kernel, linux-mediatek, devicetree, linux-kernel, taiten.peng, daniel.thompson, amit.kucheria, manivannanece23 Hi Matthias, On Wed, Jul 18, 2018 at 05:07:38PM +0200, Matthias Brugger wrote: > > > On 17/07/18 16:04, Manivannan Sadhasivam wrote: > > Hi Matthias, > > > > On Mon, Jul 16, 2018 at 03:24:44PM +0200, Matthias Brugger wrote: > >> Hi Manivanna, > >> > >> On 05/06/18 18:34, Manivannan Sadhasivam wrote: > >>> Add initial device tree support for Mediatek X20 Development Board > >>> based on MT6797 Deca core SoC. This board is one of the 96Boards > >>> Consumer Edition platform. > >>> > >>> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > >>> --- > >>> arch/arm64/boot/dts/mediatek/Makefile | 1 + > >>> .../boot/dts/mediatek/mt6797-x20-dev.dts | 33 +++++++++++++++++++ > >>> 2 files changed, 34 insertions(+) > >>> create mode 100644 arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts > >>> > >>> diff --git a/arch/arm64/boot/dts/mediatek/Makefile b/arch/arm64/boot/dts/mediatek/Makefile > >>> index ac17f60f998c..5b7fd6ad96e4 100644 > >>> --- a/arch/arm64/boot/dts/mediatek/Makefile > >>> +++ b/arch/arm64/boot/dts/mediatek/Makefile > >>> @@ -3,5 +3,6 @@ dtb-$(CONFIG_ARCH_MEDIATEK) += mt2712-evb.dtb > >>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt6755-evb.dtb > >>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt6795-evb.dtb > >>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-evb.dtb > >>> +dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-x20-dev.dtb > >>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt7622-rfb1.dtb > >>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt8173-evb.dtb > >>> diff --git a/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts > >>> new file mode 100644 > >>> index 000000000000..2c09ca95d9e2 > >>> --- /dev/null > >>> +++ b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts > >>> @@ -0,0 +1,33 @@ > >>> +// SPDX-License-Identifier: GPL-2.0 > >>> +/* > >>> + * Device Tree Source for MediaTek X20 Development Board > >>> + * > >>> + * Copyright (C) 2018, Linaro Ltd. > >>> + * > >>> + */ > >>> + > >>> +/dts-v1/; > >>> + > >>> +#include "mt6797.dtsi" > >>> + > >>> +/ { > >>> + model = "Mediatek X20 Development Board"; > >>> + compatible = "archermind,mt6797-x20-dev", "mediatek,mt6797"; > >>> + > >>> + aliases { > >>> + serial0 = &uart1; > >>> + }; > >> > >> Sorry for the late answer. > >> Why did you use uart1 instead of uart0? > >> I know that uart0 is used by the bootloader, but because of their exotic flash > >> procedure I find it quite useful to see this messages in my serial connection. > >> Of course you can only use 921600 baudrate then. > >> > > > > The reason is, most of the Mezzanine boards use USB to UART bridge on the > > UART1 port available on the Low Speed Expansion header. Additionally, the > > convention followed by most of the Consumer Edition 96Boards is to use UART1 > > as the debug serial port. That's why eventhough the bootloader is using UART0, > > I forced the kernel to use UART1. > > > > Sounds reasonable, I don't have this 96 boards adapter and I forgot about it. > That's fine :) > >>> + > >>> + memory@40000000 { > >>> + device_type = "memory"; > >>> + reg = <0 0x40000000 0 0x1e605000>; > >> > >> How comes this strange number for the memory size? > >> > > > > Actually, the memory size is extracted from the downstream kernel [1]. > > > > Hm, Archermind "Helio X20 Development Board Hardware Manual" talks about 2GB > LPDDR3-SDRAM. Would you mind to check if that is actually true? > Yes, it is 2GB. Shall I change the size to 0x80000000? Thanks, Mani > >> BTW do you know the differences between the EVB board and the x20 one? > >> I used the evb dts for booting up to now. > >> > > > > Sorry, I don't know the difference between both boards! > > Sure, no problem :) > > Regards, > Matthias > > > > > Thanks, > > Mani > > > > [1] https://github.com/helio-x20/linux/blob/helio-x20/arch/arm64/boot/dts/amt6797_64_open.dts#L8 > > > >> Best regards, > >> Matthias > >> > >>> + }; > >>> + > >>> + chosen { > >>> + stdout-path = "serial0:115200n8"; > >>> + }; > >>> +}; > >>> + > >>> +&uart1 { > >>> + status = "okay"; > >>> +}; > >>> ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: Add Mediatek X20 Development Board support 2018-07-18 16:43 ` Manivannan Sadhasivam @ 2018-07-19 9:03 ` Matthias Brugger 2018-07-19 10:57 ` Manivannan Sadhasivam 0 siblings, 1 reply; 10+ messages in thread From: Matthias Brugger @ 2018-07-19 9:03 UTC (permalink / raw) To: Manivannan Sadhasivam Cc: robh+dt, linux-arm-kernel, linux-mediatek, devicetree, linux-kernel, taiten.peng, daniel.thompson, amit.kucheria, manivannanece23 On 18/07/18 18:43, Manivannan Sadhasivam wrote: > Hi Matthias, > > On Wed, Jul 18, 2018 at 05:07:38PM +0200, Matthias Brugger wrote: >> >> >> On 17/07/18 16:04, Manivannan Sadhasivam wrote: >>> Hi Matthias, >>> >>> On Mon, Jul 16, 2018 at 03:24:44PM +0200, Matthias Brugger wrote: >>>> Hi Manivanna, >>>> >>>> On 05/06/18 18:34, Manivannan Sadhasivam wrote: >>>>> Add initial device tree support for Mediatek X20 Development Board >>>>> based on MT6797 Deca core SoC. This board is one of the 96Boards >>>>> Consumer Edition platform. >>>>> >>>>> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> >>>>> --- >>>>> arch/arm64/boot/dts/mediatek/Makefile | 1 + >>>>> .../boot/dts/mediatek/mt6797-x20-dev.dts | 33 +++++++++++++++++++ >>>>> 2 files changed, 34 insertions(+) >>>>> create mode 100644 arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts >>>>> >>>>> diff --git a/arch/arm64/boot/dts/mediatek/Makefile b/arch/arm64/boot/dts/mediatek/Makefile >>>>> index ac17f60f998c..5b7fd6ad96e4 100644 >>>>> --- a/arch/arm64/boot/dts/mediatek/Makefile >>>>> +++ b/arch/arm64/boot/dts/mediatek/Makefile >>>>> @@ -3,5 +3,6 @@ dtb-$(CONFIG_ARCH_MEDIATEK) += mt2712-evb.dtb >>>>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt6755-evb.dtb >>>>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt6795-evb.dtb >>>>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-evb.dtb >>>>> +dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-x20-dev.dtb >>>>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt7622-rfb1.dtb >>>>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt8173-evb.dtb >>>>> diff --git a/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts >>>>> new file mode 100644 >>>>> index 000000000000..2c09ca95d9e2 >>>>> --- /dev/null >>>>> +++ b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts >>>>> @@ -0,0 +1,33 @@ >>>>> +// SPDX-License-Identifier: GPL-2.0 >>>>> +/* >>>>> + * Device Tree Source for MediaTek X20 Development Board >>>>> + * >>>>> + * Copyright (C) 2018, Linaro Ltd. >>>>> + * >>>>> + */ >>>>> + >>>>> +/dts-v1/; >>>>> + >>>>> +#include "mt6797.dtsi" >>>>> + >>>>> +/ { >>>>> + model = "Mediatek X20 Development Board"; >>>>> + compatible = "archermind,mt6797-x20-dev", "mediatek,mt6797"; >>>>> + >>>>> + aliases { >>>>> + serial0 = &uart1; >>>>> + }; >>>> >>>> Sorry for the late answer. >>>> Why did you use uart1 instead of uart0? >>>> I know that uart0 is used by the bootloader, but because of their exotic flash >>>> procedure I find it quite useful to see this messages in my serial connection. >>>> Of course you can only use 921600 baudrate then. >>>> >>> >>> The reason is, most of the Mezzanine boards use USB to UART bridge on the >>> UART1 port available on the Low Speed Expansion header. Additionally, the >>> convention followed by most of the Consumer Edition 96Boards is to use UART1 >>> as the debug serial port. That's why eventhough the bootloader is using UART0, >>> I forced the kernel to use UART1. >>> >> >> Sounds reasonable, I don't have this 96 boards adapter and I forgot about it. >> > > That's fine :) > >>>>> + >>>>> + memory@40000000 { >>>>> + device_type = "memory"; >>>>> + reg = <0 0x40000000 0 0x1e605000>; >>>> >>>> How comes this strange number for the memory size? >>>> >>> >>> Actually, the memory size is extracted from the downstream kernel [1]. >>> >> >> Hm, Archermind "Helio X20 Development Board Hardware Manual" talks about 2GB >> LPDDR3-SDRAM. Would you mind to check if that is actually true? >> > > Yes, it is 2GB. Shall I change the size to 0x80000000? > Best would be if you boot the machine with this value and see if things look sane. Then resubmit. The other option is to resubmit and I'll check it tomorrow :) Let me know if you can't do it for tomorrow morning. I plan to submit the pull requests tomorrow as I'm through nearly all open patches (yes, two days earlier \o/ ) Best regards, Matthias > Thanks, > Mani > >>>> BTW do you know the differences between the EVB board and the x20 one? >>>> I used the evb dts for booting up to now. >>>> >>> >>> Sorry, I don't know the difference between both boards! >> >> Sure, no problem :) >> >> Regards, >> Matthias >> >>> >>> Thanks, >>> Mani >>> >>> [1] https://github.com/helio-x20/linux/blob/helio-x20/arch/arm64/boot/dts/amt6797_64_open.dts#L8 >>> >>>> Best regards, >>>> Matthias >>>> >>>>> + }; >>>>> + >>>>> + chosen { >>>>> + stdout-path = "serial0:115200n8"; >>>>> + }; >>>>> +}; >>>>> + >>>>> +&uart1 { >>>>> + status = "okay"; >>>>> +}; >>>>> ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: Add Mediatek X20 Development Board support 2018-07-19 9:03 ` Matthias Brugger @ 2018-07-19 10:57 ` Manivannan Sadhasivam 0 siblings, 0 replies; 10+ messages in thread From: Manivannan Sadhasivam @ 2018-07-19 10:57 UTC (permalink / raw) To: Matthias Brugger Cc: robh+dt, linux-arm-kernel, linux-mediatek, devicetree, linux-kernel, taiten.peng, daniel.thompson, amit.kucheria, manivannanece23 On Thu, Jul 19, 2018 at 11:03:54AM +0200, Matthias Brugger wrote: > > > On 18/07/18 18:43, Manivannan Sadhasivam wrote: > > Hi Matthias, > > > > On Wed, Jul 18, 2018 at 05:07:38PM +0200, Matthias Brugger wrote: > >> > >> > >> On 17/07/18 16:04, Manivannan Sadhasivam wrote: > >>> Hi Matthias, > >>> > >>> On Mon, Jul 16, 2018 at 03:24:44PM +0200, Matthias Brugger wrote: > >>>> Hi Manivanna, > >>>> > >>>> On 05/06/18 18:34, Manivannan Sadhasivam wrote: > >>>>> Add initial device tree support for Mediatek X20 Development Board > >>>>> based on MT6797 Deca core SoC. This board is one of the 96Boards > >>>>> Consumer Edition platform. > >>>>> > >>>>> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > >>>>> --- > >>>>> arch/arm64/boot/dts/mediatek/Makefile | 1 + > >>>>> .../boot/dts/mediatek/mt6797-x20-dev.dts | 33 +++++++++++++++++++ > >>>>> 2 files changed, 34 insertions(+) > >>>>> create mode 100644 arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts > >>>>> > >>>>> diff --git a/arch/arm64/boot/dts/mediatek/Makefile b/arch/arm64/boot/dts/mediatek/Makefile > >>>>> index ac17f60f998c..5b7fd6ad96e4 100644 > >>>>> --- a/arch/arm64/boot/dts/mediatek/Makefile > >>>>> +++ b/arch/arm64/boot/dts/mediatek/Makefile > >>>>> @@ -3,5 +3,6 @@ dtb-$(CONFIG_ARCH_MEDIATEK) += mt2712-evb.dtb > >>>>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt6755-evb.dtb > >>>>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt6795-evb.dtb > >>>>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-evb.dtb > >>>>> +dtb-$(CONFIG_ARCH_MEDIATEK) += mt6797-x20-dev.dtb > >>>>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt7622-rfb1.dtb > >>>>> dtb-$(CONFIG_ARCH_MEDIATEK) += mt8173-evb.dtb > >>>>> diff --git a/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts > >>>>> new file mode 100644 > >>>>> index 000000000000..2c09ca95d9e2 > >>>>> --- /dev/null > >>>>> +++ b/arch/arm64/boot/dts/mediatek/mt6797-x20-dev.dts > >>>>> @@ -0,0 +1,33 @@ > >>>>> +// SPDX-License-Identifier: GPL-2.0 > >>>>> +/* > >>>>> + * Device Tree Source for MediaTek X20 Development Board > >>>>> + * > >>>>> + * Copyright (C) 2018, Linaro Ltd. > >>>>> + * > >>>>> + */ > >>>>> + > >>>>> +/dts-v1/; > >>>>> + > >>>>> +#include "mt6797.dtsi" > >>>>> + > >>>>> +/ { > >>>>> + model = "Mediatek X20 Development Board"; > >>>>> + compatible = "archermind,mt6797-x20-dev", "mediatek,mt6797"; > >>>>> + > >>>>> + aliases { > >>>>> + serial0 = &uart1; > >>>>> + }; > >>>> > >>>> Sorry for the late answer. > >>>> Why did you use uart1 instead of uart0? > >>>> I know that uart0 is used by the bootloader, but because of their exotic flash > >>>> procedure I find it quite useful to see this messages in my serial connection. > >>>> Of course you can only use 921600 baudrate then. > >>>> > >>> > >>> The reason is, most of the Mezzanine boards use USB to UART bridge on the > >>> UART1 port available on the Low Speed Expansion header. Additionally, the > >>> convention followed by most of the Consumer Edition 96Boards is to use UART1 > >>> as the debug serial port. That's why eventhough the bootloader is using UART0, > >>> I forced the kernel to use UART1. > >>> > >> > >> Sounds reasonable, I don't have this 96 boards adapter and I forgot about it. > >> > > > > That's fine :) > > > >>>>> + > >>>>> + memory@40000000 { > >>>>> + device_type = "memory"; > >>>>> + reg = <0 0x40000000 0 0x1e605000>; > >>>> > >>>> How comes this strange number for the memory size? > >>>> > >>> > >>> Actually, the memory size is extracted from the downstream kernel [1]. > >>> > >> > >> Hm, Archermind "Helio X20 Development Board Hardware Manual" talks about 2GB > >> LPDDR3-SDRAM. Would you mind to check if that is actually true? > >> > > > > Yes, it is 2GB. Shall I change the size to 0x80000000? > > > > Best would be if you boot the machine with this value and see if things look > sane. Then resubmit. The other option is to resubmit and I'll check it tomorrow :) > Tested with the modified size, 0x80000000. Board is booting without any issues, so pushed out the v3. Thanks, Mani > Let me know if you can't do it for tomorrow morning. I plan to submit the pull > requests tomorrow as I'm through nearly all open patches (yes, two days earlier > \o/ ) > > Best regards, > Matthias > > > Thanks, > > Mani > > > >>>> BTW do you know the differences between the EVB board and the x20 one? > >>>> I used the evb dts for booting up to now. > >>>> > >>> > >>> Sorry, I don't know the difference between both boards! > >> > >> Sure, no problem :) > >> > >> Regards, > >> Matthias > >> > >>> > >>> Thanks, > >>> Mani > >>> > >>> [1] https://github.com/helio-x20/linux/blob/helio-x20/arch/arm64/boot/dts/amt6797_64_open.dts#L8 > >>> > >>>> Best regards, > >>>> Matthias > >>>> > >>>>> + }; > >>>>> + > >>>>> + chosen { > >>>>> + stdout-path = "serial0:115200n8"; > >>>>> + }; > >>>>> +}; > >>>>> + > >>>>> +&uart1 { > >>>>> + status = "okay"; > >>>>> +}; > >>>>> ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2018-07-19 10:58 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-06-05 16:34 [PATCH v2 0/2] Add Mediatek X20 Development Board support Manivannan Sadhasivam 2018-06-05 16:34 ` [PATCH v2 1/2] dt-bindings: arm: mediatek: Document Mediatek X20 Development Board Manivannan Sadhasivam 2018-06-12 18:17 ` Rob Herring 2018-06-05 16:34 ` [PATCH v2 2/2] arm64: dts: Add Mediatek X20 Development Board support Manivannan Sadhasivam 2018-07-16 13:24 ` Matthias Brugger 2018-07-17 14:04 ` Manivannan Sadhasivam 2018-07-18 15:07 ` Matthias Brugger 2018-07-18 16:43 ` Manivannan Sadhasivam 2018-07-19 9:03 ` Matthias Brugger 2018-07-19 10:57 ` Manivannan Sadhasivam
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).