Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751863AbcL1RVM (ORCPT <rfc822;w@1wt.eu>);
        Wed, 28 Dec 2016 12:21:12 -0500
Received: from smtprelay2.synopsys.com ([198.182.60.111]:44625 "EHLO
        smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1751177AbcL1RVK (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Wed, 28 Dec 2016 12:21:10 -0500
Subject: Re: [PATCH] pci: rename *host* directory to *controller*
To: Bjorn Helgaas <helgaas@kernel.org>,
        Joao Pinto <Joao.Pinto@synopsys.com>
References: <1482912577-31356-1-git-send-email-kishon@ti.com>
 <20161228092228.GA14025@infradead.org>
 <fa775e80-68cc-0d76-855d-c7e64f8bf1fc@synopsys.com>
 <20161228164118.GB19653@bhelgaas-glaptop.roam.corp.google.com>
 <1ee66ce2-c321-2341-c964-c8b32218ca7e@synopsys.com>
CC: Christoph Hellwig <hch@infradead.org>,
        Kishon Vijay Abraham I <kishon@ti.com>,
        Bjorn Helgaas <bhelgaas@google.com>, <linux-kernel@vger.kernel.org>,
        <linux-pci@vger.kernel.org>, <nsekhar@ti.com>,
        Linus Torvalds <torvalds@linux-foundation.org>
From: Joao Pinto <Joao.Pinto@synopsys.com>
Message-ID: <426f0cb8-b9d4-7c0d-5eea-0ae0d4b01668@synopsys.com>
Date: Wed, 28 Dec 2016 17:20:27 +0000
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101
 Thunderbird/45.5.1
MIME-Version: 1.0
In-Reply-To: <1ee66ce2-c321-2341-c964-c8b32218ca7e@synopsys.com>
Content-Type: text/plain; charset="windows-1252"
Content-Transfer-Encoding: 8bit
X-Originating-IP: [10.107.19.116]
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 3165
Lines: 77

?s 5:17 PM de 12/28/2016, Joao Pinto escreveu:
> ?s 4:41 PM de 12/28/2016, Bjorn Helgaas escreveu:
>> On Wed, Dec 28, 2016 at 01:57:13PM +0000, Joao Pinto wrote:
>>> ?s 9:22 AM de 12/28/2016, Christoph Hellwig escreveu:
>>>> On Wed, Dec 28, 2016 at 01:39:37PM +0530, Kishon Vijay Abraham I wrote:
>>>>> As discussed during our LPC discussions, I'm posting the rename patch
>>>>> here. I'll post the rest of EP series before the next merge window.
>>>>>
>>>>> There might be hiccups because of this renaming but feel this is
>>>>> necessary for long-term maintenance.
>>>>
>>>> if we do this rename it would be great to get it to Linus NOW after
>>>> -rc1 as that minimizes the impact on the 4.11 merge window.
>>>
>>> Rename it to controller is a bit vague I thing since we have the PCI Endpoint IP
>>> also. Wouldn't be better to name it rc_controller?
>>
>> I think Kishon's whole goal is to add PCI Endpoint IP, so he wants a
>> neutral name that can cover both RC and Endpoint.
>>
>> I'm not a huge fan of "controller" because it feels a little bit long
>> and while I suppose it technically does include the concept of the PCI
>> interface of an endpoint, it still suggests more of the host side to
>> me.
>>
>> Doesn't USB have a similar situation?  I see there's a
>> drivers/usb/host/ (probably where we copied from in the first place).
>> Is a USB gadget the USB analog of what you're doing?  How do they
>> share code between the master/slave sides?
>>
> 
> The usb/host contains the implemnentations by the spec of the several
> *hci (USB Host) and then you can have for example the USB 3.0 Designware
> Host specific ops in dwc3 and for USB 2.0 in dwc2/.
> For device purposes it uses the core/ and then some of the device functions
> are extended from the gadget/ package in which you can use mass_storage and
> other types of functions.
> 
> In our case in PCI we have the core functions inside /drivers/pci and the host
> mangled inside host. I suggest:
> 
> drivers/pci
>  drivers/pci/core/
>  drivers/pci/core/hotplug
>  drivers/pci/core/pcie
>  drivers/pci/core/<all other files inside pci/ today>
>  drivers/pci/host
>  drivers/pci/dwc -> here would be pcie-designware and the specific vendor drivers

Correction:
drivers/pci/host/dwc -> here would be pcie-designware and the specific vendor
drivers

>  drivers/pci/<vendorN> -> here would be the drivers for vendorN controller

Correction:
drivers/pci/host/<vendorN> -> here would be the drivers for vendorN controller

>  drivers/pci/endpoint -> common code
>  drivers/pci/endpoint/dwc -> implementation of Synopsys specific endpoint ops
>  drivers/pci/<vendorN> -> implementation of other vendors specific endpoint ops
> 

Thanks

> Joao
> 
>> There's a drivers/ntb/hw/.  I don't know if "hw" is the *best* name,
>> but it's short and it at least conveys the idea that this code is
>> hardware-specific, not generic.
>>
>>>>>  drivers/pci/{host => controller}/Kconfig           |    0
>>>>>  drivers/pci/{host => controller}/Makefile          |    0
>>>>>  drivers/pci/{host => controller}/pci-aardvark.c    |    0
>>>>>  drivers/pci/{host => controller}/pci-dra7xx.c      |    0
>>>>> ...
>