* Re: [PATCH v4 1/2] iio: filter: admv8818: fix range calculation
[not found] <20250225134612.577022-1-sam.winchenbach@framepointer.org>
@ 2025-02-27 20:23 ` kernel test robot
2025-02-27 21:20 ` Sam Winchenbach
0 siblings, 1 reply; 2+ messages in thread
From: kernel test robot @ 2025-02-27 20:23 UTC (permalink / raw)
To: Sam Winchenbach, linux-kernel
Cc: llvm, oe-kbuild-all, lars, Michael.Hennerich, antoniu.miclaus,
jic23, robh, krzk+dt, conor+dt, linux-iio, devicetree,
sam.winchenbach
Hi Sam,
kernel test robot noticed the following build errors:
[auto build test ERROR on jic23-iio/togreg]
[also build test ERROR on linus/master v6.14-rc4 next-20250227]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Sam-Winchenbach/dt-bindings-iio-filter-Add-lpf-hpf-freq-margins/20250225-215003
base: https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg
patch link: https://lore.kernel.org/r/20250225134612.577022-1-sam.winchenbach%40framepointer.org
patch subject: [PATCH v4 1/2] iio: filter: admv8818: fix range calculation
config: um-allmodconfig (https://download.01.org/0day-ci/archive/20250228/202502280434.DHtcsf7x-lkp@intel.com/config)
compiler: clang version 21.0.0git (https://github.com/llvm/llvm-project 204dcafec0ecf0db81d420d2de57b02ada6b09ec)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250228/202502280434.DHtcsf7x-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202502280434.DHtcsf7x-lkp@intel.com/
All errors (new ones prefixed by >>):
In file included from drivers/iio/filter/admv8818.c:17:
In file included from include/linux/regmap.h:20:
In file included from include/linux/iopoll.h:14:
In file included from include/linux/io.h:14:
In file included from arch/um/include/asm/io.h:24:
include/asm-generic/io.h:549:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
549 | val = __raw_readb(PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:567:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
567 | val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
| ~~~~~~~~~~ ^
include/uapi/linux/byteorder/little_endian.h:37:51: note: expanded from macro '__le16_to_cpu'
37 | #define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
| ^
In file included from drivers/iio/filter/admv8818.c:17:
In file included from include/linux/regmap.h:20:
In file included from include/linux/iopoll.h:14:
In file included from include/linux/io.h:14:
In file included from arch/um/include/asm/io.h:24:
include/asm-generic/io.h:585:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
585 | val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
| ~~~~~~~~~~ ^
include/uapi/linux/byteorder/little_endian.h:35:51: note: expanded from macro '__le32_to_cpu'
35 | #define __le32_to_cpu(x) ((__force __u32)(__le32)(x))
| ^
In file included from drivers/iio/filter/admv8818.c:17:
In file included from include/linux/regmap.h:20:
In file included from include/linux/iopoll.h:14:
In file included from include/linux/io.h:14:
In file included from arch/um/include/asm/io.h:24:
include/asm-generic/io.h:601:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
601 | __raw_writeb(value, PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:616:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
616 | __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:631:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
631 | __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:724:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
724 | readsb(PCI_IOBASE + addr, buffer, count);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:737:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
737 | readsw(PCI_IOBASE + addr, buffer, count);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:750:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
750 | readsl(PCI_IOBASE + addr, buffer, count);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:764:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
764 | writesb(PCI_IOBASE + addr, buffer, count);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:778:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
778 | writesw(PCI_IOBASE + addr, buffer, count);
| ~~~~~~~~~~ ^
include/asm-generic/io.h:792:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
792 | writesl(PCI_IOBASE + addr, buffer, count);
| ~~~~~~~~~~ ^
>> drivers/iio/filter/admv8818.c:304:34: error: use of undeclared identifier 'lfp_corner_target'; did you mean 'lpf_corner_target'?
304 | ret = __admv8818_lpf_select(st, lfp_corner_target);
| ^~~~~~~~~~~~~~~~~
| lpf_corner_target
drivers/iio/filter/admv8818.c:283:25: note: 'lpf_corner_target' declared here
283 | u64 hpf_corner_target, lpf_corner_target;
| ^
12 warnings and 1 error generated.
vim +304 drivers/iio/filter/admv8818.c
279
280 static int admv8818_rfin_band_select(struct admv8818_state *st)
281 {
282 int ret;
283 u64 hpf_corner_target, lpf_corner_target;
284
285 st->cf_hz = clk_get_rate(st->clkin);
286
287 // Check for underflow
288 if (st->cf_hz > st->hpf_margin_hz)
289 hpf_corner_target = st->cf_hz - st->hpf_margin_hz;
290 else
291 hpf_corner_target = 0;
292
293 // Check for overflow
294 lpf_corner_target = st->cf_hz + st->lpf_margin_hz;
295 if (lpf_corner_target < st->cf_hz)
296 lpf_corner_target = U64_MAX;
297
298 mutex_lock(&st->lock);
299
300 ret = __admv8818_hpf_select(st, hpf_corner_target);
301 if (ret)
302 goto exit;
303
> 304 ret = __admv8818_lpf_select(st, lfp_corner_target);
305 exit:
306 mutex_unlock(&st->lock);
307 return ret;
308 }
309
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH v4 1/2] iio: filter: admv8818: fix range calculation
2025-02-27 20:23 ` [PATCH v4 1/2] iio: filter: admv8818: fix range calculation kernel test robot
@ 2025-02-27 21:20 ` Sam Winchenbach
0 siblings, 0 replies; 2+ messages in thread
From: Sam Winchenbach @ 2025-02-27 21:20 UTC (permalink / raw)
To: kernel test robot
Cc: linux-kernel, llvm, oe-kbuild-all, lars, Michael.Hennerich,
antoniu.miclaus, jic23, robh, krzk+dt, conor+dt, linux-iio,
devicetree
On Fri, Feb 28, 2025 at 04:23:44AM +0800, kernel test robot wrote:
> Hi Sam,
>
> kernel test robot noticed the following build errors:
>
--- snip ---
> >> drivers/iio/filter/admv8818.c:304:34: error: use of undeclared identifier 'lfp_corner_target'; did you mean 'lpf_corner_target'?
> 304 | ret = __admv8818_lpf_select(st, lfp_corner_target);
> | ^~~~~~~~~~~~~~~~~
> | lpf_corner_target
> drivers/iio/filter/admv8818.c:283:25: note: 'lpf_corner_target' declared here
> 283 | u64 hpf_corner_target, lpf_corner_target;
> | ^
> 12 warnings and 1 error generated.
Well this is embarassing. I must have have had a fat-finger accident when
inspecting the result of applying the patch to torvalds/master.
I will fix this in v5 once we come to a concensus on the DTS changes.
--- snip ---
> 303
> > 304 ret = __admv8818_lpf_select(st, lfp_corner_target);
> 305 exit:
> 306 mutex_unlock(&st->lock);
> 307 return ret;
> 308 }
> 309
>
> --
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-02-27 21:20 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20250225134612.577022-1-sam.winchenbach@framepointer.org>
2025-02-27 20:23 ` [PATCH v4 1/2] iio: filter: admv8818: fix range calculation kernel test robot
2025-02-27 21:20 ` Sam Winchenbach
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox