[oe] [PATCH] u-boot_git: Ethernet MACID display fix for CraneBoard

Paul Menzel paulepanter at users.sourceforge.net
Fri May 6 08:00:01 UTC 2011


Dear Anil,


thank you for the next version.

Am Freitag, den 06.05.2011, 12:45 +0530 schrieb Anil Kumar:
> CraneBoard u-boot displays the MAC ID wrongly if MAC ID has
>       preceding zeros. This patch fixes it and updates Ethernet
>       NRST GPIO from 34 to 65.

Unfortunately you forgot to add your Signed-off-by line. You do not need
to resend the patch, but just reply to this message with the line.

One question though you did not answer.

1. Did you send this patch upstream?

Some more remarks for the future.

2. Please mark patch iteration with version information. This patch
should have had »[PATCH v2]« in the subject.
3. The commit message does not have to be indented.
4. Please add below the »---« a short information what changed in the
patch iteration, like

v2: add commit message
and finish this by an additional »---«.

> ---
>  ...t-MACID-display-fix-for-am3517-craneboard.patch |  108 ++++++++++++++++++++
>  recipes/u-boot/u-boot_git.bb                       |    3 +-
>  2 files changed, 110 insertions(+), 1 deletions(-)
>  create mode 100644 recipes/u-boot/u-boot-git/am3517-crane/0004-Ethernet-MACID-display-fix-for-am3517-craneboard.patch

[…]

Koen, will you push this or do you want me to do this?


Thank you,

Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20110506/596899bc/attachment-0002.sig>


More information about the Openembedded-devel mailing list