From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bartosz Golaszewski Subject: Re: [PATCH 4/6] gpio: mxc: use devres for irq generic chip Date: Wed, 2 Aug 2017 13:54:09 +0200 Message-ID: References: <20170802075126.17637-1-brgl@bgdev.pl> <20170802075126.17637-5-brgl@bgdev.pl> <2153314.vmO4TSUiTO@ws-stein> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: Received: from mail-qk0-f178.google.com ([209.85.220.178]:36846 "EHLO mail-qk0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753018AbdHBLyK (ORCPT ); Wed, 2 Aug 2017 07:54:10 -0400 Received: by mail-qk0-f178.google.com with SMTP id d136so25095827qkg.3 for ; Wed, 02 Aug 2017 04:54:10 -0700 (PDT) In-Reply-To: Sender: linux-gpio-owner@vger.kernel.org List-Id: linux-gpio@vger.kernel.org To: Linus Walleij Cc: Andy Shevchenko , Alexander Stein , "linux-gpio@vger.kernel.org" , "linux-kernel@vger.kernel.org" 2017-08-02 13:41 GMT+02:00 Linus Walleij : > On Wed, Aug 2, 2017 at 10:32 AM, Andy Shevchenko > wrote: >> On Wed, Aug 2, 2017 at 11:09 AM, Alexander Stein >> wrote: >>> On Wednesday 02 August 2017 09:51:24, Bartosz Golaszewski wrote: >>>> Use resource managed variants of irq_alloc_generic_chip() and >>>> irq_setup_generic_chip(). >>> >>> Is this really useful for drivers which can only be built-in? >> >> But you still can unbind the driver and its ->remove() will be called >> (in case of no remove, devres still on the table), right? > > Maybe the patches need to be combines with a > .suppress_bind_attrs = true in the driver struct? > > I backed out the patches I applied, I thought the series were older, > sorry stressed at work today. > gpio-sodaville sets .supress_bind_attrs to true. Other built-in drivers need updates for that, but I think this could go in a separate series as using devres doesn't affect the bind/unbind functionality/issue. Thanks, Bartosz