[alsa-devel] [PATCH v6 1/5] ASoC: codecs: Add msm8916-wcd analog codec

Mark Brown broonie at kernel.org
Mon Oct 24 19:00:52 CEST 2016


On Thu, Oct 20, 2016 at 03:20:45PM +0100, Srinivas Kandagatla wrote:

> +static int pm8916_wcd_analog_parse_dt(struct device *dev,
> +				       struct pm8916_wcd_analog_priv *priv)
> +{

> +	priv->mclk = devm_clk_get(dev, "mclk");
> +	if (IS_ERR(priv->mclk)) {
> +		dev_err(dev, "failed to get mclk\n");
> +		return PTR_ERR(priv->mclk);
> +	}
> +	for (i = 0; i < ARRAY_SIZE(supply_names); i++)
> +		priv->supplies[i].supply = supply_names[i];
> +
> +	ret = devm_regulator_bulk_get(dev, ARRAY_SIZE(priv->supplies),
> +				    priv->supplies);

The above isn't parsing the DT, it's just requesting resources - how
those resources are mapped is hidden behind the clock and regulator
APIs.  Please send a followup fixing this.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20161024/b769c7f2/attachment-0001.sig>


More information about the Alsa-devel mailing list