Received: by 2002:a05:6358:9144:b0:117:f937:c515 with SMTP id r4csp2125208rwr; Sat, 6 May 2023 05:27:37 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ7QVYOeb7xyIalOrKP60WLSeTraim6S0c9R1CJH8QKwZXKUzDYmRt15MpEfcH5PTeLY2J5U X-Received: by 2002:a17:903:1d1:b0:19a:bbd0:c5cc with SMTP id e17-20020a17090301d100b0019abbd0c5ccmr4749863plh.60.1683376056679; Sat, 06 May 2023 05:27:36 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1683376056; cv=none; d=google.com; s=arc-20160816; b=gjFZGMsuSpl4ylz3o+E6w3CEEM9K5kBGxFDE8RVgW/LnUXIjNfc/Pzisn0WQBhSQC5 2H/RmNwG7KEsp4JC+Jrm9awteSwZ/TDcfQpmAuksccCIuKgxDHegRKPvK3gRy+N0StvZ m3A4U7+oGODdvwEyt69OPyjWenqT7Qmr9EqPek4px5ULEmwvxvcO99EU6jk6KLup1Wi1 gugVwyDIGRLUB3XT+oeh3dhWRw31dT6Sa9w4+XwyKGe8u7EaCHmox1iW6tB7JklH8/KL K6OE5ggdyndFMDOfGElSWx2njvgyeMW0Hbn/eu0nqzs8MdEGBykJAeB5GT5Ep5DDiPKS zrvQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id; bh=Pu7uvs3eJPt2xGbCYT+ltwjwlF7JURooj2lP9rNhXRU=; b=If5n6R7o2XBxi0rVjpgLF/fwlsmKwF1jDVqJBIE1bJFIGi5xNJW+xolPqw+j1RSN0f w7n2NrJG8U5B6oGQPnVK/cywlhRaQDxWCGA5MBDAioUno/muWUKujzBsefbp+auXQKP/ WduotjugSuN0OuvIp6wtgZ9Lsj6r6raOg9fcVk9uTAQtLYErQl2jpfSCDfoO10Gotz93 h2npRj9YzvyJDAdzu1g/wyhIQUxEtas+2p/aGeKOYauyd5EoTT+DndFZhZOXmpktJC3Z haPGAhz+lbDdONWlcT2DFDZHMLUZipBLNYhyXfW73ARp2156UigiBH7js36dUqgcOilg R6qA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id f7-20020a170902860700b0019f25dae4f5si3632421plo.198.2023.05.06.05.27.21; Sat, 06 May 2023 05:27:36 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232208AbjEFM0X (ORCPT + 99 others); Sat, 6 May 2023 08:26:23 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52786 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232273AbjEFM0K (ORCPT ); Sat, 6 May 2023 08:26:10 -0400 Received: from ex01.ufhost.com (ex01.ufhost.com [61.152.239.75]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id AFE39150E8; Sat, 6 May 2023 05:26:04 -0700 (PDT) Received: from EXMBX166.cuchost.com (unknown [175.102.18.54]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "EXMBX166", Issuer "EXMBX166" (not verified)) by ex01.ufhost.com (Postfix) with ESMTP id 953FA24E272; Sat, 6 May 2023 20:26:02 +0800 (CST) Received: from EXMBX062.cuchost.com (172.16.6.62) by EXMBX166.cuchost.com (172.16.6.76) with Microsoft SMTP Server (TLS) id 15.0.1497.42; Sat, 6 May 2023 20:26:02 +0800 Received: from [192.168.125.107] (183.27.98.219) by EXMBX062.cuchost.com (172.16.6.62) with Microsoft SMTP Server (TLS) id 15.0.1497.42; Sat, 6 May 2023 20:26:01 +0800 Message-ID: Date: Sat, 6 May 2023 20:26:01 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 Subject: Re: [RESEND v2 1/6] dt-bindings: power: Add JH7110 AON PMU support Content-Language: en-US To: Conor Dooley CC: Conor Dooley , Krzysztof Kozlowski , Rob Herring , "Krzysztof Kozlowski" , Emil Renner Berthing , Paul Walmsley , Palmer Dabbelt , Albert Ou , Walker Chen , Hal Feng , , , , , References: <089e24d1-588a-4a56-f00b-0b35d1d99295@linaro.org> <1ac26c1a-1726-515d-6598-849a07ed0b86@linaro.org> <5adda0ad-965c-fbf0-878c-9d41d28b5c39@starfivetech.com> <86693969-59bf-5bcc-42a3-b6e94a0d6f3e@linaro.org> <20230504-worshiper-ongoing-5581e1f2c2c4@wendy> <2f473307-2219-61a4-fa66-5848fe566cf0@starfivetech.com> <20230505-magician-poet-724c96020c2f@wendy> <20230506-dating-twiddling-b364de21ed2b@spud> From: Changhuang Liang In-Reply-To: <20230506-dating-twiddling-b364de21ed2b@spud> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Originating-IP: [183.27.98.219] X-ClientProxiedBy: EXCAS066.cuchost.com (172.16.6.26) To EXMBX062.cuchost.com (172.16.6.62) X-YovoleRuleAgent: yovoleflag X-Spam-Status: No, score=-6.2 required=5.0 tests=BAYES_00,NICE_REPLY_A, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2023/5/6 18:17, Conor Dooley wrote: > On Sat, May 06, 2023 at 09:45:07AM +0800, Changhuang Liang wrote: >> >> >> On 2023/5/5 20:38, Conor Dooley wrote: >>> On Fri, May 05, 2023 at 09:29:15AM +0800, Changhuang Liang wrote: >>> >>>> But if keep this "starfive,jh7110-aon-syscon" compatible. Which .yaml match to >>>> it? Use this series dt-bindings or syscon series dt-bindings. >>> >>> There is no syscon series anymore, it's part of the PLL series now: >>> https://lore.kernel.org/linux-clk/20230414024157.53203-1-xingyu.wu@starfivetech.com/ >>> >>> I don't really care what you, Walker & Xingyu decide to do, but add the >>> binding in one series in a complete form. It's far less confusing to >>> have only have one version of the binding on the go at once. >>> >> >> Due to the current aon pmu needs to be adjusted, it affects the syscon in PLL series. >> So It's inevitable to change syscon in PLL series. >> >> My current idea is PLL series don't add the aon_syscon node. I will add it in my >> aon pmu series in next version > > That's fine. Rob was happy with the clock related parts, which was the > original source of confusion there. > >> like this: >> >> aon_syscon: syscon@17010000 { >> compatible = "starfive,jh7110-aon-pmu", "syscon"; > > The syscon does a bunch of things of which one is a pmu. I don't see a > reason to name this other than "starfive,jh100-aon-syscon". > OK, will replace "starfive,jh7110-aon-pmu" with "starfive,jh100-aon-syscon" in this series. Thanks, Changhuang