From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751371AbdH1OIs (ORCPT ); Mon, 28 Aug 2017 10:08:48 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:38054 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751170AbdH1OIr (ORCPT ); Mon, 28 Aug 2017 10:08:47 -0400 Date: Mon, 28 Aug 2017 16:08:52 +0200 From: Greg Kroah-Hartman To: Hans de Goede Cc: Arnd Bergmann , linux-kernel@vger.kernel.org Subject: Re: [PATCH] MAINTAINERS: Add files under drivers/virt/* to CHAR and MISC DRIVERS section Message-ID: <20170828140852.GA30541@kroah.com> References: <20170825115623.22785-1-hdegoede@redhat.com> <20170825131006.GA14726@kroah.com> <0d43b1d1-1976-dad5-fe45-de854417ef4f@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <0d43b1d1-1976-dad5-fe45-de854417ef4f@redhat.com> User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Aug 25, 2017 at 03:11:25PM +0200, Hans de Goede wrote: > Hi, > > On 25-08-17 15:10, Greg Kroah-Hartman wrote: > > On Fri, Aug 25, 2017 at 01:56:23PM +0200, Hans de Goede wrote: > > > Cc: Greg Kroah-Hartman > > > Cc: Arnd Bergmann > > > Signed-off-by: Hans de Goede > > > --- > > > MAINTAINERS | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > > index 6a349b5c0c2a..db633caa5a1d 100644 > > > --- a/MAINTAINERS > > > +++ b/MAINTAINERS > > > @@ -3284,6 +3284,7 @@ T: git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git > > > S: Supported > > > F: drivers/char/* > > > F: drivers/misc/* > > > +F: drivers/virt/* > > > > Really? I'm now in charge of this directory too? That's news to me... > > This was suggested by Arnd, as he preferred putting the vboxguest driver > under drivers/virt rather then drivers/misc. Note that currently there is > only one .c file under drivers/virt: fsl_hypervisor.c Ok, but how about we wait until there is more than one file in here? And what happened to all of the other "virt" drivers, why are they not here? Perhaps using this directory isn't really a good idea? I don't know... thanks, greg k-h