staging: emxx_udc: use min helper macro for variable comparison
authorDeepak R Varma <drv@mailo.com>
Sun, 6 Nov 2022 10:21:58 +0000 (15:51 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 8 Nov 2022 15:26:58 +0000 (16:26 +0100)
Simplify code by using min helper macros in place of lengthy if/else
block oriented logical evaluation and value assignment. This issue is
identified by coccicheck using the minmax.cocci file.

Signed-off-by: Deepak R Varma <drv@mailo.com>
Link: https://lore.kernel.org/r/Y2eKxoUWtbPY/88b@qemulion
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/emxx_udc/emxx_udc.c

index b6abd37..b4e1917 100644 (file)
@@ -1004,10 +1004,7 @@ static int _nbu2ss_in_dma(struct nbu2ss_udc *udc, struct nbu2ss_ep *ep,
        /* MAX Packet Size */
        mpkt = _nbu2ss_readl(&preg->EP_REGS[num].EP_PCKT_ADRS) & EPN_MPKT;
 
-       if ((DMA_MAX_COUNT * mpkt) < length)
-               i_write_length = DMA_MAX_COUNT * mpkt;
-       else
-               i_write_length = length;
+       i_write_length = min(DMA_MAX_COUNT * mpkt, length);
 
        /*------------------------------------------------------------*/
        /* Number of transmission packets */