From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f171.google.com (mail-pf1-f171.google.com [209.85.210.171]) (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 6C56D3B47D5 for ; Mon, 27 Apr 2026 10:10:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.171 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777284623; cv=none; b=JjVrBbYilXvKHdOnCDIvGggixrmBfdho1wyIkB4r8mQx+ViTWhcMZgS9sUFW0MQsADPaQunaBBs8QT6RwmSSz1B0MK1DT3+whbJxLvHKNfJ1BKsyriEeURMm15Op6jaWgQ6ZXs7ahOQ5Ho+cIxfWe1wEphBptYK3qliv1LJ+HDQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777284623; c=relaxed/simple; bh=blwxWP3h9amTde2/60twniELsnmcE0y3nMKAe9J7CRQ=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=qX0VjCyWcThEoJ10BjNOWPljEGnNiblW9sqYFPTHshrOl0tlhIc/PaCuE1qSl3EHD4pzcQQKLLIvIdXo7dcNVrxcanUPi4IHjLpsRINf8C+oQXkgv0anO1cilGS93OClWHOby0Xlb7YVEV51o3ckgDUA5B6KmpQDQlE9gcItuwI= 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=dHFi/5vs; arc=none smtp.client-ip=209.85.210.171 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="dHFi/5vs" Received: by mail-pf1-f171.google.com with SMTP id d2e1a72fcca58-8296d553142so5489858b3a.3 for ; Mon, 27 Apr 2026 03:10:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1777284622; x=1777889422; 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; bh=ErQtAdu2cnmpY8o88AkDpSJug3ccvxHHCZ8c7OJd9d0=; b=dHFi/5vs8j1aCm7U98CaBkzf5T87T+290C7/cQyTrNNwIQke+1qohmAjOeXjJu9KCN uOmj6xXTxj98LjcloaoaXpwTcdFWxPzcZuluu2jA3zbTKzarhCSjoIrL99Kv6ZIcsrRu a9baiqHm46shnl3HSTM3gOKMmU9f6vWL0SJZ6B3YEmbfTKJSZUlLMoCJWBSmIJ7vID5H oKKOUJL0DjwLrGa7QXJQlG12I7uS5P491Zs+463GRdedqPsFpad35dl4XBZ3u4CWqE/+ kXznIyv0/TMLBoDxy3pPvhwbenZpqMCL68UFcxMqac78qOHXq97P3eesvNubPz0uGFPj apgg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777284622; x=1777889422; 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; bh=ErQtAdu2cnmpY8o88AkDpSJug3ccvxHHCZ8c7OJd9d0=; b=ee2n1fS9UzS6Fk5Y219XLFyi35vzjUiP1Y+R4NDpAUJDpegx3CpFkAvNzCik4sH3JL GP7asGWyCqbLHkjwc0K2j04OzjoOqYWlxpip97V0uZ+U0YCfzCaiqZnUYOZ/3Y9rt7qs iDoHtoprly676tYjTBW46QCm34bj05Mh0e07GHUgRRRHT2BlrHH75UOHH7rkvzo7TKOK tl54YBgHswAhJ5EEhBdt/jpKdhFDGs26n3ksVjVSiPZbpvCTKDPP4k5HRKq/lt+5H45+ +9P5OzjxwpqCVngDa16D7Q+sSdXOKuJMwIuHgp7fiCp8BezYa5zAvIRMBdUrzzwdYdKa EO6A== X-Forwarded-Encrypted: i=1; AFNElJ9ZSBzCTAsLfj3mT1wm4GnFMqh3Ke0lMV8Ze39rDIfjeWFvcUYEae/gI/gcNOzVKxzlAvktNrCqGVA=@vger.kernel.org X-Gm-Message-State: AOJu0YxP4P7L49wgJ2SiYG2DYlVRs45a4RhUPs+cyL5Rcf2AjNjpww75 GHA7f/6XIKweCTu1yv4heobnfJw7nvWdzoFfPR1WqEauwA7wp6vNieA7 X-Gm-Gg: AeBDieujDwxRRiaobE7OGCBPsWUUsdrxkVPnihSYMu+ZugseUnNqRvZHAp7LrHB1WSW zf/h+Ix8YejoViT4R+vojkwOlSWH/YxaoQdr2W6D9NO/+h1dEbYZZA16WVRCYg1MT5JtK1CtVvi vX/D2IhpqQQoERzM8insZo3rbwLRwOfROfHb3DHSlKUf+9yrV4/ivrQMjhOsoMO6kdCIPzGOPe+ JpHFNNOT3t/HvQpByNd4nFyDG1H0PpMdpC+SFmMVoPsbuVG+1wqw5ZJuqvFfMvzJLXqRSFtenFU C0PGT2YiMjTUqSFJYXTPy7Hk/FM4kq/LhDz8ZJ2ozh5u0+E0CD/vs5aFq5G/kY3WDjj8GDCmmA+ 9wBn3bMwJJWj3R8bNuPNmMOSG6DTN999mnMWtesuQeRY42KiWi1zZhIRzsoO/5BmTL7+WTVp2AD Z8o4dYh3nn+d0eKHbses4X2OOtuKwKzyXXMHNSAE1bRSXpsLjTCLKAVS+2YrXgeaXgvn/DJ4Qhc 0CpvJWCL9V+2VegpTQHoq6LvXa9JUxEJlmuhqFALZXubwJD/FerX4xEXRF4 X-Received: by 2002:a05:6a00:439b:b0:82f:8696:5a75 with SMTP id d2e1a72fcca58-82f8c8bb700mr44837198b3a.28.1777284621666; Mon, 27 Apr 2026 03:10:21 -0700 (PDT) Received: from lord-daniel-VivoBook-ASUSLaptop-K3502ZA-S3502ZA.. ([2405:201:31:d016:e577:22da:dc9:7f6c]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-82f8ebba485sm39534225b3a.38.2026.04.27.03.10.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 27 Apr 2026 03:10:20 -0700 (PDT) From: Piyush Patle To: ak@it-klinger.de, jic23@kernel.org Cc: dlechner@baylibre.com, nuno.sa@analog.com, andy@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v4 2/7] iio: adc: hx711: move scale computation to per-device storage Date: Mon, 27 Apr 2026 15:39:33 +0530 Message-ID: <20260427100950.33936-3-piyushpatle228@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260427100950.33936-1-piyushpatle228@gmail.com> References: <20260427100950.33936-1-piyushpatle228@gmail.com> Precedence: bulk X-Mailing-List: linux-iio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit The gain-to-scale table hx711_gain_to_scale[] is a global array whose .scale fields are overwritten in hx711_probe() using the AVDD voltage read at probe time. When two HX711 sensors are connected to supplies at different voltages, the second probe call overwrites the scale values computed for the first sensor, silently corrupting its readings. Fix this by removing the .scale field from the global table, making the table const, and adding a per-instance gain_scale[] array to hx711_data. Populate gain_scale[] in hx711_probe() using the device's own AVDD regulator voltage. Update hx711_get_gain_to_scale() and hx711_get_scale_to_gain() to take the per-instance array as a parameter, and update hx711_scale_available_show() to retrieve it via iio_priv(). No functional change for single-sensor configurations. Signed-off-by: Piyush Patle --- Changes in v4: - New patch. Fixes a pre-existing bug reported during v3 review: the global hx711_gain_to_scale[].scale is overwritten on each probe call, breaking multiple sensor instances with different AVDD supplies. --- drivers/iio/adc/hx711.c | 32 +++++++++++++++++--------------- 1 file changed, 17 insertions(+), 15 deletions(-) diff --git a/drivers/iio/adc/hx711.c b/drivers/iio/adc/hx711.c index 1db8b68a8f64..0776a23bf59b 100644 --- a/drivers/iio/adc/hx711.c +++ b/drivers/iio/adc/hx711.c @@ -28,22 +28,20 @@ struct hx711_gain_to_scale { int gain; int gain_pulse; - int scale; int channel; }; /* * .scale depends on AVDD which in turn is known as soon as the regulator - * is available - * therefore we set .scale in hx711_probe() + * is available; it is stored per device in hx711_data.gain_scale[] * * channel A in documentation is channel 0 in source code * channel B in documentation is channel 1 in source code */ -static struct hx711_gain_to_scale hx711_gain_to_scale[HX711_GAIN_MAX] = { - { 128, 1, 0, 0 }, - { 32, 2, 0, 1 }, - { 64, 3, 0, 0 } +static const struct hx711_gain_to_scale hx711_gain_to_scale[HX711_GAIN_MAX] = { + { 128, 1, 0 }, + { 32, 2, 1 }, + { 64, 3, 0 } }; static int hx711_get_gain_to_pulse(int gain) @@ -56,22 +54,22 @@ static int hx711_get_gain_to_pulse(int gain) return 1; } -static int hx711_get_gain_to_scale(int gain) +static int hx711_get_gain_to_scale(const int *gain_scale, int gain) { int i; for (i = 0; i < HX711_GAIN_MAX; i++) if (hx711_gain_to_scale[i].gain == gain) - return hx711_gain_to_scale[i].scale; + return gain_scale[i]; return 0; } -static int hx711_get_scale_to_gain(int scale) +static int hx711_get_scale_to_gain(const int *gain_scale, int scale) { int i; for (i = 0; i < HX711_GAIN_MAX; i++) - if (hx711_gain_to_scale[i].scale == scale) + if (gain_scale[i] == scale) return hx711_gain_to_scale[i].gain; return -EINVAL; } @@ -82,6 +80,7 @@ struct hx711_data { struct gpio_desc *gpiod_dout; int gain_set; /* gain set on device */ int gain_chan_a; /* gain for channel A */ + int gain_scale[HX711_GAIN_MAX]; struct mutex lock; /* * triggered buffer @@ -290,7 +289,8 @@ static int hx711_read_raw(struct iio_dev *indio_dev, *val = 0; mutex_lock(&hx711_data->lock); - *val2 = hx711_get_gain_to_scale(hx711_data->gain_set); + *val2 = hx711_get_gain_to_scale(hx711_data->gain_scale, + hx711_data->gain_set); mutex_unlock(&hx711_data->lock); @@ -321,7 +321,7 @@ static int hx711_write_raw(struct iio_dev *indio_dev, mutex_lock(&hx711_data->lock); - gain = hx711_get_scale_to_gain(val2); + gain = hx711_get_scale_to_gain(hx711_data->gain_scale, val2); if (gain < 0) { mutex_unlock(&hx711_data->lock); return gain; @@ -388,12 +388,14 @@ static ssize_t hx711_scale_available_show(struct device *dev, { struct iio_dev_attr *iio_attr = to_iio_dev_attr(attr); int channel = iio_attr->address; + struct hx711_data *hx711_data = + iio_priv(dev_to_iio_dev(dev)); int i, len = 0; for (i = 0; i < HX711_GAIN_MAX; i++) if (hx711_gain_to_scale[i].channel == channel) len += sprintf(buf + len, "0.%09d ", - hx711_gain_to_scale[i].scale); + hx711_data->gain_scale[i]); len += sprintf(buf + len, "\n"); @@ -511,7 +513,7 @@ static int hx711_probe(struct platform_device *pdev) ret *= 100; for (i = 0; i < HX711_GAIN_MAX; i++) - hx711_gain_to_scale[i].scale = + hx711_data->gain_scale[i] = ret / hx711_gain_to_scale[i].gain / 1678; hx711_data->gain_set = 128; -- 2.43.0