From mboxrd@z Thu Jan 1 00:00:00 1970 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 2BD66ECAAD8 for ; Tue, 13 Sep 2022 22:20:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:References: List-Owner; bh=ovDCcIyPXlPqqf1OBRHTEZtKWwsEVM3o2ecEwCfQ9KM=; b=ZbZF0RQ6Zw5Pfu 2kJTsahX+2cKmmzWH0mXcO8NeZ2tYewm+3RKST/sz+H4mBOMasmZ3vkwgQ7ure6NqH8BoMB05MYVO thxQIHoGNX4inObTdGhfGI+lwPHzkYjTkitvLbx6dmK6shYXXNhZqWAX+JNcvBxb/2JBJqpbgookv hjpgYL4HH3ZXkE9wxbJibx+CvaoS/g4VXg359Rrirr7UNDbEEDnE/rvUEessrd1MkZwgxlV5jp96L f1tbN5bavQdOhwklEL/oPPiBcqj4R6D6247DZ1iJ2TRq9y0jYKYPTH8vFpkBSyYT5WiS+Kui/g7m/ wZ2PANl7AzArVeOW35YQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oYEFY-000Htr-Gg; Tue, 13 Sep 2022 22:19:24 +0000 Received: from ams.source.kernel.org ([2604:1380:4601:e00::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oYEFV-000HqZ-9q for linux-arm-kernel@lists.infradead.org; Tue, 13 Sep 2022 22:19:22 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id C7C7CB810F5; Tue, 13 Sep 2022 22:19:19 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 51E8FC433C1; Tue, 13 Sep 2022 22:19:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1663107558; bh=vOM7sqZItl/trS+7jw/NfA7mCPUKtLKs8p8juFEz1LQ=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=PqY2sMOHIFT9Hb81j94J0rf9pS4s4sfT4NMscI+2mfmuako+a2b4cqOx8dG/wawnr Jh5hfF3unFix49Lcr+LmcEVF6JQ+zzbUJj/toSoRvoTw/5VVUpbrR+xLpAu3fZauyk Rx8/zEI88uPJ/xbsGWJTH2tyyZ7PzfNw0miGbHeXfoVKCy50rqL7VeYow23AQUEnq4 n/hKChC9+JbfnKuz5cl04SNOvomK8erZlvC948pmx9bnHood+2BbE8a07mJ5/Ue7fe LCJW18jHOZcHWunYifNqxyn1pYn/oFCUMLwPNzLJF4XSCJR4oIHlec/buR4H9G26Lo cO2ze6p4tuRDw== Date: Tue, 13 Sep 2022 17:19:16 -0500 From: Bjorn Helgaas To: Frank Li Cc: maz@kernel.org, tglx@linutronix.de, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kw@linux.com, bhelgaas@google.com, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-pci@vger.kernel.org, peng.fan@nxp.com, aisheng.dong@nxp.com, jdmason@kudzu.us, kernel@pengutronix.de, festevam@gmail.com, linux-imx@nxp.com, kishon@ti.com, lorenzo.pieralisi@arm.com, ntb@lists.linux.dev, lznuaa@gmail.com, imx@lists.linux.dev, manivannan.sadhasivam@linaro.org Subject: Re: [PATCH v10 5/6] PCI: endpoint: makeup pci-epf-vntb.c Message-ID: <20220913221916.GA627631@bhelgaas> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220913210957.3493826-6-Frank.Li@nxp.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220913_151921_516542_90BFB3A1 X-CRM114-Status: GOOD ( 16.88 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org In subject, capitalize and change "makeup" to "Clean up": PCI: endpoint: pci-epf-vntb: Clean up On Tue, Sep 13, 2022 at 04:09:56PM -0500, Frank Li wrote: > Remove unused field: epf_db_phy. > Remove __iomem before epf_db. > Remove dupicate check if (readl(ntb->epf_db + i * 4)). > Using readl_relaxed instead of readl. > Using marco ENTRY_SIZE instead of number 4 at all place. Add "()" after function names. s/marco/macro/ It would be nice if "ENTRY_SIZE" had a hint about what kind of entry we're talking about. Since this is a collection of random cleanups, I noticed a typo in epf_ntb_configure_interrupt() kernel-doc: s/capaiblity/capability/ The struct epf_ntb_ctrl definition is also whitespace-damaged. The members of struct epf_ntb_ctrl and struct epf_ntb should follow the same indentation style. Some members of struct epf_ntb_ctrl are indented with a tab, others with space. Either make them all tabs and indent struct epf_ntb similarly, or indent the struct epf_ntb_ctrl members with a single space. The comments in the file have a whole collection of ways to spell vhost: Virtual Host, VHOST, VHost, vHOST, vhost. Make them all the same, please. You can use "Virtual Host (VHOST)" or whatever the first time if you want to use the short version later. Same for host/HOST/etc. I don't want to read things like this: @ntb: NTB device that facilitates communication between HOST and vHOST2 Wrapper to initialize a particular EPC interface and start the workqueue to check for commands from host. and wonder whether "host" is supposed to be the same as "HOST". Also, why does that say "vHOST*2*"? There are several instances of "HOST1" and "HOST2" (and "vHOST2"). Should those appear somewhere in the diagram at the top of the file? The diagram starts with "/**" which means it's kernel-doc, but the diagram is not kernel-doc. Please run this: scripts/kernel-doc -v -none drivers/pci/endpoint/functions/pci-epf-vntb.c and fix all the warnings. Bjorn _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel