public inbox for linux-next@vger.kernel.org 
 help / color / mirror / Atom feed
From: Kukjin Kim <kgene.kim@samsung•com>
To: 'Takashi Iwai' <tiwai@suse•de>,
	'Russell King' <rmk+kernel@arm•linux.org.uk>
Cc: 'Stephen Rothwell' <sfr@canb•auug.org.au>,
	linux-next@vger•kernel.org, linux-kernel@vger•kernel.org,
	'Jassi Brar' <jassi.brar@samsung•com>
Subject: RE: linux-next: manual merge of the sound tree with the s5p tree
Date: Fri, 24 Dec 2010 10:32:33 +0900	[thread overview]
Message-ID: <008f01cba30a$72f5c160$58e14420$%kim@samsung.com> (raw)
In-Reply-To: <s5h7hf0tvct.wl%tiwai@suse.de>

Takashi Iwai wrote:
> 
> At Thu, 23 Dec 2010 20:28:27 +0900,
> Kukjin Kim wrote:
> >
> > Takashi Iwai wrote:
> > >
> >
> > (snip)
> >
> > > > > > Anyway I need above commits from sound-2.6.git in my tree...
> > > > > > so how method is better to me instead of cherry-pick it?
> > > > >
> > > > > You can merge topic/asoc branch of sound git tree.
> > > > > This brach contains the all necessary commits for ASoC and is
almost
> > > > > never rebased.
> > > >
> > > > Hi,
> > > >
> > > > But I can't merge it because there is no 'topic/asoc' branch in
> > > > sound-2.6.git.
> > > >
(git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound-2.6.git)
> > > >
> > > > Is there another repository?
> > >
> > > OK, Mark's tree doesn't contain.  But you can merge for-2.6.38 tree
there.
> > >
> > > The main sound git tree is found in
> > >     git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-2.6.git
> > >
> > Aha~ :-)
> >
> > > > And should I merge it in my tree even though don't need all of ASoC
> > commits
> > > > :-( ?
> > >
> > > In general yes because this is the tree to be merged to 2.6.38.
> > >
> > Yeah, I know it.
> >
> > > > I'm not sure how many commits are in there...
> > >
> > > You can keep two branches.  One contains only your fixes, and another
> > > is the merged branch from yours and from sound git tree.  Expose the
> > > latter merged tree for linux-next.
> > >
> > Yeah...however, firstly, my tree will be sent to rmk during 38 merge
window,
> > if I merge from sound git, for avoiding build error I should send all
> > including them not only my patches. I think there is no need to
rmk...hmm
> >
> > So I did 'cherry-pick' only needed 2 commits in my tree.
> 
> And that caused problems :)
> 
Yup...you're right. As I said, I couldn't suitable some branch for me.

> > > There are many other ways to manage such things, but the above has
> > > worked for me well, at least.
> >
> > Yeah I think so...but is there another better way in this case?
Hehehe...
> 
> Well, ideally, create a separate branch containing these cross-tree
> commits based on vanilla.  Then pull this branch to both arm (or your)
> and sound trees.  In that way, both trees share the same commits in
> the least amount.
> 
Yeah...

> I basically don't like to rebase and the commits have been already in
> sound tree, but if this is the only way and Russell prefers this, it
> can be a possible option.
> 
I agree with you...now, rebase stuff is unnecessary load to you.

Ok...firstly, need to ask to Russell.
(Added Russell in To)

Hi Russell,

Sorry for bothering...
If possible, please read this thread :-)

So...

May/Should I merge sound tree for avoiding build error and conflict between
each other for 37 merge window instead of cherry-pick?
Or sound maintainer should rebase his tree so that can make some branch
which can support to avoid build error for me?
...Or because Stephen already informed about that so you or Linus can care
just current tree during merge window?

Which one is better to us? Or another method? Please let me know.

Thanks and Merry Christmas everyone :-)

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung•com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

  reply	other threads:[~2010-12-24  1:32 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-23  0:53 linux-next: manual merge of the sound tree with the s5p tree Stephen Rothwell
2010-12-23 10:31 ` Kukjin Kim
2010-12-23 11:03   ` Takashi Iwai
2010-12-23 11:11     ` Kukjin Kim
2010-12-23 11:17       ` Takashi Iwai
2010-12-23 11:28         ` Kukjin Kim
2010-12-23 14:30           ` Takashi Iwai
2010-12-24  1:32             ` Kukjin Kim [this message]
2010-12-24  8:27               ` Russell King
2010-12-28  2:24                 ` Kukjin Kim
2010-12-28 15:39                   ` Mark Brown
2010-12-28 16:23                     ` Russell King
2010-12-28 16:40                       ` Mark Brown
2010-12-30  0:54                         ` Kukjin Kim
2010-12-23 11:50       ` Mark Brown
2010-12-23 11:54         ` Piotr Hosowicz
2010-12-23 11:56           ` Mark Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='008f01cba30a$72f5c160$58e14420$%kim@samsung.com' \
    --to=kgene.kim@samsung$(echo .)com \
    --cc=jassi.brar@samsung$(echo .)com \
    --cc=linux-kernel@vger$(echo .)kernel.org \
    --cc=linux-next@vger$(echo .)kernel.org \
    --cc=rmk+kernel@arm$(echo .)linux.org.uk \
    --cc=sfr@canb$(echo .)auug.org.au \
    --cc=tiwai@suse$(echo .)de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox