Re: N900 sleep mode (in 4.5-rc0, if that matters)

Tony Lindgren
2016-02-09T17:24:14Z

* Pavel Machek <pavel@ucw.cz> [160207 13:24]:
> Hi!
>
> > > (I assume I have to insmod and rmmod, right? Because powersave is not
> > > entered if I simply compile-out usb).
> >
> > Depending on what the bootloader does and probably also if
> > USB was used during the booting.. So yeah you may need to modprobe
> > and then rmmod.
> >
> > > Would you have commit ids for those rmmod fixes? It might be good to
> > > push them into stable, and I should try again with them applied...
> >
> > 055555fc459 ("usb: musb: core: Fix handling of the phy notifications")
> > 03e43528ab68 ("usb: musb: Fix unbalanced pm_runtime_enable")
>
> Ok, with that, I can insmod and rmmod. But I still get:
>
> 00001fff 48005020 (fa005020) cm_idlest_per blocking bits: 0007e000

I think the cm_idlest_per is fine.

> ffdffebd 48004a20 (fa004a20) cm_idlest1_core blocking bits: 00200042
> 0000000d 48004a28 (fa004a28) cm_idlest3_core

Bit 21 in cm_idlest1_core is for MCSPI4 so WLAN. Does that go
down if do sleep 5; cat /sys/kernel/debug/pm_debug/count ?

If not, the're PM runtime missing or broken somewhere.

FYI, below is my omap3 usb test script that I use to start and
stop USB, it also works on n900. And after stopping it n900
continues hitting deeper idle states just fine.

That is with these two phy patches applied on v4.5-rc3:

"phy: twl4030-usb: Fix unbalanced pm_runtime_enable on module reload")
"phy: twl4030-usb: Relase usb phy on unload")

Maybe I gave you wrong info on the USB patches needed.. But these
two above are needed on v4.5-rc3. And looks like Kishon has now
applied them.

Oh and I also have the MMC PM runtime regression fix here that I
have not yet posted. Will post today. But I guess you're on
v4.4 right now still.

Regards,

Tony

8< --------
#!/bin/bash

# Change for your UDC controller
phy_module=phy_twl4030_usb
udc_glue=omap2430
udc_module=musb_hdrc
udc_name=musb-hdrc.0.auto

start_usb_gadgets() {
vendor=0x1d6b
product=0x0106
file=$1

mount -t configfs none /sys/kernel/config
mkdir /sys/kernel/config/usb_gadget/g1
old_pwd=$(pwd)
cd /sys/kernel/config/usb_gadget/g1

echo $product > idProduct
echo $vendor > idVendor
mkdir strings/0x409
echo 123456789 > strings/0x409/serialnumber
echo foo > strings/0x409/manufacturer
echo "Multi Gadget" > strings/0x409/product

mkdir configs/c.1
echo 100 > configs/c.1/MaxPower
mkdir configs/c.1/strings/0x409
echo "Config 100mA" > configs/c.1/strings/0x409/configuration

mkdir configs/c.5
echo 500 > configs/c.5/MaxPower
mkdir configs/c.5/strings/0x409
echo "Config 500mA" > configs/c.5/strings/0x409/configuration

mkdir functions/mass_storage.0
echo $file > functions/mass_storage.0/lun.0/file
ln -s functions/mass_storage.0 configs/c.1
ln -s functions/mass_storage.0 configs/c.5

mkdir functions/acm.0
ln -s functions/acm.0 configs/c.1
ln -s functions/acm.0 configs/c.5

mkdir functions/ecm.0
ln -s functions/ecm.0 configs/c.1
ln -s functions/ecm.0 configs/c.5

# Adding rndis seems to cause alignment trap or some
# random oops on reboot after rmmod $udc_glue

echo $udc_name > /sys/kernel/config/usb_gadget/g1/UDC
cd $old_pwd
}

stop_usb_gadgets() {
old_pwd=$(pwd)
cd /sys/kernel/config/usb_gadget/g1

echo "" > /sys/kernel/config/usb_gadget/g1/UDC

rm configs/c.1/ecm.0
rm configs/c.5/ecm.0
rmdir functions/ecm.0

rm configs/c.1/acm.0
rm configs/c.5/acm.0
rmdir functions/acm.0

echo "" > functions/mass_storage.0/lun.0/file
rm configs/c.1/mass_storage.0
rm configs/c.5/mass_storage.0
rmdir functions/mass_storage.0

rmmod $udc_glue
rmmod $phy_module
rmmod $udc_module

cd $old_pwd
}

case $1 in
start)
modprobe libcomposite
modprobe $phy_module
modprobe $udc_glue
start_usb_gadgets ""
;;
stop)
stop_usb_gadgets
;;
*)
;;
esac