Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752377Ab3IRLZw (ORCPT ); Wed, 18 Sep 2013 07:25:52 -0400 Received: from mail-lb0-f171.google.com ([209.85.217.171]:42835 "EHLO mail-lb0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751903Ab3IRLZu (ORCPT ); Wed, 18 Sep 2013 07:25:50 -0400 Date: Wed, 18 Sep 2013 16:25:38 +0500 From: "Zubair Lutfullah :" To: Jonathan Cameron Cc: "Zubair Lutfullah :" , Dmitry Torokhov , linux-iio@vger.kernel.org, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, bigeasy@linutronix.de, gregkh@linuxfoundation.org Subject: Re: [PATCH 2/2] iio: ti_am335x_adc: Add continuous sampling support Message-ID: <20130918112536.GA17126@gmail.com> References: <1379393047-11772-1-git-send-email-zubair.lutfullah@gmail.com> <1379393047-11772-3-git-send-email-zubair.lutfullah@gmail.com> <20130918042726.GB13196@core.coreip.homeip.net> <20130918065406.GA13451@gmail.com> <53771776-0d33-436d-9687-995ed0d6345d@email.android.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <53771776-0d33-436d-9687-995ed0d6345d@email.android.com> 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 Content-Length: 1509 Lines: 38 On Wed, Sep 18, 2013 at 10:39:42AM +0100, Jonathan Cameron wrote: > "Zubair Lutfullah :" wrote: > >On Tue, Sep 17, 2013 at 09:27:27PM -0700, Dmitry Torokhov wrote: > >> Hi Zubair, > >> > >> On Tue, Sep 17, 2013 at 09:44:07AM +0500, Zubair Lutfullah wrote: > >> > + > >> > + ret = devm_request_threaded_irq(indio_dev->dev.parent, > >> > + irq, > >> > + pollfunc_th, pollfunc_bh, > >> > + flags, indio_dev->name, > >> > + indio_dev); > >> > + if (ret) > >> > + goto error_kfifo_free; ... > >> > + > >> > +error_free_irq: > >> > + devm_free_irq(indio_dev->dev.parent, irq, indio_dev); > >> > >> What is the point of using devm_* here if you are doing explicit > >> management of the resource anyway (you explicitly release it in all > >> Dmitry > > The devm interfaces ensure this is all cleaned when the device is removed thus avoiding the need to free the stuff explicitly. > Device will get freed on deliberate remove and on an error from probe. Hence you can drop all calls to devm free. The devm free functions are only needed if you wish to free in order to reallocate. This might happen if you want to change a buffer size for instance. > Thank-you for the feedback. Updated and resent the series. Zubair > -- -- 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/