Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Sa, Nuno" <Nuno.Sa@analog.com>
To:     Jakob Koschel <jakobkoschel@gmail.com>,
        Jonathan Cameron <jic23@kernel.org>
CC:     Lars-Peter Clausen <lars@metafoo.de>,
        Dan Carpenter <dan.carpenter@oracle.com>,
        "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Mike Rapoport <rppt@kernel.org>,
        Brian Johannesmeyer <bjohannesmeyer@gmail.com>,
        Cristiano Giuffrida <c.giuffrida@vu.nl>,
        "Bos, H.J." <h.j.bos@vu.nl>
Subject: RE: [PATCH 2/3] iio: ssp_sensors: replace usage of found with
 dedicated list iterator variable
Thread-Topic: [PATCH 2/3] iio: ssp_sensors: replace usage of found with
 dedicated list iterator variable
Thread-Index: AQHYRVQWzVTHnxHR10i24e3FWXGAAqzbAR8g
Date:   Fri, 1 Apr 2022 12:43:21 +0000
Message-ID: <PH0PR03MB67869AFC7667ABCD9A57CB9099E09@PH0PR03MB6786.namprd03.prod.outlook.com>
References: <20220331230632.957634-1-jakobkoschel@gmail.com>
 <20220331230632.957634-2-jakobkoschel@gmail.com>
In-Reply-To: <20220331230632.957634-2-jakobkoschel@gmail.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 2
x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?orVnz4qiXyq4J6seUMT4ZxgiTpKhzp4SzkkMtM2zA0g7UrdkbGbq2Xb7UX?=
 =?iso-8859-1?Q?bcZETTcM46bfmyf9LVpoK6ntotJ/wrQKf+6Y5eomH+RKyDxwea/SJFE8ns?=
 =?iso-8859-1?Q?9z+mvvkTeHluB9KZH17/Y0IIi2kpJ5cjT1DhJdrwIAuCcFgvmIIYEz0rmx?=
 =?iso-8859-1?Q?1qqrSgyVnjZPzF/uHged8xVn6C6dbCOfaVRLyh02bpbS9OzS1Ld4HD6azq?=
 =?iso-8859-1?Q?HMkqrjUURDHag22322XSpbLsJo5bACZGpTo9ycT+3bS8zPh21LxQ/jtHrg?=
 =?iso-8859-1?Q?IE7DBUAiPtDsGiQOmrWZrNwTEi7sF8hEXs8/0GLG9rN4QW5RZXVFl4UeFx?=
 =?iso-8859-1?Q?VwJ3P2zqgwNKt+D/lcVD9hsabTLe45Ah457EhPl28ToZp1T1GUngDJJG36?=
 =?iso-8859-1?Q?C5wC9p/YnzHJGUksGLaXvwqbkIjn9Gb5AErp+B1Qo85kPAm8MzUrR+tEa6?=
 =?iso-8859-1?Q?phKeWX6jAAalU2klHVIcjQxJl8+lwgJMVX3QUPSL9WGUFI0Mu8wqTn+4w+?=
 =?iso-8859-1?Q?pOkQSnZGTRNXQlVLb2YJpt4bOVhw1504N8/TOAx5A+e8Qmf+Gtsa1TrKXt?=
 =?iso-8859-1?Q?wldZ30zDhXK52CD5iiUPrGZT7FJy6bsTQ0k095T7DL7aqOk/DEVERu8ilI?=
 =?iso-8859-1?Q?xPgzLAlCzUExJXF0wMST+yTHcg+tQuFPccVZVJuwv9wD3THSe0foa39N9w?=
 =?iso-8859-1?Q?eC5MtMECf+fhvQdBbRGsRN8ATD0St5dcKfdimoEMQeaFsBJ7wADfJb0Cru?=
 =?iso-8859-1?Q?JVZmVwJPoXFCm7Zmszni0z5rydRw3SQhjqbIsLbYmKlf6RTGgMdFQjAs3l?=
 =?iso-8859-1?Q?eQ4KkB+BzzG4JbRZg1+QXIePuaU/7KfTma6+wZTaMMTbhX9jFccJJXKlYY?=
 =?iso-8859-1?Q?tiMgi6cQDHdTibhIdPoRGOs9+VqyBT9zImgyr0QBNsq2wz+VC9A33ot1ep?=
 =?iso-8859-1?Q?zSG3zy8cPR0C9Uh1avgM2tUDsWutrqe3pYDhQjUhYrEh5AbyOPt2pmu0ku?=
 =?iso-8859-1?Q?xZ0EhZrfm/EJyKQLMQ6KMwKjaUtZWoz9QtDXtUclU6Q6wUaVYfN3un6eze?=
 =?iso-8859-1?Q?atZgCSwwPZJ4VUYwLd261z49E+rv9c4sy0CA6MfZU3dLWV29S5wEdfiV8Q?=
 =?iso-8859-1?Q?5i+V5/MTg1a1ZH8XcCuW2t6oIc0MHcMC2FrTzj6WHa9Lqzyn50iaKBmds6?=
 =?iso-8859-1?Q?WAUNPB77IeT4kGsF7+9XNaGStd1oomacDtlfvVuJsOPntL58dbV3udhCnh?=
 =?iso-8859-1?Q?psUg7T/A1U+F56EmF2KhtmuoU3oJqHJhIiDyRzhdCdfpx0aJHsbd2QuFI9?=
 =?iso-8859-1?Q?c0Y2mJtCh1VMWQkjwKO7pzkK84F6UK8M/GBuaHgoF5IqZugZYAXylGEloS?=
 =?iso-8859-1?Q?YZmnJQ2orRIC5v1SSxRiYashxgyQTrw+t7bfVrbR0thw2ZdbLEJ/Lv4j1v?=
 =?iso-8859-1?Q?Nha5siEy9UDofhI8jyurYcr/XIRPlX9dA6KvVlXCYreII/jq14gZIgF0EO?=
 =?iso-8859-1?Q?o/oA28BZfYkWDsJR0x1bM07cCmzVEzwBStLqBW3QE2WXzaONTWivMRduum?=
 =?iso-8859-1?Q?0e6ow6XA2xUuN6F1fM4RVZHYWbg8wENzucckWJpwRD+JIJJv+NFH3pHYwL?=
 =?iso-8859-1?Q?W3nIsHC2RBlkl494BahocBmYxxjRLhdBEbNEnDtDktC6KJn618mO/sEhp4?=
 =?iso-8859-1?Q?9TF0fAOg0m2chsM3IIHe7bqCwOD/8kScm7t6isM7qtgXH7uSTb2sKfhx22?=
 =?iso-8859-1?Q?iUItUb3z9vByVGKSi6wCoQY071QOP0P3j4p/Deyayp+Z9+D6XbymkXTflu?=
 =?iso-8859-1?Q?A/oay1Fj8VeiqL/gktIARv66WqPujBmhKtv/9JwUjrPY6Wu4WQ4GG4W4ZL?=
 =?iso-8859-1?Q?hA?=
