From: Laurence Withers <lwithers@guralp.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 1/8] da850: indicate cache usage disable in config file
Date: Thu, 18 Aug 2011 15:20:09 +0000 [thread overview]
Message-ID: <20110818152008.GI7769@guralp.com> (raw)
In-Reply-To: <CAPcSpEKap-am_aEVGs+rt7NsJNDMY2DFLm+k=wioedkJVg2dAw@mail.gmail.com>
On Thu, Aug 18, 2011 at 11:13:19AM -0400, Ben Gardiner wrote:
> I understand that both Laurence and Stefan (cc'd) have confirmed that
> there dcache issues with the EMA; I am assuming that the 'issue'
> results in a delay in tftp'ing...
No; the issue was that packets were being sent on to the wire with incorrect
IP checksums. It twigged that the problem was cache coherency when I put a
full hexdump of each outgoing packet in and suddenly the outbound packets
started being valid.
> Disabling the caches will slow down decompression which will slow down
> boot overall when booting from flash.
>
> Rather than masking the issue by disabling caches and slowing down
> u-boot for it's users perhaps TI should be fixing the EMAC drivers'
> cache bugs instead?
Indeed, the correct solution is to properly manage the net buffers with
respect to the cache, although I have not attempted that change myself. I
have seen some patches start to flow that make changes in drivers to work
correctly with caches enabled, such as
http://lists.denx.de/pipermail/u-boot/2011-August/098484.html .
Bye for now,
--
Laurence Withers, <lwithers@guralp.com> http://www.guralp.com/
Direct tel:+447753988197 or tel:+443333408643 Software Engineer
General support queries: <support@guralp.com> CMG-DCM CMG-EAM CMG-NAM
next prev parent reply other threads:[~2011-08-18 15:20 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-17 14:14 [U-Boot] [PATCH v3 0/8] feature additions and fixes for da850 nagabhushana.netagunte at ti.com
2011-08-17 14:14 ` [U-Boot] [PATCH v3 1/8] da850: indicate cache usage disable in config file nagabhushana.netagunte at ti.com
2011-08-18 15:13 ` Ben Gardiner
2011-08-18 15:20 ` Laurence Withers [this message]
2011-08-29 10:56 ` Netagunte, Nagabhushana
2011-08-31 5:39 ` Netagunte, Nagabhushana
2011-08-31 13:23 ` Ben Gardiner
2011-08-17 14:14 ` [U-Boot] [PATCH v3 2/8] da8xx: add support for multiple PLL controllers nagabhushana.netagunte at ti.com
2011-08-17 14:14 ` [U-Boot] [PATCH v3 3/8] da850: add NOR boot mode support nagabhushana.netagunte at ti.com
2011-08-17 14:14 ` [U-Boot] [PATCH v3 4/8] da850: modify the U-Boot prompt string nagabhushana.netagunte at ti.com
2011-08-17 14:14 ` [U-Boot] [PATCH v3 5/8] da850: add support to wake up DSP during board init nagabhushana.netagunte at ti.com
2011-08-17 14:14 ` [U-Boot] [PATCH v3 6/8] da850: add support for Spectrum Digital AM18xx EVM nagabhushana.netagunte at ti.com
2011-08-17 14:14 ` [U-Boot] [PATCH v3 7/8] da850: fix the channel number for EMAC teardown init nagabhushana.netagunte at ti.com
2011-08-17 14:14 ` [U-Boot] [PATCH v3 8/8] da850: modifications for Logic PD Rev.3 AM18xx EVM nagabhushana.netagunte at ti.com
2011-08-31 5:36 ` [U-Boot] [PATCH v3 0/8] feature additions and fixes for da850 Netagunte, Nagabhushana
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110818152008.GI7769@guralp.com \
--to=lwithers@guralp.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.