From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Subject: Re: Fwd: Tegra 30 System hangs because of commit "pinctrl: move subsystem mutex to pinctrl_dev struct" Date: Mon, 29 Apr 2013 13:59:39 -0600 Message-ID: <517ED12B.8080502@wwwdotorg.org> References: Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Bryan Wu Cc: "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, patrice.chotard-qxv4g6HH51o@public.gmane.org List-Id: linux-tegra@vger.kernel.org On 04/29/13 12:41, Bryan Wu wrote: > Forward to ARM and Tegra mail list. > > ---------- Forwarded message ---------- > From: Bryan Wu > Date: Mon, Apr 29, 2013 at 11:15 AM > Subject: Tegra 30 System hangs because of commit "pinctrl: move > subsystem mutex to pinctrl_dev struct" > To: patrice.chotard-qxv4g6HH51o@public.gmane.org, Linus Walleij > Cc: Stephen Warren , > "linux-next-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" > > > Hi Patrice and Linus, > > I'm running linux-next on my Tegra 30 system everyday. But since > next-20130426 release, the linux-next kernel will hang on my Tegra 30 > boards like Cardhu and Beaver. > > After some bisect and comparing to next-20130424, I found the commit > "pinctrl: move subsystem mutex to pinctrl_dev struct" introduced this > issue. If I simply revert this patch in next-20130429, system boots up > as before. > > Could you please help to check this? I would suggest simply dropping or reverting this patch; I had pointed out a long time ago that it was unlikely to be correct. My guess/suspicion is a recursive lock attempt during processing hogs during pinctrl device registration, or perhaps during dt->map conversion. Bryan, are you able to confirm this?