From: Albert ARIBAUD <albert.aribaud@free.fr>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V3 2/5] mv_egiga: support SoCs other than kirkwood
Date: Mon, 12 Jul 2010 17:54:07 +0200 [thread overview]
Message-ID: <4C3B3A9F.5080605@free.fr> (raw)
In-Reply-To: <AANLkTin966hMFkHKkrEOfelnAwKOUSq1dTaV-E_ghqwS@mail.gmail.com>
Le 12/07/2010 16:19, Ben Warren a ?crit :
> I suggest that:
>
> - I switch the file names from mv_egiga to "mvgbe" (to be consistent
> with Prafulla's comment on mv_sata becoming mvsata), and
>
> - I replace mv_egiga/MV_EGIGA symbols with mvgbe/MVGBE.
>
> That will retain (as much) clarity and uniformity (as egiga does),
> which is what I think Ben is looking for, and it'll keep name length
> at a minimum, which should satisfy Prafulla.
>
> Ben, Prafulla (and others as well, of course), do you agree?
>
> OK with me.
Thanks Ben. I assume Prafulla will agree too. I'll regenerate a patch
set with all changes requested by Prafulla, keeping the symbol-change in
a single patch as discussed elsewhere in this thread.
Note that the last patch of the series (edminiv2 support) will only
build above the CONFIG_SKIP_LOCAL_MAC_RANDOMIZATION patch that I posted
separately.
Amicalement,
--
Albert.
next prev parent reply other threads:[~2010-07-12 15:54 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-11 8:01 [U-Boot] [PATCH V3 0/5] Marvell egiga multiple SoC support Albert Aribaud
2010-07-11 8:01 ` [U-Boot] [PATCH V3 1/5] net: rename: kirkwood_egiga as mv_egiga Albert Aribaud
2010-07-11 8:01 ` [U-Boot] [PATCH V3 2/5] mv_egiga: support SoCs other than kirkwood Albert Aribaud
2010-07-11 8:01 ` [U-Boot] [PATCH V3 3/5] mv_egiga: CONFIG_SKIP_LOCAL_MAC_RANDOMIZATION Albert Aribaud
2010-07-11 8:01 ` [U-Boot] [PATCH V3 4/5] mv_egiga: add support for orion5x egiga controller Albert Aribaud
2010-07-11 8:01 ` [U-Boot] [PATCH V3 5/5] edminiv2: add ethernet support Albert Aribaud
2010-07-12 6:00 ` Prafulla Wadaskar
2010-07-12 6:14 ` Albert ARIBAUD
2010-07-12 6:31 ` Ben Warren
2010-07-12 6:05 ` [U-Boot] [PATCH V3 4/5] mv_egiga: add support for orion5x egiga controller Prafulla Wadaskar
2010-07-12 5:53 ` [U-Boot] [PATCH V3 3/5] mv_egiga: CONFIG_SKIP_LOCAL_MAC_RANDOMIZATION Prafulla Wadaskar
2010-07-12 5:45 ` [U-Boot] [PATCH V3 2/5] mv_egiga: support SoCs other than kirkwood Prafulla Wadaskar
2010-07-12 6:23 ` Ben Warren
2010-07-12 6:53 ` Prafulla Wadaskar
2010-07-12 8:53 ` Albert ARIBAUD
2010-07-12 14:19 ` Ben Warren
2010-07-12 15:54 ` Albert ARIBAUD [this message]
2010-07-12 16:27 ` Albert ARIBAUD
2010-07-12 5:49 ` Prafulla Wadaskar
2010-07-12 8:56 ` Albert ARIBAUD
2010-07-12 14:23 ` Ben Warren
2010-07-13 5:38 ` Prafulla Wadaskar
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=4C3B3A9F.5080605@free.fr \
--to=albert.aribaud@free.fr \
--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.