Re: [PATCH] staging: unisys: fix format string %Lx to %llx for u64

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, May 06, 2016 at 03:07:04AM +0000, Kershner, David A wrote:
> 
> 
> > -----Original Message-----
> > From: Greg KH [mailto:gregkh@xxxxxxxxxxxxxxxxxxx]
> > Sent: Thursday, May 5, 2016 9:46 PM
> > To: Kershner, David A <David.Kershner@xxxxxxxxxx>
> > Cc: driverdev-devel@xxxxxxxxxxxxxxxxxxxxxx; *S-Par-Maintainer
> > <SParMaintainer@xxxxxxxxxx>; Arfvidson, Erik
> > <Erik.Arfvidson@xxxxxxxxxx>
> > Subject: Re: [PATCH] staging: unisys: fix format string %Lx to %llx for u64
> > 
> > On Thu, May 05, 2016 at 10:25:59PM -0400, David Kershner wrote:
> > > From: Erik Arfvidson <erik.arfvidson@xxxxxxxxxx>
> > >
> > > this patch fixes the following sonarqube issue.
> > > %Lx in format string (no. 1) requires 'unsigned long long'
> > > but the argument type is 'unsigned long long'
> > >
> > > Signed-off-by: Erik Arfvidson <erik.arfvidson@xxxxxxxxxx>
> > > Signed-off-by: David Kershner <david.kershner@xxxxxxxxxx>
> > > ---
> > >  drivers/staging/unisys/visorbus/visorbus_main.c | 10 +++++-----
> > >  1 file changed, 5 insertions(+), 5 deletions(-)
> > 
> > 
> > Please send patches as series, otherwise I have no idea what order to
> > apply them, so I'll just give up and delete them all from my queue.
> > 
> > You know better than this :(
> > 
> 
> Sorry, didn't send as a series cause they can be applied independently of
> each other.

How am I supposed to know that?  Would you like to be on the receiving
end of a random assortment of patches with no way to know if they are
independant or not?

> Should I consider all the patches sent dropped and send as a series
> this time?

Yes please.

thanks,

greg k-h
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux