A historical FIXME suggested converting a dev_info() message to dev_dbg() after all Greybus modules reported valid IDs. Greybus hardware is stable today and the firmware filename is still useful to log at info level, so remove the obsolete FIXME and keep the existing behaviour. Replace it with a concise descriptive comment.
Signed-off-by: Ayaan Mirza Baig ayaanmirzabaig85@gmail.com --- drivers/staging/greybus/bootrom.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/staging/greybus/bootrom.c b/drivers/staging/greybus/bootrom.c index 156212301d58..8665be86dae6 100644 --- a/drivers/staging/greybus/bootrom.c +++ b/drivers/staging/greybus/bootrom.c @@ -165,10 +165,7 @@ static int find_firmware(struct gb_bootrom *bootrom, u8 stage) intf->ddbl1_manufacturer_id, intf->ddbl1_product_id, intf->vendor_id, intf->product_id);
- // FIXME: - // Turn to dev_dbg later after everyone has valid bootloaders with good - // ids, but leave this as dev_info for now to make it easier to track - // down "empty" vid/pid modules. + /* Log the firmware filename being requested */ dev_info(&connection->bundle->dev, "Firmware file '%s' requested\n", firmware_name);