RE: linux-next: manual merge of the sound tree with the s5p tree

[Kukjin Kim]

Russell King wrote:
> 
> On Fri, Dec 24, 2010 at 10:32:33AM +0900, Kukjin Kim wrote:
> > Takashi Iwai wrote:
> > > 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?
> 
Russell, thanks for your reply :-)

> IMHO it is never appropriate to cherry pick commits from someone elses
> tree into your own tree as it creates duplicates in the history.
> 
Absolutely, you're right. I missed.

> What would be better is to talk to the person who owns the tree with
> the commits you're interested in, and see about merging the minimal
> set of commits you need into your own tree.  They may tell you the
> name of a branch for you to merge into your tree (preferred), or more
> obscurely tell you a commit id in their tree.
> 
Yeah, would be nice to me ;-)

> Now there are two options:
> - If you know the branch you need, you can then either pull that branch
>   into your tree, and commit the dependent patches on top of that.
> 
Unfortunately, as I know, there is no suitable branch for it and me in
sound-2.6 tree now :-(

> - If you only have a commit ID in the remote tree, you can fetch their
>   tree first, use gitk to visualize the history and check you have it,
>   and then do:
>     git merge --no-commit <commitid>
>   in your tree.  Then commit the result with your own commit message
>   explaining what's going on.
> 
Hmm...happened error when merged but I don't know how can solve that :-(

IMHO, your first option is better to me but it depends on Takashi...hehe...
Anyway Thanks and Happy New year!


Takashi, as Russell's first option, could you please make some branch which
includes following to me?
Of course, I know you need rebase you tree for it...so sorry for bothering.

git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-2.6.git
#topic/asoc

commit 83e37b8e400ca51cc97946815b3055daacd92fa8
Author: Jassi Brar <jassi.brar@xxxxxxxxxxx>
Date:   Mon Nov 22 15:35:53 2010 +0900

    ARM: Samsung: Define common audio-dma device

    The ASoC uses common DMA driver for Audio devices. So it makes
    sense to a common audio-dma device shared across all platforms.

    Signed-off-by: Jassi Brar <jassi.brar@xxxxxxxxxxx>
    Acked-by: Kukjin Kim <kgene.kim@xxxxxxxxxxx>
    Acked-by: Liam Girdwood <lrg@xxxxxxxxxxxxxxx>
    Signed-off-by: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>

commit 58bb4072132c54d832082cc6eac396a6db009cbd
Author: Jassi Brar <jassi.brar@xxxxxxxxxxx>
Date:   Mon Nov 22 15:35:50 2010 +0900

    ASoC: Samsung: Rename DMA device

    Some Samsung SoCs have a PCM(DSP) controller. So the name
    s3c24xx-pcm-audio for DMA driver is not very appropraite.
    This patch moves :-
        s3c24xx-pcm-audio -> samsung-audio

    Signed-off-by: Jassi Brar <jassi.brar@xxxxxxxxxxx>
    Acked-by: Kukjin Kim <kgene.kim@xxxxxxxxxxx>
    Acked-by: Liam Girdwood <lrg@xxxxxxxxxxxxxxx>
    Signed-off-by: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>

Please let me know your opinion.
Happy New year ;-)

Thanks.

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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/