Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965152AbaGPPgB (ORCPT ); Wed, 16 Jul 2014 11:36:01 -0400 Received: from mga09.intel.com ([134.134.136.24]:48140 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933938AbaGPPf6 (ORCPT ); Wed, 16 Jul 2014 11:35:58 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.01,673,1400050800"; d="scan'208";a="574130911" Message-ID: <53C69BCF.8030307@iki.fi> Date: Wed, 16 Jul 2014 18:35:43 +0300 From: Sakari Ailus User-Agent: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:29.0) Gecko/20100101 SeaMonkey/2.26.1 MIME-Version: 1.0 To: Jacek Anaszewski , linux-leds@vger.kernel.org, devicetree@vger.kernel.org, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org CC: kyungmin.park@samsung.com, b.zolnierkie@samsung.com, Bryan Wu , Richard Purdie Subject: Re: [PATCH/RFC v4 02/21] leds: implement sysfs interface locking mechanism References: <1405087464-13762-1-git-send-email-j.anaszewski@samsung.com> <1405087464-13762-3-git-send-email-j.anaszewski@samsung.com> In-Reply-To: <1405087464-13762-3-git-send-email-j.anaszewski@samsung.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Jacek, Thank you for the update! Jacek Anaszewski wrote: ... > diff --git a/drivers/leds/led-core.c b/drivers/leds/led-core.c > index 71b40d3..4d7cb31 100644 > --- a/drivers/leds/led-core.c > +++ b/drivers/leds/led-core.c > @@ -126,3 +126,21 @@ void led_set_brightness(struct led_classdev *led_cdev, > __led_set_brightness(led_cdev, brightness); > } > EXPORT_SYMBOL(led_set_brightness); > + > +/* Caller must ensure led_cdev->led_lock held */ > +void led_sysfs_lock(struct led_classdev *led_cdev) > +{ > + WARN_ON(!mutex_is_locked(&led_cdev->led_lock)); How about lockdep_assert_held() instead? -- Sakari Ailus e-mail: sakari.ailus@iki.fi XMPP: sailus@retiisi.org.uk -- 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/