From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755724Ab3EVHzG (ORCPT ); Wed, 22 May 2013 03:55:06 -0400 Received: from mga02.intel.com ([134.134.136.20]:11219 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755432Ab3EVHzC (ORCPT ); Wed, 22 May 2013 03:55:02 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.87,718,1363158000"; d="scan'208";a="338079820" Date: Wed, 22 May 2013 10:58:26 +0300 From: Mika Westerberg To: Andy Shevchenko Cc: Linus Walleij , David Cohen , linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 1/4] gpio-langwell: initialize lock before usage Message-ID: <20130522075826.GU11878@intel.com> References: <1369208859-16514-1-git-send-email-andriy.shevchenko@linux.intel.com> <1369208859-16514-2-git-send-email-andriy.shevchenko@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1369208859-16514-2-git-send-email-andriy.shevchenko@linux.intel.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, May 22, 2013 at 10:47:36AM +0300, Andy Shevchenko wrote: > Otherwise we will end up with traceback from LOCKDEP: > > INFO: trying to register non-static key. > the code is fine but needs lockdep annotation. > turning off the locking correctness validator. > CPU: 0 PID: 1 Comm: swapper/0 Not tainted 3.10.0-rc2-next-20130521-00028-g09aa9fc #487 > 00000000 00000000 f6c55c54 c1541fe4 f6040bf8 f6c55c8c c1069ef1 c1726bc1 > c1726cc8 c1726c9e 00000000 f6c584e0 f6c58000 f6c55ce8 00000000 f6040bf8 > f6040bf8 00000046 f6c58000 f6c55d00 c106a18d 00000a2b 00000003 00004f02 > Call Trace: > [] dump_stack+0x49/0x77 > [] register_lock_class+0x58/0x260 > [] __lock_acquire+0x94/0xcff > [] ? __lock_acquire+0xccf/0xcff > [] lock_acquire+0xcc/0x10d > [] ? lnw_irq_type+0x63/0xe9 > [] _raw_spin_lock_irqsave+0x32/0x42 > [] ? lnw_irq_type+0x63/0xe9 > [] lnw_irq_type+0x63/0xe9 > [] __irq_set_trigger+0x98/0x123 > [] irq_set_irq_type+0x2f/0x51 > [] ? irq_set_irq_type+0x2f/0x51 > [] ? lnw_irq_type+0xe9/0xe9 > [] lnw_gpio_irq_map+0x32/0x3b > [] irq_domain_add_legacy+0xe2/0x107 > [] irq_domain_add_simple+0x47/0x60 > [] lnw_gpio_probe+0x119/0x217 > [] pci_device_probe+0x5a/0x92 > ... > > Signed-off-by: Andy Shevchenko Acked-by: Mika Westerberg