From patchwork Sat Nov 5 15:11:11 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Christian Zigotzky X-Patchwork-Id: 691575 Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from lists.ozlabs.org (lists.ozlabs.org [103.22.144.68]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3tB3nK6MvFz9vF6 for ; Sun, 6 Nov 2016 03:19:33 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=xenosoft.de header.i=@xenosoft.de header.b="paQhHy/R"; dkim-atps=neutral Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) by lists.ozlabs.org (Postfix) with ESMTP id 3tB3nK4x1mzDvY9 for ; Sun, 6 Nov 2016 03:19:33 +1100 (AEDT) Authentication-Results: lists.ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=xenosoft.de header.i=@xenosoft.de header.b="paQhHy/R"; dkim-atps=neutral X-Original-To: linuxppc-dev@lists.ozlabs.org Delivered-To: linuxppc-dev@lists.ozlabs.org X-Greylist: delayed 371 seconds by postgrey-1.36 at bilbo; Sun, 06 Nov 2016 03:17:37 AEDT Authentication-Results: lists.ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=xenosoft.de header.i=@xenosoft.de header.b="paQhHy/R"; dkim-atps=neutral Received: from mo6-p00-ob.smtp.rzone.de (mo6-p00-ob.smtp.rzone.de [IPv6:2a01:238:20a:202:5300::1]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3tB3l50FHLzDvQb for ; Sun, 6 Nov 2016 03:17:36 +1100 (AEDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1478362652; l=2991; s=domk; d=xenosoft.de; h=Content-Type:In-Reply-To:MIME-Version:Date:From:To:References: Subject; bh=3IbOK5WicRNLVPdLufuFT+hnePHQEf6599GHcM7Msm8=; b=paQhHy/Rs2mw/OQH3LtMGstuBF+PCxzcA3/o/2/4o3YybvRdOxKp8VPTZO/9/EtpdJ xdtOhMt32tEO3wi5mjm7skjgBLHTHrXijEUlTuPdFY2UOTn9cbT5KNvvIWtwYiVuErY9 zrWIQHzIQkY7xZmDyAD2AND0rFQ0BOKDBIafo= X-RZG-AUTH: :L2QefEenb+UdBJSdRCXu93KJ1bmSGnhMdmO4ZELgFcC9VNGwt8x55myBVvoozZdu/nXQ X-RZG-CLASS-ID: mo00 Received: from [192.168.1.106] (ppp-83-171-152-60.dynamic.mnet-online.de [83.171.152.60]) by smtp.strato.de (RZmta 39.8 DYNA|AUTH) with ESMTPSA id z04e0fsA5GB9bb0 (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA (curve secp521r1 with 521 ECDH bits, eq. 15360 bits RSA)) (Client did not present a certificate); Sat, 5 Nov 2016 17:11:09 +0100 (CET) Subject: Problem with the commit "console: don't prefer first registered if DT specifies stdout-path" References: <86E27593-C6C2-4BBA-995E-95A97BB8CB12@xenosoft.de> <14903BF2-A5B9-430B-B63E-AAEC3FC454F2@xenosoft.de> <398c3cc1-80c5-30 ce-c05a-4280290e54d0@xenosoft.de> <75F756E5-E8B8-4382-9632-15C3DE572A75@xenosoft.de> <23567bc3-f9e9-3a77-3def-6820ce3285df@candw.ms> <5a77bef0-50ac-801e-e4 86-08b524abaf0f@xenosoft.de> <19E9D3FF-6FD3-49D7-8E9B-2660DBBFBD6E@xenosoft.de> <9da8c949-c1a2-3032-a31f-61bafff270f9@candw.ms> <7ea85187-cc4b-b3fb-77c5-57f6c63b3aeb@xenosoft.de> <0f04fb38-a0d3-3fb4-6232-2951e0e9b666@xenosoft.de> <0ff108f9-4112-7f13-2c9d-8881e5416431@xenosoft.de> To: paul.burton@imgtec.com, linuxppc-dev@lists.ozlabs.org From: Christian Zigotzky Message-ID: Date: Sat, 5 Nov 2016 16:11:11 +0100 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <0ff108f9-4112-7f13-2c9d-8881e5416431@xenosoft.de> X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org Sender: "Linuxppc-dev" Hello Paul, After your commit "console: don't prefer first registered if DT specifies stdout-path" [1] we couldn't see any kernel boot messages anymore. I restored the old code and after that I was able to see the messages again. System: Nemo board with P.A. Semi PA6T PowerPC CPU I understand your idea of this patch but unfortunately it doesn't work with our system. There isn't a possibility to modify the DT. We have created a patch for removing your commit but I hope you have a better idea. Cheers, Christian [1] https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=05fd007e46296afb24d15c7d589d535e5a5b9d5c diff -rupN a/drivers/of/base.c b/drivers/of/base.c --- a/drivers/of/base.c 2016-10-29 22:52:02.000000000 +0200 +++ b/drivers/of/base.c 2016-11-04 11:59:07.049568281 +0100 @@ -2077,8 +2077,6 @@ void of_alias_scan(void * (*dt_alloc)(u6 name = of_get_property(of_aliases, "stdout", NULL); if (name) of_stdout = of_find_node_opts_by_path(name, &of_stdout_options); - if (of_stdout) - console_set_by_of(); } if (!of_aliases) diff -rupN a/include/linux/console.h b/include/linux/console.h --- a/include/linux/console.h 2016-10-29 22:52:02.000000000 +0200 +++ b/include/linux/console.h 2016-11-04 12:00:22.808701652 +0100 @@ -173,12 +173,6 @@ static inline void console_sysfs_notify( #endif extern bool console_suspend_enabled; -#ifdef CONFIG_OF -extern void console_set_by_of(void); -#else -static inline void console_set_by_of(void) {} -#endif - /* Suspend and resume console messages over PM events */ extern void suspend_console(void); extern void resume_console(void); diff -rupN a/kernel/printk/printk.c b/kernel/printk/printk.c --- a/kernel/printk/printk.c 2016-10-29 22:52:02.000000000 +0200 +++ b/kernel/printk/printk.c 2016-11-04 12:04:36.083094450 +0100 @@ -253,17 +253,6 @@ static int preferred_console = -1; int console_set_on_cmdline; EXPORT_SYMBOL(console_set_on_cmdline); -#ifdef CONFIG_OF -static bool of_specified_console; - -void console_set_by_of(void) -{ - of_specified_console = true; -} -#else -# define of_specified_console false -#endif - /* Flag: console code may call schedule() */ static int console_may_schedule; @@ -2657,7 +2646,7 @@ void register_console(struct console *ne * didn't select a console we take the first one * that registers here. */ - if (preferred_console < 0 && !of_specified_console) { + if (preferred_console < 0) { if (newcon->index < 0) newcon->index = 0; if (newcon->setup == NULL ||