Re: [PATCH v2 1/2] pmbus: add regulator support

[Mark Brown]

On Fri, Aug 22, 2014 at 04:11:33PM -0500, atull@xxxxxxxxxxxxxxxxxxxxx wrote:

> +	if (pdata && pdata->reg_init_data) {
> +		config.init_data = pdata->reg_init_data;
> +	} else {
> +		config.init_data = of_get_regulator_init_data(dev, np);
> +		if (!config.init_data)
> +			return -ENOMEM;
> +	}

It should be fine to start with no init data - just let the regulator
core worry about it.  This will allow users to read back the state even
if they can't change anything which is useful for system bringup or
general debugging.

Attachment: signature.asc
Description: Digital signature