Subject: tw32_f() in tg3_write_mem()

From: "David S. Miller" <[email-masked]>
Date: Sat, 29 Apr 2006 19:22:02 -0700 (PDT)

At least for the TG3PCI_MEM_WIN_DATA register, I don't know how safe
it is to use tw32_f() there.  Reads from a location can have side
effects, so doing a forced readback after a write could be dangerous.

And it isn't needed, as the tw32_f() done as we set the
TG3PCI_MEM_WIN_BASE_ADDR back to zero will flush all posted
writes.

Agreed?