Re: [PATCH v3] Add bt8xxgpio driver

[Michael Buesch]

On Thursday 10 July 2008 22:02:12 David Brownell wrote:
> See Documentation/gpio.txt:
> 
> + Note that these operations include I/O barriers on platforms
> + which need to use them; drivers don't need to add them explicitly.
> 
> That's the key thing:  drivers using I/O calls should
> not need to insert bus or platform specific calls to
> make sure the calls take effect.


Well, as I said, I am OK with an I/O-barrier, which is mmiowb().
But flushing, by reading the register back, is not acceptable for
me in this case, unless someone can show me hardware that needs this.
And even then I'd implement the flushing only conditionally for
that hardware.

-- 
Greetings Michael.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/