Re: [PATCH 3/3] platform/chrome: cros_ec_ucsi: Implement UCSI PDC driver
From: kernel test robot
Date: Thu Mar 28 2024 - 05:06:10 EST
Hi Pavan,
kernel test robot noticed the following build warnings:
[auto build test WARNING on 4cece764965020c22cff7665b18a012006359095]
url: https://github.com/intel-lab-lkp/linux/commits/Pavan-Holla/usb-typec-ucsi-Provide-interface-for-UCSI-transport/20240326-074003
base: 4cece764965020c22cff7665b18a012006359095
patch link: https://lore.kernel.org/r/20240325-public-ucsi-h-v1-3-7c7e888edc0a%40chromium.org
patch subject: [PATCH 3/3] platform/chrome: cros_ec_ucsi: Implement UCSI PDC driver
config: x86_64-randconfig-123-20240328 (https://download.01.org/0day-ci/archive/20240328/202403281639.AVfLPAVm-lkp@xxxxxxxxx/config)
compiler: gcc-12 (Ubuntu 12.3.0-9ubuntu2) 12.3.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240328/202403281639.AVfLPAVm-lkp@xxxxxxxxx/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@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202403281639.AVfLPAVm-lkp@xxxxxxxxx/
sparse warnings: (new ones prefixed by >>)
>> drivers/platform/chrome/cros_ec_ucsi.c:119:24: sparse: sparse: symbol 'cros_ucsi_ops' was not declared. Should it be static?
vim +/cros_ucsi_ops +119 drivers/platform/chrome/cros_ec_ucsi.c
118
> 119 struct ucsi_operations cros_ucsi_ops = {
120 .read = cros_ucsi_read,
121 .async_write = cros_ucsi_async_write,
122 .sync_write = cros_ucsi_sync_write,
123 };
124
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki