* [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support @ 2017-01-08 13:30 Andreas Färber 2017-01-08 13:30 ` [PATCH 1/4] Documentation: devicetree: Add vendor prefix for AsiaRF Andreas Färber ` (4 more replies) 0 siblings, 5 replies; 13+ messages in thread From: Andreas Färber @ 2017-01-08 13:30 UTC (permalink / raw) To: linux-mediatek Cc: Matthias Brugger, Paul Lai, linux-arm-kernel, linux-kernel, devicetree, Andreas Färber Andreas Färber (4): Documentation: devicetree: Add vendor prefix for AsiaRF Documentation: devicetree: arm: mediatek: Add Geek Force board ARM: dts: mt7623: Add Geek Force config MAINTAINERS: Extend ARM/Mediatek SoC support section Documentation/devicetree/bindings/arm/mediatek.txt | 3 + .../devicetree/bindings/vendor-prefixes.txt | 1 + MAINTAINERS | 2 + arch/arm/boot/dts/Makefile | 1 + arch/arm/boot/dts/mt7623-geekforce.dts | 77 ++++++++++++++++++++++ 5 files changed, 84 insertions(+) create mode 100644 arch/arm/boot/dts/mt7623-geekforce.dts -- 2.10.2 ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 1/4] Documentation: devicetree: Add vendor prefix for AsiaRF 2017-01-08 13:30 [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support Andreas Färber @ 2017-01-08 13:30 ` Andreas Färber 2017-01-10 5:35 ` Rob Herring 2017-01-08 13:30 ` [PATCH 2/4] Documentation: devicetree: arm: mediatek: Add Geek Force board Andreas Färber ` (3 subsequent siblings) 4 siblings, 1 reply; 13+ messages in thread From: Andreas Färber @ 2017-01-08 13:30 UTC (permalink / raw) To: linux-mediatek Cc: Matthias Brugger, Paul Lai, linux-arm-kernel, linux-kernel, devicetree, Andreas Färber, Rob Herring, Mark Rutland Signed-off-by: Andreas Färber <afaerber@suse.de> --- Documentation/devicetree/bindings/vendor-prefixes.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt index 4ec84b7..01d222b 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.txt +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt @@ -34,6 +34,7 @@ armadeus ARMadeus Systems SARL arrow Arrow Electronics artesyn Artesyn Embedded Technologies Inc. asahi-kasei Asahi Kasei Corp. +asiarf AsiaRF Co., Ltd. aspeed ASPEED Technology Inc. atlas Atlas Scientific LLC atmel Atmel Corporation -- 2.10.2 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 1/4] Documentation: devicetree: Add vendor prefix for AsiaRF 2017-01-08 13:30 ` [PATCH 1/4] Documentation: devicetree: Add vendor prefix for AsiaRF Andreas Färber @ 2017-01-10 5:35 ` Rob Herring 0 siblings, 0 replies; 13+ messages in thread From: Rob Herring @ 2017-01-10 5:35 UTC (permalink / raw) To: Andreas Färber Cc: linux-mediatek, Matthias Brugger, Paul Lai, linux-arm-kernel, linux-kernel, devicetree, Mark Rutland On Sun, Jan 08, 2017 at 02:30:57PM +0100, Andreas Färber wrote: > Signed-off-by: Andreas Färber <afaerber@suse.de> > --- > Documentation/devicetree/bindings/vendor-prefixes.txt | 1 + > 1 file changed, 1 insertion(+) Acked-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 2/4] Documentation: devicetree: arm: mediatek: Add Geek Force board 2017-01-08 13:30 [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support Andreas Färber 2017-01-08 13:30 ` [PATCH 1/4] Documentation: devicetree: Add vendor prefix for AsiaRF Andreas Färber @ 2017-01-08 13:30 ` Andreas Färber 2017-01-10 5:36 ` Rob Herring 2017-01-08 13:30 ` [PATCH 3/4] ARM: dts: mt7623: Add Geek Force config Andreas Färber ` (2 subsequent siblings) 4 siblings, 1 reply; 13+ messages in thread From: Andreas Färber @ 2017-01-08 13:30 UTC (permalink / raw) To: linux-mediatek Cc: Matthias Brugger, Paul Lai, linux-arm-kernel, linux-kernel, devicetree, Andreas Färber, Rob Herring, Mark Rutland Signed-off-by: Andreas Färber <afaerber@suse.de> --- 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 c860b24..f533758 100644 --- a/Documentation/devicetree/bindings/arm/mediatek.txt +++ b/Documentation/devicetree/bindings/arm/mediatek.txt @@ -41,6 +41,9 @@ Supported boards: - Evaluation board for MT7623: Required root node properties: - compatible = "mediatek,mt7623-evb", "mediatek,mt7623"; +- AsiaRF Geek Force board: + Required root node properties: + - compatible = "asiarf,geekforce", "mediatek,mt7623"; - MTK mt8127 tablet moose EVB: Required root node properties: - compatible = "mediatek,mt8127-moose", "mediatek,mt8127"; -- 2.10.2 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 2/4] Documentation: devicetree: arm: mediatek: Add Geek Force board 2017-01-08 13:30 ` [PATCH 2/4] Documentation: devicetree: arm: mediatek: Add Geek Force board Andreas Färber @ 2017-01-10 5:36 ` Rob Herring 0 siblings, 0 replies; 13+ messages in thread From: Rob Herring @ 2017-01-10 5:36 UTC (permalink / raw) To: Andreas Färber Cc: linux-mediatek, Matthias Brugger, Paul Lai, linux-arm-kernel, linux-kernel, devicetree, Mark Rutland On Sun, Jan 08, 2017 at 02:30:58PM +0100, Andreas Färber wrote: > Signed-off-by: Andreas Färber <afaerber@suse.de> > --- > Documentation/devicetree/bindings/arm/mediatek.txt | 3 +++ > 1 file changed, 3 insertions(+) Acked-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 3/4] ARM: dts: mt7623: Add Geek Force config 2017-01-08 13:30 [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support Andreas Färber 2017-01-08 13:30 ` [PATCH 1/4] Documentation: devicetree: Add vendor prefix for AsiaRF Andreas Färber 2017-01-08 13:30 ` [PATCH 2/4] Documentation: devicetree: arm: mediatek: Add Geek Force board Andreas Färber @ 2017-01-08 13:30 ` Andreas Färber 2017-01-08 13:31 ` [PATCH 4/4] MAINTAINERS: Extend ARM/Mediatek SoC support section Andreas Färber 2017-01-10 7:00 ` [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support John Crispin 4 siblings, 0 replies; 13+ messages in thread From: Andreas Färber @ 2017-01-08 13:30 UTC (permalink / raw) To: linux-mediatek Cc: Matthias Brugger, Paul Lai, linux-arm-kernel, linux-kernel, devicetree, Andreas Färber, Rob Herring, Mark Rutland, Russell King Signed-off-by: Andreas Färber <afaerber@suse.de> --- arch/arm/boot/dts/Makefile | 1 + arch/arm/boot/dts/mt7623-geekforce.dts | 77 ++++++++++++++++++++++++++++++++++ 2 files changed, 78 insertions(+) create mode 100644 arch/arm/boot/dts/mt7623-geekforce.dts diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile index fa68843..fc1d4be 100644 --- a/arch/arm/boot/dts/Makefile +++ b/arch/arm/boot/dts/Makefile @@ -987,6 +987,7 @@ dtb-$(CONFIG_ARCH_MEDIATEK) += \ mt6589-aquaris5.dtb \ mt6592-evb.dtb \ mt7623-evb.dtb \ + mt7623-geekforce.dtb \ mt8127-moose.dtb \ mt8135-evbp1.dtb dtb-$(CONFIG_ARCH_ZX) += zx296702-ad1.dtb diff --git a/arch/arm/boot/dts/mt7623-geekforce.dts b/arch/arm/boot/dts/mt7623-geekforce.dts new file mode 100644 index 0000000..ab4cecd --- /dev/null +++ b/arch/arm/boot/dts/mt7623-geekforce.dts @@ -0,0 +1,77 @@ +/* + * Copyright (c) 2016-2017 Andreas Färber + * + * This file is dual-licensed: you can use it either under the terms + * of the GPL or the X11 license, at your option. Note that this dual + * licensing only applies to this file, and not this project as a + * whole. + * + * a) This file is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; either version 2 of the + * License, or (at your option) any later version. + * + * This file 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. + * + * Or, alternatively, + * + * b) Permission is hereby granted, free of charge, to any person + * obtaining a copy of this software and associated documentation + * files (the "Software"), to deal in the Software without + * restriction, including without limitation the rights to use, + * copy, modify, merge, publish, distribute, sublicense, and/or + * sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following + * conditions: + * + * The above copyright notice and this permission notice shall be + * included in all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES + * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT + * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, + * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR + * OTHER DEALINGS IN THE SOFTWARE. + */ + +/dts-v1/; + +#include "mt7623.dtsi" + +/ { + model = "AsiaRF Geek Force"; + compatible = "asiarf,geekforce", "mediatek,mt7623"; + + aliases { + serial0 = &uart0; + serial1 = &uart1; + serial2 = &uart2; + }; + + chosen { + stdout-path = "serial2:115200n8"; + }; + + memory { + reg = <0 0x80000000 0 0x40000000>; + }; +}; + +/* on Raspberry Pi connector */ +&uart0 { + status = "okay"; +}; + +&uart1 { + status = "okay"; +}; + +&uart2 { + status = "okay"; +}; -- 2.10.2 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 4/4] MAINTAINERS: Extend ARM/Mediatek SoC support section 2017-01-08 13:30 [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support Andreas Färber ` (2 preceding siblings ...) 2017-01-08 13:30 ` [PATCH 3/4] ARM: dts: mt7623: Add Geek Force config Andreas Färber @ 2017-01-08 13:31 ` Andreas Färber 2017-01-10 7:00 ` [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support John Crispin 4 siblings, 0 replies; 13+ messages in thread From: Andreas Färber @ 2017-01-08 13:31 UTC (permalink / raw) To: linux-mediatek Cc: Matthias Brugger, Paul Lai, linux-arm-kernel, linux-kernel, devicetree, Andreas Färber Catch mt7623 and arm64 dts subdirectory. Cc: Matthias Brugger <matthias.bgg@gmail.com> Signed-off-by: Andreas Färber <afaerber@suse.de> --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 93a983a..7f5a629 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1522,8 +1522,10 @@ L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) L: linux-mediatek@lists.infradead.org (moderated for non-subscribers) S: Maintained F: arch/arm/boot/dts/mt6* +F: arch/arm/boot/dts/mt7* F: arch/arm/boot/dts/mt8* F: arch/arm/mach-mediatek/ +F: arch/arm64/boot/dts/mediatek/ N: mtk K: mediatek -- 2.10.2 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support 2017-01-08 13:30 [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support Andreas Färber ` (3 preceding siblings ...) 2017-01-08 13:31 ` [PATCH 4/4] MAINTAINERS: Extend ARM/Mediatek SoC support section Andreas Färber @ 2017-01-10 7:00 ` John Crispin 2017-01-10 9:48 ` Andreas Färber 4 siblings, 1 reply; 13+ messages in thread From: John Crispin @ 2017-01-10 7:00 UTC (permalink / raw) To: Andreas Färber, linux-mediatek Cc: devicetree, Paul Lai, linux-kernel, Matthias Brugger, linux-arm-kernel On 08/01/2017 14:30, Andreas Färber wrote: > > Andreas Färber (4): > Documentation: devicetree: Add vendor prefix for AsiaRF > Documentation: devicetree: arm: mediatek: Add Geek Force board > ARM: dts: mt7623: Add Geek Force config > MAINTAINERS: Extend ARM/Mediatek SoC support section > Hi, i need to NAK this series. the asiarf board is nothing more than the official MTK EVB with AsiaRF written on it. this board is already supported by linux (arch/arm/boot/dts/mt7623-evb.dts) please extend the EVB dts file nstead of adding a duplicate and letting the original bitrot. John > Documentation/devicetree/bindings/arm/mediatek.txt | 3 + > .../devicetree/bindings/vendor-prefixes.txt | 1 + > MAINTAINERS | 2 + > arch/arm/boot/dts/Makefile | 1 + > arch/arm/boot/dts/mt7623-geekforce.dts | 77 ++++++++++++++++++++++ > 5 files changed, 84 insertions(+) > create mode 100644 arch/arm/boot/dts/mt7623-geekforce.dts > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support 2017-01-10 7:00 ` [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support John Crispin @ 2017-01-10 9:48 ` Andreas Färber 2017-01-10 10:18 ` John Crispin 0 siblings, 1 reply; 13+ messages in thread From: Andreas Färber @ 2017-01-10 9:48 UTC (permalink / raw) To: John Crispin, linux-mediatek Cc: devicetree, Paul Lai, linux-kernel, Matthias Brugger, linux-arm-kernel Hi, Am 10.01.2017 um 08:00 schrieb John Crispin: > On 08/01/2017 14:30, Andreas Färber wrote: >> >> Andreas Färber (4): >> Documentation: devicetree: Add vendor prefix for AsiaRF >> Documentation: devicetree: arm: mediatek: Add Geek Force board >> ARM: dts: mt7623: Add Geek Force config >> MAINTAINERS: Extend ARM/Mediatek SoC support section >> > > Hi, > > i need to NAK this series. the asiarf board is nothing more than the > official MTK EVB with AsiaRF written on it. this board is already > supported by linux (arch/arm/boot/dts/mt7623-evb.dts) please extend the > EVB dts file nstead of adding a duplicate and letting the original bitrot. Well, I disagree. First of all I'm not letting "the original" bitrot, because I have nothing to do with that .dts! If anyone is to blame for letting it bitrot since February 2016, pick your own nose: http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/log/arch/arm/boot/dts/mt7623-evb.dts Second, I have no Mediatek documentation or even picture to identify any similarities between my board and that Mediatek EVB, so no, I can't hack on the -evb.dts file. I wrote my .dts from scratch, not even having access to /proc/device-tree on its 3.10 kernel for comparison. Third, by your argumentation we shouldn't be adding, e.g., Odroid .dts files either because they were based on a Samsung SMDK, or .dts files for Amlogic TV boxes because they're almost identical to reference designs, etc. Users need to know which .dts file to choose, so having a sane .dts filename is warranted. Depending on how similar they are, one could either #include the -evb.dts or factor out a shared .dtsi, but that takes us back to the previous point of hardly anyone having access to EVB information to identify such a subset. Therefore duplicating trivial nodes is the method of choice for all practical purposes - mt7623.dtsi is getting reused just fine. Comparing our two .dts files, mine has two more UART nodes enabled, the U-Boot bootloader's baudrate set to actually get serial output, a different board compatible string for identification, and I chose the new dual-licensing header that is being requested for new DT files. For lack of schematics I figured out UART1 by testing - continuity tests for GND, console=ttySx,115200n8 and trial-and-error for RX/TX. Obviously I can't do that for a board I don't have access to. UART2 and UART0 pins were clear, but only UART2 was obvious from ttyMT2. Do you actually have access to a Geek Force board yourself, or what are you basing your claims on? Mine looks different from the Indiegogo picture and thus has different identification from that on https://wikidevi.com/wiki/AsiaRF_WS2977 (WS3301, MT7623N RFB_V10). If you confirm the EVB's baudrate I can happily send that part your way. I've seen 921600 on the Helio X20 96board for instance. Also, none of what you've said justifies NAK'ing patch 4/4, which applies to any mt7* and arm64 .dts, including yours. While we're at it, I noticed that mainline has a "mediatek,mt7623-eth" network driver but no corresponding .dtsi node. Talk about bitrot... Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support 2017-01-10 9:48 ` Andreas Färber @ 2017-01-10 10:18 ` John Crispin 2017-01-12 8:23 ` John Crispin 2017-01-16 19:31 ` Andreas Färber 0 siblings, 2 replies; 13+ messages in thread From: John Crispin @ 2017-01-10 10:18 UTC (permalink / raw) To: Andreas Färber, linux-mediatek Cc: Matthias Brugger, devicetree, linux-kernel, Paul Lai, linux-arm-kernel (resend, hit the wrong reply button) On 10/01/2017 10:48, Andreas Färber wrote: > Hi, > > Am 10.01.2017 um 08:00 schrieb John Crispin: >> On 08/01/2017 14:30, Andreas Färber wrote: >>> >>> Andreas Färber (4): >>> Documentation: devicetree: Add vendor prefix for AsiaRF >>> Documentation: devicetree: arm: mediatek: Add Geek Force board >>> ARM: dts: mt7623: Add Geek Force config >>> MAINTAINERS: Extend ARM/Mediatek SoC support section >>> >> >> Hi, >> >> i need to NAK this series. the asiarf board is nothing more than the >> official MTK EVB with AsiaRF written on it. this board is already >> supported by linux (arch/arm/boot/dts/mt7623-evb.dts) please extend the >> EVB dts file nstead of adding a duplicate and letting the original bitrot. > > Well, I disagree. reading the rest of the email you seem to be quite agro about this. > > First of all I'm not letting "the original" bitrot, because I have > nothing to do with that .dts! If anyone is to blame for letting it > bitrot since February 2016, pick your own nose: > > http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/log/arch/arm/boot/dts/mt7623-evb.dts what should i pick my nose about ? i made mt7623 work, then waited for 4.10-rc1 to be out for clk-mt2701 so that i can continue adding the missing support > Second, I have no Mediatek documentation or even picture to identify any > similarities between my board and that Mediatek EVB, so no, I can't hack > on the -evb.dts file. I wrote my .dts from scratch, not even having > access to /proc/device-tree on its 3.10 kernel for comparison. ok, that info is most likely under NDA > > Third, by your argumentation we shouldn't be adding, e.g., Odroid .dts > files either because they were based on a Samsung SMDK, or .dts files > for Amlogic TV boxes because they're almost identical to reference > designs, etc. > Users need to know which .dts file to choose, so having a sane .dts > filename is warranted. Depending on how similar they are, one could > either #include the -evb.dts or factor out a shared .dtsi, but that > takes us back to the previous point of hardly anyone having access to > EVB information to identify such a subset. Therefore duplicating trivial > nodes is the method of choice for all practical purposes - mt7623.dtsi > is getting reused just fine. > in that case add a dtsi file for the EVB and include it in your geek board.dts and only update the compat string. > Comparing our two .dts files, mine has two more UART nodes enabled, the > U-Boot bootloader's baudrate set to actually get serial output, a > different board compatible string for identification, and I chose the > new dual-licensing header that is being requested for new DT files. 1) at the time we adde this the uart support was not ready 2) the bootloader i am using is a custom built one hence the random baudrate 3) you can just updae the license if you want to, no problem > For lack of schematics I figured out UART1 by testing - continuity tests > for GND, console=ttySx,115200n8 and trial-and-error for RX/TX. Obviously > I can't do that for a board I don't have access to. > UART2 and UART0 pins were clear, but only UART2 was obvious from ttyMT2. you do have the EVB directly in front of you > Do you actually have access to a Geek Force board yourself, or what are > you basing your claims on? Mine looks different from the Indiegogo > picture and thus has different identification from that on > https://wikidevi.com/wiki/AsiaRF_WS2977 (WS3301, MT7623N RFB_V10). i dont need the geek board as i have the EVB and they are identical according to MTK > If you confirm the EVB's baudrate I can happily send that part your way. > I've seen 921600 on the Helio X20 96board for instance. see above > Also, none of what you've said justifies NAK'ing patch 4/4, which > applies to any mt7* and arm64 .dts, including yours. agreed, i never even mentioned 4/4 > While we're at it, I noticed that mainline has a "mediatek,mt7623-eth" > network driver but no corresponding .dtsi node. Talk about bitrot... the idea is that we work together to make thins optimal. this is not a you or is right. this is about the FOSS peer review process. please dont be so agro. to me it seems suboptimal to support 2 dts files for the same board. John > > Regards, > Andreas > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support 2017-01-10 10:18 ` John Crispin @ 2017-01-12 8:23 ` John Crispin 2017-01-16 19:34 ` Andreas Färber 2017-01-16 19:31 ` Andreas Färber 1 sibling, 1 reply; 13+ messages in thread From: John Crispin @ 2017-01-12 8:23 UTC (permalink / raw) To: Andreas Färber, linux-mediatek Cc: Matthias Brugger, devicetree, linux-kernel, Paul Lai, linux-arm-kernel Hi Andreas, had a look last night why the ethernet dtsi was not added and it obviously was not added as we were waiting for the clk-mt2701 to be merged. the ethernet dtsi will have phandles pointing at the clk nodes which did not exist at the time. same is true for the PWM code. i sat down last night and worked out what pending patches i still have for mt7623 and out of the ~80 required to get v4.4 working i only need around 10 for v4.10-rc1. i started to rebase these patches last night and will have time to test them tomorrow or early next week. as the pwrap node alone is around 200 lines of devicetree we need to figure out a way to add this to the dts files without duplicating it. i'll try to post a series early next week that we can then discuss and rebase your geekboard patches on. John ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support 2017-01-12 8:23 ` John Crispin @ 2017-01-16 19:34 ` Andreas Färber 0 siblings, 0 replies; 13+ messages in thread From: Andreas Färber @ 2017-01-16 19:34 UTC (permalink / raw) To: John Crispin, linux-mediatek Cc: Matthias Brugger, devicetree, linux-kernel, Paul Lai, linux-arm-kernel Hi John, Am 12.01.2017 um 09:23 schrieb John Crispin: > had a look last night why the ethernet dtsi was not added and it > obviously was not added as we were waiting for the clk-mt2701 to be > merged. the ethernet dtsi will have phandles pointing at the clk nodes > which did not exist at the time. same is true for the PWM code. > > i sat down last night and worked out what pending patches i still have > for mt7623 and out of the ~80 required to get v4.4 working i only need > around 10 for v4.10-rc1. > > i started to rebase these patches last night and will have time to test > them tomorrow or early next week. as the pwrap node alone is around 200 > lines of devicetree we need to figure out a way to add this to the dts > files without duplicating it. i'll try to post a series early next week > that we can then discuss and rebase your geekboard patches on. Sounds like a plan. Many thanks! Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support 2017-01-10 10:18 ` John Crispin 2017-01-12 8:23 ` John Crispin @ 2017-01-16 19:31 ` Andreas Färber 1 sibling, 0 replies; 13+ messages in thread From: Andreas Färber @ 2017-01-16 19:31 UTC (permalink / raw) To: John Crispin, linux-mediatek Cc: Matthias Brugger, devicetree, linux-kernel, Paul Lai, linux-arm-kernel Am 10.01.2017 um 11:18 schrieb John Crispin: > On 10/01/2017 10:48, Andreas Färber wrote: >> Am 10.01.2017 um 08:00 schrieb John Crispin: >>> On 08/01/2017 14:30, Andreas Färber wrote: >>>> >>>> Andreas Färber (4): >>>> Documentation: devicetree: Add vendor prefix for AsiaRF >>>> Documentation: devicetree: arm: mediatek: Add Geek Force board >>>> ARM: dts: mt7623: Add Geek Force config >>>> MAINTAINERS: Extend ARM/Mediatek SoC support section >>>> >>> >>> Hi, >>> >>> i need to NAK this series. the asiarf board is nothing more than the >>> official MTK EVB with AsiaRF written on it. this board is already >>> supported by linux (arch/arm/boot/dts/mt7623-evb.dts) please extend the >>> EVB dts file nstead of adding a duplicate and letting the original > bitrot. >> >> Well, I disagree. > > reading the rest of the email you seem to be quite agro about this. Please re-read your reply above and my comments and reconsider your attitude when replying in the future. It's not about whether these patches go in or not, it's about your wording. And that continues with aggressively throwing around the term "agro" in two mails already. I don't see anything aggressive or angry in my original cover letter that would've prompted your reply and can only interpret that as your own frustration with your mt7623 progress. Instead you should've taken the time to explain a bit better what you really meant, then we could've spared or shortened this lengthy discussion and have a v2 already. >> First of all I'm not letting "the original" bitrot, because I have >> nothing to do with that .dts! If anyone is to blame for letting it >> bitrot since February 2016, pick your own nose: >> >> > http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/log/arch/arm/boot/dts/mt7623-evb.dts > > what should i pick my nose about ? Don't blame _me_ for "bitrot" of a file _you_ added almost a year ago that did not see any updates since. Nothing wrong with you not working on something, just don't blame me for it then, especially when I don't (knowingly) have that hardware and have never come in touch with it before. I am new to linux-mediatek and had checked LAKML in vein for any mt7623 patches before sending mine. [snip] >> Third, by your argumentation we shouldn't be adding, e.g., Odroid .dts >> files either because they were based on a Samsung SMDK, or .dts files >> for Amlogic TV boxes because they're almost identical to reference >> designs, etc. >> Users need to know which .dts file to choose, so having a sane .dts >> filename is warranted. Depending on how similar they are, one could >> either #include the -evb.dts or factor out a shared .dtsi, but that >> takes us back to the previous point of hardly anyone having access to >> EVB information to identify such a subset. Therefore duplicating trivial >> nodes is the method of choice for all practical purposes - mt7623.dtsi >> is getting reused just fine. >> > > in that case add a dtsi file for the EVB and include it in your geek > board.dts and only update the compat string. The question that I am not in a position to answer is: Are those two boards identical or just very similar? Paul in CC can hopefully clarify this when he is back. Matthias as maintainer has also remained silent. >> Comparing our two .dts files, mine has two more UART nodes enabled, the >> U-Boot bootloader's baudrate set to actually get serial output, a >> different board compatible string for identification, and I chose the >> new dual-licensing header that is being requested for new DT files. > > 1) at the time we adde this the uart support was not ready > 2) the bootloader i am using is a custom built one hence the random baudrate Well, did the _original_ bootloader use 115200? In that case we could update -evb.dts with it, and you could still override it via console= for your custom build. Or maybe having source access you could even contribute to mainline U-Boot, so we can all work on the same codebase? I read a BPi-R2 is coming up with mt7623, so booting without appended .dtb and uImage will come in handy for more people than just EVB owners and Geek Force backers. I'd happily contribute to making bootefi command work. > 3) you can just updae the license if you want to, no problem OK, will gladly look into the feasibility. >> For lack of schematics I figured out UART1 by testing - continuity tests >> for GND, console=ttySx,115200n8 and trial-and-error for RX/TX. Obviously >> I can't do that for a board I don't have access to. >> UART2 and UART0 pins were clear, but only UART2 was obvious from ttyMT2. > > you do have the EVB directly in front of you > >> Do you actually have access to a Geek Force board yourself, or what are >> you basing your claims on? Mine looks different from the Indiegogo >> picture and thus has different identification from that on >> https://wikidevi.com/wiki/AsiaRF_WS2977 (WS3301, MT7623N RFB_V10). > > i dont need the geek board as i have the EVB and they are identical > according to MTK > >> If you confirm the EVB's baudrate I can happily send that part your way. >> I've seen 921600 on the Helio X20 96board for instance. > > see above So... 4) add my UART nodes to your -evb.dts? Any nitpicks on the actual patch 3/4? >> Also, none of what you've said justifies NAK'ing patch 4/4, which >> applies to any mt7* and arm64 .dts, including yours. > > agreed, i never even mentioned 4/4 You replied to the cover letter 0/4 "i need to NAK this series" - and series includes all four patches. Therefore my frustration with your hip-shot reply. If you haven't read the patch, don't NAK it! In fact still no one replied to it at all, even now that you're aware of its existence. >> While we're at it, I noticed that mainline has a "mediatek,mt7623-eth" >> network driver but no corresponding .dtsi node. Talk about bitrot... > > the idea is that we work together to make thins optimal. this is not a > you or is right. this is about the FOSS peer review process. please dont > be so agro. See above, please reconsider your tone. Peer review is no excuse for rude and destructive behavior towards contributors you don't know. Had you looked at the lists and codebase you would know that I am not new to FOSS. And please stop writing "agro", it sounds like some pothead making a peace sign; not helpful among professional software developers. Taking the time to start your sentences with a capital letter would also be a respectful gesture. > to me it seems suboptimal to support 2 dts files for the same board. If it is the identical board, then we are in violent agreement. I just find all your replies to me offensive so far, I don't know you and thus have no reason to trust your unsubstantiated claims about my board. Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2017-01-16 20:24 UTC | newest] Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-01-08 13:30 [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support Andreas Färber 2017-01-08 13:30 ` [PATCH 1/4] Documentation: devicetree: Add vendor prefix for AsiaRF Andreas Färber 2017-01-10 5:35 ` Rob Herring 2017-01-08 13:30 ` [PATCH 2/4] Documentation: devicetree: arm: mediatek: Add Geek Force board Andreas Färber 2017-01-10 5:36 ` Rob Herring 2017-01-08 13:30 ` [PATCH 3/4] ARM: dts: mt7623: Add Geek Force config Andreas Färber 2017-01-08 13:31 ` [PATCH 4/4] MAINTAINERS: Extend ARM/Mediatek SoC support section Andreas Färber 2017-01-10 7:00 ` [PATCH 0/4] ARM: dts: mt7623: Add initial Geek Force support John Crispin 2017-01-10 9:48 ` Andreas Färber 2017-01-10 10:18 ` John Crispin 2017-01-12 8:23 ` John Crispin 2017-01-16 19:34 ` Andreas Färber 2017-01-16 19:31 ` Andreas Färber
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).