On 03/13/2014 02:20 PM, Mark Brown wrote:
On Tue, Mar 11, 2014 at 12:57:32PM +0200, Jyri Sarha wrote:
This commit adds a bare bones driver support for TLV320AIC31XX family
audio codecs. The driver adds basic stereo playback trough headphone
and speaker outputs and mono capture trough microphone inputs.
Applied, thanks. One thing it'd be good to fix:
+Optional properties:
+
+- gpio-reset - gpio pin number used for codec reset
+- ai31xx-micbias-vg - MicBias Voltage setting
This is optional but...
+ of_property_read_u32(np, "ai31xx-micbias-vg", &value);
+ switch (value) {
+ case MICBIAS_2_0V:
+ case MICBIAS_2_5V:
+ case MICBIAS_AVDDV:
+ aic31xx->pdata.micbias_vg = value;
+ break;
+ default:
+ dev_err(aic31xx->dev,
+ "Bad ai31xx-micbias-vg value %d DT\n",
+ value);
+ aic31xx->pdata.micbias_vg = MICBIAS_2_0V;
+ }
...you'll get an error if it's missing (and an uninitialised memory
read).
Actually you don't. The value-variable is initialized to MICBIAS_2_0V
and if "ai31xx-micbias-vg" is not present of_property_read_u32 leaves
the variable untouched, but I'll send a patch for fixing the unused
variable working shortly.
Forget about the v6 version of the patch.
Thanks,
Jyri
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html