[frank-w-bpi-r2-4.14:5.14-hdmilarb2 42/48] drivers/iommu/of_iommu.c:30:53: warning: cast to smaller integer type 'unsigned int' from 'const struct iommu_ops *'
by kernel test robot
tree: https://github.com/frank-w/BPI-R2-4.14 5.14-hdmilarb2
head: 77b4c96140f9bec5bb54fc804b401af1c9a6b4b5
commit: a03e47787fdf1b8cf17dff02113ca385b4aaa131 [42/48] iommu, more debug
config: x86_64-randconfig-a015-20210718 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 5d5b08761f944d5b9822d582378333cc4b36a0a7)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://github.com/frank-w/BPI-R2-4.14/commit/a03e47787fdf1b8cf17dff02113...
git remote add frank-w-bpi-r2-4.14 https://github.com/frank-w/BPI-R2-4.14
git fetch --no-tags frank-w-bpi-r2-4.14 5.14-hdmilarb2
git checkout a03e47787fdf1b8cf17dff02113ca385b4aaa131
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
In file included from drivers/iommu/of_iommu.c:9:
include/linux/iommu.h:576:51: warning: cast to smaller integer type 'unsigned int' from 'struct iommu_fwspec *' [-Wpointer-to-int-cast]
dev_err(dev,"%s:%d 0x%08x",__FUNCTION__,__LINE__,(unsigned int)fwspec);
^~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:112:32: note: expanded from macro 'dev_err'
_dev_err(dev, dev_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~~~
>> drivers/iommu/of_iommu.c:30:53: warning: cast to smaller integer type 'unsigned int' from 'const struct iommu_ops *' [-Wpointer-to-int-cast]
dev_err(dev,"%s:%d ops:%08x",__FUNCTION__,__LINE__,(unsigned int)ops);
^~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:112:32: note: expanded from macro 'dev_err'
_dev_err(dev, dev_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~~~
>> drivers/iommu/of_iommu.c:32:56: warning: cast to smaller integer type 'unsigned int' from 'int (*)(struct device *, struct of_phandle_args *)' [-Wpointer-to-int-cast]
dev_err(dev,"%s:%d xlate:%08x",__FUNCTION__,__LINE__,(unsigned int)ops->of_xlate);
^~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:112:32: note: expanded from macro 'dev_err'
_dev_err(dev, dev_fmt(fmt), ##__VA_ARGS__)
^~~~~~~~~~~
3 warnings generated.
vim +30 drivers/iommu/of_iommu.c
21
22 static int of_iommu_xlate(struct device *dev,
23 struct of_phandle_args *iommu_spec)
24 {
25 const struct iommu_ops *ops;
26 struct fwnode_handle *fwnode = &iommu_spec->np->fwnode;
27 int ret;
28
29 ops = iommu_ops_from_fwnode(fwnode);
> 30 dev_err(dev,"%s:%d ops:%08x",__FUNCTION__,__LINE__,(unsigned int)ops);
31 if (ops)
> 32 dev_err(dev,"%s:%d xlate:%08x",__FUNCTION__,__LINE__,(unsigned int)ops->of_xlate);
33
34 if ((ops && !ops->of_xlate) ||
35 !of_device_is_available(iommu_spec->np))
36 {
37 dev_err(dev,"no iommu");
38 return NO_IOMMU;
39 }
40 ret = iommu_fwspec_init(dev, &iommu_spec->np->fwnode, ops);
41 if (ret)
42 return ret;
43 /*
44 * The otherwise-empty fwspec handily serves to indicate the specific
45 * IOMMU device we're waiting for, which will be useful if we ever get
46 * a proper probe-ordering dependency mechanism in future.
47 */
48 if (!ops)
49 return driver_deferred_probe_check_state(dev);
50
51 if (!try_module_get(ops->owner))
52 return -ENODEV;
53
54 ret = ops->of_xlate(dev, iommu_spec);
55 module_put(ops->owner);
56 return ret;
57 }
58
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months
Re: [PATCH 2/3] hwmon: (occ) Remove sequence numbering and checksum calculation
by kernel test robot
Hi Eddie,
I love your patch! Perhaps something to improve:
[auto build test WARNING on hwmon/hwmon-next]
[also build test WARNING on linux/master linus/master v5.14-rc1 next-20210716]
[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]
url: https://github.com/0day-ci/linux/commits/Eddie-James/OCC-fsi-and-hwmon-Se...
base: https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git hwmon-next
config: i386-randconfig-s001-20210718 (attached as .config)
compiler: gcc-10 (Ubuntu 10.3.0-1ubuntu1~20.04) 10.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.3-341-g8af24329-dirty
# https://github.com/0day-ci/linux/commit/5e8ecc23325ef0310d83a4520071aae18...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Eddie-James/OCC-fsi-and-hwmon-Set-sequence-number-in-submit-interface/20210718-103535
git checkout 5e8ecc23325ef0310d83a4520071aae18418f88a
# save the attached .config to linux build tree
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=i386 SHELL=/bin/bash drivers/hwmon/occ/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
sparse warnings: (new ones prefixed by >>)
>> drivers/hwmon/occ/p8_i2c.c:104:30: sparse: sparse: incompatible types in comparison expression (different type sizes):
>> drivers/hwmon/occ/p8_i2c.c:104:30: sparse: unsigned int *
>> drivers/hwmon/occ/p8_i2c.c:104:30: sparse: unsigned long *
drivers/hwmon/occ/p8_i2c.c:107:42: sparse: sparse: incompatible types in comparison expression (different type sizes):
drivers/hwmon/occ/p8_i2c.c:107:42: sparse: unsigned int *
drivers/hwmon/occ/p8_i2c.c:107:42: sparse: unsigned long *
vim +104 drivers/hwmon/occ/p8_i2c.c
98
99 static int p8_i2c_occ_putscom_be(struct i2c_client *client, u32 address,
100 u8 *data, size_t len)
101 {
102 __be32 data0 = 0, data1 = 0;
103
> 104 memcpy(&data0, data, min(len, 4UL));
105 if (len > 4UL) {
106 len -= 4;
107 memcpy(&data1, data + 4, min(len, 4UL));
108 }
109
110 return p8_i2c_occ_putscom_u32(client, address, be32_to_cpu(data0),
111 be32_to_cpu(data1));
112 }
113
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months
Re: [PATCH 2/3] hwmon: (occ) Remove sequence numbering and checksum calculation
by kernel test robot
Hi Eddie,
I love your patch! Perhaps something to improve:
[auto build test WARNING on hwmon/hwmon-next]
[also build test WARNING on linux/master linus/master v5.14-rc1 next-20210716]
[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]
url: https://github.com/0day-ci/linux/commits/Eddie-James/OCC-fsi-and-hwmon-Se...
base: https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git hwmon-next
config: arm-randconfig-r033-20210718 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 5d5b08761f944d5b9822d582378333cc4b36a0a7)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
# https://github.com/0day-ci/linux/commit/5e8ecc23325ef0310d83a4520071aae18...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Eddie-James/OCC-fsi-and-hwmon-Set-sequence-number-in-submit-interface/20210718-103535
git checkout 5e8ecc23325ef0310d83a4520071aae18418f88a
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/hwmon/occ/p8_i2c.c:104:23: warning: comparison of distinct pointer types ('typeof (len) *' (aka 'unsigned int *') and 'typeof (4UL) *' (aka 'unsigned long *')) [-Wcompare-distinct-pointer-types]
memcpy(&data0, data, min(len, 4UL));
^~~~~~~~~~~~~
include/linux/minmax.h:45:19: note: expanded from macro 'min'
#define min(x, y) __careful_cmp(x, y, <)
^~~~~~~~~~~~~~~~~~~~~~
include/linux/minmax.h:36:24: note: expanded from macro '__careful_cmp'
__builtin_choose_expr(__safe_cmp(x, y), \
^~~~~~~~~~~~~~~~
include/linux/minmax.h:26:4: note: expanded from macro '__safe_cmp'
(__typecheck(x, y) && __no_side_effects(x, y))
^~~~~~~~~~~~~~~~~
include/linux/minmax.h:20:28: note: expanded from macro '__typecheck'
(!!(sizeof((typeof(x) *)1 == (typeof(y) *)1)))
~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~
drivers/hwmon/occ/p8_i2c.c:107:28: warning: comparison of distinct pointer types ('typeof (len) *' (aka 'unsigned int *') and 'typeof (4UL) *' (aka 'unsigned long *')) [-Wcompare-distinct-pointer-types]
memcpy(&data1, data + 4, min(len, 4UL));
^~~~~~~~~~~~~
include/linux/minmax.h:45:19: note: expanded from macro 'min'
#define min(x, y) __careful_cmp(x, y, <)
^~~~~~~~~~~~~~~~~~~~~~
include/linux/minmax.h:36:24: note: expanded from macro '__careful_cmp'
__builtin_choose_expr(__safe_cmp(x, y), \
^~~~~~~~~~~~~~~~
include/linux/minmax.h:26:4: note: expanded from macro '__safe_cmp'
(__typecheck(x, y) && __no_side_effects(x, y))
^~~~~~~~~~~~~~~~~
include/linux/minmax.h:20:28: note: expanded from macro '__typecheck'
(!!(sizeof((typeof(x) *)1 == (typeof(y) *)1)))
~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~
2 warnings generated.
vim +104 drivers/hwmon/occ/p8_i2c.c
98
99 static int p8_i2c_occ_putscom_be(struct i2c_client *client, u32 address,
100 u8 *data, size_t len)
101 {
102 __be32 data0 = 0, data1 = 0;
103
> 104 memcpy(&data0, data, min(len, 4UL));
105 if (len > 4UL) {
106 len -= 4;
107 memcpy(&data1, data + 4, min(len, 4UL));
108 }
109
110 return p8_i2c_occ_putscom_u32(client, address, be32_to_cpu(data0),
111 be32_to_cpu(data1));
112 }
113
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months
Re: [PATCH] ceph: Convert from atomic_t to refcount_t on ceph_snap_realm->nref
by kernel test robot
Hi Xiyu,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on ceph-client/for-linus]
[also build test WARNING on v5.14-rc1 next-20210716]
[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]
url: https://github.com/0day-ci/linux/commits/Xiyu-Yang/ceph-Convert-from-atom...
base: https://github.com/ceph/ceph-client.git for-linus
config: arc-randconfig-r016-20210718 (attached as .config)
compiler: arceb-elf-gcc (GCC) 10.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/af8e5c68ccb6a1e5aead78e10d6d0441c...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Xiyu-Yang/ceph-Convert-from-atomic_t-to-refcount_t-on-ceph_snap_realm-nref/20210718-111108
git checkout af8e5c68ccb6a1e5aead78e10d6d0441c032ba66
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-10.3.0 make.cross O=build_dir ARCH=arc SHELL=/bin/bash fs/ceph/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
In file included from fs/ceph/super.h:6,
from fs/ceph/quota.c:10:
>> include/linux/ceph/ceph_debug.h:5: warning: "pr_fmt" redefined
5 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
|
In file included from include/linux/kernel.h:17,
from include/asm-generic/bug.h:20,
from arch/arc/include/asm/bug.h:30,
from include/linux/bug.h:5,
from include/linux/refcount.h:96,
from fs/ceph/super.h:5,
from fs/ceph/quota.c:10:
include/linux/printk.h:301: note: this is the location of the previous definition
301 | #define pr_fmt(fmt) fmt
|
vim +/pr_fmt +5 include/linux/ceph/ceph_debug.h
de57606c23afde fs/ceph/ceph_debug.h Sage Weil 2009-10-06 4
de57606c23afde fs/ceph/ceph_debug.h Sage Weil 2009-10-06 @5 #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
de57606c23afde fs/ceph/ceph_debug.h Sage Weil 2009-10-06 6
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months
Re: [PATCH v2 01/12] KVM: x86: Report host tsc and realtime values in KVM_GET_CLOCK
by kernel test robot
Hi Oliver,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on kvm/queue]
[also build test WARNING on vhost/linux-next v5.14-rc1 next-20210716]
[cannot apply to kvmarm/next]
[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]
url: https://github.com/0day-ci/linux/commits/Oliver-Upton/KVM-Add-idempotent-...
base: https://git.kernel.org/pub/scm/virt/kvm/kvm.git queue
config: x86_64-randconfig-a014-20210718 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 5d5b08761f944d5b9822d582378333cc4b36a0a7)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://github.com/0day-ci/linux/commit/72b6d584e6ac692038a7d70e8782f0dfa...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Oliver-Upton/KVM-Add-idempotent-controls-for-migrating-system-counter-state/20210718-103407
git checkout 72b6d584e6ac692038a7d70e8782f0dfa179e5fb
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> arch/x86/kvm/x86.c:2784: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
* Returns true if realtime and TSC values were written back to the caller.
vim +2784 arch/x86/kvm/x86.c
2782
2783 /**
> 2784 * Returns true if realtime and TSC values were written back to the caller.
2785 * Returns false if a clock triplet cannot be obtained, such as if the host's
2786 * realtime clock is not based on the TSC.
2787 */
2788 static bool get_kvmclock_and_realtime(struct kvm *kvm, u64 *kvmclock_ns,
2789 u64 *realtime_ns, u64 *tsc)
2790 {
2791 struct kvm_arch *ka = &kvm->arch;
2792 struct pvclock_vcpu_time_info hv_clock;
2793 unsigned long flags;
2794 bool ret = false;
2795
2796 spin_lock_irqsave(&ka->pvclock_gtod_sync_lock, flags);
2797 if (!ka->use_master_clock) {
2798 spin_unlock_irqrestore(&ka->pvclock_gtod_sync_lock, flags);
2799 *kvmclock_ns = get_kvmclock_base_ns() + ka->kvmclock_offset;
2800 return false;
2801 }
2802
2803 hv_clock.tsc_timestamp = ka->master_cycle_now;
2804 hv_clock.system_time = ka->master_kernel_ns + ka->kvmclock_offset;
2805 spin_unlock_irqrestore(&ka->pvclock_gtod_sync_lock, flags);
2806
2807 /* both __this_cpu_read() and rdtsc() should be on the same cpu */
2808 get_cpu();
2809
2810 if (__this_cpu_read(cpu_tsc_khz)) {
2811 struct timespec64 ts;
2812 u64 tsc_val;
2813
2814 kvm_get_time_scale(NSEC_PER_SEC, __this_cpu_read(cpu_tsc_khz) * 1000LL,
2815 &hv_clock.tsc_shift,
2816 &hv_clock.tsc_to_system_mul);
2817
2818 if (kvm_get_walltime_and_clockread(&ts, &tsc_val)) {
2819 *realtime_ns = ts.tv_nsec + NSEC_PER_SEC * ts.tv_sec;
2820 *tsc = tsc_val;
2821 ret = true;
2822 }
2823
2824 *kvmclock_ns = __pvclock_read_cycles(&hv_clock, tsc_val);
2825 } else
2826 *kvmclock_ns = get_kvmclock_base_ns() + ka->kvmclock_offset;
2827
2828 put_cpu();
2829
2830 return ret;
2831 }
2832
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months
Re: [PATCH] dmaengine: xilinx: Add empty device_config function
by kernel test robot
Hi Marek,
I love your patch! Perhaps something to improve:
[auto build test WARNING on v5.14-rc1]
[also build test WARNING on next-20210716]
[cannot apply to xlnx/master]
[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]
url: https://github.com/0day-ci/linux/commits/Marek-Vasut/dmaengine-xilinx-Add...
base: e73f0f0ee7541171d89f2e2491130c7771ba58d3
config: arm-defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 10.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/89cc716182246c22a6c7bdb666952b3ce...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Marek-Vasut/dmaengine-xilinx-Add-empty-device_config-function/20210718-111119
git checkout 89cc716182246c22a6c7bdb666952b3cec945a0a
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-10.3.0 make.cross O=build_dir ARCH=arm SHELL=/bin/bash drivers/dma/xilinx/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
drivers/dma/xilinx/xilinx_dma.c:800: warning: expecting prototype for xilinx_dma_tx_descriptor(). Prototype was for xilinx_dma_alloc_tx_descriptor() instead
>> drivers/dma/xilinx/xilinx_dma.c:1659: warning: expecting prototype for xilinx_dma_issue_pending(). Prototype was for xilinx_dma_device_config() instead
drivers/dma/xilinx/xilinx_dma.c:2489: warning: expecting prototype for xilinx_dma_channel_set_config(). Prototype was for xilinx_vdma_channel_set_config() instead
vim +1659 drivers/dma/xilinx/xilinx_dma.c
1651
1652 /**
1653 * xilinx_dma_issue_pending - Configure the DMA channel
1654 * @dchan: DMA channel
1655 * @config: channel configuration
1656 */
1657 static int xilinx_dma_device_config(struct dma_chan *dchan,
1658 struct dma_slave_config *config)
> 1659 {
1660 return 0;
1661 }
1662
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months
[omap-audio:peter/linux-next-wip 21/63] sound/soc/codecs/tas2555.c:96:6: warning: variable 'ret' set but not used
by kernel test robot
tree: https://github.com/omap-audio/linux-audio peter/linux-next-wip
head: afd9de4bdc2361d1f36b24f414d1b2806b8ae755
commit: f546bca85f3cf64646dbd21dcc779633cc176038 [21/63] ASoC: Initial driver for TAS2555 Smart Amplifier
config: x86_64-randconfig-a013-20210718 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 5d5b08761f944d5b9822d582378333cc4b36a0a7)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://github.com/omap-audio/linux-audio/commit/f546bca85f3cf64646dbd21d...
git remote add omap-audio https://github.com/omap-audio/linux-audio
git fetch --no-tags omap-audio peter/linux-next-wip
git checkout f546bca85f3cf64646dbd21dcc779633cc176038
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> sound/soc/codecs/tas2555.c:96:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
int ret;
^
1 warning generated.
vim +/ret +96 sound/soc/codecs/tas2555.c
90
91 static unsigned int tas2555_read(struct snd_soc_component *component,
92 unsigned int reg)
93 {
94 struct tas2555_priv *tas2555 = snd_soc_component_get_drvdata(component);
95 unsigned int value = 0;
> 96 int ret;
97
98 dev_dbg(component->dev, "%s: BOOK:PAGE:REG %u:%u:%u\n", __func__,
99 TAS2555_BOOK_ID(reg), TAS2555_PAGE_ID(reg),
100 TAS2555_PAGE_REG(reg));
101 tas2555_change_book(tas2555, TAS2555_BOOK_ID(reg));
102 ret = regmap_read(tas2555->regmap, TAS2555_BOOK_REG(reg), &value);
103 return value;
104 }
105
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months
Re: [PATCH] net: phy: Add RGMII_ID/TXID/RXID handling to the DP83822 driver
by kernel test robot
Hi Marek,
I love your patch! Yet something to improve:
[auto build test ERROR on net/master]
[also build test ERROR on ipvs/master linus/master v5.14-rc1 next-20210716]
[cannot apply to net-next/master]
[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]
url: https://github.com/0day-ci/linux/commits/Marek-Vasut/net-phy-Add-RGMII_ID...
base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net.git 20192d9c9f6ae447c461285c915502ffbddf5696
config: arm-randconfig-r034-20210718 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 5d5b08761f944d5b9822d582378333cc4b36a0a7)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
# https://github.com/0day-ci/linux/commit/531a8b9dc73d7244ee6452e4b951f4637...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Marek-Vasut/net-phy-Add-RGMII_ID-TXID-RXID-handling-to-the-DP83822-driver/20210718-111900
git checkout 531a8b9dc73d7244ee6452e4b951f4637da20ded
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross O=build_dir ARCH=arm SHELL=/bin/bash drivers/net/phy/ fs/ceph/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
>> drivers/net/phy/dp83822.c:335:2: error: use of undeclared identifier 'ret'
ret = phy_clear_bits_mmd(phydev, DP83822_DEVADDR,
^
drivers/net/phy/dp83822.c:337:6: error: use of undeclared identifier 'ret'
if (ret < 0)
^
drivers/net/phy/dp83822.c:338:10: error: use of undeclared identifier 'ret'
return ret;
^
drivers/net/phy/dp83822.c:342:3: error: use of undeclared identifier 'ret'
ret = phy_modify_mmd(phydev, DP83822_DEVADDR, MII_DP83822_RCSR,
^
drivers/net/phy/dp83822.c:346:3: error: use of undeclared identifier 'ret'
ret = phy_modify_mmd(phydev, DP83822_DEVADDR, MII_DP83822_RCSR,
^
drivers/net/phy/dp83822.c:349:6: error: use of undeclared identifier 'ret'
if (ret < 0)
^
drivers/net/phy/dp83822.c:350:10: error: use of undeclared identifier 'ret'
return ret;
^
drivers/net/phy/dp83822.c:354:3: error: use of undeclared identifier 'ret'
ret = phy_modify_mmd(phydev, DP83822_DEVADDR, MII_DP83822_RCSR,
^
drivers/net/phy/dp83822.c:358:3: error: use of undeclared identifier 'ret'
ret = phy_modify_mmd(phydev, DP83822_DEVADDR, MII_DP83822_RCSR,
^
drivers/net/phy/dp83822.c:362:9: error: use of undeclared identifier 'ret'
return ret;
^
10 errors generated.
vim +/ret +335 drivers/net/phy/dp83822.c
330
331 static int dp8382x_disable_wol(struct phy_device *phydev)
332 {
333 u16 val = DP83822_WOL_EN | DP83822_WOL_MAGIC_EN | DP83822_WOL_SECURE_ON;
334
> 335 ret = phy_clear_bits_mmd(phydev, DP83822_DEVADDR,
336 MII_DP83822_WOL_CFG, val);
337 if (ret < 0)
338 return ret;
339
340 if (phydev->interface == PHY_INTERFACE_MODE_RGMII_ID ||
341 phydev->interface == PHY_INTERFACE_MODE_RGMII_RXID) {
342 ret = phy_modify_mmd(phydev, DP83822_DEVADDR, MII_DP83822_RCSR,
343 DP83822_RGMII_RX_CLOCK_SHIFT,
344 DP83822_RGMII_RX_CLOCK_SHIFT);
345 } else {
346 ret = phy_modify_mmd(phydev, DP83822_DEVADDR, MII_DP83822_RCSR,
347 DP83822_RGMII_RX_CLOCK_SHIFT, 0);
348 }
349 if (ret < 0)
350 return ret;
351
352 if (phydev->interface == PHY_INTERFACE_MODE_RGMII_ID ||
353 phydev->interface == PHY_INTERFACE_MODE_RGMII_TXID) {
354 ret = phy_modify_mmd(phydev, DP83822_DEVADDR, MII_DP83822_RCSR,
355 DP83822_RGMII_TX_CLOCK_SHIFT,
356 DP83822_RGMII_TX_CLOCK_SHIFT);
357 } else {
358 ret = phy_modify_mmd(phydev, DP83822_DEVADDR, MII_DP83822_RCSR,
359 DP83822_RGMII_TX_CLOCK_SHIFT, 0);
360 }
361
362 return ret;
363 }
364
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 2 months