Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753429AbbEZOwt (ORCPT ); Tue, 26 May 2015 10:52:49 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:41852 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751929AbbEZOwr (ORCPT ); Tue, 26 May 2015 10:52:47 -0400 Date: Tue, 26 May 2015 09:50:43 -0500 From: Felipe Balbi To: "Lu, Baolu" CC: David Cohen , Heikki Krogerus , Greg Kroah-Hartman , , Subject: Re: [PATCH v3 1/1] usb: ulpi: ulpi_init should be executed in subsys_initcall Message-ID: <20150526145043.GC26599@saruman.tx.rr.com> Reply-To: References: <1432294155-18039-1-git-send-email-baolu.lu@linux.intel.com> <20150522160845.GA31627@psi-dev26.jf.intel.com> <5562C000.8010302@linux.intel.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="uXxzq0nDebZQVNAZ" Content-Disposition: inline In-Reply-To: <5562C000.8010302@linux.intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2298 Lines: 63 --uXxzq0nDebZQVNAZ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Mon, May 25, 2015 at 02:24:00PM +0800, Lu, Baolu wrote: >=20 >=20 > On 05/23/2015 12:08 AM, David Cohen wrote: > >Hi, > > > >On Fri, May 22, 2015 at 07:29:15PM +0800, Lu Baolu wrote: > >>Phy drivers and the ulpi interface providers depend on the > >>registeration of the ulpi bus. Ulpi registers the bus in > >>module_init(). This could result in a load order issue, i.e. > >It's still not an issue :( > >I'd say "unnecessary probe delays". >=20 > I managed to boot a kernel built from the top of Felipe's > remotes/origin/next branch under an Ubuntu environment > on Intel's Bay Trail tablet. >=20 > The same panic (as I found in the Android environment previously) > shows up as well. And if I replace module_init() with sys_initcall(), > the panic disappears. the problem is something else... Moving things around in the init levels is just a workaround for another issue. Seems like there's some missing EPROBE_DEFER somewhere. --=20 balbi --uXxzq0nDebZQVNAZ Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJVZIhDAAoJEIaOsuA1yqREzjEQAKFO7fHSmno7xZmT7X5PJs2T /s9eFIKF3M9mtomZ8ucTUBRCVj7CLEGyw/0MspbBctDHj0LqCIr2i2lT+ltKBIds M3tysgH07GWFy2ABTamoCZgmPxWIUh0FN7t1Y734rNKBDHdcczDLuVwQ7yiEJsvP /QAwaFXHvqpODatqocGixFdO+PHP6THvm/6w4mzi4vf+GP0yCTdG0eq1dX6Mr8Xv CYmR2r8CZc6aQJslSuC3YDXiIVAMs5XedFgtupryeiAye7FdwXEBn40BG3olSrWf bCOi22eEYHP8Wi+6k9DBQQSYIl8lxXvAxTJor07BZDAiYzEF6N2VcU2IelOBUDL6 SSzdeORHcWp10PFZGC0hoJgrEDXi1lElfrCqpPlm59nhNwQT9sm2T2egtmJjjYxG qPFZzGBU/lYHwOCjvhSerJc6vzjz+dz9vMAGHtAZkBRr9dpNKm9G0SR+NYx1vAxi v+ML9nE8apax2pzq/jbjGPzUMHoRrGPh29qzWltIHEBWAaydMS2VWcQ6qwlTa9Mo pD67Oo8ATfXc+WADoM6aiCeiXL5Bny/Lai6YXQAS96ehwVcSlsqLXZD5VS0npbQw GViUlZXm7+oRHEDNgz1xHKOQ3Ob8lTuqGAEGIajrdUWLUhkMKLhl1lEi1F5JdwXg kOMPBgyRhgOaSx7X4K+R =+6Ro -----END PGP SIGNATURE----- --uXxzq0nDebZQVNAZ-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/