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. -- balbi
Attachment:
signature.asc
Description: Digital signature
