On Tue, Jan 31, 2023 at 01:12:26PM +0000, Sahin, Okan wrote: > 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");" I have had no comments on that line. -- With Best Regards, Andy Shevchenko
