From mboxrd@z Thu Jan 1 00:00:00 1970 From: Philip Balister Date: Tue, 13 Sep 2011 09:54:44 -0400 Subject: [U-Boot] [PATCH v2] OMAP3: Overo: Update GPMC timing for ethernet chip. In-Reply-To: <4E6CE3F6.4080206@balister.org> References: <1315524373-18336-1-git-send-email-philip@opensdr.com> <4E6CE3F6.4080206@balister.org> Message-ID: <4E6F60A4.2010906@balister.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 09/11/2011 12:38 PM, Philip Balister wrote: > I see I should have copied a maintainer on this patch. Which tree will > this sort of path go upstream via? It seems like my choices are ARM of > OMAP3. > > I've got a number of other patches that should go upstream that are > Overo centric. In general, what route should they take upstream? Ping? Any tips for getting patched committed would be helpful, I'm sure I'm doing something wrong. Philip > > Philip > > On 09/09/2011 03:16 AM, Enric Balletb? i Serra wrote: >> 2011/9/9 Philip Balister: >>> The existing timing does not quite meet the minimum requirements >>> in the LAN9221 datasheet. The timing in this patch solves problems >>> noticed on some parts. The patch also combines the CS configuration >>> for the overo and igep0020 boards per request. >>> >>> Signed-off-by: Philip Balister >>> --- >>> Changes for v2: Added igep0020 board per Enric Balletb? i Serra. Compile >>> tested only. >>> >>> arch/arm/include/asm/arch-omap3/omap_gpmc.h | 9 +++++++++ >>> board/isee/igep0020/igep0020.c | 1 + >>> board/isee/igep0020/igep0020.h | 9 --------- >>> board/overo/overo.c | 1 + >>> board/overo/overo.h | 9 --------- >>> 5 files changed, 11 insertions(+), 18 deletions(-) >>> >>> diff --git a/arch/arm/include/asm/arch-omap3/omap_gpmc.h b/arch/arm/include/asm/arch-omap3/omap_gpmc.h >>> index bd22bce..800e4ee 100644 >>> --- a/arch/arm/include/asm/arch-omap3/omap_gpmc.h >>> +++ b/arch/arm/include/asm/arch-omap3/omap_gpmc.h >>> @@ -80,4 +80,13 @@ >>> } >>> #endif >>> >>> +/* GPMC CS configuration for an SMSC LAN9221 ethernet controller */ >>> +#define NET_LAN9221_GPMC_CONFIG1 0x00001000 >>> +#define NET_LAN9221_GPMC_CONFIG2 0x00060700 >>> +#define NET_LAN9221_GPMC_CONFIG3 0x00020201 >>> +#define NET_LAN9221_GPMC_CONFIG4 0x06000700 >>> +#define NET_LAN9221_GPMC_CONFIG5 0x0006090A >>> +#define NET_LAN9221_GPMC_CONFIG6 0x87030000 >>> +#define NET_LAN9221_GPMC_CONFIG7 0x00000f6c >>> + >>> #endif /* __ASM_ARCH_OMAP_GPMC_H */ >>> diff --git a/board/isee/igep0020/igep0020.c b/board/isee/igep0020/igep0020.c >>> index 36cc924..a5210c9 100644 >>> --- a/board/isee/igep0020/igep0020.c >>> +++ b/board/isee/igep0020/igep0020.c >>> @@ -29,6 +29,7 @@ >>> #include >>> #include >>> #include >>> +#include >>> #include >>> #include "igep0020.h" >>> >>> diff --git a/board/isee/igep0020/igep0020.h b/board/isee/igep0020/igep0020.h >>> index c08d758..3d6e15f 100644 >>> --- a/board/isee/igep0020/igep0020.h >>> +++ b/board/isee/igep0020/igep0020.h >>> @@ -29,15 +29,6 @@ const omap3_sysinfo sysinfo = { >>> "ONENAND", >>> }; >>> >>> -/* GPMC CS 5 connected to an SMSC LAN9221 ethernet controller */ >>> -#define NET_LAN9221_GPMC_CONFIG1 0x00001000 >>> -#define NET_LAN9221_GPMC_CONFIG2 0x00080701 >>> -#define NET_LAN9221_GPMC_CONFIG3 0x00020201 >>> -#define NET_LAN9221_GPMC_CONFIG4 0x08030703 >>> -#define NET_LAN9221_GPMC_CONFIG5 0x00060908 >>> -#define NET_LAN9221_GPMC_CONFIG6 0x87030000 >>> -#define NET_LAN9221_GPMC_CONFIG7 0x00000f6c >>> - >>> static void setup_net_chip(void); >>> >>> /* >>> diff --git a/board/overo/overo.c b/board/overo/overo.c >>> index 4eafdb1..bcd7d62 100644 >>> --- a/board/overo/overo.c >>> +++ b/board/overo/overo.c >>> @@ -37,6 +37,7 @@ >>> #include >>> #include >>> #include >>> +#include >>> #include >>> #include "overo.h" >>> >>> diff --git a/board/overo/overo.h b/board/overo/overo.h >>> index 68e1243..57f2457 100644 >>> --- a/board/overo/overo.h >>> +++ b/board/overo/overo.h >>> @@ -33,15 +33,6 @@ const omap3_sysinfo sysinfo = { >>> #endif >>> }; >>> >>> -/* GPMC CS 5 connected to an SMSC LAN9221 ethernet controller */ >>> -#define NET_LAN9221_GPMC_CONFIG1 0x00001000 >>> -#define NET_LAN9221_GPMC_CONFIG2 0x00080701 >>> -#define NET_LAN9221_GPMC_CONFIG3 0x00020201 >>> -#define NET_LAN9221_GPMC_CONFIG4 0x08030703 >>> -#define NET_LAN9221_GPMC_CONFIG5 0x00060908 >>> -#define NET_LAN9221_GPMC_CONFIG6 0x87030000 >>> -#define NET_LAN9221_GPMC_CONFIG7 0x00000f6c >>> - >>> /* >>> * IEN - Input Enable >>> * IDIS - Input Disable >>> -- >>> 1.7.4.4 >>> >>> >> >> Tested on igep0020 machine (aka IGEP v2). Thanks Philip >> >> Acked-by: Enric Balletbo i Serra >> Tested-by: Enric Balletbo i Serra >> _______________________________________________ >> U-Boot mailing list >> U-Boot at lists.denx.de >> http://lists.denx.de/mailman/listinfo/u-boot > > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot