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 A1EA5C0502A for ; Tue, 30 Aug 2022 09:20:51 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231829AbiH3JUs (ORCPT ); Tue, 30 Aug 2022 05:20:48 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50558 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231761AbiH3JUU (ORCPT ); Tue, 30 Aug 2022 05:20:20 -0400 Received: from mail-pj1-x102b.google.com (mail-pj1-x102b.google.com [IPv6:2607:f8b0:4864:20::102b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8AD7E2A712 for ; Tue, 30 Aug 2022 02:20:01 -0700 (PDT) Received: by mail-pj1-x102b.google.com with SMTP id n65-20020a17090a5ac700b001fbb4fad865so11230892pji.1 for ; Tue, 30 Aug 2022 02:20:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc; bh=1stEK7orHgTbYEfwbeXC8fEwW/r5cq766TAqvkrU7s8=; b=nmk9fHCkrfrblpy/7kg08OX3ped77vmORbwkjjnbEVvUoXKOEKmX0Y1VGcflg+qh+d X5tuUyECrJp6R8RxGK62QsdF+fQbRQzNyCiGISzE5m5StPKqzt0BZ7Glg//ELPo5oSVG etz+NTjXyBz9fyw10DCx3yIeBzqheARABobLgWEFIgvMnpni+6W7Y8HuUMHy2YNYqJHY m9emxannnhCC1uNXxj8tgTkk/x5/54Y8WlBugCyPUt6MTgXTeO99BmfTb1wvKi4O0XP4 9fnghGoemu9Z13rG/mPuPrqOM5DJ8YuChJLBsl3tIg+uxjn5TpFRQH9ubuzE0FRZUxDL RPnw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc; bh=1stEK7orHgTbYEfwbeXC8fEwW/r5cq766TAqvkrU7s8=; b=7KZrsnfxrLVQOmScrKpuVsMa9UsQV8t/64G0zEX7ofh0GiMxgIW7+649gZNux/FAbf C6ZTF40xOpUW8BZegy4XdY6YiVpzmi/dMYXDgAV7k1QhgAtlxDLlkFTvNqIrH/pKSmDe XTKIC2TIfl6eLkgDgBySj4CLz+cP7wcLkXKQKNlmKUcVgxHPUf0FVqZbxtUyhRPS/l5l dlkPfmIY3VzXONKq1w1cCUajAgABKtuDksqGxxIzgJMWVtTKgRci3GPbUXwREKIUz1U0 CN4QJBOfGt5odVRkFkesMOrtppRvKMH3dlFJYWgAvpl3YgW7AgsAwXgYmYacdluTNV2t TVtw== X-Gm-Message-State: ACgBeo1IQpvvJ/WZ7XVTVqEoW1u2mrUhxDttvQYXc+e9UNv+z9fFJAHW G9261vYRCdLL7XSNoTnJTViLOnMTU9PY X-Google-Smtp-Source: AA6agR71f7NK6QRTQvMbVYh6yrfXsyqBOXunCOCPdx/lBWuvieJpGZ4JeK25bmov9unRjiB3MYANpA== X-Received: by 2002:a17:902:f64a:b0:172:7576:2124 with SMTP id m10-20020a170902f64a00b0017275762124mr19856760plg.155.1661851200915; Tue, 30 Aug 2022 02:20:00 -0700 (PDT) Received: from thinkpad ([117.193.209.245]) by smtp.gmail.com with ESMTPSA id q10-20020a170902eb8a00b001749e8eee4fsm5100555plg.226.2022.08.30.02.19.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 30 Aug 2022 02:20:00 -0700 (PDT) Date: Tue, 30 Aug 2022 14:49:53 +0530 From: Manivannan Sadhasivam To: Marc Zyngier Cc: linus.walleij@linaro.org, bjorn.andersson@linaro.org, robimarko@gmail.com, linux-arm-msm@vger.kernel.org, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, johan+linaro@kernel.org, steev@kali.org Subject: Re: [PATCH v2] pinctrl: qcom: spmi-gpio: Make IRQCHIP immutable Message-ID: <20220830091953.GC135982@thinkpad> References: <20220830081212.164709-1-manivannan.sadhasivam@linaro.org> <20220830084441.GB135982@thinkpad> <61371efca85fbbd360b3ede9a258ee69@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <61371efca85fbbd360b3ede9a258ee69@kernel.org> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Aug 30, 2022 at 09:48:28AM +0100, Marc Zyngier wrote: > On 2022-08-30 09:44, Manivannan Sadhasivam wrote: > > On Tue, Aug 30, 2022 at 09:26:51AM +0100, Marc Zyngier wrote: > > > On 2022-08-30 09:12, Manivannan Sadhasivam wrote: > > > > The IRQCHIP implementation used inside the gpiochips are not supposed to > > > > > > lower case > > > > > > > be changed during runtime. So let's make the one inside the spmi-gpio > > > > gpiochip immutable. > > > > > > > > This fixes the below warning during boot: > > > > gpio gpiochip0: (c440000.spmi:pmic@0:gpio@c000): not an immutable > > > > chip, please consider fixing it! > > > > > > > > Separate callbacks need to be provided for irq_{mask/unmask} pointers > > > > since > > > > the callbacks are supposed to mask/unmask the corresponding parent IRQ > > > > in > > > > addition to changing the gpio_desc flags. > > > > > > This is all part of the existing documentation, so I don't think > > > this is really needed. > > > > > > > Yes it is documented, but developers usually refer the commits doing the > > similar > > thing while doing these kind of conversions. For them, this text serves > > as a > > quick documentation. > > If they can't be bothered to read the documentation, why would they > consider reading unrelated commits? > > > > > So I prefer to keep it in the commit message. > > I still think this is pointless. > Okay. I'll remove it in v3. Thanks, Mani > M. > -- > Jazz is not dead. It just smells funny... -- மணிவண்ணன் சதாசிவம்