[Buildroot] [PATCH 1/1] package/gadgetfs-test: add an upstream URL to Config.in

[Buildroot] [PATCH 1/1] package/gadgetfs-test: add an upstream URL to Config.in

[Mark Corbin]

Add an upstream URL to the help text in Config.in. This
addresses the 'Missing' URL status in the package stats
web page output.

Signed-off-by: Mark Corbin <mark@dibsco.co.uk>
---
 package/gadgetfs-test/Config.in | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/package/gadgetfs-test/Config.in b/package/gadgetfs-test/Config.in
index ca14c91b41..c22134c4a5 100644
--- a/package/gadgetfs-test/Config.in
+++ b/package/gadgetfs-test/Config.in
@@ -4,6 +4,8 @@ config BR2_PACKAGE_GADGETFS_TEST
 	help
 	  Test program for gadgetfs from linux-usb.org
 
+	  https://mirror.egtvedt.no/avr32linux.org/twiki/pub/Main/GadgetFsTest/
+
 if BR2_PACKAGE_GADGETFS_TEST
 
 config BR2_PACKAGE_GADGETFS_TEST_USE_AIO
-- 
2.20.1

[Buildroot] [PATCH 1/1] package/gadgetfs-test: add an upstream URL to Config.in

[Thomas Petazzoni]

On Tue, 26 Nov 2019 15:35:05 +0000
Mark Corbin <mark@dibsco.co.uk> wrote:

> Add an upstream URL to the help text in Config.in. This
> addresses the 'Missing' URL status in the package stats
> web page output.
> 
> Signed-off-by: Mark Corbin <mark@dibsco.co.uk>
> ---
>  package/gadgetfs-test/Config.in | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/package/gadgetfs-test/Config.in b/package/gadgetfs-test/Config.in
> index ca14c91b41..c22134c4a5 100644
> --- a/package/gadgetfs-test/Config.in
> +++ b/package/gadgetfs-test/Config.in
> @@ -4,6 +4,8 @@ config BR2_PACKAGE_GADGETFS_TEST
>  	help
>  	  Test program for gadgetfs from linux-usb.org
>  
> +	  https://mirror.egtvedt.no/avr32linux.org/twiki/pub/Main/GadgetFsTest/

Same here, I don't think this adds much value.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [PATCH 1/1] package/gadgetfs-test: add an upstream URL to Config.in

[Thomas Petazzoni]

On Tue, 26 Nov 2019 16:59:56 +0100
Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote:

> On Tue, 26 Nov 2019 15:35:05 +0000
> Mark Corbin <mark@dibsco.co.uk> wrote:
> 
> > Add an upstream URL to the help text in Config.in. This
> > addresses the 'Missing' URL status in the package stats
> > web page output.
> > 
> > Signed-off-by: Mark Corbin <mark@dibsco.co.uk>
> > ---
> >  package/gadgetfs-test/Config.in | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/package/gadgetfs-test/Config.in b/package/gadgetfs-test/Config.in
> > index ca14c91b41..c22134c4a5 100644
> > --- a/package/gadgetfs-test/Config.in
> > +++ b/package/gadgetfs-test/Config.in
> > @@ -4,6 +4,8 @@ config BR2_PACKAGE_GADGETFS_TEST
> >  	help
> >  	  Test program for gadgetfs from linux-usb.org
> >  
> > +	  https://mirror.egtvedt.no/avr32linux.org/twiki/pub/Main/GadgetFsTest/  
> 
> Same here, I don't think this adds much value.

To me, this package is another candidate for removal. This is some
really old unmaintained stuff, I'm pretty sure there are much better
things these days to test USB gadget.

Peter ?

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [PATCH 1/1] package/gadgetfs-test: add an upstream URL to Config.in

[Peter Korsgaard]

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@bootlin.com> writes:

Hi,

 >> >  	  Test program for gadgetfs from linux-usb.org
 >> >  
 >> > +	  https://mirror.egtvedt.no/avr32linux.org/twiki/pub/Main/GadgetFsTest/  
 >> 
 >> Same here, I don't think this adds much value.

 > To me, this package is another candidate for removal. This is some
 > really old unmaintained stuff, I'm pretty sure there are much better
 > things these days to test USB gadget.

 > Peter ?

I believe so as well. The kernel has some example code under
tools/usb. Gadgetfs is obsoleted by functionfs (2010), which works with
the composite framework. The user space API to setup a composite device
without a custom kernel driver using configfs is somewhat newer though
(3.11). We have the libusbgx package to handle that.

It may make sense to package https://github.com/kopasiak/gt, which is a
command line tool built on top of libusbgx.

-- 
Bye, Peter Korsgaard