| Message ID | 1416221874-22092-1-git-send-email-rf@opensource.wolfsonmicro.com |
|---|---|
| State | Needs Review / ACK, archived |
| Headers | show |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | warning | total: 1 errors, 0 warnings, 0 lines checked |
| robh/patch-applied | success |
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.
diff --git a/Documentation/devicetree/bindings/sound/samsung-i2s.txt b/Documentation/devicetree/bindings/sound/samsung-i2s.txt index d188296..c3f3950 100644 --- a/Documentation/devicetree/bindings/sound/samsung-i2s.txt +++ b/Documentation/devicetree/bindings/sound/samsung-i2s.txt @@ -40,6 +40,8 @@ Optional SoC Specific Properties: sub system(used in secondary sound source). - pinctrl-0: Should specify pin control groups used for this controller. - pinctrl-names: Should contain only one value - "default". +- samsung,supports-192khz: specify this (without a value) if you want to allow + 192000 sample rate over the I2S link. Example: diff --git a/include/linux/platform_data/asoc-s3c.h b/include/linux/platform_data/asoc-s3c.h index 5e0bc77..6d0702d 100644 --- a/include/linux/platform_data/asoc-s3c.h +++ b/include/linux/platform_data/asoc-s3c.h @@ -28,6 +28,7 @@ struct samsung_i2s { #define QUIRK_NEED_RSTCLR (1 << 3) #define QUIRK_SUPPORTS_TDM (1 << 4) #define QUIRK_SUPPORTS_IDMA (1 << 5) +#define QUIRK_SUPPORTS_192KHZ (1 << 6) /* Quirks of the I2S controller */ u32 quirks; dma_addr_t idma_addr; diff --git a/sound/soc/samsung/i2s.c b/sound/soc/samsung/i2s.c index 947352d..f688e89 100644 --- a/sound/soc/samsung/i2s.c +++ b/sound/soc/samsung/i2s.c @@ -1219,6 +1219,9 @@ static int samsung_i2s_probe(struct platform_device *pdev) "specified"); } } + + if (of_find_property(np, "samsung,supports-192khz", NULL)) + quirks |= QUIRK_SUPPORTS_192KHZ; } res = platform_get_resource(pdev, IORESOURCE_MEM, 0); @@ -1247,6 +1250,11 @@ static int samsung_i2s_probe(struct platform_device *pdev) if (quirks & QUIRK_PRI_6CHAN) pri_dai->i2s_dai_drv.playback.channels_max = 6; + if (quirks & QUIRK_SUPPORTS_192KHZ) { + pri_dai->i2s_dai_drv.playback.rates |= SNDRV_PCM_RATE_192000; + pri_dai->i2s_dai_drv.capture.rates |= SNDRV_PCM_RATE_192000; + } + if (quirks & QUIRK_SEC_DAI) { sec_dai = i2s_alloc_dai(pdev, true); if (!sec_dai) {
Adds QUIRK_SUPPORTS_192KHZ to allow 192kHz rate to be selected for hardware that supports it. Signed-off-by: Richard Fitzgerald <rf@opensource.wolfsonmicro.com> --- .../devicetree/bindings/sound/samsung-i2s.txt | 2 ++ include/linux/platform_data/asoc-s3c.h | 1 + sound/soc/samsung/i2s.c | 8 ++++++++ 3 files changed, 11 insertions(+), 0 deletions(-)