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 90EEEC4332F for ; Mon, 31 Jan 2022 22:39:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234225AbiAaWjK (ORCPT ); Mon, 31 Jan 2022 17:39:10 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37328 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234172AbiAaWjK (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 DB3EFC06173D for ; Mon, 31 Jan 2022 14:39:09 -0800 (PST) Received: by mail-ot1-x330.google.com with SMTP id n6-20020a9d6f06000000b005a0750019a7so14494089otq.5 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=ZhNBY34kpaQmrjBaohhik11zOlZVN1x45dVLuRgNLFU3WMLENM8EwHb5BqUjyWaJDM FUE27nJYTiZyllLsxm52nHlQ1JUcULCIATGYC/FHV0iUzXQLb5ltSk+ChWcG6FQXiZIy 6Gv6gmsj5KZVupN2nX1ebMFCaa6Mo2Fhhg50+NXdlY8Wz5ZTH+Y4SdwxnKN1y+S8eSxo e6pobVNZRI4UpjnN5aaRTCrlOSIqGGJpyzY/4p6THPASoWXnSPBoZqd8IxqyHMrkJ9R9 7CoBXXEFCRh80oM2ObYgZBioyoQwLaFSyC8gKUw06rgK1/0ozji1C1mX0fcYF+AZdAru Uo/A== X-Gm-Message-State: AOAM530r+sauAhPpBKmPHbceBRFtg2BW0ZIiSupV4elKmD0Dv8/da1Ce N4vx4qmijTjMmaOWCBYscC92cQ== 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: linux-arm-msm@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 >