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 09BD0C433F5 for ; Mon, 31 Jan 2022 22:39:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234265AbiAaWjK (ORCPT ); Mon, 31 Jan 2022 17:39:10 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37330 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234201AbiAaWjK (ORCPT ); Mon, 31 Jan 2022 17:39:10 -0500 Received: from mail-ot1-x330.google.com (mail-ot1-x330.google.com [IPv6:2607:f8b0:4864:20::330]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E19E3C06173E for ; Mon, 31 Jan 2022 14:39:09 -0800 (PST) Received: by mail-ot1-x330.google.com with SMTP id o9-20020a9d7189000000b0059ee49b4f0fso14501392otj.2 for ; Mon, 31 Jan 2022 14:39:09 -0800 (PST) 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=I2YeCNPA7JUrutaCl69AyRbHGGFWusCcn2hr3SGCW1s=; b=foDZPqDmjYJKTvX6XkRIv+WJyzkPaJu2DQxeqscZqPbLvLmMYwSSLZIZxpUiDkoKGg anOqt8nWUzxsywZ1RzvNUZj6UFVrIrsyLyZcAgyq8DcLWkdmuUJMUS/IQBXm51d0qFi7 ggpkddcxJ4N5F80UlNz8oAXkqLuVxdAFgnLVUDNSuGUsaiS/CXuPvIEFdlr1tZ2kBB9K fbtTNlFg2tWvV6zY7UwsFqazV83XlfBPl5QXW/g20crr+jqGqgdJr2hr12/UeQy6Aqo1 hnq22qK94l1Ui+Z/qUvmwY8Idt2hHe+0uA2dw488uUsD6FC5hy5raxDbwu87Dv11Vpn+ o8qw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=I2YeCNPA7JUrutaCl69AyRbHGGFWusCcn2hr3SGCW1s=; b=w1gLlGdMiQF0fI3gMXX6T8IQyH6o6blLbfvUpjsZI3J6kSEhVJEA0FusesOREz2oiQ oC4ZFwUHioDkIrIW/vSvBx1tAJGmpEDftBoRI/E6gQoFqE7bA+TLMXy3W2hY/Mo7G4An TytSX8Wupo5buvnxLeGaqFsEwaTIII8Di75u4O1lMcUs3wVVCVEpXJvPKuSrx0xLmyku m7m+wm9oSEf+fjhA0jGW1JwotV8TKlI5hipvOumC40+RJv4JITdHyF6aeBkIaOLx5ZvA nM+8GR5VaS/UP1iuAUpgRMUIpbC9eY3km7BPhqZEwSHPvdmwM1qKNwRT4ckys0pK+Wmw 4Dxw== X-Gm-Message-State: AOAM533j95J0Ht8x3hXdpXXoXYMuK6lvdaFSXaGwseRETcRC8pjF/yuw W0kcFb1JxTqdM6PCgfYSrLk3pA== X-Google-Smtp-Source: ABdhPJz3DrrbNejbTueCAz6MmMNxgUOQKSR3d4PLal0CVNEOYgaNKJjs/dRMEoEqJtXTi6yuHVSS/A== X-Received: by 2002:a9d:12d7:: with SMTP id g81mr12610035otg.82.1643668749113; Mon, 31 Jan 2022 14:39:09 -0800 (PST) Received: from builder.lan ([2600:1700:a0:3dc8:3697:f6ff:fe85:aac9]) by smtp.gmail.com with ESMTPSA id t4sm13313906oie.14.2022.01.31.14.39.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 31 Jan 2022 14:39:08 -0800 (PST) Date: Mon, 31 Jan 2022 16:39:06 -0600 From: Bjorn Andersson To: Luca Weiss Cc: linux-arm-msm@vger.kernel.org, ~postmarketos/upstreaming@lists.sr.ht, phone-devel@vger.kernel.org, Vladimir Lypak , Konrad Dybcio , Andy Gross , Rob Herring , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 13/15] arm64: dts: qcom: Add MSM8953+PM8953 device tree Message-ID: References: <20220112194118.178026-1-luca@z3ntu.xyz> <20220112194118.178026-14-luca@z3ntu.xyz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220112194118.178026-14-luca@z3ntu.xyz> Precedence: bulk List-ID: X-Mailing-List: phone-devel@vger.kernel.org On Wed 12 Jan 13:41 CST 2022, Luca Weiss wrote: > From: Vladimir Lypak > > The combination MSM8953 + PM8953 is commonly used, so add a > device tree where common power supplies etc. can be configured. > > Signed-off-by: Vladimir Lypak > Signed-off-by: Luca Weiss > Reviewed-by: Konrad Dybcio I would prefer if we stick with the current scheme and just push this into the device dts (or possibly some vendor-common dtsi if that's applicable). Simply just to follow what we do on other platforms. PS. I see some patches has been applied, but as you resubmit this series please split it per maintainer to make it obvious to each maintainer that they should pick their part(s). Thanks, Bjorn > --- > arch/arm64/boot/dts/qcom/msm8953-pm8953.dtsi | 50 ++++++++++++++++++++ > 1 file changed, 50 insertions(+) > create mode 100644 arch/arm64/boot/dts/qcom/msm8953-pm8953.dtsi > > diff --git a/arch/arm64/boot/dts/qcom/msm8953-pm8953.dtsi b/arch/arm64/boot/dts/qcom/msm8953-pm8953.dtsi > new file mode 100644 > index 000000000000..b5f20fc9488e > --- /dev/null > +++ b/arch/arm64/boot/dts/qcom/msm8953-pm8953.dtsi > @@ -0,0 +1,50 @@ > +// SPDX-License-Identifier: BSD-3-Clause > +/* Copyright (c) 2022, The Linux Foundation. All rights reserved. */ > + > +#include "msm8953.dtsi" > +#include "pm8953.dtsi" > + > +&hsusb_phy { > + vdd-supply = <&pm8953_l3>; > + vdda-pll-supply = <&pm8953_l7>; > + vdda-phy-dpdm-supply = <&pm8953_l13>; > +}; > + > +&sdhc_1 { > + vmmc-supply = <&pm8953_l8>; > + vqmmc-supply = <&pm8953_l5>; > +}; > + > +&sdhc_2 { > + vmmc-supply = <&pm8953_l11>; > + vqmmc-supply = <&pm8953_l12>; > +}; > + > +&rpm_requests { > + smd_rpm_regulators: pm8953-regulators { > + compatible = "qcom,rpm-pm8953-regulators"; > + > + pm8953_s1: s1 {}; > + pm8953_s3: s3 {}; > + pm8953_s4: s4 {}; > + > + pm8953_l1: l1 {}; > + pm8953_l2: l2 {}; > + pm8953_l3: l3 {}; > + pm8953_l5: l5 {}; > + pm8953_l6: l6 {}; > + pm8953_l7: l7 {}; > + pm8953_l8: l8 {}; > + pm8953_l9: l9 {}; > + pm8953_l10: l10 {}; > + pm8953_l11: l11 {}; > + pm8953_l12: l12 {}; > + pm8953_l13: l13 {}; > + pm8953_l15: l15 {}; > + pm8953_l16: l16 {}; > + pm8953_l17: l17 {}; > + pm8953_l19: l19 {}; > + pm8953_l22: l22 {}; > + pm8953_l23: l23 {}; > + }; > +}; > -- > 2.34.1 >