Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753947Ab3EIKjo (ORCPT ); Thu, 9 May 2013 06:39:44 -0400 Received: from mail-ie0-f177.google.com ([209.85.223.177]:52183 "EHLO mail-ie0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753316Ab3EIKjm (ORCPT ); Thu, 9 May 2013 06:39:42 -0400 MIME-Version: 1.0 In-Reply-To: <20130508141706.GF2703@arm.com> References: <87sj27xxjr.fsf@rustcorp.com.au> <87ip2wr8tp.fsf@rustcorp.com.au> <87txmfo0rl.fsf@rustcorp.com.au> <5189234D.3040208@codeaurora.org> <20130508141706.GF2703@arm.com> From: Grant Likely Date: Thu, 9 May 2013 11:39:22 +0100 X-Google-Sender-Auth: iM8WnLaWwubf3WBh3wU8hmg1r5o Message-ID: Subject: Re: [PATCH 0/2] Early printk support for virtio console devices. To: Catalin Marinas Cc: Peter Maydell , "linaro-kernel@lists.linaro.org" , "patches@linaro.org" , Rusty Russell , Alexander Graf , "linux-kernel@vger.kernel.org List" , Christopher Covington , Amit Shah , "kvmarm@lists.cs.columbia.edu" , "linux-arm-kernel@lists.infradead.org" Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2077 Lines: 39 On Wed, May 8, 2013 at 3:17 PM, Catalin Marinas wrote: > On Tue, May 07, 2013 at 04:58:23PM +0100, Peter Maydell wrote: >> On 7 May 2013 16:52, Christopher Covington wrote: >> This mixes up "information that the user provides to the >> kernel" (ie configuration) with "information that QEMU or >> kvmtool should provide to the kernel" (ie hardware >> description), and would require QEMU/kvmtool to parse >> the user's commandline tool to figure out if they >> needed to override it or edit it (or alternatively, >> would require the user to know internal details of >> QEMU/kvmtool's address map for the guest). I think it >> would be nicer to keep them separate. > > I'm not sure it's worth it, the earlyprintk is meant for debugging the > early kernel booting code, not for general use (you have the normal > console for this). Someone doing platform port should know the address > of the uart and pass the right information on the kernel command line to > help debug early booting issues. > > But if someone comes up with some bindings, I won't reject the patch. Nicolas and I have tossed back and forth the idea of having a trivial binding in the DT which specifies a couple of physical addresses and a mask; one write register, one status register, and the mask to say which bit to care about in the status. Basically the bare minimum to say "here is something that will output to an already set up serial device" without any reference to what it actually is. Something like that would theoretically work with any device that implements or emulates a UART write register. The thinking is that if it is trivial to parse then it becomes more useful as an early printk target and usable by the zImage wrapper (arm32 bit of course). Neither of us ever followed up on it though. g. -- 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/