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:     "Bolboaca, Ramona" <Ramona.Bolboaca@analog.com>,
        "jic23@kernel.org" <jic23@kernel.org>,
        "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
CC:     "Bolboaca, Ramona" <Ramona.Bolboaca@analog.com>
Subject: RE: [PATCH v2 0/8] Remove adis_initial_startup usage
Thread-Topic: [PATCH v2 0/8] Remove adis_initial_startup usage
Thread-Index: AQHY71uS1QH1RLZlHUSZW0mLk8EYpa4tIjSg
Date:   Thu, 3 Nov 2022 12:35:31 +0000
Message-ID: <SJ0PR03MB677857576EF31B737F6D3DF599389@SJ0PR03MB6778.namprd03.prod.outlook.com>
References: <20221103080847.162509-1-ramona.bolboaca@analog.com>
In-Reply-To: <20221103080847.162509-1-ramona.bolboaca@analog.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?YUUI24c/fsZAccR4PKpUkwoolLTaQSVQyOQZEN8Lq6snopjnJ+XdnLB+Qa?=
 =?iso-8859-1?Q?QQlKxzypNEpUiYaSY+FPoQcMx3PLrRC8+f8yJWtnOXe8vKeqMGBYFmFuO4?=
 =?iso-8859-1?Q?Q0/x1o1VjlNcO0WaC835bZ5ACT+P/AWQK6qnyyvFTpCkQn7G34h+4DInWN?=
 =?iso-8859-1?Q?G4y/Br53Z4N6lin0B1RxsC8GVzLHwBda3i/NEuc4rcL521amoc+EfcQI5b?=
 =?iso-8859-1?Q?B7EuTThf3sn33FbtRQSJinPuNXF+hH6ZjGdJ2VZj/i2yJfAArR9GIgVHRV?=
 =?iso-8859-1?Q?1rFEmfhxeTf3/0++AyJsn7yZOZeikl4HxhCi4Y5gZjyaqwzzSNS8c1E9AE?=
 =?iso-8859-1?Q?6vfAOvWQB5Mo5YRgKuu2bcc4WLOHfWia9qIBYxzlz06KT+ESTNd92GnzRY?=
 =?iso-8859-1?Q?YAy2d+5GKbisBp/iQnfNmVJ/YG9CiYEmrz7bp/Oe37G4CTH9kwG7roar/S?=
 =?iso-8859-1?Q?yUUJN4QW9FDY+nxWnqhvHFF4Finali8n12qXx48LTg36eEnr69RWlJ1WZU?=
 =?iso-8859-1?Q?ePy8mvgBK6Nmf/XMZrjTmZFOed9COpTLj4YOM1VRXBmvkwfV+D5AxKCeL2?=
 =?iso-8859-1?Q?1anyOWiNfL1VZLo44BR5Rrx68EinH8s3IbkD6BZzDTWLRoWB5xcFOrZbf3?=
 =?iso-8859-1?Q?m8hR2VXfQVrnIIKAcpVGv71akVtJEYY2komVYw+f11lY3vLgJgRv6/QkKV?=
 =?iso-8859-1?Q?24gEwVLbGKtWKfwQlj4q7+KyS6n17BG8l1q8fQsHVp0majF1B55p/si3bx?=
 =?iso-8859-1?Q?siQTc7kho2xmR27L/4A6C/GByJ/vQtLKZNzOCAHG9Jl92Lyl/9tHpZtdH3?=
 =?iso-8859-1?Q?e8Jhef6oo0v5FhZQDSFo1GIKmQrndnHWiv9Z0iUBToLSToh/68Uy8tIBvV?=
 =?iso-8859-1?Q?BMvSVfiGIurMRqNvt0iWx2+ZO5yOp74OES+L7JzTRYcZJFDhM1MxKI/Ocm?=
 =?iso-8859-1?Q?3EG74iwliz0Bv8weCCTyJJmYo80YHCsVqqxZshmO3MuzDMg8uik/D7f9XY?=
 =?iso-8859-1?Q?q0FzVGPmTkCo7cppl8fAb5Zk6DaahFUk1YU2Ayj8CY/aAv99+ggQTEpNEO?=
 =?iso-8859-1?Q?qq5IkQnTHVsR3AaE7D3GrxuBgV66royb9pKlJJnZJCupzFvHk84BhIA+Mz?=
 =?iso-8859-1?Q?mKhXXqeqMAuM4RqAUuPR7+Kx8zitLctGGtAvu36N800SY2VaZV2eNzMeZB?=
 =?iso-8859-1?Q?eg029dchyB04Jbof7CB1qEiejK0ub7e0kReF+W4dBGuwIc9RklI5Zwsfg0?=
 =?iso-8859-1?Q?77aa3hpAh9bOCDgtm7zIJGRScPtTGCgkVqkMx/o2VdlVtXSgoX8au8Gyeg?=
 =?iso-8859-1?Q?aURlXmZ8cBRcXYApGjBADD3sewT2FTcVn/AmJKd7iC+Vg8yPXybPeGSlA6?=
 =?iso-8859-1?Q?xJMW0AZYl464S1725ss5F9jaReUzO9T66/2dsJV25DRKmL75HDuGWP6PRm?=
 =?iso-8859-1?Q?+PR5TmNVeSfGkZGeXavWgRk//CTXAICx878kX/XExfthufH+Qy8PPzDu31?=
 =?iso-8859-1?Q?9VpL7pMUipsgPENgP7gT2YikRZFLc1al7UR1TSkjKc+V95xGm2hjfikrL5?=
 =?iso-8859-1?Q?dVQA0PUniR1fPWmNaHlwSlNVBJyZb5hzsn/P/NpEkPSNZ7mA2cCl9mFttU?=
 =?iso-8859-1?Q?kGVB+mBoL0r/zCvmr6iaD/A73EcAXFDWUI?=
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: SJ0PR03MB6778.namprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c5438dac-0d0f-4cc3-565e-08dabd97e5a8
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Nov 2022 12:35:31.6916
 (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: G17jVMI/PBdFf+dhnsbKf5wYs4JEgqVvXgnkQrVmXGvttvfOanxfYMylqHKeXCjxyqaXfmCRo/y8aphlRCsogw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO6PR03MB6257
X-Proofpoint-ORIG-GUID: hxpnvKsJevejRsZG4Ftuu216EITn3mkx
X-Proofpoint-GUID: hxpnvKsJevejRsZG4Ftuu216EITn3mkx
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.895,Hydra:6.0.545,FMLib:17.11.122.1
 definitions=2022-11-03_02,2022-11-03_01,2022-06-22_01
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 mlxscore=0 malwarescore=0
 suspectscore=0 bulkscore=0 phishscore=0 lowpriorityscore=0 mlxlogscore=999
 clxscore=1015 adultscore=0 impostorscore=0 priorityscore=1501 spamscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2210170000
 definitions=main-2211030086
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,SPF_HELO_NONE,SPF_PASS 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

> From: Ramona Bolboaca <ramona.bolboaca@analog.com>
> Sent: Thursday, November 3, 2022 9:09 AM
> To: jic23@kernel.org; linux-iio@vger.kernel.org; linux-
> kernel@vger.kernel.org
> Cc: Bolboaca, Ramona <Ramona.Bolboaca@analog.com>
> Subject: [PATCH v2 0/8] Remove adis_initial_startup usage
>=20
>=20
> Remove 'adis_initial_startup()' usage due to the fact that it leads to a
> deadlock.
> The same mutex is acquired twice, without releasing it, once inside
> 'adis_initial_startup()' and once inside 'adis_enable_irq()'.
> Instead of 'adis_initial_startup()', use '__adis_initial_startup()'.
>=20
> Ramona Bolboaca (8):
>   iio: accel: adis16201: Fix deadlock in probe
>   iio: accel: adis16209: Fix deadlock in probe
>   iio: gyro: adis16136: Fix deadlock in probe
>   iio: gyro: adis16260: Fix deadlock in probe
>   iio: imu: adis16400: Fix deadlock in probe
>   staging: iio: accel: adis16203: Fix deadlock in probe
>   staging: iio: accel: adis16240: Fix deadlock in probe
>   iio: imu: adis: Remove adis_initial_startup function
>=20
>  drivers/iio/accel/adis16201.c         |  2 +-
>  drivers/iio/accel/adis16209.c         |  2 +-
>  drivers/iio/gyro/adis16136.c          |  2 +-
>  drivers/iio/gyro/adis16260.c          |  2 +-
>  drivers/iio/imu/adis16400.c           |  2 +-
>  drivers/staging/iio/accel/adis16203.c |  2 +-
>  drivers/staging/iio/accel/adis16240.c |  2 +-
>  include/linux/iio/imu/adis.h          | 12 ------------
>  8 files changed, 7 insertions(+), 19 deletions(-)
>=20

You could have placed your v2 changelog in the cover letter.
Moreover it's the same for all patches... Anyways:=20

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

- Nuno S=E1