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 A3CC4C0015E for ; Wed, 19 Jul 2023 06:07:00 +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:In-Reply-To:MIME-Version:References: 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=aZf+VjnET4fPywodoT1a6C+x9SBXPnmE/AnVLf7nWkQ=; b=ZJHUTvDoa7gp/k iqX3U09VbarE89NUxN/AVdFbgSmj5N3h8i6kafAwS+5jsFZbKWPEfkiHLdIpg8cejmRdg5tCQ9a7v caHTMdB8gNGUQnUJmgfNMBDox0iGzmYb1ZDQSsDpvSfPMEPvaB45b3e8tc6xR2JJhIHkQBr28LNKf dor5fvK1/PIFmwcl8Ov4imDGobHBfUJbPVJMZAdr8Tijf2d2JxHwUPXLCkUY8tTzgMWcw3nPeaau0 UkVVy5tKK1WEawLOzqstrAbffI1ZGbSSbwe7kXtX/qQJegonEFpZX29csILZ5z6X8muuHWlnG+Zxq 3MuwY6UHmmYsja0jAW/g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qM0Ke-005Y66-0H; Wed, 19 Jul 2023 06:06:40 +0000 Received: from desiato.infradead.org ([90.155.92.199]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qM0KZ-005Y3y-2l for linux-arm-kernel@bombadil.infradead.org; Wed, 19 Jul 2023 06:06:36 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=In-Reply-To:Content-Transfer-Encoding: Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Sender:Reply-To:Content-ID:Content-Description; bh=WDdlGti6p/avA7z7hsRpCKJ7gi9p9GsqIKK7OGoj/EY=; b=ApCdT791IZzG4jeZXlAYd8uWU0 +RlfRpt8IQm3gIyCo9LB6bd8XDamYe88bYHVTdWp/68Wv4mkFC3YsjY9aMSKxZTK0gKbRc2LUMuNG u1w13zS/dRmMVcUbQ2ouujJP9/SlBUWL2ckYYd8jkij5nhODv9mF84JCh5WJb09D0ImYhRVepol0h iJFPvhfCJCxULv4wB8ZXYnvhpozZ9iCK3p5+dD6jvksGKE8+OBA/Mp6mUzvbqBLabEIzyWmuHn0Ih Me1Y3fZALKeLM1pI5MYknVjArRemw1WDeTrYdqC6sLsGjGiBbBvAwLzrG3DGPe2XQDDAYSCLIpd4B ZSE/NxTw==; Received: from dfw.source.kernel.org ([139.178.84.217]) by desiato.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qLwSC-00CRS0-3A for linux-arm-kernel@lists.infradead.org; Wed, 19 Jul 2023 01:58:17 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id E4A0161626; Wed, 19 Jul 2023 01:58:03 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 56C73C433C9; Wed, 19 Jul 2023 01:57:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1689731883; bh=RawEnqc1EkIgdkG2i7me4l9fFj/6mUCWCz+xGcSTwAc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Q7a7oEr8cy6h2A3djds9QBxY5rCG1G+E9+SWbmSrMqgK/lMSCQqJOb3xV05iUsCoN nN6poCxArph65dloIeJ0lF/SQIjMWHOg/QGU9+9zKLscwn/9IyVDUEPKN3JGOUtYxb ypkBXycMSAg6oIOYs243x+kWrPKEsXsYtRMpjL0vtYqc9xY9U7Pa8+/mIqgxE6IGz4 bdqzzqjWpHwNci/K3JKCV9qOeABLl36BBNRZF2eIpp7L8J4+vkPY3H4qBmGyZlZh8F kTbnIpsumRmJMlPnAKBna4b9QoBYH+7ZXgVIaMa0XzboSZDIikXDlO44OiXiTWJbYi th22DTt01xFtQ== Date: Wed, 19 Jul 2023 09:57:50 +0800 From: Shawn Guo To: Tim Harvey Cc: linux-arm-kernel@lists.infradead.org, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] arm64: dts: freescale: Add imx8mp-venice-gw71xx-2x Message-ID: <20230719015750.GR9559@dragon> References: <20230607170724.2016988-1-tharvey@gateworks.com> <20230607170724.2016988-2-tharvey@gateworks.com> <20230717002717.GI9559@dragon> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230719_025814_914861_C28C3F35 X-CRM114-Status: GOOD ( 28.51 ) 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="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org T24gVHVlLCBKdWwgMTgsIDIwMjMgYXQgMDI6MTY6MjJQTSAtMDcwMCwgVGltIEhhcnZleSB3cm90 ZToKPiBPbiBTdW4sIEp1bCAxNiwgMjAyMyBhdCA1OjI34oCvUE0gU2hhd24gR3VvIDxzaGF3bmd1 b0BrZXJuZWwub3JnPiB3cm90ZToKPiA+Cj4gPiBPbiBXZWQsIEp1biAwNywgMjAyMyBhdCAxMDow NzoyNEFNIC0wNzAwLCBUaW0gSGFydmV5IHdyb3RlOgo+ID4gPiBUaGUgR2F0ZXdvcmtzIGlteDht cC12ZW5pY2UtZ3c3MXh4LTJ4IGNvbnNpc3RzIG9mIGEgU09NICsgYmFzZWJvYXJkLgo+ID4gPgo+ ID4gPiBUaGUgR1c3MDJ4IFNPTSBjb250YWlucyB0aGUgZm9sbG93aW5nOgo+ID4gPiAgLSBpLk1Y OE0gUGx1cyBTb0MKPiA+ID4gIC0gTFBERFI0IG1lbW9yeQo+ID4gPiAgLSBlTU1DIEJvb3QgZGV2 aWNlCj4gPiA+ICAtIEdhdGV3b3JrcyBTeXN0ZW0gQ29udHJvbGxlciAoR1NDKSB3aXRoIGludGVn cmF0ZWQgRUVQUk9NLCBidXR0b24KPiA+ID4gICAgY29udHJvbGxlciwgYW5kIEFEQydzCj4gPiA+ ICAtIFBNSUMKPiA+ID4gIC0gU09NIGNvbm5lY3RvciBwcm92aWRpbmc6Cj4gPiA+ICAgLSBlUW9T IEdiRSBNSUkKPiA+ID4gICAtIDF4IFNQSQo+ID4gPiAgIC0gMnggSTJDCj4gPiA+ICAgLSA0eCBV QVJUCj4gPiA+ICAgLSAyeCBVU0IgMy4wCj4gPiA+ICAgLSAxeCBQQ0kKPiA+ID4gICAtIDF4IFNE SU8gKDQtYml0IDMuM1YpCj4gPiA+ICAgLSAxeCBTRElPICg0LWJpdCAzLjNWLzEuOFYpCj4gPiA+ ICAgLSBHUElPCj4gPiA+Cj4gPiA+IFRoZSBHVzcxeHggQmFzZWJvYXJkIGNvbnRhaW5zIHRoZSBm b2xsb3dpbmc6Cj4gPiA+ICAtIEdQUwo+ID4gPiAgLSBSSjQ1IEdiRSAoZVFvUykKPiA+ID4gIC0g b2ZmLWJvYXJkIEkvTyBjb25uZWN0b3Igd2l0aCBVQVJULCBJMkMsIFNQSSwgR1BJTwo+ID4gPiAg LSBFRVJQT00KPiA+ID4gIC0gUENJZSBjbG9jayBnZW5lcmF0b3IKPiA+ID4gIC0gZnVsbC1sZW5n dGggbWluaVBDSWUgc29ja2V0IHdpdGggUENJL1VTQjMgKHZpYSBtdXgpIGFuZCBVU0IyLjAKPiA+ ID4gIC0gVVNCIFR5cGUtQyB3aXRoIFVTQiAyLjAgaG9zdCBhbmQgcGVyaXBoZXJhbCBzdXBwb3J0 Cj4gPiA+ICAtIFdpZGUgcmFuZ2UgREMgaW5wdXQgc3VwcGx5Cj4gPiA+Cj4gPiA+IFNpZ25lZC1v ZmYtYnk6IFRpbSBIYXJ2ZXkgPHRoYXJ2ZXlAZ2F0ZXdvcmtzLmNvbT4KPiA+ID4gLS0tCj4gPiA+ ICBhcmNoL2FybTY0L2Jvb3QvZHRzL2ZyZWVzY2FsZS9NYWtlZmlsZSAgICAgICAgfCAgIDEgKwo+ ID4gPiAgLi4uL2R0cy9mcmVlc2NhbGUvaW14OG1wLXZlbmljZS1ndzcxeHgtMnguZHRzIHwgIDE5 ICsrCj4gPiA+ICAuLi4vZHRzL2ZyZWVzY2FsZS9pbXg4bXAtdmVuaWNlLWd3NzF4eC5kdHNpICAg fCAyMzYgKysrKysrKysrKysrKysrKysrCj4gPiA+ICAzIGZpbGVzIGNoYW5nZWQsIDI1NiBpbnNl cnRpb25zKCspCj4gPiA+ICBjcmVhdGUgbW9kZSAxMDA2NDQgYXJjaC9hcm02NC9ib290L2R0cy9m cmVlc2NhbGUvaW14OG1wLXZlbmljZS1ndzcxeHgtMnguZHRzCj4gPiA+ICBjcmVhdGUgbW9kZSAx MDA2NDQgYXJjaC9hcm02NC9ib290L2R0cy9mcmVlc2NhbGUvaW14OG1wLXZlbmljZS1ndzcxeHgu ZHRzaQo+ID4gPgo+ID4gPiBkaWZmIC0tZ2l0IGEvYXJjaC9hcm02NC9ib290L2R0cy9mcmVlc2Nh bGUvTWFrZWZpbGUgYi9hcmNoL2FybTY0L2Jvb3QvZHRzL2ZyZWVzY2FsZS9NYWtlZmlsZQo+ID4g PiBpbmRleCA2MmYyMmI3N2IzOGIuLmIzYmI4MjNkMDE2OCAxMDA2NDQKPiA+ID4gLS0tIGEvYXJj aC9hcm02NC9ib290L2R0cy9mcmVlc2NhbGUvTWFrZWZpbGUKPiA+ID4gKysrIGIvYXJjaC9hcm02 NC9ib290L2R0cy9mcmVlc2NhbGUvTWFrZWZpbGUKPiA+ID4gQEAgLTEwMSw2ICsxMDEsNyBAQCBk dGItJChDT05GSUdfQVJDSF9NWEMpICs9IGlteDhtcC1pY29yZS1teDhtcC1lZGltbTIuMi5kdGIK PiA+ID4gIGR0Yi0kKENPTkZJR19BUkNIX01YQykgKz0gaW14OG1wLW1zYy1zbTJzLWVwMS5kdGIK PiA+ID4gIGR0Yi0kKENPTkZJR19BUkNIX01YQykgKz0gaW14OG1wLXBoeWJvYXJkLXBvbGx1eC1y ZGsuZHRiCj4gPiA+ICBkdGItJChDT05GSUdfQVJDSF9NWEMpICs9IGlteDhtcC10cW1hOG1wcWwt bWJhOG1weGwuZHRiCj4gPiA+ICtkdGItJChDT05GSUdfQVJDSF9NWEMpICs9IGlteDhtcC12ZW5p Y2UtZ3c3MXh4LTJ4LmR0Ygo+ID4gPiAgZHRiLSQoQ09ORklHX0FSQ0hfTVhDKSArPSBpbXg4bXAt dmVuaWNlLWd3NzR4eC5kdGIKPiA+ID4gIGR0Yi0kKENPTkZJR19BUkNIX01YQykgKz0gaW14OG1w LXZlbmljZS1ndzc5MDUtMnguZHRiCj4gPiA+ICBkdGItJChDT05GSUdfQVJDSF9NWEMpICs9IGlt eDhtcC12ZXJkaW4tbm9ud2lmaS1kYWhsaWEuZHRiCj4gPiA+IGRpZmYgLS1naXQgYS9hcmNoL2Fy bTY0L2Jvb3QvZHRzL2ZyZWVzY2FsZS9pbXg4bXAtdmVuaWNlLWd3NzF4eC0yeC5kdHMgYi9hcmNo L2FybTY0L2Jvb3QvZHRzL2ZyZWVzY2FsZS9pbXg4bXAtdmVuaWNlLWd3NzF4eC0yeC5kdHMKPiA+ ID4gbmV3IGZpbGUgbW9kZSAxMDA2NDQKPiA+ID4gaW5kZXggMDAwMDAwMDAwMDAwLi41MzEyMGZj OWNkN2YKPiA+ID4gLS0tIC9kZXYvbnVsbAo+ID4gPiArKysgYi9hcmNoL2FybTY0L2Jvb3QvZHRz L2ZyZWVzY2FsZS9pbXg4bXAtdmVuaWNlLWd3NzF4eC0yeC5kdHMKPiA+ID4gQEAgLTAsMCArMSwx OSBAQAo+ID4gPiArLy8gU1BEWC1MaWNlbnNlLUlkZW50aWZpZXI6IChHUEwtMi4wKyBPUiBNSVQp Cj4gPiA+ICsvKgo+ID4gPiArICogQ29weXJpZ2h0IDIwMjMgR2F0ZXdvcmtzIENvcnBvcmF0aW9u Cj4gPiA+ICsgKi8KPiA+ID4gKwo+ID4gPiArL2R0cy12MS87Cj4gPiA+ICsKPiA+ID4gKyNpbmNs dWRlICJpbXg4bXAuZHRzaSIKPiA+ID4gKyNpbmNsdWRlICJpbXg4bXAtdmVuaWNlLWd3NzAyeC5k dHNpIgo+ID4gPiArI2luY2x1ZGUgImlteDhtcC12ZW5pY2UtZ3c3MXh4LmR0c2kiCj4gPiA+ICsK PiA+ID4gKy8gewo+ID4gPiArICAgICBtb2RlbCA9ICJHYXRld29ya3MgVmVuaWNlIEdXNzF4eC0y eCBpLk1YOE1QIERldmVsb3BtZW50IEtpdCI7Cj4gPiA+ICsgICAgIGNvbXBhdGlibGUgPSAiZ2F0 ZXdvcmtzLGlteDhtcC1ndzcxeHgtMngiLCAiZnNsLGlteDhtcCI7Cj4gPiA+ICsKPiA+ID4gKyAg ICAgY2hvc2VuIHsKPiA+ID4gKyAgICAgICAgICAgICBzdGRvdXQtcGF0aCA9ICZ1YXJ0MjsKPiA+ ID4gKyAgICAgfTsKPiA+ID4gK307Cj4gPiA+IGRpZmYgLS1naXQgYS9hcmNoL2FybTY0L2Jvb3Qv ZHRzL2ZyZWVzY2FsZS9pbXg4bXAtdmVuaWNlLWd3NzF4eC5kdHNpIGIvYXJjaC9hcm02NC9ib290 L2R0cy9mcmVlc2NhbGUvaW14OG1wLXZlbmljZS1ndzcxeHguZHRzaQo+ID4gPiBuZXcgZmlsZSBt b2RlIDEwMDY0NAo+ID4gPiBpbmRleCAwMDAwMDAwMDAwMDAuLjg2OTk5ZjUyZDRiMgo+ID4gPiAt LS0gL2Rldi9udWxsCj4gPiA+ICsrKyBiL2FyY2gvYXJtNjQvYm9vdC9kdHMvZnJlZXNjYWxlL2lt eDhtcC12ZW5pY2UtZ3c3MXh4LmR0c2kKPiA+ID4gQEAgLTAsMCArMSwyMzYgQEAKPiA+ID4gKy8v IFNQRFgtTGljZW5zZS1JZGVudGlmaWVyOiAoR1BMLTIuMCsgT1IgTUlUKQo+ID4gPiArLyoKPiA+ ID4gKyAqIENvcHlyaWdodCAyMDIzIEdhdGV3b3JrcyBDb3Jwb3JhdGlvbgo+ID4gPiArICovCj4g PiA+ICsKPiA+ID4gKyNpbmNsdWRlIDxkdC1iaW5kaW5ncy9ncGlvL2dwaW8uaD4KPiA+ID4gKyNp bmNsdWRlIDxkdC1iaW5kaW5ncy9sZWRzL2NvbW1vbi5oPgo+ID4gPiArI2luY2x1ZGUgPGR0LWJp bmRpbmdzL3BoeS9waHktaW14OC1wY2llLmg+Cj4gPiA+ICsKPiA+ID4gKy8gewo+ID4gPiArICAg ICBsZWQtY29udHJvbGxlciB7Cj4gPiA+ICsgICAgICAgICAgICAgY29tcGF0aWJsZSA9ICJncGlv LWxlZHMiOwo+ID4gPiArICAgICAgICAgICAgIHBpbmN0cmwtbmFtZXMgPSAiZGVmYXVsdCI7Cj4g PiA+ICsgICAgICAgICAgICAgcGluY3RybC0wID0gPCZwaW5jdHJsX2dwaW9fbGVkcz47Cj4gPiA+ ICsKPiA+ID4gKyAgICAgICAgICAgICBsZWQtMCB7Cj4gPiA+ICsgICAgICAgICAgICAgICAgICAg ICBmdW5jdGlvbiA9IExFRF9GVU5DVElPTl9TVEFUVVM7Cj4gPiA+ICsgICAgICAgICAgICAgICAg ICAgICBjb2xvciA9IDxMRURfQ09MT1JfSURfR1JFRU4+Owo+ID4gPiArICAgICAgICAgICAgICAg ICAgICAgZ3Bpb3MgPSA8JmdwaW80IDEgR1BJT19BQ1RJVkVfSElHSD47Cj4gPiA+ICsgICAgICAg ICAgICAgICAgICAgICBkZWZhdWx0LXN0YXRlID0gIm9uIjsKPiA+ID4gKyAgICAgICAgICAgICAg ICAgICAgIGxpbnV4LGRlZmF1bHQtdHJpZ2dlciA9ICJoZWFydGJlYXQiOwo+ID4gPiArICAgICAg ICAgICAgIH07Cj4gPiA+ICsKPiA+ID4gKyAgICAgICAgICAgICBsZWQtMSB7Cj4gPiA+ICsgICAg ICAgICAgICAgICAgICAgICBmdW5jdGlvbiA9IExFRF9GVU5DVElPTl9TVEFUVVM7Cj4gPiA+ICsg ICAgICAgICAgICAgICAgICAgICBjb2xvciA9IDxMRURfQ09MT1JfSURfUkVEPjsKPiA+ID4gKyAg ICAgICAgICAgICAgICAgICAgIGdwaW9zID0gPCZncGlvNCA1IEdQSU9fQUNUSVZFX0hJR0g+Owo+ ID4gPiArICAgICAgICAgICAgICAgICAgICAgZGVmYXVsdC1zdGF0ZSA9ICJvZmYiOwo+ID4gPiAr ICAgICAgICAgICAgIH07Cj4gPiA+ICsgICAgIH07Cj4gPiA+ICsKPiA+ID4gKyAgICAgcGNpZTBf cmVmY2xrOiBwY2llMC1yZWZjbGsgewo+ID4KPiA+IENhbiB3ZSBuYW1lIHRoZSBub2RlIGNsb2Nr LXh4eD8KPiA+Cj4gPiA+ICsgICAgICAgICAgICAgY29tcGF0aWJsZSA9ICJmaXhlZC1jbG9jayI7 Cj4gPiA+ICsgICAgICAgICAgICAgI2Nsb2NrLWNlbGxzID0gPDA+Owo+ID4gPiArICAgICAgICAg ICAgIGNsb2NrLWZyZXF1ZW5jeSA9IDwxMDAwMDAwMDA+Owo+ID4gPiArICAgICB9Owo+ID4gPiAr Cj4gPiA+ICsgICAgIHBwcyB7Cj4gPiA+ICsgICAgICAgICAgICAgY29tcGF0aWJsZSA9ICJwcHMt Z3BpbyI7Cj4gPiA+ICsgICAgICAgICAgICAgcGluY3RybC1uYW1lcyA9ICJkZWZhdWx0IjsKPiA+ ID4gKyAgICAgICAgICAgICBwaW5jdHJsLTAgPSA8JnBpbmN0cmxfcHBzPjsKPiA+ID4gKyAgICAg ICAgICAgICBncGlvcyA9IDwmZ3BpbzQgMyBHUElPX0FDVElWRV9ISUdIPjsKPiA+ID4gKyAgICAg ICAgICAgICBzdGF0dXMgPSAib2theSI7Cj4gPiA+ICsgICAgIH07Cj4gPiA+ICt9Owo+ID4gPiAr Cj4gPiA+ICsvKiBvZmYtYm9hcmQgaGVhZGVyICovCj4gPiA+ICsmZWNzcGkyIHsKPiA+ID4gKyAg ICAgcGluY3RybC1uYW1lcyA9ICJkZWZhdWx0IjsKPiA+ID4gKyAgICAgcGluY3RybC0wID0gPCZw aW5jdHJsX3NwaTI+Owo+ID4gPiArICAgICBjcy1ncGlvcyA9IDwmZ3BpbzUgMTMgR1BJT19BQ1RJ VkVfTE9XPjsKPiA+ID4gKyAgICAgc3RhdHVzID0gIm9rYXkiOwo+ID4gPiArfTsKPiA+ID4gKwo+ ID4gPiArJmdwaW80IHsKPiA+ID4gKyAgICAgZ3Bpby1saW5lLW5hbWVzID0KPiA+ID4gKyAgICAg ICAgICAgICAiIiwgIiIsICIiLCAiIiwKPiA+ID4gKyAgICAgICAgICAgICAiIiwgIiIsICIiLCAi IiwKPiA+ID4gKyAgICAgICAgICAgICAiZGlvMSIsICIiLCAiIiwgImRpbzAiLAo+ID4gPiArICAg ICAgICAgICAgICIiLCAiIiwgInBjaV91c2Jfc2VsIiwgIiIsCj4gPiA+ICsgICAgICAgICAgICAg IiIsICIiLCAiIiwgIiIsCj4gPiA+ICsgICAgICAgICAgICAgIiIsICIiLCAiIiwgIiIsCj4gPiA+ ICsgICAgICAgICAgICAgImRpbzMiLCAiIiwgImRpbzIiLCAiIiwKPiA+ID4gKyAgICAgICAgICAg ICAicGNpX3dkaXMjIiwgIiIsICIiLCAiIjsKPiA+ID4gK307Cj4gPiA+ICsKPiA+ID4gKyZpMmMy IHsKPiA+ID4gKyAgICAgY2xvY2stZnJlcXVlbmN5ID0gPDQwMDAwMD47Cj4gPiA+ICsgICAgIHBp bmN0cmwtbmFtZXMgPSAiZGVmYXVsdCI7Cj4gPiA+ICsgICAgIHBpbmN0cmwtMCA9IDwmcGluY3Ry bF9pMmMyPjsKPiA+ID4gKyAgICAgc3RhdHVzID0gIm9rYXkiOwo+ID4gPiArCj4gPiA+ICsgICAg IGFjY2VsZXJvbWV0ZXJAMTkgewo+ID4gPiArICAgICAgICAgICAgIHBpbmN0cmwtbmFtZXMgPSAi ZGVmYXVsdCI7Cj4gPiA+ICsgICAgICAgICAgICAgcGluY3RybC0wID0gPCZwaW5jdHJsX2FjY2Vs PjsKPiA+ID4gKyAgICAgICAgICAgICBjb21wYXRpYmxlID0gInN0LGxpczJkZTEyIjsKPiA+ID4g KyAgICAgICAgICAgICByZWcgPSA8MHgxOT47Cj4gPgo+ID4gQ2FuIHdlIHN0YXJ0IHRoZSBwcm9w ZXJ0eSBsaXN0IGZyb20gdGhlc2UgdHdvPwo+ID4KPiA+ID4gKyAgICAgICAgICAgICBzdCxkcmR5 LWludC1waW4gPSA8MT47Cj4gPiA+ICsgICAgICAgICAgICAgaW50ZXJydXB0LXBhcmVudCA9IDwm Z3BpbzQ+Owo+ID4gPiArICAgICAgICAgICAgIGludGVycnVwdHMgPSA8MjEgSVJRX1RZUEVfTEVW RUxfTE9XPjsKPiA+ID4gKyAgICAgICAgICAgICBpbnRlcnJ1cHQtbmFtZXMgPSAiSU5UMSI7Cj4g PiA+ICsgICAgIH07Cj4gPiA+ICt9Owo+ID4gPiArCj4gPiA+ICsmcGNpZV9waHkgewo+ID4gPiAr ICAgICBmc2wscmVmY2xrLXBhZC1tb2RlID0gPElNWDhfUENJRV9SRUZDTEtfUEFEX0lOUFVUPjsK PiA+ID4gKyAgICAgZnNsLGNsa3JlcS11bnN1cHBvcnRlZDsKPiA+ID4gKyAgICAgY2xvY2tzID0g PCZwY2llMF9yZWZjbGs+Owo+ID4gPiArICAgICBjbG9jay1uYW1lcyA9ICJyZWYiOwo+ID4gPiAr ICAgICBzdGF0dXMgPSAib2theSI7Cj4gPiA+ICt9Owo+ID4gPiArCj4gPiA+ICsmcGNpZSB7Cj4g PiA+ICsgICAgIHBpbmN0cmwtbmFtZXMgPSAiZGVmYXVsdCI7Cj4gPiA+ICsgICAgIHBpbmN0cmwt MCA9IDwmcGluY3RybF9wY2llMD47Cj4gPiA+ICsgICAgIHJlc2V0LWdwaW8gPSA8JmdwaW80IDI5 IEdQSU9fQUNUSVZFX0xPVz47Cj4gPiA+ICsgICAgIHN0YXR1cyA9ICJva2F5IjsKPiA+ID4gK307 Cj4gPiA+ICsKPiA+ID4gKy8qIEdQUyAqLwo+ID4gPiArJnVhcnQxIHsKPiA+ID4gKyAgICAgcGlu Y3RybC1uYW1lcyA9ICJkZWZhdWx0IjsKPiA+ID4gKyAgICAgcGluY3RybC0wID0gPCZwaW5jdHJs X3VhcnQxPjsKPiA+ID4gKyAgICAgc3RhdHVzID0gIm9rYXkiOwo+ID4gPiArfTsKPiA+ID4gKwo+ ID4gPiArLyogb2ZmLWJvYXJkIGhlYWRlciAqLwo+ID4gPiArJnVhcnQzIHsKPiA+ID4gKyAgICAg cGluY3RybC1uYW1lcyA9ICJkZWZhdWx0IjsKPiA+ID4gKyAgICAgcGluY3RybC0wID0gPCZwaW5j dHJsX3VhcnQzPjsKPiA+ID4gKyAgICAgc3RhdHVzID0gIm9rYXkiOwo+ID4gPiArfTsKPiA+ID4g Kwo+ID4gPiArLyogVVNCMSBUeXBlLUMgZnJvbnQgcGFuZWwgKi8KPiA+ID4gKyZ1c2IzXzAgewo+ ID4gPiArICAgICBwaW5jdHJsLW5hbWVzID0gImRlZmF1bHQiOwo+ID4gPiArICAgICBwaW5jdHJs LTAgPSA8JnBpbmN0cmxfdXNiMT47Cj4gPiA+ICsgICAgIGZzbCxvdmVyLWN1cnJlbnQtYWN0aXZl LWxvdzsKPiA+ID4gKyAgICAgc3RhdHVzID0gIm9rYXkiOwo+ID4gPiArfTsKPiA+ID4gKwo+ID4g PiArJnVzYjNfcGh5MCB7Cj4gPiA+ICsgICAgIHN0YXR1cyA9ICJva2F5IjsKPiA+ID4gK307Cj4g PiA+ICsKPiA+ID4gKyZ1c2JfZHdjM18wIHsKPiA+ID4gKyAgICAgLyogZHVhbCByb2xlIGlzIGlt cGxlbWVudGVkIGJ1dCBub3QgYSBmdWxsIGZlYXR1cmVkIE9URyAqLwo+ID4gPiArICAgICBhZHAt ZGlzYWJsZTsKPiA+ID4gKyAgICAgaG5wLWRpc2FibGU7Cj4gPiA+ICsgICAgIHNycC1kaXNhYmxl Owo+ID4gPiArICAgICBkcl9tb2RlID0gIm90ZyI7Cj4gPiA+ICsgICAgIHVzYi1yb2xlLXN3aXRj aDsKPiA+ID4gKyAgICAgcm9sZS1zd2l0Y2gtZGVmYXVsdC1tb2RlID0gInBlcmlwaGVyYWwiOwo+ ID4gPiArICAgICBzdGF0dXMgPSAib2theSI7Cj4gPiA+ICsKPiA+ID4gKyAgICAgY29ubmVjdG9y IHsKPiA+ID4gKyAgICAgICAgICAgICBwaW5jdHJsLW5hbWVzID0gImRlZmF1bHQiOwo+ID4gPiAr ICAgICAgICAgICAgIHBpbmN0cmwtMCA9IDwmcGluY3RybF91c2Jjb24xPjsKPiA+ID4gKyAgICAg ICAgICAgICBjb21wYXRpYmxlID0gImdwaW8tdXNiLWItY29ubmVjdG9yIiwgInVzYi1iLWNvbm5l Y3RvciI7Cj4gPgo+ID4gU3RhcnQgdGhlIHByb3BlcnRpZXMgZnJvbSAnY29tcGF0aWJsZSc/Cj4g Pgo+ID4gU2hhd24KPiA+Cj4gCj4gSGkgU2hhd24sCj4gCj4gTXkgbm90ZXMgc2F5IHRoYXQgcGlu Y3RybCBzaG91bGQgY29tZSBmaXJzdCAoaWYgbmVlZGVkKSBmb2xsb3dlZCBieQo+IGNvbXBhdGli bGUsIHJlZywgb3RoZXIgcHJvcHMsIGFuZCBmaW5hbGx5IHN0YXR1cyAoaWYgbmVlZGVkKS4gQXNz dW1pbmcKPiBJJ20gd3JvbmcgYWJvdXQgdGhhdCB3aGVyZSBpbiB0aGUgbGlzdCBzaG91bGQgcGlu Y3RybCBiZSBhbmQgaXMgdGhpcwo+IGRvY3VtZW50ZWQgYW55d2hlcmUgZm9yIGZ1dHVyZSByZWZl cmVuY2U/CgpJdCdzIG1vcmUgYSByZWNvbW1lbmRlZCBpZGlvbSB0aGFuIGRvY3VtZW50ZWQvaGFy ZCBydWxlLgoKLSBjb21wYXRpYmxlCi0gcmVnCi0gZ2VuZXJhbCBwcm9wZXJ0aWVzCi0gdmVuZG9y IHNwZWNpZmljIHByb3BlcnRpZXMKLSBzdGF0dXMKClNoYXduCgpfX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsaW51eC1hcm0ta2VybmVsIG1haWxpbmcgbGlz dApsaW51eC1hcm0ta2VybmVsQGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3RzLmluZnJh ZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1hcm0ta2VybmVsCg== 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 DD54FEB64DC for ; Wed, 19 Jul 2023 01:58:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229450AbjGSB6M (ORCPT ); Tue, 18 Jul 2023 21:58:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43328 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229536AbjGSB6L (ORCPT ); Tue, 18 Jul 2023 21:58:11 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5C3521BE9; Tue, 18 Jul 2023 18:58:04 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id E4D0761683; Wed, 19 Jul 2023 01:58:03 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 56C73C433C9; Wed, 19 Jul 2023 01:57:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1689731883; bh=RawEnqc1EkIgdkG2i7me4l9fFj/6mUCWCz+xGcSTwAc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Q7a7oEr8cy6h2A3djds9QBxY5rCG1G+E9+SWbmSrMqgK/lMSCQqJOb3xV05iUsCoN nN6poCxArph65dloIeJ0lF/SQIjMWHOg/QGU9+9zKLscwn/9IyVDUEPKN3JGOUtYxb ypkBXycMSAg6oIOYs243x+kWrPKEsXsYtRMpjL0vtYqc9xY9U7Pa8+/mIqgxE6IGz4 bdqzzqjWpHwNci/K3JKCV9qOeABLl36BBNRZF2eIpp7L8J4+vkPY3H4qBmGyZlZh8F kTbnIpsumRmJMlPnAKBna4b9QoBYH+7ZXgVIaMa0XzboSZDIikXDlO44OiXiTWJbYi th22DTt01xFtQ== Date: Wed, 19 Jul 2023 09:57:50 +0800 From: Shawn Guo To: Tim Harvey Cc: linux-arm-kernel@lists.infradead.org, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] arm64: dts: freescale: Add imx8mp-venice-gw71xx-2x Message-ID: <20230719015750.GR9559@dragon> References: <20230607170724.2016988-1-tharvey@gateworks.com> <20230607170724.2016988-2-tharvey@gateworks.com> <20230717002717.GI9559@dragon> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Tue, Jul 18, 2023 at 02:16:22PM -0700, Tim Harvey wrote: > On Sun, Jul 16, 2023 at 5:27 PM Shawn Guo wrote: > > > > On Wed, Jun 07, 2023 at 10:07:24AM -0700, Tim Harvey wrote: > > > The Gateworks imx8mp-venice-gw71xx-2x consists of a SOM + baseboard. > > > > > > The GW702x SOM contains the following: > > > - i.MX8M Plus SoC > > > - LPDDR4 memory > > > - eMMC Boot device > > > - Gateworks System Controller (GSC) with integrated EEPROM, button > > > controller, and ADC's > > > - PMIC > > > - SOM connector providing: > > > - eQoS GbE MII > > > - 1x SPI > > > - 2x I2C > > > - 4x UART > > > - 2x USB 3.0 > > > - 1x PCI > > > - 1x SDIO (4-bit 3.3V) > > > - 1x SDIO (4-bit 3.3V/1.8V) > > > - GPIO > > > > > > The GW71xx Baseboard contains the following: > > > - GPS > > > - RJ45 GbE (eQoS) > > > - off-board I/O connector with UART, I2C, SPI, GPIO > > > - EERPOM > > > - PCIe clock generator > > > - full-length miniPCIe socket with PCI/USB3 (via mux) and USB2.0 > > > - USB Type-C with USB 2.0 host and peripheral support > > > - Wide range DC input supply > > > > > > Signed-off-by: Tim Harvey > > > --- > > > arch/arm64/boot/dts/freescale/Makefile | 1 + > > > .../dts/freescale/imx8mp-venice-gw71xx-2x.dts | 19 ++ > > > .../dts/freescale/imx8mp-venice-gw71xx.dtsi | 236 ++++++++++++++++++ > > > 3 files changed, 256 insertions(+) > > > create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-venice-gw71xx-2x.dts > > > create mode 100644 arch/arm64/boot/dts/freescale/imx8mp-venice-gw71xx.dtsi > > > > > > diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile > > > index 62f22b77b38b..b3bb823d0168 100644 > > > --- a/arch/arm64/boot/dts/freescale/Makefile > > > +++ b/arch/arm64/boot/dts/freescale/Makefile > > > @@ -101,6 +101,7 @@ dtb-$(CONFIG_ARCH_MXC) += imx8mp-icore-mx8mp-edimm2.2.dtb > > > dtb-$(CONFIG_ARCH_MXC) += imx8mp-msc-sm2s-ep1.dtb > > > dtb-$(CONFIG_ARCH_MXC) += imx8mp-phyboard-pollux-rdk.dtb > > > dtb-$(CONFIG_ARCH_MXC) += imx8mp-tqma8mpql-mba8mpxl.dtb > > > +dtb-$(CONFIG_ARCH_MXC) += imx8mp-venice-gw71xx-2x.dtb > > > dtb-$(CONFIG_ARCH_MXC) += imx8mp-venice-gw74xx.dtb > > > dtb-$(CONFIG_ARCH_MXC) += imx8mp-venice-gw7905-2x.dtb > > > dtb-$(CONFIG_ARCH_MXC) += imx8mp-verdin-nonwifi-dahlia.dtb > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw71xx-2x.dts b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw71xx-2x.dts > > > new file mode 100644 > > > index 000000000000..53120fc9cd7f > > > --- /dev/null > > > +++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw71xx-2x.dts > > > @@ -0,0 +1,19 @@ > > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > > +/* > > > + * Copyright 2023 Gateworks Corporation > > > + */ > > > + > > > +/dts-v1/; > > > + > > > +#include "imx8mp.dtsi" > > > +#include "imx8mp-venice-gw702x.dtsi" > > > +#include "imx8mp-venice-gw71xx.dtsi" > > > + > > > +/ { > > > + model = "Gateworks Venice GW71xx-2x i.MX8MP Development Kit"; > > > + compatible = "gateworks,imx8mp-gw71xx-2x", "fsl,imx8mp"; > > > + > > > + chosen { > > > + stdout-path = &uart2; > > > + }; > > > +}; > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-venice-gw71xx.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw71xx.dtsi > > > new file mode 100644 > > > index 000000000000..86999f52d4b2 > > > --- /dev/null > > > +++ b/arch/arm64/boot/dts/freescale/imx8mp-venice-gw71xx.dtsi > > > @@ -0,0 +1,236 @@ > > > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > > > +/* > > > + * Copyright 2023 Gateworks Corporation > > > + */ > > > + > > > +#include > > > +#include > > > +#include > > > + > > > +/ { > > > + led-controller { > > > + compatible = "gpio-leds"; > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&pinctrl_gpio_leds>; > > > + > > > + led-0 { > > > + function = LED_FUNCTION_STATUS; > > > + color = ; > > > + gpios = <&gpio4 1 GPIO_ACTIVE_HIGH>; > > > + default-state = "on"; > > > + linux,default-trigger = "heartbeat"; > > > + }; > > > + > > > + led-1 { > > > + function = LED_FUNCTION_STATUS; > > > + color = ; > > > + gpios = <&gpio4 5 GPIO_ACTIVE_HIGH>; > > > + default-state = "off"; > > > + }; > > > + }; > > > + > > > + pcie0_refclk: pcie0-refclk { > > > > Can we name the node clock-xxx? > > > > > + compatible = "fixed-clock"; > > > + #clock-cells = <0>; > > > + clock-frequency = <100000000>; > > > + }; > > > + > > > + pps { > > > + compatible = "pps-gpio"; > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&pinctrl_pps>; > > > + gpios = <&gpio4 3 GPIO_ACTIVE_HIGH>; > > > + status = "okay"; > > > + }; > > > +}; > > > + > > > +/* off-board header */ > > > +&ecspi2 { > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&pinctrl_spi2>; > > > + cs-gpios = <&gpio5 13 GPIO_ACTIVE_LOW>; > > > + status = "okay"; > > > +}; > > > + > > > +&gpio4 { > > > + gpio-line-names = > > > + "", "", "", "", > > > + "", "", "", "", > > > + "dio1", "", "", "dio0", > > > + "", "", "pci_usb_sel", "", > > > + "", "", "", "", > > > + "", "", "", "", > > > + "dio3", "", "dio2", "", > > > + "pci_wdis#", "", "", ""; > > > +}; > > > + > > > +&i2c2 { > > > + clock-frequency = <400000>; > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&pinctrl_i2c2>; > > > + status = "okay"; > > > + > > > + accelerometer@19 { > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&pinctrl_accel>; > > > + compatible = "st,lis2de12"; > > > + reg = <0x19>; > > > > Can we start the property list from these two? > > > > > + st,drdy-int-pin = <1>; > > > + interrupt-parent = <&gpio4>; > > > + interrupts = <21 IRQ_TYPE_LEVEL_LOW>; > > > + interrupt-names = "INT1"; > > > + }; > > > +}; > > > + > > > +&pcie_phy { > > > + fsl,refclk-pad-mode = ; > > > + fsl,clkreq-unsupported; > > > + clocks = <&pcie0_refclk>; > > > + clock-names = "ref"; > > > + status = "okay"; > > > +}; > > > + > > > +&pcie { > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&pinctrl_pcie0>; > > > + reset-gpio = <&gpio4 29 GPIO_ACTIVE_LOW>; > > > + status = "okay"; > > > +}; > > > + > > > +/* GPS */ > > > +&uart1 { > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&pinctrl_uart1>; > > > + status = "okay"; > > > +}; > > > + > > > +/* off-board header */ > > > +&uart3 { > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&pinctrl_uart3>; > > > + status = "okay"; > > > +}; > > > + > > > +/* USB1 Type-C front panel */ > > > +&usb3_0 { > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&pinctrl_usb1>; > > > + fsl,over-current-active-low; > > > + status = "okay"; > > > +}; > > > + > > > +&usb3_phy0 { > > > + status = "okay"; > > > +}; > > > + > > > +&usb_dwc3_0 { > > > + /* dual role is implemented but not a full featured OTG */ > > > + adp-disable; > > > + hnp-disable; > > > + srp-disable; > > > + dr_mode = "otg"; > > > + usb-role-switch; > > > + role-switch-default-mode = "peripheral"; > > > + status = "okay"; > > > + > > > + connector { > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&pinctrl_usbcon1>; > > > + compatible = "gpio-usb-b-connector", "usb-b-connector"; > > > > Start the properties from 'compatible'? > > > > Shawn > > > > Hi Shawn, > > My notes say that pinctrl should come first (if needed) followed by > compatible, reg, other props, and finally status (if needed). Assuming > I'm wrong about that where in the list should pinctrl be and is this > documented anywhere for future reference? It's more a recommended idiom than documented/hard rule. - compatible - reg - general properties - vendor specific properties - status Shawn