Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752696AbbEHKMs (ORCPT ); Fri, 8 May 2015 06:12:48 -0400 Received: from mail-qg0-f50.google.com ([209.85.192.50]:33337 "EHLO mail-qg0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751113AbbEHKMp (ORCPT ); Fri, 8 May 2015 06:12:45 -0400 Message-ID: <554C8C15.4070404@hurleysoftware.com> Date: Fri, 08 May 2015 06:12:37 -0400 From: Peter Hurley User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Sascha Hauer CC: "Maciej W. Rozycki" , kernel@pengutronix.de, Robert Schwebel , linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org Subject: Re: earlycon: no match? References: <20150503211054.GR15254@pengutronix.de> <55477BC1.6000805@hurleysoftware.com> <20150504194229.GR15254@pengutronix.de> <5547D4DB.8060300@hurleysoftware.com> <20150504205241.GT15254@pengutronix.de> <5547E447.9020604@hurleysoftware.com> <554B9F43.6000702@hurleysoftware.com> <20150508063433.GA6325@pengutronix.de> In-Reply-To: <20150508063433.GA6325@pengutronix.de> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1929 Lines: 49 On 05/08/2015 02:34 AM, Sascha Hauer wrote: > On Thu, May 07, 2015 at 01:22:11PM -0400, Peter Hurley wrote: >> On 05/07/2015 01:09 PM, Maciej W. Rozycki wrote: >>> On Mon, 4 May 2015, Peter Hurley wrote: >>> >>>>>> Since 2007, 'console=' is a early param synonym for 'earlycon='; IOW, >>>>>> the message is new but not the behavior. >>>>> >>>>> "console=" had nothing to do with early param before. >>>> >>>> *Yes, it has* since the commit I referenced in the previous email and >>>> the email before that when I noted that 'console=' and 'earlycon=' are >>>> synonyms if CONFIG_SERIAL_EARLYCON=y, and it has been that way since >>>> 2007. >>>> >>>> The only thing that has changed is that I added a diagnostic; _to repeat_, >>>> the earlycon matching code has always run in this case, only the >>>> diagnostic is new. >>> >>> What's the point of having two parameters as synonyms whose syntax is not >>> compatible to each other in the general case? I'd expect the following >>> cases to be handled: >>> >>> 1. Regular console only (no early console requested) => `console=foo...'. >>> >>> 2. Both early and regular console => `earlycon=blah... console=foo...'. >>> >>> 3. Early console handing over to regular console => `earlycon=blah...'. >> >> 4. Early console only => `earlycon=blah...' >> >> How to distinguish between 3 & 4? > > Given that the aliasing only makes sense for console=uart, and > console=uart8250, we can do the following. Not exactly the nicest > code, but only slighty more ugly than what we have now in > do_early_param() I already sent Greg a patch to remove the egregious and unsightly false positive warning. Thanks anyway. Regards, Peter Hurley -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/