public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Peter.Chubb at data61.csiro.au <Peter.Chubb@data61.csiro.au>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3] net: Fix cache misalignment message after network load operations
Date: Wed, 14 Sep 2016 04:48:23 +0000	[thread overview]
Message-ID: <84bmzrf5ns.wl-Peter.Chubb@data61.csiro.au> (raw)
In-Reply-To: <57D8D083.3060901@denx.de>

>>>>> "Heiko" == Heiko Schocher <hs@denx.de> writes:

Heiko> Hello Peter, Am 14.09.2016 um 05:49 schrieb
Heiko> Peter.Chubb at data61.csiro.au:
>> After any operation that downloads a file (e.g., pxe get, or dhcp),
>> the buffer containing the downloaded data is flushed.  This is
>> unnecessary and annoying.  Unnecessary, because the network driver
>> should already have fliushed the cache for the DMAed area, and
>> annoying because it generates a cache misalignment message.
>> 
>> Signed-off-by: Peter Chubb <peter.chubb@data61.csiro.au>
>> ---
>> cmd/net.c | 3 --- 1 file changed, 3 deletions(-)

Heiko> I posted a fix for this here:
Heiko> http://patchwork.ozlabs.org/patch/663489/

Heiko> but I did not remove the flush operation ... can we really
Heiko> remove it?
 I believe so -- removing it was suggested by Joe Hershberger.

Heiko> If so, you can add my Acked-by: Heiko Schocher <hs@denx.de>

OK. thanks.
-- 
Dr Peter Chubb         Tel: +61 2 9490 5852      http://ts.data61.csiro.au/
Trustworthy Systems Group                           Data61 (formerly NICTA)

  reply	other threads:[~2016-09-14  4:48 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-14  3:49 [U-Boot] [PATCH v3] net: Fix cache misalignment message after network load operations Peter.Chubb at data61.csiro.au
2016-09-14  4:22 ` Heiko Schocher
2016-09-14  4:48   ` Peter.Chubb at data61.csiro.au [this message]
2016-09-23 20:53 ` Joe Hershberger
2016-10-06  0:14 ` [U-Boot] [U-Boot, " Tom Rini
2016-10-11 20:50   ` Joe Hershberger
2016-10-13 17:39 ` [U-Boot] " Joe Hershberger

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=84bmzrf5ns.wl-Peter.Chubb@data61.csiro.au \
    --to=peter.chubb@data61.csiro.au \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox