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 X-Spam-Level: X-Spam-Status: No, score=-4.1 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1F80CC2BB1D for ; Wed, 8 Apr 2020 23:46:27 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id E7D132087E for ; Wed, 8 Apr 2020 23:46:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1586389587; bh=jdERzFagcZJpT6zeQ9ptPHc+W/HgbAkXikMkMpGMIc8=; h=Date:From:To:Cc:Subject:In-Reply-To:List-ID:From; b=2l56w5V/4O4Fwu8uu9CYE7LOrbVkaANc2KIaRTB3Z2Zsmmno1Tz7tTUdMwnC6mlg/ T5ZZBX/tgLsCmAvzEbDe9rQHnlTtKX0APnJlqw7UmlZRVlmKiVk0B7GMXM90iAKLsm CN8C6DmcBFpS+VYzFXq1ZMuJfyM5FlKQrSg3C8nw= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726521AbgDHXqZ (ORCPT ); Wed, 8 Apr 2020 19:46:25 -0400 Received: from mail.kernel.org ([198.145.29.99]:42304 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726508AbgDHXqZ (ORCPT ); Wed, 8 Apr 2020 19:46:25 -0400 Received: from localhost (mobile-166-175-188-68.mycingular.net [166.175.188.68]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 8821C20787; Wed, 8 Apr 2020 23:46:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1586389585; bh=jdERzFagcZJpT6zeQ9ptPHc+W/HgbAkXikMkMpGMIc8=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=UBEF4PPNtgYeAzJH79lLGGULnX41ZtOGL+nLtZMe5rqcdjOevaJL0F7SJcJqnQ/gL cbk/jdvfvN7/J57YJCjLt9hgnh47iQLIXE+WlxuVESZ/NcOWOUykYPQC8tj5yfO0K9 NtF4OXMIed85faV2LxGjwdhmPKjCAsn4lB7mBgEc= Date: Wed, 8 Apr 2020 18:46:22 -0500 From: Bjorn Helgaas To: Lad Prabhakar Cc: Yoshihiro Shimoda , Lorenzo Pieralisi , Rob Herring , Andrew Murray , Kishon Vijay Abraham I , Marek Vasut , Gustavo Pimentel , Jingoo Han , Shawn Lin , Tom Joseph , Heiko Stuebner , Geert Uytterhoeven , linux-pci@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-renesas-soc@vger.kernel.org, Chris Paterson , linux-rockchip@lists.infradead.org, Lad Prabhakar Subject: Re: [PATCH v7 4/8] PCI: endpoint: Add support to handle multiple base for mapping outbound memory Message-ID: <20200408234622.GA150772@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1586360280-10956-5-git-send-email-prabhakar.mahadev-lad.rj@bp.renesas.com> Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org On Wed, Apr 08, 2020 at 04:37:56PM +0100, Lad Prabhakar wrote: > R-Car PCIe controller has support to map multiple memory regions for > mapping the outbound memory in local system also the controller limits > single allocation for each region (that is, once a chunk is used from the > region it cannot be used to allocate a new one). This features inspires to > add support for handling multiple memory bases in endpoint framework. > > With this patch pci_epc_mem_init() initializes address space for endpoint > controller which support single window and whereas __pci_epc_mem_init() > now accepts pointer to multiple windows supported by endpoint controller. Adding a double underscore prefix usually indicates an internal function that skips some checking. It doesn't seem like quite the right thing for this external interface that adds functionality. Maybe the name could include something like "multi"? > Signed-off-by: Lad Prabhakar This needs an ack from Kishon, of course. This patch seems like it does several things that could possibly be split into separate patches? - Change pci_epc_mem_init() interface to add page_size argument (the only one that touches cadence & rockchip; it would be nice if this were a tiny patch) - Add struct pci_epc_mem_window - Add a pci_epc_multi_mem_init() or similar, implement pci_epc_mem_init() in terms of it (as you already do) Bjorn From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bjorn Helgaas Subject: Re: [PATCH v7 4/8] PCI: endpoint: Add support to handle multiple base for mapping outbound memory Date: Wed, 8 Apr 2020 18:46:22 -0500 Message-ID: <20200408234622.GA150772@google.com> References: <1586360280-10956-5-git-send-email-prabhakar.mahadev-lad.rj@bp.renesas.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1586360280-10956-5-git-send-email-prabhakar.mahadev-lad.rj-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Lad Prabhakar Cc: Yoshihiro Shimoda , Lorenzo Pieralisi , Rob Herring , Andrew Murray , Kishon Vijay Abraham I , Marek Vasut , Gustavo Pimentel , Jingoo Han , Shawn Lin , Tom Joseph , Heiko Stuebner , Geert Uytterhoeven , linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Chris Paterson , linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Lad Prabhakar List-Id: linux-rockchip.vger.kernel.org On Wed, Apr 08, 2020 at 04:37:56PM +0100, Lad Prabhakar wrote: > R-Car PCIe controller has support to map multiple memory regions for > mapping the outbound memory in local system also the controller limits > single allocation for each region (that is, once a chunk is used from the > region it cannot be used to allocate a new one). This features inspires to > add support for handling multiple memory bases in endpoint framework. > > With this patch pci_epc_mem_init() initializes address space for endpoint > controller which support single window and whereas __pci_epc_mem_init() > now accepts pointer to multiple windows supported by endpoint controller. Adding a double underscore prefix usually indicates an internal function that skips some checking. It doesn't seem like quite the right thing for this external interface that adds functionality. Maybe the name could include something like "multi"? > Signed-off-by: Lad Prabhakar This needs an ack from Kishon, of course. This patch seems like it does several things that could possibly be split into separate patches? - Change pci_epc_mem_init() interface to add page_size argument (the only one that touches cadence & rockchip; it would be nice if this were a tiny patch) - Add struct pci_epc_mem_window - Add a pci_epc_multi_mem_init() or similar, implement pci_epc_mem_init() in terms of it (as you already do) Bjorn 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 X-Spam-Level: X-Spam-Status: No, score=-4.0 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8E654C2BA2B for ; Wed, 8 Apr 2020 23:46:29 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 5B2FD2082F for ; Wed, 8 Apr 2020 23:46:29 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="YnYiyTuc"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="UBEF4PPN" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5B2FD2082F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Message-ID: Subject: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=yU7vEocYbltUo1N3XDvd+7Ni1gEbnqsIIsO2oZryfjU=; b=YnYiyTucnanT8F qOZrJVbkX58SFHY45UbvCr67ecOuVMx+10Fiw3fw6LQ3f29bQM5whmfYABH7I+dmRjs7L7s859hN1 LuM1fdr8ioMnXCYrKgNx4pFVuoHwAhN2gwrSM7YI+aO0Yl+iIGPYRuRsbJ6lmH+2FSusfNPvBTUEN M/Mn9j3jmC2f2X3snfkmPE3KwNd/mDV4VKK6KbokuRXDZgjgBydSwMJNkbSzpGDZBIEediMvKU08O jutEJMcMsCDKbgNOkPPORdzb6+z3uruUdyJT813KMtUafkGfnvGfzVZDwPExHfskemBdrStl8Py1N iOLFt3j8t8zuLkL4waLA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jMKOq-0006Nd-9i; Wed, 08 Apr 2020 23:46:28 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jMKOn-0006ND-HG; Wed, 08 Apr 2020 23:46:26 +0000 Received: from localhost (mobile-166-175-188-68.mycingular.net [166.175.188.68]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 8821C20787; Wed, 8 Apr 2020 23:46:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1586389585; bh=jdERzFagcZJpT6zeQ9ptPHc+W/HgbAkXikMkMpGMIc8=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=UBEF4PPNtgYeAzJH79lLGGULnX41ZtOGL+nLtZMe5rqcdjOevaJL0F7SJcJqnQ/gL cbk/jdvfvN7/J57YJCjLt9hgnh47iQLIXE+WlxuVESZ/NcOWOUykYPQC8tj5yfO0K9 NtF4OXMIed85faV2LxGjwdhmPKjCAsn4lB7mBgEc= Date: Wed, 8 Apr 2020 18:46:22 -0500 From: Bjorn Helgaas To: Lad Prabhakar Subject: Re: [PATCH v7 4/8] PCI: endpoint: Add support to handle multiple base for mapping outbound memory Message-ID: <20200408234622.GA150772@google.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1586360280-10956-5-git-send-email-prabhakar.mahadev-lad.rj@bp.renesas.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200408_164625_590463_0A02B2B4 X-CRM114-Status: GOOD ( 15.28 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, Chris Paterson , Lorenzo Pieralisi , Heiko Stuebner , Geert Uytterhoeven , Gustavo Pimentel , Yoshihiro Shimoda , linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Kishon Vijay Abraham I , linux-renesas-soc@vger.kernel.org, linux-rockchip@lists.infradead.org, Tom Joseph , Rob Herring , Lad Prabhakar , Jingoo Han , Andrew Murray , Shawn Lin , linux-arm-kernel@lists.infradead.org, Marek Vasut Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Apr 08, 2020 at 04:37:56PM +0100, Lad Prabhakar wrote: > R-Car PCIe controller has support to map multiple memory regions for > mapping the outbound memory in local system also the controller limits > single allocation for each region (that is, once a chunk is used from the > region it cannot be used to allocate a new one). This features inspires to > add support for handling multiple memory bases in endpoint framework. > > With this patch pci_epc_mem_init() initializes address space for endpoint > controller which support single window and whereas __pci_epc_mem_init() > now accepts pointer to multiple windows supported by endpoint controller. Adding a double underscore prefix usually indicates an internal function that skips some checking. It doesn't seem like quite the right thing for this external interface that adds functionality. Maybe the name could include something like "multi"? > Signed-off-by: Lad Prabhakar This needs an ack from Kishon, of course. This patch seems like it does several things that could possibly be split into separate patches? - Change pci_epc_mem_init() interface to add page_size argument (the only one that touches cadence & rockchip; it would be nice if this were a tiny patch) - Add struct pci_epc_mem_window - Add a pci_epc_multi_mem_init() or similar, implement pci_epc_mem_init() in terms of it (as you already do) Bjorn _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel