Re: [PATCH v3] staging: greybus: bootrom: replace dev_info with dev_dbg for firmware name
From: kernel test robot
Date: Mon May 04 2026 - 21:56:20 EST
Hi Bentley,
kernel test robot noticed the following build warnings:
[auto build test WARNING on staging/staging-testing]
url: https://github.com/intel-lab-lkp/linux/commits/Bentley-Blacketer/staging-greybus-bootrom-replace-dev_info-with-dev_dbg-for-firmware-name/20260505-034304
base: staging/staging-testing
patch link: https://lore.kernel.org/r/20260430173045.4619-1-sonionwhat%40gmail.com
patch subject: [PATCH v3] staging: greybus: bootrom: replace dev_info with dev_dbg for firmware name
config: hexagon-randconfig-r071-20260505 (https://download.01.org/0day-ci/archive/20260505/202605050911.O85GmqxE-lkp@xxxxxxxxx/config)
compiler: clang version 16.0.6 (https://github.com/llvm/llvm-project 7cbf1a2591520c2491aa35339f227775f4d3adf6)
smatch: v0.5.0-9065-ge9cc34fd
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/202605050911.O85GmqxE-lkp@xxxxxxxxx/
smatch warnings:
drivers/staging/greybus/bootrom.c:166 find_firmware() warn: inconsistent indenting
vim +166 drivers/staging/greybus/bootrom.c
f1e941a6e4b71a drivers/staging/greybus/firmware.c Viresh Kumar 2015-11-26 142
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 143 /* This returns path of the firmware blob on the disk */
68793c4c8824a0 drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-22 144 static int find_firmware(struct gb_bootrom *bootrom, u8 stage)
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 145 {
5a53e02eaf223c drivers/staging/greybus/bootrom.c Viresh Kumar 2016-04-03 146 struct gb_connection *connection = bootrom->connection;
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 147 struct gb_interface *intf = connection->bundle->intf;
56c78715eaaeba drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-16 148 char firmware_name[49];
fc41c2da44c510 drivers/staging/greybus/firmware.c Eli Sennesh 2016-01-08 149 int rc;
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 150
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 151 /* Already have a firmware, free it */
5a53e02eaf223c drivers/staging/greybus/bootrom.c Viresh Kumar 2016-04-03 152 free_firmware(bootrom);
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 153
56c78715eaaeba drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-16 154 /* Bootrom protocol is only supported for loading Stage 2 firmware */
56c78715eaaeba drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-16 155 if (stage != 2) {
56c78715eaaeba drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-16 156 dev_err(&connection->bundle->dev, "Invalid boot stage: %u\n",
56c78715eaaeba drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-16 157 stage);
56c78715eaaeba drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-16 158 return -EINVAL;
56c78715eaaeba drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-16 159 }
56c78715eaaeba drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-16 160
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 161 /*
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 162 * Create firmware name
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 163 *
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 164 * XXX Name it properly..
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 165 */
af0b4d5a19e3d4 drivers/staging/greybus/firmware.c Johan Hovold 2015-08-28 @166 snprintf(firmware_name, sizeof(firmware_name),
8a704565ebda96 drivers/staging/greybus/bootrom.c Greg Kroah-Hartman 2016-07-20 167 FW_NAME_PREFIX "%08x_%08x_%08x_%08x_s2l.tftf",
b32a5c5346bdaa drivers/staging/greybus/firmware.c Viresh Kumar 2015-12-22 168 intf->ddbl1_manufacturer_id, intf->ddbl1_product_id,
56c78715eaaeba drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-16 169 intf->vendor_id, intf->product_id);
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 170
6472c9b9299508 drivers/staging/greybus/bootrom.c Bentley Blacketer 2026-04-30 171 dev_dbg(&connection->bundle->dev, "Firmware file '%s' requested\n",
eb8fafdfb9fce9 drivers/staging/greybus/firmware.c Greg Kroah-Hartman 2016-01-20 172 firmware_name);
5a53e02eaf223c drivers/staging/greybus/bootrom.c Viresh Kumar 2016-04-03 173 rc = request_firmware(&bootrom->fw, firmware_name,
0a72bd36df9478 drivers/staging/greybus/firmware.c Greg Kroah-Hartman 2015-10-14 174 &connection->bundle->dev);
2d6f1c29988c0f drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-22 175 if (rc) {
68793c4c8824a0 drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-22 176 dev_err(&connection->bundle->dev,
68793c4c8824a0 drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-22 177 "failed to find %s firmware (%d)\n", firmware_name, rc);
2d6f1c29988c0f drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-22 178 }
2d6f1c29988c0f drivers/staging/greybus/bootrom.c Viresh Kumar 2016-07-22 179
fc41c2da44c510 drivers/staging/greybus/firmware.c Eli Sennesh 2016-01-08 180 return rc;
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 181 }
90f1b617d88f14 drivers/staging/greybus/firmware.c Viresh Kumar 2015-08-12 182
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki