Received: by 2002:a05:7412:d8a:b0:e2:908c:2ebd with SMTP id b10csp2468984rdg; Mon, 16 Oct 2023 05:43:18 -0700 (PDT) X-Google-Smtp-Source: AGHT+IE1eVzhwuSRbEfXdlfvihepItr9KIzNheJD7BVXqJX4hp3O6EU1uec0hTkZE6zAQ+jRRuEr X-Received: by 2002:a17:903:41c1:b0:1c6:30d1:7214 with SMTP id u1-20020a17090341c100b001c630d17214mr33200905ple.55.1697460197924; Mon, 16 Oct 2023 05:43:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1697460197; cv=none; d=google.com; s=arc-20160816; b=0zrqYNtnHzqD0YsVqGV2XjWLPgYvffgDmMXg4sBQmgcmkEvpgTNiiuUk1mxDZgpUJu g1MrOOfLr+AFdkQ/LtRGpxRovL6/bwLIuQ1/JjnjQSUrKo1FU+o1yZeDHJBZ27i68hdz KI5UZNJIPtLGUpRPu7eFuZHbmvugxkdrHL5mki7IS5Y8R5LCBt+crdIy3I2cGYnZVp4y 5bDfheQXffl3HdV0d6AcMxzESrRVFr6W96Q3v/mWIvDFEk7POLHBsJlVvropIvVJvxvm TBljWX22b3jJJb3jvtEqxECRI2roFfzws14FDEOesDIBN8HvL9DeKxpu5Wzhz+Jv7txF D5+w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=Tb5AOkfc3GSuO5Ciat2viUgVFSJvlxY/wYiGaxPQOYo=; fh=Z8b89HXImf/Il29ubQ390gX4WzjHDZQ9m/iurUCqrCQ=; b=a9Wqmf+ItfuIJrxZjT58KoBKS5yyFFW0/qyFvU9RfSVz+i+5EoehHsT3y3SBOpQDAR 9VxkZWxzkkA2CkPPuYL+qyKl3/RA5edYKuQgpnWAEPv4VyvMxE5bDGClDQsaF0kS5GMf PSDTgOBpr1PrBZ1CB3uIVj7cJWhH3WO0LwofG7thIR3P2vaRiqKrWGF7HRQDXDC3BXfC OsqYybwDOt1t1ifSNZ7zWqW/5c2T7RnhbP4+eEth59i7KQLYAdAY5iOLqyRIk9hgp/3V 8rYMYosV0/m3wspSC20DzO0WS0CBIuWqSsBACqN6JxxOji2lUEIYu7ruhnWnqjJkqWL3 k+6Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=Gi7eI2lC; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from agentk.vger.email (agentk.vger.email. [23.128.96.32]) by mx.google.com with ESMTPS id u12-20020a170903124c00b001b9ed021929si10880745plh.225.2023.10.16.05.43.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 Oct 2023 05:43:17 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) client-ip=23.128.96.32; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=Gi7eI2lC; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by agentk.vger.email (Postfix) with ESMTP id 955C5808D49D; Mon, 16 Oct 2023 05:43:14 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at agentk.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233562AbjJPMnA (ORCPT + 99 others); Mon, 16 Oct 2023 08:43:00 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53156 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233563AbjJPMm4 (ORCPT ); Mon, 16 Oct 2023 08:42:56 -0400 Received: from mail-yw1-x1130.google.com (mail-yw1-x1130.google.com [IPv6:2607:f8b0:4864:20::1130]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C7F7010C for ; Mon, 16 Oct 2023 05:42:51 -0700 (PDT) Received: by mail-yw1-x1130.google.com with SMTP id 00721157ae682-5a7be88e9ccso56156707b3.2 for ; Mon, 16 Oct 2023 05:42:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1697460171; x=1698064971; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=Tb5AOkfc3GSuO5Ciat2viUgVFSJvlxY/wYiGaxPQOYo=; b=Gi7eI2lCcGITDA3+EGEqozoKkHAzpJhDq3Y0KRFU1b8ICINBVtgDGDYwf9WVfM/vxF 7sxC/bXvQhAThoLCKDRCrhkxgsEFRH8UqikrAgURc4RqwNwhSHrd/i/0vrpwTMnCGRmx vl6VLRc1vZYEAJ1bDWIDl0qkWcmdEYdW0UOOPebYaXaqs8PqWuKNJgnsYubJjTkoBJ46 /oWDryrSUQMNTcvr+K9Uh2elFI6WxJjacKv3xSSSA/TOmtJukQDKq+BcMxqpq4jAzOlZ FOEjOmMrpCf2ADcLKGqo6oAcutkZdwpwfwHBYqLr4akAwTuQuzzp+4umt6crQrdyLLva vdEQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1697460171; x=1698064971; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=Tb5AOkfc3GSuO5Ciat2viUgVFSJvlxY/wYiGaxPQOYo=; b=KwvwxcjNtlTxz0+shVyq4n8AAuGQdBg+gR9lxJNy+oojgTVfR/3Rr0G/qjH65J1FD/ gxA/xdDtPntmGvcNUCdj8ml04XQ23ofa0ePEq4EJ+0coU34SVT3KqpwMSmQFuP3B0Ti+ vl2dsEDgO2nzSFHeXDgqVyaUHtGFdA+az0LCQH7F+SiLi3c21n5bQP73Hra5MhBiOiyX AK28kfj0TRdgy5Iv4Rcg6k6BVN5uhuiCCfnrpNHoACTD5rwMz/kbfCNBo6S7+msncu8C 5s4plFjj8nfW7vyPihq5FGanEAvAQCD+mTmmIZ4ZizLtLiyHUFNr0IfqBNK7kwlj73AL 6CzQ== X-Gm-Message-State: AOJu0YzMcl3XfaQMpB9SvMUl+6xwgbRjtX43X/STuRmwRpfnOFUoj/Or sT8+Z5MoK28khk0dbLpqO4o= X-Received: by 2002:a81:ac5b:0:b0:5a7:afc9:3579 with SMTP id z27-20020a81ac5b000000b005a7afc93579mr19492476ywj.18.1697460170675; Mon, 16 Oct 2023 05:42:50 -0700 (PDT) Received: from gilbert-PC ([105.112.31.148]) by smtp.gmail.com with ESMTPSA id i132-20020a81548a000000b005a7d9fca87dsm2148071ywb.107.2023.10.16.05.42.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 Oct 2023 05:42:48 -0700 (PDT) Date: Mon, 16 Oct 2023 13:42:42 +0100 From: Gilbert Adikankwu To: Julia Lawall , outreachy@lists.linux.dev Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, gregkh@linuxfoundation.org Subject: Re: your mail Message-ID: References: <3c8e9839-78f0-bb72-a195-ee4c54b68c7f@inria.fr> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <3c8e9839-78f0-bb72-a195-ee4c54b68c7f@inria.fr> X-Spam-Status: No, score=-0.6 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Mon, 16 Oct 2023 05:43:14 -0700 (PDT) On Mon, Oct 16, 2023 at 02:34:48PM +0200, Julia Lawall wrote: > > > On Mon, 16 Oct 2023, Gilbert Adikankwu wrote: > > > linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org > > Bcc: > > Subject: Re: [PATCH] staging: emxx_udc: Remove unnecessary parentheses around > > condition tests > > Reply-To: > > In-Reply-To: <6b60ed7-9d97-2071-44f8-83b173191ed@inria.fr> > > > > On Mon, Oct 16, 2023 at 02:15:06PM +0200, Julia Lawall wrote: > > > > > > > > > On Mon, 16 Oct 2023, Gilbert Adikankwu wrote: > > > > > > > Fix 47 warnings detected by checkpatch.pl about unnecessary parenthesis > > > > around condition tests. > > > > > > If you need to make any changes to the patch, there is no need to give the > > > count of the changes. It doesn't matter if it's 47, 46, 35, etc. > > > > > > julia > > > > > Hi Julia, > > > > I added the number because I saw I similar commit on the logs that did > > so. (commit b83970f23f36f0e2968872140e69f68118d82fe3) > > OK, I still think it's pointless... The person who looks at the commit 5 > years from now won't care about this information. They care about what > you did and why. > > julia > Ok that make sense. I will revise it. Do I send revision patch now or later today? > > > > > > > > > Signed-off-by: Gilbert Adikankwu > > > > --- > > > > drivers/staging/emxx_udc/emxx_udc.c | 72 ++++++++++++++--------------- > > > > 1 file changed, 36 insertions(+), 36 deletions(-) > > > > > > > > diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emxx_udc/emxx_udc.c > > > > index eb63daaca702..e8ddd691b788 100644 > > > > --- a/drivers/staging/emxx_udc/emxx_udc.c > > > > +++ b/drivers/staging/emxx_udc/emxx_udc.c > > > > @@ -149,8 +149,8 @@ static void _nbu2ss_ep0_complete(struct usb_ep *_ep, struct usb_request *_req) > > > > /* SET_FEATURE */ > > > > recipient = (u8)(p_ctrl->bRequestType & USB_RECIP_MASK); > > > > selector = le16_to_cpu(p_ctrl->wValue); > > > > - if ((recipient == USB_RECIP_DEVICE) && > > > > - (selector == USB_DEVICE_TEST_MODE)) { > > > > + if (recipient == USB_RECIP_DEVICE && > > > > + selector == USB_DEVICE_TEST_MODE) { > > > > wIndex = le16_to_cpu(p_ctrl->wIndex); > > > > test_mode = (u32)(wIndex >> 8); > > > > _nbu2ss_set_test_mode(udc, test_mode); > > > > @@ -287,7 +287,7 @@ static int _nbu2ss_epn_exit(struct nbu2ss_udc *udc, struct nbu2ss_ep *ep) > > > > u32 num; > > > > u32 data; > > > > > > > > - if ((ep->epnum == 0) || (udc->vbus_active == 0)) > > > > + if (ep->epnum == 0 || udc->vbus_active == 0) > > > > return -EINVAL; > > > > > > > > num = ep->epnum - 1; > > > > @@ -336,7 +336,7 @@ static void _nbu2ss_ep_dma_init(struct nbu2ss_udc *udc, struct nbu2ss_ep *ep) > > > > u32 data; > > > > > > > > data = _nbu2ss_readl(&udc->p_regs->USBSSCONF); > > > > - if (((ep->epnum == 0) || (data & (1 << ep->epnum)) == 0)) > > > > + if (ep->epnum == 0 || (data & (1 << ep->epnum)) == 0) > > > > return; /* Not Support DMA */ > > > > > > > > num = ep->epnum - 1; > > > > @@ -380,7 +380,7 @@ static void _nbu2ss_ep_dma_exit(struct nbu2ss_udc *udc, struct nbu2ss_ep *ep) > > > > return; /* VBUS OFF */ > > > > > > > > data = _nbu2ss_readl(&preg->USBSSCONF); > > > > - if ((ep->epnum == 0) || ((data & (1 << ep->epnum)) == 0)) > > > > + if (ep->epnum == 0 || (data & (1 << ep->epnum)) == 0) > > > > return; /* Not Support DMA */ > > > > > > > > num = ep->epnum - 1; > > > > @@ -560,7 +560,7 @@ static int ep0_out_overbytes(struct nbu2ss_udc *udc, u8 *p_buf, u32 length) > > > > union usb_reg_access temp_32; > > > > union usb_reg_access *p_buf_32 = (union usb_reg_access *)p_buf; > > > > > > > > - if ((length > 0) && (length < sizeof(u32))) { > > > > + if (length > 0 && length < sizeof(u32)) { > > > > temp_32.dw = _nbu2ss_readl(&udc->p_regs->EP0_READ); > > > > for (i = 0 ; i < length ; i++) > > > > p_buf_32->byte.DATA[i] = temp_32.byte.DATA[i]; > > > > @@ -608,7 +608,7 @@ static int ep0_in_overbytes(struct nbu2ss_udc *udc, > > > > union usb_reg_access temp_32; > > > > union usb_reg_access *p_buf_32 = (union usb_reg_access *)p_buf; > > > > > > > > - if ((i_remain_size > 0) && (i_remain_size < sizeof(u32))) { > > > > + if (i_remain_size > 0 && i_remain_size < sizeof(u32)) { > > > > for (i = 0 ; i < i_remain_size ; i++) > > > > temp_32.byte.DATA[i] = p_buf_32->byte.DATA[i]; > > > > _nbu2ss_ep_in_end(udc, 0, temp_32.dw, i_remain_size); > > > > @@ -701,7 +701,7 @@ static int _nbu2ss_ep0_in_transfer(struct nbu2ss_udc *udc, > > > > return result; > > > > } > > > > > > > > - if ((i_remain_size < sizeof(u32)) && (result != EP0_PACKETSIZE)) { > > > > + if (i_remain_size < sizeof(u32) && result != EP0_PACKETSIZE) { > > > > p_buffer += result; > > > > result += ep0_in_overbytes(udc, p_buffer, i_remain_size); > > > > req->div_len = result; > > > > @@ -738,7 +738,7 @@ static int _nbu2ss_ep0_out_transfer(struct nbu2ss_udc *udc, > > > > req->req.actual += result; > > > > i_recv_length -= result; > > > > > > > > - if ((i_recv_length > 0) && (i_recv_length < sizeof(u32))) { > > > > + if (i_recv_length > 0 && i_recv_length < sizeof(u32)) { > > > > p_buffer += result; > > > > i_remain_size -= result; > > > > > > > > @@ -891,8 +891,8 @@ static int _nbu2ss_epn_out_pio(struct nbu2ss_udc *udc, struct nbu2ss_ep *ep, > > > > > > > > req->req.actual += result; > > > > > > > > - if ((req->req.actual == req->req.length) || > > > > - ((req->req.actual % ep->ep.maxpacket) != 0)) { > > > > + if (req->req.actual == req->req.length || > > > > + (req->req.actual % ep->ep.maxpacket) != 0) { > > > > result = 0; > > > > } > > > > > > > > @@ -914,8 +914,8 @@ static int _nbu2ss_epn_out_data(struct nbu2ss_udc *udc, struct nbu2ss_ep *ep, > > > > > > > > i_buf_size = min((req->req.length - req->req.actual), data_size); > > > > > > > > - if ((ep->ep_type != USB_ENDPOINT_XFER_INT) && (req->req.dma != 0) && > > > > - (i_buf_size >= sizeof(u32))) { > > > > + if (ep->ep_type != USB_ENDPOINT_XFER_INT && req->req.dma != 0 && > > > > + i_buf_size >= sizeof(u32)) { > > > > nret = _nbu2ss_out_dma(udc, req, num, i_buf_size); > > > > } else { > > > > i_buf_size = min_t(u32, i_buf_size, ep->ep.maxpacket); > > > > @@ -954,8 +954,8 @@ static int _nbu2ss_epn_out_transfer(struct nbu2ss_udc *udc, > > > > } > > > > } > > > > } else { > > > > - if ((req->req.actual == req->req.length) || > > > > - ((req->req.actual % ep->ep.maxpacket) != 0)) { > > > > + if (req->req.actual == req->req.length || > > > > + (req->req.actual % ep->ep.maxpacket) != 0) { > > > > result = 0; > > > > } > > > > } > > > > @@ -1106,8 +1106,8 @@ static int _nbu2ss_epn_in_data(struct nbu2ss_udc *udc, struct nbu2ss_ep *ep, > > > > > > > > num = ep->epnum - 1; > > > > > > > > - if ((ep->ep_type != USB_ENDPOINT_XFER_INT) && (req->req.dma != 0) && > > > > - (data_size >= sizeof(u32))) { > > > > + if (ep->ep_type != USB_ENDPOINT_XFER_INT && req->req.dma != 0 && > > > > + data_size >= sizeof(u32)) { > > > > nret = _nbu2ss_in_dma(udc, ep, req, num, data_size); > > > > } else { > > > > data_size = min_t(u32, data_size, ep->ep.maxpacket); > > > > @@ -1238,7 +1238,7 @@ static void _nbu2ss_endpoint_toggle_reset(struct nbu2ss_udc *udc, u8 ep_adrs) > > > > u8 num; > > > > u32 data; > > > > > > > > - if ((ep_adrs == 0) || (ep_adrs == 0x80)) > > > > + if (ep_adrs == 0 || ep_adrs == 0x80) > > > > return; > > > > > > > > num = (ep_adrs & 0x7F) - 1; > > > > @@ -1261,7 +1261,7 @@ static void _nbu2ss_set_endpoint_stall(struct nbu2ss_udc *udc, > > > > struct nbu2ss_ep *ep; > > > > struct fc_regs __iomem *preg = udc->p_regs; > > > > > > > > - if ((ep_adrs == 0) || (ep_adrs == 0x80)) { > > > > + if (ep_adrs == 0 || ep_adrs == 0x80) { > > > > if (bstall) { > > > > /* Set STALL */ > > > > _nbu2ss_bitset(&preg->EP0_CONTROL, EP0_STL); > > > > @@ -1392,8 +1392,8 @@ static inline int _nbu2ss_req_feature(struct nbu2ss_udc *udc, bool bset) > > > > u8 ep_adrs; > > > > int result = -EOPNOTSUPP; > > > > > > > > - if ((udc->ctrl.wLength != 0x0000) || > > > > - (direction != USB_DIR_OUT)) { > > > > + if (udc->ctrl.wLength != 0x0000 || > > > > + direction != USB_DIR_OUT) { > > > > return -EINVAL; > > > > } > > > > > > > > @@ -1480,7 +1480,7 @@ static int std_req_get_status(struct nbu2ss_udc *udc) > > > > u8 ep_adrs; > > > > int result = -EINVAL; > > > > > > > > - if ((udc->ctrl.wValue != 0x0000) || (direction != USB_DIR_IN)) > > > > + if (udc->ctrl.wValue != 0x0000 || direction != USB_DIR_IN) > > > > return result; > > > > > > > > length = > > > > @@ -1542,9 +1542,9 @@ static int std_req_set_address(struct nbu2ss_udc *udc) > > > > int result = 0; > > > > u32 wValue = le16_to_cpu(udc->ctrl.wValue); > > > > > > > > - if ((udc->ctrl.bRequestType != 0x00) || > > > > - (udc->ctrl.wIndex != 0x0000) || > > > > - (udc->ctrl.wLength != 0x0000)) { > > > > + if (udc->ctrl.bRequestType != 0x00 || > > > > + udc->ctrl.wIndex != 0x0000 || > > > > + udc->ctrl.wLength != 0x0000) { > > > > return -EINVAL; > > > > } > > > > > > > > @@ -1564,9 +1564,9 @@ static int std_req_set_configuration(struct nbu2ss_udc *udc) > > > > { > > > > u32 config_value = (u32)(le16_to_cpu(udc->ctrl.wValue) & 0x00ff); > > > > > > > > - if ((udc->ctrl.wIndex != 0x0000) || > > > > - (udc->ctrl.wLength != 0x0000) || > > > > - (udc->ctrl.bRequestType != 0x00)) { > > > > + if (udc->ctrl.wIndex != 0x0000 || > > > > + udc->ctrl.wLength != 0x0000 || > > > > + udc->ctrl.bRequestType != 0x00) { > > > > return -EINVAL; > > > > } > > > > > > > > @@ -1838,8 +1838,8 @@ static void _nbu2ss_ep_done(struct nbu2ss_ep *ep, > > > > } > > > > > > > > #ifdef USE_DMA > > > > - if ((ep->direct == USB_DIR_OUT) && (ep->epnum > 0) && > > > > - (req->req.dma != 0)) > > > > + if (ep->direct == USB_DIR_OUT && ep->epnum > 0 && > > > > + req->req.dma != 0) > > > > _nbu2ss_dma_unmap_single(udc, ep, req, USB_DIR_OUT); > > > > #endif > > > > > > > > @@ -1931,7 +1931,7 @@ static inline void _nbu2ss_epn_in_dma_int(struct nbu2ss_udc *udc, > > > > mpkt = ep->ep.maxpacket; > > > > size = preq->actual % mpkt; > > > > if (size > 0) { > > > > - if (((preq->actual & 0x03) == 0) && (size < mpkt)) > > > > + if ((preq->actual & 0x03) == 0 && size < mpkt) > > > > _nbu2ss_ep_in_end(udc, ep->epnum, 0, 0); > > > > } else { > > > > _nbu2ss_epn_in_int(udc, ep, req); > > > > @@ -2428,8 +2428,8 @@ static int nbu2ss_ep_enable(struct usb_ep *_ep, > > > > } > > > > > > > > ep_type = usb_endpoint_type(desc); > > > > - if ((ep_type == USB_ENDPOINT_XFER_CONTROL) || > > > > - (ep_type == USB_ENDPOINT_XFER_ISOC)) { > > > > + if (ep_type == USB_ENDPOINT_XFER_CONTROL || > > > > + ep_type == USB_ENDPOINT_XFER_ISOC) { > > > > pr_err(" *** %s, bat bmAttributes\n", __func__); > > > > return -EINVAL; > > > > } > > > > @@ -2438,7 +2438,7 @@ static int nbu2ss_ep_enable(struct usb_ep *_ep, > > > > if (udc->vbus_active == 0) > > > > return -ESHUTDOWN; > > > > > > > > - if ((!udc->driver) || (udc->gadget.speed == USB_SPEED_UNKNOWN)) { > > > > + if (!udc->driver || udc->gadget.speed == USB_SPEED_UNKNOWN) { > > > > dev_err(ep->udc->dev, " *** %s, udc !!\n", __func__); > > > > return -ESHUTDOWN; > > > > } > > > > @@ -2603,8 +2603,8 @@ static int nbu2ss_ep_queue(struct usb_ep *_ep, > > > > } > > > > } > > > > > > > > - if ((ep->epnum > 0) && (ep->direct == USB_DIR_OUT) && > > > > - (req->req.dma != 0)) > > > > + if (ep->epnum > 0 && ep->direct == USB_DIR_OUT && > > > > + req->req.dma != 0) > > > > _nbu2ss_dma_map_single(udc, ep, req, USB_DIR_OUT); > > > > #endif > > > > > > > > -- > > > > 2.34.1 > > > > > > > > > > > > > >