[U-Boot] [PATCH] e1000: fix bugs from recent commits
Wolfgang Denk
wd at denx.de
Fri Oct 28 22:40:17 CEST 2011
Dear "Moffett, Kyle D",
In message <FEA1AB06-A66F-4180-A2B3-076E7C719FC9 at boeing.com> you wrote:
>
> The correct E1000_WRITE_FLUSH macro should be:
> #define E1000_WRITE_FLUSH(a) \
> do { uint32_t x =3D E1000_READ_REG(a, STATUS); (void)x; } while(0=
> )
>
> It shouldn't return a value, it's just ensuring that writes are properly
> posted to the PCI bus.
Really? Should that not be guaranteed by the memory barriers inside
the E1000_READ_REG() code?
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: wd at denx.de
"Here's a fish hangs in the net like a poor man's right in the law.
'Twill hardly come out." - Shakespeare, Pericles, Act II, Scene 1
More information about the U-Boot
mailing list