All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
@ 2012-07-17  5:57 nitin.a.kamble
  2012-07-17  5:57 ` [PATCH 1/1] meta: crownbay.scc change emgd branch name nitin.a.kamble
  2012-07-18 14:13 ` [PATCH 0/1] meta: update crownbay.scc for newer emgd driver Tom Zanussi
  0 siblings, 2 replies; 10+ messages in thread
From: nitin.a.kamble @ 2012-07-17  5:57 UTC (permalink / raw)
  To: yocto, tom.zanussi, bruce.ashfield

From: Nitin A Kamble <nitin.a.kamble@intel.com>

This commit changes emgd kernel driver for crownbay bsp to the emgd-1.14.

Thanks,
Nitin

The following changes since commit c2b5ee363bf2612932301488f3ac0644ff70fc7c:

  meta: disable CONFIG_MTD_NAND_VERIFY_WRITE (2012-07-14 00:21:49 -0400)

are available in the git repository at:
  git://git.yoctoproject.org/linux-yocto-2.6.37-contrib nitin/meta
  http://git.yoctoproject.org/cgit.cgi/linux-yocto-2.6.37-contrib/log/?h=nitin/meta

Nitin A Kamble (1):
  meta: crownbay.scc change emgd branch name

 .../bsp/crownbay/crownbay-standard.scc             |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

-- 
1.7.3.4



^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH 1/1] meta: crownbay.scc change emgd branch name
  2012-07-17  5:57 [PATCH 0/1] meta: update crownbay.scc for newer emgd driver nitin.a.kamble
@ 2012-07-17  5:57 ` nitin.a.kamble
  2012-07-18 14:13 ` [PATCH 0/1] meta: update crownbay.scc for newer emgd driver Tom Zanussi
  1 sibling, 0 replies; 10+ messages in thread
From: nitin.a.kamble @ 2012-07-17  5:57 UTC (permalink / raw)
  To: yocto, tom.zanussi, bruce.ashfield

From: Nitin A Kamble <nitin.a.kamble@intel.com>

Signed-off-by: Nitin A Kamble <nitin.a.kamble@intel.com>
---
 .../bsp/crownbay/crownbay-standard.scc             |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta/cfg/kernel-cache/bsp/crownbay/crownbay-standard.scc b/meta/cfg/kernel-cache/bsp/crownbay/crownbay-standard.scc
index 4990371..cbda721 100644
--- a/meta/cfg/kernel-cache/bsp/crownbay/crownbay-standard.scc
+++ b/meta/cfg/kernel-cache/bsp/crownbay/crownbay-standard.scc
@@ -6,7 +6,7 @@ include ktypes/standard
 branch crownbay
 
 include features/emgd/emgd-1.10.scc
-git merge emgd-1.10
+git merge emgd-1.14
 
 include crownbay.scc
 
-- 
1.7.3.4



^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
  2012-07-17  5:57 [PATCH 0/1] meta: update crownbay.scc for newer emgd driver nitin.a.kamble
  2012-07-17  5:57 ` [PATCH 1/1] meta: crownbay.scc change emgd branch name nitin.a.kamble
@ 2012-07-18 14:13 ` Tom Zanussi
  2012-07-18 16:47   ` Bruce Ashfield
  2012-07-19  2:58   ` Kamble, Nitin A
  1 sibling, 2 replies; 10+ messages in thread
From: Tom Zanussi @ 2012-07-18 14:13 UTC (permalink / raw)
  To: nitin.a.kamble; +Cc: yocto

On Mon, 2012-07-16 at 22:57 -0700, nitin.a.kamble@intel.com wrote:
> From: Nitin A Kamble <nitin.a.kamble@intel.com>
> 
> This commit changes emgd kernel driver for crownbay bsp to the emgd-1.14.
> 

Nitin,

Doesn't switching this over in the kenrnel require emgd-1.14 userspace
recipe updates to work?  Did I miss those?

Tom

> Thanks,
> Nitin
> 
> The following changes since commit c2b5ee363bf2612932301488f3ac0644ff70fc7c:
> 
>   meta: disable CONFIG_MTD_NAND_VERIFY_WRITE (2012-07-14 00:21:49 -0400)
> 
> are available in the git repository at:
>   git://git.yoctoproject.org/linux-yocto-2.6.37-contrib nitin/meta
>   http://git.yoctoproject.org/cgit.cgi/linux-yocto-2.6.37-contrib/log/?h=nitin/meta
> 
> Nitin A Kamble (1):
>   meta: crownbay.scc change emgd branch name
> 
>  .../bsp/crownbay/crownbay-standard.scc             |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 




^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
  2012-07-18 14:13 ` [PATCH 0/1] meta: update crownbay.scc for newer emgd driver Tom Zanussi
@ 2012-07-18 16:47   ` Bruce Ashfield
  2012-07-19  2:58   ` Kamble, Nitin A
  1 sibling, 0 replies; 10+ messages in thread
From: Bruce Ashfield @ 2012-07-18 16:47 UTC (permalink / raw)
  To: Tom Zanussi; +Cc: yocto

On 12-07-18 10:13 AM, Tom Zanussi wrote:
> On Mon, 2012-07-16 at 22:57 -0700, nitin.a.kamble@intel.com wrote:
>> From: Nitin A Kamble<nitin.a.kamble@intel.com>
>>
>> This commit changes emgd kernel driver for crownbay bsp to the emgd-1.14.
>>
>
> Nitin,
>
> Doesn't switching this over in the kenrnel require emgd-1.14 userspace
> recipe updates to work?  Did I miss those?

Good question. I'm holding on merging and pushing this until we
are sure that the whole stack will work ..

Cheers,

Bruce

>
> Tom
>
>> Thanks,
>> Nitin
>>
>> The following changes since commit c2b5ee363bf2612932301488f3ac0644ff70fc7c:
>>
>>    meta: disable CONFIG_MTD_NAND_VERIFY_WRITE (2012-07-14 00:21:49 -0400)
>>
>> are available in the git repository at:
>>    git://git.yoctoproject.org/linux-yocto-2.6.37-contrib nitin/meta
>>    http://git.yoctoproject.org/cgit.cgi/linux-yocto-2.6.37-contrib/log/?h=nitin/meta
>>
>> Nitin A Kamble (1):
>>    meta: crownbay.scc change emgd branch name
>>
>>   .../bsp/crownbay/crownbay-standard.scc             |    2 +-
>>   1 files changed, 1 insertions(+), 1 deletions(-)
>>
>
>



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
  2012-07-18 14:13 ` [PATCH 0/1] meta: update crownbay.scc for newer emgd driver Tom Zanussi
  2012-07-18 16:47   ` Bruce Ashfield
@ 2012-07-19  2:58   ` Kamble, Nitin A
  2012-07-19  3:26     ` Tom Zanussi
  1 sibling, 1 reply; 10+ messages in thread
From: Kamble, Nitin A @ 2012-07-19  2:58 UTC (permalink / raw)
  To: Zanussi, Tom; +Cc: yocto



> -----Original Message-----
> From: Zanussi, Tom
> Sent: Wednesday, July 18, 2012 7:43 PM
> To: Kamble, Nitin A
> Cc: yocto@yoctoproject.org; bruce.ashfield@windriver.com
> Subject: Re: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
> 
> On Mon, 2012-07-16 at 22:57 -0700, nitin.a.kamble@intel.com wrote:
> > From: Nitin A Kamble <nitin.a.kamble@intel.com>
> >
> > This commit changes emgd kernel driver for crownbay bsp to the emgd-
> 1.14.
> >
> 
> Nitin,
> 
> Doesn't switching this over in the kenrnel require emgd-1.14 userspace
> recipe updates to work?  Did I miss those?
> 
> Tom
> 
Tom,
    I am holding the emgd 1.14 userspace commits until the kernel repository change goes upstream. The kernel recipe will need new SRCREV (with emgd 1.14 kernel parts) for the user level emgd 1.14 parts to work. And that dependency will get satisfied once this commit is in.

Nitin


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
  2012-07-19  2:58   ` Kamble, Nitin A
