[U-Boot] [PATCH] da850evm: basic MII EMAC support

Ben Gardiner bengardiner at nanometrics.ca
Mon Sep 13 14:55:47 CEST 2010


Hi Mike,

On Sat, Sep 11, 2010 at 12:01 AM, Mike Frysinger <vapier at gentoo.org> wrote:
> On Friday, September 10, 2010 16:10:16 Ben Gardiner wrote:
>> The current da850evm support in u-boot/master [1] omits any use of
>> the davinci EMAC. This patch adds basic support for the EMAC using
>> the MII PHY found on the baseboard of the EVM. The MAC address is
>> read from the environment variable 'ethadd'. Note that this is
>> different from the da850evm support in the u-boot omapl1 tree
>> where the MAC address is read from SPI flash [2].
>>
>> CC: Sandeep Paulraj <s-paulraj at ti.com>
>> CC: Sudhakar Rajashekhara <sudhakar.raj at ti.com>
>
> where is your sign-off-by ?  patches cant be merged without it.

Right. My mistake, sorry.

>> [1] http://git.denx.de/?p=u-boot.git
>> [2] http://arago-project.org/git/projects/?p=u-boot-omapl1.git
>
> these URLs are useless.  just drop them.

Thanks for the feedback on these -- I have been wondering whether URL
footnotes were desirable in commit messages. I put them there since I
can remember that it is sometimes difficult to find the trees that are
being referred to when you are a newcomer.  Would SHA1's and/or git
URLs be more acceptable? Or is the point that nothing before the '---'
should have links?

Best Regards,
Ben Gardiner

---
Nanometrics Inc.
http://www.nanometrics.ca


More information about the U-Boot mailing list