Re: [U-Boot] [PATCH V3 2/5] mv_egiga: support SoCs other than kirkwood

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,