All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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 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.