x-ms-exchange-antispam-messagedata-1: zOsSx7HcqoRBfA==
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: analog.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PH0PR03MB6786.namprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 763f85af-5a24-4ffe-11c3-08da13dd348d
X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Apr 2022 12:43:21.5925
 (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: khCIDRUwhqRrFGJx4Tr+0sgdWjWcEDXVr2H0DvtOnw4JmASU2kdPPi9NzKXeDVHcNMUdYShkNnQ89OVtNLPszw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW5PR03MB6960
X-Proofpoint-ORIG-GUID: LyNk0mjSCb4XdwVh-FwtvnHlQLCPgT7C
X-Proofpoint-GUID: LyNk0mjSCb4XdwVh-FwtvnHlQLCPgT7C
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.850,Hydra:6.0.425,FMLib:17.11.64.514
 definitions=2022-04-01_04,2022-03-31_01,2022-02-23_01
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 clxscore=1015
 mlxscore=0 bulkscore=0 priorityscore=1501 phishscore=0 malwarescore=0
 adultscore=0 lowpriorityscore=0 impostorscore=0 mlxlogscore=999
 suspectscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2202240000 definitions=main-2204010058
X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,SPF_PASS,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
> From: Jakob Koschel <jakobkoschel@gmail.com>
> Sent: Friday, April 1, 2022 1:07 AM
> To: Jonathan Cameron <jic23@kernel.org>
> Cc: Lars-Peter Clausen <lars@metafoo.de>; Dan Carpenter
> <dan.carpenter@oracle.com>; Jakob Koschel
> <jakobkoschel@gmail.com>; linux-iio@vger.kernel.org; linux-
> kernel@vger.kernel.org; Mike Rapoport <rppt@kernel.org>; Brian
> Johannesmeyer <bjohannesmeyer@gmail.com>; Cristiano Giuffrida
> <c.giuffrida@vu.nl>; Bos, H.J. <h.j.bos@vu.nl>
> Subject: [PATCH 2/3] iio: ssp_sensors: replace usage of found with
> dedicated list iterator variable
>=20
> [External]
>=20
> To move the list iterator variable into the list_for_each_entry_*()
> macro in the future it should be avoided to use the list iterator
> variable after the loop body.
>=20
> To *never* use the list iterator variable after the loop it was
> concluded to use a separate iterator variable instead of a
> found boolean [1].
>=20
> This removes the need to use a found variable and simply checking if
> the variable was set, can determine if the break/goto was hit.
>=20
> Link: https://urldefense.com/v3/__https://lore.kernel.org/all/CAHk-
> =3DwgRr_D8CB-D9Kg-
> c=3DEHreAsk5SqXPwr9Y7k9sA6cWXJ6w@mail.gmail.com/__;!!A3Ni8CS0y
> 2Y!tqcYepOSYys02RcvKD7LiZk3mHwPmSLa_1S0QglWPRZSCoPnXk3ccjY
> _XzsDDA$  [1]
> Signed-off-by: Jakob Koschel <jakobkoschel@gmail.com>
> ---

Reviewed-by: Nuno S=E1 <nuno.sa@analog.com>