]> git.openfabrics.org - ~emulex/infiniband.git/commitdiff
staging: emxx_udc: Use min_t instead of min
authorEbru Akagunduz <ebru.akagunduz@gmail.com>
Thu, 2 Oct 2014 20:32:07 +0000 (23:32 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 2 Oct 2014 20:51:03 +0000 (13:51 -0700)
Use min_t instead of min function in emxx_udc.c

Fix checkpatch.pl warnings:
WARNING: min() should probably be min_t(u32, iBufSize, ep->ep.maxpacket)
WARNING: min() should probably be min_t(u32, data_size, ep->ep.maxpacket)
WARNING: min() should probably be min_t(u16, udc->ctrl.wLength, sizeof(status_data))

Changes in v2:
 - Fixed min function call as min_t

Signed-off-by: Ebru Akagunduz <ebru.akagunduz@gmail.com>
Reviewed-by: Josh Triplett <josh@joshtriplett.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/emxx_udc/emxx_udc.c

index 44fd58a955928ff68358439e0c19f38e2335b39c..1ae0013fb11889a8953d86719f49ccc59dca542a 100644 (file)
@@ -971,7 +971,7 @@ static int _nbu2ss_epn_out_data(
                && (iBufSize  >= sizeof(u32))) {
                nret = _nbu2ss_out_dma(udc, req, num, iBufSize);
        } else {
-               iBufSize = min(iBufSize, (u32)ep->ep.maxpacket);
+               iBufSize = min_t(u32, iBufSize, ep->ep.maxpacket);
                nret = _nbu2ss_epn_out_pio(udc, ep, req, iBufSize);
        }
 
@@ -1185,7 +1185,7 @@ static int _nbu2ss_epn_in_data(
                && (data_size >= sizeof(u32))) {
                nret = _nbu2ss_in_dma(udc, ep, req, num, data_size);
        } else {
-               data_size = min(data_size, (u32)ep->ep.maxpacket);
+               data_size = min_t(u32, data_size, ep->ep.maxpacket);
                nret = _nbu2ss_epn_in_pio(udc, ep, req, data_size);
        }
 
@@ -1603,7 +1603,7 @@ static int std_req_get_status(struct nbu2ss_udc *udc)
                return result;
        }
 
-       length = min(udc->ctrl.wLength, (u16)sizeof(status_data));
+       length = min_t(u16, udc->ctrl.wLength, sizeof(status_data));
 
        switch (recipient) {
        case USB_RECIP_DEVICE: