Hi Greg, On Tue, Jun 20, 2017 at 03:24:27PM -0500, Gustavo A. R. Silva wrote: > Simplify return logic to avoid unnecessary variable assignment. > > Signed-off-by: Gustavo A. R. Silva <garsilva@xxxxxxxxxxxxxx> My tree is already closed for v4.13. if you want to pick it up directly, here is my Acked-by, or I will take it for v4.14. Acked-by: Bin Liu <b-liu@xxxxxx> Regards, -Bin. > --- > drivers/usb/musb/musb_host.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/usb/musb/musb_host.c b/drivers/usb/musb/musb_host.c > index dbe617a..76decb8 100644 > --- a/drivers/usb/musb/musb_host.c > +++ b/drivers/usb/musb/musb_host.c > @@ -1540,7 +1540,7 @@ static int musb_rx_dma_iso_cppi41(struct dma_controller *dma, > struct dma_channel *channel = hw_ep->rx_channel; > void __iomem *epio = hw_ep->regs; > dma_addr_t *buf; > - u32 length, res; > + u32 length; > u16 val; > > buf = (void *)urb->iso_frame_desc[qh->iso_idx].offset + > @@ -1552,10 +1552,8 @@ static int musb_rx_dma_iso_cppi41(struct dma_controller *dma, > val |= MUSB_RXCSR_DMAENAB; > musb_writew(hw_ep->regs, MUSB_RXCSR, val); > > - res = dma->channel_program(channel, qh->maxpacket, 0, > + return dma->channel_program(channel, qh->maxpacket, 0, > (u32)buf, length); > - > - return res; > } > #else > static inline int musb_rx_dma_iso_cppi41(struct dma_controller *dma, > -- > 2.5.0 > -- 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
