From: Ben Warren <biggerbadderben@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/3][Net][ARM] Moved Davinci Ethernet driver to drivers/net
Date: Tue, 28 Apr 2009 13:04:59 -0700 [thread overview]
Message-ID: <49F7616B.8080200@gmail.com> (raw)
In-Reply-To: <200904281254.39931.david-b@pacbell.net>
David Brownell wrote:
> On Tuesday 28 April 2009, Ben Warren wrote:
>
>> The driver has been renamed dm644x_emac.c
>>
>
> Let me suggest "davinci_emac.c" ... the same controller
> is in some non-dm644x DaVinci silicon (like dm365), and
> a gigabit flavor is in the dm6467 chip.
>
>
>
OK. I named it this because the top of the file mentioned it was ripped
off from 'dm644x_emac.c' (not Linux)
>>
>> -COBJS = timer.o ether.o lxt972.o dp83848.o
>> +COBJS = timer.o lxt972.o dp83848.o
>> SOBJS = lowlevel_init.o reset.o
>>
>
> Given the recent cleanups in cpu/arm926ejs/davinci/Makefile,
> that won't quite apply. I suggest you pull some of the
> changes from arm-next.
>
OK, I'll look into that.
> Procedure note: this is exactly why Linux now has a
> shared "-next" tree ... to make sure interdependencies
> between development branches can be sorted out before
> the next merge window, so the merge window can be spent
> addressing more substantive issues (if any).
>
>
Sounds good. I believe there is a 'next' branch on the U-boot mainline,
and quite often we do use it, but probably should do so earlier in the
process.
Thanks for the help and advice!
Ben
next prev parent reply other threads:[~2009-04-28 20:04 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-04-28 17:12 [U-Boot] [PATCH 0/3][Net][ARM] Davinci Ethernet driver cleanup Ben Warren
2009-04-28 17:12 ` [U-Boot] [PATCH 1/3][Net][ARM] Moved Davinci Ethernet driver to drivers/net Ben Warren
2009-04-28 17:12 ` [U-Boot] [PATCH 2/3][Net][ARM] Initial cleanup of Davinci Ethernet driver Ben Warren
2009-04-28 17:12 ` [U-Boot] [PATCH 3/3][Net][ARM] Switched dm644x_emac Ethernet driver to use newer API Ben Warren
2009-04-29 20:13 ` Jean-Christophe PLAGNIOL-VILLARD
2009-04-29 20:29 ` Ben Warren
2009-04-29 20:49 ` Jean-Christophe PLAGNIOL-VILLARD
2009-04-29 22:11 ` Ben Warren
2009-04-29 23:19 ` Jean-Christophe PLAGNIOL-VILLARD
2009-04-30 4:28 ` David Brownell
2009-04-28 19:54 ` [U-Boot] [PATCH 1/3][Net][ARM] Moved Davinci Ethernet driver to drivers/net David Brownell
2009-04-28 20:04 ` Ben Warren [this message]
2009-04-29 23:32 ` [U-Boot] [PATCH 0/3][Net][ARM] Davinci Ethernet driver cleanup David Brownell
2009-04-29 23:38 ` Ben Warren
2009-04-30 5:59 ` Jean-Christophe PLAGNIOL-VILLARD
[not found] ` <C9D59C82B94F474B872F2092A87F26148147B6A3@dlee07.ent.ti.com>
2009-05-12 17:43 ` Ben Warren
2009-05-12 18:28 ` Jean-Christophe PLAGNIOL-VILLARD
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=49F7616B.8080200@gmail.com \
--to=biggerbadderben@gmail.com \
--cc=u-boot@lists.denx.de \
/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