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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 28E4DC433FE for ; Mon, 20 Dec 2021 15:16:32 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236938AbhLTPQa (ORCPT ); Mon, 20 Dec 2021 10:16:30 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39734 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237162AbhLTPOb (ORCPT ); Mon, 20 Dec 2021 10:14:31 -0500 Received: from mail-ed1-x52c.google.com (mail-ed1-x52c.google.com [IPv6:2a00:1450:4864:20::52c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 38BE0C0D940D for ; Mon, 20 Dec 2021 06:57:40 -0800 (PST) Received: by mail-ed1-x52c.google.com with SMTP id j21so35008570edt.9 for ; Mon, 20 Dec 2021 06:57:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20210112.gappssmtp.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=HUHmWuXX8VlTF9hpA1y1c+iXS8fi7L43NGXx9Hiegjg=; b=dvs02sSsAlxhiB6e0PVeHzVhBcEy2reW7+Wj9pgM3BKiZ/Gjqf8TFrL6XaVgX4k2HS ap3rX8JYih7erDXKhcGrmWwesnfbgjInwgVz6xHzv80g88dKY98/ke7o2GkeXbFvhLuG XZp3zYy0h/alewh36FJK29g0IxhZsEU8nsl+vCEP23gL/tf03swDjvG7KrM/ZyLLomjz Tw99P2+OhpdT3ADZD3wwlo+0ppbE7S1tqHD1tglcdmSivbKrKQZC8LN748Q9cfgLwc8h cyb+Y6c2gBfmq+eS/SHjTliQCRXg5SMEqfVqUtHaCXQVaIHhtA4UrJScca/BLpyfcprj wwQg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=HUHmWuXX8VlTF9hpA1y1c+iXS8fi7L43NGXx9Hiegjg=; b=Rk50fV0MYpwCAWJO0ycxeFBEhOTNP52Z5VR/HbGB7zvGmGfglDw1g+axXUCh+WeCHM XkeMljpjgDp2LR2nuPktrtKTcj8oYQta6bfWpTBWwkoalNq01fnMR3bAi+O8zFw0LJ7f h88Og/mBZWm2g/apb16NQtzPGnJCRgkYPFmv0NdlpQv57jXaews0vMphCPnMXWhBq7HH F4vOUmq7nc/DcbpBNhvD7y19CmtbrG4pP+xxZahgnFZK8rjmdHSm3yxLgNhI0+eWcjAB n8LQkm2G8rst6PgdYXkWzJOAdZTH1vhQ4e8qnK6TFULZy2xpghCS/88ucep0N0HmNKj5 Wx5w== X-Gm-Message-State: AOAM531TC6Hf/pSZuWmy3C4q50XOI+d2as+vwdxopPKjQO4MI5LcSktH DyYY+OD0VychNNEx87NZgIRWa28fqAodbFQUiAqfpw== X-Google-Smtp-Source: ABdhPJz9B8jdW71VfZkn6ik656/P3HPcergQlD1ReF7ru9atjKHTO6NZEt3lypROwqtqHQM4b1hb7fZyG+IO7ljSSQw= X-Received: by 2002:a17:907:6289:: with SMTP id nd9mr13119600ejc.101.1640012258769; Mon, 20 Dec 2021 06:57:38 -0800 (PST) MIME-Version: 1.0 References: <20211217153555.9413-1-marcelo.jimenez@gmail.com> In-Reply-To: From: Bartosz Golaszewski Date: Mon, 20 Dec 2021 15:57:28 +0100 Message-ID: Subject: Re: [PATCH] gpio: Revert regression in sysfs-gpio (gpiolib.c) To: Marcelo Roberto Jimenez , Thorsten Leemhuis Cc: stable , regressions@lists.linux.dev, Linus Walleij , "open list:GPIO SUBSYSTEM" , Thierry Reding , Vidya Sagar , Geert Uytterhoeven , Stephen Rothwell , Edmond Chung , Andrew Chant , Will McVicker , Sergio Tanzilli Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-gpio@vger.kernel.org On Sat, Dec 18, 2021 at 7:28 AM Thorsten Leemhuis wrote: > > [TLDR: I'm adding this regression to regzbot, the Linux kernel > regression tracking bot; most text you find below is compiled from a few > templates paragraphs some of you might have seen already.] > > On 17.12.21 16:35, Marcelo Roberto Jimenez wrote: > > Some GPIO lines have stopped working after the patch > > commit 2ab73c6d8323f ("gpio: Support GPIO controllers without pin-ranges") > > > > And this has supposedly been fixed in the following patches > > commit 89ad556b7f96a ("gpio: Avoid using pin ranges with !PINCTRL") > > commit 6dbbf84603961 ("gpiolib: Don't free if pin ranges are not defined") > > There seems to be a backstory here. Are there any entries and bug > trackers or earlier discussions everyone that looks into this should be > aware of? > Agreed with Thorsten. I'd like to first try to determine what's wrong before reverting those, as they are correct in theory but maybe the implementation missed something. Have you tried tracing the execution on your platform in order to see what the driver is doing? Bart