From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lr2-f12.google.com (mail-lr2-f12.google.com [74.125.230.76]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 991304FB9D7 for ; Mon, 21 Sep 2026 18:31:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.230.76 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790015505; cv=none; b=E65nOxlY3XeMvHYyLncJs5UbZMJmsFZdmWcugjHMi5yk27rJeZk7ExVYrRzsmtj+DXxiSL0E4S/s9C41kIdmm2y+7uUqugipxNTQ25PcZ4562h0wKImXjSWJirGdFVNgZQZuvUhTfFHubpVpfsWp3ohxrMwLxM0nx/F6/JR1Z0E= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790015505; c=relaxed/simple; bh=2f3W7ewvZD9vQ7bzg12iNZ75xkP2pWmWjFYvl7E8R3Q=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=ZmINCxSMlXwK/W/q0mxkYXvzdXShzG6ibsUotvJORrVBEwteUXscLkiXYk3thSxOccL1DaYtqbWutqCAXiQ07sxBDDOOIRSbK6S/CN8S6aob36x6NPTXz9it9DJNA7qftjaH07Tz4juJUZmdFdwm0NPVMh9xbXsf5uD3P38qHLw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=OSakWhV2; arc=none smtp.client-ip=74.125.230.76 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="OSakWhV2" Received: by mail-lr2-f12.google.com with SMTP id 38308e7fff4ca-3a2ff148dfdso21857481fa.1 for ; Mon, 21 Sep 2026 11:31:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1790015501; x=1790620301; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=2j1Tv4agtRXuOvbEOgkc069No6p6dhHi73ng2NMw2ME=; b=OSakWhV2yLjW3FW1HXrPiz1nWPCmH7q1wgcyraM8zegSxdjbeyr1qIXU4WO5RMcoN5 gKaqzU+sdezgoQl1Yqd53vJKwozUfP1Zxef5w/LZVzJD3YdP40GU53pEdITdfoSwhTbn heMbWN+aW62XX/6nU8reP8b4FVQlWmg+Cq998cAoz7ShqtKA/OwVgxAgOIZcVPr/TypO gxtMSLktoHWKcz8b4/IM3cstulCf1GsbWEqStiLxCb470ZlUkMu4jGXMpNQq8PD/o9C1 bMABk1qzGAig52PTFZspZCzczgbi80L8Akc63032I2MoTikgrfOUAf61ULG1vHoz91Wy tsdQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1790015501; x=1790620301; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=2j1Tv4agtRXuOvbEOgkc069No6p6dhHi73ng2NMw2ME=; b=V8l6nR/qjIpguh3JiCrfWp3ulRyHern53mgYgCjznnIYeJExC5Rk+SUVRZFk5ss5Xl uXslb+2wN2wSby0J4phv+/Ch7BuM0kyIVOh3Lpcr/9Pz3l1BhdwQDkb6C+ggFVCaJJzZ n4jFdxYSsnRtkmie9opw80Y2jAdf8ov60a9BKOnPuenZSDcIgVMVAlQYrPClwGWbr1TJ 1X/XNy397n5bs9Bdv8BG7I7ZcDWWUYWRgWi5CssOIWUhcpWjMSJerov43WnnWCPi6pnA beaCuYafO9DLlNrXjgSdXUSZca4Lo4jrvVtssL54KhSyXm/6gPI1l7OJFaZ32L82fYpn gXEw== X-Forwarded-Encrypted: i=1; AKwUvBzQmFEr4f80dWPIjyLYbaRGYzPlY2ZFr9sJTtkh9oZhM+krPxzwyw0FRJDaZWLqA9u8yZvzZ/vmtNDJr+s=@vger.kernel.org X-Gm-Message-State: AFuF++lUk0tfKCnP1tlLBWONM29uVX14/8qQP1gmjxE5K4cqzoGNFB6/ Lf+5Ug4Zs+es6jhxFADSp5o89y8sK7cP8f9d4dfJtCRa2t3IcOyH8hmQ X-Gm-Gg: AYBFou2irPecWt+m4BGyZhqJKIRkYa2WKcRTSkqoPhWTZ5RIFX/UvB3pGuFAqWgC01b m6papfcOc71vnOMb8Y0sMmWzsj7ISV+Amf+yj6L1JfTflHtcP3/jP3uV+b2nXhpJ3V2VeGIRmXo O613UiOHUcvUKHKn9hfaPgi1e4jGVpzQ1LpelNGXKtYLN0junvhqWVH+lKJIsjhWqxwCkBRFBGS PPfAU8sQ1Eyg/PT733X4MJGnnDdYkWaIvGrsCim2ZHdJJxwe3TnUw0jndWhVdnAZ3WRfgViI27r u9LMWEbSfDFJTnTpeUp10Jfsy+uIk4DJmQetJFWcWw9FapWsIyaOKy7CdlrDJzyEN9uBXJcGQ5h +K5K1papECpAZJTo5LLSS89qrFRCszCMi2ZUGH/t62WlJqDmfETQb4DLTjRSax2io6ZNqc/Tde1 dB8PcWgF+DwfWeu6/exgJcFOLkFCFcfpftn7VaULg6U41UjfTIRLfugrvBHLasC9RS6dh5YNa9I LbGkanHC0VEd0MhVM8QqsVCqxkd8fEfSYrTdqX1yK2SJu+Syvio59suUZ3HZJDlu7TUTw== X-Received: by 2002:a05:651c:1508:b0:3a6:74a:53cf with SMTP id 38308e7fff4ca-3a6074a5560mr19625551fa.28.1790015500539; Mon, 21 Sep 2026 11:31:40 -0700 (PDT) Received: from yantra.. (dsl-trebng21-58c187-236.dhcp.inet.fi. [88.193.135.236]) by smtp.gmail.com with ESMTPSA id 38308e7fff4ca-3a60761e5d6sm24260141fa.29.2026.09.21.11.31.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 21 Sep 2026 11:31:39 -0700 (PDT) From: Rupesh Majhi To: Andy Shevchenko , Bill Wendling , David Lechner , Eddie James , Joel Stanley , Jonathan Cameron , Justin Stitt , Nathan Chancellor , Nick Desaulniers , =?UTF-8?q?Nuno=20S=C3=A1?= Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, llvm@lists.linux.dev, Rupesh Majhi Subject: [PATCH v8 03/10] iio: pressure: dps310: use get_unaligned_be24() for the 24-bit results Date: Mon, 21 Sep 2026 21:31:25 +0300 Message-ID: <20260921183132.233136-4-zoone.rupert@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260921183132.233136-1-zoone.rupert@gmail.com> References: <20260921183132.233136-1-zoone.rupert@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Pressure and temperature results are big-endian 24-bit values. Build them with get_unaligned_be24() instead of open-coded shifts. The coefficients are packed into the bytes rather than byte-aligned, so they stay as they are. Assisted-by: LLM Signed-off-by: Rupesh Majhi --- drivers/iio/pressure/dps310.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/iio/pressure/dps310.c b/drivers/iio/pressure/dps310.c index bd7da4f12749..e7f173e08e01 100644 --- a/drivers/iio/pressure/dps310.c +++ b/drivers/iio/pressure/dps310.c @@ -19,6 +19,7 @@ #include #include #include +#include #include #include @@ -468,7 +469,6 @@ static int dps310_read_pres_raw(struct dps310_data *data) int rc; int rate; int timeout; - s32 raw; u8 val[3]; if (mutex_lock_interruptible(&data->lock)) @@ -489,8 +489,7 @@ static int dps310_read_pres_raw(struct dps310_data *data) if (rc < 0) goto done; - raw = (val[0] << 16) | (val[1] << 8) | val[2]; - data->pressure_raw = sign_extend32(raw, 23); + data->pressure_raw = sign_extend32(get_unaligned_be24(val), 23); done: mutex_unlock(&data->lock); @@ -502,14 +501,12 @@ static int dps310_read_temp_ready(struct dps310_data *data) { int rc; u8 val[3]; - s32 raw; rc = regmap_bulk_read(data->regmap, DPS310_TMP_BASE, val, sizeof(val)); if (rc < 0) return rc; - raw = (val[0] << 16) | (val[1] << 8) | val[2]; - data->temp_raw = sign_extend32(raw, 23); + data->temp_raw = sign_extend32(get_unaligned_be24(val), 23); return 0; } -- 2.43.0