From mboxrd@z Thu Jan 1 00:00:00 1970 From: Linus Walleij Subject: Re: [RFT PATCH v2] gpiolib: allow gpio irqchip to map irqs dynamically Date: Mon, 9 Oct 2017 21:57:58 +0200 Message-ID: References: <20170721164900.7070-1-grygorii.strashko@ti.com> <20170928083348.GI4630@lahna.fi.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: Received: from mail-it0-f45.google.com ([209.85.214.45]:56933 "EHLO mail-it0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755072AbdJIT57 (ORCPT ); Mon, 9 Oct 2017 15:57:59 -0400 Received: by mail-it0-f45.google.com with SMTP id g18so14233136itg.5 for ; Mon, 09 Oct 2017 12:57:59 -0700 (PDT) In-Reply-To: Sender: linux-gpio-owner@vger.kernel.org List-Id: linux-gpio@vger.kernel.org To: Grygorii Strashko Cc: Mika Westerberg , "linux-gpio@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Jerome Brunet , Chris Gorman On Mon, Oct 9, 2017 at 8:10 PM, Grygorii Strashko wrote: > At the moment it was merged there were no user of irq_base except gpio-mockup.c. OK I feel calmer. > And actually there are shouldn't as calling irq_create_mapping() in cycle > will not guarantee sequential Linux IRQ numbers allocation. Yeah. I always felt that was bad practice. But we should really kill off that irq_base member from gpio_chip. It looks dangerous to have arond. Yours, Linus Walleij