Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Frank Li <Frank.Li@nxp.com>
To:     Peter Chen <peter.chen@kernel.org>,
        Pawel Laszczak <pawell@cadence.com>,
        Roger Quadros <rogerq@kernel.org>,
        Aswath Govindraju <a-govindraju@ti.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        linux-usb@vger.kernel.org (open list:CADENCE USB3 DRD IP DRIVER),
        linux-kernel@vger.kernel.org (open list)
Cc:     imx@lists.linux.dev
Subject: [PATCH 2/2] usb: cdns3: optimize OUT transfer by copying only actual received data
Date:   Thu, 18 May 2023 16:49:46 -0400
Message-Id: <20230518204947.3770236-2-Frank.Li@nxp.com>
In-Reply-To: <20230518204947.3770236-1-Frank.Li@nxp.com>
References: <20230518204947.3770236-1-Frank.Li@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?OgciEXco+u6x4MmN/LfqcUvXtmSnIH/+AzaRR3AcuuBoZ8mXj+g3IO4JTKq0?=
 =?us-ascii?Q?wgKQECO8RUovfYJ4DqdIsMdRSGP0FSr1tM03DvLHwJ/8FUTZoIr39CxM13u9?=
 =?us-ascii?Q?cx+RzpSwVuOwqFE4WLjQagGKplRrzD0Si61avluKWRJm7d2DDSPoFLOkyrc8?=
 =?us-ascii?Q?kzOVnioAIbuEbPPnoGTHK04Ko+K74+YMrkEU+uU2fEYQAxw8+B7LKrZ4q35X?=
 =?us-ascii?Q?OeTFE83vyYeO+hthQ1LA23eOQT6InL/K5pnL+dhg8RYiCf79D08Ad31mv20r?=
 =?us-ascii?Q?gATMWrDKYcxEj2fvo4kOG/f0ekWoR18n55vM/lkaXO9ck31CQG/wwu3Gvv7y?=
 =?us-ascii?Q?M2EW9XD3gs0DwMpR2YjsAYeLdGTN6yfHxd0L+rv7tEVB/sYcRUIkHKSpdMu6?=
 =?us-ascii?Q?Y/0pA8ppi1AjcvIKj4hzgCYxuFqaa4Fd0i19Kfo66VP22oMgF82taDJ5XmCS?=
 =?us-ascii?Q?BJZ6qb5hU5+diwC1lyJZ9Wq4fy65AnQP8jz5AeYgiNcOuZMboxG+vjTdeJJb?=
 =?us-ascii?Q?rDJXGPjcKyExnodVsSUFEzAeCWEWPFyYdL8Fki9S2jjanSjj9Wk7nxMcGfLO?=
 =?us-ascii?Q?j3u1wU/DI3y9Z3qeG1KLRn+pKdI83AKPT8IeX6lnV/kao7C3speLmbsc1PvS?=
 =?us-ascii?Q?u28QhvBpS0vd2IePqpcKbrW9wGYNl+3Amx4hDOylySaC+fEvMxnZNbjy+Qp0?=
 =?us-ascii?Q?oIaXAIJKgX5s74RKSKVxvY8TjV+Ilq4senNsk6c66x4l62TEf24jqI63M9s7?=
 =?us-ascii?Q?pkYzhZeNzp4s2V71fFab+7BgFnt7h5BO2FYIeWGLsbfx+QsSwoX0B3veXidC?=
 =?us-ascii?Q?O/tQLGAE15pha+OvmVB2vTr0Lb6QgxJJXMD5vLAUO0LjIg+gA5nI79tKEZn9?=
 =?us-ascii?Q?I2Cy9C5mccKbdNXL2hIB8n5VpaxCjkNpGy1NHED8x0TEAIRDUw5HGS7wE9Bk?=
 =?us-ascii?Q?YxCRxdEp1SxQfuKUtw+FpqJ/FnJJP1YbSMg/A4xoSKosP4jJlYhU0BvFgGxs?=
 =?us-ascii?Q?zqMtBgIXjBOJU8MGqWON8LxwQXsKOIEAmz/wDq+/Z963TlOjuHsQwlB0I6fC?=
 =?us-ascii?Q?XcnoDcmYUTycj5Tz8acLv3T4Qbp00LBItMAH5ykNjOIUvpreDgCTfZn5GmSh?=
 =?us-ascii?Q?2Rd3S9BIAii6lV2RIWMZo1kxVuI4pBTe4RdvIzvrLBFpt9AfOtIOSrcwIen1?=
 =?us-ascii?Q?W9nZoCNzfrb8Z11ksExnzOFL0186OGcponuEfUgeRNsWkJJZi9oghP07E5Us?=
 =?us-ascii?Q?PUAl5Ra/6j33PjCSAcce35PpZg1uabUx4ukGAshOfltp1iV9B8uozexNYfZM?=
 =?us-ascii?Q?nDfI0x9tYGGIsKOP5+8rlOiAnkpP/YstY2wEDYSOhHBrb4YUrq5tEl5tOoCs?=
 =?us-ascii?Q?LpYFnvm2Z9MOyiteYL/5c7nLAz8fVOBJcSoITd7W0wO0OLvaUhH0kfQqvMOx?=
 =?us-ascii?Q?I7cwq17B9fYpOTeJiowsjeH34aS9DEiVtOp+UpHonjb4tDTJ8bkR/nMaVwyR?=
 =?us-ascii?Q?AfcwV039rdMv9pG3/aQ5h4YdfDt4dlKJfvJGa1ULTlmaJqN9RkzTXZTDauJC?=
 =?us-ascii?Q?GrcTGh1ZVEbq22dtsL4=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1aa8cf26-6237-4324-78d9-08db57e1774d
X-MS-Exchange-CrossTenant-AuthSource: AM6PR04MB4838.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 May 2023 20:50:08.6792
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: vMwivOhbEMb1zFVNSAkowk1QSCpKRGzc/MyV/yVict+Ka/JhcK7LKMkI2CjEiN7YDY/YqdlGhpMIjf2ros49vA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB8493
X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        RCVD_IN_DNSWL_NONE,SPF_HELO_PASS,T_SCC_BODY_TEXT_LINE,T_SPF_PERMERROR
        autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Previously, the entire length of the request, which is equal to or greater
than the actual data, was dma synced and memcpy when using the bounce
buffer. Actually only the actual data indicated by request->actual need be
synced and copied.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 drivers/usb/cdns3/cdns3-gadget.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/usb/cdns3/cdns3-gadget.c b/drivers/usb/cdns3/cdns3-gadget.c
index 09a0882a4e97..ea19253fd2d0 100644
--- a/drivers/usb/cdns3/cdns3-gadget.c
+++ b/drivers/usb/cdns3/cdns3-gadget.c
@@ -809,10 +809,10 @@ void cdns3_gadget_giveback(struct cdns3_endpoint *priv_ep,
 		/* Make DMA buffer CPU accessible */
 		dma_sync_single_for_cpu(priv_dev->sysdev,
 			priv_req->aligned_buf->dma,
-			priv_req->aligned_buf->size,
+			request->actual,
 			priv_req->aligned_buf->dir);
 		memcpy(request->buf, priv_req->aligned_buf->buf,
-		       request->length);
+		       request->actual);
 	}
 
 	priv_req->flags &= ~(REQUEST_PENDING | REQUEST_UNALIGNED);
-- 
2.34.1