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 3E9FEC433EF for ; Wed, 22 Dec 2021 09:07:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239516AbhLVJHH (ORCPT ); Wed, 22 Dec 2021 04:07:07 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45024 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236355AbhLVJHG (ORCPT ); Wed, 22 Dec 2021 04:07:06 -0500 Received: from mail-ed1-x52f.google.com (mail-ed1-x52f.google.com [IPv6:2a00:1450:4864:20::52f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 078BCC06173F for ; Wed, 22 Dec 2021 01:07:06 -0800 (PST) Received: by mail-ed1-x52f.google.com with SMTP id o6so5983196edc.4 for ; Wed, 22 Dec 2021 01:07:05 -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=OH9lUZGvbIYsFS6r93OdVkoyWwer11g866A2zNPeehk=; b=r3WN6KoV0SzdgtxvaGk6qQynqc2rvj3SjmK9MBxRug7Pb+EVJpUK3t+F+yktx2DuJc OBYJ2S/tGlHwp1SwDX3Tk8FzNeauleOXb8tli4nGClkgMNhBrB5Ix0fTy1fZ49TXXAdD 2zmt7dP0B7bijNj6jcyR5tlg2IMGqdpZNx90la65lXq+rIS/tpWu1Xm0N3FCPJn1YdV3 sVjtI99DrXEc8h+K9oqIjtQ/ZRjSkQmJ/zpQHi4QlRjTedfonW7XGiqlDGfZBk2S4CWi A5DG5wlrtcO62lxYpYfdJNGpLp/TihBJ092A4EYWpnn0zu66tWHrhZlqA6gWE9bQckw8 h80A== 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=OH9lUZGvbIYsFS6r93OdVkoyWwer11g866A2zNPeehk=; b=AlrYAI3752Moh17qw3+LzWqekGMwA5+5uVF0JwsObnhocPKBZOAF2AL1Nq4XMVw3uv bhDhKNT03cIhJHoauRLakSbstLM2DUtLT2GjtEa7mIDnnAInTGcopDODlLOC65eXXCd3 nW7SA5OUgn9kx3rJL3ZsWqTNuY2/J8omN0UoVzg+ctS7CRhZ0D29whAkQQjgIGzNzcNT oRZDRXwLUJ9okTgK0ufTm4C63xKkaDcgUQqhE+mSZs9drXExT4etl+8lrqLppn9RV3Mx +025YPggBAsbyfBtuf6TVMwCuT+bNaLBvOeLdLpG4kDuCVooM/3NgGT8sAVZ/uwXgimo Ns2Q== X-Gm-Message-State: AOAM532HzFQzPgntmraaXR8Z3YZEKetgfii/fWqrvjtAVVbhD+UcaqNn 3AXYl0ankChswCfdai5R+nYfCSbcH20mPCkGbON1VQ== X-Google-Smtp-Source: ABdhPJwlTOwQkimbTWuIr5y0SujyhX/GH7oxW5rFLOJAM7Fb2ZmmwI2kQCMkhG9muS1yczRmhCgRLzxKggSNitGnEZ0= X-Received: by 2002:a17:907:7208:: with SMTP id dr8mr1727842ejc.697.1640164024530; Wed, 22 Dec 2021 01:07:04 -0800 (PST) MIME-Version: 1.0 References: <20211220132439.1194-1-andriy.shevchenko@linux.intel.com> In-Reply-To: From: Bartosz Golaszewski Date: Wed, 22 Dec 2021 10:06:53 +0100 Message-ID: Subject: Re: [PATCH v1 1/1] gpio: Remove unused local OF node pointers To: Andy Shevchenko Cc: Andy Shevchenko , Sergio Paracuellos , Marc Zyngier , bcm-kernel-feedback-list , "open list:GPIO SUBSYSTEM" , Linux ARM , Linux Kernel Mailing List , "moderated list:ARM/Mediatek SoC..." , linux-unisoc@lists.infradead.org, Gregory Fong , Linus Walleij , Florian Fainelli , Matthias Brugger , Manivannan Sadhasivam , Stephen Rothwell Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Dec 21, 2021 at 9:21 PM Andy Shevchenko wrote: > > On Tue, Dec 21, 2021 at 9:13 PM Bartosz Golaszewski wrote: > > On Mon, Dec 20, 2021 at 4:11 PM Andy Shevchenko > > wrote: > > > On Mon, Dec 20, 2021 at 5:09 PM Bartosz Golaszewski wrote: > > > > On Mon, Dec 20, 2021 at 2:24 PM Andy Shevchenko > > > > wrote: > > ... > > > > > > - gc->label = devm_kasprintf(dev, GFP_KERNEL, "%pOF", dev->of_node); > > > > > + gc->label = devm_kasprintf(dev, GFP_KERNEL, "%pOF", np); > > > > > > > > This is unrelated though. Can you make it a separate patch? > > > > > > It actually makes use of the local np IIRC. > > > It's related, no? > > > > No, the np pointer is already used elsewhere. You just replace the > > dev->of_node here with np. Rightfully so, but it's not part of this > > commit logically. > > I see. Probably I missed that somehow. > Nevertheless, do you want a new version or can you just revert that > particular change? > > -- > With Best Regards, > Andy Shevchenko I removed it and queued the rest. Bart