[PATCH] iio: adc: ade9000: remove unused AD9000_CHANNELS_PER_PHASE macro

Giorgi Tchankvetadze posted 1 patch 1 month, 1 week ago
drivers/iio/adc/ade9000.c | 1 -
1 file changed, 1 deletion(-)
[PATCH] iio: adc: ade9000: remove unused AD9000_CHANNELS_PER_PHASE macro
Posted by Giorgi Tchankvetadze 1 month, 1 week ago
Remove the AD9000_CHANNELS_PER_PHASE macro which is unused, has a
misspelled prefix (AD9000 instead of ADE9000), and has an incorrect
value (10 instead of 11, there are 33 total channels and 3 phases, so 11
for each).

Signed-off-by: Giorgi Tchankvetadze <giorgitchankvetadze1997@gmail.com>
---
 drivers/iio/adc/ade9000.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/iio/adc/ade9000.c b/drivers/iio/adc/ade9000.c
index 5dcc26a08970..4be8df34428d 100644
--- a/drivers/iio/adc/ade9000.c
+++ b/drivers/iio/adc/ade9000.c
@@ -283,7 +283,6 @@ enum ade9000_wfb_cfg {
 #define ADE9000_PHASE_C_POS_BIT		BIT(6)

 #define ADE9000_MAX_PHASE_NR		3
-#define AD9000_CHANNELS_PER_PHASE	10

 /*
  * Calculate register address for multi-phase device.
--
2.52.0
Re: [PATCH] iio: adc: ade9000: remove unused AD9000_CHANNELS_PER_PHASE macro
Posted by Andy Shevchenko 1 month, 1 week ago
On Fri, Feb 27, 2026 at 12:58:13PM +0400, Giorgi Tchankvetadze wrote:
> Remove the AD9000_CHANNELS_PER_PHASE macro which is unused, has a
> misspelled prefix (AD9000 instead of ADE9000), and has an incorrect
> value (10 instead of 11, there are 33 total channels and 3 phases, so 11
> for each).

for --> per

-- 
With Best Regards,
Andy Shevchenko
Re: [PATCH] iio: adc: ade9000: remove unused AD9000_CHANNELS_PER_PHASE macro
Posted by Jonathan Cameron 1 month, 1 week ago
On Fri, 27 Feb 2026 11:07:42 +0200
Andy Shevchenko <andriy.shevchenko@intel.com> wrote:

> On Fri, Feb 27, 2026 at 12:58:13PM +0400, Giorgi Tchankvetadze wrote:
> > Remove the AD9000_CHANNELS_PER_PHASE macro which is unused, has a
> > misspelled prefix (AD9000 instead of ADE9000), and has an incorrect
> > value (10 instead of 11, there are 33 total channels and 3 phases, so 11
> > for each).  
> 
> for --> per

Hmm. per each is weird.  per phase makes more sense so I'll change it to that.

Applied with that commit message tweak.
Thanks

Jonathan

>