Messages in this thread | | | Subject | Re: [PATCH] scsi: BusLogic: use %X for u32 sized integer rather than %lX | From | Khalid Aziz <> | Date | Fri, 30 Jul 2021 15:57:16 -0600 |
| |
On 7/30/21 3:50 AM, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > An earlier fix changed the print format specifier for adapter->bios_addr > to use %lX however the integer is a u32 so the fix was wrong. Fix this > by using the correct %X format specifier. > > Addresses-Coverity: ("Invalid type in argument") > Fixes: 43622697117c ("scsi: BusLogic: use %lX for unsigned long rather than %X") > Signed-off-by: Colin Ian King <colin.king@canonical.com> > --- > drivers/scsi/BusLogic.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/scsi/BusLogic.c b/drivers/scsi/BusLogic.c > index adddcd589941..bd615db5c58c 100644 > --- a/drivers/scsi/BusLogic.c > +++ b/drivers/scsi/BusLogic.c > @@ -1711,7 +1711,7 @@ static bool __init blogic_reportconfig(struct blogic_adapter *adapter) > if (adapter->adapter_bus_type != BLOGIC_PCI_BUS) { > blogic_info(" DMA Channel: None, ", adapter); > if (adapter->bios_addr > 0) > - blogic_info("BIOS Address: 0x%lX, ", adapter, > + blogic_info("BIOS Address: 0x%X, ", adapter, > adapter->bios_addr); > else > blogic_info("BIOS Address: None, ", adapter); >
Acked-by: Khalid Aziz <khalid@gonehiking.org>
| |