From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755081AbcBEPnq (ORCPT ); Fri, 5 Feb 2016 10:43:46 -0500 Received: from mout.kundenserver.de ([217.72.192.73]:51523 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751804AbcBEPnp (ORCPT ); Fri, 5 Feb 2016 10:43:45 -0500 From: Arnd Bergmann To: Joao Pinto Cc: Bjorn Helgaas , Vineet.Gupta1@synopsys.com, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, linux-snps-arc@lists.infradead.org, CARLOS.PALMINHA@synopsys.com, Alexey.Brodkin@synopsys.com, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org Subject: Re: [PATCH v8 2/2] add new platform driver for PCI RC Date: Fri, 05 Feb 2016 16:43:05 +0100 Message-ID: <3217683.43eupG8Vx4@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <56B4B6FB.1040500@synopsys.com> References: <4427983.0G6mVNKP5W@wuerfel> <56B4B6FB.1040500@synopsys.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:HP1ZKz6xAfIadLtdpO1WkJ05YA4TNHtEReOD/tS9lrV+OvtI/7R hHXk1+cfNHe/hSuahCveen8dP711PWKCSFsOAAe76jjQY7z8CxLyoAprR0dKcWfHbEy/axY eo2R2+uimOLfxYM6SknbkcY/ocYKpD2w7Q/ps7XCMOyI/IXA7mfOAcIAtKa9BzFgrA9io/s b450c9wuE6roehxm37vFA== X-UI-Out-Filterresults: notjunk:1;V01:K0:X9vqrAhPOYs=:o/7IAahij3N+tpF1Z/V9tw HO8+wukOeDJ7hRLFL8ftQ1pTr4KmKecz5wYYRb2MUs4OqkSKDQIi04br9yHiSnnJOTkYGGvvZ IiaqErQ5oDkSqDlmVZ/Jld9TNqJdveFhCwTciMvjDMbdHEvZWGd7Q7CbOABDg7GBBnDSLSx5P tJ+8Dkq9mfWPvFCCRnQ7e3kz3Yjmpl7VWURNC98UPShb6sj3Pc4ulpPtjZo8k1iGvNijopJ5h X25oEur7IM41+cZDcZzoqhx5AF1Ymcw6D2WhH3mj6JAHL7WRPKXFGURUgp195SGmYgCsAXLjP 2J/WqWvztnhjQdSypcRKcwU4SZh/bIp0dPwg3ocCs9R3yeaVQiW223wnpQueVZIyxkKdBQXYK kh9OhXNVuJAzOTPbyTXLYVbFpj9rpge/hhhyqwUEKZDfUm7lOz22+VPogAi6YhAUidnv+EAZi Sna4/DcP275MQIf9aB8PHKOjIs6sBwAZWmKYOC8k/nYjhcslWoVN+c7klYQIHwAat82DfBEkk T4w0AXlruLyC5Y8s330+9UqGnz6bRlURZbSSgGuplNKC8m5bJ3d253q1phH/gRw9cSk/xss3i nh7/METHxK95qf1bdaWEMRYoX1def+s7GMu62rf1xavCLsPTYdcNyPDWQuMQzwiU10d9GmSfU mncsxSF2rhDAY0O3lKVhxw3gq4mqjsT3A4YP4/aneVYVPsgFnTS23VnRS6oJXGrnJ++F57PHC 2Hi+baC4VnYQ5wf5 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday 05 February 2016 14:51:39 Joao Pinto wrote: > > It is a driver that is useful for PCIe RC prototyping and to be a reference > platform driver for DesignWare PCIe RC, I don't know if merging some of the > driver's code into pcie-designware is really necessary depends on the usefulness > of it. I would suggest that bigger step to be done in a 2nd stage since I will > be around to maintain what's necessary. Agree? > > I made a patch that is applicable to Bjorn's host/pcie-synopsys that tries to > match your suggestions and Bjorn's. Could you please comment check it? I think it would be useful to do this as generic as possible, and you seem to be the right person to integrate it into the generic driver as you have access to the hardware documents. Normally the folks writing the drivers are just guessing based on what little information they get out of manuals or vendor-provided drivers, but I'm sure that not all of that makes sense. Arnd