Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale (ICache issue)

2010-08-10 Thread Ilya Yanok
Hi Kim, 20.07.2010 4:33, Kim Phillips wrote: > turning icache on early causes a different pattern of bus accesses when > fetching code, and this might trigger bus accesses that interfere with > code that sets bus configuration registers, such as the acr. Can you > test this patch please?: This

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-07-20 Thread Ilya Yanok
Hi Kim, 20.07.2010 4:33, Kim Phillips wrote: >>> Thanks for your advice but this doesn't help... > turning icache on early causes a different pattern of bus accesses when > fetching code, and this might trigger bus accesses that interfere with > code that sets bus configuration registers, such a

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-07-19 Thread Wolfgang Denk
Dear Kim Phillips, In message <20100719193356.a02add7e.kim.phill...@freescale.com> you wrote: > > +++ b/arch/powerpc/cpu/mpc83xx/acr.c > @@ -0,0 +1,29 @@ > +/* > + * Copyright (C) 2010 Freescale Semiconductor, Inc. > + * > + * See file CREDITS for list of people who contributed to this > + * proje

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-07-19 Thread Kim Phillips
On Thu, 24 Jun 2010 19:00:54 -0500 Kim Phillips wrote: > On Thu, 24 Jun 2010 23:36:49 +0400 > Ilya Yanok wrote: > > > Hi Kim, > > > > On 24.06.2010 22:00, Kim Phillips wrote: > > >> I've enabled icache and now the board sometimes resets twice after > > >> U-Boot 'reset' command: > > >> >

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-07-01 Thread Ilya Yanok
Hi Kim, On 01.07.2010 04:30, Kim Phillips wrote: >> This patch provides support for MPC8308ERDB development board from >> > so, according to this document: > > http://cache.freescale.com/files/32bit/doc/user_guide/MPC8308RDBUG.pdf > > the board name is just 'mpc8308rdb', not 'mpc8308erdb'.

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-30 Thread Kim Phillips
On Mon, 28 Jun 2010 16:45:29 +0400 Ilya Yanok wrote: > This patch provides support for MPC8308ERDB development board from so, according to this document: http://cache.freescale.com/files/32bit/doc/user_guide/MPC8308RDBUG.pdf the board name is just 'mpc8308rdb', not 'mpc8308erdb'. AFAICT, ther

[U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-28 Thread Ilya Yanok
This patch provides support for MPC8308ERDB development board from Freescale with a minimal set of features: Dual UART is supported NOR flash is supported Both TSEC Ethernet controllers are supported PCI Express initialization is supported The following features are enabled in configuration bu

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-24 Thread Aggrwal Poonam-B10812
> -Original Message- > From: u-boot-boun...@lists.denx.de [mailto:u-boot-boun...@lists.denx.de] > On Behalf Of Ilya Yanok > Sent: Friday, June 25, 2010 1:07 AM > To: Phillips Kim-R1AAHA > Cc: u-boot@lists.denx.de > Subject: Re: [U-Boot] [PATCH 2/2] MPC8308ERDB

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-24 Thread Ilya Yanok
Hi Kim, On 24.06.2010 22:00, Kim Phillips wrote: >> I've enabled icache and now the board sometimes resets twice after >> U-Boot 'reset' command: >> > >> (It doesn't always stop at "DRAM:" line but that position is most frequent) >> Maybe you have some ideas on this subject? >> > h

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-24 Thread Kim Phillips
On Thu, 24 Jun 2010 19:59:13 +0400 Ilya Yanok wrote: > On 23.06.2010 04:17, Kim Phillips wrote: > >> +#define CONFIG_SYS_HID0_FINAL (HID0_ENABLE_MACHINE_CHECK | \ > >> + HID0_ENABLE_DYNAMIC_POWER_MANAGMENT) > >> > > | HID0_ENABLE_INSTRUCTION_CACHE? > > I've en

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-24 Thread Ilya Yanok
Dear Kim, On 23.06.2010 04:17, Kim Phillips wrote: >> +#define CONFIG_SYS_HID0_FINAL (HID0_ENABLE_MACHINE_CHECK | \ >> + HID0_ENABLE_DYNAMIC_POWER_MANAGMENT) >> > | HID0_ENABLE_INSTRUCTION_CACHE? > I've enabled icache and now the board sometimes resets

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-23 Thread Wolfgang Denk
Dear Ilya Yanok, In message <4c227cd8.2080...@emcraft.com> you wrote: > > > missing a doc/README.mpc8308erdb file. > > Hm.. Wolfgang, do we really need this? Only if there is somethign spoecial about this board which is worth being documented. Best regards, Wolfgang Denk -- DENX Software En

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-23 Thread Ilya Yanok
Hi Kim, Thanks for your review. On 23.06.2010 04:17, Kim Phillips wrote: >> MAKEALL |1 + >> Makefile |3 + >> board/freescale/mpc8308erdb/Makefile | 52 +++ >> board/freescale/mpc8308erdb/config.mk |1

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-23 Thread Ilya Yanok
Hi Ben, On 22.06.2010 23:10, Ben Warren wrote: > +int board_eth_init(bd_t *bis) > +{ > +cpu_eth_init(bis);/* Initialize TSECs first */ > I think it's wrong to ignore the return code here. >>> What makes you think so? What can we do with the return code here? >>> Print

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-23 Thread Ilya Yanok
Hi Wolfgang, On 22.06.2010 22:14, Wolfgang Denk wrote: >>> Entry to MAINTAINERS missing. >>> >> Should I add you as a maintainer or myself? >> > You did the actual work... > but I fear I won't be able to maintain it... I don't have much free time to spend now... >>> + return

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-22 Thread Kim Phillips
On Sun, 20 Jun 2010 21:32:48 +0400 Ilya Yanok wrote: > This patch provides support for MPC8308ERDB development board from > Freescale with a minimal set of features: > Dual UART is supported > NOR flash is supported > Both TSEC Ethernet controllers are supported > PCI Express initialization i

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-22 Thread Ben Warren
Wolfgang & Ilya, Sorry for responding to both of you at the same time... On 6/22/2010 11:14 AM, Wolfgang Denk wrote: > Dear Ilya Yanok, > > In message<4c1f5a54.4050...@emcraft.com> you wrote: > >> >>> Entry to MAINTAINERS missing. >>> >> Should I add you as a maintainer or myse

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-22 Thread Wolfgang Denk
Dear Ilya Yanok, In message <4c1f5a54.4050...@emcraft.com> you wrote: > > > Entry to MAINTAINERS missing. > > Should I add you as a maintainer or myself? You did the actual work... > >> +int board_eth_init(bd_t *bis) > >> +{ > >> + cpu_eth_init(bis); /* Initialize TSECs first */ > >>

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-21 Thread Ilya Yanok
Dear Wolfgang, On 21.06.2010 11:44, Wolfgang Denk wrote: >> MAKEALL |1 + >> Makefile |3 + >> board/freescale/mpc8308erdb/Makefile | 52 +++ >> board/freescale/mpc8308erdb/config.mk |1 + >> board/free

Re: [U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-21 Thread Wolfgang Denk
Dear Ilya Yanok, In message <1277055168-18596-3-git-send-email-ya...@emcraft.com> you wrote: > This patch provides support for MPC8308ERDB development board from > Freescale with a minimal set of features: > Dual UART is supported > NOR flash is supported > Both TSEC Ethernet controllers are su

[U-Boot] [PATCH 2/2] MPC8308ERDB: minimal support for devboard from Freescale

2010-06-20 Thread Ilya Yanok
This patch provides support for MPC8308ERDB development board from Freescale with a minimal set of features: Dual UART is supported NOR flash is supported Both TSEC Ethernet controllers are supported PCI Express initialization is supported The following features are enabled in configuration bu