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=-15.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable 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 97954C2B9F4 for ; Mon, 14 Jun 2021 16:10:22 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 74A1B61378 for ; Mon, 14 Jun 2021 16:10:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233776AbhFNQMY (ORCPT ); Mon, 14 Jun 2021 12:12:24 -0400 Received: from mail-oo1-f42.google.com ([209.85.161.42]:45927 "EHLO mail-oo1-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233968AbhFNQMY (ORCPT ); Mon, 14 Jun 2021 12:12:24 -0400 Received: by mail-oo1-f42.google.com with SMTP id q20-20020a4a6c140000b029024915d1bd7cso2769131ooc.12 for ; Mon, 14 Jun 2021 09:10:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=WxkwNm6ZYNWIa0Lk9ntav/xspFP7K0YWVjmpKlkqGHM=; b=MWIRhnftb8ptws/gw00LvdIp2qhm1pyKeix9DxnZhFxe/rLAFyLuhnb38KFxVFY21U 3qLBPzwgtR/Zmw7YzqRBONHujLPBW4+yapefexTWW0MVYi2rQOyDaqUjbQN7jKkb1zdR r23kkH0xw1v2vJk+VmHiPi9exwOB/LvoNC7Ak0VjKQFTo51DdmHO137vb6gHMMnQE0d0 UHbtQT79HZg+zxRbowzUu+UtJliuRU1cDeKNYxQzP9J9ThSy/+fF5Hlvkp77e3BjnVgl mG8FAT0sZNXqf9eBKUJMORz6vQvy8P6biOO8WI2jV3NWqs5S76l3Ctu1k/w71JWtukmn jByA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=WxkwNm6ZYNWIa0Lk9ntav/xspFP7K0YWVjmpKlkqGHM=; b=dFGv09Og65m+vRozxTdHem7WQiztgndzZjYEXlX5vqg9htoTt1wrqDKfz9JG2S+cry rMLr0Vjmdw3WIZYscb9FxF/PPQJTGx0wNOg6NbDG6fGUR9VgW7sG9YUUVDqiFJ3jTPvP plmWflmgnAOvkitDkHeS0k1jxtjT3H3YOGiWPOkuc4YytuyQ/Ow/zy2eDzVfC1mzRVh6 xHvNSyxDzZ7sY5VALXiggm0i951rlkaxSqaC0QdU9Og4Wiq5KzSrWDpy+39vvPe8+Rrv t6FJ9VggzyythmD/AHtEDkyQU23C2B5EEKJY5SMo+alrM+G4pn8kvgwlAxLlmuXx3PH4 81mA== X-Gm-Message-State: AOAM533U3fV44V7MvJYPcg12uqPzayXGiOy2yqAszUmXDvRsYNXwDXmN eWBkgxXFwiKDeleMxp9009ojxA== X-Google-Smtp-Source: ABdhPJyKjW8ZTmKRNbtA7KEhYwH5Q70ndZULWrCYGmo8BGF/gqf4pAvVS2xZT8D5iVtAzeG7g/aryQ== X-Received: by 2002:a4a:9210:: with SMTP id f16mr13429698ooh.17.1623686960994; Mon, 14 Jun 2021 09:09:20 -0700 (PDT) Received: from yoga (104-57-184-186.lightspeed.austtx.sbcglobal.net. [104.57.184.186]) by smtp.gmail.com with ESMTPSA id g25sm3433972otn.81.2021.06.14.09.09.19 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 14 Jun 2021 09:09:20 -0700 (PDT) Date: Mon, 14 Jun 2021 11:09:18 -0500 From: Bjorn Andersson To: Bhupesh Sharma Cc: linux-arm-msm@vger.kernel.org, Linus Walleij , Liam Girdwood , Mark Brown , Vinod Koul , Rob Herring , Andy Gross , devicetree , Linux Kernel Mailing List , "open list:GPIO SUBSYSTEM" , bhupesh.linux@gmail.com Subject: Re: [PATCH 2/8] dt-bindings: pinctrl: qcom,pmic-gpio: Add compatible for SA8155p-adp Message-ID: References: <20210607113840.15435-1-bhupesh.sharma@linaro.org> <20210607113840.15435-3-bhupesh.sharma@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Mon 14 Jun 03:13 CDT 2021, Bhupesh Sharma wrote: > Hi Bjorn, > > On Fri, 11 Jun 2021 at 08:21, Bjorn Andersson > wrote: > > > > On Mon 07 Jun 06:38 CDT 2021, Bhupesh Sharma wrote: > > > > > Add pmic-gpio compatible strings for pmm8155au_1 and pmm8155au_2 pmics > > > found on SA8155p-adp board. > > > > > > Cc: Linus Walleij > > > Cc: Liam Girdwood > > > Cc: Mark Brown > > > Cc: Bjorn Andersson > > > Cc: Vinod Koul > > > Cc: Rob Herring > > > Cc: Andy Gross > > > Cc: devicetree@vger.kernel.org > > > Cc: linux-kernel@vger.kernel.org > > > Cc: linux-gpio@vger.kernel.org > > > Cc: bhupesh.linux@gmail.com > > > Signed-off-by: Bhupesh Sharma > > > --- > > > Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.txt | 5 +++++ > > > 1 file changed, 5 insertions(+) > > > > > > diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.txt b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.txt > > > index f6a9760558a6..ee4721f1c477 100644 > > > --- a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.txt > > > +++ b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.txt > > > @@ -27,6 +27,8 @@ PMIC's from Qualcomm. > > > "qcom,pm660l-gpio" > > > "qcom,pm8150-gpio" > > > "qcom,pm8150b-gpio" > > > + "qcom,pmm8155au-1-gpio" > > > + "qcom,pmm8155au-2-gpio" > > > > As with the regulator this seems to be a single component. > > > > > "qcom,pm8350-gpio" > > > "qcom,pm8350b-gpio" > > > "qcom,pm8350c-gpio" > > > @@ -116,6 +118,9 @@ to specify in a pin configuration subnode: > > > and gpio8) > > > gpio1-gpio12 for pm8150b (holes on gpio3, gpio4, gpio7) > > > gpio1-gpio12 for pm8150l (hole on gpio7) > > > + gpio1-gpio10 for pmm8155au-1 (holes on gpio2, gpio5, gpio7 > > > + and gpio8) > > > + gpio1-gpio10 for pmm8155au-2 (holes on gpio2, gpio5, gpio7) > > > > In the schematics all 10 pins are wired on both of these PMICs, so I > > don't think there are holes. Please omit the comment. > > But if we look at the downstream dts (see [1]), we clearly have holes > on gpio 2, 5 and 7 on PMM8155AU_2 whereas if we see [2], we can see > PMM8155AU_1 has holes on gpio 2, 5, 7 and 8. > > As I understand, the schematics mention some optional configurations > as well which might not be available depending on the default board > configuration. > > [1]. https://source.codeaurora.org/quic/la/kernel/msm-4.14/tree/arch/arm64/boot/dts/qcom/sa8155-pmic-overlay.dtsi?h=LV.AU.0.1.0.r1-15900-gen3meta.0#n92 > [2]. https://source.codeaurora.org/quic/la/kernel/msm-4.14/tree/arch/arm64/boot/dts/qcom/sa8155-pmic-overlay.dtsi?h=LV.AU.0.1.0.r1-15900-gen3meta.0#n36 > Perhaps these "holes" means that they are reserved for use by one of the other subsystems in the SoC - similar to how we have gpio-reserved-ranges in the TLMM. Because according to the PMS8155AU documentation and the schematics, there's no physical holes there. As such, the binding and driver should not describe these "holes". If they aren't used by Linux, then the DT won't reference them - and if there's reasons for prohibiting Linux from touching the registers we should use gpio-reserved-ranges, as this will depend on firmware configuration. Regards, Bjorn > Regards, > Bhupesh > > > > > > gpio1-gpio10 for pm8350 > > > gpio1-gpio8 for pm8350b > > > gpio1-gpio9 for pm8350c > > > -- > > > 2.31.1 > > >