usb: gadget: bdc: remove unnecessary AND operation when get ep maxp
authorChunfeng Yun <chunfeng.yun@mediatek.com>
Fri, 13 Aug 2021 06:30:52 +0000 (14:30 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 13 Aug 2021 11:05:51 +0000 (13:05 +0200)
usb_endpoint_maxp() already returns actual max packet size, no need
to AND 0x7ff.

Acked-by: Felipe Balbi <balbi@kernel.org>
Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
Link: https://lore.kernel.org/r/1628836253-7432-6-git-send-email-chunfeng.yun@mediatek.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/gadget/udc/bdc/bdc_cmd.c

index 995f79c..6788731 100644 (file)
@@ -153,7 +153,6 @@ int bdc_config_ep(struct bdc *bdc, struct bdc_ep *ep)
        si = clamp_val(si, 1, 16) - 1;
 
        mps = usb_endpoint_maxp(desc);
-       mps &= 0x7ff;
        param2 |= mps << MP_SHIFT;
        param2 |= usb_endpoint_type(desc) << EPT_SHIFT;