From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCHv2 05/15] ARM: mvebu: add stdout-path to all armada-*.dts Date: Mon, 23 Feb 2015 10:50:26 -0600 Message-ID: References: <1424451874-25375-1-git-send-email-thomas.petazzoni@free-electrons.com> <1424451874-25375-6-git-send-email-thomas.petazzoni@free-electrons.com> <54EB4E9B.6010001@free-electrons.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <54EB4E9B.6010001-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Gregory CLEMENT Cc: Thomas Petazzoni , Jason Cooper , Andrew Lunn , Sebastian Hesselbarth , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Mike Turquette , Stephen Boyd , Linus Walleij , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "linux-gpio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Tawfik Bayouk , Nadav Haklai , Lior Amsalem , Ezequiel Garcia , Maxime Ripard List-Id: linux-gpio@vger.kernel.org On Mon, Feb 23, 2015 at 10:00 AM, Gregory CLEMENT wrote: > Hi Thomas, > > On 20/02/2015 18:04, Thomas Petazzoni wrote: >> This commit adds the stdout-path property in /chosen for all Armada >> boards that were not yet carrying this property. > > I though the main motivation for using the stdout-path property was > for removing the earlyprintk in the command line. Arnd told that there > should be a mean to use stdout-path instead of earlyprintk. > > So what about going further and removing "earlyprintk" in the same time? Not exactly, stdout-path allows for removing "console" from the command line. earlyprintk is a debug/developer option, so it should not be in a default command line IMO. So bootargs should be removed entirely. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: robherring2@gmail.com (Rob Herring) Date: Mon, 23 Feb 2015 10:50:26 -0600 Subject: [PATCHv2 05/15] ARM: mvebu: add stdout-path to all armada-*.dts In-Reply-To: <54EB4E9B.6010001@free-electrons.com> References: <1424451874-25375-1-git-send-email-thomas.petazzoni@free-electrons.com> <1424451874-25375-6-git-send-email-thomas.petazzoni@free-electrons.com> <54EB4E9B.6010001@free-electrons.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, Feb 23, 2015 at 10:00 AM, Gregory CLEMENT wrote: > Hi Thomas, > > On 20/02/2015 18:04, Thomas Petazzoni wrote: >> This commit adds the stdout-path property in /chosen for all Armada >> boards that were not yet carrying this property. > > I though the main motivation for using the stdout-path property was > for removing the earlyprintk in the command line. Arnd told that there > should be a mean to use stdout-path instead of earlyprintk. > > So what about going further and removing "earlyprintk" in the same time? Not exactly, stdout-path allows for removing "console" from the command line. earlyprintk is a debug/developer option, so it should not be in a default command line IMO. So bootargs should be removed entirely. Rob