From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Wed, 21 Aug 2024 14:40:13 +0200 Received: from metis.whiteo.stw.pengutronix.de ([2a0a:edc0:2:b01:1d::104]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1sgkdJ-00Buud-0q for lore@lore.pengutronix.de; Wed, 21 Aug 2024 14:40:13 +0200 Received: from localhost ([127.0.0.1] helo=metis.whiteo.stw.pengutronix.de) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1sgkdI-0004Ap-Lh; Wed, 21 Aug 2024 14:40:12 +0200 Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.whiteo.stw.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1sgkdD-0004AS-UP; Wed, 21 Aug 2024 14:40:07 +0200 Received: from [2a0a:edc0:2:b01:1d::c5] (helo=pty.whiteo.stw.pengutronix.de) by drehscheibe.grey.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1sgkdD-0020rO-H5; Wed, 21 Aug 2024 14:40:07 +0200 Received: from rsc by pty.whiteo.stw.pengutronix.de with local (Exim 4.96) (envelope-from ) id 1sgkdD-00FWNh-1K; Wed, 21 Aug 2024 14:40:07 +0200 Date: Wed, 21 Aug 2024 14:40:07 +0200 From: Robert Schwebel To: Ahmad Fatoum Message-ID: References: <20240821095133.1471779-1-o.rempel@pengutronix.de> <20240821095133.1471779-6-o.rempel@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-Accept-Language: de,en X-Accept-Content-Type: text/plain Subject: Re: [DistroKit] [PATCH v1 5/9] mips/mipsel: barebox: use CONSOLE_ACTIVATE_NONE X-BeenThere: distrokit@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: DistroKit Mailinglist List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Oleksij Rempel , distrokit@pengutronix.de Sender: "DistroKit" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: distrokit-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false Hi Ahmad, On Wed, Aug 21, 2024 at 12:07:00PM +0200, Ahmad Fatoum wrote: > On 21.08.24 11:51, Oleksij Rempel wrote: > > For system booting with device tree, the default console should be > > specifie via /chosen/stdout-path and barebox should not unconditionally > > enable any consoles. This avoids issues with probe order > > (CONFIG_CONSOLE_ACTIVATE_FIRST) and crosstalk with serial devices that > > don't expect console output (CONFIG_CONSOLE_ACTIVATE_ALL). > > Did you test this? > > arch/mips/dts/qemu-malta.dts doesn't have a chosen /chosen/stdout-path, > so this change would most likely render the console completely silent. > > In general, we will want to use the new CONFIG_CONSOLE_ACTIVATE_ALL_FALLBACK > option[1]. It would enable all consoles as a fallback if none was activated > explicitly. It hasn't hit a barebox release yet though. > > For Malta, barebox should probably get a /chosen/stdout-path if there's indeed > no board code or environment activating any consoles. > > [1]: https://lore.barebox.org/barebox/20240730083451.2203330-1-a.fatoum@pengutronix.de/ Could you quickly add a patch on top of this series on top of next? Genereally I don't care much about MIPS any more, and we even quickly discussed about removing it from DistroKit in the near future. Currently this is stopping me from merging next into master, and this is already the case for several months - not because anything broke, but because the tooling became more picky. Not satisfying. rsc -- Pengutronix e.K. | Dipl.-Ing. Robert Schwebel | Steuerwalder Str. 21 | https://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 |