From:   "Sahin, Okan" <Okan.Sahin@analog.com>
To:     Andy Shevchenko <andriy.shevchenko@linux.intel.com>
CC:     Lee Jones <lee@kernel.org>, Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Liam Girdwood <lgirdwood@gmail.com>,
        Mark Brown <broonie@kernel.org>,
        Jonathan Cameron <jic23@kernel.org>,
        Lars-Peter Clausen <lars@metafoo.de>,
        ChiYuan Huang <cy_huang@richtek.com>,
        "Bolboaca, Ramona" <Ramona.Bolboaca@analog.com>,
        Caleb Connolly <caleb.connolly@linaro.org>,
        William Breathitt Gray <william.gray@linaro.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>
Subject: RE: [PATCH v3 3/5] drivers: regulator: Add ADI MAX77541/MAX77540
 Regulator Support
Thread-Topic: [PATCH v3 3/5] drivers: regulator: Add ADI MAX77541/MAX77540
 Regulator Support
Thread-Index: AQHZKwfLM9q3yL8ciUSLt6aTy9zYOq6j1SEAgBRcQ/CAAFZ5AIAADHeA
Date:   Tue, 31 Jan 2023 13:12:26 +0000
Message-ID: <BY5PR03MB5153112086B2B9F899E8524AE7D09@BY5PR03MB5153.namprd03.prod.outlook.com>
References: <20230118063822.14521-1-okan.sahin@analog.com>
 <20230118063822.14521-4-okan.sahin@analog.com>
 <Y8erlpofdk24vwCC@smile.fi.intel.com>
 <MN2PR03MB5168EC97926AB33D4D806FCCE7D09@MN2PR03MB5168.namprd03.prod.outlook.com>
 <Y9kIfEj70XdO3Q/7@smile.fi.intel.com>
