From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752784AbcI1XZ0 (ORCPT ); Wed, 28 Sep 2016 19:25:26 -0400 Received: from bombadil.infradead.org ([198.137.202.9]:37901 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751257AbcI1XZS (ORCPT ); Wed, 28 Sep 2016 19:25:18 -0400 Date: Wed, 28 Sep 2016 16:25:12 -0700 From: Darren Hart To: hpa@zytor.com, vadimp@mellanox.com, linux-kernel@vger.kernel.org, mingo@kernel.org, tglx@linutronix.de Cc: linux-tip-commits@vger.kernel.org Subject: Re: [tip:x86/platform] x86/platform/mellanox: Introduce support for Mellanox systems platform Message-ID: <20160928232512.GA7630@f23x64.localdomain> References: <1474578822-33805-1-git-send-email-vadimp@mellanox.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.7.0 (2016-08-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Sep 22, 2016 at 01:18:52PM -0700, tip-bot for Vadim Pasternak wrote: > Commit-ID: 58cbbee2391ce3876e6eee80a4f2a7f025859c52 > Gitweb: http://git.kernel.org/tip/58cbbee2391ce3876e6eee80a4f2a7f025859c52 > Author: Vadim Pasternak > AuthorDate: Thu, 22 Sep 2016 21:13:42 +0000 > Committer: Thomas Gleixner > CommitDate: Thu, 22 Sep 2016 22:13:10 +0200 > > x86/platform/mellanox: Introduce support for Mellanox systems platform > > Enable system support for the Mellanox Technologies platform, which > provides support for the next Mellanox basic systems: "msx6710", > "msx6720", "msb7700", "msn2700", "msx1410", "msn2410", "msb7800", > "msn2740", "msn2100" and also various number of derivative systems from > the above basic types. > > The Kconfig controlling compilation of this code is: MLX_PLATFORM > > Signed-off-by: Vadim Pasternak > Cc: jiri@resnulli.us > Cc: gregkh@linuxfoundation.org > Cc: platform-driver-x86@vger.kernel.org > Cc: geert@linux-m68k.org > Cc: linux@roeck-us.net > Cc: akpm@linux-foundation.org > Cc: mchehab@kernel.org > Cc: davem@davemloft.net > Cc: kvalo@codeaurora.org > Link: http://lkml.kernel.org/r/1474578822-33805-1-git-send-email-vadimp@mellanox.com > Signed-off-by: Thomas Gleixner > > --- > MAINTAINERS | 6 + > arch/x86/Kconfig | 12 ++ > arch/x86/platform/Makefile | 1 + > arch/x86/platform/mellanox/Makefile | 1 + > arch/x86/platform/mellanox/mlx-platform.c | 266 ++++++++++++++++++++++++++++++ > 5 files changed, 286 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index a0ce40f..2adc6ef 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -7668,6 +7668,12 @@ W: http://www.mellanox.com > Q: http://patchwork.ozlabs.org/project/netdev/list/ > F: drivers/net/ethernet/mellanox/mlxsw/ > > +MELLANOX PLATFORM DRIVER > +M: Vadim Pasternak > +L: platform-driver-x86@vger.kernel.org > +S: Supported > +F: arch/x86/platform/mellanox/mlx-platform.c This through me as I was trying to reconcile this series with another mellanox platform driver from Vadim to the drivers/platform/x86 tree as this one also came to the platform-driver-x86 mailing list. There are no other entries in MAINTAINERS that use this list without files under drivers/platform/x86. Thomas, HPA, Ingo, do you also use the platform-driver-x86 mailing list for arch/x86/platform? I had assumed this was only for drivers/platform/x86. I can definitely understand how contributors might get confused.... I think I may have confused myself actually :-) My two cents too late, this driver seems like it would better placed in drivers/platform/x86 as it isn't architectural (or SoC specific) in the same way most of the others in arch/x86/platform are, but instead is more akin to the end-product laptop drivers and such in drivers/platform/x86 which build platform data from DMI strings, ACPI HIDs, etc. If this isn't the right distinction between the two similarly named trees ... how would you like to distinguish them? I'll volunteer to get that documented someplace convenient to make it easier to decide what goes where. -- Darren Hart Intel Open Source Technology Center