From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andreas Kemnade Subject: Re: power management problems in ehci-omap Date: Tue, 6 Feb 2018 19:03:47 +0100 Message-ID: <20180206185452.34d01d77@aktux> References: <20180204000335.29812776@kemnade.info> <20180204093831.44322452@kemnade.info> <20180206074201.479405e7@aktux> <20180206160452.GA21573@atomide.com> <20180206175509.4d80e930@aktux> <20180206171737.GB21573@atomide.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; boundary="Sig_/LAojc=+8fasEJgmIGcD22W8"; protocol="application/pgp-signature" Return-path: In-Reply-To: <20180206171737.GB21573-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org> Sender: linux-usb-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Tony Lindgren Cc: Michael Nazzareno Trimarchi , USB list , Linux OMAP Mailing List , Discussions about the Letux Kernel , Roger Quadros List-Id: linux-omap@vger.kernel.org --Sig_/LAojc=+8fasEJgmIGcD22W8 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Tue, 6 Feb 2018 09:17:37 -0800 Tony Lindgren wrote: > * Andreas Kemnade [180206 16:56]: > > On Tue, 6 Feb 2018 08:04:52 -0800 > > Tony Lindgren wrote: > > =20 > > > * Andreas Kemnade [180206 06:42]: =20 > > > > rechecked with a board with really nothing connected there > > > > Same behaviour =20 > > >=20 > > > I've just verified that my test board power consumption goes > > > back to normal after rmmod ehci-omap with v4.15. > > > =20 > > yes, for me too, I initially used a test script which does an > > echo rmmod ehci-omap > >=20 > > without a real > > rmmod ehci-omap =20 >=20 > Ah OK :) >=20 > > It just seems to be consistent with my observations in a fully booted > > system (where many things can play a role). Sorry for that confusion. = =20 >=20 > Try with a minimal set of modules first, then modprobe and rmmod one > at a time until you find the module breaking PM? >=20 Yes, that is basically what I do with this script: https://misc.andi.de1.cc/measure4.sh and https://misc.andi.de1.cc/measure5.sh I start from a bare kernel, check currents, load ehci-omap (I have also debugged many other pm things that way) check currents again.=20 But since my rough observations with a fully loaded system seem to match with the=20 echo rmmod behaviour, I did not notice my mistake. > You probably know this already, but just in case it helps.. >=20 > First idle the UARTs and enable off mode with something like: >=20 > uarts=3D$(find /sys/class/tty/tty[SO]*/device/power/ -type d) > for uart in $uarts; do > echo 3000 > $uart/autosuspend_delay_ms 2>&1 > done >=20 > uarts=3D$(find /sys/class/tty/tty[SO]*/power/ -type d 2>/dev/null) > for uart in $uarts; do > echo enabled > $uart/wakeup 2>&1 > echo auto > $uart/control 2>&1 > done >=20 hmm, this looks a bit like runtime suspend. I mean suspend aka echo mem >/sys/power/state > echo -n 1 > /sys/kernel/debug/pm_debug/enable_off_mode >=20 In earlier times this just caused trouble and a little lower current, maybe it is time to check again. > Then if using omap3, the attached debug hack patch can be used to > see which devices are not idling: >=20 Yes, I will try out. It it about detecting whether the soc itself went into low power mode. But it does not check e.g. whether the phy is put into low power mode correctly. So ehci-usb in soc might be powered off and phy is still on. When I go to suspend, I get this message: "Successfully put all powerdomains to target state" @Nikolaus: Can you check IR at the end of the suspend time in https://misc.andi.de1.cc/measure5.sh the second suspend compared with the first whether the phy (the USB 2233) stays on.=20 Regards, Andreas --Sig_/LAojc=+8fasEJgmIGcD22W8 Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEE7sDbhY5mwNpwYgrAfb1qx03ikyQFAlp57gMACgkQfb1qx03i kyToBQ//aPZPPLsFSsL5feZs/yqq21zzc1Ytnjb57/tYKRClBU25cZ3+bP9/0qlw JFnIZYadX18ySaxt39JoYRKwg/YjDishEjhas86GT6J7k2dXsmV4qJpjZeYdLeTs gsE2SLeO4lBbYDIPM1oldhn0IyVocFWUkn9Lqvf109NMGs19AsIRzI7IY/RxpHJk FD5YSUPeRean0AqyWnEM1TvsWlXsTMxUw8eqUxcfx1XWGH/4Dii9FzhCeCvXoRyh MzBtUH4Qfa2EJh8Tc6OM6ntcOtTu0IxAOp48txpOi49xXbxZBITCGyBwuibuC4cM +hTozqP+GUQyx7RaV3tnkJ6CyDUy1Ok5Sf8DEuVHiOJobwOeae6SBDgx2MgYpISX oWtZNBf15lVNToou1P1lHYNYd1bnSXBKPt58kyVmDbeiCH7KbkMPbOZsI7AQ4/6r N2rRJzp+EQji/UmIOMzcP5uwyrjvwmKapdpwhGcofC8thD/tKKisPINxIQPOFO/e +8py/qe+Wkc56qZHrkf6Pz995eF1nA7SkyrXfI6EN0mP4csxFJXatY76G2xrmFwE tT1aWAnBAozKa6D/bOJaBww8z46M3WCzyAWvRrlP40K20biLADXjt0cRigZxh5yg BHK6jj7XpXeLRC7pjZVnR4ktKAEWHzfRB3Zz0UPSXq0qAFmf3vk= =4wI3 -----END PGP SIGNATURE----- --Sig_/LAojc=+8fasEJgmIGcD22W8-- -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: power management problems in ehci-omap From: Andreas Kemnade Message-Id: <20180206185452.34d01d77@aktux> Date: Tue, 6 Feb 2018 19:03:47 +0100 To: Tony Lindgren Cc: Michael Nazzareno Trimarchi , USB list , Linux OMAP Mailing List , Discussions about the Letux Kernel , Roger Quadros List-ID: T24gVHVlLCA2IEZlYiAyMDE4IDA5OjE3OjM3IC0wODAwClRvbnkgTGluZGdyZW4gPHRvbnlAYXRv bWlkZS5jb20+IHdyb3RlOgoKPiAqIEFuZHJlYXMgS2VtbmFkZSA8YW5kcmVhc0BrZW1uYWRlLmlu Zm8+IFsxODAyMDYgMTY6NTZdOgo+ID4gT24gVHVlLCA2IEZlYiAyMDE4IDA4OjA0OjUyIC0wODAw Cj4gPiBUb255IExpbmRncmVuIDx0b255QGF0b21pZGUuY29tPiB3cm90ZToKPiA+ICAgCj4gPiA+ ICogQW5kcmVhcyBLZW1uYWRlIDxhbmRyZWFzQGtlbW5hZGUuaW5mbz4gWzE4MDIwNiAwNjo0Ml06 ICAKPiA+ID4gPiByZWNoZWNrZWQgd2l0aCBhIGJvYXJkIHdpdGggcmVhbGx5IG5vdGhpbmcgY29u bmVjdGVkIHRoZXJlCj4gPiA+ID4gU2FtZSBiZWhhdmlvdXIgICAgCj4gPiA+IAo+ID4gPiBJJ3Zl IGp1c3QgdmVyaWZpZWQgdGhhdCBteSB0ZXN0IGJvYXJkIHBvd2VyIGNvbnN1bXB0aW9uIGdvZXMK PiA+ID4gYmFjayB0byBub3JtYWwgYWZ0ZXIgcm1tb2QgZWhjaS1vbWFwIHdpdGggdjQuMTUuCj4g PiA+ICAgCj4gPiB5ZXMsIGZvciBtZSB0b28sIEkgaW5pdGlhbGx5IHVzZWQgYSB0ZXN0IHNjcmlw dCB3aGljaCBkb2VzIGFuCj4gPiBlY2hvIHJtbW9kIGVoY2ktb21hcAo+ID4gCj4gPiB3aXRob3V0 IGEgcmVhbAo+ID4gcm1tb2QgZWhjaS1vbWFwICAKPiAKPiBBaCBPSyA6KQo+IAo+ID4gSXQganVz dCBzZWVtcyB0byBiZSBjb25zaXN0ZW50IHdpdGggbXkgb2JzZXJ2YXRpb25zIGluIGEgZnVsbHkg Ym9vdGVkCj4gPiBzeXN0ZW0gKHdoZXJlIG1hbnkgdGhpbmdzIGNhbiBwbGF5IGEgcm9sZSkuIFNv cnJ5IGZvciB0aGF0IGNvbmZ1c2lvbi4gIAo+IAo+IFRyeSB3aXRoIGEgbWluaW1hbCBzZXQgb2Yg bW9kdWxlcyBmaXJzdCwgdGhlbiBtb2Rwcm9iZSBhbmQgcm1tb2Qgb25lCj4gYXQgYSB0aW1lIHVu dGlsIHlvdSBmaW5kIHRoZSBtb2R1bGUgYnJlYWtpbmcgUE0/Cj4gClllcywgdGhhdCBpcyBiYXNp Y2FsbHkgd2hhdCBJIGRvIHdpdGggdGhpcyBzY3JpcHQ6CgpodHRwczovL21pc2MuYW5kaS5kZTEu Y2MvbWVhc3VyZTQuc2gKCmFuZAoKaHR0cHM6Ly9taXNjLmFuZGkuZGUxLmNjL21lYXN1cmU1LnNo CgpJIHN0YXJ0IGZyb20gYSBiYXJlIGtlcm5lbCwgY2hlY2sgY3VycmVudHMsIGxvYWQgZWhjaS1v bWFwIChJIGhhdmUgYWxzbwpkZWJ1Z2dlZCBtYW55IG90aGVyIHBtIHRoaW5ncyB0aGF0IHdheSkK Y2hlY2sgY3VycmVudHMgYWdhaW4uIApCdXQgc2luY2UgbXkgcm91Z2ggb2JzZXJ2YXRpb25zIHdp dGggYSBmdWxseSBsb2FkZWQgc3lzdGVtCnNlZW0gdG8gbWF0Y2ggd2l0aCB0aGUgCmVjaG8gcm1t b2QgYmVoYXZpb3VyLCBJIGRpZCBub3Qgbm90aWNlIG15IG1pc3Rha2UuCgo+IFlvdSBwcm9iYWJs eSBrbm93IHRoaXMgYWxyZWFkeSwgYnV0IGp1c3QgaW4gY2FzZSBpdCBoZWxwcy4uCj4gCj4gRmly c3QgaWRsZSB0aGUgVUFSVHMgYW5kIGVuYWJsZSBvZmYgbW9kZSB3aXRoIHNvbWV0aGluZyBsaWtl Ogo+IAo+IHVhcnRzPSQoZmluZCAvc3lzL2NsYXNzL3R0eS90dHlbU09dKi9kZXZpY2UvcG93ZXIv IC10eXBlIGQpCj4gZm9yIHVhcnQgaW4gJHVhcnRzOyBkbwo+IAllY2hvIDMwMDAgPiAkdWFydC9h dXRvc3VzcGVuZF9kZWxheV9tcyAyPiYxCj4gZG9uZQo+IAo+IHVhcnRzPSQoZmluZCAvc3lzL2Ns YXNzL3R0eS90dHlbU09dKi9wb3dlci8gLXR5cGUgZCAyPi9kZXYvbnVsbCkKPiBmb3IgdWFydCBp biAkdWFydHM7IGRvCj4gCWVjaG8gZW5hYmxlZCA+ICR1YXJ0L3dha2V1cCAyPiYxCj4gCWVjaG8g YXV0byA+ICR1YXJ0L2NvbnRyb2wgMj4mMQo+IGRvbmUKPiAKCmhtbSwgdGhpcyBsb29rcyBhIGJp dCBsaWtlIHJ1bnRpbWUgc3VzcGVuZC4KCkkgbWVhbiBzdXNwZW5kIGFrYSBlY2hvIG1lbSA+L3N5 cy9wb3dlci9zdGF0ZQoKPiBlY2hvIC1uIDEgPiAvc3lzL2tlcm5lbC9kZWJ1Zy9wbV9kZWJ1Zy9l bmFibGVfb2ZmX21vZGUKPiAKSW4gZWFybGllciB0aW1lcyB0aGlzIGp1c3QgY2F1c2VkIHRyb3Vi bGUgYW5kIGEgbGl0dGxlIGxvd2VyIGN1cnJlbnQsCm1heWJlIGl0IGlzIHRpbWUgdG8gY2hlY2sg YWdhaW4uCgo+IFRoZW4gaWYgdXNpbmcgb21hcDMsIHRoZSBhdHRhY2hlZCBkZWJ1ZyBoYWNrIHBh dGNoIGNhbiBiZSB1c2VkIHRvCj4gc2VlIHdoaWNoIGRldmljZXMgYXJlIG5vdCBpZGxpbmc6Cj4g ClllcywgSSB3aWxsIHRyeSBvdXQuIEl0IGl0IGFib3V0IGRldGVjdGluZyB3aGV0aGVyIHRoZSBz b2MgaXRzZWxmIHdlbnQKaW50byBsb3cgcG93ZXIgbW9kZS4gQnV0IGl0IGRvZXMgbm90IGNoZWNr IGUuZy4gd2hldGhlciB0aGUgcGh5IGlzIHB1dAppbnRvIGxvdyBwb3dlciBtb2RlIGNvcnJlY3Rs eS4gU28gZWhjaS11c2IgaW4gc29jIG1pZ2h0IGJlIHBvd2VyZWQgb2ZmCmFuZCBwaHkgaXMgc3Rp bGwgb24uCgpXaGVuIEkgZ28gdG8gc3VzcGVuZCwgSSBnZXQgdGhpcyBtZXNzYWdlOgoiU3VjY2Vz c2Z1bGx5IHB1dCBhbGwgcG93ZXJkb21haW5zIHRvIHRhcmdldCBzdGF0ZSIKCkBOaWtvbGF1czog Q2FuIHlvdSBjaGVjayBJUiBhdCB0aGUgZW5kIG9mIHRoZSBzdXNwZW5kIHRpbWUgaW4KaHR0cHM6 Ly9taXNjLmFuZGkuZGUxLmNjL21lYXN1cmU1LnNoCnRoZSBzZWNvbmQgc3VzcGVuZCBjb21wYXJl ZCB3aXRoIHRoZSBmaXJzdCB3aGV0aGVyIHRoZSBwaHkgKHRoZSBVU0IKMjIzMykgc3RheXMgb24u IAoKUmVnYXJkcywKQW5kcmVhcwo=