@ 2012-07-19  3:26     ` Tom Zanussi
  2012-07-19 14:39       ` Kamble, Nitin A
  0 siblings, 1 reply; 10+ messages in thread
From: Tom Zanussi @ 2012-07-19  3:26 UTC (permalink / raw)
  To: Kamble, Nitin A; +Cc: yocto

On Wed, 2012-07-18 at 19:58 -0700, Kamble, Nitin A wrote:
> 
> > -----Original Message-----
> > From: Zanussi, Tom
> > Sent: Wednesday, July 18, 2012 7:43 PM
> > To: Kamble, Nitin A
> > Cc: yocto@yoctoproject.org; bruce.ashfield@windriver.com
> > Subject: Re: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
> > 
> > On Mon, 2012-07-16 at 22:57 -0700, nitin.a.kamble@intel.com wrote:
> > > From: Nitin A Kamble <nitin.a.kamble@intel.com>
> > >
> > > This commit changes emgd kernel driver for crownbay bsp to the emgd-
> > 1.14.
> > >
> > 
> > Nitin,
> > 
> > Doesn't switching this over in the kenrnel require emgd-1.14 userspace
> > recipe updates to work?  Did I miss those?
> > 
> > Tom
> > 
> Tom,
>     I am holding the emgd 1.14 userspace commits until the kernel repository change goes upstream. The kernel recipe will need new SRCREV (with emgd 1.14 kernel parts) for the user level emgd 1.14 parts to work. And that dependency will get satisfied once this commit is in.
> 

Yeah, I understand, the crownbay is typically the most complicated
update for all of those reasons - simultaneous new kernel
version/recipe, new emgd branch, and new emgd userspace recipe all at
the same time.

In the past I've submitted them all at the same time for simultaneous
review and some coordination with Bruce - once the kernel parts are in,
the kernel recipe and the userspace parts can then be pulled in along
with a simple follow-on patch to update the SRCREVs Bruce give us.

