From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755713AbdETGn4 (ORCPT ); Sat, 20 May 2017 02:43:56 -0400 Received: from foss.arm.com ([217.140.101.70]:51214 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751818AbdETGnz (ORCPT ); Sat, 20 May 2017 02:43:55 -0400 From: Marc Zyngier To: Matthias Brugger Cc: laurentiu.tudor@nxp.com, gregkh@linuxfoundation.org, stuyoder@gmail.com, devel@driverdev.osuosl.org, arnd@arndb.de, ruxandra.radulescu@nxp.com, Stuart Yoder , roy.pledge@nxp.com, linux-kernel@vger.kernel.org, agraf@suse.de, catalin.horghidan@nxp.com, ioana.ciornei@nxp.com, Thomas Gleixner , leoyang.li@nxp.com, bharat.bhushan@nxp.com, Jason Cooper , linux-arm-kernel@lists.infradead.org, Rob Herring Subject: Re: [PATCH 3/3][v4] staging: fsl-mc: move bus driver out of staging In-Reply-To: (Matthias Brugger's message of "Fri, 19 May 2017 15:41:43 +0200") Organization: ARM Ltd References: <20170519131305.5068-1-laurentiu.tudor@nxp.com> <20170519131305.5068-4-laurentiu.tudor@nxp.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) Date: Sat, 20 May 2017 07:43:15 +0100 Message-ID: <86tw4g6nb0.fsf@arm.com> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, May 19 2017 at 02:41:43 PM, Matthias Brugger wrote: > On 19/05/17 15:13, laurentiu.tudor@nxp.com wrote: >> From: Stuart Yoder >> >> Move the source files out of staging into their final locations: >> -include files in drivers/staging/fsl-mc/include go to include/linux/fsl >> -irq-gic-v3-its-fsl-mc-msi.c goes to drivers/irqchip > > This driver has as compatible "arm,gic-v3-its". I wonder if this is > correct and if it should be moved like this out of staging. This is no different from the way we handle *any* bus that uses the GICv3 ITS as an MSI controller. Each bus provides its glue code that latches onto the ITS node, and calls into the generic code. Now, when it comes to moving this out of staging, here is my concern: There is mention of a userspace tool (restool) used to control the HW. Where is this tool? Where is the user ABI documented? Thanks, M. -- Jazz is not dead. It just smells funny.