Re: [PATCH 1/3] usb: gadget: function: phonet: balance usb_ep_disable calls

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

 



On Tue 2015-02-03 13:18:59, Felipe Balbi wrote:
> On Tue, Feb 03, 2015 at 05:17:28PM +0100, Pali Rohár wrote:
> > On Tuesday 03 February 2015 16:43:45 Felipe Balbi wrote:
> > > Hi,
> > > 
> > > On Tue, Feb 03, 2015 at 04:31:51PM +0100, Pali Rohár wrote:
> > > > On Tuesday 03 February 2015 00:15:19 Felipe Balbi wrote:
> > > > > f_phonet's ->set_alt() method will call usb_ep_disable()
> > > > > potentially on an endpoint which is already disabled.
> > > > > That's something the gadget/function driver must
> > > > > guarantee that it's always balanced.
> > > > > 
> > > > > In order to balance the calls, just make sure the endpoint
> > > > > was enabled before by means of checking the validity of
> > > > > driver_data.
> > > > > 
> > > > > Reported-by: Pali Rohár <pali.rohar@xxxxxxxxx>
> > > > > Signed-off-by: Felipe Balbi <balbi@xxxxxx>
> > > > > ---
> > > > 
> > > > Your patches cause that kernel does not print any error
> > > > message to n900 screen anymore and reboot device in 10
> > > > seconds. I did not loaded any external modules.
> > > 
> > > > In qemu I see this crash in early boot:
> > > alright, so n900's working fine. I'll wait until you debug
> > > qemu a little more, thank you
> > 
> > NO! It does not working, see ^^^^. It break n900 totally!
> 
> settle down a bit more. I don't have the HW you have and things are
> working fine on boards I _do_ have, there's not much more I can do to
> help without you doing your homework. Debug a bit more and bring more
> information as to what's going on, until then you're on your own.

I'm not sure what you are smoking, but Pali is doing more then enough
of his homework. No, it is not okay for you to break n900, and it is
not okay for you to break qemu.

In fact, you should do _your_ homework and install n900 qemu now. It
is not Pali's homework to debug your stuff for you.

								Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




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

  Powered by Linux