Received-SPF: pass (google.com: domain of linux-kernel+bounces-26946-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Date: Tue, 16 Jan 2024 13:04:08 +0900
From: Dominique Martinet <asmadeus@codewreck.org>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: Michael Grzeschik <m.grzeschik@pengutronix.de>,
	Eric Van Hensbergen <ericvh@kernel.org>,
	Latchesar Ionkov <lucho@ionkov.net>,
	Christian Schoenebeck <linux_oss@crudebyte.com>,
	Jonathan Corbet <corbet@lwn.net>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	v9fs@lists.linux.dev, linux-doc@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org,
	kernel@pengutronix.de
Subject: Re: [PATCH 1/3] usb: gadget: function: 9pfs
Message-ID: <ZaYAONB-fUB3gjBl@codewreck.org>
References: <20240116-ml-topic-u9p-v1-0-ad8c306f9a4e@pengutronix.de>
 <20240116-ml-topic-u9p-v1-1-ad8c306f9a4e@pengutronix.de>
 <4856923e-3ce8-4372-9451-f9c8aa157111@rowland.harvard.edu>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <4856923e-3ce8-4372-9451-f9c8aa157111@rowland.harvard.edu>

Alan Stern wrote on Mon, Jan 15, 2024 at 10:17:34PM -0500:
> > diff --git a/drivers/usb/gadget/Kconfig b/drivers/usb/gadget/Kconfig
> > index b3592bcb0f966..72cdecaef6aa9 100644
> > --- a/drivers/usb/gadget/Kconfig
> > +++ b/drivers/usb/gadget/Kconfig
> > @@ -153,6 +153,10 @@ config USB_F_ACM
> >  config USB_F_SS_LB
> >  	tristate
> >  
> > +config USB_F_9PFS
> > +	tristate
> > +	select NET_9P
> > +
> >  config USB_U_SERIAL
> >  	tristate
> >  
> > @@ -363,6 +367,13 @@ config USB_CONFIGFS_F_LB_SS
> >  	  test software, like the "usbtest" driver, to put your hardware
> >  	  and its driver through a basic set of functional tests.
> >  
> > +config USB_CONFIGFS_F_9PFS
> > +	bool "9pfs over usb gadget"
> > +	depends on USB_CONFIGFS
> > +	select USB_F_9PFS
> > +	help
> > +	  9pfs support for usb gadget
> 
> This may be a dumb question, but what is the purpose of this CONFIG
> symbol?  It doesn't get used by any of the patches in this series, as
> far as I can see.

USB_F_9PFS cannot be selected directly in menuconfig so this allows
configuring the build option -- that appears to be how the other usb
gadgets are configured so I assume it's done that way for consistency
more than out of necessity (I don't see a problem in making the build
system use USB_CONFIGFS_F_9PFS directly, it'd just be different from the
rest)

-- 
Dominique Martinet | Asmadeus