Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752295AbbFCAQr (ORCPT ); Tue, 2 Jun 2015 20:16:47 -0400 Received: from mga01.intel.com ([192.55.52.88]:3982 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751815AbbFCAQj convert rfc822-to-8bit (ORCPT ); Tue, 2 Jun 2015 20:16:39 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,543,1427785200"; d="scan'208";a="739786102" From: "Anderson, Stuart R" To: Bin Gao , Thomas Gleixner CC: Ingo Molnar , Ingo Molnar , "H. Peter Anvin" , Greg Kroah-Hartman , One Thousand Gnomes , Peter Hurley , Jiri Slaby , Borislav Petkov , "linux-kernel@vger.kernel.org" Subject: RE: [PATCH v5 2/2] arch/x86: remove pci uart early console from early_prink.c Thread-Topic: [PATCH v5 2/2] arch/x86: remove pci uart early console from early_prink.c Thread-Index: AQHQnYPY/DgnhIrWHk2EgE5xK+jJSJ2Z6Hcg Date: Wed, 3 Jun 2015 00:16:36 +0000 Message-ID: <518B79536261F8469CEA9CC56FA8F74503353E1D@FMSMSX105.amr.corp.intel.com> References: <20150529184123.GB13090@worksta> <20150602115433.GA20697@gmail.com> <20150602192036.GA52512@worksta> <20150602224305.GB64269@worksta> In-Reply-To: <20150602224305.GB64269@worksta> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.1.200.108] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1879 Lines: 34 Bin, et al, What we are losing here is the ability to specify a UART by its bus address instead of just supplying the memory or io address. There are some cases where this is useful, though I admit it is probably not going to be widely used. I have seen a platform where the location of the UART moves depending on the firmware version, but the bus address (B:D.F) did not change. There are also some platforms where you do not know the address until you boot the OS and can use the UART to login and find the address of the UART (oops. Chicken and egg problem). Also, I was going to soon send a patch to allow "pciserial32" for the case where the UART registers are 32-bit aligned instead of 8-bit aligned. Stuart -----Original Message----- From: Bin Gao [mailto:bin.gao@linux.intel.com] Sent: Tuesday, June 02, 2015 3:43 PM To: Thomas Gleixner Cc: Ingo Molnar; Ingo Molnar; H. Peter Anvin; Greg Kroah-Hartman; One Thousand Gnomes; Peter Hurley; Jiri Slaby; Borislav Petkov; Anderson, Stuart R; linux-kernel@vger.kernel.org Subject: Re: [PATCH v5 2/2] arch/x86: remove pci uart early console from early_prink.c On Tue, Jun 02, 2015 at 11:07:39PM +0200, Thomas Gleixner wrote: > What about the memory mapped uarts which have been source of trouble > in the past? > > Thanks, > > tglx Not sure which specific early console you are referring to. Currently we have serial, ttyS, vga, dbgp, xen, efi and pciserial in arch/x86/kernel/early_printk.c, and only pciserial is memory mapped (but it's being removed). And this patch only touches pciserial/serial parts so ideally shouldn't have impact on other early consoles. -Bin -- 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/