The thing is that if Bruce pulls in the patch to switch to emgd-1.14 and
only at that point do you post the userspace recipe, and it ends up
needing some rework, in the meantime the graphics remain non-functional
(I know, they already are, thanks to the package re-ordering patches
breaking the current recipe, but hopefully the 1.14 changes will be in
and we don't have to worry about it any more.).

Does that make sense, and can you just post the userspace and kernel
recipe parts before we give Bruce the go-ahead in making the kernel
changes?  (I'd also like to try it out myself as well...).

Thanks,

Tom 

> Nitin
> 




^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
  2012-07-19  3:26     ` Tom Zanussi
@ 2012-07-19 14:39       ` Kamble, Nitin A
  2012-07-19 14:57         ` Tom Zanussi
  0 siblings, 1 reply; 10+ messages in thread
From: Kamble, Nitin A @ 2012-07-19 14:39 UTC (permalink / raw)
  To: Zanussi, Tom; +Cc: yocto



> -----Original Message-----
> From: Zanussi, Tom
> Sent: Thursday, July 19, 2012 8:57 AM
> To: Kamble, Nitin A
> Cc: yocto@yoctoproject.org; bruce.ashfield@windriver.com
> Subject: RE: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
> 
> On Wed, 2012-07-18 at 19:58 -0700, Kamble, Nitin A wrote:
> >
> > > -----Original Message-----
> > > From: Zanussi, Tom
> > > Sent: Wednesday, July 18, 2012 7:43 PM
> > > To: Kamble, Nitin A
> > > Cc: yocto@yoctoproject.org; bruce.ashfield@windriver.com
> > > Subject: Re: [PATCH 0/1] meta: update crownbay.scc for newer emgd
> > > driver
> > >
> > > On Mon, 2012-07-16 at 22:57 -0700, nitin.a.kamble@intel.com wrote:
> > > > From: Nitin A Kamble <nitin.a.kamble@intel.com>
> > > >
> > > > This commit changes emgd kernel driver for crownbay bsp to the
> > > > emgd-
> > > 1.14.
> > > >
> > >
> > > Nitin,
> > >
> > > Doesn't switching this over in the kenrnel require emgd-1.14
> > > userspace recipe updates to work?  Did I miss those?
> > >
> > > Tom
> > >
> > Tom,
> >     I am holding the emgd 1.14 userspace commits until the kernel repository
> change goes upstream. The kernel recipe will need new SRCREV (with emgd
> 1.14 kernel parts) for the user level emgd 1.14 parts to work. And that
> dependency will get satisfied once this commit is in.
> >
> 
> Yeah, I understand, the crownbay is typically the most complicated update
> for all of those reasons - simultaneous new kernel version/recipe, new emgd
> branch, and new emgd userspace recipe all at the same time.
> 
> In the past I've submitted them all at the same time for simultaneous review
> and some coordination with Bruce - once the kernel parts are in, the kernel
> recipe and the userspace parts can then be pulled in along with a simple
> follow-on patch to update the SRCREVs Bruce give us.
> 
> The thing is that if Bruce pulls in the patch to switch to emgd-1.14 and only at
> that point do you post the userspace recipe, and it ends up needing some
> rework, in the meantime the graphics remain non-functional (I know, they
> already are, thanks to the package re-ordering patches breaking the current
> recipe, but hopefully the 1.14 changes will be in and we don't have to worry
> about it any more.).
> 
> Does that make sense, and can you just post the userspace and kernel recipe
> parts before we give Bruce the go-ahead in making the kernel changes?  (I'd
> also like to try it out myself as well...).
> 
Tom,
  Until the crownbay bsp's Linux-yocto kernel recipe's meta SRCREV is updated, these kernel repo changes have no effect in the build process. Right ? So as I see, this commit does not make anything worst (or better).
I am sending the userland recipe commits anyway; So that you can also test it out.


Thanks,
Nitin




^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
  2012-07-19 14:39       ` Kamble, Nitin A
@ 2012-07-19 14:57         ` Tom Zanussi
  2012-07-19 15:25           ` Kamble, Nitin A
  0 siblings, 1 reply; 10+ messages in thread
From: Tom Zanussi @ 2012-07-19 14:57 UTC (permalink / raw)
  To: Kamble, Nitin A; +Cc: yocto

On Thu, 2012-07-19 at 07:39 -0700, Kamble, Nitin A wrote:
> 
> > -----Original Message-----
> > From: Zanussi, Tom
> > Sent: Thursday, July 19, 2012 8:57 AM
> > To: Kamble, Nitin A
> > Cc: yocto@yoctoproject.org; bruce.ashfield@windriver.com
> > Subject: RE: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
> > 
> > On Wed, 2012-07-18 at 19:58 -0700, Kamble, Nitin A wrote:
> > >
> > > > -----Original Message-----
> > > > From: Zanussi, Tom
> > > > Sent: Wednesday, July 18, 2012 7:43 PM
> > > > To: Kamble, Nitin A
> > > > Cc: yocto@yoctoproject.org; bruce.ashfield@windriver.com
> > > > Subject: Re: [PATCH 0/1] meta: update crownbay.scc for newer emgd
> > > > driver
> > > >
> > > > On Mon, 2012-07-16 at 22:57 -0700, nitin.a.kamble@intel.com wrote:
> > > > > From: Nitin A Kamble <nitin.a.kamble@intel.com>
> > > > >
> > > > > This commit changes emgd kernel driver for crownbay bsp to the
> > > > > emgd-
> > > > 1.14.
> > > > >
> > > >
> > > > Nitin,
> > > >
> > > > Doesn't switching this over in the kenrnel require emgd-1.14
> > > > userspace recipe updates to work?  Did I miss those?
> > > >
> > > > Tom
> > > >
> > > Tom,
> > >     I am holding the emgd 1.14 userspace commits until the kernel repository
> > change goes upstream. The kernel recipe will need new SRCREV (with emgd
> > 1.14 kernel parts) for the user level emgd 1.14 parts to work. And that
> > dependency will get satisfied once this commit is in.
> > >
> > 
> > Yeah, I understand, the crownbay is typically the most complicated update
> > for all of those reasons - simultaneous new kernel version/recipe, new emgd
> > branch, and new emgd userspace recipe all at the same time.
> > 
> > In the past I've submitted them all at the same time for simultaneous review
> > and some coordination with Bruce - once the kernel parts are in, the kernel
> > recipe and the userspace parts can then be pulled in along with a simple
> > follow-on patch to update the SRCREVs Bruce give us.
> > 
> > The thing is that if Bruce pulls in the patch to switch to emgd-1.14 and only at
> > that point do you post the userspace recipe, and it ends up needing some
> > rework, in the meantime the graphics remain non-functional (I know, they
> > already are, thanks to the package re-ordering patches breaking the current
> > recipe, but hopefully the 1.14 changes will be in and we don't have to worry
> > about it any more.).
> > 
> > Does that make sense, and can you just post the userspace and kernel recipe
> > parts before we give Bruce the go-ahead in making the kernel changes?  (I'd
> > also like to try it out myself as well...).
> > 
> Tom,
>   Until the crownbay bsp's Linux-yocto kernel recipe's meta SRCREV is updated, these kernel repo changes have no effect in the build process. Right ? So as I see, this commit does not make anything worst (or better).
> I am sending the userland recipe commits anyway; So that you can also test it out.
> 

Right, but what if we have to update the meta SRCREV for a different
reason in the meantime (like we did last week for instance).  We're then
forced to take the emgd commit as well.

In any case, it's your BSP so it's up to you how you want to manage it,
I'm just trying to save you and the users some headaches.  It always
made sense to me to do it all at the same time, all else being equal...

Tom

> 
> Thanks,
> Nitin
> 
> 
> 




^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
  2012-07-19 14:57         ` Tom Zanussi
@ 2012-07-19 15:25           ` Kamble, Nitin A
  2012-07-20  4:53             ` Bruce Ashfield
  0 siblings, 1 reply; 10+ messages in thread
From: Kamble, Nitin A @ 2012-07-19 15:25 UTC (permalink / raw)
  To: Zanussi, Tom; +Cc: yocto



> -----Original Message-----
> From: Zanussi, Tom
> Sent: Thursday, July 19, 2012 8:27 PM
> To: Kamble, Nitin A
> Cc: yocto@yoctoproject.org; bruce.ashfield@windriver.com
> Subject: RE: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
> 
> On Thu, 2012-07-19 at 07:39 -0700, Kamble, Nitin A wrote:
> >
> > > -----Original Message-----
> > > From: Zanussi, Tom
> > > Sent: Thursday, July 19, 2012 8:57 AM
> > > To: Kamble, Nitin A
> > > Cc: yocto@yoctoproject.org; bruce.ashfield@windriver.com
> > > Subject: RE: [PATCH 0/1] meta: update crownbay.scc for newer emgd
> > > driver
> > >
> > > On Wed, 2012-07-18 at 19:58 -0700, Kamble, Nitin A wrote:
> > > >
> > > > > -----Original Message-----
> > > > > From: Zanussi, Tom
> > > > > Sent: Wednesday, July 18, 2012 7:43 PM
> > > > > To: Kamble, Nitin A
> > > > > Cc: yocto@yoctoproject.org; bruce.ashfield@windriver.com
> > > > > Subject: Re: [PATCH 0/1] meta: update crownbay.scc for newer
> > > > > emgd driver
> > > > >
> > > > > On Mon, 2012-07-16 at 22:57 -0700, nitin.a.kamble@intel.com wrote:
> > > > > > From: Nitin A Kamble <nitin.a.kamble@intel.com>
> > > > > >
> > > > > > This commit changes emgd kernel driver for crownbay bsp to the
> > > > > > emgd-
> > > > > 1.14.
> > > > > >
> > > > >
> > > > > Nitin,
> > > > >
> > > > > Doesn't switching this over in the kenrnel require emgd-1.14
> > > > > userspace recipe updates to work?  Did I miss those?
> > > > >
> > > > > Tom
> > > > >
> > > > Tom,
> > > >     I am holding the emgd 1.14 userspace commits until the kernel
> > > > repository
> > > change goes upstream. The kernel recipe will need new SRCREV (with
> > > emgd
> > > 1.14 kernel parts) for the user level emgd 1.14 parts to work. And
> > > that dependency will get satisfied once this commit is in.
> > > >
> > >
> > > Yeah, I understand, the crownbay is typically the most complicated
> > > update for all of those reasons - simultaneous new kernel
> > > version/recipe, new emgd branch, and new emgd userspace recipe all at
> the same time.
> > >
> > > In the past I've submitted them all at the same time for
> > > simultaneous review and some coordination with Bruce - once the
> > > kernel parts are in, the kernel recipe and the userspace parts can
> > > then be pulled in along with a simple follow-on patch to update the
> SRCREVs Bruce give us.
> > >
> > > The thing is that if Bruce pulls in the patch to switch to emgd-1.14
> > > and only at that point do you post the userspace recipe, and it ends
> > > up needing some rework, in the meantime the graphics remain
> > > non-functional (I know, they already are, thanks to the package
> > > re-ordering patches breaking the current recipe, but hopefully the
> > > 1.14 changes will be in and we don't have to worry about it any more.).
> > >
> > > Does that make sense, and can you just post the userspace and kernel
> > > recipe parts before we give Bruce the go-ahead in making the kernel
> > > changes?  (I'd also like to try it out myself as well...).
> > >
> > Tom,
> >   Until the crownbay bsp's Linux-yocto kernel recipe's meta SRCREV is
> updated, these kernel repo changes have no effect in the build process.
> Right ? So as I see, this commit does not make anything worst (or better).
> > I am sending the userland recipe commits anyway; So that you can also test
> it out.
> >
> 
> Right, but what if we have to update the meta SRCREV for a different reason
> in the meantime (like we did last week for instance).  We're then forced to
> take the emgd commit as well.

I see the issue now. And I don't see a perfect solution to the issue. Because of commits in two different repos there always going to be this kind of a sync issue. The best is pull the commits in both repositories at the same time.

Thanks,
Nitin

> 
> In any case, it's your BSP so it's up to you how you want to manage it, I'm just
> trying to save you and the users some headaches.  It always made sense to
> me to do it all at the same time, all else being equal...
> 
> Tom
> 
> >
> > Thanks,
> > Nitin
> >
> >
> >
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
  2012-07-19 15:25           ` Kamble, Nitin A
@ 2012-07-20  4:53             ` Bruce Ashfield
  0 siblings, 0 replies; 10+ messages in thread
From: Bruce Ashfield @ 2012-07-20  4:53 UTC (permalink / raw)
  To: Kamble, Nitin A; +Cc: yocto

On 12-07-19 11:25 AM, Kamble, Nitin A wrote:
>
>
>> -----Original Message-----
>> From: Zanussi, Tom
>> Sent: Thursday, July 19, 2012 8:27 PM
>> To: Kamble, Nitin A
>> Cc: yocto@yoctoproject.org; bruce.ashfield@windriver.com
>> Subject: RE: [PATCH 0/1] meta: update crownbay.scc for newer emgd driver
>>
>> On Thu, 2012-07-19 at 07:39 -0700, Kamble, Nitin A wrote:
>>>
>>>> -----Original Message-----
>>>> From: Zanussi, Tom
>>>> Sent: Thursday, July 19, 2012 8:57 AM
>>>> To: Kamble, Nitin A
>>>> Cc: yocto@yoctoproject.org; bruce.ashfield@windriver.com
>>>> Subject: RE: [PATCH 0/1] meta: update crownbay.scc for newer emgd
>>>> driver
>>>>
>>>> On Wed, 2012-07-18 at 19:58 -0700, Kamble, Nitin A wrote:
>>>>>
>>>>>> -----Original Message-----
>>>>>> From: Zanussi, Tom
>>>>>> Sent: Wednesday, July 18, 2012 7:43 PM
>>>>>> To: Kamble, Nitin A
>>>>>> Cc: yocto@yoctoproject.org; bruce.ashfield@windriver.com
>>>>>> Subject: Re: [PATCH 0/1] meta: update crownbay.scc for newer
>>>>>> emgd driver
>>>>>>
>>>>>> On Mon, 2012-07-16 at 22:57 -0700, nitin.a.kamble@intel.com wrote:
>>>>>>> From: Nitin A Kamble<nitin.a.kamble@intel.com>
>>>>>>>
>>>>>>> This commit changes emgd kernel driver for crownbay bsp to the
>>>>>>> emgd-
>>>>>> 1.14.
>>>>>>>
>>>>>>
>>>>>> Nitin,
>>>>>>
>>>>>> Doesn't switching this over in the kenrnel require emgd-1.14
>>>>>> userspace recipe updates to work?  Did I miss those?
>>>>>>
>>>>>> Tom
>>>>>>
>>>>> Tom,
>>>>>      I am holding the emgd 1.14 userspace commits until the kernel
>>>>> repository
>>>> change goes upstream. The kernel recipe will need new SRCREV (with
>>>> emgd
>>>> 1.14 kernel parts) for the user level emgd 1.14 parts to work. And
>>>> that dependency will get satisfied once this commit is in.
>>>>>
>>>>
>>>> Yeah, I understand, the crownbay is typically the most complicated
>>>> update for all of those reasons - simultaneous new kernel
>>>> version/recipe, new emgd branch, and new emgd userspace recipe all at
>> the same time.
>>>>
>>>> In the past I've submitted them all at the same time for
>>>> simultaneous review and some coordination with Bruce - once the
>>>> kernel parts are in, the kernel recipe and the userspace parts can
>>>> then be pulled in along with a simple follow-on patch to update the
>> SRCREVs Bruce give us.
>>>>
>>>> The thing is that if Bruce pulls in the patch to switch to emgd-1.14
>>>> and only at that point do you post the userspace recipe, and it ends
>>>> up needing some rework, in the meantime the graphics remain
>>>> non-functional (I know, they already are, thanks to the package
>>>> re-ordering patches breaking the current recipe, but hopefully the
>>>> 1.14 changes will be in and we don't have to worry about it any more.).
>>>>
>>>> Does that make sense, and can you just post the userspace and kernel
>>>> recipe parts before we give Bruce the go-ahead in making the kernel
>>>> changes?  (I'd also like to try it out myself as well...).
>>>>
>>> Tom,
>>>    Until the crownbay bsp's Linux-yocto kernel recipe's meta SRCREV is
>> updated, these kernel repo changes have no effect in the build process.
>> Right ? So as I see, this commit does not make anything worst (or better).
>>> I am sending the userland recipe commits anyway; So that you can also test
>> it out.
>>>
>>
>> Right, but what if we have to update the meta SRCREV for a different reason
>> in the meantime (like we did last week for instance).  We're then forced to
>> take the emgd commit as well.
>
> I see the issue now. And I don't see a perfect solution to the issue. Because of commits in two different repos there always going to be this kind of a sync issue. The best is pull the commits in both repositories at the same time.

FYI: everything is staged and I have a SRCREV commit prepared for this.

When I hear a clear "yes" on the userspace changes, I can send the pull
request immediately.

Cheers,

Bruce

>
> Thanks,
> Nitin
>
>>
>> In any case, it's your BSP so it's up to you how you want to manage it, I'm just
>> trying to save you and the users some headaches.  It always made sense to
>> me to do it all at the same time, all else being equal...
>>
>> Tom
>>
>>>
>>> Thanks,
>>> Nitin
>>>
>>>
>>>
>>
>



^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2012-07-20  4:53 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-17  5:57 [PATCH 0/1] meta: update crownbay.scc for newer emgd driver nitin.a.kamble
2012-07-17  5:57 ` [PATCH 1/1] meta: crownbay.scc change emgd branch name nitin.a.kamble
2012-07-18 14:13 ` [PATCH 0/1] meta: update crownbay.scc for newer emgd driver Tom Zanussi
2012-07-18 16:47   ` Bruce Ashfield
2012-07-19  2:58   ` Kamble, Nitin A
2012-07-19  3:26     ` Tom Zanussi
2012-07-19 14:39       ` Kamble, Nitin A
2012-07-19 14:57         ` Tom Zanussi
2012-07-19 15:25           ` Kamble, Nitin A
2012-07-20  4:53             ` Bruce Ashfield

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.