Re: [PATCH] usb: testusb: Fix for showing the connection speed

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Sep 02, 2021 at 10:42:52AM +0200, Greg KH wrote:
> On Thu, Sep 02, 2021 at 02:03:15PM +0530, Faizel K B wrote:
> > On Wed, Sep 01, 2021 at 08:24:08PM +0200, Greg KH wrote:
> > > On Wed, Sep 01, 2021 at 11:43:51PM +0530, Faizel K B wrote:
> > > > Signed-off-by: Faizel K B <faizel.kb@xxxxxxxxxxxxx>
> > > > ---
> > > >  tools/usb/testusb.c | 14 ++++++++------
> > > >  1 file changed, 8 insertions(+), 6 deletions(-)
> > > > 
> > > > diff --git a/tools/usb/testusb.c b/tools/usb/testusb.c
> > > > index ee8208b2f946..69c3ead25313 100644
> > > > --- a/tools/usb/testusb.c
> > > > +++ b/tools/usb/testusb.c
> > > > @@ -265,12 +265,6 @@ static int find_testdev(const char *name, const struct stat *sb, int flag)
> > > >  	}
> > > >  
> > > >  	entry->ifnum = ifnum;
> > > > -
> > > > -	/* FIXME update USBDEVFS_CONNECTINFO so it tells about high speed etc */
> > > > -
> > > > -	fprintf(stderr, "%s speed\t%s\t%u\n",
> > > > -		speed(entry->speed), entry->name, entry->ifnum);
> > > > -
> > > >  	entry->next = testdevs;
> > > >  	testdevs = entry;
> > > >  	return 0;
> > > > @@ -299,6 +293,14 @@ static void *handle_testdev (void *arg)
> > > >  		return 0;
> > > >  	}
> > > >  
> > > > +	status  =  ioctl(fd, USBDEVFS_GET_SPEED, NULL);
> > > > +	if (status < 0)
> > > > +		fprintf(stderr, "USBDEVFS_GET_SPEED failed %d\n", status);
> > > > +	else
> > > > +		dev->speed = status;
> > > > +	fprintf(stderr, "%s speed\t%s\t%u\n",
> > > > +			speed(dev->speed), dev->name, dev->ifnum);
> > > > +
> > > >  restart:
> > > >  	for (i = 0; i < TEST_CASES; i++) {
> > > >  		if (dev->test != -1 && dev->test != i)
> > > > -- 
> > > > 2.25.1
> > > > 
> > > 
> > > Hi,
> > > 
> > > This is the friendly patch-bot of Greg Kroah-Hartman.  You have sent him
> > > a patch that has triggered this response.  He used to manually respond
> > > to these common problems, but in order to save his sanity (he kept
> > > writing the same thing over and over, yet to different people), I was
> > > created.  Hopefully you will not take offence and will fix the problem
> > > in your patch and resubmit it so that it can be accepted into the Linux
> > > kernel tree.
> > > 
> > > You are receiving this message because of the following common error(s)
> > > as indicated below:
> > > 
> > > - You did not specify a description of why the patch is needed, or
> > >   possibly, any description at all, in the email body.  Please read the
> > >   section entitled "The canonical patch format" in the kernel file,
> > >   Documentation/SubmittingPatches for what is needed in order to
> > >   properly describe the change.
> > > 
> > > - This looks like a new version of a previously submitted patch, but you
> > >   did not list below the --- line any changes from the previous version.
> > >   Please read the section entitled "The canonical patch format" in the
> > >   kernel file, Documentation/SubmittingPatches for what needs to be done
> > >   here to properly describe this.
> > > 
> > > If you wish to discuss this problem further, or you have questions about
> > > how to resolve this issue, please feel free to respond to this email and
> > > Greg will reply once he has dug out from the pending patches received
> > > from other developers.
> > > 
> > > thanks,
> > > 
> > > greg k-h's patch email bot
> > Is it expected to be a separate patch (version 2)? Since the change is
> > only in the updated 'real name' for the 'from' field and 'Signed Off' field
> > in response with the previous message from the bot.
> 
> Yes, of course, how else would we know this is a newer patch?
> 
> thanks,
> 
> greg k-h

Ok. Will send it as new version.




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux