Re: [PATCH] regulator: qcom_spmi: Add support for pmi8994

On 11/02/2017 11:28 PM, Stephen Boyd wrote:
> On 11/02/2017 02:24 AM, Rajendra Nayak wrote:
>> Document the regulators available on pmi8994 and add support for
>> this PMIC to the SPMI PMIC regulator driver.
>>
>> Signed-off-by: Rajendra Nayak <[email protected]>
> 
> Reviewed-by: Stephen Boyd <[email protected]>
> Tested-by: Stephen Boyd <[email protected]>

thanks

> 
>> diff --git a/drivers/regulator/qcom_spmi-regulator.c b/drivers/regulator/qcom_spmi-regulator.c
>> index 16c5f84..9f8eb5d 100644
>> --- a/drivers/regulator/qcom_spmi-regulator.c
>> +++ b/drivers/regulator/qcom_spmi-regulator.c
>> @@ -1619,11 +1619,20 @@ static int spmi_regulator_of_parse(struct device_node *node,
>>  	{ }
>>  };
>>  
>> +static const struct spmi_regulator_data pmi8994_regulators[] = {
>> +	{ "s1", 0x1400, "vdd_s1", },
>> +	{ "s2", 0x1700, "vdd_s2", },
>> +	{ "s3", 0x1a00, "vdd_s3", },
>> +	{ "l1", 0x4000, "vdd_l1", },
>> +        { }
> 
> Should be a tab instead of spaces?

Oops, looks like Mark already applied this :/

> 
>> +};
> 

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation