Messages in this thread | | | From | "Sahin, Okan" <> | Subject | RE: [PATCH v3 3/5] drivers: regulator: Add ADI MAX77541/MAX77540 Regulator Support | Date | Tue, 31 Jan 2023 13:12:26 +0000 |
| |
On Tue, 31 Jan 2022 3:24 PM Andy Shevchenko <andriy.shevchenko@linux.intel.com> 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@linux.intel.com> 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
| |