[PATCH] stagging:iio:ad9834: add devicetree property support

[Lars-Peter Clausen]

Hi,

Thanks for the patch.

On 09/11/2016 12:52 PM, Gwenhael Goavec-Merou wrote:
> +static struct ad9834_platform_data *ad9834_parse_dt(struct spi_device *spi)
> +{
> +	struct ad9834_platform_data *pdata;
> +	struct device_node *np = spi->dev.of_node;
> +
> +	pdata = devm_kzalloc(&spi->dev, sizeof(*pdata), GFP_KERNEL);
> +	if (!pdata)
> +		return ERR_PTR(-ENOMEM);
> +
> +	if (of_property_read_u32(np, "mclk", &pdata->mclk))
> +		return ERR_PTR(-ENODEV);

The input clock should be using the standard clock bindings.

> +	if (of_property_read_u32(np, "freq0", &pdata->freq0))
> +		return ERR_PTR(-ENODEV);
> +	if (of_property_read_u32(np, "freq1", &pdata->freq1))
> +		return ERR_PTR(-ENODEV);
> +	if (of_property_read_u16(np, "phase0", &pdata->phase0))
> +		return ERR_PTR(-ENODEV);
> +	if (of_property_read_u16(np, "phase1", &pdata->phase1))
> +		return ERR_PTR(-ENODEV);
> +	pdata->en_div2 = of_property_read_bool(np, "en_div2");
> +	pdata->en_signbit_msb_out = of_property_read_bool(np,
> +					"en_signbit_msb_out");

The other attributes seem to be more runtime configuration data, rather than
hardware description. Maybe just choose a fixed default.

- Lars