RE: [PATCH v3 3/5] drivers: regulator: Add ADI MAX77541/MAX77540 Regulator Support
From: Sahin, Okan
Date: Tue Jan 31 2023 - 08:13:26 EST
On Tue, 31 Jan 2022 3:24 PM
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
>On Tue, Jan 31, 2023 at 07:20:52AM +0000, Sahin, Okan wrote:
>> On Wed, 18 Jan 2022 11:20 AM
>> Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
>> >On Wed, Jan 18, 2023 at 09:38:10AM +0300, Okan Sahin wrote:
>
>...
>
>> >> + rdev = devm_regulator_register(dev,
>> >> + &desc[i], &config);
>> >
>> >This is perfectly one line.
>> Thank you, I will arrange it.
>> >
>> >> + if (IS_ERR(rdev))
>> >> + return dev_err_probe(dev, PTR_ERR(rdev),
>> >> + "Failed to register regulator\n");
>> >> + }
>> >> +
>> >> + return 0;
>> >> +}
>
>> However, this one is not fit when I set max-line-length argument as 80
>> in checkpatch script. What do you suggest? This line has 99 characters.
>
>Which line do you refer to?
I am referring "return dev_err_probe(dev, PTR_ERR(rdev), "Failed to register regulator\n");"
>
>--
>With Best Regards,
>Andy Shevchenko
>
Best Regards,
Okan Sahin