Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751297AbdFASGZ (ORCPT ); Thu, 1 Jun 2017 14:06:25 -0400 Received: from mail-pf0-f181.google.com ([209.85.192.181]:35258 "EHLO mail-pf0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751274AbdFASGX (ORCPT ); Thu, 1 Jun 2017 14:06:23 -0400 MIME-Version: 1.0 In-Reply-To: References: <1495471182-12490-1-git-send-email-oza.oza@broadcom.com> <1495471182-12490-3-git-send-email-oza.oza@broadcom.com> From: Oza Oza Date: Thu, 1 Jun 2017 23:36:17 +0530 Message-ID: Subject: Re: [PATCH v7 2/3] PCI: Add support for PCI inbound windows resources To: Bjorn Helgaas Cc: Joerg Roedel , Robin Murphy , "open list:INTEL IOMMU (VT-d)" , "linux-pci@vger.kernel.org" , "linux-kernel@vger.kernel.org" , linux-arm , "devicetree@vger.kernel.org" , bcm-kernel-feedback-list , Oza Pawandeep Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2682 Lines: 66 On Thu, Jun 1, 2017 at 10:38 PM, Bjorn Helgaas wrote: > On Wed, May 31, 2017 at 11:17 AM, Oza Oza wrote: >> On Wed, May 31, 2017 at 4:12 AM, Bjorn Helgaas wrote: >>> On Mon, May 22, 2017 at 11:39 AM, Oza Pawandeep wrote: >>>> This patch adds support for inbound memory window >>>> for PCI RC drivers. >>>> >>>> It defines new function pci_create_root_bus2 which >>>> takes inbound resources as an argument and fills in the >>>> memory resource to PCI internal host bridge structure >>>> as inbound_windows. >>>> >>>> Legacy RC driver could continue to use pci_create_root_bus, >>>> but any RC driver who wants to reseve IOVAS for their >>>> inbound memory holes, should use new API pci_create_root_bus2. >>>> >>>> Signed-off-by: Oza Pawandeep >>>> ... >>> >>>> +struct pci_bus *pci_create_root_bus2(struct device *parent, int bus, >>>> + struct pci_ops *ops, void *sysdata, struct list_head *resources, >>>> + struct list_head *in_res) >>>> +{ >>>> + return pci_create_root_bus_msi(parent, bus, ops, sysdata, >>>> + resources, in_res, NULL); >>>> +} >>>> +EXPORT_SYMBOL_GPL(pci_create_root_bus2); >>> >>> Based on your response to Lorenzo's "[RFC/RFT PATCH 03/18] PCI: >>> Introduce pci_scan_root_bus_bridge()", I'm hoping you can avoid adding >>> yet another variant of pci_create_root_bus(). >>> >>> So I think I can wait for that to settle out and look for a v8? >>> >>> Bjorn >> >> Sure Bjorn, please wait for v8. >> >> But there is one more associated patch >> [PATCH v7 1/3] OF/PCI: Export inbound memory interface to PCI RC >> which basically aims to provide an interface to RC drivers for their >> inbound resources. >> RC driver already get their outbound resources from >> of_pci_get_host_bridge_resources, >> similar attempt for inbound dma-ranges. > > Not sure I understand. Patch 1/3 adds of_pci_get_dma_ranges(), but > none of the patches adds a caller, so I don't see the point of it yet. > > In general, if I'm expecting another revision of one patch in a > series, I expect the next revision to include *all* the patches in the > series. I normally don't pick out and apply individual patches from > the series. > > Bjorn Yes, it does not get called by anybody, because it is supposed to be called by RC drivers who want to reserve IOVAs, not all the PCI host bridge driver might call it, but certainly iproc based PCI driver has to call it. Then I will have PATCH v8, and with that, I will include PCI RC driver patch calling it as well. Thanks for the Review. Regards, Oza.