From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.3 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 40603C4321A for ; Tue, 11 Jun 2019 07:37:26 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 1B75B212F5 for ; Tue, 11 Jun 2019 07:37:26 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2404364AbfFKHhZ (ORCPT ); Tue, 11 Jun 2019 03:37:25 -0400 Received: from muru.com ([72.249.23.125]:52734 "EHLO muru.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2404009AbfFKHhZ (ORCPT ); Tue, 11 Jun 2019 03:37:25 -0400 Received: from atomide.com (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id 23EA580C0; Tue, 11 Jun 2019 07:37:45 +0000 (UTC) Date: Tue, 11 Jun 2019 00:37:20 -0700 From: Tony Lindgren To: Grygorii Strashko Cc: Russell King , Linus Walleij , Bartosz Golaszewski , linux-omap@vger.kernel.org, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, Santosh Shilimkar , Aaro Koskinen Subject: Re: [PATCH-next 00/20] gpio: gpio-omap: set of fixes and big clean-up Message-ID: <20190611073720.GB5447@atomide.com> References: <20190610171103.30903-1-grygorii.strashko@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190610171103.30903-1-grygorii.strashko@ti.com> User-Agent: Mutt/1.11.4 (2019-03-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, * Grygorii Strashko [190610 10:11]: > Hi Linus, Russell, Tony, All, > > This series contains set of patches from Russell King which were circulated > internally for quite some time already and I fill it's reasonable to move > future discussion upstream (and also avoid rebasing). > Fisrt two patches are fixes and the rest are big, great clean up > from Russell King. > > Personally, I like this clean up and refactoring very much and don't want > it to be lost. Adding Aaro to Cc too hopefully for more testing. Yes this is very nice and behaves for my idle test cases. I've also boot tested omap1 osk and it still works just fine for NFSroot. FYI, after this series, the only issue I'm aware of still remaining is the lost edge wake-up interrupts for L4 PER idle that I posted a WIP fix as "[PATCH] gpio: gpio-omap: Fix lost edge wake-up interrupts". But that one still needs a bit more work and is a separate fix from this series. So for this whole series, please feel free to add: Tested-by: Tony Lindgren > Code can be found at: > git@git.ti.com:~gragst/ti-linux-kernel/gragsts-ti-linux-kernel.git > branch: > lkml-next-gpio-clean-up > > Russell King (20): > gpio: gpio-omap: ensure irq is enabled before wakeup > gpio: gpio-omap: fix lack of irqstatus_raw0 for OMAP4 > gpio: gpio-omap: remove remainder of list management > gpio: gpio-omap: clean up edge interrupt handling > gpio: gpio-omap: remove irq_ack method > gpio: gpio-omap: move omap_gpio_request() and omap_gpio_free() > gpio: gpio-omap: simplify omap_gpio_get_direction() > gpio: gpio-omap: simplify get() method > gpio: gpio-omap: simplify get_multiple() > gpio: gpio-omap: simplify set_multiple() > gpio: gpio-omap: simplify bank->level_mask > gpio: gpio-omap: simplify read-modify-write > gpio: gpio-omap: simplify omap_toggle_gpio_edge_triggering() > gpio: gpio-omap: simplify omap_set_gpio_irqenable() > gpio: gpio-omap: remove dataout variation in context handling > gpio: gpio-omap: clean up omap_gpio_restore_context() > gpio: gpio-omap: constify register tables > gpio: gpio-omap: clean up wakeup handling > gpio: gpio-omap: irq_startup() must not return error codes > gpio: gpio-omap: clean up register access in omap2_set_gpio_debounce() > > drivers/gpio/gpio-omap.c | 497 ++++++++---------------- > include/linux/platform_data/gpio-omap.h | 2 +- > 2 files changed, 161 insertions(+), 338 deletions(-) > > -- > 2.17.1 >