Re: [PATCH] USB: storage: fix compile warning

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

 



On Tue, Dec 10, 2013 at 01:18:48PM +0800, Ming Lei wrote:
> On Tue, Dec 10, 2013 at 1:05 PM, Greg Kroah-Hartman
> <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> > On Tue, Dec 10, 2013 at 12:37:24PM +0800, Ming Lei wrote:
> >> This patch should fix the below compile warning:
> >>
> >> drivers/usb/storage/protocol.c: In function 'usb_stor_access_xfer_buf':
> >> drivers/usb/storage/protocol.c:155:22: warning: comparison of distinct
> >> pointer types lacks a cast [enabled by default]
> >>
> >> Reported-by: kbuild test robot <fengguang.wu@xxxxxxxxx>
> >> Reported-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>
> >> ---
> >>  drivers/usb/storage/protocol.c |    3 ++-
> >>  1 file changed, 2 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/drivers/usb/storage/protocol.c b/drivers/usb/storage/protocol.c
> >> index f54e5fe..599d88f 100644
> >> --- a/drivers/usb/storage/protocol.c
> >> +++ b/drivers/usb/storage/protocol.c
> >> @@ -152,7 +152,8 @@ unsigned int usb_stor_access_xfer_buf(unsigned char *buffer,
> >>               return cnt;
> >>
> >>       while (sg_miter_next(&miter) && cnt < buflen) {
> >> -             unsigned int len = min(miter.length, buflen - cnt);
> >> +             unsigned int len = min_t(unsigned, miter.length,
> >
> > s/unsigned,/unsigned int/
> 
> Actually, the two are same, and I may change to 'unsigned int' if
> you require that, also signed-off-by is missed for the patch.

Ok, care to fix both of these and resend?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux