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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CALeDE9M9QQtJOhOjNdsAmpWkAFG9DvLOOiyPyqbbp00cuN+BMQ@mail.gmail.com>
Date:   Fri, 17 Nov 2017 12:16:22 +0000
From:   Peter Robinson <pbrobinson@...il.com>
To:     Eric Anholt <eric@...olt.net>
Cc:     Marcel Holtmann <marcel@...tmann.org>,
        Stefan Wahren <stefan.wahren@...e.com>,
        Matt Hart <matt@...tface.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        bcm-kernel-feedback-list@...adcom.com,
        linux-rpi-kernel@...ts.infradead.org,
        Loic Poulain <loic.poulain@...il.com>,
        linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [GIT PULL 1/2] bcm2835-dt-fixes-2017-10-06

>>>>>>> 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

We already have that one included in the Fedora kernel, I wonder what
else I'm doing wrong.