From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by mx1.pokylinux.org (Postfix) with ESMTP id 1B75F4C800A9 for ; Mon, 11 Jul 2011 11:56:22 -0500 (CDT) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga102.jf.intel.com with ESMTP; 11 Jul 2011 09:56:21 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.65,516,1304319600"; d="scan'208";a="24518239" Received: from unknown (HELO [10.255.13.208]) ([10.255.13.208]) by orsmga002.jf.intel.com with ESMTP; 11 Jul 2011 09:56:19 -0700 From: Tom Zanussi To: Darren Hart In-Reply-To: <4E1B2982.4020102@linux.intel.com> References: <8b824f53292e21ccb9de938b2c115adf0a9b1aa0.1310314387.git.tom.zanussi@intel.com> <4E1B2982.4020102@linux.intel.com> Date: Mon, 11 Jul 2011 11:56:17 -0500 Message-ID: <1310403377.2313.36.camel@elmorro> Mime-Version: 1.0 X-Mailer: Evolution 2.28.3 Cc: "yocto@yoctoproject.org" Subject: Re: [PATCH 1/3][KERNEL] meta: add eg20t feature X-BeenThere: yocto@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Discussion of all things Yocto List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 11 Jul 2011 16:56:22 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Mon, 2011-07-11 at 09:49 -0700, Darren Hart wrote: > That is quite the assortment of config bits - are all of these necessary > to explicitly support the eg20t? I'd like to see minimal config > fragments both for simplicity as well as avoiding unnecessary kernel > tool warnings about redefinitions. > Yes, I agree, I would also like to see these split up, but I think that would be better done as part of our ongoing general option cleanup. The particular reason for doing this now is to simply re-use the existing crownbay bits as-is for a new BSP. Once I have time and can get back to the option cleanup I started with the romley options, I'll clean this up too, but at the moment have bigger fish to fry... Tom > -- > Darren > > On 07/10/2011 09:26 AM, tom.zanussi@intel.com wrote: > > From: Tom Zanussi > > > > Option group for Intel Platform Controller Hub EG20T (Topcliff). > > > > Signed-off-by: Tom Zanussi > > --- > > meta/cfg/kernel-cache/features/eg20t/eg20t.cfg | 39 ++++++++++++++++++++++++ > > meta/cfg/kernel-cache/features/eg20t/eg20t.scc | 1 + > > 2 files changed, 40 insertions(+), 0 deletions(-) > > create mode 100644 meta/cfg/kernel-cache/features/eg20t/eg20t.cfg > > create mode 100644 meta/cfg/kernel-cache/features/eg20t/eg20t.scc > > > > diff --git a/meta/cfg/kernel-cache/features/eg20t/eg20t.cfg b/meta/cfg/kernel-cache/features/eg20t/eg20t.cfg > > new file mode 100644 > > index 0000000..225581c > > --- /dev/null > > +++ b/meta/cfg/kernel-cache/features/eg20t/eg20t.cfg > > @@ -0,0 +1,39 @@ > > +# Hardware support for the Platform Controller Hub EG20T > > + > > +CONFIG_PCH_DMA=y > > +CONFIG_PCH_UART_DMA=y > > +CONFIG_SERIAL_8250_PCI=y > > +CONFIG_SERIAL_8250_PCH=y > > +CONFIG_SERIAL_8250_PCH_DMA=y > > + > > +CONFIG_SERIAL_PCH_UART=y > > +CONFIG_SERIAL_CORE=y > > + > > +CONFIG_SPI=y > > +CONFIG_SPI_BITBANG=y > > +CONFIG_SPI_GPIO=y > > +CONFIG_SPI_TOPCLIFF_PCH=y > > + > > +CONFIG_MISC_DEVICES=y > > +CONFIG_PCH_PHUB=y > > + > > +CONFIG_NETDEVICES=y > > +CONFIG_NETDEV_1000=y > > +CONFIG_PCH_GBE=y > > + > > +CONFIG_PCH_USBDEV=y > > +CONFIG_USB_GADGET=y > > +CONFIG_USB_GADGET_EG20T=y > > +CONFIG_USB_GADGET_DUALSPEED=y > > +CONFIG_USB_GADGET_SELECTED=y > > + > > +CONFIG_I2C_EG20T=y > > + > > +CONFIG_GPIOLIB=y > > +CONFIG_GPIO_PCH=y > > + > > +CONFIG_PCH_IEEE1588=y > > + > > +CONFIG_CAN=y > > +CONFIG_CAN_DEV=y > > +CONFIG_PCH_CAN=y > > diff --git a/meta/cfg/kernel-cache/features/eg20t/eg20t.scc b/meta/cfg/kernel-cache/features/eg20t/eg20t.scc > > new file mode 100644 > > index 0000000..d690469 > > --- /dev/null > > +++ b/meta/cfg/kernel-cache/features/eg20t/eg20t.scc > > @@ -0,0 +1 @@ > > +kconf hardware eg20t.cfg >