From patchwork Mon Sep 9 06:14:50 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Umang Jain X-Patchwork-Id: 13795920 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id BBE72CD4F4C for ; Mon, 9 Sep 2024 06:16:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: MIME-Version:Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-Type: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; bh=Qmpet/hhx7JjIRUGtS/R+EbcLR4WdLCM3E47IPGV6sY=; b=sINiwIL6hp/DS/YEKFghXhbDzk YVmPIZOpIwPnKy4F0LC9ZQCviF7U8rCA0atOVFP7HZGojbh78nqmeVf6n1YR6EWsKpySvqhCBxofS 60UDNZIOscXMldldCivZEV0zg6GmYbAaMBE2snFLFD7cbYjhBwbVAmhMhqDqhJWpdcW/XHI+nj4Gk Oz4VFARGg6/hg6fkDpFoffphcysCdfR7E19p3Owm6QlN5B5J2jU/xGaRiWlhJ0GMN9f6658mVqkMr wS34Ha/hZ1WKbk37mSeXWGDOLiNs7zEAw4UrZ1QgFf6gA9SGwyKLHCZNfIsXpeX1ELhjz941+x0AO b5038BnA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1snXhE-00000000bT8-26Qa; Mon, 09 Sep 2024 06:16:20 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1snXgD-00000000bKQ-22UH; Mon, 09 Sep 2024 06:15:19 +0000 Received: from localhost.localdomain (unknown [IPv6:2405:201:2015:f873:55f8:639e:8e9f:12ec]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id E429E5A4; Mon, 9 Sep 2024 08:13:53 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1725862435; bh=A2rObrp0MBKP+bTZ+KGvyWu2AnfcTnyVZ/IYdlbhSL8=; h=From:To:Cc:Subject:Date:From; b=AJYcFsC1V1h0AWpxdZ3DOItXF0eTTo4+PIkJEsQF4AsY2Rk3vBGU2Ns+iVl+1WSWX T2ha1+Dh5kItlb5WyRAH/g54sk0S/1DJbEmtRzt9rE2h/UwtCNWeZvqYJzCd8X6/vi DpZ6SbHzx1YVzAwj1Ng0IKwug4bFduvVAPYU60xE= From: Umang Jain To: Broadcom internal kernel review list , Greg Kroah-Hartman Cc: linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-staging@lists.linux.dev, Kieran Bingham , Arnd Bergmann , Stefan Wahren , Laurent Pinchart , Umang Jain Subject: [PATCH v5 0/7] staging: vchiq_core: Refactor vchiq_bulk_transfer() logic Date: Mon, 9 Sep 2024 11:44:50 +0530 Message-ID: <20240909061457.255452-1-umang.jain@ideasonboard.com> X-Mailer: git-send-email 2.45.0 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240908_231517_692484_B0EBE8B6 X-CRM114-Status: GOOD ( 11.99 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Series mostly refactors vchiq_bulk_transfer() code according to Arnd's review comment from [1]: > You can also wrap vchiq_bulk_transfer() in order to have > four separate functions based on the different 'mode' > values and have them only take the arguments they actually > need. I didn't wrap vchiq_bulk_transfer(), instead created four differnet function, one for each mode. This will pave the way to address his second comment: > Ideally this should be cleaned up in a way that completely > avoids passing both user and kernel data at the same time. which is not part of this series and will be done on top as arguments shuffling will have to fix the sparse warnings that exists currently. Patch 1/7 first moves the VCHIQ_BULK_MODE_WAITING logic out of vchiq_bulk_transfer Patch 2/7 then moves the core logic to vchiq_bulk_transfer() which can be shared in subsequent patches Patch 3/7 and 4/7 refactors remaining bulk transfer modes Patch 5/7 finally drops the vchiq_bulk_transfer() as not needed patch 6/7 and 7/7 are drive by patches, noticed during development. [1]: https://lore.kernel.org/linux-staging/3d3b7368-93b2-4c0d-845e-4099c2de9dc1@app.fastmail.com/ changes in v5: - Fix regression for VCHIQ_BULK_MODE_BLOCKING in 3/7 (Fixes strange hang from v4 with 'vchiq_test -p') Changes in v4: - In 1/7 return 0 direcly instead of assigning status = 0 - In 3/7, fix null assignment of bulk waiter CHanges in v3: - Directly return if service == NULL in 1/7 and 4/7 - Drop set-but-unused variable from 1/7 Changes in v2: - Directly return if service == NULL in 3/7 - Prefix non-static functions with "vchiq_" (appropriate driver prefix) - Use typed argument for 'userdata' in 1/7 Umang Jain (7): staging: vchiq: Factor out bulk transfer for VCHIQ_BULK_MODE_WAITING staging: vchiq_core: Simplify vchiq_bulk_transfer() staging: vchiq_core: Factor out bulk transfer for blocking mode staging: vchiq_core: Factor out bulk transfer for (no/)callback mode staging: vchiq_core: Drop vchiq_bulk_transfer() staging: vchiq_core: Remove unused function argument staging: vchiq_core: Pass enumerated flag instead of int .../interface/vchiq_arm/vchiq_arm.c | 20 +- .../interface/vchiq_arm/vchiq_core.c | 330 +++++++++++------- .../interface/vchiq_arm/vchiq_core.h | 16 +- .../interface/vchiq_arm/vchiq_dev.c | 15 +- 4 files changed, 239 insertions(+), 142 deletions(-) Tested-by: Stefan Wahren