Re: [U-Boot] [PATCH] 440spe MQ initialization
Hi Wolfgang, On Friday 17 October 2008, Wolfgang Denk wrote: > > So it seems that the current code relies on some already preset (default) > > values in those registers. I don't really like this. I'll try to provide > > a patch that configures those registers completely in a short while. > > "In a short while" means very few hours? Yes. I already tested with Yuri's patch and it doesn't change the resulting register values. So the current code already sets the correct values, it just doesn't set all bits explicitly. I'll provide another patch in a few minutes to set these bits explicitly, since this makes it clearer which bits are really configured. > If Yuri is sopposed to test > this, we have little time left. The release is scheduled for > tomorrow. Understood. He can test with the current U-Boot version without setting the MQ registers in the DMA/RAID Linux driver. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
Dear Stefan Roese, In message <[EMAIL PROTECTED]> you wrote: > > So it seems that the current code relies on some already preset (default) > values in those registers. I don't really like this. I'll try to provide a > patch that configures those registers completely in a short while. "In a short while" means very few hours? If Yuri is sopposed to test this, we have little time left. The release is scheduled for tomorrow. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] When the bosses talk about improving productivity, they are never talking about themselves. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
Hi Wolfgang, On Friday 17 October 2008, Wolfgang Denk wrote: > > Set the MQ Read Passing & MCIF Cycle limits to the recommended by AMCC > > values. This fixes the occasional 440SPe hard locking issues when the > > 440SPe's dedicated DMA engines are used (e.g. by the h/w accelerated RAID > > driver). > > > > Previously the appropriate initialization had been made in Linux, by the > > ppc440spe ADMA driver, which is wrong because modifying the MQ > > configuration registers after normal operation has begun is not supported > > and could have unpredictable results. > > > > Signed-off-by: Yuri Tikhonov <[EMAIL PROTECTED]> > > --- > > cpu/ppc4xx/44x_spd_ddr2.c | 10 ++ > > include/asm-ppc/ppc4xx-sdram.h |5 + > > 2 files changed, 11 insertions(+), 4 deletions(-) > > I must admit that I lost track in the discussion following this > posting what the real state of affairs is now. Do we need to change > anything in U-Boot, or not, and why not? I also lost track of whats really needed, I'm afraid. That's why I asked about this a few weeks ago. Here an extract from the last mail: On Thursday 25 September 2008, Prodyut Hazarika wrote: > > So please let me know if Yuri's original MQ patch should be applied and > > if there is another PLB arbiter patch that should be applied too. > > Yuri's patch does the following > - set RLPM (read passing limit) bits in MQ register to 1, > - set WRCL (MCIF cycle limit) in MQ registers to 1. > In my testing, as far as I can remember, these values were set to 1 by > default. I had tested with Canyonlands, Kilauea, Katmai and Glacier boards. > So unless Yuri found that these values were not default values in some > processor, Yuri's patch should be applied. So it seems that the current code relies on some already preset (default) values in those registers. I don't really like this. I'll try to provide a patch that configures those registers completely in a short while. It would be great if Yuri could test this on a target to see if we are still missing something. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
Dear Stefan, In message <[EMAIL PROTECTED]> Yuri Tikhonov wrote: > > Set the MQ Read Passing & MCIF Cycle limits to the recommended by AMCC > values. This fixes the occasional 440SPe hard locking issues when the 440SPe's > dedicated DMA engines are used (e.g. by the h/w accelerated RAID driver). > > Previously the appropriate initialization had been made in Linux, by the > ppc440spe ADMA driver, which is wrong because modifying the MQ configuration > registers after normal operation has begun is not supported and could > have unpredictable results. > > Signed-off-by: Yuri Tikhonov <[EMAIL PROTECTED]> > --- > cpu/ppc4xx/44x_spd_ddr2.c | 10 ++ > include/asm-ppc/ppc4xx-sdram.h |5 + > 2 files changed, 11 insertions(+), 4 deletions(-) I must admit that I lost track in the discussion following this posting what the real state of affairs is now. Do we need to change anything in U-Boot, or not, and why not? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED] The ideal situation is to have massive computing power right at home. Something that dims the streetlights and shrinks the picture on the neighbours' TVs when you boot it up. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
> I have to admit that I am a little confused now. Are there now any patches > that should be applied to U-Boot regarding the MQ and/or the PLB arbiter > configuration? There is no patch that needs to be applied for PLB arbiter configuration. My earlier patch covered that. > It seems to me that Yuri's configuration from the RAID driver > (which will be dropped in the RAID driver soon) is not fully covered in the > current U-Boot version. Yuri's configuration from RAID driver should be droppred coz changing MQ/PLB4 registers from Linux is not recommended > So please let me know if Yuri's original MQ patch should be applied and if > there is another PLB arbiter patch that should be applied too. Yuri's patch does the following - set RLPM (read passing limit) bits in MQ register to 1, - set WRCL (MCIF cycle limit) in MQ registers to 1. In my testing, as far as I can remember, these values were set to 1 by default. I had tested with Canyonlands, Kilauea, Katmai and Glacier boards. So unless Yuri found that these values were not default values in some processor, Yuri's patch should be applied. Thanks, Prodyut ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
On Tuesday 23 September 2008, Yuri Tikhonov wrote: > >> Is it OK, or should we remove these strings from the Linux driver, > >> assuming U-Boot has already done this ? > > > > Please go ahead and remove these from the linux driver. > > Thanks for confirmation, will do. I have to admit that I am a little confused now. Are there now any patches that should be applied to U-Boot regarding the MQ and/or the PLB arbiter configuration? It seems to me that Yuri's configuration from the RAID driver (which will be dropped in the RAID driver soon) is not fully covered in the current U-Boot version. So please let me know if Yuri's original MQ patch should be applied and if there is another PLB arbiter patch that should be applied too. Thanks. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
Hi Prodyut, On Tuesday, September 23, 2008 you wrote: > Hi Yuri, >> Is it OK, or should we remove these strings from the Linux driver, >> assuming U-Boot has already done this ? > Please go ahead and remove these from the linux driver. Thanks for confirmation, will do. > And I think this driver has not been submitted to the powerpc tree, since I > still don't see it in Josh's or Linus's tree. Yep, for now it's present in the linux-2.6-denx tree only. We are about to post this to MLs along with our RAID-6 related changes soon. >> From: Stefan Roese [mailto:[EMAIL PROTECTED] >> Sent: Tue 9/23/2008 2:43 AM >> To: Yuri Tikhonov >> Cc: u-boot@lists.denx.de; Prodyut Hazarika; Olga Buchonina >> Subject: Re: [U-Boot] [PATCH] 440spe MQ initialization >> On Tuesday 23 September 2008, Yuri Tikhonov wrote: >>> BTW, when I said "recommended by AMCC" in the patch description I >>> referred to the following information forwarded to me by Wolfgang >>> Denk on Tue Mar 18 2008: >>> >>> --- >>> Dear Yuri, >>> >>> here is some additional (and hopefully helpful) information from AMCC >>> >>> regarding the observed hangs on the katmai board: >>> > If possible, can you please check if you still see the lock up when >>> > you program MQ as follows: >>> > set value in HB and if you are using LL also as follows: >>> > MQ0_CF1H = 0x80001C80 >>> > MQ0_CF1L = 0x80001C80 >>> > Additionally, make sure that your PLB settings are: >>> > PLB0_ACR = 0xDF00 ( 4 deep read and 2 deep write) >>> > PLB1_ACR = 0xDF00 ( 4 deep read and 2 deep write) >>> > Please let me know if this fixes the issue. >>> > I also would like to know how you are programming your DMA and how >>> > is the traffic is pipelined. >>> > Regards, >>> > Olga Buchonina >>> > AMCC PowerPC Applications Engineering >> Understood. I just would like to see an ACK from AMCC on this since they just >> updated this MQ init code. >> Best regards, >> Stefan >> = >> DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel >> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany >> Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] >> = > -- > Yuri Tikhonov, Senior Software Engineer > Emcraft Systems, www.emcraft.com Regards, Yuri -- Yuri Tikhonov, Senior Software Engineer Emcraft Systems, www.emcraft.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
Hi Yuri, > Is it OK, or should we remove these strings from the Linux driver, > assuming U-Boot has already done this ? Please go ahead and remove these from the linux driver. And I think this driver has not been submitted to the powerpc tree, since I still don't see it in Josh's or Linus's tree. Thanks, Prodyut > From: Stefan Roese [mailto:[EMAIL PROTECTED] > Sent: Tue 9/23/2008 2:43 AM > To: Yuri Tikhonov > Cc: u-boot@lists.denx.de; Prodyut Hazarika; Olga Buchonina > Subject: Re: [U-Boot] [PATCH] 440spe MQ initialization > On Tuesday 23 September 2008, Yuri Tikhonov wrote: >> BTW, when I said "recommended by AMCC" in the patch description I >> referred to the following information forwarded to me by Wolfgang >> Denk on Tue Mar 18 2008: >> >> --- >> Dear Yuri, >> >> here is some additional (and hopefully helpful) information from AMCC >> >> regarding the observed hangs on the katmai board: >> > If possible, can you please check if you still see the lock up when >> > you program MQ as follows: >> > set value in HB and if you are using LL also as follows: >> > MQ0_CF1H = 0x80001C80 >> > MQ0_CF1L = 0x80001C80 >> > Additionally, make sure that your PLB settings are: >> > PLB0_ACR = 0xDF00 ( 4 deep read and 2 deep write) >> > PLB1_ACR = 0xDF00 ( 4 deep read and 2 deep write) >> > Please let me know if this fixes the issue. >> > I also would like to know how you are programming your DMA and how >> > is the traffic is pipelined. >> > Regards, >> > Olga Buchonina >> > AMCC PowerPC Applications Engineering > Understood. I just would like to see an ACK from AMCC on this since they just > updated this MQ init code. > Best regards, > Stefan > = > DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] > = -- Yuri Tikhonov, Senior Software Engineer Emcraft Systems, www.emcraft.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
Hello Prodyut, On Tuesday, September 23, 2008 you wrote: > Hi Stefan/Yuri, > I looked at the changes. > I had already submitted the changes in an earlier patch. This code > to set the PLB4 arbiter depth was moved to cpu_init_f function in > cpu/ppc4xx/cpu_init.c since it is common across many processor families. > > Please don't do the same thing again in the program_memory_queue function. Well, actually my patch doesn't have a deal with the PLB configuring, but with MQ only. So, it's OK here. But then another question arises: in Linux, in ppc440spe_configure_raid_devices(), beside the other things we do the following: /* Configure PLB as follows: * PLB: 0xDF00. This means * - Priority level 00 fair priority, * - Priority level 01 fair priority, * - Priority level 11 fair priority, * - High Bus Utilization enabled, * - 4 Deep read pipe, * - 2 Deep write pipe. */ mask = (1 << PLB_ACR_PPM0) | (1 << PLB_ACR_PPM1) | (1 << PLB_ACR_PPM3) | (1 << PLB_ACR_HBU) | ((3 & PLB_ACR_RDP_MSK) << PLB_ACR_RDP) | (1 << PLB_ACR_WRP); mtdcr(DCRN_PLB0_ACR, mask); mtdcr(DCRN_PLB1_ACR, mask); Is it OK, or should we remove these strings from the Linux driver, assuming U-Boot has already done this ? Thanks in advance, Yuri > From: Stefan Roese [mailto:[EMAIL PROTECTED] > Sent: Tue 9/23/2008 2:43 AM > To: Yuri Tikhonov > Cc: u-boot@lists.denx.de; Prodyut Hazarika; Olga Buchonina > Subject: Re: [U-Boot] [PATCH] 440spe MQ initialization > On Tuesday 23 September 2008, Yuri Tikhonov wrote: >> BTW, when I said "recommended by AMCC" in the patch description I >> referred to the following information forwarded to me by Wolfgang >> Denk on Tue Mar 18 2008: >> >> --- >> Dear Yuri, >> >> here is some additional (and hopefully helpful) information from AMCC >> >> regarding the observed hangs on the katmai board: >> > If possible, can you please check if you still see the lock up when >> > you program MQ as follows: >> > set value in HB and if you are using LL also as follows: >> > MQ0_CF1H = 0x80001C80 >> > MQ0_CF1L = 0x80001C80 >> > Additionally, make sure that your PLB settings are: >> > PLB0_ACR = 0xDF00 ( 4 deep read and 2 deep write) >> > PLB1_ACR = 0xDF00 ( 4 deep read and 2 deep write) >> > Please let me know if this fixes the issue. >> > I also would like to know how you are programming your DMA and how >> > is the traffic is pipelined. >> > Regards, >> > Olga Buchonina >> > AMCC PowerPC Applications Engineering > Understood. I just would like to see an ACK from AMCC on this since they just > updated this MQ init code. > Best regards, > Stefan > = > DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] > = -- Yuri Tikhonov, Senior Software Engineer Emcraft Systems, www.emcraft.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
Hi Yuri/Stefan, I remember duriing my testing that the default Read passing limit (RPML) and MCIF limit (WRCL) was 1. So there was no need to set these registers again to the same values. Thanks, Prodyut : Tue 9/23/2008 2:43 AM To: Yuri Tikhonov Cc: u-boot@lists.denx.de; Prodyut Hazarika; Olga Buchonina Subject: Re: [U-Boot] [PATCH] 440spe MQ initialization On Tuesday 23 September 2008, Yuri Tikhonov wrote: > BTW, when I said "recommended by AMCC" in the patch description I > referred to the following information forwarded to me by Wolfgang > Denk on Tue Mar 18 2008: > > --- > Dear Yuri, > > here is some additional (and hopefully helpful) information from AMCC > > regarding the observed hangs on the katmai board: > > If possible, can you please check if you still see the lock up when > > you program MQ as follows: > > set value in HB and if you are using LL also as follows: > > MQ0_CF1H = 0x80001C80 > > MQ0_CF1L = 0x80001C80 > > Additionally, make sure that your PLB settings are: > > PLB0_ACR = 0xDF00 ( 4 deep read and 2 deep write) > > PLB1_ACR = 0xDF00 ( 4 deep read and 2 deep write) > > Please let me know if this fixes the issue. > > I also would like to know how you are programming your DMA and how > > is the traffic is pipelined. > > Regards, > > Olga Buchonina > > AMCC PowerPC Applications Engineering Understood. I just would like to see an ACK from AMCC on this since they just updated this MQ init code. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
Hi Stefan/Yuri, I looked at the changes. I had already submitted the changes in an earlier patch. This code to set the PLB4 arbiter depth was moved to cpu_init_f function in cpu/ppc4xx/cpu_init.c since it is common across many processor families. Please don't do the same thing again in the program_memory_queue function. Thanks, Prodyut From: Stefan Roese [mailto:[EMAIL PROTECTED] Sent: Tue 9/23/2008 2:43 AM To: Yuri Tikhonov Cc: u-boot@lists.denx.de; Prodyut Hazarika; Olga Buchonina Subject: Re: [U-Boot] [PATCH] 440spe MQ initialization On Tuesday 23 September 2008, Yuri Tikhonov wrote: > BTW, when I said "recommended by AMCC" in the patch description I > referred to the following information forwarded to me by Wolfgang > Denk on Tue Mar 18 2008: > > --- > Dear Yuri, > > here is some additional (and hopefully helpful) information from AMCC > > regarding the observed hangs on the katmai board: > > If possible, can you please check if you still see the lock up when > > you program MQ as follows: > > set value in HB and if you are using LL also as follows: > > MQ0_CF1H = 0x80001C80 > > MQ0_CF1L = 0x80001C80 > > Additionally, make sure that your PLB settings are: > > PLB0_ACR = 0xDF00 ( 4 deep read and 2 deep write) > > PLB1_ACR = 0xDF00 ( 4 deep read and 2 deep write) > > Please let me know if this fixes the issue. > > I also would like to know how you are programming your DMA and how > > is the traffic is pipelined. > > Regards, > > Olga Buchonina > > AMCC PowerPC Applications Engineering Understood. I just would like to see an ACK from AMCC on this since they just updated this MQ init code. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
Yuriy/Stefan- This is AMCC confirmation about configuration. Regards, Olga Buchonina AMCC PowerPC Applications Engineering -Original Message- From: Stefan Roese [mailto:[EMAIL PROTECTED] Sent: Tuesday, September 23, 2008 4:43 AM To: Yuri Tikhonov Cc: u-boot@lists.denx.de; Prodyut Hazarika; Olga Buchonina Subject: Re: [U-Boot] [PATCH] 440spe MQ initialization On Tuesday 23 September 2008, Yuri Tikhonov wrote: > BTW, when I said "recommended by AMCC" in the patch description I > referred to the following information forwarded to me by Wolfgang Denk > on Tue Mar 18 2008: > > --- > Dear Yuri, > > here is some additional (and hopefully helpful) information from AMCC > > regarding the observed hangs on the katmai board: > > If possible, can you please check if you still see the lock up when > > you program MQ as follows: > > set value in HB and if you are using LL also as follows: > > MQ0_CF1H = 0x80001C80 > > MQ0_CF1L = 0x80001C80 > > Additionally, make sure that your PLB settings are: > > PLB0_ACR = 0xDF00 ( 4 deep read and 2 deep write) PLB1_ACR = > > 0xDF00 ( 4 deep read and 2 deep write) Please let me know if > > this fixes the issue. > > I also would like to know how you are programming your DMA and how > > is the traffic is pipelined. > > Regards, > > Olga Buchonina > > AMCC PowerPC Applications Engineering Understood. I just would like to see an ACK from AMCC on this since they just updated this MQ init code. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = CONFIDENTIALITY NOTICE: This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and contains information that is confidential and proprietary to Applied Micro Circuits Corporation or its subsidiaries. It is to be used solely for the purpose of furthering the parties' business relationship. All unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
On Tuesday 23 September 2008, Yuri Tikhonov wrote: > BTW, when I said "recommended by AMCC" in the patch description I > referred to the following information forwarded to me by Wolfgang > Denk on Tue Mar 18 2008: > > --- > Dear Yuri, > > here is some additional (and hopefully helpful) information from AMCC > > regarding the observed hangs on the katmai board: > > If possible, can you please check if you still see the lock up when > > you program MQ as follows: > > set value in HB and if you are using LL also as follows: > > MQ0_CF1H = 0x80001C80 > > MQ0_CF1L = 0x80001C80 > > Additionally, make sure that your PLB settings are: > > PLB0_ACR = 0xDF00 ( 4 deep read and 2 deep write) > > PLB1_ACR = 0xDF00 ( 4 deep read and 2 deep write) > > Please let me know if this fixes the issue. > > I also would like to know how you are programming your DMA and how > > is the traffic is pipelined. > > Regards, > > Olga Buchonina > > AMCC PowerPC Applications Engineering Understood. I just would like to see an ACK from AMCC on this since they just updated this MQ init code. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
Hello, BTW, when I said "recommended by AMCC" in the patch description I referred to the following information forwarded to me by Wolfgang Denk on Tue Mar 18 2008: --- Dear Yuri, here is some additional (and hopefully helpful) information from AMCC regarding the observed hangs on the katmai board: > If possible, can you please check if you still see the lock up when > you program MQ as follows: > set value in HB and if you are using LL also as follows: > MQ0_CF1H = 0x80001C80 > MQ0_CF1L = 0x80001C80 > Additionally, make sure that your PLB settings are: > PLB0_ACR = 0xDF00 ( 4 deep read and 2 deep write) > PLB1_ACR = 0xDF00 ( 4 deep read and 2 deep write) > Please let me know if this fixes the issue. > I also would like to know how you are programming your DMA and how > is the traffic is pipelined. > Regards, > Olga Buchonina > AMCC PowerPC Applications Engineering Best regards, Wolfgang Denk --- On Tuesday, September 23, 2008 you wrote: > Hi Yuri, > On Tuesday 23 September 2008, Yuri Tikhonov wrote: >> Set the MQ Read Passing & MCIF Cycle limits to the recommended by AMCC >> values. This fixes the occasional 440SPe hard locking issues when the >> 440SPe's dedicated DMA engines are used (e.g. by the h/w accelerated RAID >> driver). >> >> Previously the appropriate initialization had been made in Linux, by the >> ppc440spe ADMA driver, which is wrong because modifying the MQ >> configuration registers after normal operation has begun is not supported >> and could have unpredictable results. > AMCC just recently updated the 440SP(e) MQ initialization with this patch: > commit 079589bcfb24ba11068460276a3cc9549ab5346f > Author: Prodyut Hazarika <[EMAIL PROTECTED]> > Date: Wed Aug 20 09:38:51 2008 -0700 > ppc4xx: Optimize PLB4 Arbiter and Memory Queue settings for PPC440SP/SPe, > PPC405EX and PPC460EX/GT/SX > - Read pipeline depth set to 4 for PPC440SP/SPE, PPC405EX, PPC460EX/GT/SX > processors > - Moved PLB4 Arbiter register definitions to ppc4xx.h since it is shared > across processors (405 and 440/460) > - Optimize Memory Queue settings for PPC440SP/SPE and PPC460EX/GT/SX > processors > - Add register bit definitions for Memory Queue Configuration registers > Signed-off-by: Prodyut Hazarika <[EMAIL PROTECTED]> > Signed-off-by: Stefan Roese <[EMAIL PROTECTED]> > I have a bad feeling changing this "optimized" settings without AMCC's > specific ACK. > Prodyut, are you ok with Yuri's change? > Thanks. > Best regards, > Stefan > = > DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] > = Regards, Yuri -- Yuri Tikhonov, Senior Software Engineer Emcraft Systems, www.emcraft.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] 440spe MQ initialization
Hi Yuri, On Tuesday 23 September 2008, Yuri Tikhonov wrote: > Set the MQ Read Passing & MCIF Cycle limits to the recommended by AMCC > values. This fixes the occasional 440SPe hard locking issues when the > 440SPe's dedicated DMA engines are used (e.g. by the h/w accelerated RAID > driver). > > Previously the appropriate initialization had been made in Linux, by the > ppc440spe ADMA driver, which is wrong because modifying the MQ > configuration registers after normal operation has begun is not supported > and could have unpredictable results. AMCC just recently updated the 440SP(e) MQ initialization with this patch: commit 079589bcfb24ba11068460276a3cc9549ab5346f Author: Prodyut Hazarika <[EMAIL PROTECTED]> Date: Wed Aug 20 09:38:51 2008 -0700 ppc4xx: Optimize PLB4 Arbiter and Memory Queue settings for PPC440SP/SPe, PPC405EX and PPC460EX/GT/SX - Read pipeline depth set to 4 for PPC440SP/SPE, PPC405EX, PPC460EX/GT/SX processors - Moved PLB4 Arbiter register definitions to ppc4xx.h since it is shared across processors (405 and 440/460) - Optimize Memory Queue settings for PPC440SP/SPE and PPC460EX/GT/SX processors - Add register bit definitions for Memory Queue Configuration registers Signed-off-by: Prodyut Hazarika <[EMAIL PROTECTED]> Signed-off-by: Stefan Roese <[EMAIL PROTECTED]> I have a bad feeling changing this "optimized" settings without AMCC's specific ACK. Prodyut, are you ok with Yuri's change? Thanks. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot