From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751236AbdAQQdZ (ORCPT ); Tue, 17 Jan 2017 11:33:25 -0500 Received: from fllnx209.ext.ti.com ([198.47.19.16]:51612 "EHLO fllnx209.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751129AbdAQQdW (ORCPT ); Tue, 17 Jan 2017 11:33:22 -0500 Subject: Re: [PATCH V3 0/5] gpio: davinci: Redesign driver to accommodate ngpios in one gpio chip To: References: <1484669955-5349-1-git-send-email-j-keerthy@ti.com> CC: , , , , From: Keerthy Message-ID: <8e91ed3d-6b60-606c-790f-cc5b6cf5ea41@ti.com> Date: Tue, 17 Jan 2017 22:02:41 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <1484669955-5349-1-git-send-email-j-keerthy@ti.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday 17 January 2017 09:49 PM, Keerthy wrote: > The Davinci GPIO driver is implemented to work with one monolithic > Davinci GPIO platform device which may have up to Y(144) gpios. > The Davinci GPIO driver instantiates number of GPIO chips with > max 32 gpio pins per each during initialization and one IRQ domain. > So, the current GPIO's opjects structure is: > > Davinci GPIO controller > |- ------| > ... |--- irq_domain (hwirq [0..143]) > |- ------| > > Current driver creates one chip for every 32 GPIOs in a controller. > This was a limitation earlier now there is no need for that. Hence > redesigning the driver to create one gpio chip for all the ngpio > in the controller. > > |- ------|--- irq_domain (hwirq [0..143]). > > The previous discussion on this can be found here: > https://www.spinics.net/lists/linux-omap/msg132869.html > > The series is posted on top of: > > https://lkml.org/lkml/2017/1/4/94 This is based on "devel" branch of the gpio tree. > > Changes in v2: > > * Optimized the re-design patch. > * Added couple of code clean ups after the re-design. > * Included v2 of https://patchwork.ozlabs.org/patch/710855/ > > Changes in v3: > > * Removed one of the clean up patches. As the macros were needed. > > Keerthy (5): > gpio: davinci: Remove gpio2regs function to accommodate multi > instances > gpio: davinci: Remove unwanted blank line > gpio: davinci: Redesign driver to accommodate ngpios in one gpio chip > gpio: davinci: Add support for multiple GPIO controllers > gpio: davinci: Remove custom .xlate > gpio: davinci: Remove redundant macros > > drivers/gpio/gpio-davinci.c | 174 +++++++++++++---------------- > include/linux/platform_data/gpio-davinci.h | 20 ++-- > 2 files changed, 90 insertions(+), 104 deletions(-) >