[PATCH] i2c-cpm: Add flexibility for I2C clock frequency and filter.

[David Gibson]

On Fri, Oct 31, 2008 at 12:40:47PM +0100, Jochen Friedrich wrote:
> Hi Mike,
> 
> > This patch adds the ability to enable the digital filter in the device
> > tree (with the "clock-filter" boolean property) and automates the
> > predivider selection according to the clock-frequency and clock-filter
> > properties.
> 
> looks good.
> 
> David, is "clock-filter" an appropriate dts property for this purpose or
> would you prefer a different name?

Hrm, well the name seems fine, but then, device-specific properties
are device-specific so it's pretty much up to the device binding to
pick a name.

What does worry me, however, is the description says it's about
whether the driver "should" enable the filter.  Generally the device
tree doesn't attempt to say what users "should" do with the hardware,
just what the characteristics of the hardware are.

What's the underlying difference here that affects the driver's choice
to enable the filter or not?

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson