diff for duplicates of <438381472136802@web24g.yandex.ru> diff --git a/a/1.txt b/N1/1.txt index 12abbfa..80cf865 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -4,48 +4,48 @@ > Hi, > > On 25-08-16 16:12, Icenowy Zheng wrote: ->> ?25.08.2016, 16:03, "Maxime Ripard" <maxime.ripard@free-electrons.com>: ->>> ?Hi, +>> 25.08.2016, 16:03, "Maxime Ripard" <maxime.ripard@free-electrons.com>: +>>> Hi, >>> ->>> ?On Thu, Aug 25, 2016 at 02:57:24PM +0800, Icenowy Zheng wrote: ->>>> ??>>>> diff --git a/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts b/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts ->>>> ??>>>> index 78823d8..3ac22d4 100644 ->>>> ??>>>> --- a/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts ->>>> ??>>>> +++ b/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts ->>>> ??>>>> @@ -48,6 +48,10 @@ ->>>> ??>>>> model = "INet-D978 Rev 02"; ->>>> ??>>>> compatible = "primux,inet-d978-rev2", "allwinner,sun8i-a33"; ->>>> ??>>>> ->>>> ??>>>> + aliases { ->>>> ??>>>> + serial1 = &uart1; ->>>> ??>>>> + }; ->>>> ??>>>> + ->>>> ??>>> ->>>> ??>>> Is there any other UART in the system? ->>>> ??>> ->>>> ??>> serial0 is defined in sun8i-reference-design-tablet.dtsi, as r_uart. ->>>> ??> ->>>> ??> If your board does not have r_uart pads, then the right thing to do ->>>> ??> would be to disable it. You can then have uart1 as serial0. +>>> On Thu, Aug 25, 2016 at 02:57:24PM +0800, Icenowy Zheng wrote: +>>>> >>>> diff --git a/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts b/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts +>>>> >>>> index 78823d8..3ac22d4 100644 +>>>> >>>> --- a/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts +>>>> >>>> +++ b/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts +>>>> >>>> @@ -48,6 +48,10 @@ +>>>> >>>> model = "INet-D978 Rev 02"; +>>>> >>>> compatible = "primux,inet-d978-rev2", "allwinner,sun8i-a33"; +>>>> >>>> +>>>> >>>> + aliases { +>>>> >>>> + serial1 = &uart1; +>>>> >>>> + }; +>>>> >>>> + +>>>> >>> +>>>> >>> Is there any other UART in the system? +>>>> >> +>>>> >> serial0 is defined in sun8i-reference-design-tablet.dtsi, as r_uart. +>>>> > +>>>> > If your board does not have r_uart pads, then the right thing to do +>>>> > would be to disable it. You can then have uart1 as serial0. >>>> ->>>> ??I don't know it. The RX/TX is wired on PF2/PF4 on the board (UART0 muxed with ->>>> ??MMC) +>>>> I don't know it. The RX/TX is wired on PF2/PF4 on the board (UART0 muxed with +>>>> MMC) >>> ->>> ?Then you don't support it and you need to disable it. +>>> Then you don't support it and you need to disable it. >>> ->>>> ??But many codes will require a ttyS0 as console... So I think the ->>>> ??bluetooth should be located at ttyS1... +>>>> But many codes will require a ttyS0 as console... So I think the +>>>> bluetooth should be located at ttyS1... >>> ->>> ?Most of them would use /dev/console anyway. What do you have in mind? +>>> Most of them would use /dev/console anyway. What do you have in mind? >> ->> ?As a general configuration, /dev/ttyS0 will be indicated as the default console= ->> ?value... (Otherwise more boards will fail) +>> As a general configuration, /dev/ttyS0 will be indicated as the default console= +>> value... (Otherwise more boards will fail) >> ->> ?Or maybe we can specify uart0 as serial0, and leave it as disabled? (Thus if the ->> ?debug port is needed, we can easily apply a overlay with &mmc0 status as disabled, ->> ?and &uart0 status as okay) +>> Or maybe we can specify uart0 as serial0, and leave it as disabled? (Thus if the +>> debug port is needed, we can easily apply a overlay with &mmc0 status as disabled, +>> and &uart0 status as okay) >> ->> ?On all my A33 boards, the official boot0, u-boot all uses uart0 as debugging port. +>> On all my A33 boards, the official boot0, u-boot all uses uart0 as debugging port. > > If no uart is available (and being muxed with the mmc counts as not being available) > then you should not have any serial alias in the dts. @@ -65,3 +65,8 @@ Can I have no ttyS0 and just make bluetooth ttyS1? > Regards, > > Hans + +_______________________________________________ +linux-arm-kernel mailing list +linux-arm-kernel@lists.infradead.org +http://lists.infradead.org/mailman/listinfo/linux-arm-kernel diff --git a/a/content_digest b/N1/content_digest index 9efaa46..c26149c 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -7,10 +7,18 @@ "ref\020160825080334.GD7776@lukather\0" "ref\0116041472134355@web12g.yandex.ru\0" "ref\06580ba1b-f542-58d4-9f0a-fdd8e9b86092@redhat.com\0" - "From\0icenowy@aosc.xyz (Icenowy Zheng)\0" - "Subject\0[PATCH 2/2] ARM: dts: sun8i: enable UART1 for iNet D978 Rev2 board\0" + "From\0Icenowy Zheng <icenowy@aosc.xyz>\0" + "Subject\0Re: [PATCH 2/2] ARM: dts: sun8i: enable UART1 for iNet D978 Rev2 board\0" "Date\0Thu, 25 Aug 2016 22:53:22 +0800\0" - "To\0linux-arm-kernel@lists.infradead.org\0" + "To\0Hans de Goede <hdegoede@redhat.com>" + " Maxime Ripard <maxime.ripard@free-electrons.com>\0" + "Cc\0Mark Rutland <mark.rutland@arm.com>" + devicetree@vger.kernel.org <devicetree@vger.kernel.org> + Russell King <linux@armlinux.org.uk> + linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> + Chen-Yu Tsai <wens@csie.org> + Rob Herring <robh+dt@kernel.org> + " linux-arm-kernel@lists.infradead.org <linux-arm-kernel@lists.infradead.org>\0" "\00:1\0" "b\0" "\n" @@ -19,48 +27,48 @@ "> Hi,\n" ">\n" "> On 25-08-16 16:12, Icenowy Zheng wrote:\n" - ">> ?25.08.2016, 16:03, \"Maxime Ripard\" <maxime.ripard@free-electrons.com>:\n" - ">>> ?Hi,\n" + ">> \302\24025.08.2016, 16:03, \"Maxime Ripard\" <maxime.ripard@free-electrons.com>:\n" + ">>> \302\240Hi,\n" ">>>\n" - ">>> ?On Thu, Aug 25, 2016 at 02:57:24PM +0800, Icenowy Zheng wrote:\n" - ">>>> ??>>>> diff --git a/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts b/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts\n" - ">>>> ??>>>> index 78823d8..3ac22d4 100644\n" - ">>>> ??>>>> --- a/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts\n" - ">>>> ??>>>> +++ b/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts\n" - ">>>> ??>>>> @@ -48,6 +48,10 @@\n" - ">>>> ??>>>> model = \"INet-D978 Rev 02\";\n" - ">>>> ??>>>> compatible = \"primux,inet-d978-rev2\", \"allwinner,sun8i-a33\";\n" - ">>>> ??>>>>\n" - ">>>> ??>>>> + aliases {\n" - ">>>> ??>>>> + serial1 = &uart1;\n" - ">>>> ??>>>> + };\n" - ">>>> ??>>>> +\n" - ">>>> ??>>>\n" - ">>>> ??>>> Is there any other UART in the system?\n" - ">>>> ??>>\n" - ">>>> ??>> serial0 is defined in sun8i-reference-design-tablet.dtsi, as r_uart.\n" - ">>>> ??>\n" - ">>>> ??> If your board does not have r_uart pads, then the right thing to do\n" - ">>>> ??> would be to disable it. You can then have uart1 as serial0.\n" + ">>> \302\240On Thu, Aug 25, 2016 at 02:57:24PM +0800, Icenowy Zheng wrote:\n" + ">>>> \302\240\302\240>>>> diff --git a/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts b/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts\n" + ">>>> \302\240\302\240>>>> index 78823d8..3ac22d4 100644\n" + ">>>> \302\240\302\240>>>> --- a/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts\n" + ">>>> \302\240\302\240>>>> +++ b/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts\n" + ">>>> \302\240\302\240>>>> @@ -48,6 +48,10 @@\n" + ">>>> \302\240\302\240>>>> model = \"INet-D978 Rev 02\";\n" + ">>>> \302\240\302\240>>>> compatible = \"primux,inet-d978-rev2\", \"allwinner,sun8i-a33\";\n" + ">>>> \302\240\302\240>>>>\n" + ">>>> \302\240\302\240>>>> + aliases {\n" + ">>>> \302\240\302\240>>>> + serial1 = &uart1;\n" + ">>>> \302\240\302\240>>>> + };\n" + ">>>> \302\240\302\240>>>> +\n" + ">>>> \302\240\302\240>>>\n" + ">>>> \302\240\302\240>>> Is there any other UART in the system?\n" + ">>>> \302\240\302\240>>\n" + ">>>> \302\240\302\240>> serial0 is defined in sun8i-reference-design-tablet.dtsi, as r_uart.\n" + ">>>> \302\240\302\240>\n" + ">>>> \302\240\302\240> If your board does not have r_uart pads, then the right thing to do\n" + ">>>> \302\240\302\240> would be to disable it. You can then have uart1 as serial0.\n" ">>>>\n" - ">>>> ??I don't know it. The RX/TX is wired on PF2/PF4 on the board (UART0 muxed with\n" - ">>>> ??MMC)\n" + ">>>> \302\240\302\240I don't know it. The RX/TX is wired on PF2/PF4 on the board (UART0 muxed with\n" + ">>>> \302\240\302\240MMC)\n" ">>>\n" - ">>> ?Then you don't support it and you need to disable it.\n" + ">>> \302\240Then you don't support it and you need to disable it.\n" ">>>\n" - ">>>> ??But many codes will require a ttyS0 as console... So I think the\n" - ">>>> ??bluetooth should be located at ttyS1...\n" + ">>>> \302\240\302\240But many codes will require a ttyS0 as console... So I think the\n" + ">>>> \302\240\302\240bluetooth should be located at ttyS1...\n" ">>>\n" - ">>> ?Most of them would use /dev/console anyway. What do you have in mind?\n" + ">>> \302\240Most of them would use /dev/console anyway. What do you have in mind?\n" ">>\n" - ">> ?As a general configuration, /dev/ttyS0 will be indicated as the default console=\n" - ">> ?value... (Otherwise more boards will fail)\n" + ">> \302\240As a general configuration, /dev/ttyS0 will be indicated as the default console=\n" + ">> \302\240value... (Otherwise more boards will fail)\n" ">>\n" - ">> ?Or maybe we can specify uart0 as serial0, and leave it as disabled? (Thus if the\n" - ">> ?debug port is needed, we can easily apply a overlay with &mmc0 status as disabled,\n" - ">> ?and &uart0 status as okay)\n" + ">> \302\240Or maybe we can specify uart0 as serial0, and leave it as disabled? (Thus if the\n" + ">> \302\240debug port is needed, we can easily apply a overlay with &mmc0 status as disabled,\n" + ">> \302\240and &uart0 status as okay)\n" ">>\n" - ">> ?On all my A33 boards, the official boot0, u-boot all uses uart0 as debugging port.\n" + ">> \302\240On all my A33 boards, the official boot0, u-boot all uses uart0 as debugging port.\n" ">\n" "> If no uart is available (and being muxed with the mmc counts as not being available)\n" "> then you should not have any serial alias in the dts.\n" @@ -79,6 +87,11 @@ ">\n" "> Regards,\n" ">\n" - > Hans + "> Hans\n" + "\n" + "_______________________________________________\n" + "linux-arm-kernel mailing list\n" + "linux-arm-kernel@lists.infradead.org\n" + http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -691fa44b8bffe36ae8d546a0b560eac1cbcbd82486ed8ae53955dbcb895c1ad2 +d73e354cbe7348227d3df0f897470b755a2d17135743378124af76db4dc75c5d
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.