On Mon, Nov 17, 2014 at 10:57:54AM +0000, Richard Fitzgerald wrote:
Adds QUIRK_SUPPORTS_192KHZ to allow 192kHz rate to be selected for hardware that supports it.
+- samsung,supports-192khz: specify this (without a value) if you want to allow
- 192000 sample rate over the I2S link.
Two things here. One is that the first half is a funny way of saying "boolean property" (the other standard idiom is "present if..."). The other is that "if you want to allow" suggests that this is a bit of runtime configuration (which shouldn't be in device tree at all). If there's a quirk here it should be a fixed property of the hardware, if this is a system integration choice then it should at most be specified in the machine driver and the I2S controller should just advertise whatever it's physically capable of.