Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752027AbcJZFJs (ORCPT ); Wed, 26 Oct 2016 01:09:48 -0400 Received: from mail-db5eur01on0050.outbound.protection.outlook.com ([104.47.2.50]:16328 "EHLO EUR01-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751259AbcJZFJr (ORCPT ); Wed, 26 Oct 2016 01:09:47 -0400 X-Greylist: delayed 48347 seconds by postgrey-1.27 at vger.kernel.org; Wed, 26 Oct 2016 01:09:46 EDT From: Stuart Yoder To: Alexander Graf , "gregkh@linuxfoundation.org" CC: German Rivera , "devel@driverdev.osuosl.org" , "linux-kernel@vger.kernel.org" , "arnd@arndb.de" , Leo Li Subject: RE: [PATCH 0/9] staging: fsl-mc: move bus driver out of staging, add dpio Thread-Topic: [PATCH 0/9] staging: fsl-mc: move bus driver out of staging, add dpio Thread-Index: AQHSK6SDMbgkZ31CaUmyp/yko3J4kqC3sA8AgAAQrNA= Date: Wed, 26 Oct 2016 02:35:57 +0000 Message-ID: References: <1477058509-12547-1-git-send-email-stuart.yoder@nxp.com> <580E1BDE.6050302@suse.de> In-Reply-To: <580E1BDE.6050302@suse.de> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=stuart.yoder@nxp.com; x-originating-ip: [162.203.173.167] x-ms-office365-filtering-correlation-id: 9a3dac0f-1132-491c-348b-08d3fd48d0f5 x-microsoft-exchange-diagnostics: 1;AM2PR04MB0897;7:gAgEZIz5Gfjo0p4FetDLXUeuPcbzG4to0NhG+pSg39bKo/dfym9D3A0X05GzQ48B7wnMgQGGGInzgjxGMGgTnDp4WA3Vnjxc4oD5vsDh4Qvvp3FIUS9w1D2i3Ad0/KqEbS6hr3akSmMZCtN4E2Wlvysqv7NNdeLmEnvGzc66Yctmmcwu1hoOy/O7stFoZ4Bfd1mtwYR/7uTXFvwn5EZEsKXyj5xolUkefJGxfN2Sue4UguU/A/+JjhakuGpKamr9HyyrfoglV8djyeVQiIVI8lSBypni9aGkYYHV1eSJ0DwiTvT3v8niG2KBOWtnxq37rNGdCREKORjNOwSmGQAVfdyKf9aRuJsx55G8lPtKrG8= x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:AM2PR04MB0897; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(9452136761055)(185117386973197); x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(6040176)(6045068)(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001)(6055026)(6046068)(6072074);SRVR:AM2PR04MB0897;BCL:0;PCL:0;RULEID:;SRVR:AM2PR04MB0897; x-forefront-prvs: 0107098B6C x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(6009001)(7916002)(13464003)(24454002)(189002)(377454003)(199003)(106116001)(101416001)(2900100001)(81156014)(81166006)(8676002)(33656002)(122556002)(10400500002)(87936001)(77096005)(3660700001)(76576001)(4326007)(2501003)(3280700002)(7736002)(7846002)(305945005)(2906002)(6116002)(3846002)(102836003)(54356999)(2950100002)(74316002)(68736007)(66066001)(7696004)(86362001)(76176999)(50986999)(5660300001)(9686002)(92566002)(5001770100001)(97736004)(8936002)(19580405001)(5002640100001)(586003)(106356001)(105586002)(189998001)(19580395003);DIR:OUT;SFP:1101;SCL:1;SRVR:AM2PR04MB0897;H:VI1PR0401MB2638.eurprd04.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-originalarrivaltime: 26 Oct 2016 02:35:57.1752 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM2PR04MB0897 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id u9Q59uV9027020 Content-Length: 4974 Lines: 106 > -----Original Message----- > From: Alexander Graf [mailto:agraf@suse.de] > Sent: Monday, October 24, 2016 9:34 AM > To: Stuart Yoder ; gregkh@linuxfoundation.org > Cc: German Rivera ; devel@driverdev.osuosl.org; linux-kernel@vger.kernel.org; > arnd@arndb.de; Leo Li > Subject: Re: [PATCH 0/9] staging: fsl-mc: move bus driver out of staging, add dpio > > Hi Stuart, > > On 10/21/2016 04:01 PM, Stuart Yoder wrote: > > This patch series: A) addresses the final item in the staging > > TODO list for the fsl-mc bus driver-- adding a functional driver > > on top of the bus driver, and B) requests that the fsl-mc bus driver > > be moved out of staging. > > Awesome, it's great to see progress again! :) > > > The proposed destination for the bus driver is drivers/bus. > > Proposed location for global header files for fsl-mc and dpaa2 > > is include/linux/fsl. > > > > The functional driver added is for the DPIO object which provides > > queuing services for other DPAA2 drivers. An overview of the > > I thought the idea of the TODO item was to have a full-fledged user of > the bus, like a full network driver. The TODO item reads: > > > -* Add at least one device driver for a DPAA2 object (child device of the > > - fsl-mc bus). Most likely candidate for this is adding DPAA2 Ethernet > > - driver support, which depends on drivers for several objects: DPNI, > > - DPIO, DPMAC. Other pre-requisites include: DPIO is a "full fleged user" of the bus. But, yes, it does provide infrastructure services and so does not have a standalone I/O function. > which to me indicates that DPIO is only part of that goal. Of course I'm > the last person blocking progress to move the driver out of staging. But > are we at the right point yet? I thought the goal was to demonstrate a driver on top of the fsl-mc bus driver because without that it would have been difficult to validate/review that the bus infrastructure was correct. The DPIO driver demonstrates full use of the bus driver infrastructure-- getting probed, discovering and mapping mmio regions, initializing the device, initializing interrupts. > To me the topmost important bit of having this outside of staging is > actually missing in the TODO list (probably since it's obvious): Have > stable, reliable, responsible maintainership for the code. > > So far I've seen German do the initial push upstream, then there was > silence for a while. Now some time passed and you push a few bits here > and there again. All of the efforts are great and very appreciated, but > I'm missing the "maintainer" figure. Some peer to German and you who > oversees the whole thing, reviews your patches and devotes at least 2-3 > days a week to only upstream fsl-mc work. Someone like York for U-Boot > or Scott for general Linux work. > > Without that, there's too much of a chance that the code will stay > incomplete, bitrot, etc. And that'd be bad for everyone involved. I > think the concept behind fsl-mc is great and exactly what people need, > so we should make sure it succeeds. I agree we need that. We are actively working on getting an additional maintainer (or two), and until we can get the right person(s) I'm willing to fill that role. We're not going to let this code bitrot. I actually think getting the bus driver out of staging will help spur broader involvment by NXP engineers in the fsl-mc bus support. There are enhancements like a resource management interface for user space, an interface to see the MC log buffer, SMMU-related hooks for the fsl-mc bus, and vfio for the fsl-mc bus. All that stuff is on hold until we get the bus driver out of staging. The directive we have is to add no new features until the bus driver is out. For example, the ARM SMMU driver has an include of , but I don't see the SMMU maintainers accepting the following in arm-smmu.c: #include <../drivers/staging/fsl-mc/include/mc.h> Given that the fsl-mc bus TODO list is done, there is not a whole lot for a new maintainer to do to the bus driver itself until we get the driver out of staging (aside from reviewing another DPAA2 object driver that would also go into staging). Once the bus driver + dpio is out staging it also opens up the door for other DPAA2 drivers-- network, crypto, DMA, L2 switch, decompression/compression, and others to be upstreamed. I didn't think we wanted all of those to go into staging, but we were waiting until some 1 driver was accepted first, proving the bus infrastructure is sound. I was hoping DPI could be that proof of concept. So, in short, I think getting the bus driver and DPIO out of staging will open some parallel development and will also provide more opportunities for some new maintainers to get involved, because there will be more to review and do. However, if you want things to stay in staging for now, I will resubmit and put DPIO there. Thanks, Stuart