From: tomasz.figa@gmail.com (Tomasz Figa)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] Samsung Clock changes for v3.14
Date: Thu, 09 Jan 2014 20:08:32 +0100 [thread overview]
Message-ID: <5724394.ipj3s2zcuB@flatron> (raw)
In-Reply-To: <20140109004956.7168.34121@quantum>
On Wednesday 08 of January 2014 16:49:56 Mike Turquette wrote:
> Quoting Mike Turquette (2014-01-08 14:59:43)
> > Quoting Tomasz Figa (2014-01-08 14:43:24)
> > > On Wednesday 08 of January 2014 14:07:49 Mike Turquette wrote:
> > > > Quoting Tomasz Figa (2014-01-08 11:13:38)
> > > > > Hi Mike,
> > > > >
> > > > > Please consider pulling following Samsung Clock changes for v3.14.
> > > > >
> > > > > The following changes since commit 2bb00c68e094271b79deac993893461cc051b721:
> > > >
> > > > Hi Tomasz,
> > > >
> > > > Commit 2bb00c68 is the tip of the Samsung clk fixes that I'm about to
> > > > send out. Can you rebase this request onto the tip of clk-next?
> > >
> > > I guess I can do it, but wouldn't this introduce merge conflicts, since
> > > some of the fixes are changing the same parts of the code?
> >
> > Yes it will cause conflicts, but that is sometimes OK.
> >
> > >
> > > I based my samsung-next branch on top of your clk-next with my fixes
> > > branch merged, since that's what will end up in Linus' tree anyway. Was it
> > > not the right thing to do?
> >
> > Well maybe I am missing something, but I think it is not the best thing.
> > The problem is that those patches will sort of be merged twice. Again
> > maybe I am missing something. The same commit IDs will be used so
> > perhaps it is OK...
> >
> > Part of this trouble is caused by the simple way that I fork clk-next
> > from v3.xx-rc1. Since I never rebase that branch to a more recent -rc
> > then there is a delta between the fixes that go in the queue of patches
> > for the next merge window.
> >
> > I guess one way that other subsystems handle this is by having something
> > like clk-next-late which is a new branch based on a later -rc.
> >
> > How awful is the merge conflict resolution?
>
> Thanks for working this out on IRC. I've taken this pull request into
> clk-next for 3.14.
Great, thanks.
One more thing. Could you update your public tree to let patches get some
wider testing in linux-next before the merge window?
Best regards,
Tomasz
prev parent reply other threads:[~2014-01-09 19:08 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-08 19:13 [GIT PULL] Samsung Clock changes for v3.14 Tomasz Figa
2014-01-08 22:07 ` Mike Turquette
2014-01-08 22:43 ` Tomasz Figa
2014-01-08 22:59 ` Mike Turquette
2014-01-09 0:49 ` Mike Turquette
2014-01-09 19:08 ` Tomasz Figa [this message]
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=5724394.ipj3s2zcuB@flatron \
--to=tomasz.figa@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
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;
as well as URLs for NNTP newsgroup(s).