From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dilip Kota Subject: Re: [PATCH v3 2/2] dwc: PCI: intel: Intel PCIe RC controller driver Date: Mon, 16 Sep 2019 11:03:39 +0800 Message-ID: <52ef93fd-d705-4163-d194-a1b749b2e7df@linux.intel.com> References: <20190906112044.GF9720@e119886-lin.cambridge.arm.com> <959a5f9b-2646-96e3-6a0f-0af1051ae1cb@linux.intel.com> <20190909083117.GH9720@e119886-lin.cambridge.arm.com> <22857835-1f98-b251-c94b-16b4b0a6dba2@linux.intel.com> <20190911103058.GP9720@e119886-lin.cambridge.arm.com> <20190912082517.GA9720@e119886-lin.cambridge.arm.com> <20190913101203.GE2680@smile.fi.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20190913101203.GE2680@smile.fi.intel.com> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: "andriy.shevchenko@intel.com" Cc: Gustavo Pimentel , Andrew Murray , "jingoohan1@gmail.com" , "lorenzo.pieralisi@arm.com" , "robh@kernel.org" , "martin.blumenstingl@googlemail.com" , "linux-pci@vger.kernel.org" , "hch@infradead.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "cheol.yong.kim@intel.com" , "chuanhua.lei@linux.intel.com" , "qi-ming.wu@intel.com" List-Id: devicetree@vger.kernel.org On 9/13/2019 6:12 PM, andriy.shevchenko@intel.com wrote: > On Fri, Sep 13, 2019 at 05:20:26PM +0800, Dilip Kota wrote: >> On 9/12/2019 6:49 PM, Gustavo Pimentel wrote: >>> On Thu, Sep 12, 2019 at 10:23:31, Dilip Kota >>> wrote: >>> Hi, I just return from parental leave, therefore I still trying to get >>> the pace in mailing list discussion. >>> >>> However your suggestion looks good, I agree that can go into DesignWare >>> driver to be available to all. >> Thanks Gustavo for the confirmation, i will add it in the next patch version >>> Just a small request, please do in general: >>> s/designware/DesignWare >> Sorry, i didnt understand this. > It means the reviewer asks you to name DesignWare in this form, > i.o.w. designware -> DesignWare. > > `man 1 sed` gives you more about it :-) Thanks Andy for clarifying it. Could you please also comment let me know your opinion on the driver naming. Below is the mail snippet. ====== >>> Add support to PCIe RC controller on Intel Universal >>> Gateway SoC. PCIe controller is based of Synopsys >>> Designware pci core. >>> +config PCIE_INTEL_AXI > [Andy]: I think that name here is too generic. Classical x86 seems not using this. [Dilip Kota]: This PCIe driver is for the Intel Gateway SoCs. So how about naming it is as "pcie-intel-gw"; pcie-intel-gw.c and Kconfig as PCIE_INTEL_GW. Andrew Murray is ok with this naming, please let me know your view. ======= Regards, Dilip >