From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932243AbdKAMFq (ORCPT ); Wed, 1 Nov 2017 08:05:46 -0400 Received: from mout.kundenserver.de ([212.227.126.134]:51644 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751738AbdKAMFo (ORCPT ); Wed, 1 Nov 2017 08:05:44 -0400 Subject: Re: [GIT PULL 1/2] bcm2835-dt-fixes-2017-10-06 To: Peter Robinson , Matt Hart Cc: Marcel Holtmann , linux-kernel@vger.kernel.org, bcm-kernel-feedback-list@broadcom.com, linux-rpi-kernel@lists.infradead.org, Loic Poulain , linux-arm-kernel@lists.infradead.org, Eric Anholt References: <20171006220504.7776-1-eric@anholt.net> From: Stefan Wahren Message-ID: <910a98ca-57f6-baca-73ac-f6c8c1ff04db@i2se.com> Date: Wed, 1 Nov 2017 13:05:21 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Content-Language: de-DE X-Provags-ID: V03:K0:b4R66cSrbPVJ2c/oSHrGrUXVcahxqWEiHciF9xtvDRZeghfFY+r uebRgqecdsDsAYPLEsNeiaXTEtsfHMX24jzZ/ZGBz9UtEh2d4CKdPKld7hsNVfPV8/8BsMg tDgS2pOnPzLJpqtunTHvIp1kwXmVHbnKN02a2eG97HeU7qhxOB4yPAIWySqHdJrz9wnZm2P 6KuKcaN7+Q/76FVsWV3jA== X-UI-Out-Filterresults: notjunk:1;V01:K0:MPSH3ySzjjE=:+cvWBUdQY33HVQc+VbhkR+ Pag2IHGO3nTa8arftgMMmA8eN4ywv+9kuS+2MKBFY+0U+GMaIkfXym0EIkg2mSTJ/F1tiI6We TCVZRFD2XAH0XEsZQEzdKeVncjoBujXNZ6XBts8+B7hE8iFMh439Uri3Kys4sErmuNm6bO+xN GnX5/cRmMoKBvS2s6/2ty2C2jm5nU+IMK8omBBc9T5CONpGrZrtNLnv3CTeNoL5qaYbo8o21l ndk3lqNMPErhMnaP2YM5Bo1ORrqMMOeakz+oDH/saDzQHuban/aqJ7UwilqGNuR99LEBXeIDM ilk74e7OjYPm5KCBrBT+DuRf10JEoHtee0z6eOEpuQEhrg3RADV9PamCkv5P62D2fiFhf8pRA tDNc81jqLYPrVgIEXEM/nnZUR+HGcpdyw2YnFM6AOJUX3D6G1RvUAQU/J9yB+bfPNXBWT2ecV jWGq3I/hptCtsmdPyXbdMtoi5jgwXsd89aN1D9n4d8wqnpUA79YbP2LwFuGl9QeWvz1YS/M75 r4/VsExL0yaVlY9XTgT3CIAXNpBHJ/KGdnnVduIR3YO6SAZTe1Cx4SNwSZ86Q+5hPAhR/AZ/S DexMjCWURf/f15DIglODat9T317iEr/3VSwRNL0di2/jtzfAFTqlm20d2/qjnJ1SGTMISQRXk qMP5UdBdu6dpQkR5VsP4/qsNVhvy2J807ZrKam6cmRQMzt9m2oxVKUjL3cnS0coOJNPBUR1le 1DX3P+pP1u2ekZLFwpt/gxif/bUrVPu5fGJGBzWYnMoKCn4EAWLwti8U/J0= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Matt, hi Peter, Am 01.11.2017 um 10:37 schrieb Peter Robinson: > On Tue, Oct 31, 2017 at 5:00 PM, Matt Hart wrote: >> On 6 October 2017 at 23:14, Florian Fainelli wrote: >>> On 10/06/2017 03:05 PM, Eric Anholt wrote: >>>> Hi Florian. Here's a patch that's gone through a couple of revisions >>>> on the list, that seriously fixes up the default serial behavior -- >>>> previously, without the right config.txt/cmdline bits, you'd often end >>>> up with a hang with no output before booting completed. It would be >>>> great to get it into 4.14. >>>> >>>> The following changes since commit 2bd6bf03f4c1c59381d62c61d03f6cc3fe71f66e: >>>> >>>> Linux 4.14-rc1 (2017-09-16 15:47:51 -0700) >>>> >>>> are available in the git repository at: >>>> >>>> git://github.com/anholt/linux tags/bcm2835-dt-fixes-2017-10-06 >>>> >>>> for you to fetch changes up to f08f58a2bf68900a84e782b8c7ad701c0654173c: >>>> >>>> ARM: dts: bcm283x: Fix console path on RPi3 (2017-10-06 13:04:56 -0700) >>>> >>>> ---------------------------------------------------------------- >>>> This pull request brings in a fix for default serial console setup on >>>> RPi3, so it now comes up with no config.txt/cmdline.txt settings in >>>> the firmware. >>>> >>>> ---------------------------------------------------------------- >>> Merged, thanks Eric >>> >>> -- >>> Florian >> Hi all, >> >> I realise this patch has been in mainline for a more than a week, but >> I've only recently had time to bisect broken rpi-3 boots in kernelci >> and found f08f58a2bf68900a84e782b8c7ad701c0654173c as the cause. >> >> Kernelci boots the rpi3 using uboot, with console=ttyS0 in the command >> line which now fails: >> https://kernelci.org/boot/id/59f787f059b5147794ef1dcf/ >> >> Is there an alias I can use for both before and after this patch? >> >> 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? Which u-boot version do you use? Best regards Stefan