From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by yocto-www.yoctoproject.org (Postfix, from userid 118) id 75127E00769; Fri, 19 Sep 2014 04:16:15 -0700 (PDT) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on yocto-www.yoctoproject.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00 autolearn=ham version=3.3.1 X-Spam-HAM-Report: * -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% * [score: 0.0000] Received: from smtp.webfaction.com (mail6.webfaction.com [74.55.86.74]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id 0EDB1E006EF for ; Fri, 19 Sep 2014 04:16:10 -0700 (PDT) Received: from [192.168.1.10] (c-68-38-40-177.hsd1.nj.comcast.net [68.38.40.177]) by smtp.webfaction.com (Postfix) with ESMTP id C38382294B51; Fri, 19 Sep 2014 11:16:08 +0000 (UTC) Message-ID: <541C1077.2090200@mindchasers.com> Date: Fri, 19 Sep 2014 07:16:07 -0400 From: Bob Cochran User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.1.1 MIME-Version: 1.0 To: "zhenhua.luo@freescale.com" , "meta-freescale@yoctoproject.org" References: <541A54FD.7000004@mindchasers.com> In-Reply-To: Cc: Scott Wood , Heinz Wrobel Subject: Re: Should all T4240 errata be applied to T1040? X-BeenThere: meta-freescale@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Usage and development list for the meta-fsl-* layers List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 19 Sep 2014 11:16:15 -0000 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit On 09/19/2014 06:04 AM, zhenhua.luo@freescale.com wrote: > Hi Bob, > > Thanks for the great finding. > > Defining the CONFIG_FMAN_T4240 for T1 is inaccurate and potentially dangerous for the correct function. The manual explicitly states for TNTSKS that exceeding the max supported values is not permitted, but it doesn't actually state that the reset value is the maximum permissible value. > > As for the correct way to define things, we probably need to share integrations across common SoCs per, e.g., Table A-5 in the T1040 DPAA RM to make SW match our docs properly. A configuration "T4240" is likely a bad one anyway if we should name it effectively "FMAN_V3H_384KiB" vs. "FMAN_V3H_512KiB" for B4860 rev 2. So it is good to change "FMAN_V3L" for T1 to match the documentation and part behavior. > > Also the correct way is to stop configuring such things at compile time to avoid separate kernel build for every FMan revision. Thank you for the quick reply Zhenhua. Your proposed solution makes sense to me. I'm seeing other registers that have their values set to FMAN_v3H rather than FMAN_v3L values, and I'm trying to track them down in the source to see whether they are being incorrectly set (or maybe the doc is wrong?). Perhaps I'll send you a list of suspicious register settings in the next few days. I continue to investigate the DPAA to hopefully resolve CPU hangs & oops during network loading. Bob > > > Best Regards, > > Zhenhua > >> -----Original Message----- >> From: meta-freescale-bounces@yoctoproject.org [mailto:meta-freescale- >> bounces@yoctoproject.org] On Behalf Of Bob Cochran >> Sent: Thursday, September 18, 2014 11:44 AM >> To: meta-freescale@yoctoproject.org >> Subject: [meta-freescale] Should all T4240 errata be applied to T1040? >> >> I find that during initialization of my T1040rdb-64b using the kernel built with >> meta-fsl-ppc master, the FMBM_CFG2.TNTASKS field (total number of BMI >> tasks) is being overwritten to 0x7b (the reset value is documented as 0x3b in >> the T1040 reference manual). >> >> I believe this is being overwritten due to sharing the errata definitions with the >> T4240, but this particular erratum may not apply to the T1040 and may cause >> undesirable side effects. >> >> The T4240 uses DPAA FMAN_v3H, and the T1040 uses DPAA FMAN_v3L. >> >> >> Here is my understanding of how this erratum (A005127) is being applied to the >> T1040: >> >> 1) t1040_64bit_smp_defconfig defines CONFIG_FMAN_T4240 >> >> 2) By defining CONFIG_FMAN_T4240, >> drivers/net/ethernet/freescale/fman/ncsw_config.mk adds >> -I$(FMAN)/inc/integrations/T4240 to EXTRA_CFLAGS. >> >> 3) dpaa_integration_ext.h is included throughout the fman source files from >> drivers/net/ethernet/freescale/fman/inc/integrations/T4240 >> >> 4) This defines FM_WRONG_RESET_VALUES_ERRATA_FMAN_A005127, but it is >> my understanding that this erratum does not apply to the T1040. >> >> 5) This results in FMBM_CFG2.TNTASKS being overwritten with a value >> approximately twice its default value, and my concern is that the FMAN_V3L >> does not have the resources to support this many tasks. >> >> >> >> In general, I'm also wondering whether the other errata in the integration file is >> appropriate for the T1040 and whether the FMAN_v3L devices should have >> their own integration tree. >> >> >> >> >> >> -- >> _______________________________________________ >> meta-freescale mailing list >> meta-freescale@yoctoproject.org >> https://lists.yoctoproject.org/listinfo/meta-freescale >