From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Mon, 12 Jul 2010 18:27:24 +0200 Subject: [U-Boot] [PATCH V3 2/5] mv_egiga: support SoCs other than kirkwood In-Reply-To: References: <1278835300-10658-1-git-send-email-albert.aribaud@free.fr> <1278835300-10658-2-git-send-email-albert.aribaud@free.fr> <1278835300-10658-3-git-send-email-albert.aribaud@free.fr> Message-ID: <4C3B426C.7080206@free.fr> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Le 12/07/2010 07:45, Prafulla Wadaskar a ?crit : >> --- a/arch/arm/include/asm/arch-kirkwood/kirkwood.h >> +++ b/arch/arm/include/asm/arch-kirkwood/kirkwood.h >> +#if defined (CONFIG_MV_EGIGA) > > I think you don't need ifdef here > >> +#define MV_EGIGA0_BASE KW_EGIGA0_BASE >> +#define MV_EGIGA1_BASE KW_EGIGA1_BASE >> +#endif Actually the ifdef is intended to make sure these symbols do not get defined uselessly for kirkwood boards that don't have Ethernet support. Granted, their being defined without being used is no big deal. However the whole thing about the egiga driver supporting one or two ports could be streamlined (e.g., one might define KIRKWOOD_EGIGAn_BASE while not enabling port n in CONFIG_KIRKWOOD_EGIGA_PORTS) and integrate the suggestion about the PHYs that occurred some time ago: So I'll leave that one to a later patch set. Amicalement, -- Albert.