* [PATCH v5 0/2] ARM: dts: add support for Turris Omnia @ 2016-11-25 14:26 Uwe Kleine-König 2016-11-25 14:26 ` [PATCH v5 1/2] devicetree: Add vendor prefix for CZ.NIC Uwe Kleine-König 2016-11-25 14:26 ` [PATCH v5 2/2] ARM: dts: add support for Turris Omnia Uwe Kleine-König 0 siblings, 2 replies; 17+ messages in thread From: Uwe Kleine-König @ 2016-11-25 14:26 UTC (permalink / raw) To: Jason Cooper, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth Cc: Mark Rutland, devicetree, Tomas Hlavacek, Rob Herring, linux-arm-kernel, Bedřicha Košatu Hello, This is basicly my v3 plus some of the changes that Tomas changed for his RFC patch (implicitly "v4"). So compared to v3 the following changed: - Add more documentation (mostly from Tomas) - Add more mbus ranges (by Tomas) - Move link to schematic below copyright header (suggested by Andrew) - reenable rtc as this is not that broken on production board - fix spi chip select pinmuxing Uwe Kleine-König (2): devicetree: Add vendor prefix for CZ.NIC ARM: dts: add support for Turris Omnia .../devicetree/bindings/vendor-prefixes.txt | 1 + arch/arm/boot/dts/Makefile | 1 + arch/arm/boot/dts/armada-385-turris-omnia.dts | 334 +++++++++++++++++++++ 3 files changed, 336 insertions(+) create mode 100644 arch/arm/boot/dts/armada-385-turris-omnia.dts -- 2.10.2 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH v5 1/2] devicetree: Add vendor prefix for CZ.NIC 2016-11-25 14:26 [PATCH v5 0/2] ARM: dts: add support for Turris Omnia Uwe Kleine-König @ 2016-11-25 14:26 ` Uwe Kleine-König [not found] ` <20161125142658.21690-2-uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> 2016-11-25 14:26 ` [PATCH v5 2/2] ARM: dts: add support for Turris Omnia Uwe Kleine-König 1 sibling, 1 reply; 17+ messages in thread From: Uwe Kleine-König @ 2016-11-25 14:26 UTC (permalink / raw) To: Jason Cooper, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth Cc: Mark Rutland, devicetree, Tomas Hlavacek, Rob Herring, linux-arm-kernel, Bedřicha Košatu Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org> --- 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 f0a48ea78659..ae9fce9fed03 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.txt +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt @@ -67,6 +67,7 @@ creative Creative Technology Ltd crystalfontz Crystalfontz America, Inc. cubietech Cubietech, Ltd. cypress Cypress Semiconductor Corporation +cznic CZ.NIC, z.s.p.o. dallas Maxim Integrated Products (formerly Dallas Semiconductor) davicom DAVICOM Semiconductor, Inc. delta Delta Electronics, Inc. -- 2.10.2 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 17+ messages in thread
[parent not found: <20161125142658.21690-2-uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org>]
* Re: [PATCH v5 1/2] devicetree: Add vendor prefix for CZ.NIC [not found] ` <20161125142658.21690-2-uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> @ 2016-11-25 16:15 ` Gregory CLEMENT 0 siblings, 0 replies; 17+ messages in thread From: Gregory CLEMENT @ 2016-11-25 16:15 UTC (permalink / raw) To: Uwe Kleine-König Cc: Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Tomas Hlavacek, Rob Herring, Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Bedřicha Košatu Hi Uwe, On ven., nov. 25 2016, Uwe Kleine-König <uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> wrote: > Signed-off-by: Uwe Kleine-König <uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> Applied on mvebu/dt and I added back the Acked-by from Rob Herring. Thanks, Gregory > --- > 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 f0a48ea78659..ae9fce9fed03 100644 > --- a/Documentation/devicetree/bindings/vendor-prefixes.txt > +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt > @@ -67,6 +67,7 @@ creative Creative Technology Ltd > crystalfontz Crystalfontz America, Inc. > cubietech Cubietech, Ltd. > cypress Cypress Semiconductor Corporation > +cznic CZ.NIC, z.s.p.o. > dallas Maxim Integrated Products (formerly Dallas Semiconductor) > davicom DAVICOM Semiconductor, Inc. > delta Delta Electronics, Inc. > -- > 2.10.2 > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH v5 2/2] ARM: dts: add support for Turris Omnia 2016-11-25 14:26 [PATCH v5 0/2] ARM: dts: add support for Turris Omnia Uwe Kleine-König 2016-11-25 14:26 ` [PATCH v5 1/2] devicetree: Add vendor prefix for CZ.NIC Uwe Kleine-König @ 2016-11-25 14:26 ` Uwe Kleine-König [not found] ` <20161125142658.21690-3-uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> 1 sibling, 1 reply; 17+ messages in thread From: Uwe Kleine-König @ 2016-11-25 14:26 UTC (permalink / raw) To: Jason Cooper, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth Cc: Mark Rutland, devicetree, Tomas Hlavacek, Rob Herring, linux-arm-kernel, Bedřicha Košatu This machine is an open hardware router by cz.nic driven by a Marvell Armada 385. Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org> --- arch/arm/boot/dts/Makefile | 1 + arch/arm/boot/dts/armada-385-turris-omnia.dts | 334 ++++++++++++++++++++++++++ 2 files changed, 335 insertions(+) create mode 100644 arch/arm/boot/dts/armada-385-turris-omnia.dts diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile index befcd2619902..f1d3b9ff257e 100644 --- a/arch/arm/boot/dts/Makefile +++ b/arch/arm/boot/dts/Makefile @@ -920,6 +920,7 @@ dtb-$(CONFIG_MACH_ARMADA_38X) += \ armada-385-db-ap.dtb \ armada-385-linksys-caiman.dtb \ armada-385-linksys-cobra.dtb \ + armada-385-turris-omnia.dtb \ armada-388-clearfog.dtb \ armada-388-db.dtb \ armada-388-gp.dtb \ diff --git a/arch/arm/boot/dts/armada-385-turris-omnia.dts b/arch/arm/boot/dts/armada-385-turris-omnia.dts new file mode 100644 index 000000000000..bcc10c285889 --- /dev/null +++ b/arch/arm/boot/dts/armada-385-turris-omnia.dts @@ -0,0 +1,334 @@ +/* + * Device Tree file for the Turris Omnia + * + * Copyright (C) 2016 Uwe Kleine-König <uwe@kleine-koenig.org> + * Copyright (C) 2016 Tomas Hlavacek <tmshlvkc@gmail.com> + * + * 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 licensed under the terms of the GNU General Public + * License version 2. This program is licensed "as is" without + * any warranty of any kind, whether express or implied. + * + * 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. + */ + +/* + * Schematic available at https://www.turris.cz/doc/_media/rtrom01-schema.pdf + */ + +/dts-v1/; + +#include <dt-bindings/gpio/gpio.h> +#include <dt-bindings/input/input.h> +#include "armada-385.dtsi" + +/ { + model = "Turris Omnia"; + compatible = "cznic,turris-omnia", "marvell,armada385", "marvell,armada380"; + + chosen { + stdout-path = &uart0; + }; + + memory { + device_type = "memory"; + reg = <0x00000000 0x40000000>; /* 1024 MB */ + }; + + soc { + ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000 + MBUS_ID(0x01, 0x1d) 0 0xfff00000 0x100000 + MBUS_ID(0x09, 0x19) 0 0xf1100000 0x10000 + MBUS_ID(0x09, 0x15) 0 0xf1110000 0x10000>; + + internal-regs { + + /* USB part of the PCIe2/USB 2.0 port */ + usb@58000 { + status = "okay"; + }; + + sata@a8000 { + status = "okay"; + }; + + sdhci@d8000 { + pinctrl-names = "default"; + pinctrl-0 = <&sdhci_pins>; + status = "okay"; + + bus-width = <8>; + no-1-8-v; + non-removable; + }; + + usb3@f0000 { + status = "okay"; + }; + + usb3@f8000 { + status = "okay"; + }; + }; + + pcie-controller { + status = "okay"; + + pcie@1,0 { + /* Port 0, Lane 0 */ + status = "okay"; + }; + + pcie@2,0 { + /* Port 1, Lane 0 */ + status = "okay"; + }; + + pcie@3,0 { + /* Port 2, Lane 0 */ + status = "okay"; + }; + }; + }; +}; + +/* Connected to 88E6176 switch, port 6 */ +ð0 { + pinctrl-names = "default"; + pinctrl-0 = <&ge0_rgmii_pins>; + status = "okay"; + phy-mode = "rgmii-id"; + + fixed-link { + speed = <1000>; + full-duplex; + }; +}; + +/* Connected to 88E6176 switch, port 5 */ +ð1 { + pinctrl-names = "default"; + pinctrl-0 = <&ge1_rgmii_pins>; + status = "okay"; + phy-mode = "rgmii-id"; + + fixed-link { + speed = <1000>; + full-duplex; + }; +}; + +/* WAN port */ +ð2 { + status = "okay"; + phy-mode = "sgmii"; + phy = <&phy1>; +}; + +&i2c0 { + pinctrl-names = "default"; + pinctrl-0 = <&i2c0_pins>; + status = "okay"; + + i2cmux@70 { + compatible = "nxp,pca9547"; + #address-cells = <1>; + #size-cells = <0>; + reg = <0x70>; + status = "okay"; + + i2c@0 { + #address-cells = <1>; + #size-cells = <0>; + reg = <0>; + + /* STM32F0 command interface at address 0x2a */ + /* leds device (in STM32F0) at address 0x2b */ + + eeprom@54 { + compatible = "at,24c64"; + reg = <0x54>; + + /* The EEPROM contains data for bootloader. + * Contents: + * struct omnia_eeprom { + * u32 magic; (=0x0341a034 in LE) + * u32 ramsize; (in GiB) + * char regdomain[4]; + * u32 crc32; + * }; + */ + }; + }; + + i2c@1 { + #address-cells = <1>; + #size-cells = <0>; + reg = <1>; + + /* routed to PCIe0/mSATA connector (CN7A) */ + }; + + i2c@2 { + #address-cells = <1>; + #size-cells = <0>; + reg = <2>; + + /* routed to PCIe1/USB2 connector (CN61A) */ + }; + + i2c@3 { + #address-cells = <1>; + #size-cells = <0>; + reg = <3>; + + /* routed to PCIe2 connector (CN62A) */ + }; + + i2c@4 { + #address-cells = <1>; + #size-cells = <0>; + reg = <4>; + + /* routed to SFP+ */ + }; + + i2c@5 { + #address-cells = <1>; + #size-cells = <0>; + reg = <5>; + + /* ATSHA204A at address 0x64 */ + }; + + i2c@6 { + #address-cells = <1>; + #size-cells = <0>; + reg = <6>; + + /* exposed on pin header */ + }; + + i2c@7 { + #address-cells = <1>; + #size-cells = <0>; + reg = <7>; + + pcawan: gpio@71 { + /* + * GPIO expander for SFP+ signals and + * and phy irq + */ + compatible = "nxp,pca9538"; + reg = <0x71>; + + pinctrl-names = "default"; + pinctrl-0 = <&pcawan_pins>; + + interrupt-parent = <&gpio1>; + interrupts = <14 IRQ_TYPE_LEVEL_LOW>; + + gpio-controller; + #gpio-cells = <2>; + }; + }; + }; +}; + +&mdio { + pinctrl-names = "default"; + pinctrl-0 = <&mdio_pins>; + status = "okay"; + + phy1: phy@1 { + status = "okay"; + compatible = "ethernet-phy-id0141.0DD1", "ethernet-phy-ieee802.3-c22"; + reg = <1>; + + /* irq is connected to &pcawan pin 7 */ + }; + + /* Switch MV88E7176 at address 0x10 */ +}; + +&pinctrl { + pcawan_pins: pcawan-pins { + marvell,pins = "mpp46"; + marvell,function = "gpio"; + }; + + spi0cs0_pins: spi0cs0-pins { + marvell,pins = "mpp25"; + marvell,function = "spi0"; + }; + + spi0cs1_pins: spi0cs1-pins { + marvell,pins = "mpp26"; + marvell,function = "spi0"; + }; +}; + +&spi0 { + pinctrl-names = "default"; + pinctrl-0 = <&spi0_pins &spi0cs0_pins>; + status = "okay"; + + spi-nor@0 { + compatible = "spansion,s25fl164k", "jedec,spi-nor"; + #address-cells = <1>; + #size-cells = <1>; + reg = <0>; + spi-max-frequency = <40000000>; + + partition@0 { + reg = <0x0 0x00100000>; + label = "U-Boot"; + }; + + partition@1 { + reg = <0x00100000 0x00700000>; + label = "Rescue system"; + }; + }; + + /* MISO, MOSI, SCLK and CS1 are routed to pin header CN11 */ +}; + +&uart0 { + /* Pin header CN10 */ + pinctrl-names = "default"; + pinctrl-0 = <&uart0_pins>; + status = "okay"; +}; + +&uart1 { + /* Pin header CN11 */ + pinctrl-names = "default"; + pinctrl-0 = <&uart1_pins>; + status = "okay"; +}; -- 2.10.2 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 17+ messages in thread
[parent not found: <20161125142658.21690-3-uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org>]
* Re: [PATCH v5 2/2] ARM: dts: add support for Turris Omnia [not found] ` <20161125142658.21690-3-uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> @ 2016-11-25 14:32 ` Andrew Lunn 2016-11-25 16:16 ` Gregory CLEMENT 2016-11-27 19:22 ` Andreas Färber 2 siblings, 0 replies; 17+ messages in thread From: Andrew Lunn @ 2016-11-25 14:32 UTC (permalink / raw) To: Uwe Kleine-König Cc: Jason Cooper, Gregory Clement, Sebastian Hesselbarth, Tomas Hlavacek, Rob Herring, Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Bed??icha Ko??atu On Fri, Nov 25, 2016 at 03:26:58PM +0100, Uwe Kleine-König wrote: > This machine is an open hardware router by cz.nic driven by a > Marvell Armada 385. > > Signed-off-by: Uwe Kleine-König <uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> Reviewed-by: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org> Andrew -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v5 2/2] ARM: dts: add support for Turris Omnia [not found] ` <20161125142658.21690-3-uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> 2016-11-25 14:32 ` Andrew Lunn @ 2016-11-25 16:16 ` Gregory CLEMENT [not found] ` <87lgw7ilg9.fsf-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> 2016-11-27 19:22 ` Andreas Färber 2 siblings, 1 reply; 17+ messages in thread From: Gregory CLEMENT @ 2016-11-25 16:16 UTC (permalink / raw) To: Uwe Kleine-König Cc: Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Tomas Hlavacek, Rob Herring, Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Bedřicha Košatu Hi Uwe, On ven., nov. 25 2016, Uwe Kleine-König <uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> wrote: > This machine is an open hardware router by cz.nic driven by a > Marvell Armada 385. > > Signed-off-by: Uwe Kleine-König <uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> Applied on mvebu/dt with few changes: First I added the SoB from Thomas Hlavacek you initially planed to add as seen on IRC. The other change I did are mentioned inline. Thanks, Gregory [...] > +&spi0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&spi0_pins &spi0cs0_pins>; > + status = "okay"; > + > + spi-nor@0 { > + compatible = "spansion,s25fl164k", "jedec,spi-nor"; > + #address-cells = <1>; > + #size-cells = <1>; > + reg = <0>; > + spi-max-frequency = <40000000>; > + + + partitions { + compatible = "fixed-partitions"; + #address-cells = <1>; + #size-cells = <1>; + it is mandatory since v4.4 to use this pattern for partitions. > + partition@0 { > + reg = <0x0 0x00100000>; > + label = "U-Boot"; > + }; > + > + partition@1 { @0x100000 We should use the reg value here ^ > + reg = <0x00100000 0x00700000>; > + label = "Rescue system"; > + }; + }; -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 17+ messages in thread
[parent not found: <87lgw7ilg9.fsf-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>]
* Re: [PATCH v5 2/2] ARM: dts: add support for Turris Omnia [not found] ` <87lgw7ilg9.fsf-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> @ 2016-11-27 16:00 ` Andreas Färber [not found] ` <a7f47999-ec83-6cc8-8119-0087dee17bac-l3A5Bk7waGM@public.gmane.org> 0 siblings, 1 reply; 17+ messages in thread From: Andreas Färber @ 2016-11-27 16:00 UTC (permalink / raw) To: Gregory CLEMENT, Uwe Kleine-König Cc: Mark Rutland, Andrew Lunn, Jason Cooper, devicetree-u79uwXL29TY76Z2rM5mHXA, Tomas Hlavacek, Rob Herring, Bedřicha Košatu, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Sebastian Hesselbarth, Michal Hrusecky Am 25.11.2016 um 17:16 schrieb Gregory CLEMENT: > On ven., nov. 25 2016, Uwe Kleine-König <uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> wrote: >> This machine is an open hardware router by cz.nic driven by a >> Marvell Armada 385. >> >> Signed-off-by: Uwe Kleine-König <uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> > > Applied on mvebu/dt with few changes: [...] >> +&spi0 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&spi0_pins &spi0cs0_pins>; >> + status = "okay"; >> + >> + spi-nor@0 { >> + compatible = "spansion,s25fl164k", "jedec,spi-nor"; >> + #address-cells = <1>; >> + #size-cells = <1>; >> + reg = <0>; >> + spi-max-frequency = <40000000>; >> + > > + > + partitions { > + compatible = "fixed-partitions"; > + #address-cells = <1>; > + #size-cells = <1>; > + > it is mandatory since v4.4 to use this pattern for partitions. > > >> + partition@0 { >> + reg = <0x0 0x00100000>; >> + label = "U-Boot"; >> + }; >> + >> + partition@1 { > @0x100000 > We should use the reg value here ^ The unit name should be without 0x though. In your tree you seem to have it correctly. @Uwe: Note that I had already told CZ.NIC's Michal ~two weeks ago that I have a WIP .dts for the Omnia - looks like no one knows what the other is doing. :( My branch includes cleanups for 385 .dtsi and bug fixes for the switch that I am not seeing in your series: https://github.com/afaerber/linux/commits/omnia-next I am still looking into phy backtraces when the network interfaces go down. @Gregory: Can we please follow up with cleaning up these ugly internal-regs and pcie-controller nodes for consistency? Regards, Andreas >> + reg = <0x00100000 0x00700000>; >> + label = "Rescue system"; >> + }; > > + }; -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 17+ messages in thread
[parent not found: <a7f47999-ec83-6cc8-8119-0087dee17bac-l3A5Bk7waGM@public.gmane.org>]
* Re: [PATCH v5 2/2] ARM: dts: add support for Turris Omnia [not found] ` <a7f47999-ec83-6cc8-8119-0087dee17bac-l3A5Bk7waGM@public.gmane.org> @ 2016-11-27 16:07 ` Andrew Lunn [not found] ` <20161127160731.GD4574-g2DYL2Zd6BY@public.gmane.org> 2016-11-27 21:22 ` Andreas Färber 1 sibling, 1 reply; 17+ messages in thread From: Andrew Lunn @ 2016-11-27 16:07 UTC (permalink / raw) To: Andreas Färber Cc: Gregory CLEMENT, Uwe Kleine-König, Mark Rutland, Jason Cooper, devicetree-u79uwXL29TY76Z2rM5mHXA, Tomas Hlavacek, Rob Herring, Bed??icha Ko??atu, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Sebastian Hesselbarth, Michal Hrusecky > @Uwe: Note that I had already told CZ.NIC's Michal ~two weeks ago that I > have a WIP .dts for the Omnia - looks like no one knows what the other > is doing. Hi Andreas Did you post to the list? Comment on the earlier versions of the patches? The list is the please to coordinate these activities. Andrew -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 17+ messages in thread
[parent not found: <20161127160731.GD4574-g2DYL2Zd6BY@public.gmane.org>]
* Re: [PATCH v5 2/2] ARM: dts: add support for Turris Omnia [not found] ` <20161127160731.GD4574-g2DYL2Zd6BY@public.gmane.org> @ 2016-11-27 16:20 ` Andreas Färber 0 siblings, 0 replies; 17+ messages in thread From: Andreas Färber @ 2016-11-27 16:20 UTC (permalink / raw) To: Andrew Lunn Cc: Gregory CLEMENT, Uwe Kleine-König, Mark Rutland, Jason Cooper, devicetree-u79uwXL29TY76Z2rM5mHXA, Tomas Hlavacek, Rob Herring, Bed??icha Ko??atu, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Sebastian Hesselbarth, Michal Hrusecky Hi, Am 27.11.2016 um 17:07 schrieb Andrew Lunn: >> @Uwe: Note that I had already told CZ.NIC's Michal ~two weeks ago that I >> have a WIP .dts for the Omnia - looks like no one knows what the other >> is doing. > > Hi Andreas > > Did you post to the list? Comment on the earlier versions of the > patches? The list is the please to coordinate these activities. No, it was not yet fully working (only WAN NIC) and I don't have a habit of spamming the list with RFCs. Also I was away the last two weekends. I would've expected to get CC'ed for review though, since CZ.NIC should've been aware of my work. https://lists.opensuse.org/opensuse-arm/2016-11/msg00005.html Cheers, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v5 2/2] ARM: dts: add support for Turris Omnia [not found] ` <a7f47999-ec83-6cc8-8119-0087dee17bac-l3A5Bk7waGM@public.gmane.org> 2016-11-27 16:07 ` Andrew Lunn @ 2016-11-27 21:22 ` Andreas Färber 1 sibling, 0 replies; 17+ messages in thread From: Andreas Färber @ 2016-11-27 21:22 UTC (permalink / raw) To: Uwe Kleine-König, Andrew Lunn Cc: Gregory CLEMENT, Mark Rutland, Michal Hrusecky, Jason Cooper, devicetree-u79uwXL29TY76Z2rM5mHXA, Tomas Hlavacek, Rob Herring, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Bedřicha Košatu, Sebastian Hesselbarth Am 27.11.2016 um 17:00 schrieb Andreas Färber: > @Uwe: Note that I had already told CZ.NIC's Michal ~two weeks ago that I > have a WIP .dts for the Omnia - looks like no one knows what the other > is doing. :( My branch includes cleanups for 385 .dtsi and bug fixes for > the switch that I am not seeing in your series: > > https://github.com/afaerber/linux/commits/omnia-next Archived at https://github.com/afaerber/linux/commits/omnia-next.pre-uwe It seems like four out of my five switch probing bug fixes were already resolved by Andrew in the meantime. Remaining one plus 88E6176 mini-series sent out. Cheers, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v5 2/2] ARM: dts: add support for Turris Omnia [not found] ` <20161125142658.21690-3-uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> 2016-11-25 14:32 ` Andrew Lunn 2016-11-25 16:16 ` Gregory CLEMENT @ 2016-11-27 19:22 ` Andreas Färber [not found] ` <71af60f5-b657-cab4-32a8-00a604fc656e-l3A5Bk7waGM@public.gmane.org> 2 siblings, 1 reply; 17+ messages in thread From: Andreas Färber @ 2016-11-27 19:22 UTC (permalink / raw) To: Uwe Kleine-König, Gregory Clement Cc: Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA, Tomas Hlavacek, Rob Herring, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Bedřicha Košatu Am 25.11.2016 um 15:26 schrieb Uwe Kleine-König: > diff --git a/arch/arm/boot/dts/armada-385-turris-omnia.dts b/arch/arm/boot/dts/armada-385-turris-omnia.dts > new file mode 100644 > index 000000000000..bcc10c285889 > --- /dev/null > +++ b/arch/arm/boot/dts/armada-385-turris-omnia.dts [...] > + chosen { > + stdout-path = &uart0; > + }; I notice that the other 38x boards (and thus my previous Omnia .dts) use "serial0:115200n8". Can we really rely on the driver defaults here? Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 17+ messages in thread
[parent not found: <71af60f5-b657-cab4-32a8-00a604fc656e-l3A5Bk7waGM@public.gmane.org>]
* [PATCH] ARM: dts: mvebu: Fix armada-385-turris-omnia stdout-path [not found] ` <71af60f5-b657-cab4-32a8-00a604fc656e-l3A5Bk7waGM@public.gmane.org> @ 2016-11-27 19:37 ` Andreas Färber [not found] ` <1480275444-4220-1-git-send-email-afaerber-l3A5Bk7waGM@public.gmane.org> 2016-11-27 19:39 ` [PATCH v5 2/2] ARM: dts: add support for Turris Omnia Andreas Färber 1 sibling, 1 reply; 17+ messages in thread From: Andreas Färber @ 2016-11-27 19:37 UTC (permalink / raw) To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r Cc: Uwe Kleine-König, Michal Hrusecki, Tomas Hlavacek, Bedřicha Košatu, Andreas Färber, Jason Cooper, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth, Rob Herring, Mark Rutland, Russell King, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA Specify the baudrate. Fixes: 26ca8b52d6e1 ("ARM: dts: add support for Turris Omnia") Cc: Uwe Kleine-König <uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> Signed-off-by: Andreas Färber <afaerber-l3A5Bk7waGM@public.gmane.org> --- arch/arm/boot/dts/armada-385-turris-omnia.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/armada-385-turris-omnia.dts b/arch/arm/boot/dts/armada-385-turris-omnia.dts index f53cb8b73610..2eff012287d4 100644 --- a/arch/arm/boot/dts/armada-385-turris-omnia.dts +++ b/arch/arm/boot/dts/armada-385-turris-omnia.dts @@ -52,7 +52,7 @@ compatible = "cznic,turris-omnia", "marvell,armada385", "marvell,armada380"; chosen { - stdout-path = &uart0; + stdout-path = "serial0:115200n8"; }; memory { -- 2.6.6 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 17+ messages in thread
[parent not found: <1480275444-4220-1-git-send-email-afaerber-l3A5Bk7waGM@public.gmane.org>]
* Re: [PATCH] ARM: dts: mvebu: Fix armada-385-turris-omnia stdout-path [not found] ` <1480275444-4220-1-git-send-email-afaerber-l3A5Bk7waGM@public.gmane.org> @ 2016-11-27 21:10 ` Andrew Lunn 2016-11-27 21:25 ` Uwe Kleine-König 1 sibling, 0 replies; 17+ messages in thread From: Andrew Lunn @ 2016-11-27 21:10 UTC (permalink / raw) To: Andreas Färber Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Uwe Kleine-König, Michal Hrusecki, Tomas Hlavacek, Bed??icha Ko??atu, Jason Cooper, Gregory Clement, Sebastian Hesselbarth, Rob Herring, Mark Rutland, Russell King, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA On Sun, Nov 27, 2016 at 08:37:24PM +0100, Andreas Färber wrote: > Specify the baudrate. Hi Andreas Please put each patch/patchset in a new thread. > Fixes: 26ca8b52d6e1 ("ARM: dts: add support for Turris Omnia") > Cc: Uwe Kleine-König <uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> > Signed-off-by: Andreas Färber <afaerber-l3A5Bk7waGM@public.gmane.org> Reviewed-by: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org> Andrew -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] ARM: dts: mvebu: Fix armada-385-turris-omnia stdout-path [not found] ` <1480275444-4220-1-git-send-email-afaerber-l3A5Bk7waGM@public.gmane.org> 2016-11-27 21:10 ` Andrew Lunn @ 2016-11-27 21:25 ` Uwe Kleine-König [not found] ` <20161127212528.wmqbwciz5ltnfws3-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> 1 sibling, 1 reply; 17+ messages in thread From: Uwe Kleine-König @ 2016-11-27 21:25 UTC (permalink / raw) To: Andreas Färber Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Mark Rutland, Andrew Lunn, Jason Cooper, devicetree-u79uwXL29TY76Z2rM5mHXA, Tomas Hlavacek, Russell King, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Gregory Clement, Sebastian Hesselbarth, Bedřicha Košatu, Michal Hrusecki On Sun, Nov 27, 2016 at 08:37:24PM +0100, Andreas Färber wrote: > Specify the baudrate. > > Fixes: 26ca8b52d6e1 ("ARM: dts: add support for Turris Omnia") > Cc: Uwe Kleine-König <uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> > Signed-off-by: Andreas Färber <afaerber-l3A5Bk7waGM@public.gmane.org> You said with plain &uart0 the kernel uses a wrong baud rate? That's strange. For me it works and I think it's the intended behaviour to dermine the baud rate setup by the bootloader and use this. I'd prefer it this way over hard coding the baud rate. > arch/arm/boot/dts/armada-385-turris-omnia.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/armada-385-turris-omnia.dts b/arch/arm/boot/dts/armada-385-turris-omnia.dts > index f53cb8b73610..2eff012287d4 100644 > --- a/arch/arm/boot/dts/armada-385-turris-omnia.dts > +++ b/arch/arm/boot/dts/armada-385-turris-omnia.dts > @@ -52,7 +52,7 @@ > compatible = "cznic,turris-omnia", "marvell,armada385", "marvell,armada380"; > > chosen { > - stdout-path = &uart0; > + stdout-path = "serial0:115200n8"; > }; > > memory { This has the downside to depend on the alias. Not sure this is considered modern. An alternative would be: stdout-path = "/soc/internal-regs/serial@12000:115200n8"; (maybe there even exists syntactic sugar to express this using &uart0?) Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ | -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 17+ messages in thread
[parent not found: <20161127212528.wmqbwciz5ltnfws3-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>]
* Re: [PATCH] ARM: dts: mvebu: Fix armada-385-turris-omnia stdout-path [not found] ` <20161127212528.wmqbwciz5ltnfws3-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> @ 2016-11-27 22:14 ` Andreas Färber [not found] ` <0f51c7a1-6d52-3e3e-edfe-e7b10917ecab-l3A5Bk7waGM@public.gmane.org> 0 siblings, 1 reply; 17+ messages in thread From: Andreas Färber @ 2016-11-27 22:14 UTC (permalink / raw) To: Uwe Kleine-König Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Mark Rutland, Andrew Lunn, Jason Cooper, devicetree-u79uwXL29TY76Z2rM5mHXA, Tomas Hlavacek, Russell King, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Gregory Clement, Sebastian Hesselbarth, Bedřicha Košatu, Michal Hrusecki Am 27.11.2016 um 22:25 schrieb Uwe Kleine-König: > On Sun, Nov 27, 2016 at 08:37:24PM +0100, Andreas Färber wrote: >> Specify the baudrate. >> >> Fixes: 26ca8b52d6e1 ("ARM: dts: add support for Turris Omnia") >> Cc: Uwe Kleine-König <uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> >> Signed-off-by: Andreas Färber <afaerber-l3A5Bk7waGM@public.gmane.org> > > You said with plain &uart0 the kernel uses a wrong baud rate? That's > strange. For me it works and I think it's the intended behaviour to > dermine the baud rate setup by the bootloader and use this. IIRC the 8250 driver defaults to 9600n8 if unspecified. Kernel tested: 4.9.0-rc2-next-20161028-00010-g4fb44d9-dirty Maybe you used some console= argument overriding it? > I'd prefer it this way over hard coding the baud rate. > >> arch/arm/boot/dts/armada-385-turris-omnia.dts | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/arm/boot/dts/armada-385-turris-omnia.dts b/arch/arm/boot/dts/armada-385-turris-omnia.dts >> index f53cb8b73610..2eff012287d4 100644 >> --- a/arch/arm/boot/dts/armada-385-turris-omnia.dts >> +++ b/arch/arm/boot/dts/armada-385-turris-omnia.dts >> @@ -52,7 +52,7 @@ >> compatible = "cznic,turris-omnia", "marvell,armada385", "marvell,armada380"; >> >> chosen { >> - stdout-path = &uart0; >> + stdout-path = "serial0:115200n8"; >> }; >> >> memory { > > This has the downside to depend on the alias. Not sure this is > considered modern. An alternative would be: > > stdout-path = "/soc/internal-regs/serial@12000:115200n8"; Please don't unroll the path I'm trying to abstract elsewhere. Like I said, the "serialX:115200n8" syntax is what all other Armada 38x boards use: git grep stdout-path -- arch/arm/boot/dts/ | grep armada-38 arch/arm/boot/dts/armada-385-db-ap.dts: stdout-path = "serial1:115200n8"; arch/arm/boot/dts/armada-385-linksys.dtsi: stdout-path = "serial0:115200n8"; arch/arm/boot/dts/armada-388-clearfog.dts: stdout-path = "serial0:115200n8"; arch/arm/boot/dts/armada-388-db.dts: stdout-path = "serial0:115200n8"; arch/arm/boot/dts/armada-388-gp.dts: stdout-path = "serial0:115200n8"; arch/arm/boot/dts/armada-388-rd.dts: stdout-path = "serial0:115200n8"; The alias is needed to reliably determine the tty device number and is set "globally" in armada-38x.dtsi, so why is it a problem to rely on? Regards, Andreas > > (maybe there even exists syntactic sugar to express this using &uart0?) -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 17+ messages in thread
[parent not found: <0f51c7a1-6d52-3e3e-edfe-e7b10917ecab-l3A5Bk7waGM@public.gmane.org>]
* Re: [PATCH] ARM: dts: mvebu: Fix armada-385-turris-omnia stdout-path [not found] ` <0f51c7a1-6d52-3e3e-edfe-e7b10917ecab-l3A5Bk7waGM@public.gmane.org> @ 2016-11-27 22:30 ` Uwe Kleine-König 0 siblings, 0 replies; 17+ messages in thread From: Uwe Kleine-König @ 2016-11-27 22:30 UTC (permalink / raw) To: Andreas Färber Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Mark Rutland, Andrew Lunn, Jason Cooper, devicetree-u79uwXL29TY76Z2rM5mHXA, Tomas Hlavacek, Russell King, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Gregory Clement, Sebastian Hesselbarth, Bedřicha Košatu, Michal Hrusecki Hello, On Sun, Nov 27, 2016 at 11:14:37PM +0100, Andreas Färber wrote: > Am 27.11.2016 um 22:25 schrieb Uwe Kleine-König: > > On Sun, Nov 27, 2016 at 08:37:24PM +0100, Andreas Färber wrote: > >> Specify the baudrate. > >> > >> Fixes: 26ca8b52d6e1 ("ARM: dts: add support for Turris Omnia") > >> Cc: Uwe Kleine-König <uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org> > >> Signed-off-by: Andreas Färber <afaerber-l3A5Bk7waGM@public.gmane.org> > > > > You said with plain &uart0 the kernel uses a wrong baud rate? That's > > strange. For me it works and I think it's the intended behaviour to > > dermine the baud rate setup by the bootloader and use this. > > IIRC the 8250 driver defaults to 9600n8 if unspecified. > > Kernel tested: 4.9.0-rc2-next-20161028-00010-g4fb44d9-dirty > > Maybe you used some console= argument overriding it? Yes, you're right. > > I'd prefer it this way over hard coding the baud rate. > > > >> arch/arm/boot/dts/armada-385-turris-omnia.dts | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> diff --git a/arch/arm/boot/dts/armada-385-turris-omnia.dts b/arch/arm/boot/dts/armada-385-turris-omnia.dts > >> index f53cb8b73610..2eff012287d4 100644 > >> --- a/arch/arm/boot/dts/armada-385-turris-omnia.dts > >> +++ b/arch/arm/boot/dts/armada-385-turris-omnia.dts > >> @@ -52,7 +52,7 @@ > >> compatible = "cznic,turris-omnia", "marvell,armada385", "marvell,armada380"; > >> > >> chosen { > >> - stdout-path = &uart0; > >> + stdout-path = "serial0:115200n8"; > >> }; > >> > >> memory { > > > > This has the downside to depend on the alias. Not sure this is > > considered modern. An alternative would be: > > > > stdout-path = "/soc/internal-regs/serial@12000:115200n8"; > > Please don't unroll the path I'm trying to abstract elsewhere. Yeah, specifying the path isn't nice. > Like I said, the "serialX:115200n8" syntax is what all other Armada 38x > boards use: This isn't a reason to not think about better alternatives. An if something like: stdout-path = &uart0 + ":115200n8"; would be possible, I'd definitely prefer it over "serial0:115200n8". > git grep stdout-path -- arch/arm/boot/dts/ | grep armada-38 > arch/arm/boot/dts/armada-385-db-ap.dts: stdout-path = "serial1:115200n8"; > arch/arm/boot/dts/armada-385-linksys.dtsi: stdout-path = > "serial0:115200n8"; > arch/arm/boot/dts/armada-388-clearfog.dts: stdout-path = > "serial0:115200n8"; > arch/arm/boot/dts/armada-388-db.dts: stdout-path = "serial0:115200n8"; > arch/arm/boot/dts/armada-388-gp.dts: stdout-path = "serial0:115200n8"; > arch/arm/boot/dts/armada-388-rd.dts: stdout-path = "serial0:115200n8"; > > The alias is needed to reliably determine the tty device number and is > set "globally" in armada-38x.dtsi, so why is it a problem to rely on? AFAIK aliases are seen as (still necessary) evil by the dt people. So if you can stop making use of them, that would be nice. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ | -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v5 2/2] ARM: dts: add support for Turris Omnia [not found] ` <71af60f5-b657-cab4-32a8-00a604fc656e-l3A5Bk7waGM@public.gmane.org> 2016-11-27 19:37 ` [PATCH] ARM: dts: mvebu: Fix armada-385-turris-omnia stdout-path Andreas Färber @ 2016-11-27 19:39 ` Andreas Färber 1 sibling, 0 replies; 17+ messages in thread From: Andreas Färber @ 2016-11-27 19:39 UTC (permalink / raw) To: Uwe Kleine-König, Gregory Clement Cc: Mark Rutland, Andrew Lunn, Jason Cooper, devicetree-u79uwXL29TY76Z2rM5mHXA, Tomas Hlavacek, Rob Herring, Bedřicha Košatu, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Sebastian Hesselbarth Am 27.11.2016 um 20:22 schrieb Andreas Färber: > Am 25.11.2016 um 15:26 schrieb Uwe Kleine-König: >> diff --git a/arch/arm/boot/dts/armada-385-turris-omnia.dts b/arch/arm/boot/dts/armada-385-turris-omnia.dts >> new file mode 100644 >> index 000000000000..bcc10c285889 >> --- /dev/null >> +++ b/arch/arm/boot/dts/armada-385-turris-omnia.dts > [...] >> + chosen { >> + stdout-path = &uart0; >> + }; > > I notice that the other 38x boards (and thus my previous Omnia .dts) use > "serial0:115200n8". Can we really rely on the driver defaults here? Answering my own question: No, with the mvebu/dt .dts I do not get any serial output. Patch sent. Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2016-11-27 22:30 UTC | newest]
Thread overview: 17+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-25 14:26 [PATCH v5 0/2] ARM: dts: add support for Turris Omnia Uwe Kleine-König
2016-11-25 14:26 ` [PATCH v5 1/2] devicetree: Add vendor prefix for CZ.NIC Uwe Kleine-König
[not found] ` <20161125142658.21690-2-uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org>
2016-11-25 16:15 ` Gregory CLEMENT
2016-11-25 14:26 ` [PATCH v5 2/2] ARM: dts: add support for Turris Omnia Uwe Kleine-König
[not found] ` <20161125142658.21690-3-uwe-rXY34ruvC2xidJT2blvkqNi2O/JbrIOy@public.gmane.org>
2016-11-25 14:32 ` Andrew Lunn
2016-11-25 16:16 ` Gregory CLEMENT
[not found] ` <87lgw7ilg9.fsf-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2016-11-27 16:00 ` Andreas Färber
[not found] ` <a7f47999-ec83-6cc8-8119-0087dee17bac-l3A5Bk7waGM@public.gmane.org>
2016-11-27 16:07 ` Andrew Lunn
[not found] ` <20161127160731.GD4574-g2DYL2Zd6BY@public.gmane.org>
2016-11-27 16:20 ` Andreas Färber
2016-11-27 21:22 ` Andreas Färber
2016-11-27 19:22 ` Andreas Färber
[not found] ` <71af60f5-b657-cab4-32a8-00a604fc656e-l3A5Bk7waGM@public.gmane.org>
2016-11-27 19:37 ` [PATCH] ARM: dts: mvebu: Fix armada-385-turris-omnia stdout-path Andreas Färber
[not found] ` <1480275444-4220-1-git-send-email-afaerber-l3A5Bk7waGM@public.gmane.org>
2016-11-27 21:10 ` Andrew Lunn
2016-11-27 21:25 ` Uwe Kleine-König
[not found] ` <20161127212528.wmqbwciz5ltnfws3-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2016-11-27 22:14 ` Andreas Färber
[not found] ` <0f51c7a1-6d52-3e3e-edfe-e7b10917ecab-l3A5Bk7waGM@public.gmane.org>
2016-11-27 22:30 ` Uwe Kleine-König
2016-11-27 19:39 ` [PATCH v5 2/2] ARM: dts: add support for Turris Omnia 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).