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 _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev