Re: [meta-intel] [Patch v2 00/11] machinesetuptool commits

2014-10-10 Thread Kamble, Nitin A
On 10/10/14, 2:18 PM, "Darren Hart" wrote: >On 10/10/14, 11:46, "Kamble, Nitin A" wrote: > >> >>On 10/10/14, 9:40 AM, "Darren Hart" wrote: >> >>> >>>Another concern is the statement below about not allowing for a default >>>configuration. Like other recent changes to meta-intel, this should b

Re: [meta-intel] [PATCH 1/3] crownbay-noemgd: disable the gma500_gfx module

2014-10-10 Thread Tom Zanussi
On Fri, 2014-10-10 at 14:14 -0700, Darren Hart wrote: > On 10/10/14, 12:00, "Tom Zanussi" wrote: > > >On Fri, 2014-10-10 at 12:46 -0500, Kamble, Nitin A wrote: > >> > >> On 10/10/14, 10:31 AM, "Zanussi, Tom" wrote: > >> > >> >On Fri, 2014-10-10 at 12:25 -0500, Kamble, Nitin A wrote: > >> >> >

Re: [meta-intel] [Patch v2 00/11] machinesetuptool commits

2014-10-10 Thread Burton, Ross
On 10 October 2014 22:18, Darren Hart wrote: > The primary issue with eliminating the defaults from intel-corei7-64 and > providing the default via machinesetuptool, is we then make > machinesetuptool a requirement for that BSP for basic functionality. That > isn't acceptable in my opinion. Agree

Re: [meta-intel] [Patch v2 00/11] machinesetuptool commits

2014-10-10 Thread Darren Hart
On 10/10/14, 11:46, "Kamble, Nitin A" wrote: > >On 10/10/14, 9:40 AM, "Darren Hart" wrote: > >> >>Another concern is the statement below about not allowing for a default >>configuration. Like other recent changes to meta-intel, this should be an >>opt-in thing, and as such, the BSPs need to be a

Re: [meta-intel] [PATCH 1/3] crownbay-noemgd: disable the gma500_gfx module

2014-10-10 Thread Darren Hart
On 10/10/14, 12:00, "Tom Zanussi" wrote: >On Fri, 2014-10-10 at 12:46 -0500, Kamble, Nitin A wrote: >> >> On 10/10/14, 10:31 AM, "Zanussi, Tom" wrote: >> >> >On Fri, 2014-10-10 at 12:25 -0500, Kamble, Nitin A wrote: >> >> >> >> On 10/10/14, 6:37 AM, "Zanussi, Tom" wrote: >> >> >> >> >On Thu

Re: [meta-intel] [PATCH 1/3] crownbay-noemgd: disable the gma500_gfx module

2014-10-10 Thread Darren Hart
On 10/10/14, 11:48, "Kamble, Nitin A" wrote: > > >On 10/10/14, 1:34 PM, "Darren Hart" wrote: > >>On 10/10/14, 10:46, "Kamble, Nitin A" wrote: >> >>> >>> >>>On 10/10/14, 10:31 AM, "Zanussi, Tom" wrote: >>> On Fri, 2014-10-10 at 12:25 -0500, Kamble, Nitin A wrote: > > On 10/10/14, 6

Re: [meta-intel] [PATCH 1/3] crownbay-noemgd: disable the gma500_gfx module

2014-10-10 Thread Tom Zanussi
On Fri, 2014-10-10 at 12:46 -0500, Kamble, Nitin A wrote: > > On 10/10/14, 10:31 AM, "Zanussi, Tom" wrote: > > >On Fri, 2014-10-10 at 12:25 -0500, Kamble, Nitin A wrote: > >> > >> On 10/10/14, 6:37 AM, "Zanussi, Tom" wrote: > >> > >> >On Thu, 2014-10-09 at 15:11 -0700, nitin.a.kam...@intel.co

Re: [meta-intel] [PATCH 1/3] crownbay-noemgd: disable the gma500_gfx module

2014-10-10 Thread Kamble, Nitin A
On 10/10/14, 1:34 PM, "Darren Hart" wrote: >On 10/10/14, 10:46, "Kamble, Nitin A" wrote: > >> >> >>On 10/10/14, 10:31 AM, "Zanussi, Tom" wrote: >> >>>On Fri, 2014-10-10 at 12:25 -0500, Kamble, Nitin A wrote: On 10/10/14, 6:37 AM, "Zanussi, Tom" wrote: >On Thu, 2014-10-0

Re: [meta-intel] [Patch v2 00/11] machinesetuptool commits

2014-10-10 Thread Kamble, Nitin A
On 10/10/14, 9:40 AM, "Darren Hart" wrote: >Hi Nitin, > >This description helps to identify _what_ the tool is intended to do, but >it still does not address the _how_, which is important context to have >when reviewing the patches. There was also no mention of the sources to >the tool itself,

Re: [meta-intel] [PATCH 1/3] crownbay-noemgd: disable the gma500_gfx module

2014-10-10 Thread Darren Hart
On 10/10/14, 10:46, "Kamble, Nitin A" wrote: > > >On 10/10/14, 10:31 AM, "Zanussi, Tom" wrote: > >>On Fri, 2014-10-10 at 12:25 -0500, Kamble, Nitin A wrote: >>> >>> On 10/10/14, 6:37 AM, "Zanussi, Tom" wrote: >>> >>> >On Thu, 2014-10-09 at 15:11 -0700, nitin.a.kam...@intel.com wrote: >>> >> F

Re: [meta-intel] [Patch v2 02/11] machinesetuptool: add nuc machine config to the intel-corei7-64 BSP

2014-10-10 Thread Darren Hart
On 10/8/14, 5:32, "nitin.a.kam...@intel.com" wrote: >From: Nitin A Kamble > >Add machine configuration for the NUC platform in the intel-corei7-64 BSP >image. Hi Nitin, A few comments inline below... > >Signed-off-by: Nitin A Kamble >--- > .../machinesetuptool/intel-corei7-64/nuc-alsa.conf

Re: [meta-intel] [PATCH 1/3] crownbay-noemgd: disable the gma500_gfx module

2014-10-10 Thread Kamble, Nitin A
On 10/10/14, 10:31 AM, "Zanussi, Tom" wrote: >On Fri, 2014-10-10 at 12:25 -0500, Kamble, Nitin A wrote: >> >> On 10/10/14, 6:37 AM, "Zanussi, Tom" wrote: >> >> >On Thu, 2014-10-09 at 15:11 -0700, nitin.a.kam...@intel.com wrote: >> >> From: Nitin A Kamble >> >> >> >> The gmx500 graphics dri

Re: [meta-intel] [PATCH 1/3] crownbay-noemgd: disable the gma500_gfx module

2014-10-10 Thread Tom Zanussi
On Fri, 2014-10-10 at 12:25 -0500, Kamble, Nitin A wrote: > > On 10/10/14, 6:37 AM, "Zanussi, Tom" wrote: > > >On Thu, 2014-10-09 at 15:11 -0700, nitin.a.kam...@intel.com wrote: > >> From: Nitin A Kamble > >> > >> The gmx500 graphics driver does not work on this BSP, but it takes > >> the owne

Re: [meta-intel] [PATCH 1/3] crownbay-noemgd: disable the gma500_gfx module

2014-10-10 Thread Kamble, Nitin A
On 10/10/14, 6:37 AM, "Zanussi, Tom" wrote: >On Thu, 2014-10-09 at 15:11 -0700, nitin.a.kam...@intel.com wrote: >> From: Nitin A Kamble >> >> The gmx500 graphics driver does not work on this BSP, but it takes >> the ownership of the graphics hardware at boot time, blocking other >> drivers fr

Re: [meta-intel] [Patch v2 01/11] machinesetuptool: a new recipe for setup of a machine

2014-10-10 Thread Darren Hart
On 10/8/14, 5:32, "nitin.a.kam...@intel.com" wrote: >From: Nitin A Kamble > >a new recipe which recipe provides the ability to setup a BSP image for a >specific machine or platform at the boot time. The base recipe does not >provide any machine configuration files, those are provided by individu

Re: [meta-intel] [Patch v2 00/11] machinesetuptool commits

2014-10-10 Thread Darren Hart
Hi Nitin, This description helps to identify _what_ the tool is intended to do, but it still does not address the _how_, which is important context to have when reviewing the patches. There was also no mention of the sources to the tool itself, also important for context. One gap below is Gummibo

Re: [meta-intel] [PATCH 1/3] crownbay-noemgd: disable the gma500_gfx module

2014-10-10 Thread Tom Zanussi
On Thu, 2014-10-09 at 15:11 -0700, nitin.a.kam...@intel.com wrote: > From: Nitin A Kamble > > The gmx500 graphics driver does not work on this BSP, but it takes > the ownership of the graphics hardware at boot time, blocking other > drivers from using the graphics hardware. > > Fix the issue by