From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5ED171DE3BB for ; Sun, 24 Aug 2025 16:39:32 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756053572; cv=none; b=T+q6m4o2Oy6lNL3q3C+aLRbVaVTPrdMBhIK0u+U63asFNqBjxkeP4bDivmJFBbr8dMkhYJsycAJsvHImcmkLUdRFbWZbJRwHc6xE0uRI+MwJJL9epVXWOwC+J21WjR3KSs0DQMVmi+uF3tZqqHCsys0jzZfZE+9i7SUpiTBkCCk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756053572; c=relaxed/simple; bh=xr8yoJnCqK8XTgilfNmdiGE8Gn6IGFLEQ4OQw/l6TG8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=j81H1z3LEKKYTLQF35gBZ2yRxcR95rDUeEBgyvXikeTGLJbAg1CB5Zd0jOrGkaEGG3nBJt7wXK1pppOFDOS7iKufoKWb5cjCQDbJdsL2WJ0vypzrjYQpY4XXCEhQzZXFPaASIBipSNy98W6/EZnT6sCyqkv+zXlYyaVQKyg5s3c= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=AfH1+GT9; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="AfH1+GT9" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4DFA9C4CEEB; Sun, 24 Aug 2025 16:39:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1756053572; bh=xr8yoJnCqK8XTgilfNmdiGE8Gn6IGFLEQ4OQw/l6TG8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=AfH1+GT9J9Prb0RVEZmEure02vM8SS84JODEisVt3TLCDhV9pvv1DwwZBI5aLSa6y HOTktnYoW+xRkGGdPUoLYdlUk6xBkuqVCCKIWV6xXdKjVjlhJbVbd9EmrRO3EKdVQT 49oLig7lq3qNy8uBKWS75YB0iIG/AdgPgqA0KFYqy9Gpy/ZlJYVzxkRBM2r+u4Jgry LrvCXWH5Ro+O/EjLycUG/ZHqHNfsp5z700Sk5NXSvR7h1rOlPihYUCMLSJECGU5uIt GYIo1BZYDdisJ13oIdr1OmJkxEYEUP1fk3oQPxlkQtj12GlM5l47QcJ1x8p0qzClcb lrqAcDCa3atWg== From: Sasha Levin To: stable@vger.kernel.org Cc: Jean-Baptiste Maneyrol , Andy Shevchenko , Sean Nyekjaer , Jonathan Cameron , Sasha Levin Subject: [PATCH 5.10.y] iio: imu: inv_icm42600: change invalid data error to -EBUSY Date: Sun, 24 Aug 2025 12:39:29 -0400 Message-ID: <20250824163929.4031504-1-sashal@kernel.org> X-Mailer: git-send-email 2.50.1 In-Reply-To: <2025082315-delirious-sandy-654a@gregkh> References: <2025082315-delirious-sandy-654a@gregkh> Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Jean-Baptiste Maneyrol [ Upstream commit dfdc31e7ccf3ac1d5ec01d5120c71e14745e3dd8 ] Temperature sensor returns the temperature of the mechanical parts of the chip. If both accel and gyro are off, the temperature sensor is also automatically turned off and returns invalid data. In this case, returning -EBUSY error code is better then -EINVAL and indicates userspace that it needs to retry reading temperature in another context. Fixes: bc3eb0207fb5 ("iio: imu: inv_icm42600: add temperature sensor support") Signed-off-by: Jean-Baptiste Maneyrol Cc: stable@vger.kernel.org Reviewed-by: Andy Shevchenko Reviewed-by: Sean Nyekjaer Link: https://patch.msgid.link/20250808-inv-icm42600-change-temperature-error-code-v1-1-986fbf63b77d@tdk.com Signed-off-by: Jonathan Cameron [ Adjust context ] Signed-off-by: Sasha Levin --- drivers/iio/imu/inv_icm42600/inv_icm42600_temp.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/iio/imu/inv_icm42600/inv_icm42600_temp.c b/drivers/iio/imu/inv_icm42600/inv_icm42600_temp.c index 91f0f381082b..8926b48d7661 100644 --- a/drivers/iio/imu/inv_icm42600/inv_icm42600_temp.c +++ b/drivers/iio/imu/inv_icm42600/inv_icm42600_temp.c @@ -32,8 +32,12 @@ static int inv_icm42600_temp_read(struct inv_icm42600_state *st, int16_t *temp) goto exit; *temp = (int16_t)be16_to_cpup(raw); + /* + * Temperature data is invalid if both accel and gyro are off. + * Return -EBUSY in this case. + */ if (*temp == INV_ICM42600_DATA_INVALID) - ret = -EINVAL; + ret = -EBUSY; exit: mutex_unlock(&st->lock); -- 2.50.1