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 59F8EC43334 for ; Wed, 15 Jun 2022 03:40:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1345968AbiFODkK (ORCPT ); Tue, 14 Jun 2022 23:40:10 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53180 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229924AbiFODkJ (ORCPT ); Tue, 14 Jun 2022 23:40:09 -0400 Received: from mx1.cqplus1.com (unknown [113.204.237.245]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 42F2A4D9E9 for ; Tue, 14 Jun 2022 20:40:00 -0700 (PDT) X-MailGates: (flag:1,DYNAMIC,RELAY,NOHOST,LAN:PASS)(compute_score:DELIVE R,40,3) Received: from 172.27.96.203 by mx1.cqplus1.com with MailGates ESMTP Server V5.0(28649:0:AUTH_RELAY) (envelope-from ); Wed, 15 Jun 2022 11:31:54 +0800 (CST) Received: from CQEXMAIL01.cqplus1.com (172.27.96.203) by CQEXMAIL01.cqplus1.com (172.27.96.203) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.6; Wed, 15 Jun 2022 11:31:50 +0800 Received: from CQEXMAIL01.cqplus1.com ([::1]) by CQEXMAIL01.cqplus1.com ([::1]) with mapi id 15.01.2507.006; Wed, 15 Jun 2022 11:31:50 +0800 From: =?utf-8?B?cWluamlhblvopoPlgaVd?= To: Arnd Bergmann CC: Stephen Boyd , Krzysztof Kozlowski , Rob Herring , Michael Turquette , Russell King - ARM Linux , Olof Johansson , SoC Team , Linux ARM , DTML , Linux Kernel Mailing List , linux-clk Subject: RE: [PATCH v18 10/10] ARM: dts: Add Sunplus SP7021-Demo-V3 board device tree Thread-Topic: [PATCH v18 10/10] ARM: dts: Add Sunplus SP7021-Demo-V3 board device tree Thread-Index: AQHYf8r27MMUrQucmUWH+5A1oaj77q1OMQ+AgAGftCA= Date: Wed, 15 Jun 2022 03:31:50 +0000 Message-ID: References: <0574ed0ad1e9ec4c12645fe2d2dde8f701d285e5.1655194858.git.qinjian@cqplus1.com> In-Reply-To: Accept-Language: zh-CN, en-US Content-Language: zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [172.28.110.18] Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-clk@vger.kernel.org PiANCj4gSSdsbCB3YWl0IGZvciBhIGZpbmFsIEFjayBmcm9tIEtyenlzenRvZiBiZWZvcmUgYXBw bHlpbmcgdGhpcy4NCj4gDQo+IA0KPiBBIGZldyBtb3JlIGRldGFpbHMgSSBub3RpY2VkOg0KPiAN Cj4gDQo+ID4gKy8gew0KPiA+ICsgICAgICAgY29tcGF0aWJsZSA9ICJzdW5wbHVzLHNwNzAyMSI7 DQo+ID4gKyAgICAgICBtb2RlbCA9ICJTdW5wbHVzIFNQNzAyMSI7DQo+ID4gKw0KPiA+ICsgICAg ICAgYWxpYXNlcyB7DQo+ID4gKyAgICAgICAgICAgICAgIHNlcmlhbDAgPSAmdWFydDA7DQo+ID4g KyAgICAgICAgICAgICAgIHNlcmlhbDEgPSAmdWFydDE7DQo+ID4gKyAgICAgICAgICAgICAgIHNl cmlhbDIgPSAmdWFydDI7DQo+ID4gKyAgICAgICAgICAgICAgIHNlcmlhbDMgPSAmdWFydDM7DQo+ ID4gKyAgICAgICAgICAgICAgIHNlcmlhbDQgPSAmdWFydDQ7DQo+ID4gKyAgICAgICB9Ow0KPiAN Cj4gU29tZSBvZiB0aGVzZSBhcmUgZGlzYWJsZWQsIHByZXN1bWFibHkgYmVjYXVzZSB0aGV5IGFy ZSBub3QNCj4gYWN0dWFsbHkgY29ubmVjdGVkIG9uIGFsbCBib2FyZHMuIEJldHRlciBtb3ZlIHRo ZSBhbGlhc2VzIHRvIHRoZSAuZHRzIGZpbGUNCj4gYW5kIG9ubHkgbGlzdCB0aGUgcG9ydHMgdGhh dCBhcmUgaW4gZmFjdCB1c2VkLCB1c2luZyB0aGUgbnVtYmVyaW5nIHRoYXQNCj4gbWF0Y2hlcyB0 aGUgbGFiZWxzIG9uIHRoZSBib2FyZCwgbm90IHRoZSBudW1iZXJzIGluc2lkZSBvZiB0aGUgU29D DQo+IA0KPiA+ICsNCj4gPiArICAgICAgIHNvYyB7DQo+ID4gKyAgICAgICAgICAgICAgIGNvbXBh dGlibGUgPSAic2ltcGxlLWJ1cyI7DQo+ID4gKyAgICAgICAgICAgICAgICNhZGRyZXNzLWNlbGxz ID0gPDE+Ow0KPiA+ICsgICAgICAgICAgICAgICAjc2l6ZS1jZWxscyA9IDwxPjsNCj4gPiArICAg ICAgICAgICAgICAgcmFuZ2VzOw0KPiA+ICsgICAgICAgICAgICAgICBpbnRlcnJ1cHQtcGFyZW50 ID0gPCZpbnRjPjsNCj4gPiArDQo+ID4gKyAgICAgICAgICAgICAgIGludGM6IGludGVycnVwdC1j b250cm9sbGVyQDljMDAwNzgwIHsNCj4gPiArICAgICAgICAgICAgICAgICAgICAgICBjb21wYXRp YmxlID0gInN1bnBsdXMsc3A3MDIxLWludGMiOw0KPiA+ICsgICAgICAgICAgICAgICAgICAgICAg IHJlZyA9IDwweDljMDAwNzgwIDB4ODA+LCA8MHg5YzAwMGE4MCAweDgwPjsNCj4gPiArICAgICAg ICAgICAgICAgICAgICAgICBpbnRlcnJ1cHQtY29udHJvbGxlcjsNCj4gPiArICAgICAgICAgICAg ICAgICAgICAgICAjaW50ZXJydXB0LWNlbGxzID0gPDI+Ow0KPiA+ICsgICAgICAgICAgICAgICB9 Ow0KPiANCj4gQWxsIGNoaWxkIG5vZGVzIGJlbG93IC9zb2MgaGF2ZSByZWdpc3RlcnMgaW4gdGhl IDB4OWMwMDAwMDAgcmFuZ2UuIEkgd291bGQNCj4gc3VnZ2VzdCB5b3UgdXNlIGEgbWF0Y2hpbmcg J3JhbmdlcycgcHJvcGVydHkgdG8gdHJhbnNsYXRlIHRoZXNlIGludG8gYQ0KPiB6ZXJvLWJhc2Vk DQo+IGFkZHJlc3MgbGlrZQ0KPiANCj4gL3NvYyB7DQo+ICAgICAgICByYW5nZXMgPSA8MCAweDlj MDAwMDAwIDB4MTAwMDA+Ow0KPiANCj4gICAgICAgaW50ZXJydXB0LWNvbnRyb2xsZXJANzgwIHsN Cj4gICAgICAgICAgICAgIHJlZyA9IDwweDc4MCAweDgwPiwgPDB4YTgwIDB4ODA+Ow0KPiAgICAg ICAgICAgICAgLi4uDQo+ICAgICAgIH07DQo+IH07DQo+IA0KPiA+ICsgICAgICAgICAgICAgICBj bGtjOiBjbG9jay1jb250cm9sbGVyQDljMDAwMDA0IHsNCj4gPiArICAgICAgICAgICAgICAgICAg ICAgICBjb21wYXRpYmxlID0gInN1bnBsdXMsc3A3MDIxLWNsa2MiOw0KPiA+ICsgICAgICAgICAg ICAgICAgICAgICAgIHJlZyA9IDwweDljMDAwMDA0IDB4Mjg+LA0KPiA+ICsgICAgICAgICAgICAg ICAgICAgICAgICAgICAgIDwweDljMDAwMjAwIDB4NDQ+LA0KPiA+ICsgICAgICAgICAgICAgICAg ICAgICAgICAgICAgIDwweDljMDAwMjY4IDB4MDQ+Ow0KPiA+ICsgICAgICAgICAgICAgICAgICAg ICAgIGNsb2NrcyA9IDwmZXh0Y2xrPjsNCj4gPiArICAgICAgICAgICAgICAgICAgICAgICAjY2xv Y2stY2VsbHMgPSA8MT47DQo+ID4gKyAgICAgICAgICAgICAgIH07DQo+ID4gKw0KPiA+ICsgICAg ICAgICAgICAgICByc3RjOiByZXNldEA5YzAwMDA1NCB7DQo+ID4gKyAgICAgICAgICAgICAgICAg ICAgICAgY29tcGF0aWJsZSA9ICJzdW5wbHVzLHNwNzAyMS1yZXNldCI7DQo+ID4gKyAgICAgICAg ICAgICAgICAgICAgICAgcmVnID0gPDB4OWMwMDAwNTQgMHgyOD47DQo+ID4gKyAgICAgICAgICAg ICAgICAgICAgICAgI3Jlc2V0LWNlbGxzID0gPDE+Ow0KPiA+ICsgICAgICAgICAgICAgICB9Ow0K PiANCj4gDQo+IE1heWJlIHNvcnQgdGhlIG5vZGVzIGJ5IGFkZHJlc3MsIG9yIHBvc3NpYmx5IGJ5 IG5hbWUuDQo+IA0KPiAgICAgICAgQXJuZA0KDQpUaGFua3MgZm9yIHlvdXIgY29tbWVudHMsIEkn bGwgZml4IHRoZXNlIGluIG5leHQgcGF0Y2guDQoNCg== 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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 7FF59C43334 for ; Wed, 15 Jun 2022 03:40:55 +0000 (UTC) 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:In-Reply-To:References: Message-ID:Date:Subject:CC:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=xDVqJk3b+g7duFLCNlFQXAmEgXAy1Nad+NqDXyBg1R8=; b=e2n6KrRmP5wd6X yNqDf1K/tXuvsf70CtUueThKfQbFmkGt/D6GHX9KBFgws3w48eX2RtsdyMS6HzwXQlbQYr/kpuwvR BzFwT0hA5NUNXbzMMrA05BgQ6XCTbe4jsrLhMVG71ZccKWf0V1/KhRO3p4petEEfd9rHsijDsSH02 tG53rzoXUXgbmnhvtnZbGQ/9rkOluhtt8yzd0GBVsvafjCkquc/XPuvtxZYhNVIpUkEvC67+iKcmZ raV+e6CtNpVog1oiprT+/whZzWO3WYFvDAnPsZZY82qn3AOibkyyVzmD9JWdcaLkdVQtarsRXEPAM 4m5fsO/FzBAm8Mi/Z2ig==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o1Js5-00CNQZ-EE; Wed, 15 Jun 2022 03:39:09 +0000 Received: from mx1.cqplus1.com ([113.204.237.245]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o1Js1-00CNPo-BS for linux-arm-kernel@lists.infradead.org; Wed, 15 Jun 2022 03:39:07 +0000 X-MailGates: (flag:1,DYNAMIC,RELAY,NOHOST,LAN:PASS)(compute_score:DELIVE R,40,3) Received: from 172.27.96.203 by mx1.cqplus1.com with MailGates ESMTP Server V5.0(28649:0:AUTH_RELAY) (envelope-from ); Wed, 15 Jun 2022 11:31:54 +0800 (CST) Received: from CQEXMAIL01.cqplus1.com (172.27.96.203) by CQEXMAIL01.cqplus1.com (172.27.96.203) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.6; Wed, 15 Jun 2022 11:31:50 +0800 Received: from CQEXMAIL01.cqplus1.com ([::1]) by CQEXMAIL01.cqplus1.com ([::1]) with mapi id 15.01.2507.006; Wed, 15 Jun 2022 11:31:50 +0800 From: =?utf-8?B?cWluamlhblvopoPlgaVd?= To: Arnd Bergmann CC: Stephen Boyd , Krzysztof Kozlowski , Rob Herring , Michael Turquette , Russell King - ARM Linux , Olof Johansson , SoC Team , Linux ARM , DTML , Linux Kernel Mailing List , linux-clk Subject: RE: [PATCH v18 10/10] ARM: dts: Add Sunplus SP7021-Demo-V3 board device tree Thread-Topic: [PATCH v18 10/10] ARM: dts: Add Sunplus SP7021-Demo-V3 board device tree Thread-Index: AQHYf8r27MMUrQucmUWH+5A1oaj77q1OMQ+AgAGftCA= Date: Wed, 15 Jun 2022 03:31:50 +0000 Message-ID: References: <0574ed0ad1e9ec4c12645fe2d2dde8f701d285e5.1655194858.git.qinjian@cqplus1.com> In-Reply-To: Accept-Language: zh-CN, en-US Content-Language: zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [172.28.110.18] MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220614_203905_782182_5750D4AD X-CRM114-Status: GOOD ( 13.93 ) 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 > > I'll wait for a final Ack from Krzysztof before applying this. > > > A few more details I noticed: > > > > +/ { > > + compatible = "sunplus,sp7021"; > > + model = "Sunplus SP7021"; > > + > > + aliases { > > + serial0 = &uart0; > > + serial1 = &uart1; > > + serial2 = &uart2; > > + serial3 = &uart3; > > + serial4 = &uart4; > > + }; > > Some of these are disabled, presumably because they are not > actually connected on all boards. Better move the aliases to the .dts file > and only list the ports that are in fact used, using the numbering that > matches the labels on the board, not the numbers inside of the SoC > > > + > > + soc { > > + compatible = "simple-bus"; > > + #address-cells = <1>; > > + #size-cells = <1>; > > + ranges; > > + interrupt-parent = <&intc>; > > + > > + intc: interrupt-controller@9c000780 { > > + compatible = "sunplus,sp7021-intc"; > > + reg = <0x9c000780 0x80>, <0x9c000a80 0x80>; > > + interrupt-controller; > > + #interrupt-cells = <2>; > > + }; > > All child nodes below /soc have registers in the 0x9c000000 range. I would > suggest you use a matching 'ranges' property to translate these into a > zero-based > address like > > /soc { > ranges = <0 0x9c000000 0x10000>; > > interrupt-controller@780 { > reg = <0x780 0x80>, <0xa80 0x80>; > ... > }; > }; > > > + clkc: clock-controller@9c000004 { > > + compatible = "sunplus,sp7021-clkc"; > > + reg = <0x9c000004 0x28>, > > + <0x9c000200 0x44>, > > + <0x9c000268 0x04>; > > + clocks = <&extclk>; > > + #clock-cells = <1>; > > + }; > > + > > + rstc: reset@9c000054 { > > + compatible = "sunplus,sp7021-reset"; > > + reg = <0x9c000054 0x28>; > > + #reset-cells = <1>; > > + }; > > > Maybe sort the nodes by address, or possibly by name. > > Arnd Thanks for your comments, I'll fix these in next patch. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel