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=-21.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_2 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 81929C4338F for ; Mon, 26 Jul 2021 07:56:37 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 422E7600D1 for ; Mon, 26 Jul 2021 07:56:37 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 422E7600D1 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=SgH0lFPqsF93HlvVylp7nqhchmSLsd4MnZqBGSd7mYE=; b=akE5r6j/tWO6Eh G4PhEkXyMsxAHEOG1iCUGBKtP66fTn+8BPzEQ9BMb8EyQ7tEE+fIfpddn0qQ7+ajp2PzFf6eWE5Se fBH9aGlr3UEZ0Ujazof1LNkVM9E7I7FoiWG+jJEa0Q1wUJnxtIsvvrvoW/L37k7Zku+8QK+LPcZfz YBMmYEPcsG8j1n4SgJ4ufUc7G6JQDfRL1136HopG4jl2Q83+OJipcwKR/GOJsN+hYacTe+LWRzBcb OHo9mSaq6JcMVufN0EGelwCb+/T/c3ksloCOmsIcup2wLmtPIKhRRNr69FoKx9toj2J0o82XiTQoT A0xc4mM5dwXy96AOmTiQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m7vHH-00A0Ud-SZ; Mon, 26 Jul 2021 07:43:56 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m7vHC-00A0TF-Vg for linux-arm-kernel@lists.infradead.org; Mon, 26 Jul 2021 07:43:52 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 8B2F160C51; Mon, 26 Jul 2021 07:43:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1627285430; bh=NHQc2BQbV+EpmfsJG7sWI5huE7s0E6egpdCnikBrACM=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=eNU1PUe114yOBv0o1RHTLCE8znJx8Uk9ZBT6kFZTYU0a36Vy2xEhw8bNjEs9IHVfv jtyTc1mto+46eT036JLZFXRTMUTvnaG5rbH7bwhtv6y/nBKUI+6doUtSbz6VnG5V2Q 98nUMQQeKIIFkZEFKtUOG3UGllOAFVwO1/c5GMEMKLoiUPJrvVnP4EqsmDlFpc/URt P863pG8JKLDUzEvJ5phAjW2D2jqGX8IrcEs9STx2ZXty+XtfTn5a96bqngV9Ooic8P YsI7PoYZBKlYRru/mnpfyk09rrt/JuPPZXgs+9KmuxlGxLiFttcXMRIOshl8JfeBvb srgn7c56rCJFw== Date: Mon, 26 Jul 2021 09:43:44 +0200 From: Mauro Carvalho Chehab To: Wei Xu , Greg Kroah-Hartman Cc: , , Rob Herring , Rob Herring , , , , "xuwei (O)" Subject: Re: [PATCH 1/2] dts: hisilicon: add support for the PMIC found on Hikey 970 Message-ID: <20210726094344.0a415bd0@coco.lan> In-Reply-To: <60FE57B5.2010603@hisilicon.com> References: <4a3583dd683512c2a4a138e88d4c889e51bf48e8.1627116285.git.mchehab+huawei@kernel.org> <60FE20AC.3050400@hisilicon.com> <20210726081812.54431189@coco.lan> <60FE57B5.2010603@hisilicon.com> X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.30; x86_64-redhat-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210726_004351_089903_80A1A758 X-CRM114-Status: GOOD ( 34.06 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Em Mon, 26 Jul 2021 14:35:33 +0800 Wei Xu escreveu: > Hi Mauro, > > On 2021/7/26 14:19, Mauro Carvalho Chehab wrote: > > Em Mon, 26 Jul 2021 10:40:44 +0800 > > Wei Xu escreveu: > > > >> Hi Mauro, > >> > >> On 2021/7/24 16:55, Mauro Carvalho Chehab wrote: > >>> Add a device tree for the HiSilicon 6421v600 SPMI PMIC, used > >>> on HiKey970 board. > >>> > >>> As we now have support for it, change the fixed regulators > >>> used by the SD I/O to use the proper LDO supplies. > >>> > >>> Signed-off-by: Mauro Carvalho Chehab > >>> --- > >>> .../boot/dts/hisilicon/hi3670-hikey970.dts | 22 +---- > >>> .../boot/dts/hisilicon/hikey970-pmic.dtsi | 86 +++++++++++++++++++ > >>> 2 files changed, 89 insertions(+), 19 deletions(-) > >>> create mode 100644 arch/arm64/boot/dts/hisilicon/hikey970-pmic.dtsi > >>> > >>> diff --git a/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts b/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts > >>> index d8abf442ee7e..7c32f5fd5cc5 100644 > >>> --- a/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts > >>> +++ b/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts > >>> @@ -12,6 +12,7 @@ > >>> > >>> #include "hi3670.dtsi" > >>> #include "hikey970-pinctrl.dtsi" > >>> +#include "hikey970-pmic.dtsi" > >>> > >>> / { > >>> model = "HiKey970"; > >>> @@ -39,23 +40,6 @@ memory@0 { > >>> reg = <0x0 0x0 0x0 0x0>; > >>> }; > >>> > >>> - sd_1v8: regulator-1v8 { > >>> - compatible = "regulator-fixed"; > >>> - regulator-name = "fixed-1.8V"; > >>> - regulator-min-microvolt = <1800000>; > >>> - regulator-max-microvolt = <1800000>; > >>> - regulator-always-on; > >>> - }; > >>> - > >>> - sd_3v3: regulator-3v3 { > >>> - compatible = "regulator-fixed"; > >>> - regulator-name = "fixed-3.3V"; > >>> - regulator-min-microvolt = <3300000>; > >>> - regulator-max-microvolt = <3300000>; > >>> - regulator-boot-on; > >>> - regulator-always-on; > >>> - }; > >>> - > >>> wlan_en: wlan-en-1-8v { > >>> compatible = "regulator-fixed"; > >>> regulator-name = "wlan-en-regulator"; > >>> @@ -402,8 +386,8 @@ &dwmmc1 { > >>> pinctrl-0 = <&sd_pmx_func > >>> &sd_clk_cfg_func > >>> &sd_cfg_func>; > >>> - vmmc-supply = <&sd_3v3>; > >>> - vqmmc-supply = <&sd_1v8>; > >>> + vmmc-supply = <&ldo16>; > >>> + vqmmc-supply = <&ldo9>; > >>> status = "okay"; > >>> }; > >>> > >>> diff --git a/arch/arm64/boot/dts/hisilicon/hikey970-pmic.dtsi b/arch/arm64/boot/dts/hisilicon/hikey970-pmic.dtsi > >>> new file mode 100644 > >>> index 000000000000..970047f2dabd > >>> --- /dev/null > >>> +++ b/arch/arm64/boot/dts/hisilicon/hikey970-pmic.dtsi > >>> @@ -0,0 +1,86 @@ > >>> +// SPDX-License-Identifier: GPL-2.0 > >>> +/* > >>> + * dts file for Hi6421v600 SPMI PMIC used at the HiKey970 Development Board > >>> + * > >>> + * Copyright (C) 2020, Huawei Tech. Co., Ltd. > >>> + */ > >>> + > >>> +#include > >>> + > >>> +/ { > >>> + spmi: spmi@fff24000 { > >>> + compatible = "hisilicon,kirin970-spmi-controller"; > >>> + #address-cells = <2>; > >>> + #size-cells = <0>; > >>> + status = "okay"; > >>> + reg = <0x0 0xfff24000 0x0 0x1000>; > >>> + hisilicon,spmi-channel = <2>; > >>> + > >>> + pmic: pmic@0 { > >>> + compatible = "hisilicon,hi6421-spmi"; > >> > >> Should this be "hisilicon,hi6421-pmic" which is already in the binding document "mfd/hi6421.txt"? > >> Others are OK to me. > > > > No. > > > > hisilicon,hi6421-pmic is used by this driver > > drivers/mfd/hi6421-pmic-core.c > > > > which provides support for the Hi6421 variants that don't use a MIPI > > SPMI bus. > > > > The "hisilicon,hi6421-spmi" compatible is for the new driver: > > drivers/mfd/hi6421-spmi-pmic.c > > > > And it is defined at: > > Documentation/devicetree/bindings/mfd/hisilicon,hi6421-spmi-pmic.yaml > > > > Both DT bindings and the driver were promoted from staging on this patch: > > > > https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/commit/?h=staging-next&id=9bd9e0de1cf5b89c4854be505ac0a418ddcc01bf > > Got it. Thanks for your kind explanation! > So I am fine with this patch. > Do you need my ack now or later I can merge it once it is out of staging. Whatever works best for you and Greg. It should be noticed that the driver should be out of staging for v5.15-rc1 (as the patch moving it out of staging was already merged at Greg's tree). This patch is needed by the PCI driver patchset I'm currently working, so the best would be if this patch could also be merged for v5.15-rc1 (either via HiSilicon tree or via staging tree). I suspect that having your ack and merging this via staging would work best, as this would ensure that both changes will be merged upstream at the same time. But, as I said, it is up to you both ;-) Thanks, Mauro _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel