[PATCH v3 2/2] regulator: Add support for MAX77686.
Yadwinder Singh Brar
yadi.brar01 at gmail.com
Wed May 23 00:17:28 EDT 2012
Hi,
On Wed, May 23, 2012 at 7:20 AM, <jonghwa3.lee at samsung.com> wrote:
> Hi, again.
> On 2012년 05월 22일 14:57, yadi.brar01 at gmail.com wrote:
>
>
>> +static __devinit int max77686_pmic_probe(struct platform_device *pdev)
>> +{
>
>> +
>> + for (i = 0; i < pdata->num_regulators; i++) {
>> + config.dev = max77686->dev;
>> + config.init_data = pdata->regulators[i].init_data;
>> + config.driver_data = max77686;
>> + config.regmap = iodev->regmap;
>> +
>> + rdev[i] = regulator_register(®ulators[i], &config);
>
>
> I'm sorry that i missed one. You have to register all regulators
> unconditionally. Mark brown commented about this to my former patch.
>
> 'No, you should unconditionally register all regulators the device
> physically has. This is useful for debug and simplifies the code.'
> - from Mark Brown
>
Yes, we are registering all regulators here.
As pdata->num_regulators will be equal to ARRAY_SIZE(regulators)
Thanks,
Yadwinder.
More information about the linux-arm-kernel
mailing list