On Wed, 30 Mar 2011 17:07:11 +0100, Chris Wilson <ch...@chris-wilson.co.uk> 
wrote:

> +clear_err:
> +     I915_WRITE(GMBUS1 + reg_offset, GMBUS_SW_CLR_INT);
> +     POSTING_READ(GMBUS1 + reg_offset);
> +     I915_WRITE(GMBUS1 + reg_offset, 0);

Any posting read needed here?

> +
> +done:
> +     I915_WRITE(GMBUS0 + reg_offset, 0);

What's this new write doing in the non-error path? Do we need a posting
read after it?

-- 
keith.pack...@intel.com

Attachment: pgpkb2hVh5uez.pgp
Description: PGP signature

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to