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 1BE65C46467 for ; Sat, 17 Dec 2022 14:22:37 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229625AbiLQOWg (ORCPT ); Sat, 17 Dec 2022 09:22:36 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40508 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229475AbiLQOWf (ORCPT ); Sat, 17 Dec 2022 09:22:35 -0500 Received: from mail-lf1-x12a.google.com (mail-lf1-x12a.google.com [IPv6:2a00:1450:4864:20::12a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8949210078 for ; Sat, 17 Dec 2022 06:22:34 -0800 (PST) Received: by mail-lf1-x12a.google.com with SMTP id cf42so7661581lfb.1 for ; Sat, 17 Dec 2022 06:22:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=kZ5cHVqgvVJe/N0LNOPokBoYzo9K73QgpXdouMkj7zk=; b=VjvnmjLIx2zkaHhRBd1ERHGtVh0W/mMVfljst0WgMOtK2nRySeW2dA7k9bSBlCKYSL llQXDhxKpXYdkQc/mMzKeDeti4aUjqT+54Uo62G35tUuYe55/cLg48CEKjcPXa1/AZUr 5GE9RwY/LyP+N2Vd9OZrbCLNGBRENX4yp3Dq5Lm39ugo9ALVXdwY06ozWrwfQuiMSizu UhVpvObscCcDWV4aM4X66MH8lT0iPgqRH3MwNDk+x7ulPZ8Rr6EZFfAt0dog7LKBsZug asbGSxCa8y1hYlDXC2sa3geARZWB+oOBPyHEM5D/ylpOvRAj/YPJnF6/iE8MMpo2Uj8k 9cZA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=kZ5cHVqgvVJe/N0LNOPokBoYzo9K73QgpXdouMkj7zk=; b=qjHvAecZpxjsO9S9xXfgC6uFTnxfyaYNvryzOy8U9Q6jim8ggof9DJ+uQ1hUQS5RUO WzVNxkJ54UaGZeXc/Osu/UNMDdMbGNZV1vYuGW5HpDcZ8HH157ZZJbGzb89ZCknB4KhZ G3sWDxLwHTaIIhPfkjCVLqunCUMWjYuRBe71dSNY4ecvlf7TL7hnLnvslwOTStKeWdul ZkVPb5vX3xPvxZQNZo0cIA2TDx2SrqORzNur2wPEr6CkgqJuESJkW6Wrz9tkY1yYHKRe 4oypSvelXVVPk52DV4dMx5nCV3yN0zGXmiwnhhGksZJ2kkZ+bWyKlZ/9vDKtmeTi/7oV 8sCA== X-Gm-Message-State: ANoB5pkJoDnbIGT9nKJ6mSqnpR1jAIS8Eoe/toUbraSXRA1uAwEw++Ny VU7FcYkM5g/YeoRu3Sm85kerNQ== X-Google-Smtp-Source: AA0mqf4fD7u4aK9UvUQ6QAv1TPkAid7ix72nzfFyxbd9Mh63m2B4TewkspgqAxkOTuZo1hosYeQIxg== X-Received: by 2002:a05:6512:308a:b0:4b6:f51e:b8b6 with SMTP id z10-20020a056512308a00b004b6f51eb8b6mr6952120lfd.56.1671286952812; Sat, 17 Dec 2022 06:22:32 -0800 (PST) Received: from [192.168.1.101] (abxh44.neoplus.adsl.tpnet.pl. [83.9.1.44]) by smtp.gmail.com with ESMTPSA id a16-20020a19f810000000b004b54ca56cf9sm522157lff.303.2022.12.17.06.22.31 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 17 Dec 2022 06:22:32 -0800 (PST) Message-ID: <32bfd604-6fb6-51c6-5847-c87832ef1ebc@linaro.org> Date: Sat, 17 Dec 2022 15:22:25 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.6.0 Subject: Re: [PATCH v2 2/3] arm64: dts: qcom: sa8295p-adp: use pm8450a dtsi Content-Language: en-US To: Eric Chanudet Cc: Andy Gross , Bjorn Andersson , Rob Herring , Krzysztof Kozlowski , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Andrew Halaney , Brian Masney References: <20221214210908.1788284-1-echanude@redhat.com> <20221214210908.1788284-2-echanude@redhat.com> <5ed623d6-3ca2-a1c4-9277-6768df5a63fe@linaro.org> <20221216195632.2wu5h4opfr46n7s4@echanude> From: Konrad Dybcio In-Reply-To: <20221216195632.2wu5h4opfr46n7s4@echanude> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 16.12.2022 20:56, Eric Chanudet wrote: > On Thu, Dec 15, 2022 at 02:01:09PM +0100, Konrad Dybcio wrote: >> On 14.12.2022 22:09, Eric Chanudet wrote: >>> Include the dtsi to use a single pmic descriptions. >>> Both sa8295p-adp and sa8540p-adp have the same spmi pmic apparently. >>> >>> Signed-off-by: Eric Chanudet >>> --- >>> arch/arm64/boot/dts/qcom/sa8295p-adp.dts | 79 +----------------------- >>> 1 file changed, 1 insertion(+), 78 deletions(-) >>> >>> diff --git a/arch/arm64/boot/dts/qcom/sa8295p-adp.dts b/arch/arm64/boot/dts/qcom/sa8295p-adp.dts >>> index 84cb6f3eeb56..889259df3287 100644 >>> --- a/arch/arm64/boot/dts/qcom/sa8295p-adp.dts >>> +++ b/arch/arm64/boot/dts/qcom/sa8295p-adp.dts >>> @@ -11,6 +11,7 @@ >>> #include >>> >>> #include "sa8540p.dtsi" >>> +#include "pm8450a.dtsi" >> I feel like naming it sa8540p-pmics.dtsi (like sc8280xp-pmics.dtsi) >> would be more representative of what's really going on (unless it's >> a single chip providing 4 virtual PMICs on different SIDs). > > I can make a v3 renaming this. The initial commit from Parikshit > mentions it is to be re-used on sa8540 based boards. > > Side note: A quick look also shows pm8450.dtsi[1] is not included by > any of its intended targets (sm8350 and sm8450 IIUC). Was this lost? sm8450 only. They were not included, as SPMI was not enabled on sm8450. They will be included in 6.3 once that's merged. Konrad > > Thanks, > > [1] https://lore.kernel.org/r/20220226205035.1826360-8-dmitry.baryshkov@linaro.org >