In-Reply-To: <Y9kIfEj70XdO3Q/7@smile.fi.intel.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?xxmt0tUTcKPJtsmpB6rd1iOdwk1aM9i6RPeMFosFYd1/Xnsgp9qxB4Q6TQhT?=
 =?us-ascii?Q?wzYWuWlppOT8s46dKrL8+RyMydfi+7IoM74wsTsAssSTwrQ2AvrFM+OvcxI6?=
 =?us-ascii?Q?xqpqxnY+2Un8M1g+VS8G4y8xEm2gtxuSTS+BVUk33SWK1lk23LJCt/6aZF8n?=
 =?us-ascii?Q?pI+IDPLtBl2j/T4GhdfQQo2LCU7aDkYW2hTSSVVWvZVOrrHR/y0VzTuO1+O/?=
 =?us-ascii?Q?PGf3LWcuXWj/EonaIN48qU6leZncs02NK23aGwrA/vrXmWccuUlm0m4VngV+?=
 =?us-ascii?Q?iv1keu4bxIJFjAjDzbYORaJfi7pPqK+OM9PGvlIJaK23o1NuxGhLhvBA2iEX?=
 =?us-ascii?Q?kH1udzHhwBG73tmp30N6nF7YoxqLqAPe9DvOnkzXXQgXnAJwovvwXyIYJKIl?=
 =?us-ascii?Q?DDE1xMt9t9V8xkZevZyWhz3LZcRY1rIzbSZb7uhDSwZ3D5D7HrC1C3UFeQe/?=
 =?us-ascii?Q?r3TQSNLHMr31/daW4GUBWMYEN1gZZUcHaEO1fp2TRvzMSHKedTVFIuQiLHHb?=
 =?us-ascii?Q?0zFckhek/tHfo2QMfurXIf9V7bkSRI8is3QUNCMnVtt3SjtWJtOIhxIgjMN2?=
 =?us-ascii?Q?pyp6L1NLNGYL5Yqw4GklyZ51iv7DMoLxgEMDE6KpvWs0RMhEE3duHDLnqki9?=
 =?us-ascii?Q?3V/zM/m/kp5a02V3YhnmiZZBUiPh0RK22QfELTDl8a+Fq8jGDvlF19u2LwHL?=
 =?us-ascii?Q?y0gCCIQDGSe0NVNEvPmvWn9KH9vRm4mDUWFaqR2mh5ht6LaJKaNJ46tHonPj?=
 =?us-ascii?Q?4CdEg56SiXOOYVSg1vuA3AJMzrzRGddUo6FYy8YIGJaUO+otaC8N8IB2UAdW?=
 =?us-ascii?Q?wewo255fT/VRIVMW+Dmxf99HTTWfOWiL2AbInQcCZExeD2qIFh9Vr+IWA6gS?=
 =?us-ascii?Q?L2IhQGtFLXe6ozaRoWdljL5H11anAYDgHBc8CnxH5D62qQDO6zgI/D5I64Dq?=
 =?us-ascii?Q?5hX5Rx1Z+wAkIwlvVHmcz3is9qlKp4Q3pVctjs7HDWu7qb+EvzqGaSnlparX?=
 =?us-ascii?Q?2AxUIvjtmIb1SKxXXC5sDmV/Joia2pqRyI/Bzpx7qHP+a4fpM3/scNFsTU5+?=
 =?us-ascii?Q?bci9Wu1Fgfy/k4j1tcTxlAbNsdsxe5ek7xLWV28yfnA1+CTHxRCMW4xKY8X/?=
 =?us-ascii?Q?/+Vs6OV/WnX84Q6kE0fZJVICwkTaoiJSTrzppnw/qsdHKVjaK1jfIUBWdWCG?=
 =?us-ascii?Q?QQXn+zvYFn6w6jGksiJgtyGc54PDfgnEqhM6wy/MRhvUDJURx4ShzDPZMTRX?=
 =?us-ascii?Q?t4IswOhWpOSa+bnOcDjVwOnu/b+Nu1Ofp126c9Yxz/L0kjaTTVErp28YR8Nv?=
 =?us-ascii?Q?rEn/woNnlHaSvy8UwqLjABjuYuZE3xUfsBgGaCj7e4hqOF1im2vuO+c8BtLE?=
 =?us-ascii?Q?iJpbMproWw6JnFeRIRa/4oLVo1g3Cwy9acM9iUofFBJ2Ut7n3PxmU9Wk5xRZ?=
 =?us-ascii?Q?cUFpA7+ImFbbRP+ZfPSZSVX/Fah+BHbeaqt9Hi4YuREZaJN7DV1NNOw28wn3?=
 =?us-ascii?Q?L7yk7PLmu8qjkQu8R1wAY+zYlqYsT4GDO7rsNm6N1DEPEHICY/ONCwbSLckY?=
 =?us-ascii?Q?Kf06sgZ2hSQYfvEyev98jnhauBbSOcYy590gKojq?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: analog.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BY5PR03MB5153.namprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c2c07b95-d56b-4773-b307-08db038ccc74
X-MS-Exchange-CrossTenant-originalarrivaltime: 31 Jan 2023 13:12:26.3405
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: eaa689b4-8f87-40e0-9c6f-7228de4d754a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: lQQ63Nq+iXDucLwohYdHrybBNINOXSnkCGfYDs6WoSXT06PbvT9ppRG5UcIXyrnR/WK7EebgL5hViPJTA0lIFA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR03MB6523
X-Proofpoint-ORIG-GUID: gcz_-5UnyoFhLctsi_JS08Q2v5-VT1p5
X-Proofpoint-GUID: gcz_-5UnyoFhLctsi_JS08Q2v5-VT1p5
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.219,Aquarius:18.0.930,Hydra:6.0.562,FMLib:17.11.122.1
 definitions=2023-01-31_08,2023-01-31_01,2022-06-22_01
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0
 lowpriorityscore=0 malwarescore=0 clxscore=1015 spamscore=0 adultscore=0
 phishscore=0 mlxscore=0 suspectscore=0 priorityscore=1501 bulkscore=0
 mlxlogscore=890 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2212070000 definitions=main-2301310116
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

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 =3D 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 registe=
r regulator\n");"
>
>--
>With Best Regards,
>Andy Shevchenko
>

Best Regards,
Okan Sahin