Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755046AbbG0XUb (ORCPT ); Mon, 27 Jul 2015 19:20:31 -0400 Received: from mga01.intel.com ([192.55.52.88]:43644 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754629AbbG0XUa (ORCPT ); Mon, 27 Jul 2015 19:20:30 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.15,558,1432623600"; d="scan'208";a="614073140" Date: Mon, 27 Jul 2015 16:33:12 -0700 From: Bin Gao To: Greg Kroah-Hartman Cc: One Thousand Gnomes , Peter Hurley , Jiri Slaby , Paul Bolle , linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v6 1/2] serial_core: add pci uart early console support Message-ID: <20150727233312.GA188620@worksta> References: <20150529183839.GA13090@worksta> <20150608181711.GA181146@worksta> <20150723222127.GA9973@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150723222127.GA9973@kroah.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1418 Lines: 44 On Thu, Jul 23, 2015 at 03:21:27PM -0700, Greg Kroah-Hartman wrote: > > +config PCI_EARLY > > + bool "Early PCI access" > > + depends on PCI > > + default n > > Default is always 'n' so this isn't needed here. Will fix this. > > diff --git a/drivers/tty/serial/serial_core.c b/drivers/tty/serial/serial_core.c > > index 0b7bb12..5b21999 100644 > > --- a/drivers/tty/serial/serial_core.c > > +++ b/drivers/tty/serial/serial_core.c > > @@ -34,10 +34,15 @@ > > #include > > #include > > #include > > +#include > > > > #include > > #include > > > > +#ifdef CONFIG_PCI_EARLY > > +#include > > +#endif > > You shouldn't need an #ifdef here, the .h file should handle it. Why > doesn't pci.h always include this if it is present? Only x86 has asm/pci-direct.h. Compiling on any other archs will fail if we don't put CONFIG_PCI_EARLY here. That's way I added a new config option CONFIG_PCI_EARLY to drivers/pci/Kconfig and let an arch to select it. Do you want me to move "#include " to linux/pci.h? > > thanks, > > greg k-h -- 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/