Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752617AbbHGJXR (ORCPT ); Fri, 7 Aug 2015 05:23:17 -0400 Received: from mail-yk0-f170.google.com ([209.85.160.170]:32829 "EHLO mail-yk0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752250AbbHGJWr (ORCPT ); Fri, 7 Aug 2015 05:22:47 -0400 MIME-Version: 1.0 In-Reply-To: References: <8fcb9ff7a8e01bec53813f0702ff606bf4982943.1438844454.git.baolin.wang@linaro.org> <20150807054554.GE13200@shlinux2> Date: Fri, 7 Aug 2015 17:22:47 +0800 Message-ID: Subject: Re: [PATCH 2/2] gadget: Support for the usb charger framework From: Baolin Wang To: Peter Chen Cc: Felipe Balbi , Mark Brown , LKML , Greg KH , "sojka@merica.cz" , Alan Stern , "andreas@gaisler.com" , "linux-usb@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1532 Lines: 45 On 7 August 2015 at 17:07, Peter Chen wrote: > >> >> /** >> >> * struct usb_udc - describes one usb device controller @@ -127,12 >> >> +128,45 @@ void usb_gadget_giveback_request(struct usb_ep *ep, } >> >> EXPORT_SYMBOL_GPL(usb_gadget_giveback_request); >> >> >> >> +int usb_gadget_register_notify(struct usb_gadget *gadget, >> >> + struct notifier_block *nb) { >> >> + unsigned long flags; >> >> + int ret; >> >> + >> >> + spin_lock_irqsave(&gadget->lock, flags); >> > >> > I find you use so many spin_lock_irqsave, any reasons for that? >> > Why mutex_lock can't be used? >> > >> >> The spin_lock_irqsave() can make it as a atomic notifier, that can make sure the >> gadget state event can be quickly reported to the user who register a notifier >> on the gadget device. Is it OK? >> > > I don't think it is a good reason, spin_lock_irqsave is usually used for protecting > data which is accessed at atomic environment. > Yes, we want the notify process is a atomic environment which do not want to be interrupted by irq or other things to make the notice can be quickly reported to the user. Also i think the notify process is less cost, thus i use the spinlock. Thanks. > Peter -- Baolin.wang Best Regards -- 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/