From nobody Thu Apr 2 17:16:39 2026 Received: from mail-dy1-f174.google.com (mail-dy1-f174.google.com [74.125.82.174]) (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 B2A54375F9A for ; Fri, 6 Mar 2026 07:17:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.82.174 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772781471; cv=none; b=nMsOIvgnyBwPIeTe7na9OT8jX3VhpNmJm3FdPy74tEwyNl0YiY7pQuFrwKuZy3F+Hv/uDB7lTYeAijW1tEssqbICg7lq0fhn3YkDJio24fKt6pihWS06+Q6cjiamt8y0wUjQ+eXWLM3u0i5uHfh2t8NXLpbw0Txb7rGRRUof8Eg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772781471; c=relaxed/simple; bh=FnnVUipcnXp2uwXZ+svU7qIgeUZRJVxaE3eFoy49gvk=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=fX2UIpF2km1AwyD33KJ7KCiWoAbDMdfRUJng2lgyHm7T8G2+Yv8APum00G/P6kCex9Rs7TS0rsl7GO02rzvPw/QmK5bI3uM/QtSQxiqF/KpfJWRueYQJusXzuEQTzPNKCnjQr70IYmJ0khed775J0GZzayP7OwhkXi7/6r/WIGo= 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=IS2j+4su; arc=none smtp.client-ip=74.125.82.174 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="IS2j+4su" Received: by mail-dy1-f174.google.com with SMTP id 5a478bee46e88-2be1c918173so8069823eec.1 for ; Thu, 05 Mar 2026 23:17:49 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1772781469; x=1773386269; 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=/ZgrgHECWYzN34F/29U4qOQ3ECWsuSmnQ/TM4j8ot7k=; b=IS2j+4suZ+lJSfCwmA0Pxk31ltBBL/S7GHgLt+hlY6ubw8weo8rrQJv6NXE/yWD1tI dAa5LZGtjoynF7G7rUkiGDNe2IiDDuM08FRDMG1U/dCWQzaVmhKJVo8wooNBQ2TMHkzP bRkGfE4OlMf9Jy5A19ea+erXc5rJMEM0AZAYE0HlesQtcHriAimmdWfltJhdPPXvDWig qnuP7DLyyOP6T9qfGDzRFcVxS8V/mQeUhxgEz9tc2krvBlg2fDJeOr1YB+rRjso+XOds MiD+BwqVQTdU+1jqLHgiODCHBG6hLbVAtdcaXBH1j0rQsrFPem3ZDvcvY0Qx+/fUS5iD nY/Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1772781469; x=1773386269; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-gg:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=/ZgrgHECWYzN34F/29U4qOQ3ECWsuSmnQ/TM4j8ot7k=; b=tlsd/xuvlp3Ix6UpslvHa4kuOdXHy1XwCh+6XpT/04XurlVD9fZgyjL26bSgjpCv2E cm+1TRuIAQOopMuIN7pC67hs0PVuIGK3w6GViZ0sZrxkzDRySFw3CHn0wxFIPjL023dt DcU0YZylgP0NXGipV9dFWPusJOWmQ2lEfk96k2yl4deEuPhlSeZtPXxA1R00I9wZ5rC3 cCSPuHJuGUXvwKZSCa3CaPNj3JXnikpZXZ9jrUlo5CzY33wPgYczxO6CxDkKMEs6+gGo d3815ITU1MxTE4aVtBfNzo9HwXvOtDhnxRx1D1YwUBa8IBp+CGuCrJ3BtvA0VGq/BbAN SIfQ== X-Forwarded-Encrypted: i=1; AJvYcCXbsKN7/VLqvWwpuxOqyOiz7AQNGKL5h8nA6BhkqfvkH7ENGMll17DRrmDYgPkGI38KU2ubkV/Zbn4MWhs=@vger.kernel.org X-Gm-Message-State: AOJu0YyD5mzzUyg3nJ5xoEn5g+L4gaamG7m+OknYxRJelUcx5zwcK9Cn 46Uz5F5xRrxFUCjSjLFqs3vdWHjYUkvkYUbI2XFP0cuKEDzQb/r2d3yu X-Gm-Gg: ATEYQzzwQWbXixzwrB7dFACbAOCVLD7RGcn6ZLYXQsGmPQChtF2/z3czodu6hYZIivD xr9smVsQfjH412gS4NXMobYjZfrwlaRq/E5N0ph8El/MsvKVytTZ6EC59Z6TpmZkNjmnDJuIyDI kq/H1a/1S3sZNGkCSJiGcK34cUMwuw2nGOqUex3fw/E+6OvoAgvrKNnsXewBExxFZwlWhryZzrA eSggyuQi6kKEom/S53IyFKk8KC2APXtHbvTe3hfQtK03M2rZDubYma9H3DrIJcp8ynDMM7AKxtt ZczlurErG2ojIOKpoP1hBXkgI/NbUcCllk/1GgDls4/FLKseqCJ7mX2FAeMatSka7B9GclOoRfI Onby0PBy9isb6Tsam5WrUq221VU43azVoxmybDjpaJ+qVbWzTXLXYjHC9/WW0fUCb71x9iNgC4l LxzGY36F0CMF4i0mNEImjWV9BxzyGttbJbsSXO+A== X-Received: by 2002:a05:7301:1f12:b0:2b7:f809:9c3f with SMTP id 5a478bee46e88-2be4e03d117mr546570eec.24.1772781468675; Thu, 05 Mar 2026 23:17:48 -0800 (PST) Received: from [192.168.1.187] ([76.50.196.152]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-2be4f96f25bsm545713eec.28.2026.03.05.23.17.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 05 Mar 2026 23:17:48 -0800 (PST) From: Michael Harris Date: Thu, 05 Mar 2026 23:17:00 -0800 Subject: [PATCH v2 3/4] staging: iio: adt7316: add config names to registers and reorder 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: <20260305-adt7316-correct-macros-v2-3-3702e3841c42@gmail.com> References: <20260305-adt7316-correct-macros-v2-0-3702e3841c42@gmail.com> In-Reply-To: <20260305-adt7316-correct-macros-v2-0-3702e3841c42@gmail.com> To: Lars-Peter Clausen , Michael Hennerich , Jonathan Cameron , Greg Kroah-Hartman Cc: David Lechner , =?utf-8?q?Nuno_S=C3=A1?= , Andy Shevchenko , linux-iio@vger.kernel.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Michael Harris X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1772781459; l=30783; i=michaelharriscode@gmail.com; s=20260130; h=from:subject:message-id; bh=FnnVUipcnXp2uwXZ+svU7qIgeUZRJVxaE3eFoy49gvk=; b=2kNg/qcuMdazf5pZYi+fpH4hNsPd5QizJoqKWEecoTJD2hHF/i9Ctby5DqmmZnMC8s+kUnMEr UqpJQjgtURoALK5Jb73QZF3yfi7HUoslVqEFCvA7DXHUdHNrIFnPMQL X-Developer-Key: i=michaelharriscode@gmail.com; a=ed25519; pk=td/zQD3XANAhdG3Kf3mSetXiynk1Ql0eM4s0+eks5DU= Add config names to macros to make it more clear which register they're affecting. Also renamed ADT7316_EN to further clarify what it's enabling. Some macros were reordered, so that mask values were below the actual mask. Signed-off-by: Michael Harris --- drivers/staging/iio/addac/adt7316.c | 299 +++++++++++++++++++-------------= ---- 1 file changed, 154 insertions(+), 145 deletions(-) diff --git a/drivers/staging/iio/addac/adt7316.c b/drivers/staging/iio/adda= c/adt7316.c index b8b66f4dd14bb59c3d29fdd569d84f0dd786db9e..ea7d97bb0f378f3a3ce1225d8d1= 3af2c5955ca56 100644 --- a/drivers/staging/iio/addac/adt7316.c +++ b/drivers/staging/iio/addac/adt7316.c @@ -90,61 +90,61 @@ /* * ADT7316 config1 */ -#define ADT7316_EN 0x1 -#define ADT7516_SEL_EX_TEMP 0x4 -#define ADT7516_SEL_AIN1_2_EX_TEMP_MASK 0x6 -#define ADT7516_SEL_AIN3 0x8 -#define ADT7316_INT_EN 0x20 -#define ADT7316_INT_POLARITY 0x40 -#define ADT7316_PD 0x80 +#define ADT7316_CONFIG1_MONITOR_EN 0x1 +#define ADT7516_CONFIG1_SEL_AIN1_2_EX_TEMP_MASK 0x6 +#define ADT7516_CONFIG1_SEL_EX_TEMP 0x4 +#define ADT7516_CONFIG1_SEL_AIN3 0x8 +#define ADT7316_CONFIG1_INT_EN 0x20 +#define ADT7316_CONFIG1_INT_POLARITY 0x40 +#define ADT7316_CONFIG1_PD 0x80 =20 /* * ADT7316 config2 */ -#define ADT7316_AD_SINGLE_CH_MASK 0x3 -#define ADT7516_AD_SINGLE_CH_MASK 0x7 -#define ADT7316_AD_SINGLE_CH_VDD 0 -#define ADT7316_AD_SINGLE_CH_IN 1 -#define ADT7316_AD_SINGLE_CH_EX 2 -#define ADT7516_AD_SINGLE_CH_AIN1 2 -#define ADT7516_AD_SINGLE_CH_AIN2 3 -#define ADT7516_AD_SINGLE_CH_AIN3 4 -#define ADT7516_AD_SINGLE_CH_AIN4 5 -#define ADT7316_AD_SINGLE_CH_MODE 0x10 -#define ADT7316_DISABLE_AVERAGING 0x20 -#define ADT7316_EN_SMBUS_TIMEOUT 0x40 -#define ADT7316_RESET 0x80 +#define ADT7316_CONFIG2_AD_SINGLE_CH_MASK 0x3 +#define ADT7516_CONFIG2_AD_SINGLE_CH_MASK 0x7 +#define ADT7316_CONFIG2_AD_SINGLE_CH_VDD 0 +#define ADT7316_CONFIG2_AD_SINGLE_CH_IN 1 +#define ADT7316_CONFIG2_AD_SINGLE_CH_EX 2 +#define ADT7516_CONFIG2_AD_SINGLE_CH_AIN1 2 +#define ADT7516_CONFIG2_AD_SINGLE_CH_AIN2 3 +#define ADT7516_CONFIG2_AD_SINGLE_CH_AIN3 4 +#define ADT7516_CONFIG2_AD_SINGLE_CH_AIN4 5 +#define ADT7316_CONFIG2_AD_SINGLE_CH_MODE 0x10 +#define ADT7316_CONFIG2_DISABLE_AVERAGING 0x20 +#define ADT7316_CONFIG2_EN_SMBUS_TIMEOUT 0x40 +#define ADT7316_CONFIG2_RESET 0x80 =20 /* * ADT7316 config3 */ -#define ADT7316_ADCLK_22_5 0x1 -#define ADT7316_DA_HIGH_RESOLUTION 0x2 -#define ADT7316_DA_EN_VIA_DAC_LDAC 0x8 -#define ADT7516_AIN_IN_VREF 0x10 -#define ADT7316_EN_IN_TEMP_PROP_DACA 0x20 -#define ADT7316_EN_EX_TEMP_PROP_DACB 0x40 +#define ADT7316_CONFIG3_ADCLK_22_5 0x1 +#define ADT7316_CONFIG3_DA_HIGH_RESOLUTION 0x2 +#define ADT7316_CONFIG3_DA_EN_VIA_DAC_LDAC 0x8 +#define ADT7516_CONFIG3_AIN_IN_VREF 0x10 +#define ADT7316_CONFIG3_EN_IN_TEMP_PROP_DACA 0x20 +#define ADT7316_CONFIG3_EN_EX_TEMP_PROP_DACB 0x40 =20 /* * ADT7316 DAC config */ -#define ADT7316_DA_2VREF_CH_MASK 0xF -#define ADT7316_DA_EN_MODE_MASK 0x30 -#define ADT7316_DA_EN_MODE_SINGLE 0x00 -#define ADT7316_DA_EN_MODE_AB_CD 0x10 -#define ADT7316_DA_EN_MODE_ABCD 0x20 -#define ADT7316_DA_EN_MODE_LDAC 0x30 -#define ADT7316_VREF_BYPASS_DAC_AB 0x40 -#define ADT7316_VREF_BYPASS_DAC_CD 0x80 +#define ADT7316_DAC_CONFIG_2VREF_CH_MASK 0xF +#define ADT7316_DAC_CONFIG_EN_MODE_MASK 0x30 +#define ADT7316_DAC_CONFIG_EN_MODE_SINGLE 0x00 +#define ADT7316_DAC_CONFIG_EN_MODE_AB_CD 0x10 +#define ADT7316_DAC_CONFIG_EN_MODE_ABCD 0x20 +#define ADT7316_DAC_CONFIG_EN_MODE_LDAC 0x30 +#define ADT7316_DAC_CONFIG_VREF_BYPASS_AB 0x40 +#define ADT7316_DAC_CONFIG_VREF_BYPASS_CD 0x80 =20 /* * ADT7316 LDAC config */ -#define ADT7316_LDAC_EN_DA_MASK 0xF -#define ADT7316_DAC_IN_VREF 0x10 -#define ADT7516_DAC_AB_IN_VREF 0x10 -#define ADT7516_DAC_CD_IN_VREF 0x20 -#define ADT7516_DAC_IN_VREF_MASK 0x30 +#define ADT7316_LDAC_CONFIG_EN_DA_MASK 0xF +#define ADT7316_LDAC_CONFIG_DAC_IN_VREF 0x10 +#define ADT7516_LDAC_CONFIG_DAC_AB_IN_VREF 0x10 +#define ADT7516_LDAC_CONFIG_DAC_CD_IN_VREF 0x20 +#define ADT7516_LDAC_CONFIG_DAC_IN_VREF_MASK 0x30 =20 /* * ADT7316 INT_MASK2 @@ -219,7 +219,8 @@ static ssize_t adt7316_show_enabled(struct device *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - return sysfs_emit(buf, "%d\n", !!(chip->config1 & ADT7316_EN)); + return sysfs_emit(buf, "%d\n", + !!(chip->config1 & ADT7316_CONFIG1_MONITOR_EN)); } =20 static ssize_t _adt7316_store_enabled(struct adt7316_chip_info *chip, @@ -229,9 +230,9 @@ static ssize_t _adt7316_store_enabled(struct adt7316_ch= ip_info *chip, int ret; =20 if (enable) - config1 =3D chip->config1 | ADT7316_EN; + config1 =3D chip->config1 | ADT7316_CONFIG1_MONITOR_EN; else - config1 =3D chip->config1 & ~ADT7316_EN; + config1 =3D chip->config1 & ~ADT7316_CONFIG1_MONITOR_EN; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_CONFIG1, config1); if (ret) @@ -277,7 +278,8 @@ static ssize_t adt7316_show_select_ex_temp(struct devic= e *dev, if ((chip->id & ID_FAMILY_MASK) !=3D ID_ADT75XX) return -EPERM; =20 - return sysfs_emit(buf, "%d\n", !!(chip->config1 & ADT7516_SEL_EX_TEMP)); + return sysfs_emit(buf, "%d\n", + !!(chip->config1 & ADT7516_CONFIG1_SEL_EX_TEMP)); } =20 static ssize_t adt7316_store_select_ex_temp(struct device *dev, @@ -293,9 +295,9 @@ static ssize_t adt7316_store_select_ex_temp(struct devi= ce *dev, if ((chip->id & ID_FAMILY_MASK) !=3D ID_ADT75XX) return -EPERM; =20 - config1 =3D chip->config1 & (~ADT7516_SEL_EX_TEMP); + config1 =3D chip->config1 & (~ADT7516_CONFIG1_SEL_EX_TEMP); if (buf[0] =3D=3D '1') - config1 |=3D ADT7516_SEL_EX_TEMP; + config1 |=3D ADT7516_CONFIG1_SEL_EX_TEMP; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_CONFIG1, config1); if (ret) @@ -318,7 +320,7 @@ static ssize_t adt7316_show_mode(struct device *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - if (chip->config2 & ADT7316_AD_SINGLE_CH_MODE) + if (chip->config2 & ADT7316_CONFIG2_AD_SINGLE_CH_MODE) return sysfs_emit(buf, "single_channel\n"); =20 return sysfs_emit(buf, "round_robin\n"); @@ -334,9 +336,9 @@ static ssize_t adt7316_store_mode(struct device *dev, u8 config2; int ret; =20 - config2 =3D chip->config2 & (~ADT7316_AD_SINGLE_CH_MODE); + config2 =3D chip->config2 & (~ADT7316_CONFIG2_AD_SINGLE_CH_MODE); if (!memcmp(buf, "single_channel", 14)) - config2 |=3D ADT7316_AD_SINGLE_CH_MODE; + config2 |=3D ADT7316_CONFIG2_AD_SINGLE_CH_MODE; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_CONFIG2, config2); if (ret) @@ -368,31 +370,31 @@ static ssize_t adt7316_show_ad_channel(struct device = *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - if (!(chip->config2 & ADT7316_AD_SINGLE_CH_MODE)) + if (!(chip->config2 & ADT7316_CONFIG2_AD_SINGLE_CH_MODE)) return -EPERM; =20 - switch (chip->config2 & ADT7516_AD_SINGLE_CH_MASK) { - case ADT7316_AD_SINGLE_CH_VDD: + switch (chip->config2 & ADT7516_CONFIG2_AD_SINGLE_CH_MASK) { + case ADT7316_CONFIG2_AD_SINGLE_CH_VDD: return sysfs_emit(buf, "0 - VDD\n"); - case ADT7316_AD_SINGLE_CH_IN: + case ADT7316_CONFIG2_AD_SINGLE_CH_IN: return sysfs_emit(buf, "1 - Internal Temperature\n"); - case ADT7316_AD_SINGLE_CH_EX: + case ADT7316_CONFIG2_AD_SINGLE_CH_EX: if (((chip->id & ID_FAMILY_MASK) =3D=3D ID_ADT75XX) && - (chip->config1 & ADT7516_SEL_AIN1_2_EX_TEMP_MASK) =3D=3D 0) + (chip->config1 & ADT7516_CONFIG1_SEL_AIN1_2_EX_TEMP_MASK) =3D=3D 0) return sysfs_emit(buf, "2 - AIN1\n"); =20 return sysfs_emit(buf, "2 - External Temperature\n"); - case ADT7516_AD_SINGLE_CH_AIN2: - if ((chip->config1 & ADT7516_SEL_AIN1_2_EX_TEMP_MASK) =3D=3D 0) + case ADT7516_CONFIG2_AD_SINGLE_CH_AIN2: + if ((chip->config1 & ADT7516_CONFIG1_SEL_AIN1_2_EX_TEMP_MASK) =3D=3D 0) return sysfs_emit(buf, "3 - AIN2\n"); =20 return sysfs_emit(buf, "N/A\n"); - case ADT7516_AD_SINGLE_CH_AIN3: - if (chip->config1 & ADT7516_SEL_AIN3) + case ADT7516_CONFIG2_AD_SINGLE_CH_AIN3: + if (chip->config1 & ADT7516_CONFIG1_SEL_AIN3) return sysfs_emit(buf, "4 - AIN3\n"); =20 return sysfs_emit(buf, "N/A\n"); - case ADT7516_AD_SINGLE_CH_AIN4: + case ADT7516_CONFIG2_AD_SINGLE_CH_AIN4: return sysfs_emit(buf, "5 - AIN4\n"); default: return sysfs_emit(buf, "N/A\n"); @@ -410,7 +412,7 @@ static ssize_t adt7316_store_ad_channel(struct device *= dev, u8 data; int ret; =20 - if (!(chip->config2 & ADT7316_AD_SINGLE_CH_MODE)) + if (!(chip->config2 & ADT7316_CONFIG2_AD_SINGLE_CH_MODE)) return -EPERM; =20 ret =3D kstrtou8(buf, 10, &data); @@ -421,12 +423,12 @@ static ssize_t adt7316_store_ad_channel(struct device= *dev, if (data > 5) return -EINVAL; =20 - config2 =3D chip->config2 & (~ADT7516_AD_SINGLE_CH_MASK); + config2 =3D chip->config2 & (~ADT7516_CONFIG2_AD_SINGLE_CH_MASK); } else { if (data > 2) return -EINVAL; =20 - config2 =3D chip->config2 & (~ADT7316_AD_SINGLE_CH_MASK); + config2 =3D chip->config2 & (~ADT7316_CONFIG2_AD_SINGLE_CH_MASK); } =20 config2 |=3D data; @@ -452,7 +454,7 @@ static ssize_t adt7316_show_all_ad_channels(struct devi= ce *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - if (!(chip->config2 & ADT7316_AD_SINGLE_CH_MODE)) + if (!(chip->config2 & ADT7316_CONFIG2_AD_SINGLE_CH_MODE)) return -EPERM; =20 if ((chip->id & ID_FAMILY_MASK) =3D=3D ID_ADT75XX) @@ -474,7 +476,7 @@ static ssize_t adt7316_show_disable_averaging(struct de= vice *dev, struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 return sysfs_emit(buf, "%d\n", - !!(chip->config2 & ADT7316_DISABLE_AVERAGING)); + !!(chip->config2 & ADT7316_CONFIG2_DISABLE_AVERAGING)); } =20 static ssize_t adt7316_store_disable_averaging(struct device *dev, @@ -487,9 +489,9 @@ static ssize_t adt7316_store_disable_averaging(struct d= evice *dev, u8 config2; int ret; =20 - config2 =3D chip->config2 & (~ADT7316_DISABLE_AVERAGING); + config2 =3D chip->config2 & (~ADT7316_CONFIG2_DISABLE_AVERAGING); if (buf[0] =3D=3D '1') - config2 |=3D ADT7316_DISABLE_AVERAGING; + config2 |=3D ADT7316_CONFIG2_DISABLE_AVERAGING; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_CONFIG2, config2); if (ret) @@ -513,7 +515,7 @@ static ssize_t adt7316_show_enable_smbus_timeout(struct= device *dev, struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 return sysfs_emit(buf, "%d\n", - !!(chip->config2 & ADT7316_EN_SMBUS_TIMEOUT)); + !!(chip->config2 & ADT7316_CONFIG2_EN_SMBUS_TIMEOUT)); } =20 static ssize_t adt7316_store_enable_smbus_timeout(struct device *dev, @@ -526,9 +528,9 @@ static ssize_t adt7316_store_enable_smbus_timeout(struc= t device *dev, u8 config2; int ret; =20 - config2 =3D chip->config2 & (~ADT7316_EN_SMBUS_TIMEOUT); + config2 =3D chip->config2 & (~ADT7316_CONFIG2_EN_SMBUS_TIMEOUT); if (buf[0] =3D=3D '1') - config2 |=3D ADT7316_EN_SMBUS_TIMEOUT; + config2 |=3D ADT7316_CONFIG2_EN_SMBUS_TIMEOUT; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_CONFIG2, config2); if (ret) @@ -551,7 +553,7 @@ static ssize_t adt7316_show_powerdown(struct device *de= v, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - return sysfs_emit(buf, "%d\n", !!(chip->config1 & ADT7316_PD)); + return sysfs_emit(buf, "%d\n", !!(chip->config1 & ADT7316_CONFIG1_PD)); } =20 static ssize_t adt7316_store_powerdown(struct device *dev, @@ -564,9 +566,9 @@ static ssize_t adt7316_store_powerdown(struct device *d= ev, u8 config1; int ret; =20 - config1 =3D chip->config1 & (~ADT7316_PD); + config1 =3D chip->config1 & (~ADT7316_CONFIG1_PD); if (buf[0] =3D=3D '1') - config1 |=3D ADT7316_PD; + config1 |=3D ADT7316_CONFIG1_PD; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_CONFIG1, config1); if (ret) @@ -589,7 +591,8 @@ static ssize_t adt7316_show_fast_ad_clock(struct device= *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - return sysfs_emit(buf, "%d\n", !!(chip->config3 & ADT7316_ADCLK_22_5)); + return sysfs_emit(buf, "%d\n", + !!(chip->config3 & ADT7316_CONFIG3_ADCLK_22_5)); } =20 static ssize_t adt7316_store_fast_ad_clock(struct device *dev, @@ -602,9 +605,9 @@ static ssize_t adt7316_store_fast_ad_clock(struct devic= e *dev, u8 config3; int ret; =20 - config3 =3D chip->config3 & (~ADT7316_ADCLK_22_5); + config3 =3D chip->config3 & (~ADT7316_CONFIG3_ADCLK_22_5); if (buf[0] =3D=3D '1') - config3 |=3D ADT7316_ADCLK_22_5; + config3 |=3D ADT7316_CONFIG3_ADCLK_22_5; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_CONFIG3, config3); if (ret) @@ -627,7 +630,7 @@ static ssize_t adt7316_show_da_high_resolution(struct d= evice *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - if (chip->config3 & ADT7316_DA_HIGH_RESOLUTION) { + if (chip->config3 & ADT7316_CONFIG3_DA_HIGH_RESOLUTION) { if (chip->id !=3D ID_ADT7318 && chip->id !=3D ID_ADT7519) return sysfs_emit(buf, "1 (10 bits)\n"); } @@ -648,9 +651,9 @@ static ssize_t adt7316_store_da_high_resolution(struct = device *dev, if (chip->id =3D=3D ID_ADT7318 || chip->id =3D=3D ID_ADT7519) return -EPERM; =20 - config3 =3D chip->config3 & (~ADT7316_DA_HIGH_RESOLUTION); + config3 =3D chip->config3 & (~ADT7316_CONFIG3_DA_HIGH_RESOLUTION); if (buf[0] =3D=3D '1') - config3 |=3D ADT7316_DA_HIGH_RESOLUTION; + config3 |=3D ADT7316_CONFIG3_DA_HIGH_RESOLUTION; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_CONFIG3, config3); if (ret) @@ -677,7 +680,7 @@ static ssize_t adt7316_show_AIN_internal_Vref(struct de= vice *dev, return -EPERM; =20 return sysfs_emit(buf, "%d\n", - !!(chip->config3 & ADT7516_AIN_IN_VREF)); + !!(chip->config3 & ADT7516_CONFIG3_AIN_IN_VREF)); } =20 static ssize_t adt7316_store_AIN_internal_Vref(struct device *dev, @@ -694,9 +697,9 @@ static ssize_t adt7316_store_AIN_internal_Vref(struct d= evice *dev, return -EPERM; =20 if (buf[0] !=3D '1') - config3 =3D chip->config3 & (~ADT7516_AIN_IN_VREF); + config3 =3D chip->config3 & (~ADT7516_CONFIG3_AIN_IN_VREF); else - config3 =3D chip->config3 | ADT7516_AIN_IN_VREF; + config3 =3D chip->config3 | ADT7516_CONFIG3_AIN_IN_VREF; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_CONFIG3, config3); if (ret) @@ -720,7 +723,7 @@ static ssize_t adt7316_show_enable_prop_DACA(struct dev= ice *dev, struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 return sysfs_emit(buf, "%d\n", - !!(chip->config3 & ADT7316_EN_IN_TEMP_PROP_DACA)); + !!(chip->config3 & ADT7316_CONFIG3_EN_IN_TEMP_PROP_DACA)); } =20 static ssize_t adt7316_store_enable_prop_DACA(struct device *dev, @@ -733,9 +736,9 @@ static ssize_t adt7316_store_enable_prop_DACA(struct de= vice *dev, u8 config3; int ret; =20 - config3 =3D chip->config3 & (~ADT7316_EN_IN_TEMP_PROP_DACA); + config3 =3D chip->config3 & (~ADT7316_CONFIG3_EN_IN_TEMP_PROP_DACA); if (buf[0] =3D=3D '1') - config3 |=3D ADT7316_EN_IN_TEMP_PROP_DACA; + config3 |=3D ADT7316_CONFIG3_EN_IN_TEMP_PROP_DACA; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_CONFIG3, config3); if (ret) @@ -759,7 +762,7 @@ static ssize_t adt7316_show_enable_prop_DACB(struct dev= ice *dev, struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 return sysfs_emit(buf, "%d\n", - !!(chip->config3 & ADT7316_EN_EX_TEMP_PROP_DACB)); + !!(chip->config3 & ADT7316_CONFIG3_EN_EX_TEMP_PROP_DACB)); } =20 static ssize_t adt7316_store_enable_prop_DACB(struct device *dev, @@ -772,9 +775,9 @@ static ssize_t adt7316_store_enable_prop_DACB(struct de= vice *dev, u8 config3; int ret; =20 - config3 =3D chip->config3 & (~ADT7316_EN_EX_TEMP_PROP_DACB); + config3 =3D chip->config3 & (~ADT7316_CONFIG3_EN_EX_TEMP_PROP_DACB); if (buf[0] =3D=3D '1') - config3 |=3D ADT7316_EN_EX_TEMP_PROP_DACB; + config3 |=3D ADT7316_CONFIG3_EN_EX_TEMP_PROP_DACB; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_CONFIG3, config3); if (ret) @@ -798,7 +801,7 @@ static ssize_t adt7316_show_DAC_2Vref_ch_mask(struct de= vice *dev, struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 return sysfs_emit(buf, "0x%x\n", - chip->dac_config & ADT7316_DA_2VREF_CH_MASK); + chip->dac_config & ADT7316_DAC_CONFIG_2VREF_CH_MASK); } =20 static ssize_t adt7316_store_DAC_2Vref_ch_mask(struct device *dev, @@ -813,10 +816,10 @@ static ssize_t adt7316_store_DAC_2Vref_ch_mask(struct= device *dev, int ret; =20 ret =3D kstrtou8(buf, 16, &data); - if (ret || data > ADT7316_DA_2VREF_CH_MASK) + if (ret || data > ADT7316_DAC_CONFIG_2VREF_CH_MASK) return -EINVAL; =20 - dac_config =3D chip->dac_config & (~ADT7316_DA_2VREF_CH_MASK); + dac_config =3D chip->dac_config & (~ADT7316_DAC_CONFIG_2VREF_CH_MASK); dac_config |=3D data; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_DAC_CONFIG, dac_config); @@ -840,20 +843,20 @@ static ssize_t adt7316_show_DAC_update_mode(struct de= vice *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - if (!(chip->config3 & ADT7316_DA_EN_VIA_DAC_LDAC)) + if (!(chip->config3 & ADT7316_CONFIG3_DA_EN_VIA_DAC_LDAC)) return sysfs_emit(buf, "manual\n"); =20 - switch (chip->dac_config & ADT7316_DA_EN_MODE_MASK) { - case ADT7316_DA_EN_MODE_SINGLE: + switch (chip->dac_config & ADT7316_DAC_CONFIG_EN_MODE_MASK) { + case ADT7316_DAC_CONFIG_EN_MODE_SINGLE: return sysfs_emit(buf, "0 - auto at any MSB DAC writing\n"); - case ADT7316_DA_EN_MODE_AB_CD: + case ADT7316_DAC_CONFIG_EN_MODE_AB_CD: return sysfs_emit(buf, "1 - auto at MSB DAC AB and CD writing\n"); - case ADT7316_DA_EN_MODE_ABCD: + case ADT7316_DAC_CONFIG_EN_MODE_ABCD: return sysfs_emit(buf, "2 - auto at MSB DAC ABCD writing\n"); - default: /* ADT7316_DA_EN_MODE_LDAC */ + default: /* ADT7316_DAC_CONFIG_EN_MODE_LDAC */ return sysfs_emit(buf, "3 - manual\n"); } } @@ -869,15 +872,15 @@ static ssize_t adt7316_store_DAC_update_mode(struct d= evice *dev, u8 data; int ret; =20 - if (!(chip->config3 & ADT7316_DA_EN_VIA_DAC_LDAC)) + if (!(chip->config3 & ADT7316_CONFIG3_DA_EN_VIA_DAC_LDAC)) return -EPERM; =20 ret =3D kstrtou8(buf, 10, &data); - if (ret || !FIELD_FIT(ADT7316_DA_EN_MODE_MASK, data)) + if (ret || !FIELD_FIT(ADT7316_DAC_CONFIG_EN_MODE_MASK, data)) return -EINVAL; =20 - dac_config =3D chip->dac_config & (~ADT7316_DA_EN_MODE_MASK); - dac_config |=3D FIELD_PREP(ADT7316_DA_EN_MODE_MASK, data); + dac_config =3D chip->dac_config & (~ADT7316_DAC_CONFIG_EN_MODE_MASK); + dac_config |=3D FIELD_PREP(ADT7316_DAC_CONFIG_EN_MODE_MASK, data); =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_DAC_CONFIG, dac_config); if (ret) @@ -900,7 +903,7 @@ static ssize_t adt7316_show_all_DAC_update_modes(struct= device *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - if (chip->config3 & ADT7316_DA_EN_VIA_DAC_LDAC) + if (chip->config3 & ADT7316_CONFIG3_DA_EN_VIA_DAC_LDAC) return sysfs_emit(buf, "0 - auto at any MSB DAC writing\n" "1 - auto at MSB DAC AB and CD writing\n" "2 - auto at MSB DAC ABCD writing\n" @@ -922,16 +925,17 @@ static ssize_t adt7316_store_update_DAC(struct device= *dev, u8 data; int ret; =20 - if (chip->config3 & ADT7316_DA_EN_VIA_DAC_LDAC) { - if ((chip->dac_config & ADT7316_DA_EN_MODE_MASK) !=3D - ADT7316_DA_EN_MODE_LDAC) + if (chip->config3 & ADT7316_CONFIG3_DA_EN_VIA_DAC_LDAC) { + if ((chip->dac_config & ADT7316_DAC_CONFIG_EN_MODE_MASK) !=3D + ADT7316_DAC_CONFIG_EN_MODE_LDAC) return -EPERM; =20 ret =3D kstrtou8(buf, 16, &data); - if (ret || data > ADT7316_LDAC_EN_DA_MASK) + if (ret || data > ADT7316_LDAC_CONFIG_EN_DA_MASK) return -EINVAL; =20 - ldac_config =3D chip->ldac_config & (~ADT7316_LDAC_EN_DA_MASK); + ldac_config =3D chip->ldac_config & + (~ADT7316_LDAC_CONFIG_EN_DA_MASK); ldac_config |=3D data; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_LDAC_CONFIG, @@ -959,7 +963,7 @@ static ssize_t adt7316_show_DA_AB_Vref_bypass(struct de= vice *dev, struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 return sysfs_emit(buf, "%d\n", - !!(chip->dac_config & ADT7316_VREF_BYPASS_DAC_AB)); + !!(chip->dac_config & ADT7316_DAC_CONFIG_VREF_BYPASS_AB)); } =20 static ssize_t adt7316_store_DA_AB_Vref_bypass(struct device *dev, @@ -972,9 +976,9 @@ static ssize_t adt7316_store_DA_AB_Vref_bypass(struct d= evice *dev, u8 dac_config; int ret; =20 - dac_config =3D chip->dac_config & (~ADT7316_VREF_BYPASS_DAC_AB); + dac_config =3D chip->dac_config & (~ADT7316_DAC_CONFIG_VREF_BYPASS_AB); if (buf[0] =3D=3D '1') - dac_config |=3D ADT7316_VREF_BYPASS_DAC_AB; + dac_config |=3D ADT7316_DAC_CONFIG_VREF_BYPASS_AB; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_DAC_CONFIG, dac_config); if (ret) @@ -998,7 +1002,7 @@ static ssize_t adt7316_show_DA_CD_Vref_bypass(struct d= evice *dev, struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 return sysfs_emit(buf, "%d\n", - !!(chip->dac_config & ADT7316_VREF_BYPASS_DAC_CD)); + !!(chip->dac_config & ADT7316_DAC_CONFIG_VREF_BYPASS_CD)); } =20 static ssize_t adt7316_store_DA_CD_Vref_bypass(struct device *dev, @@ -1011,9 +1015,9 @@ static ssize_t adt7316_store_DA_CD_Vref_bypass(struct= device *dev, u8 dac_config; int ret; =20 - dac_config =3D chip->dac_config & (~ADT7316_VREF_BYPASS_DAC_CD); + dac_config =3D chip->dac_config & (~ADT7316_DAC_CONFIG_VREF_BYPASS_CD); if (buf[0] =3D=3D '1') - dac_config |=3D ADT7316_VREF_BYPASS_DAC_CD; + dac_config |=3D ADT7316_DAC_CONFIG_VREF_BYPASS_CD; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_DAC_CONFIG, dac_config); if (ret) @@ -1038,9 +1042,10 @@ static ssize_t adt7316_show_DAC_internal_Vref(struct= device *dev, =20 if ((chip->id & ID_FAMILY_MASK) =3D=3D ID_ADT75XX) return sysfs_emit(buf, "0x%x\n", - FIELD_GET(ADT7516_DAC_IN_VREF_MASK, chip->ldac_config)); + FIELD_GET(ADT7516_LDAC_CONFIG_DAC_IN_VREF_MASK, + chip->ldac_config)); return sysfs_emit(buf, "%d\n", - !!(chip->ldac_config & ADT7316_DAC_IN_VREF)); + !!(chip->ldac_config & ADT7316_LDAC_CONFIG_DAC_IN_VREF)); } =20 static ssize_t adt7316_store_DAC_internal_Vref(struct device *dev, @@ -1059,19 +1064,22 @@ static ssize_t adt7316_store_DAC_internal_Vref(stru= ct device *dev, if (ret || data > 3) return -EINVAL; =20 - ldac_config =3D chip->ldac_config & (~ADT7516_DAC_IN_VREF_MASK); + ldac_config =3D chip->ldac_config & + (~ADT7516_LDAC_CONFIG_DAC_IN_VREF_MASK); if (data & 0x1) - ldac_config |=3D ADT7516_DAC_AB_IN_VREF; + ldac_config |=3D ADT7516_LDAC_CONFIG_DAC_AB_IN_VREF; if (data & 0x2) - ldac_config |=3D ADT7516_DAC_CD_IN_VREF; + ldac_config |=3D ADT7516_LDAC_CONFIG_DAC_CD_IN_VREF; } else { ret =3D kstrtou8(buf, 16, &data); if (ret) return -EINVAL; =20 - ldac_config =3D chip->ldac_config & (~ADT7316_DAC_IN_VREF); + ldac_config =3D chip->ldac_config & + (~ADT7316_LDAC_CONFIG_DAC_IN_VREF); if (data) - ldac_config =3D chip->ldac_config | ADT7316_DAC_IN_VREF; + ldac_config =3D chip->ldac_config | + ADT7316_LDAC_CONFIG_DAC_IN_VREF; } =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_LDAC_CONFIG, @@ -1092,13 +1100,13 @@ static IIO_DEVICE_ATTR(DAC_internal_Vref, 0644, static u8 adt7316_extract_ad_lsb(u8 lsb, int channel) { switch (channel) { - case ADT7316_AD_SINGLE_CH_EX: + case ADT7316_CONFIG2_AD_SINGLE_CH_EX: return FIELD_GET(ADT7316_LSB_EX_TEMP_AIN1_MASK, lsb); - case ADT7516_AD_SINGLE_CH_AIN2: + case ADT7516_CONFIG2_AD_SINGLE_CH_AIN2: return FIELD_GET(ADT7516_LSB_AIN2_MASK, lsb); - case ADT7516_AD_SINGLE_CH_AIN3: + case ADT7516_CONFIG2_AD_SINGLE_CH_AIN3: return FIELD_GET(ADT7516_LSB_AIN3_MASK, lsb); - case ADT7516_AD_SINGLE_CH_AIN4: + case ADT7516_CONFIG2_AD_SINGLE_CH_AIN4: return FIELD_GET(ADT7516_LSB_AIN4_MASK, lsb); default: return 0; @@ -1112,12 +1120,12 @@ static ssize_t adt7316_show_ad(struct adt7316_chip_= info *chip, u8 msb, lsb; int ret; =20 - if ((chip->config2 & ADT7316_AD_SINGLE_CH_MODE) && - channel !=3D (chip->config2 & ADT7516_AD_SINGLE_CH_MASK)) + if ((chip->config2 & ADT7316_CONFIG2_AD_SINGLE_CH_MODE) && + channel !=3D (chip->config2 & ADT7516_CONFIG2_AD_SINGLE_CH_MASK)) return -EPERM; =20 switch (channel) { - case ADT7316_AD_SINGLE_CH_IN: + case ADT7316_CONFIG2_AD_SINGLE_CH_IN: ret =3D chip->bus.read(chip->bus.client, ADT7316_LSB_IN_TEMP_VDD, &lsb); if (ret) @@ -1131,7 +1139,7 @@ static ssize_t adt7316_show_ad(struct adt7316_chip_in= fo *chip, data =3D FIELD_PREP(ADT7316_AD_MSB_MASK, msb); data |=3D lsb & ADT7316_LSB_IN_TEMP_MASK; break; - case ADT7316_AD_SINGLE_CH_VDD: + case ADT7316_CONFIG2_AD_SINGLE_CH_VDD: ret =3D chip->bus.read(chip->bus.client, ADT7316_LSB_IN_TEMP_VDD, &lsb); if (ret) @@ -1182,7 +1190,7 @@ static ssize_t adt7316_show_VDD(struct device *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - return adt7316_show_ad(chip, ADT7316_AD_SINGLE_CH_VDD, buf); + return adt7316_show_ad(chip, ADT7316_CONFIG2_AD_SINGLE_CH_VDD, buf); } static IIO_DEVICE_ATTR(VDD, 0444, adt7316_show_VDD, NULL, 0); =20 @@ -1193,7 +1201,7 @@ static ssize_t adt7316_show_in_temp(struct device *de= v, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - return adt7316_show_ad(chip, ADT7316_AD_SINGLE_CH_IN, buf); + return adt7316_show_ad(chip, ADT7316_CONFIG2_AD_SINGLE_CH_IN, buf); } =20 static IIO_DEVICE_ATTR(in_temp, 0444, adt7316_show_in_temp, NULL, 0); @@ -1205,7 +1213,7 @@ static ssize_t adt7316_show_ex_temp_AIN1(struct devic= e *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - return adt7316_show_ad(chip, ADT7316_AD_SINGLE_CH_EX, buf); + return adt7316_show_ad(chip, ADT7316_CONFIG2_AD_SINGLE_CH_EX, buf); } =20 static IIO_DEVICE_ATTR(ex_temp_AIN1, 0444, adt7316_show_ex_temp_AIN1, @@ -1219,7 +1227,7 @@ static ssize_t adt7316_show_AIN2(struct device *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - return adt7316_show_ad(chip, ADT7516_AD_SINGLE_CH_AIN2, buf); + return adt7316_show_ad(chip, ADT7516_CONFIG2_AD_SINGLE_CH_AIN2, buf); } static IIO_DEVICE_ATTR(AIN2, 0444, adt7316_show_AIN2, NULL, 0); =20 @@ -1230,7 +1238,7 @@ static ssize_t adt7316_show_AIN3(struct device *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - return adt7316_show_ad(chip, ADT7516_AD_SINGLE_CH_AIN3, buf); + return adt7316_show_ad(chip, ADT7516_CONFIG2_AD_SINGLE_CH_AIN3, buf); } static IIO_DEVICE_ATTR(AIN3, 0444, adt7316_show_AIN3, NULL, 0); =20 @@ -1241,7 +1249,7 @@ static ssize_t adt7316_show_AIN4(struct device *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - return adt7316_show_ad(chip, ADT7516_AD_SINGLE_CH_AIN4, buf); + return adt7316_show_ad(chip, ADT7516_CONFIG2_AD_SINGLE_CH_AIN4, buf); } static IIO_DEVICE_ATTR(AIN4, 0444, adt7316_show_AIN4, NULL, 0); =20 @@ -1403,9 +1411,9 @@ static ssize_t adt7316_show_DAC(struct adt7316_chip_i= nfo *chip, =20 if (channel >=3D ADT7316_DA_MSB_DATA_REGS || (channel =3D=3D 0 && - (chip->config3 & ADT7316_EN_IN_TEMP_PROP_DACA)) || + (chip->config3 & ADT7316_CONFIG3_EN_IN_TEMP_PROP_DACA)) || (channel =3D=3D 1 && - (chip->config3 & ADT7316_EN_EX_TEMP_PROP_DACB))) + (chip->config3 & ADT7316_CONFIG3_EN_EX_TEMP_PROP_DACB))) return -EPERM; =20 offset =3D chip->dac_bits - 8; @@ -1440,9 +1448,9 @@ static ssize_t adt7316_store_DAC(struct adt7316_chip_= info *chip, =20 if (channel >=3D ADT7316_DA_MSB_DATA_REGS || (channel =3D=3D 0 && - (chip->config3 & ADT7316_EN_IN_TEMP_PROP_DACA)) || + (chip->config3 & ADT7316_CONFIG3_EN_IN_TEMP_PROP_DACA)) || (channel =3D=3D 1 && - (chip->config3 & ADT7316_EN_EX_TEMP_PROP_DACB))) + (chip->config3 & ADT7316_CONFIG3_EN_EX_TEMP_PROP_DACB))) return -EPERM; =20 offset =3D chip->dac_bits - 8; @@ -1834,7 +1842,7 @@ static int adt7316_setup_irq(struct iio_dev *indio_de= v) } =20 if (irq_type & IRQF_TRIGGER_HIGH) - chip->config1 |=3D ADT7316_INT_POLARITY; + chip->config1 |=3D ADT7316_CONFIG1_INT_POLARITY; =20 return 0; } @@ -1918,7 +1926,7 @@ static inline ssize_t adt7316_show_ad_bound(struct de= vice *dev, data =3D (int)val; =20 if (!((chip->id & ID_FAMILY_MASK) =3D=3D ID_ADT75XX && - (chip->config1 & ADT7516_SEL_AIN1_2_EX_TEMP_MASK) =3D=3D 0)) { + (chip->config1 & ADT7516_CONFIG1_SEL_AIN1_2_EX_TEMP_MASK) =3D=3D 0)= ) { if (data & 0x80) data -=3D 256; } @@ -1947,7 +1955,7 @@ static inline ssize_t adt7316_set_ad_bound(struct dev= ice *dev, return -EINVAL; =20 if ((chip->id & ID_FAMILY_MASK) =3D=3D ID_ADT75XX && - (chip->config1 & ADT7516_SEL_AIN1_2_EX_TEMP_MASK) =3D=3D 0) { + (chip->config1 & ADT7516_CONFIG1_SEL_AIN1_2_EX_TEMP_MASK) =3D=3D 0) { if (data > 255 || data < 0) return -EINVAL; } else { @@ -1974,7 +1982,8 @@ static ssize_t adt7316_show_int_enabled(struct device= *dev, struct iio_dev *dev_info =3D dev_to_iio_dev(dev); struct adt7316_chip_info *chip =3D iio_priv(dev_info); =20 - return sysfs_emit(buf, "%d\n", !!(chip->config1 & ADT7316_INT_EN)); + return sysfs_emit(buf, "%d\n", + !!(chip->config1 & ADT7316_CONFIG1_INT_EN)); } =20 static ssize_t adt7316_set_int_enabled(struct device *dev, @@ -1987,9 +1996,9 @@ static ssize_t adt7316_set_int_enabled(struct device = *dev, u8 config1; int ret; =20 - config1 =3D chip->config1 & (~ADT7316_INT_EN); + config1 =3D chip->config1 & (~ADT7316_CONFIG1_INT_EN); if (buf[0] =3D=3D '1') - config1 |=3D ADT7316_INT_EN; + config1 |=3D ADT7316_CONFIG1_INT_EN; =20 ret =3D chip->bus.write(chip->bus.client, ADT7316_CONFIG1, config1); if (ret) @@ -2167,9 +2176,9 @@ int adt7316_probe(struct device *dev, struct adt7316_= bus *bus, } =20 if (!chip->ldac_pin) { - chip->config3 |=3D ADT7316_DA_EN_VIA_DAC_LDAC; + chip->config3 |=3D ADT7316_CONFIG3_DA_EN_VIA_DAC_LDAC; if ((chip->id & ID_FAMILY_MASK) =3D=3D ID_ADT75XX) - chip->config1 |=3D ADT7516_SEL_AIN3; + chip->config1 |=3D ADT7516_CONFIG1_SEL_AIN3; } chip->int_mask =3D ADT7316_TEMP_INT_MASK | ADT7316_VDD_INT_MASK; if ((chip->id & ID_FAMILY_MASK) =3D=3D ID_ADT75XX) --=20 2.53.0