From: nicolas.ferre@atmel.com (Nicolas Ferre)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] at91: drivers for 3.3
Date: Fri, 16 Dec 2011 11:21:30 +0100 [thread overview]
Message-ID: <4EEB1BAA.9050705@atmel.com> (raw)
In-Reply-To: <CAOesGMjurPDXVR1FLmLXa5NDNjpTzGpshQQ0piqHis5wsYM=Tg@mail.gmail.com>
On 12/16/2011 06:53 AM, Olof Johansson :
> Hi,
>
> On Thu, Dec 15, 2011 at 7:10 AM, Nicolas Ferre <nicolas.ferre@atmel.com> wrote:
>> Arnd, Olof,
>>
>> This is a first "drivers" part of our material for 3.3 merge window. It collects
>> Ethernet drivers changes.
>> One thing to note is that the macb.c driver is modified to support
>> device tree. We agreed with David to drag those modifications into
>> arm-soc git tree because they go on top of Jamie's work on macb-gem driver
>> already in your drivers/macb-gem branch.
>>
>> This pull request goes on top of Linus' 3.2-rc5 commit but with additional
>> dependencies with two of your branches:
>> 1/ fixes (which seems already included in 3.2-rc5)
>> 2/ drivers/macb-gem
>>
>> I had to resolve a conflict while merging the macb-gem branche with current 3.2-rc5. The
>> resolution is commited into this branch.
>> Tell me if you need more precision.
>
> The history of this branch due to the double dependencies gets a bit
> awkward. Also, it seems like you in one of your merges kept the
> 'default y' in NET_CADENCE that should have been removed.
Well, no, this line has been added into "eccab1ec" and is in mainline
since 3.2-rc3.
This line have to be kept and the merge conflict is triggered during the
merge of macb-gem on top of Linus' tree. BTW, this merge is already
performed properly in linux-next.
> Would you mind trying this instead?
>
> * Base this new branch on the previous macb-gem branch
> * Cherry-pick the fixes from 'fixes' into the new branch if truly
> needed -- I tested here and all of the patches applied cleanly.
> * Then apply these patches
> * For build testing before sending, merge into for-next to aggregate
> with the rest of the 'fixes' changes -- it's OK that we have to do a
> fixup here.
>
> That should give a clean branch with neat history for just these
> changes. About 'fixes': Since they are in for-next, and will be in
> 3.2, that dependency will be taken care of that way.
>
> Then send a fresh pull request for that new branch, please.
Ok, I will base my "drivers" branch on top of macb-gem one and send you
another pull request.
Thanks, bye,
--
Nicolas Ferre
next prev parent reply other threads:[~2011-12-16 10:21 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-15 15:10 [GIT PULL] at91: drivers for 3.3 Nicolas Ferre
2011-12-16 5:53 ` Olof Johansson
2011-12-16 10:21 ` Nicolas Ferre [this message]
[not found] ` <4EEB388A.1040906@atmel.com>
2011-12-16 14:33 ` [GIT PULL v2] " Arnd Bergmann
2011-12-20 5:29 ` Olof Johansson
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=4EEB1BAA.9050705@atmel.com \
--to=nicolas.ferre@atmel.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 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.