From nobody Tue Oct 7 10:30:58 2025 Received: from mail-oi1-f177.google.com (mail-oi1-f177.google.com [209.85.167.177]) (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 8DD662BD027 for ; Thu, 10 Jul 2025 22:40:38 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.177 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1752187240; cv=none; b=lMF3yRzcunkgPc2aqt/sVevmelqry9Rocx31TZeFJeN+g4bD+WCu4F61um83pbpB9EMMxJ/H4gXFJksR6kxaY1W36K1cU7c34G5ntciupwDCJG45mkThJwXBGOtTs8SpnbDrr85PXdV6N8AbNlM93w2X9Pv71F4mrMZDjtEpygg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1752187240; c=relaxed/simple; bh=Y4xxzpQMXT0tVAh0B+9LP1ypOj3L5Cj0Clka4wYTVnI=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=kCDWag5Gfn0nhfL8TIilYb5XJxz4+zMXlc7ni1Uz7WHDNNboBHNa2Mze9IAESKtXlwwolHEgBXr2UZ5UWKkjYIAl79spjDEHSmyvxrdjy+37Mc2xtrvf7WH+aVzG2SrUz2uvifb2FBxfXl88ghG8uLZj56EBY371GytjnPRmW5U= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b=A3o4uWvp; arc=none smtp.client-ip=209.85.167.177 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b="A3o4uWvp" Received: by mail-oi1-f177.google.com with SMTP id 5614622812f47-40b99490728so1241789b6e.0 for ; Thu, 10 Jul 2025 15:40:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1752187237; x=1752792037; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=VIBjwMYnL74TcVrAH5e3IGRZZKgXqxeLaFBUWJ6E5yw=; b=A3o4uWvpwJbT7T61DXFECIZ2RvI/bz71hV+L7EfHspzSG4ickqRBcrzEwYv8yXDBkB d8Uda8sqcDcxeXxBE3gC2aQdqcpss0nkW4zIOvs5yR3GiYRNy2eVrgRVEkYJFgZuleCh xwFBeSrcMFRpNau3pmlUXVxuo7gT6fkot37bEY//dCLLJc865ZIEcDwnUV1njWDKmuw7 n26gCn8X8zq7JTw4viT8J4GoTuDw3u9rJRqqI8Jq03yETHB21BnNJh+ZRCR5xIeuNTX7 ea2VD5dLwFFZTKgpew5A2haz8422+XmFr6HqTTtRpfIC+d3D4iGzS5Ux7bETJorHWi1g A8eQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1752187237; x=1752792037; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=VIBjwMYnL74TcVrAH5e3IGRZZKgXqxeLaFBUWJ6E5yw=; b=siSwMLYB46NKhm6sTGEH6cmOqmBqr6/bEfLbjIJ0B7kVhuWSoppv2/PRPjZsf+Ls4T ZdKAqFC2U1HivFfxmTUYf3yYrmnmNOB4ckTNxzarBd9U6h2KPt8CdI6TWuwtgR+bkJTe uAU6JmJXiv4TkuByrDcUz3ddn8Z8fSpVQj9xmTH6GqJPrayZyUIoc33yMdfN67OWqOsv gHnfOoQ7oOS4wAKcxQMhnb7HVsoFFfIAPHrLF2OUSznhRKoEw/xELaYG6OrWOZS+a/CM cBKFO5aw6CW1ncxXs7HfA6I6nICigjh6DsRV4HUgHUUTD6jdkPw8o3IQW0l3pyORjJGJ IRFw== X-Forwarded-Encrypted: i=1; AJvYcCUukbAIa2x3ZhwQRnEj463D/diyORBxa3/HK3WxiVCdm4/GVbwCa1NUAAPPh6XZdujLYvOEh2cH2FvRtoM=@vger.kernel.org X-Gm-Message-State: AOJu0YyHm0LYClzedONNUCmxvPUwa6m7Ux4li5aFV1vonVVX/CueQTZe Ne5AW/5dAgT+YE3a8uJGZ/T2htcgFJDNHfbY2KtRarvyGv01KGNTmCd3kp1SBycUfYI= X-Gm-Gg: ASbGnct4z4cxiptlE1jPNdFUOh2tUwKoVLROCt2g7eWNwYr7OHyOnda7aEmmbL1NGB/ ZxOA1EiSL8GAExXwNPG4x/56p8ohlrYF82luYkmpFXT4ulaSL6JCISwxywVat2C3+vSHcWe1BeB lLvsdxkml437+dBLtsmRtsvoN+c2A593yjbTAoKG2pizHzFeLyK1Q4QBVhnvfpWQLywMq+xKTEG tZo6WA547w506jSif/A29ZkNc1QuKjEyCrsJrQo962dW1dM7buIJSfd6iai/JV4KQ5v0JmO342i QoEeEBlW8UtuDMAOmxyP66RW15cOOVizWK36BIz9ksZz1HWKNEZXxLMpfwzKtt04YFPPNfqJF5P zZMVoGG7PWk1vKZxj7IysGNe44Q== X-Google-Smtp-Source: AGHT+IE0jBieUnW+rRGfWxll6L52XTUO/aByQNjjFVxiGHTKx/oAlmLkC6wro7rhwPBufH+QA4JFrQ== X-Received: by 2002:a05:6808:220f:b0:3f8:bbf3:3a18 with SMTP id 5614622812f47-413c4890f1cmr3917546b6e.16.1752187237575; Thu, 10 Jul 2025 15:40:37 -0700 (PDT) Received: from [127.0.1.1] ([2600:8803:e7e4:1d00:891b:7836:c92:869]) by smtp.gmail.com with ESMTPSA id 006d021491bc7-613d9d950f1sm305944eaf.18.2025.07.10.15.40.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 10 Jul 2025 15:40:37 -0700 (PDT) From: David Lechner Date: Thu, 10 Jul 2025 17:39:51 -0500 Subject: [PATCH 2/5] iio: adc: ad7173: rename odr field Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250710-iio-adc-ad7137-add-filter-support-v1-2-acffe401c4d2@baylibre.com> References: <20250710-iio-adc-ad7137-add-filter-support-v1-0-acffe401c4d2@baylibre.com> In-Reply-To: <20250710-iio-adc-ad7137-add-filter-support-v1-0-acffe401c4d2@baylibre.com> To: Michael Hennerich , Jonathan Cameron , =?utf-8?q?Nuno_S=C3=A1?= , Andy Shevchenko Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, David Lechner X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=3350; i=dlechner@baylibre.com; h=from:subject:message-id; bh=Y4xxzpQMXT0tVAh0B+9LP1ypOj3L5Cj0Clka4wYTVnI=; b=owEBbQGS/pANAwAKAcLMIAH/AY/AAcsmYgBocEFG9srtwbvcID06yiIQNe6pU879Zuxtnvmfy fKROwb7ExiJATMEAAEKAB0WIQTsGNmeYg6D1pzYaJjCzCAB/wGPwAUCaHBBRgAKCRDCzCAB/wGP wKeSB/9qDmJmuCanZotA2Zn3hgDWFta7IAV0s/RKiWUXnhfr/+xEEaT5j/eOllCOubgJuNeUxA2 HjnlYeRd0bZSEmzYuN9KVWZJ4dVvQbkiKsLae01JHtM2VgLmCaTAKlmgOOsl229QmdGhCYJcHYX h7U0bswENjIP8oj32wBm4jktVe40Pq1MpvoliTqjkhFybA+iWMz7wsQnwprfaFG9ebDFHrM79EV 8R2fs7RdinBMV1AWhKQF20AjJViCjeTcveF1TpbJS0ty3IUxNvxafYtL//kPH7smNkt0pSvLkPC KqPooEsnl5KELdG5WuhwRpcAngy/Q6nuR1hBUz6c4jqf2Imf X-Developer-Key: i=dlechner@baylibre.com; a=openpgp; fpr=8A73D82A6A1F509907F373881F8AF88C82F77C03 Rename odr to sinc5_odr_index in the channel setup structure. In a following commit, we will be adding a separate odr field for when the sinc3 filter is used instead so having sinc5 in the name will help avoid confusion. And _index makes it more clear that this is an index of the sinc5_data_rates array and not the output data rate itself. Signed-off-by: David Lechner --- drivers/iio/adc/ad7173.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/iio/adc/ad7173.c b/drivers/iio/adc/ad7173.c index 5daf21c6ba5637b2e47dcd052bdd019c3ecbb442..01d78d531d6c024dd92fff21b8b= 2afb750092b66 100644 --- a/drivers/iio/adc/ad7173.c +++ b/drivers/iio/adc/ad7173.c @@ -205,7 +205,7 @@ struct ad7173_channel_config { struct_group(config_props, bool bipolar; bool input_buf; - u8 odr; + u8 sinc5_odr_index; u8 ref_sel; ); }; @@ -582,13 +582,13 @@ static bool ad7173_setup_equal(const struct ad7173_ch= annel_config *cfg1, sizeof(struct { bool bipolar; bool input_buf; - u8 odr; + u8 sinc5_odr_index; u8 ref_sel; })); =20 return cfg1->bipolar =3D=3D cfg2->bipolar && cfg1->input_buf =3D=3D cfg2->input_buf && - cfg1->odr =3D=3D cfg2->odr && + cfg1->sinc5_odr_index =3D=3D cfg2->sinc5_odr_index && cfg1->ref_sel =3D=3D cfg2->ref_sel; } =20 @@ -650,7 +650,7 @@ static int ad7173_load_config(struct ad7173_state *st, return ret; =20 return ad_sd_write_reg(&st->sd, AD7173_REG_FILTER(free_cfg_slot), 2, - AD7173_FILTER_ODR0_MASK & cfg->odr); + AD7173_FILTER_ODR0_MASK & cfg->sinc5_odr_index); } =20 static int ad7173_config_channel(struct ad7173_state *st, int addr) @@ -1183,7 +1183,7 @@ static int ad7173_read_raw(struct iio_dev *indio_dev, return -EINVAL; } case IIO_CHAN_INFO_SAMP_FREQ: - reg =3D st->channels[chan->address].cfg.odr; + reg =3D st->channels[chan->address].cfg.sinc5_odr_index; =20 *val =3D st->info->sinc5_data_rates[reg] / MILLI; *val2 =3D (st->info->sinc5_data_rates[reg] % MILLI) * (MICRO / MILLI); @@ -1229,7 +1229,7 @@ static int ad7173_write_raw(struct iio_dev *indio_dev, break; =20 cfg =3D &st->channels[chan->address].cfg; - cfg->odr =3D i; + cfg->sinc5_odr_index =3D i; cfg->live =3D false; break; =20 @@ -1655,7 +1655,7 @@ static int ad7173_fw_parse_channel_config(struct iio_= dev *indio_dev) chan_st_priv->cfg.bipolar =3D false; chan_st_priv->cfg.input_buf =3D st->info->has_input_buf; chan_st_priv->cfg.ref_sel =3D AD7173_SETUP_REF_SEL_INT_REF; - chan_st_priv->cfg.odr =3D st->info->odr_start_value; + chan_st_priv->cfg.sinc5_odr_index =3D st->info->odr_start_value; chan_st_priv->cfg.openwire_comp_chan =3D -1; st->adc_mode |=3D AD7173_ADC_MODE_REF_EN; if (st->info->data_reg_only_16bit) @@ -1727,7 +1727,7 @@ static int ad7173_fw_parse_channel_config(struct iio_= dev *indio_dev) chan->scan_index =3D chan_index; chan->channel =3D ain[0]; chan_st_priv->cfg.input_buf =3D st->info->has_input_buf; - chan_st_priv->cfg.odr =3D st->info->odr_start_value; + chan_st_priv->cfg.sinc5_odr_index =3D st->info->odr_start_value; chan_st_priv->cfg.openwire_comp_chan =3D -1; =20 chan_st_priv->cfg.bipolar =3D fwnode_property_read_bool(child, "bipolar"= ); --=20 2.43.0