iio: adc: ad7124: Remove ability to write filter_low_pass_3db_frequency #17
top-level.yml
on: push
checks
/
checks
1m 28s
build_gcc_arm
/
build
4m 7s
build_gcc_x86_64
/
build
3m 18s
build_llvm_x86_64
/
build
6m 53s
build_gcc_aarch64
/
build
4m 28s
build_gcc_aarch64_adi_bcm2711_defconfig
/
build
build_gcc_aarch64_adi_bcm2712_defconfig
/
build
build_gcc_arm_adi_bcm2709_defconfig
/
build
build_gcc_arm_adi_bcm2711_defconfig
/
build
build_gcc_arm_adi_bcmrpi_defconfig
/
build
build_gcc_aarch64_adi_zynqmp_defconfig
/
build
build_gcc_arm_adi_versal_defconfig
/
build
build_gcc_arm_socfpga_adi_defconfig
/
build
build_gcc_arm_zynq_m2k_defconfig
/
build
build_gcc_arm_zynq_pluto_defconfig
/
build
build_gcc_arm_zynq_xcomm_adv7511_defconfig
/
build
many_devicetrees
/
build
assert_build_rpi
assert_build_xlnx
Annotations
3 errors and 15 warnings
|
build_gcc_arm / build
Process completed with exit code 1.
|
|
assert_checks
Process completed with exit code 1.
|
|
assert_checks
Some jobs didn't pass strict checks:
build_gcc_arm:
gcc_fanalyzer
And non-strict checks:
checks:
coccicheck
checkpatch
build_llvm_x86_64:
clang_analyzer
|
|
checks / checks:
drivers/iio/adc/ad7944.c#L116
coccicheck: 28-44 WARNING use flexible-array member instead (https //www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
|
|
checks / checks:
drivers/iio/adc/ad7124.c#L0
checkpatch: Possible repeated word: 'it'
user. Also it it counter-intuitive that setting the 3db frequency
|
|
build_gcc_arm / build:
drivers/iio/adc/ad7124.c#L1100
kernel_smatch: potentially one past the end of array 'chan[num_channels]'
|
|
build_gcc_arm / build:
drivers/iio/adc/ad7124.c#L1100
kernel_smatch: potentially one past the end of array 'chan[num_channels]'
|
|
build_gcc_arm / build:
drivers/iio/dac/ad3552r.c#L328
gcc_fanalayzer: use of uninitialized value 'val' [CWE-457] [-Wanalyzer-use-of-uninitialized-value]
328 | if (val1 != val)
| ~~~~~^~~~~~
'ad3552r_probe': events 1-3
|
| 617 | static int ad3552r_probe(struct spi_device *spi)
| | ^~~~~~~~~~~~~
| | |
| | (1) entry to 'ad3552r_probe'
|......
| 624 | if (!indio_dev)
| | ~
| | |
| | (2) following 'false' branch (when 'indio_dev' is non-NULL)...
|......
| 627 | dac = iio_priv(indio_dev);
| | ~
| | |
| | (3) inlined call to 'iio_priv' from 'ad3552r_probe'
|
+--> 'iio_priv': event 4
|
|./include/linux/iio/iio.h:814:25:
| 814 | return indio_dev->priv;
| | ~~~~~~~~~^~~~~~
| | |
| | (4) ...to here
|
<------+
|
'ad3552r_probe': event 5
|
|drivers/iio/dac/ad3552r.c:630:12:
| 630 | if (!dac->model_data)
| | ^
| | |
| | (5) following 'false' branch...
|
'ad3552r_probe': event 6
|
|./include/linux/mutex.h:66:9:
| 66 | __mutex_init((mutex), #mutex, &__key); \
| | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| | |
| | (6) ...to here
|
|
build_gcc_arm / build:
drivers/iio/dac/ad3552r.c#L118
gcc_fanalayzer: use of uninitialized value 'reg' [CWE-457] [-Wanalyzer-use-of-uninitialized-value]
118 | reg &= ~mask;
| ^~
'ad3552r_probe': events 1-3
|
| 617 | static int ad3552r_probe(struct spi_device *spi)
| | ^~~~~~~~~~~~~
| | |
| | (1) entry to 'ad3552r_probe'
|......
| 624 | if (!indio_dev)
| | ~
| | |
| | (2) following 'false' branch (when 'indio_dev' is non-NULL)...
|......
| 627 | dac = iio_priv(indio_dev);
| | ~
| | |
| | (3) inlined call to 'iio_priv' from 'ad3552r_probe'
|
+--> 'iio_priv': event 4
|
|./include/linux/iio/iio.h:814:25:
| 814 | return indio_dev->priv;
| | ~~~~~~~~~^~~~~~
| | |
| | (4) ...to here
|
<------+
|
'ad3552r_probe': event 5
|
|drivers/iio/dac/ad3552r.c:630:12:
| 630 | if (!dac->model_data)
| | ^
| | |
| | (5) following 'false' branch...
|
'ad3552r_probe': event 6
|
|./include/linux/mutex.h:66:9:
| 66 | __mutex_init((mutex), #mutex, &__key); \
| | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| | |
| | (6) ...to here
|
|
build_gcc_arm / build:
drivers/iio/dac/ad3552r.c#L118
gcc_fanalayzer: use of uninitialized value 'reg' [CWE-457] [-Wanalyzer-use-of-uninitialized-value]
118 | reg &= ~mask;
| ^~
'ad3552r_init': events 1-2
|
| 578 | static int ad3552r_init(struct ad3552r_desc *dac)
| | ^~~~~~~~~~~~
| | |
| | (1) entry to 'ad3552r_init'
|......
| 583 | err = ad3552r_reset(dac);
| | ~~~~~~~~~~~~~~~~~~
| | |
| | (2) calling 'ad3552r_reset' from 'ad3552r_init'
|
+--> 'ad3552r_reset': events 3-8
|
| 362 | static int ad3552r_reset(struct ad3552r_desc *dac)
| | ^~~~~~~~~~~~~
| | |
| | (3) entry to 'ad3552r_reset'
|......
| 370 | if (IS_ERR(dac->gpio_reset))
| | ~
| | |
| | (4) following 'false' branch...
|......
| 374 | if (dac->gpio_reset) {
| | ~
| | |
| | (5) ...to here
| | (6) following 'false' branch...
|......
| 380 | ret = ad3552r_update_reg_field(dac,
| | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| | |
| | (7) ...to here
| | (8) calling 'ad3552r_update_reg_field' from 'ad3552r_reset'
| 381 | AD3552R_REG_ADDR_INTERFACE_CONFIG_A,
| | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| 382 | AD3552R_MASK_SOFTWARE_RESET,
| | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| 383 | AD3552R_MASK_SOFTWARE_RESET);
| | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
|
+--> 'ad3552r_update_reg_field': events 9-12
|
| 108 | static int ad3552r_update_reg_field(struct ad3552r_desc *dac, u8 addr, u16 mask,
| | ^~~~~~~~~~~~~~~~~~~~~~~~
| | |
| | (9) entry to 'ad3552r_update_reg_field'
|......
| 112 | u16 reg;
| | ~~~
| | |
| | (10) region created on stack here
| | (11) capacity: 2 bytes
| 113 |
| 114 | ret = ad3552r_read_reg(dac, addr, ®);
| | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| | |
| | (12) calling 'ad3552r_read_reg' from 'ad3552r_update_reg_field'
|
+--> 'ad3552r_read_reg': events 13-14
|
| 88 | static int ad3552r_read_reg(struct ad3552r_desc *dac, u8 addr, u16 *val)
| | ^~~~~~~~~~~~~~~~
| | |
| | (13) entry to 'ad3552r_read_reg'
|......
| 93 | reg_len = _ad3552r_reg_len(addr);
| | ~~~~~~~~~~~~~~~~~~~~~~
| | |
| | (14) calling '_ad3552r_reg_len' from 'ad3552r_read_reg'
|
+--> '_ad3552r_reg_len': events 15-16
|
| 32 | static u8 _ad3552r_reg_len(u8 addr)
| | ^~~~~~~~~~~~~~~~
| | |
| | (15) entry to '_ad3552r_reg_len'
| 33 | {
| 34 | switch (addr) {
| | ~~~~~~
| | |
| | (16) following 'default:' branch...
|
'_ad3552r_reg_len': event 17
|
|cc1:
| (17): ...to here
|
'_ad3552r_reg_len': event 18
|
|cc1:
| (18): calling '_ad3552r_reg_len.part.0' from '_ad3552r_reg_len'
|
+--> '_ad3552r_reg_len.part.0': events 19-22
|
| 32 | static u8 _ad3552r_reg_len(u8 addr)
| | ^~~~~~~~~~~~~~~~
| | |
| | (19) entry to '_ad3552r_reg_len.part.0'
|......
| 46 | if (addr > AD3552R_REG_ADDR_HW_LDAC_24B)
| | ~
| | |
| | (20) following 'false' branch (when 'addr <= 55')...
| 47 | return 3;
| 48 | if (addr > AD3552R_REG_ADDR_HW_LDAC_16B)
| | ~
| | |
| | (21) ...to here
| | (22) following 'false' branch (when 'addr <= 40')...
|
'_ad3552r_reg_len.part.0': event 23
|
|cc1:
| (23): ...to here
|
<------+
|
'_ad3552r_reg_len': event 24
|
|cc1:
| (24): returning to '_ad3552r_reg_len' from '_ad3552r_reg_len.part.0'
|
<------+
|
'ad3552r_read_reg': events 25-26
|
| 93 | reg_len = _ad3552r_reg_len(addr);
| | ^~~~~~~~~~~~~~~~~~~~~~
| | |
| | (25) returning to 'ad3552r_read_reg' from '_ad3552r_reg_len'
| 94 | err = ad3552r_transfer(dac, addr, reg_len, buf, true);
| | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| | |
| | (26) calling 'ad3552r_transfer' from 'ad3552r_read_reg'
|
+--> 'ad3552r_transfer': events 27-31
|
| 55 | static int ad3552r_transfer(struct ad3552r_desc *dac, u8 addr, u32 len,
| | ^~~~~~~~~~~~~~~~
| | |
| | (27) entry to 'ad3552r_transfer'
|......
| 62 | buf[0] |= is_read ? AD3552R_READ_BIT : 0;
| | ~~
| | |
| | (28) following 'true' branch (when 'is_read != 0')...
| | (29) ...to here
| 63 | if (is_read)
| | ~
| | |
| | (30) following 'true' branch (when 'is_read != 0')...
| 64 | return spi_write_then_read(dac->spi, buf, 1, data, len);
| | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| | |
| | (31) ...to here
|
<------+
|
'ad3552r_read_reg': events 32-33
|
| 94 | err = ad3552r_transfer(dac, addr, reg_len, buf, true);
| | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| | |
| | (
|
|
build_llvm_x86_64 / build:
drivers/iio/dac/ad3552r.c#L328
clang_analyzer: The right operand of '!=' is a garbage value [core.UndefinedBinaryOperatorResult]
328 | if (val1 != val)
| ^
|
|
build_llvm_x86_64 / build:
drivers/iio/dac/ad3552r.c#L170
clang_analyzer: The left operand of '&' is a garbage value [core.UndefinedBinaryOperatorResult]
170 | *val = !((tmp_val & AD3552R_MASK_CH_DAC_POWERDOWN(ch)) >>
| ~~~~~~~ ^
|
|
build_llvm_x86_64 / build:
drivers/iio/dac/ad3552r.c#L161
clang_analyzer: Assigned value is garbage or undefined [core.uninitialized.Assign]
161 | *val = tmp_val;
| ^ ~~~~~~~
|
|
build_llvm_x86_64 / build:
drivers/iio/dac/ad3552r.c#L118
clang_analyzer: The left expression of the compound assignment is an uninitialized value. The computed value will also be garbage [core.uninitialized.Assign]
118 | reg &= ~mask;
| ^
|
|
build_llvm_x86_64 / build:
drivers/iio/adc/ad7944.c#L812
clang_analyzer: Assigned value is garbage or undefined [core.uninitialized.Assign]
812 | indio_dev->available_scan_masks = chain_scan_masks;
| ^ ~~~~~~~~~~~~~~~~
|
|
build_llvm_x86_64 / build:
drivers/iio/adc/ad7944.c#L640
clang_analyzer: Right shift by '32' overflows the capacity of 'unsigned long' [core.BitwiseShift]
640 | scan_masks[0] = GENMASK(n_chain_dev - 1, 0);
| ^
|
|
build_llvm_x86_64 / build:
drivers/iio/adc/ad7124.c#L1178
clang_analyzer: Value stored to 'dev' during its initialization is never read [deadcode.DeadStores]
1178 | struct device *dev = &st->sd.spi->dev;
| ^~~ ~~~~~~~~~~~~~~~~
|
|
build_llvm_x86_64 / build:
drivers/iio/adc/ad7124.c#L864
clang_analyzer: Value stored to 'dev' during its initialization is never read [deadcode.DeadStores]
864 | struct device *dev = &st->sd.spi->dev;
| ^~~ ~~~~~~~~~~~~~~~~
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
dist-adi_ci_defconfig-gcc_14-arm
Expired
|
4.45 MB |
sha256:21dd58d82ed1317cfbfceb7b08859f7e2450d4e6536795b5240bb675ff1e13dc
|
|
|
dist-adi_ci_defconfig-gcc_14-arm64
Expired
|
2.44 MB |
sha256:abd2ccd6a8cb0c3b2d82c58e291be3eec0e572d68076f28f5f8698003587b23c
|
|
|
dist-adi_ci_defconfig-gcc_14-x86
Expired
|
2.33 MB |
sha256:ba2aa43e35cb1c009578b748a067e7d746d4321fcf3d890506682462ea225f1a
|
|
|
dist-adi_ci_defconfig-llvm_19-x86
Expired
|
2.24 MB |
sha256:3676d6176aee340dc9d8365b55fdac1993a4e094952992a39a239b3c1bd78c28
|
|