Re: [GIT PULL 1/2] bcm2835-dt-fixes-2017-10-06

From: Eric Anholt <eric@anholt.net>
To: Peter Robinson <pbrobinson@gmail.com>,
	Marcel Holtmann <marcel@holtmann.org>
Cc: Stefan Wahren <stefan.wahren@i2se.com>,
	Matt Hart <matt@mattface.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	bcm-kernel-feedback-list@broadcom.com,
	linux-rpi-kernel@lists.infradead.org,
	Loic Poulain <loic.poulain@gmail.com>,
	linux-arm-kernel <linux-arm-kernel@lists.infradead.org>
Subject: Re: [GIT PULL 1/2] bcm2835-dt-fixes-2017-10-06
Date: Thu, 16 Nov 2017 12:21:39 -0800	[thread overview]
Message-ID: <87k1yqq8j0.fsf@anholt.net> (raw)
In-Reply-To: <CALeDE9OySr2LpcedpTHnz89aFYAk2CknKGHeVx+4ecFCemYZPw@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1427 bytes --]

Peter Robinson <pbrobinson@gmail.com> writes:

> Hi Marcel,
>
> Sorry for the late response.
>
>>>>>> Mainline boots fine if I use console=ttyS1 or drop the console arg
>>>>>> altogether, but as we don't currently support different boot args per
>>>>>> tree, and we don't have the manpower to track this patch in all the
>>>>>> trees we test, we will have to switch to only boot testing mainline
>>>>>> from now on.
>>>>> Yes, I've seen the same in Fedora with that patch.
>>>>>
>>>>>
>>>> could you please describe the issue more in detail.
>>>>
>>>> Does u-boot or Linux hang? Does u-boot select the wrong UART? Is there a
>>>> pinmux conflict?
>>>
>>> Basically on the RPi3 you use to get ttyAMA0 and ttyS0 but now the
>>> later is ttyS1
>>
>> which UART is which now. It should expose the Bluetooth UART correctly now. And also more importantly usable. The only missing piece with 4.14 would be the DT assigned for serdev. While 4.14 would still require a call to btattach, starting with 4.15 that should no longer be needed.
>
> Looks to be ttyAMA0
>
> [    4.480469] 3f201000.serial: ttyAMA0 at MMIO 0x3f201000 (irq = 72,
> base_baud = 0) is a PL011 rev2
> [    4.497401] serial serial0: tty port ttyAMA0 registered
>
> With 4.14 I get the following, the address doesn't look right, do you
> have a link to the serdev patch headed to 4.15?

https://github.com/anholt/linux/commit/fd3372db18d3d44ae4579243a8eacb5247d8c03a

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]