From mboxrd@z Thu Jan 1 00:00:00 1970 From: icenowy@aosc.xyz (Icenowy Zheng) Date: Thu, 25 Aug 2016 22:53:22 +0800 Subject: [PATCH 2/2] ARM: dts: sun8i: enable UART1 for iNet D978 Rev2 board In-Reply-To: <6580ba1b-f542-58d4-9f0a-fdd8e9b86092@redhat.com> References: <20160823064451.25997-1-icenowy@aosc.xyz> <20160823064451.25997-3-icenowy@aosc.xyz> <20160824193600.GS8103@lukather> <43951472083107@web23m.yandex.ru> <97321472108244@web8m.yandex.ru> <20160825080334.GD7776@lukather> <116041472134355@web12g.yandex.ru> <6580ba1b-f542-58d4-9f0a-fdd8e9b86092@redhat.com> Message-ID: <438381472136802@web24g.yandex.ru> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 25.08.2016, 22:18, "Hans de Goede" : > Hi, > > On 25-08-16 16:12, Icenowy Zheng wrote: >> ?25.08.2016, 16:03, "Maxime Ripard" : >>> ?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. >>>> >>>> ??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. >>> >>>> ??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? >> >> ?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) >> >> ?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. > > In u-boot you need to use: > > # CONFIG_REQUIRE_SERIAL_CONSOLE is not set > > In the defconfig in this case, otherwise u-boot is going to be unhappy about > not having a serial console. It's not the problem. Can I have no ttyS0 and just make bluetooth ttyS1? > > Regards, > > Hans From mboxrd@z Thu Jan 1 00:00:00 1970 From: Icenowy Zheng Subject: Re: [PATCH 2/2] ARM: dts: sun8i: enable UART1 for iNet D978 Rev2 board Date: Thu, 25 Aug 2016 22:53:22 +0800 Message-ID: <438381472136802@web24g.yandex.ru> References: <20160823064451.25997-1-icenowy@aosc.xyz> <20160823064451.25997-3-icenowy@aosc.xyz> <20160824193600.GS8103@lukather> <43951472083107@web23m.yandex.ru> <97321472108244@web8m.yandex.ru> <20160825080334.GD7776@lukather> <116041472134355@web12g.yandex.ru> <6580ba1b-f542-58d4-9f0a-fdd8e9b86092@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <6580ba1b-f542-58d4-9f0a-fdd8e9b86092@redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Hans de Goede , Maxime Ripard Cc: Mark Rutland , "devicetree@vger.kernel.org" , Russell King , "linux-kernel@vger.kernel.org" , Chen-Yu Tsai , Rob Herring , "linux-arm-kernel@lists.infradead.org" List-Id: devicetree@vger.kernel.org CgoyNS4wOC4yMDE2LCAyMjoxOCwgIkhhbnMgZGUgR29lZGUiIDxoZGVnb2VkZUByZWRoYXQuY29t PjoKPiBIaSwKPgo+IE9uIDI1LTA4LTE2IDE2OjEyLCBJY2Vub3d5IFpoZW5nIHdyb3RlOgo+PiDC oDI1LjA4LjIwMTYsIDE2OjAzLCAiTWF4aW1lIFJpcGFyZCIgPG1heGltZS5yaXBhcmRAZnJlZS1l bGVjdHJvbnMuY29tPjoKPj4+IMKgSGksCj4+Pgo+Pj4gwqBPbiBUaHUsIEF1ZyAyNSwgMjAxNiBh dCAwMjo1NzoyNFBNICswODAwLCBJY2Vub3d5IFpoZW5nIHdyb3RlOgo+Pj4+IMKgwqA+Pj4+IGRp ZmYgLS1naXQgYS9hcmNoL2FybS9ib290L2R0cy9zdW44aS1hMzMtaW5ldC1kOTc4LXJldjIuZHRz IGIvYXJjaC9hcm0vYm9vdC9kdHMvc3VuOGktYTMzLWluZXQtZDk3OC1yZXYyLmR0cwo+Pj4+IMKg wqA+Pj4+IGluZGV4IDc4ODIzZDguLjNhYzIyZDQgMTAwNjQ0Cj4+Pj4gwqDCoD4+Pj4gLS0tIGEv YXJjaC9hcm0vYm9vdC9kdHMvc3VuOGktYTMzLWluZXQtZDk3OC1yZXYyLmR0cwo+Pj4+IMKgwqA+ Pj4+ICsrKyBiL2FyY2gvYXJtL2Jvb3QvZHRzL3N1bjhpLWEzMy1pbmV0LWQ5NzgtcmV2Mi5kdHMK Pj4+PiDCoMKgPj4+PiBAQCAtNDgsNiArNDgsMTAgQEAKPj4+PiDCoMKgPj4+PiBtb2RlbCA9ICJJ TmV0LUQ5NzggUmV2IDAyIjsKPj4+PiDCoMKgPj4+PiBjb21wYXRpYmxlID0gInByaW11eCxpbmV0 LWQ5NzgtcmV2MiIsICJhbGx3aW5uZXIsc3VuOGktYTMzIjsKPj4+PiDCoMKgPj4+Pgo+Pj4+IMKg wqA+Pj4+ICsgYWxpYXNlcyB7Cj4+Pj4gwqDCoD4+Pj4gKyBzZXJpYWwxID0gJnVhcnQxOwo+Pj4+ IMKgwqA+Pj4+ICsgfTsKPj4+PiDCoMKgPj4+PiArCj4+Pj4gwqDCoD4+Pgo+Pj4+IMKgwqA+Pj4g SXMgdGhlcmUgYW55IG90aGVyIFVBUlQgaW4gdGhlIHN5c3RlbT8KPj4+PiDCoMKgPj4KPj4+PiDC oMKgPj4gc2VyaWFsMCBpcyBkZWZpbmVkIGluIHN1bjhpLXJlZmVyZW5jZS1kZXNpZ24tdGFibGV0 LmR0c2ksIGFzIHJfdWFydC4KPj4+PiDCoMKgPgo+Pj4+IMKgwqA+IElmIHlvdXIgYm9hcmQgZG9l cyBub3QgaGF2ZSByX3VhcnQgcGFkcywgdGhlbiB0aGUgcmlnaHQgdGhpbmcgdG8gZG8KPj4+PiDC oMKgPiB3b3VsZCBiZSB0byBkaXNhYmxlIGl0LiBZb3UgY2FuIHRoZW4gaGF2ZSB1YXJ0MSBhcyBz ZXJpYWwwLgo+Pj4+Cj4+Pj4gwqDCoEkgZG9uJ3Qga25vdyBpdC4gVGhlIFJYL1RYIGlzIHdpcmVk IG9uIFBGMi9QRjQgb24gdGhlIGJvYXJkIChVQVJUMCBtdXhlZCB3aXRoCj4+Pj4gwqDCoE1NQykK Pj4+Cj4+PiDCoFRoZW4geW91IGRvbid0IHN1cHBvcnQgaXQgYW5kIHlvdSBuZWVkIHRvIGRpc2Fi bGUgaXQuCj4+Pgo+Pj4+IMKgwqBCdXQgbWFueSBjb2RlcyB3aWxsIHJlcXVpcmUgYSB0dHlTMCBh cyBjb25zb2xlLi4uIFNvIEkgdGhpbmsgdGhlCj4+Pj4gwqDCoGJsdWV0b290aCBzaG91bGQgYmUg bG9jYXRlZCBhdCB0dHlTMS4uLgo+Pj4KPj4+IMKgTW9zdCBvZiB0aGVtIHdvdWxkIHVzZSAvZGV2 L2NvbnNvbGUgYW55d2F5LiBXaGF0IGRvIHlvdSBoYXZlIGluIG1pbmQ/Cj4+Cj4+IMKgQXMgYSBn ZW5lcmFsIGNvbmZpZ3VyYXRpb24sIC9kZXYvdHR5UzAgd2lsbCBiZSBpbmRpY2F0ZWQgYXMgdGhl IGRlZmF1bHQgY29uc29sZT0KPj4gwqB2YWx1ZS4uLiAoT3RoZXJ3aXNlIG1vcmUgYm9hcmRzIHdp bGwgZmFpbCkKPj4KPj4gwqBPciBtYXliZSB3ZSBjYW4gc3BlY2lmeSB1YXJ0MCBhcyBzZXJpYWww LCBhbmQgbGVhdmUgaXQgYXMgZGlzYWJsZWQ/IChUaHVzIGlmIHRoZQo+PiDCoGRlYnVnIHBvcnQg aXMgbmVlZGVkLCB3ZSBjYW4gZWFzaWx5IGFwcGx5IGEgb3ZlcmxheSB3aXRoICZtbWMwIHN0YXR1 cyBhcyBkaXNhYmxlZCwKPj4gwqBhbmQgJnVhcnQwIHN0YXR1cyBhcyBva2F5KQo+Pgo+PiDCoE9u IGFsbCBteSBBMzMgYm9hcmRzLCB0aGUgb2ZmaWNpYWwgYm9vdDAsIHUtYm9vdCBhbGwgdXNlcyB1 YXJ0MCBhcyBkZWJ1Z2dpbmcgcG9ydC4KPgo+IElmIG5vIHVhcnQgaXMgYXZhaWxhYmxlIChhbmQg YmVpbmcgbXV4ZWQgd2l0aCB0aGUgbW1jIGNvdW50cyBhcyBub3QgYmVpbmcgYXZhaWxhYmxlKQo+ IHRoZW4geW91IHNob3VsZCBub3QgaGF2ZSBhbnkgc2VyaWFsIGFsaWFzIGluIHRoZSBkdHMuCj4K PiBJbiB1LWJvb3QgeW91IG5lZWQgdG8gdXNlOgo+Cj4gIyBDT05GSUdfUkVRVUlSRV9TRVJJQUxf Q09OU09MRSBpcyBub3Qgc2V0Cj4KPiBJbiB0aGUgZGVmY29uZmlnIGluIHRoaXMgY2FzZSwgb3Ro ZXJ3aXNlIHUtYm9vdCBpcyBnb2luZyB0byBiZSB1bmhhcHB5IGFib3V0Cj4gbm90IGhhdmluZyBh IHNlcmlhbCBjb25zb2xlLgoKSXQncyBub3QgdGhlIHByb2JsZW0uCgpDYW4gSSBoYXZlIG5vIHR0 eVMwIGFuZCBqdXN0IG1ha2UgYmx1ZXRvb3RoIHR0eVMxPwoKPgo+IFJlZ2FyZHMsCj4KPiBIYW5z CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsaW51eC1h cm0ta2VybmVsIG1haWxpbmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxpc3RzLmluZnJhZGVhZC5v cmcKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1hcm0t a2VybmVsCg==