From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752867AbdEVHo1 (ORCPT ); Mon, 22 May 2017 03:44:27 -0400 Received: from mail.free-electrons.com ([62.4.15.54]:59101 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752580AbdEVHo0 (ORCPT ); Mon, 22 May 2017 03:44:26 -0400 Date: Mon, 22 May 2017 09:44:14 +0200 From: Maxime Ripard To: Kevin Hilman Cc: Rask Ingemann Lambertsen , Chen-Yu Tsai , Russell King , linux-arm-kernel , lkml Subject: Re: [PATCH 4/4] ARM: multi_v7_defconfig: Switch AXP20x driver from module to built-in Message-ID: <20170522074414.3qz2wpw4cs2btkhv@flea.lan> References: <937914a68e4e7d6db2d12540cc6253b200978e2d.1486590875.git.rask@formelder.dk> <49ad3e66452bd78da7b9e581129f76633c56e609.1486590875.git.rask@formelder.dk> <20170210084213.uzf25soh7bl73a6g@lukather> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="f2spcziqu4xh5xpc" Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170428 (1.8.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --f2spcziqu4xh5xpc Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi Kevin, On Thu, May 18, 2017 at 11:59:50AM -0700, Kevin Hilman wrote: > On Fri, Mar 17, 2017 at 10:39 AM, Kevin Hilman wro= te: > > On Fri, Feb 10, 2017 at 12:42 AM, Maxime Ripard > > wrote: > >> On Wed, Feb 08, 2017 at 11:09:31PM +0100, Rask Ingemann Lambertsen wro= te: > >>> The AXP20X regulator support is currently built as a module, which me= ans > >>> it's not available until the root fs has been mounted, but the boot l= oader > >>> might not have enabled the required regulators, so build their drivers > >>> into the kernel. > >>> > >>> Signed-off-by: Rask Ingemann Lambertsen > >> > >> Queued for 4.12. > > > > Hello, kernelci.org is reporting boot failures on sun5i-r8-chip in > > linux-next[1] for a few days and with a variety of defconfigs. I > > bisected it[2] down to this patch. > > > > I verified that reverting this patch on top of next-20170310 makes my > > chip board boot again. >=20 > FYI... this board is still broken in linux-next (and now in mainline), > and reverting $SUBJECT patch still makes it work. >=20 > Is nobody else using mainline on this board? I thought about that during the weekend, and it might just be a symptom. The CHIP has brown out issues, especially when you enable the WiFi chip, which should happen around the time of the failure when the PMIC regulator support is compiled as a module. We mitigate that in upstream's U-Boot by enabling the two regulators for the WiFi chip in U-boot, which levels a bit the current over the boot. You have a few ways to prevent that from happening. Having a better power supply / cable will help, I'm not sure how reasonable that is. Another thing that can work is, if your USB plugs can take it, to increase the overcurrent trigger in the PMIC, ideally in U-Boot. The last, and probably cleaner one, would be to just power it through the 5v input on its header, and not the USB. There's not current limitation there, so it shouldn't cause any problems anymore. Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com --f2spcziqu4xh5xpc Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIcBAEBAgAGBQJZIpbOAAoJEBx+YmzsjxAgKwYP/R0UfB/3C4hb6BLRmKjPv2wO ahAXL26VWiXsRL1Ysio2WkzdyHiykU43xNfcNpwiuRTpQFfW9Iixlrhsc9Uy3G4P pTuoUvLOh478elEJ6UjDTI0qE198qbKPFVEQA8rSnmDmx31pWTcXpg3/1EDzBcfn feIx09wPXUMkK9dr3aF1tWhmQEc0OE5wTSWF4UFAhZVaTctgUAHY1c1W+p1vxTMO DJxnczRGALS+9I7wsVLAeRw7lMU3XPAzzZ06YPZ/CD32+nu/FiRodCOfex73sqAA NClcKJOvO/ktNw+Nv/ZI/X7oo0MYyi0FEc/iA5sx9KN00qttsVZLon+Opayp+2QB PwykmwfEWnytNy5teu2db+lEjXbH8+z/bX4esliElWVBB5bTTc+JjqKFOufl9Wq3 Q4Broov1yb13cgDq8PaoLPSuioaEmTLIeswnNgtHkG41RX1L0Vs8PKF2EXR28z0M VLIILVj3GQhDHIFGAFZNo3JUo8ZwnY6RBgTMwVHvsLDu0kr1fv5PANyBFHRguUJX BJYTL6gf/4r2m0OPR+9Jjar0Mfgk3Jjzf8N1l9a4n8KbQjTccnXzygEaVjOqIkVE U50FNGwOdT9KXUxwIrbbg3zdg9W14RVKzXYI4xB5iLCSiqAMypV081G/1xnOHYH5 DPBJ5MUSiGScB+OfRrSy =NtbH -----END PGP SIGNATURE----- --f2spcziqu4xh5xpc--