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=-8.3 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY,URIBL_BLOCKED, USER_AGENT_SANE_2 autolearn=ham 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 E8BD1C433DF for ; Mon, 12 Oct 2020 12:02:18 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 89C2221D7F for ; Mon, 12 Oct 2020 12:02:18 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=mediatek.com header.i=@mediatek.com header.b="TKHmU4GJ" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2388436AbgJLMCR (ORCPT ); Mon, 12 Oct 2020 08:02:17 -0400 Received: from Mailgw01.mediatek.com ([1.203.163.78]:49095 "EHLO mailgw01.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S2388373AbgJLMCR (ORCPT ); Mon, 12 Oct 2020 08:02:17 -0400 X-UUID: ffba327e58714f71a6d76e7305d4bd5f-20201012 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=mediatek.com; s=dk; h=Content-Transfer-Encoding:MIME-Version:Content-Type:References:In-Reply-To:Date:CC:To:From:Subject:Message-ID; bh=Bm1nD1Rni6NjuejkSKF6tuW+1OZNvmAl+g7ZgtGLPq8=; b=TKHmU4GJsUveGwDhBPx/LGSjxR3leBMs8N6y7bwJSxOlan0sk7oer/u9vRRlh3s8X7pSSOkMjjvVFrcXHQYITLDunKJ05E3CZDH3JfQSPA+Hjn/qPzO0MKzEwDmWiZq4c+B2nkb04cH3V8CaQuYNO+CUjBDthWz3zmRl1QjyOKU=; X-UUID: ffba327e58714f71a6d76e7305d4bd5f-20201012 Received: from mtkcas32.mediatek.inc [(172.27.4.253)] by mailgw01.mediatek.com (envelope-from ) (mailgw01.mediatek.com ESMTP with TLSv1.2 ECDHE-RSA-AES256-SHA384 256/256) with ESMTP id 1331938782; Mon, 12 Oct 2020 20:02:01 +0800 Received: from MTKCAS32.mediatek.inc (172.27.4.184) by MTKMBS31DR.mediatek.inc (172.27.6.102) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Mon, 12 Oct 2020 20:02:00 +0800 Received: from [10.17.3.153] (10.17.3.153) by MTKCAS32.mediatek.inc (172.27.4.170) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Mon, 12 Oct 2020 20:01:58 +0800 Message-ID: <1602504119.26323.54.camel@mhfsdcap03> Subject: Re: [PATCH v3 02/24] dt-bindings: memory: mediatek: Convert SMI to DT schema From: Yong Wu To: Krzysztof Kozlowski , Rob Herring CC: Joerg Roedel , Matthias Brugger , Rob Herring , Robin Murphy , Will Deacon , Evan Green , Tomasz Figa , , , , "linux-kernel@vger.kernel.org" , , , , Nicolas Boichat , , , , Greg Kroah-Hartman , Date: Mon, 12 Oct 2020 20:01:59 +0800 In-Reply-To: <20201012071843.GA1889@pi3> References: <20200930070647.10188-1-yong.wu@mediatek.com> <20200930070647.10188-3-yong.wu@mediatek.com> <20201002110831.GD6888@pi3> <1601958428.26323.26.camel@mhfsdcap03> <1602310691.26323.39.camel@mhfsdcap03> <20201012071843.GA1889@pi3> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.4-0ubuntu2 MIME-Version: 1.0 X-TM-SNTS-SMTP: 1FA9AD3A565A7DE7F33FFEDBC7D6A606B4EFC4E07A07FFF66367021A06DDA1692000:8 X-MTK: N Content-Transfer-Encoding: base64 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org T24gTW9uLCAyMDIwLTEwLTEyIGF0IDA5OjE4ICswMjAwLCBLcnp5c3p0b2YgS296bG93c2tpIHdy b3RlOg0KPiBPbiBTYXQsIE9jdCAxMCwgMjAyMCBhdCAwMjoxODoxMVBNICswODAwLCBZb25nIFd1 IHdyb3RlOg0KPiA+IE9uIFR1ZSwgMjAyMC0xMC0wNiBhdCAwOToxNSArMDIwMCwgS3J6eXN6dG9m IEtvemxvd3NraSB3cm90ZToNCj4gPiA+IE9uIFR1ZSwgNiBPY3QgMjAyMCBhdCAwNjoyNywgWW9u ZyBXdSA8eW9uZy53dUBtZWRpYXRlay5jb20+IHdyb3RlOg0KPiA+ID4gPg0KPiA+ID4gPiBPbiBG cmksIDIwMjAtMTAtMDIgYXQgMTM6MDggKzAyMDAsIEtyenlzenRvZiBLb3psb3dza2kgd3JvdGU6 DQo+ID4gPiA+ID4gT24gV2VkLCBTZXAgMzAsIDIwMjAgYXQgMDM6MDY6MjVQTSArMDgwMCwgWW9u ZyBXdSB3cm90ZToNCj4gPiA+ID4gPiA+IENvbnZlcnQgTWVkaWFUZWsgU01JIHRvIERUIHNjaGVt YS4NCj4gPiA+ID4gPiA+DQo+ID4gPiA+ID4gPiBTaWduZWQtb2ZmLWJ5OiBZb25nIFd1IDx5b25n Lnd1QG1lZGlhdGVrLmNvbT4NCj4gPiA+ID4gPiA+IC0tLQ0KPiA+ID4gPiA+ID4gIC4uLi9tZWRp YXRlayxzbWktY29tbW9uLnR4dCAgICAgICAgICAgICAgICAgICB8ICA0OSAtLS0tLS0tLS0NCj4g PiA+ID4gPiA+ICAuLi4vbWVkaWF0ZWssc21pLWNvbW1vbi55YW1sICAgICAgICAgICAgICAgICAg fCAxMDAgKysrKysrKysrKysrKysrKysrDQo+ID4gPiA+ID4gPiAgLi4uL21lbW9yeS1jb250cm9s bGVycy9tZWRpYXRlayxzbWktbGFyYi50eHQgIHwgIDQ5IC0tLS0tLS0tLQ0KPiA+ID4gPiA+ID4g IC4uLi9tZW1vcnktY29udHJvbGxlcnMvbWVkaWF0ZWssc21pLWxhcmIueWFtbCB8ICA5MSArKysr KysrKysrKysrKysrDQo+ID4gPiA+ID4gPiAgNCBmaWxlcyBjaGFuZ2VkLCAxOTEgaW5zZXJ0aW9u cygrKSwgOTggZGVsZXRpb25zKC0pDQo+ID4gPiA+ID4gPiAgZGVsZXRlIG1vZGUgMTAwNjQ0IERv Y3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9tZW1vcnktY29udHJvbGxlcnMvbWVkaWF0 ZWssc21pLWNvbW1vbi50eHQNCj4gPiA+ID4gPiA+ICBjcmVhdGUgbW9kZSAxMDA2NDQgRG9jdW1l bnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL21lbW9yeS1jb250cm9sbGVycy9tZWRpYXRlayxz bWktY29tbW9uLnlhbWwNCj4gPiA+ID4gPiA+ICBkZWxldGUgbW9kZSAxMDA2NDQgRG9jdW1lbnRh dGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL21lbW9yeS1jb250cm9sbGVycy9tZWRpYXRlayxzbWkt bGFyYi50eHQNCj4gPiA+ID4gPiA+ICBjcmVhdGUgbW9kZSAxMDA2NDQgRG9jdW1lbnRhdGlvbi9k ZXZpY2V0cmVlL2JpbmRpbmdzL21lbW9yeS1jb250cm9sbGVycy9tZWRpYXRlayxzbWktbGFyYi55 YW1sDQo+ID4gPiA+IC4uLg0KPiA+ID4gPiA+ID4gK3Byb3BlcnRpZXM6DQo+ID4gPiA+ID4gPiAr ICBjb21wYXRpYmxlOg0KPiA+ID4gPiA+ID4gKyAgICBvbmVPZjoNCj4gPiA+ID4gPiA+ICsgICAg ICAtIGVudW06DQo+ID4gPiA+ID4gPiArICAgICAgICAgIC0gbWVkaWF0ZWssbXQyNzAxLXNtaS1j b21tb24NCj4gPiA+ID4gPiA+ICsgICAgICAgICAgLSBtZWRpYXRlayxtdDI3MTItc21pLWNvbW1v bg0KPiA+ID4gPiA+ID4gKyAgICAgICAgICAtIG1lZGlhdGVrLG10Njc3OS1zbWktY29tbW9uDQo+ ID4gPiA+ID4gPiArICAgICAgICAgIC0gbWVkaWF0ZWssbXQ4MTczLXNtaS1jb21tb24NCj4gPiA+ ID4gPiA+ICsgICAgICAgICAgLSBtZWRpYXRlayxtdDgxODMtc21pLWNvbW1vbg0KPiA+ID4gPiA+ ID4gKw0KPiA+ID4gPiA+ID4gKyAgICAgIC0gZGVzY3JpcHRpb246IGZvciBtdDc2MjMNCj4gPiA+ ID4gPiA+ICsgICAgICAgIGl0ZW1zOg0KPiA+ID4gPiA+ID4gKyAgICAgICAgICAtIGNvbnN0OiBt ZWRpYXRlayxtdDc2MjMtc21pLWNvbW1vbg0KPiA+ID4gPiA+ID4gKyAgICAgICAgICAtIGNvbnN0 OiBtZWRpYXRlayxtdDI3MDEtc21pLWNvbW1vbg0KPiA+ID4gPiA+ID4gKw0KPiA+ID4gPiA+ID4g KyAgcmVnOg0KPiA+ID4gPiA+ID4gKyAgICBtYXhJdGVtczogMQ0KPiA+ID4gPiA+ID4gKw0KPiA+ ID4gPiA+ID4gKyAgY2xvY2tzOg0KPiA+ID4gPiA+ID4gKyAgICBkZXNjcmlwdGlvbjogfA0KPiA+ ID4gPiA+ID4gKyAgICAgIGFwYiBhbmQgc21pIGFyZSBtYW5kYXRvcnkuIHRoZSBhc3luYyBpcyBv bmx5IGZvciBnZW5lcmF0aW9uIDEgc21pIEhXLg0KPiA+ID4gPiA+ID4gKyAgICAgIGdhbHMoZ2xv YmFsIGFzeW5jIGxvY2FsIHN5bmMpIGFsc28gaXMgb3B0aW9uYWwsIGhlcmUgaXMgdGhlIGxpc3Qg d2hpY2gNCj4gPiA+ID4gPiA+ICsgICAgICByZXF1aXJlIGdhbHM6IG10Njc3OSBhbmQgbXQ4MTgz Lg0KPiA+ID4gPiA+ID4gKyAgICBtaW5JdGVtczogMg0KPiA+ID4gPiA+ID4gKyAgICBtYXhJdGVt czogNA0KPiA+ID4gPiA+ID4gKyAgICBpdGVtczoNCj4gPiA+ID4gPiA+ICsgICAgICAtIGRlc2Ny aXB0aW9uOiBhcGIgaXMgQWR2YW5jZWQgUGVyaXBoZXJhbCBCdXMgY2xvY2ssIEl0J3MgdGhlIGNs b2NrIGZvcg0KPiA+ID4gPiA+ID4gKyAgICAgICAgICBzZXR0aW5nIHRoZSByZWdpc3Rlci4NCj4g PiA+ID4gPiA+ICsgICAgICAtIGRlc2NyaXB0aW9uOiBzbWkgaXMgdGhlIGNsb2NrIGZvciB0cmFu c2ZlciBkYXRhIGFuZCBjb21tYW5kLg0KPiA+ID4gPiA+ID4gKyAgICAgIC0gZGVzY3JpcHRpb246 IGFzeW5jIGlzIGFzeW5jaHJvbm91cyBjbG9jaywgaXQgaGVscCB0cmFuc2Zvcm0gdGhlIHNtaSBj bG9jaw0KPiA+ID4gPiA+ID4gKyAgICAgICAgICBpbnRvIHRoZSBlbWkgY2xvY2sgZG9tYWluLg0K PiA+ID4gPiA+ID4gKyAgICAgIC0gZGVzY3JpcHRpb246IGdhbHMwIGlzIHRoZSBwYXRoMCBjbG9j ayBvZiBnYWxzLg0KPiA+ID4gPiA+ID4gKyAgICAgIC0gZGVzY3JpcHRpb246IGdhbHMxIGlzIHRo ZSBwYXRoMSBjbG9jayBvZiBnYWxzLg0KPiA+ID4gPiA+ID4gKw0KPiA+ID4gPiA+ID4gKyAgY2xv Y2stbmFtZXM6DQo+ID4gPiA+ID4gPiArICAgIG9uZU9mOg0KPiA+ID4gPiA+ID4gKyAgICAgIC0g aXRlbXM6DQo+ID4gPiA+ID4gPiArICAgICAgICAgIC0gY29uc3Q6IGFwYg0KPiA+ID4gPiA+ID4g KyAgICAgICAgICAtIGNvbnN0OiBzbWkNCj4gPiA+ID4gPiA+ICsgICAgICAtIGl0ZW1zOg0KPiA+ ID4gPiA+ID4gKyAgICAgICAgICAtIGNvbnN0OiBhcGINCj4gPiA+ID4gPiA+ICsgICAgICAgICAg LSBjb25zdDogc21pDQo+ID4gPiA+ID4gPiArICAgICAgICAgIC0gY29uc3Q6IGFzeW5jDQo+ID4g PiA+ID4gPiArICAgICAgLSBpdGVtczoNCj4gPiA+ID4gPiA+ICsgICAgICAgICAgLSBjb25zdDog YXBiDQo+ID4gPiA+ID4gPiArICAgICAgICAgIC0gY29uc3Q6IHNtaQ0KPiA+ID4gPiA+ID4gKyAg ICAgICAgICAtIGNvbnN0OiBnYWxzMA0KPiA+ID4gPiA+ID4gKyAgICAgICAgICAtIGNvbnN0OiBn YWxzMQ0KPiA+ID4gPiA+DQo+ID4gPiA+ID4gU2ltaWxhcmx5IHRvIG15IGNvbW1lbnQgdG8gb3Ro ZXIgcHJvcGVydGllcywgdGhpcyByZXF1aXJlbWVudCBwZXINCj4gPiA+ID4gPiBjb21wYXRpYmxl IHNob3VsZCBiZSBwYXJ0IG9mIHRoZSBzY2hlbWEgd2l0aGluICdpZi10aGVuJy4NCj4gPiA+ID4N Cj4gPiA+ID4gSSdtIG5vdCBzbyBmYW1pbGlhciB3aXRoIHRoaXMgZm9ybWF0LiBEbyB0aGlzIGhh cyAiaWYtdGhlbi0nZWxzZQ0KPiA+ID4gPiBpZictdGhlbi1lbHNlIj8NCj4gPiA+IA0KPiA+ID4g VGhlc2UgYXJlIG11dHVhbGx5IGV4Y2x1c2l2ZSBjb25kaXRpb25zLCBzbyB5b3UgY2FuIHNraXAg ZWxzZToNCj4gPiA+ICAtIGlmLXRoZW4NCj4gPiA+ICAtIGlmLXRoZW4NCj4gPiA+ICAtIGlmLXRo ZW4NCj4gPiA+IEl0IHdpbGwgYmUgbW9yZSByZWFkYWJsZSB0aGVuIHN0YWNraW5nICdpZicgdW5k ZXIgJ2Vsc2UnDQo+ID4gDQo+ID4gVGhhbmtzLiBJIHdpbGwgdXNlIHNvbWV0aGluZyBsaWtlIHRo aXM6DQo+ID4gDQo+ID4gIGFueU9mOg0KPiANCj4gVGhlbiBpdCBzaG91bGQgYmUgb25lT2YgYXMg b25seSBvbmUgY29uZGl0aW9uIGNhbiBiZSB2YWxpZC4NCg0KSSBkaWQgZG8gdGhpcyBhdCB0aGUg YmVnaW5uaW5nLiBCdXQgSSBnZXQgYSB3YXJuaW5nIGxvZyB3aGVuDQpkdF9iaW5kaW5nX2NoZWNr Lg0KDQoNCkJlbG93IGlzIG15IHNjaGVtYSBhbmQgdGhlIGRldGFpbGVkIHdhcm5pbmcgbG9nOg0K Ly89PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0NCiAgY2xvY2tzOg0KICAg IGRlc2NyaXB0aW9uOiB8DQogICAgICB4eHh4eA0KICAgIG1pbkl0ZW1zOiAyDQogICAgbWF4SXRl bXM6IDQNCiAgICBpdGVtczoNCiAgICAgIC0gZGVzY3JpcHRpb246IGFwYiBpcyB0aGUgY2xvY2sg Zm9yIHNldHRpbmcgdGhlIHJlZ2lzdGVyLg0KICAgICAgLSBkZXNjcmlwdGlvbjogc21pIGlzIHRo ZSBjbG9jayBmb3IgdHJhbnNmZXIgZGF0YSBhbmQgY29tbWFuZC4NCiAgICAgIC0gZGVzY3JpcHRp b246IGFzeW5jIGlzIGFzeW5jaHJvbm91cyBjbG9jay4NCiAgICAgIC0gZGVzY3JpcHRpb246IGdh bHMwIGlzIHRoZSBwYXRoMCBjbG9jayBvZiBnYWxzLg0KICAgICAgLSBkZXNjcmlwdGlvbjogZ2Fs czEgaXMgdGhlIHBhdGgxIGNsb2NrIG9mIGdhbHMuICAgIA0KDQogIGNsb2NrLW5hbWVzOg0KICAg IG1pbkl0ZW1zOiAyDQogICAgbWF4SXRlbXM6IDQgDQoNCnJlcXVpcmVkOg0KICAtIGNvbXBhdGli bGUNCiAgLSByZWcNCiAgLSBwb3dlci1kb21haW5zDQogIC0gY2xvY2tzDQogIC0gY2xvY2stbmFt ZXMNCg0Kb25lT2Y6DQogIC0gaWY6ICNvbmx5IGZvciBnZW4xIEhXDQogICAgICBwcm9wZXJ0aWVz Og0KICAgICAgICBjb21wYXRpYmxlOg0KICAgICAgICAgIGNvbnRhaW5zOg0KICAgICAgICAgICAg ZW51bToNCiAgICAgICAgICAgICAgLSBtZWRpYXRlayxtdDI3MDEtc21pLWNvbW1vbg0KICAgIHRo ZW46DQogICAgICAgcHJvcGVydGllczoNCiAgICAgICAgIGNsb2NrOg0KICAgICAgICAgICBpdGVt czoNCiAgICAgICAgICAgICBtaW5JdGVtczogMw0KICAgICAgICAgICAgIG1heEl0ZW1zOiAzDQog ICAgICAgICBjbG9jay1uYW1lczoNCiAgICAgICAgICAgaXRlbXM6DQogICAgICAgICAgICAgLSBj b25zdDogYXBiDQogICAgICAgICAgICAgLSBjb25zdDogc21pDQogICAgICAgICAgICAgLSBjb25z dDogYXN5bmMNCg0KICAtIGlmOiAjZm9yIGdlbjIgSFcgdGhhdCBoYXZlIGdhbHMNCiAgICAgIHBy b3BlcnRpZXM6DQogICAgICAgIGNvbXBhdGlibGU6DQogICAgICAgICAgICBlbnVtOg0KICAgICAg ICAgICAgICAtIG1lZGlhdGVrLG10Njc3OS1zbWktY29tbW9uDQogICAgICAgICAgICAgIC0gbWVk aWF0ZWssbXQ4MTgzLXNtaS1jb21tb24NCiAgICAgICAgICAgICAgLSBtZWRpYXRlayxtdDgxOTIt c21pLWNvbW1vbg0KDQogICAgdGhlbjoNCiAgICAgIHByb3BlcnRpZXM6DQogICAgICAgIGNsb2Nr Og0KICAgICAgICAgIGl0ZW1zOg0KICAgICAgICAgICAgbWluSXRlbXM6IDQNCiAgICAgICAgICAg IG1heEl0ZW1zOiA0DQogICAgICAgIGNsb2NrLW5hbWVzOg0KICAgICAgICAgIGl0ZW1zOg0KICAg ICAgICAgICAgLSBjb25zdDogYXBiDQogICAgICAgICAgICAtIGNvbnN0OiBzbWkNCiAgICAgICAg ICAgIC0gY29uc3Q6IGdhbHMwDQogICAgICAgICAgICAtIGNvbnN0OiBnYWxzMQ0KDQogICAgZWxz ZTogI2ZvciBnZW4yIEhXIHRoYXQgZG9uJ3QgaGF2ZSBnYWxzDQogICAgICBwcm9wZXJ0aWVzOg0K ICAgICAgICBjbG9jazoNCiAgICAgICAgICBpdGVtczoNCiAgICAgICAgICAgIG1pbkl0ZW1zOiAy DQogICAgICAgICAgICBtYXhJdGVtczogMg0KICAgICAgICBjbG9jay1uYW1lczoNCiAgICAgICAg ICBpdGVtczoNCiAgICAgICAgICAgIC0gY29uc3Q6IGFwYg0KICAgICAgICAgICAgLSBjb25zdDog c21pDQoNCmFkZGl0aW9uYWxQcm9wZXJ0aWVzOiBmYWxzZSANCg0KDQovLz09PT09PT09PT09d2Fy bmluZyBsb2c9PT09PT09PT09PT09PT09PT09PT0NCg0KRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVl L2JpbmRpbmdzL21lbW9yeS1jb250cm9sbGVycy9tZWRpYXRlayxzbWktY29tbW9uLmV4YW1wbGUu ZHQueWFtbA0KLi4uL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9tZW1vcnktY29u dHJvbGxlcnMvbWVkaWF0ZWssc21pLWNvbW1vbi5leGFtcGxlLmR0LnlhbWw6IHNtaUAxNDAyMjAw MDogeydjb21wYXRpYmxlJzogWydtZWRpYXRlayxtdDgxNzMtc21pLWNvbW1vbiddLCAncmVnJzog W1szMzU2ODM1ODQsIDQwOTZdXSwgJ3Bvd2VyLWRvbWFpbnMnOiBbWzQyOTQ5NjcyOTUsIDNdXSwg J2Nsb2Nrcyc6IFtbNDI5NDk2NzI5NSwgMV0sIFs0Mjk0OTY3Mjk1LCAxXV0sICdjbG9jay1uYW1l cyc6IFsnYXBiJywgJ3NtaSddLCAnJG5vZGVuYW1lJzogWydzbWlAMTQwMjIwMDAnXX0gaXMgdmFs aWQgdW5kZXIgZWFjaCBvZiB7J2Vsc2UnOiB7J3Byb3BlcnRpZXMnOiB7J2Nsb2NrJzogeydpdGVt cyc6IHsnbWF4SXRlbXMnOiAyLCAnbWluSXRlbXMnOiAyfSwgJ3R5cGUnOiAnYXJyYXknfSwgJ2Ns b2NrLW5hbWVzJzogeydhZGRpdGlvbmFsSXRlbXMnOiBGYWxzZSwgJ2l0ZW1zJzogW3snY29uc3Qn OiAnYXBiJ30sIHsnY29uc3QnOiAnc21pJ31dLCAnbWF4SXRlbXMnOiAyLCAnbWluSXRlbXMnOiAy LCAndHlwZSc6ICdhcnJheSd9fX0sICdpZic6IHsncHJvcGVydGllcyc6IHsnY29tcGF0aWJsZSc6 IHsnYWRkaXRpb25hbEl0ZW1zJzogRmFsc2UsICdpdGVtcyc6IFt7J2VudW0nOiBbJ21lZGlhdGVr LG10Njc3OS1zbWktY29tbW9uJywgJ21lZGlhdGVrLG10ODE4My1zbWktY29tbW9uJywgJ21lZGlh dGVrLG10ODE5Mi1zbWktY29tbW9uJ119XSwgJ21heEl0ZW1zJzogMSwgJ21pbkl0ZW1zJzogMSwg J3R5cGUnOiAnYXJyYXknfX19LCAndGhlbic6IHsncHJvcGVydGllcyc6IHsnY2xvY2snOiB7J2l0 ZW1zJzogeydtYXhJdGVtcyc6IDQsICdtaW5JdGVtcyc6IDR9LCAndHlwZSc6ICdhcnJheSd9LCAn Y2xvY2stbmFtZXMnOiB7J2FkZGl0aW9uYWxJdGVtcyc6IEZhbHNlLCAnaXRlbXMnOiBbeydjb25z dCc6ICdhcGInfSwgeydjb25zdCc6ICdzbWknfSwgeydjb25zdCc6ICdnYWxzMCd9LCB7J2NvbnN0 JzogJ2dhbHMxJ31dLCAnbWF4SXRlbXMnOiA0LCAnbWluSXRlbXMnOiA0LCAndHlwZSc6ICdhcnJh eSd9fX19LCB7J2lmJzogeydwcm9wZXJ0aWVzJzogeydjb21wYXRpYmxlJzogeydjb250YWlucyc6 IHsnZW51bSc6IFsnbWVkaWF0ZWssbXQyNzAxLXNtaS1jb21tb24nXX19fX0sICd0aGVuJzogeydw cm9wZXJ0aWVzJzogeydjbG9jayc6IHsnaXRlbXMnOiB7J21heEl0ZW1zJzogMywgJ21pbkl0ZW1z JzogM30sICd0eXBlJzogJ2FycmF5J30sICdjbG9jay1uYW1lcyc6IHsnYWRkaXRpb25hbEl0ZW1z JzogRmFsc2UsICdpdGVtcyc6IFt7J2NvbnN0JzogJ2FwYid9LCB7J2NvbnN0JzogJ3NtaSd9LCB7 J2NvbnN0JzogJ2FzeW5jJ31dLCAnbWF4SXRlbXMnOiAzLCAnbWluSXRlbXMnOiAzLCAndHlwZSc6 ICdhcnJheSd9fX19DQp7JyRmaWxlbmFtZSc6DQonLi4uL0RvY3VtZW50YXRpb24vZGV2aWNldHJl ZS9iaW5kaW5ncy9tZW1vcnktY29udHJvbGxlcnMvbWVkaWF0ZWssc21pLWNvbW1vbi55YW1sJywN CiAnJGlkJzoNCidodHRwOi8vZGV2aWNldHJlZS5vcmcvc2NoZW1hcy9tZW1vcnktY29udHJvbGxl cnMvbWVkaWF0ZWssc21pLWNvbW1vbi55YW1sIycsDQogJyRzY2hlbWEnOiAnaHR0cDovL2Rldmlj ZXRyZWUub3JnL21ldGEtc2NoZW1hcy9jb3JlLnlhbWwjJywNCiAnJHNlbGVjdF92YWxpZGF0b3In OiA8anNvbnNjaGVtYS52YWxpZGF0b3JzLmNyZWF0ZS48bG9jYWxzPi5WYWxpZGF0b3INCm9iamVj dCBhdCAweDdmYjNkZjM3ODkwOD4sDQogJ2FkZGl0aW9uYWxQcm9wZXJ0aWVzJzogRmFsc2UsDQog J29uZU9mJzogW3snaWYnOiB7J3Byb3BlcnRpZXMnOiB7J2NvbXBhdGlibGUnOiB7J2NvbnRhaW5z JzogeydlbnVtJzoNClsnbWVkaWF0ZWssbXQyNzAxLXNtaS1jb21tb24nXX19fX0sDQogICAgICAg ICAgICAndGhlbic6IHsncHJvcGVydGllcyc6IHsnY2xvY2snOiB7J2l0ZW1zJzogeydtYXhJdGVt cyc6IDMsDQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICdtaW5JdGVtcyc6IDN9LA0KICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICd0eXBlJzogJ2FycmF5J30sDQogICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAnY2xvY2stbmFtZXMnOiB7J2FkZGl0aW9uYWxJdGVtcyc6DQpGYWxzZSwNCiAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAnaXRlbXMn OiBbeydjb25zdCc6DQonYXBiJ30sDQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgIHsnY29uc3QnOg0KJ3NtaSd9LA0KICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICB7J2Nv bnN0JzoNCidhc3luYyd9XSwNCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAnbWF4SXRlbXMnOiAzLA0KICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICdtaW5JdGVtcyc6IDMsDQogICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgJ3R5cGUnOiAnYXJyYXknfX19fSwN CiAgICAgICAgICAgeydlbHNlJzogeydwcm9wZXJ0aWVzJzogeydjbG9jayc6IHsnaXRlbXMnOiB7 J21heEl0ZW1zJzogMiwNCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgJ21pbkl0ZW1zJzogMn0sDQogICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgJ3R5cGUnOiAnYXJyYXknfSwNCiAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICdjbG9jay1uYW1lcyc6IHsnYWRkaXRpb25hbEl0ZW1zJzoNCkZh bHNlLA0KICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICdpdGVtcyc6IFt7J2NvbnN0JzoNCidhcGInfSwNCiAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgeydjb25zdCc6DQonc21pJ31dLA0K ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICdtYXhJ dGVtcyc6IDIsDQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgJ21pbkl0ZW1zJzogMiwNCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAndHlwZSc6ICdhcnJheSd9fX0sDQogICAgICAgICAgICAnaWYnOiB7 J3Byb3BlcnRpZXMnOiB7J2NvbXBhdGlibGUnOiB7J2FkZGl0aW9uYWxJdGVtcyc6DQpGYWxzZSwN CiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAnaXRlbXMn OiBbeydlbnVtJzoNClsnbWVkaWF0ZWssbXQ2Nzc5LXNtaS1jb21tb24nLA0KDQonbWVkaWF0ZWss bXQ4MTgzLXNtaS1jb21tb24nLA0KDQonbWVkaWF0ZWssbXQ4MTkyLXNtaS1jb21tb24nXX1dLA0K ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICdtYXhJdGVt cyc6IDEsDQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg J21pbkl0ZW1zJzogMSwNCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAndHlwZSc6ICdhcnJheSd9fX0sDQogICAgICAgICAgICAndGhlbic6IHsncHJvcGVy dGllcyc6IHsnY2xvY2snOiB7J2l0ZW1zJzogeydtYXhJdGVtcyc6IDQsDQogICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICdtaW5JdGVtcyc6IDR9 LA0KICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICd0eXBlJzog J2FycmF5J30sDQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAnY2xvY2stbmFt ZXMnOiB7J2FkZGl0aW9uYWxJdGVtcyc6DQpGYWxzZSwNCiAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAnaXRlbXMnOiBbeydjb25zdCc6DQonYXBiJ30s DQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgIHsnY29uc3QnOg0KJ3NtaSd9LA0KICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICB7J2NvbnN0JzoNCidnYWxzMCd9LA0KICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICB7J2NvbnN0JzoNCidnYWxzMSd9XSwNCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAnbWF4SXRlbXMnOiA0LA0KICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICdtaW5JdGVtcyc6IDQsDQogICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgJ3R5cGUnOg0KJ2FycmF5 J319fX1dLA0KICdwYXR0ZXJuUHJvcGVydGllcyc6IHsncGluY3RybC1bMC05XSsnOiBUcnVlfSwN CiAncHJvcGVydGllcyc6IHsnJG5vZGVuYW1lJzogVHJ1ZSwNCiAgICAgICAgICAgICAgICAnY2xv Y2stbmFtZXMnOiB7J21heEl0ZW1zJzogNCwgJ21pbkl0ZW1zJzogMn0sDQogICAgICAgICAgICAg ICAgJ2Nsb2Nrcyc6IHsnYWRkaXRpb25hbEl0ZW1zJzogRmFsc2UsDQogICAgICAgICAgICAgICAg ICAgICAgICAgICAnaXRlbXMnOiBbe30sIHt9LCB7fSwge30sIHt9XSwNCiAgICAgICAgICAgICAg ICAgICAgICAgICAgICdtYXhJdGVtcyc6IDQsDQogICAgICAgICAgICAgICAgICAgICAgICAgICAn bWluSXRlbXMnOiAyLA0KICAgICAgICAgICAgICAgICAgICAgICAgICAgJ3R5cGUnOiAnYXJyYXkn fSwNCiAgICAgICAgICAgICAgICAnY29tcGF0aWJsZSc6IHsnb25lT2YnOiBbeydhZGRpdGlvbmFs SXRlbXMnOiBGYWxzZSwNCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICdpdGVtcyc6IFt7J2VudW0nOg0KWydtZWRpYXRlayxtdDI3MDEtc21pLWNvbW1vbicsDQoNCidt ZWRpYXRlayxtdDI3MTItc21pLWNvbW1vbicsDQoNCidtZWRpYXRlayxtdDY3Nzktc21pLWNvbW1v bicsDQoNCidtZWRpYXRlayxtdDgxNzMtc21pLWNvbW1vbicsDQoNCidtZWRpYXRlayxtdDgxODMt c21pLWNvbW1vbicsDQoNCidtZWRpYXRlayxtdDgxOTItc21pLWNvbW1vbiddfV0sDQogICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAnbWF4SXRlbXMnOiAxLA0KICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgJ21pbkl0ZW1zJzogMSwNCiAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICd0eXBlJzogJ2FycmF5J30sDQog ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIHsnYWRkaXRpb25hbEl0ZW1z JzogRmFsc2UsDQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAnaXRl bXMnOiBbeydjb25zdCc6DQonbWVkaWF0ZWssbXQ3NjIzLXNtaS1jb21tb24nfSwNCiAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICB7J2NvbnN0JzoNCidt ZWRpYXRlayxtdDI3MDEtc21pLWNvbW1vbid9XSwNCiAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICdtYXhJdGVtcyc6IDIsDQogICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAnbWluSXRlbXMnOiAyLA0KICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgJ3R5cGUnOiAnYXJyYXknfV19LA0KICAgICAgICAgICAgICAgICdw aGFuZGxlJzogVHJ1ZSwNCiAgICAgICAgICAgICAgICAncGluY3RybC1uYW1lcyc6IFRydWUsDQog ICAgICAgICAgICAgICAgJ3Bvd2VyLWRvbWFpbnMnOiB7J21heEl0ZW1zJzogMSwgJ21pbkl0ZW1z JzogMX0sDQogICAgICAgICAgICAgICAgJ3JlZyc6IHsnbWF4SXRlbXMnOiAxLCAnbWluSXRlbXMn OiAxfSwNCiAgICAgICAgICAgICAgICAnc3RhdHVzJzogVHJ1ZX0sDQogJ3JlcXVpcmVkJzogWydj b21wYXRpYmxlJywNCiAgICAgICAgICAgICAgJ3JlZycsDQogICAgICAgICAgICAgICdwb3dlci1k b21haW5zJywNCiAgICAgICAgICAgICAgJ2Nsb2NrcycsDQogICAgICAgICAgICAgICdjbG9jay1u YW1lcyddLA0KICdzZWxlY3QnOiB7J3Byb3BlcnRpZXMnOiB7J2NvbXBhdGlibGUnOiB7J2NvbnRh aW5zJzogeydlbnVtJzoNClsnbWVkaWF0ZWssbXQyNzAxLXNtaS1jb21tb24nLA0KDQonbWVkaWF0 ZWssbXQyNzEyLXNtaS1jb21tb24nLA0KDQonbWVkaWF0ZWssbXQ2Nzc5LXNtaS1jb21tb24nLA0K DQonbWVkaWF0ZWssbXQ3NjIzLXNtaS1jb21tb24nLA0KDQonbWVkaWF0ZWssbXQ4MTczLXNtaS1j b21tb24nLA0KDQonbWVkaWF0ZWssbXQ4MTgzLXNtaS1jb21tb24nLA0KDQonbWVkaWF0ZWssbXQ4 MTkyLXNtaS1jb21tb24nXX19fSwNCiAgICAgICAgICAgICdyZXF1aXJlZCc6IFsnY29tcGF0aWJs ZSddfSwNCiAndGl0bGUnOiAnU01JIChTbWFydCBNdWx0aW1lZGlhIEludGVyZmFjZSkgQ29tbW9u J30NCglGcm9tDQpzY2hlbWE6IC4uLi9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3Mv bWVtb3J5LWNvbnRyb2xsZXJzL21lZGlhdGVrLHNtaS1jb21tb24ueWFtbA0KDQo9PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT0NCg0KU29ycnksIEkgZGlkbid0IGZpbmQgYSBleGFtcGxl IG9mIG11bHRpICJpZi10aGVuIiBpbiBvbmVPZiBhbmQgZG9uJ3QNCmtub3cgaG93IHRvIGZpeCB0 aGlzLg0KDQpGcm9tIFsxXSwgImFsbE9mIiBsb29rcyBhbHNvIGlzIGFsbG93ZWQuLi4gSW4gdGhp cyBjYXNlLCBib3RoIGFsbE9mIGFuZA0KYW55T2YgaGF2ZSBubyB3YXJuaW5nIHdoZW4gZHRfYmlu ZGluZ19jaGVjay4gb25seSBvbmVPZiBoYXMgdGhlIHdhcm5pbmcNCmxvZyBhYm92ZS4NCg0KQW55 IHN1Z2dlc3Rpb24gaXMgYXBwcmVjaWF0ZWQuDQoNClsxXQ0KaHR0cHM6Ly9lbGl4aXIuYm9vdGxp bi5jb20vbGludXgvdjUuOS1yYzEvc291cmNlL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5k aW5ncy9leGFtcGxlLXNjaGVtYS55YW1sI0wxOTUNCg0KPiANCj4gQmVzdCByZWdhcmRzLA0KPiBL cnp5c3p0b2YNCj4gDQo+ID4gICAgLSBpZjogI2dlbjEgaHcNCj4gPiAgICAgIHRoZW46DQo+ID4g ICAgICAgIHVzZSBhcGIvc21pL2FzeW5jIGNsb2Nrcw0KPiA+IA0KPiA+ICAgIC0gaWY6ICNnZW4y IGh3IHRoYXQgaGFzIGdhbHMuDQo+ID4gICAgICB0aGVuOg0KPiA+ICAgICAgICB1c2UgYXBiL3Nt aS9nYWxzMC9nYWxzMSBjbG9ja3MNCj4gPiAgICAgIGVsc2U6ICMgZ2VuMiBodyB0aGF0IGRvZXNu J3QgaGF2ZSBnYWxzLg0KPiA+ICAgICAgICB1c2UgYXBiL3NtaSBjbG9ja3MuDQoNCg== 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=-8.0 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY,URIBL_BLOCKED,USER_AGENT_SANE_2 autolearn=ham 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 80763C43467 for ; Mon, 12 Oct 2020 12:02:20 +0000 (UTC) Received: from whitealder.osuosl.org (smtp1.osuosl.org [140.211.166.138]) (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 1A8C121D7F for ; Mon, 12 Oct 2020 12:02:19 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=mediatek.com header.i=@mediatek.com header.b="TKHmU4GJ" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1A8C121D7F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=mediatek.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=iommu-bounces@lists.linux-foundation.org Received: from localhost (localhost [127.0.0.1]) by whitealder.osuosl.org (Postfix) with ESMTP id C2EF486A46; Mon, 12 Oct 2020 12:02:19 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from whitealder.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id S3E1j2LXlZpf; Mon, 12 Oct 2020 12:02:18 +0000 (UTC) Received: from lists.linuxfoundation.org (lf-lists.osuosl.org [140.211.9.56]) by whitealder.osuosl.org (Postfix) with ESMTP id 037C086A1D; Mon, 12 Oct 2020 12:02:18 +0000 (UTC) Received: from lf-lists.osuosl.org (localhost [127.0.0.1]) by lists.linuxfoundation.org (Postfix) with ESMTP id CF063C0052; Mon, 12 Oct 2020 12:02:17 +0000 (UTC) Received: from whitealder.osuosl.org (smtp1.osuosl.org [140.211.166.138]) by lists.linuxfoundation.org (Postfix) with ESMTP id 9755DC0051 for ; Mon, 12 Oct 2020 12:02:16 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by whitealder.osuosl.org (Postfix) with ESMTP id 84D8686A1D for ; Mon, 12 Oct 2020 12:02:16 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from whitealder.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 56QotKfPU196 for ; Mon, 12 Oct 2020 12:02:14 +0000 (UTC) X-Greylist: domain auto-whitelisted by SQLgrey-1.7.6 Received: from mailgw01.mediatek.com (unknown [1.203.163.78]) by whitealder.osuosl.org (Postfix) with ESMTP id 8BE4086A27 for ; Mon, 12 Oct 2020 12:02:12 +0000 (UTC) X-UUID: ffba327e58714f71a6d76e7305d4bd5f-20201012 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=mediatek.com; s=dk; h=Content-Transfer-Encoding:MIME-Version:Content-Type:References:In-Reply-To:Date:CC:To:From:Subject:Message-ID; bh=Bm1nD1Rni6NjuejkSKF6tuW+1OZNvmAl+g7ZgtGLPq8=; b=TKHmU4GJsUveGwDhBPx/LGSjxR3leBMs8N6y7bwJSxOlan0sk7oer/u9vRRlh3s8X7pSSOkMjjvVFrcXHQYITLDunKJ05E3CZDH3JfQSPA+Hjn/qPzO0MKzEwDmWiZq4c+B2nkb04cH3V8CaQuYNO+CUjBDthWz3zmRl1QjyOKU=; X-UUID: ffba327e58714f71a6d76e7305d4bd5f-20201012 Received: from mtkcas32.mediatek.inc [(172.27.4.253)] by mailgw01.mediatek.com (envelope-from ) (mailgw01.mediatek.com ESMTP with TLSv1.2 ECDHE-RSA-AES256-SHA384 256/256) with ESMTP id 1331938782; Mon, 12 Oct 2020 20:02:01 +0800 Received: from MTKCAS32.mediatek.inc (172.27.4.184) by MTKMBS31DR.mediatek.inc (172.27.6.102) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Mon, 12 Oct 2020 20:02:00 +0800 Received: from [10.17.3.153] (10.17.3.153) by MTKCAS32.mediatek.inc (172.27.4.170) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Mon, 12 Oct 2020 20:01:58 +0800 Message-ID: <1602504119.26323.54.camel@mhfsdcap03> Subject: Re: [PATCH v3 02/24] dt-bindings: memory: mediatek: Convert SMI to DT schema From: Yong Wu To: Krzysztof Kozlowski , Rob Herring Date: Mon, 12 Oct 2020 20:01:59 +0800 In-Reply-To: <20201012071843.GA1889@pi3> References: <20200930070647.10188-1-yong.wu@mediatek.com> <20200930070647.10188-3-yong.wu@mediatek.com> <20201002110831.GD6888@pi3> <1601958428.26323.26.camel@mhfsdcap03> <1602310691.26323.39.camel@mhfsdcap03> <20201012071843.GA1889@pi3> X-Mailer: Evolution 3.10.4-0ubuntu2 MIME-Version: 1.0 X-TM-SNTS-SMTP: 1FA9AD3A565A7DE7F33FFEDBC7D6A606B4EFC4E07A07FFF66367021A06DDA1692000:8 X-MTK: N Cc: youlin.pei@mediatek.com, devicetree@vger.kernel.org, kernel-team@android.com, Nicolas Boichat , srv_heupstream@mediatek.com, chao.hao@mediatek.com, Robin Murphy , Greg Kroah-Hartman , "linux-kernel@vger.kernel.org" , Evan Green , Tomasz Figa , iommu@lists.linux-foundation.org, Rob Herring , linux-mediatek@lists.infradead.org, Matthias Brugger , ming-fan.chen@mediatek.com, anan.sun@mediatek.com, Will Deacon , linux-arm-kernel@lists.infradead.org X-BeenThere: iommu@lists.linux-foundation.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: Development issues for Linux IOMMU support List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: iommu-bounces@lists.linux-foundation.org Sender: "iommu" On Mon, 2020-10-12 at 09:18 +0200, Krzysztof Kozlowski wrote: > On Sat, Oct 10, 2020 at 02:18:11PM +0800, Yong Wu wrote: > > On Tue, 2020-10-06 at 09:15 +0200, Krzysztof Kozlowski wrote: > > > On Tue, 6 Oct 2020 at 06:27, Yong Wu wrote: > > > > > > > > On Fri, 2020-10-02 at 13:08 +0200, Krzysztof Kozlowski wrote: > > > > > On Wed, Sep 30, 2020 at 03:06:25PM +0800, Yong Wu wrote: > > > > > > Convert MediaTek SMI to DT schema. > > > > > > > > > > > > Signed-off-by: Yong Wu > > > > > > --- > > > > > > .../mediatek,smi-common.txt | 49 --------- > > > > > > .../mediatek,smi-common.yaml | 100 ++++++++++++++++++ > > > > > > .../memory-controllers/mediatek,smi-larb.txt | 49 --------- > > > > > > .../memory-controllers/mediatek,smi-larb.yaml | 91 ++++++++++++++++ > > > > > > 4 files changed, 191 insertions(+), 98 deletions(-) > > > > > > delete mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.txt > > > > > > create mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.yaml > > > > > > delete mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.txt > > > > > > create mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.yaml > > > > ... > > > > > > +properties: > > > > > > + compatible: > > > > > > + oneOf: > > > > > > + - enum: > > > > > > + - mediatek,mt2701-smi-common > > > > > > + - mediatek,mt2712-smi-common > > > > > > + - mediatek,mt6779-smi-common > > > > > > + - mediatek,mt8173-smi-common > > > > > > + - mediatek,mt8183-smi-common > > > > > > + > > > > > > + - description: for mt7623 > > > > > > + items: > > > > > > + - const: mediatek,mt7623-smi-common > > > > > > + - const: mediatek,mt2701-smi-common > > > > > > + > > > > > > + reg: > > > > > > + maxItems: 1 > > > > > > + > > > > > > + clocks: > > > > > > + description: | > > > > > > + apb and smi are mandatory. the async is only for generation 1 smi HW. > > > > > > + gals(global async local sync) also is optional, here is the list which > > > > > > + require gals: mt6779 and mt8183. > > > > > > + minItems: 2 > > > > > > + maxItems: 4 > > > > > > + items: > > > > > > + - description: apb is Advanced Peripheral Bus clock, It's the clock for > > > > > > + setting the register. > > > > > > + - description: smi is the clock for transfer data and command. > > > > > > + - description: async is asynchronous clock, it help transform the smi clock > > > > > > + into the emi clock domain. > > > > > > + - description: gals0 is the path0 clock of gals. > > > > > > + - description: gals1 is the path1 clock of gals. > > > > > > + > > > > > > + clock-names: > > > > > > + oneOf: > > > > > > + - items: > > > > > > + - const: apb > > > > > > + - const: smi > > > > > > + - items: > > > > > > + - const: apb > > > > > > + - const: smi > > > > > > + - const: async > > > > > > + - items: > > > > > > + - const: apb > > > > > > + - const: smi > > > > > > + - const: gals0 > > > > > > + - const: gals1 > > > > > > > > > > Similarly to my comment to other properties, this requirement per > > > > > compatible should be part of the schema within 'if-then'. > > > > > > > > I'm not so familiar with this format. Do this has "if-then-'else > > > > if'-then-else"? > > > > > > These are mutually exclusive conditions, so you can skip else: > > > - if-then > > > - if-then > > > - if-then > > > It will be more readable then stacking 'if' under 'else' > > > > Thanks. I will use something like this: > > > > anyOf: > > Then it should be oneOf as only one condition can be valid. I did do this at the beginning. But I get a warning log when dt_binding_check. Below is my schema and the detailed warning log: //======================================= clocks: description: | xxxxx minItems: 2 maxItems: 4 items: - description: apb is the clock for setting the register. - description: smi is the clock for transfer data and command. - description: async is asynchronous clock. - description: gals0 is the path0 clock of gals. - description: gals1 is the path1 clock of gals. clock-names: minItems: 2 maxItems: 4 required: - compatible - reg - power-domains - clocks - clock-names oneOf: - if: #only for gen1 HW properties: compatible: contains: enum: - mediatek,mt2701-smi-common then: properties: clock: items: minItems: 3 maxItems: 3 clock-names: items: - const: apb - const: smi - const: async - if: #for gen2 HW that have gals properties: compatible: enum: - mediatek,mt6779-smi-common - mediatek,mt8183-smi-common - mediatek,mt8192-smi-common then: properties: clock: items: minItems: 4 maxItems: 4 clock-names: items: - const: apb - const: smi - const: gals0 - const: gals1 else: #for gen2 HW that don't have gals properties: clock: items: minItems: 2 maxItems: 2 clock-names: items: - const: apb - const: smi additionalProperties: false //===========warning log===================== Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.example.dt.yaml .../Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.example.dt.yaml: smi@14022000: {'compatible': ['mediatek,mt8173-smi-common'], 'reg': [[335683584, 4096]], 'power-domains': [[4294967295, 3]], 'clocks': [[4294967295, 1], [4294967295, 1]], 'clock-names': ['apb', 'smi'], '$nodename': ['smi@14022000']} is valid under each of {'else': {'properties': {'clock': {'items': {'maxItems': 2, 'minItems': 2}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}], 'maxItems': 2, 'minItems': 2, 'type': 'array'}}}, 'if': {'properties': {'compatible': {'additionalItems': False, 'items': [{'enum': ['mediatek,mt6779-smi-common', 'mediatek,mt8183-smi-common', 'mediatek,mt8192-smi-common']}], 'maxItems': 1, 'minItems': 1, 'type': 'array'}}}, 'then': {'properties': {'clock': {'items': {'maxItems': 4, 'minItems': 4}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}, {'const' : 'gals0'}, {'const': 'gals1'}], 'maxItems': 4, 'minItems': 4, 'type': 'array'}}}}, {'if': {'properties': {'compatible': {'contains': {'enum': ['mediatek,mt2701-smi-common']}}}}, 'then': {'properties': {'clock': {'items': {'maxItems': 3, 'minItems': 3}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}, {'const': 'async'}], 'maxItems': 3, 'minItems': 3, 'type': 'array'}}}} {'$filename': '.../Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.yaml', '$id': 'http://devicetree.org/schemas/memory-controllers/mediatek,smi-common.yaml#', '$schema': 'http://devicetree.org/meta-schemas/core.yaml#', '$select_validator': .Validator object at 0x7fb3df378908>, 'additionalProperties': False, 'oneOf': [{'if': {'properties': {'compatible': {'contains': {'enum': ['mediatek,mt2701-smi-common']}}}}, 'then': {'properties': {'clock': {'items': {'maxItems': 3, 'minItems': 3}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}, {'const': 'async'}], 'maxItems': 3, 'minItems': 3, 'type': 'array'}}}}, {'else': {'properties': {'clock': {'items': {'maxItems': 2, 'minItems': 2}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}], 'maxItems': 2, 'minItems': 2, 'type': 'array'}}}, 'if': {'properties': {'compatible': {'additionalItems': False, 'items': [{'enum': ['mediatek,mt6779-smi-common', 'mediatek,mt8183-smi-common', 'mediatek,mt8192-smi-common']}], 'maxItems': 1, 'minItems': 1, 'type': 'array'}}}, 'then': {'properties': {'clock': {'items': {'maxItems': 4, 'minItems': 4}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}, {'const': 'gals0'}, {'const': 'gals1'}], 'maxItems': 4, 'minItems': 4, 'type': 'array'}}}}], 'patternProperties': {'pinctrl-[0-9]+': True}, 'properties': {'$nodename': True, 'clock-names': {'maxItems': 4, 'minItems': 2}, 'clocks': {'additionalItems': False, 'items': [{}, {}, {}, {}, {}], 'maxItems': 4, 'minItems': 2, 'type': 'array'}, 'compatible': {'oneOf': [{'additionalItems': False, 'items': [{'enum': ['mediatek,mt2701-smi-common', 'mediatek,mt2712-smi-common', 'mediatek,mt6779-smi-common', 'mediatek,mt8173-smi-common', 'mediatek,mt8183-smi-common', 'mediatek,mt8192-smi-common']}], 'maxItems': 1, 'minItems': 1, 'type': 'array'}, {'additionalItems': False, 'items': [{'const': 'mediatek,mt7623-smi-common'}, {'const': 'mediatek,mt2701-smi-common'}], 'maxItems': 2, 'minItems': 2, 'type': 'array'}]}, 'phandle': True, 'pinctrl-names': True, 'power-domains': {'maxItems': 1, 'minItems': 1}, 'reg': {'maxItems': 1, 'minItems': 1}, 'status': True}, 'required': ['compatible', 'reg', 'power-domains', 'clocks', 'clock-names'], 'select': {'properties': {'compatible': {'contains': {'enum': ['mediatek,mt2701-smi-common', 'mediatek,mt2712-smi-common', 'mediatek,mt6779-smi-common', 'mediatek,mt7623-smi-common', 'mediatek,mt8173-smi-common', 'mediatek,mt8183-smi-common', 'mediatek,mt8192-smi-common']}}}, 'required': ['compatible']}, 'title': 'SMI (Smart Multimedia Interface) Common'} From schema: .../Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.yaml ================================= Sorry, I didn't find a example of multi "if-then" in oneOf and don't know how to fix this. >From [1], "allOf" looks also is allowed... In this case, both allOf and anyOf have no warning when dt_binding_check. only oneOf has the warning log above. Any suggestion is appreciated. [1] https://elixir.bootlin.com/linux/v5.9-rc1/source/Documentation/devicetree/bindings/example-schema.yaml#L195 > > Best regards, > Krzysztof > > > - if: #gen1 hw > > then: > > use apb/smi/async clocks > > > > - if: #gen2 hw that has gals. > > then: > > use apb/smi/gals0/gals1 clocks > > else: # gen2 hw that doesn't have gals. > > use apb/smi clocks. _______________________________________________ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu 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=-8.0 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY,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 34DCAC433DF for ; Mon, 12 Oct 2020 12:12:33 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 9292D206BE for ; Mon, 12 Oct 2020 12:12:32 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="cjAlMpeN"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=mediatek.com header.i=@mediatek.com header.b="Q83HusVU" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9292D206BE Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=mediatek.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:Date:To:From: Subject:Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=oL02qzLvzInLfOh83AYQGv9sI0NkaLWLE1KmQMB9t54=; b=cjAlMpeN5LFk6PoWTFdvHwdiR HjYjB1BKFsXpOgwse0M2IwmZSTjMxgVrpf/KKEx5FTkiXNZtaClW6k54RcH5XpgmxCXTc2E8OHv2W 5iNv1gCdqJXDvk9M7DwFkUjEDVMbBqCvIr1qnYWB/sEfcPvSwwu7Q0VMYLBFUonBeXTWtwmDPYVks jBdKDeOF84UZoCsgEPeunRFHEjq0ahUBDW9N60VAmX2n/SjH5aSYF577FRF234boO7sC9Xb6DAiO4 Bo4hsVCAfMy7ZMCUK7YPcJuf3FOtM5UHFK90c1udySIhIgtNE5HYB/FXPc48CfvXm+nZ9ZURWGG0u I1Ll7j+GA==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kRwgh-0007hT-H5; Mon, 12 Oct 2020 12:12:23 +0000 Received: from mailgw01.mediatek.com ([216.200.240.184]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kRwge-0007gj-Ej; Mon, 12 Oct 2020 12:12:22 +0000 X-UUID: e97ac839869f40e59bd9019c6b658804-20201012 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=mediatek.com; s=dk; h=Content-Transfer-Encoding:MIME-Version:Content-Type:References:In-Reply-To:Date:CC:To:From:Subject:Message-ID; bh=Bm1nD1Rni6NjuejkSKF6tuW+1OZNvmAl+g7ZgtGLPq8=; b=Q83HusVUoIPt4LmvIp0EA8lONkXn0OY0gdobVbwH/3zi8suo4ZoXxW9Zo1S+xJGOXbrRQ99JSDoys3ijdaufuACIg8gSwjZRMOnfKizKAwa4rD5pGY8DqwRU7fSG/16zp35lCZidwyWzhL/mQQpAnJvWAFv6piT3gm68QfktNRM=; X-UUID: e97ac839869f40e59bd9019c6b658804-20201012 Received: from mtkcas66.mediatek.inc [(172.29.193.44)] by mailgw01.mediatek.com (envelope-from ) (musrelay.mediatek.com ESMTP with TLSv1.2 ECDHE-RSA-AES256-SHA384 256/256) with ESMTP id 1401816764; Mon, 12 Oct 2020 04:12:06 -0800 Received: from MTKMBS31DR.mediatek.inc (172.27.6.102) by MTKMBS62DR.mediatek.inc (172.29.94.18) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Mon, 12 Oct 2020 05:02:03 -0700 Received: from MTKCAS32.mediatek.inc (172.27.4.184) by MTKMBS31DR.mediatek.inc (172.27.6.102) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Mon, 12 Oct 2020 20:02:00 +0800 Received: from [10.17.3.153] (10.17.3.153) by MTKCAS32.mediatek.inc (172.27.4.170) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Mon, 12 Oct 2020 20:01:58 +0800 Message-ID: <1602504119.26323.54.camel@mhfsdcap03> Subject: Re: [PATCH v3 02/24] dt-bindings: memory: mediatek: Convert SMI to DT schema From: Yong Wu To: Krzysztof Kozlowski , Rob Herring Date: Mon, 12 Oct 2020 20:01:59 +0800 In-Reply-To: <20201012071843.GA1889@pi3> References: <20200930070647.10188-1-yong.wu@mediatek.com> <20200930070647.10188-3-yong.wu@mediatek.com> <20201002110831.GD6888@pi3> <1601958428.26323.26.camel@mhfsdcap03> <1602310691.26323.39.camel@mhfsdcap03> <20201012071843.GA1889@pi3> X-Mailer: Evolution 3.10.4-0ubuntu2 MIME-Version: 1.0 X-TM-SNTS-SMTP: 1FA9AD3A565A7DE7F33FFEDBC7D6A606B4EFC4E07A07FFF66367021A06DDA1692000:8 X-MTK: N X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201012_081220_749112_9966A898 X-CRM114-Status: GOOD ( 24.85 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: youlin.pei@mediatek.com, devicetree@vger.kernel.org, kernel-team@android.com, Nicolas Boichat , srv_heupstream@mediatek.com, chao.hao@mediatek.com, Robin Murphy , Joerg Roedel , Greg Kroah-Hartman , "linux-kernel@vger.kernel.org" , Evan Green , Tomasz Figa , iommu@lists.linux-foundation.org, Rob Herring , linux-mediatek@lists.infradead.org, Matthias Brugger , ming-fan.chen@mediatek.com, anan.sun@mediatek.com, Will Deacon , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org On Mon, 2020-10-12 at 09:18 +0200, Krzysztof Kozlowski wrote: > On Sat, Oct 10, 2020 at 02:18:11PM +0800, Yong Wu wrote: > > On Tue, 2020-10-06 at 09:15 +0200, Krzysztof Kozlowski wrote: > > > On Tue, 6 Oct 2020 at 06:27, Yong Wu wrote: > > > > > > > > On Fri, 2020-10-02 at 13:08 +0200, Krzysztof Kozlowski wrote: > > > > > On Wed, Sep 30, 2020 at 03:06:25PM +0800, Yong Wu wrote: > > > > > > Convert MediaTek SMI to DT schema. > > > > > > > > > > > > Signed-off-by: Yong Wu > > > > > > --- > > > > > > .../mediatek,smi-common.txt | 49 --------- > > > > > > .../mediatek,smi-common.yaml | 100 ++++++++++++++++++ > > > > > > .../memory-controllers/mediatek,smi-larb.txt | 49 --------- > > > > > > .../memory-controllers/mediatek,smi-larb.yaml | 91 ++++++++++++++++ > > > > > > 4 files changed, 191 insertions(+), 98 deletions(-) > > > > > > delete mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.txt > > > > > > create mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.yaml > > > > > > delete mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.txt > > > > > > create mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.yaml > > > > ... > > > > > > +properties: > > > > > > + compatible: > > > > > > + oneOf: > > > > > > + - enum: > > > > > > + - mediatek,mt2701-smi-common > > > > > > + - mediatek,mt2712-smi-common > > > > > > + - mediatek,mt6779-smi-common > > > > > > + - mediatek,mt8173-smi-common > > > > > > + - mediatek,mt8183-smi-common > > > > > > + > > > > > > + - description: for mt7623 > > > > > > + items: > > > > > > + - const: mediatek,mt7623-smi-common > > > > > > + - const: mediatek,mt2701-smi-common > > > > > > + > > > > > > + reg: > > > > > > + maxItems: 1 > > > > > > + > > > > > > + clocks: > > > > > > + description: | > > > > > > + apb and smi are mandatory. the async is only for generation 1 smi HW. > > > > > > + gals(global async local sync) also is optional, here is the list which > > > > > > + require gals: mt6779 and mt8183. > > > > > > + minItems: 2 > > > > > > + maxItems: 4 > > > > > > + items: > > > > > > + - description: apb is Advanced Peripheral Bus clock, It's the clock for > > > > > > + setting the register. > > > > > > + - description: smi is the clock for transfer data and command. > > > > > > + - description: async is asynchronous clock, it help transform the smi clock > > > > > > + into the emi clock domain. > > > > > > + - description: gals0 is the path0 clock of gals. > > > > > > + - description: gals1 is the path1 clock of gals. > > > > > > + > > > > > > + clock-names: > > > > > > + oneOf: > > > > > > + - items: > > > > > > + - const: apb > > > > > > + - const: smi > > > > > > + - items: > > > > > > + - const: apb > > > > > > + - const: smi > > > > > > + - const: async > > > > > > + - items: > > > > > > + - const: apb > > > > > > + - const: smi > > > > > > + - const: gals0 > > > > > > + - const: gals1 > > > > > > > > > > Similarly to my comment to other properties, this requirement per > > > > > compatible should be part of the schema within 'if-then'. > > > > > > > > I'm not so familiar with this format. Do this has "if-then-'else > > > > if'-then-else"? > > > > > > These are mutually exclusive conditions, so you can skip else: > > > - if-then > > > - if-then > > > - if-then > > > It will be more readable then stacking 'if' under 'else' > > > > Thanks. I will use something like this: > > > > anyOf: > > Then it should be oneOf as only one condition can be valid. I did do this at the beginning. But I get a warning log when dt_binding_check. Below is my schema and the detailed warning log: //======================================= clocks: description: | xxxxx minItems: 2 maxItems: 4 items: - description: apb is the clock for setting the register. - description: smi is the clock for transfer data and command. - description: async is asynchronous clock. - description: gals0 is the path0 clock of gals. - description: gals1 is the path1 clock of gals. clock-names: minItems: 2 maxItems: 4 required: - compatible - reg - power-domains - clocks - clock-names oneOf: - if: #only for gen1 HW properties: compatible: contains: enum: - mediatek,mt2701-smi-common then: properties: clock: items: minItems: 3 maxItems: 3 clock-names: items: - const: apb - const: smi - const: async - if: #for gen2 HW that have gals properties: compatible: enum: - mediatek,mt6779-smi-common - mediatek,mt8183-smi-common - mediatek,mt8192-smi-common then: properties: clock: items: minItems: 4 maxItems: 4 clock-names: items: - const: apb - const: smi - const: gals0 - const: gals1 else: #for gen2 HW that don't have gals properties: clock: items: minItems: 2 maxItems: 2 clock-names: items: - const: apb - const: smi additionalProperties: false //===========warning log===================== Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.example.dt.yaml .../Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.example.dt.yaml: smi@14022000: {'compatible': ['mediatek,mt8173-smi-common'], 'reg': [[335683584, 4096]], 'power-domains': [[4294967295, 3]], 'clocks': [[4294967295, 1], [4294967295, 1]], 'clock-names': ['apb', 'smi'], '$nodename': ['smi@14022000']} is valid under each of {'else': {'properties': {'clock': {'items': {'maxItems': 2, 'minItems': 2}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}], 'maxItems': 2, 'minItems': 2, 'type': 'array'}}}, 'if': {'properties': {'compatible': {'additionalItems': False, 'items': [{'enum': ['mediatek,mt6779-smi-common', 'mediatek,mt8183-smi-common', 'mediatek,mt8192-smi-common']}], 'maxItems': 1, 'minItems': 1, 'type': 'array'}}}, 'then': {'properties': {'clock': {'items': {'maxItems': 4, 'minItems': 4}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}, {'const' : 'gals0'}, {'const': 'gals1'}], 'maxItems': 4, 'minItems': 4, 'type': 'array'}}}}, {'if': {'properties': {'compatible': {'contains': {'enum': ['mediatek,mt2701-smi-common']}}}}, 'then': {'properties': {'clock': {'items': {'maxItems': 3, 'minItems': 3}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}, {'const': 'async'}], 'maxItems': 3, 'minItems': 3, 'type': 'array'}}}} {'$filename': '.../Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.yaml', '$id': 'http://devicetree.org/schemas/memory-controllers/mediatek,smi-common.yaml#', '$schema': 'http://devicetree.org/meta-schemas/core.yaml#', '$select_validator': .Validator object at 0x7fb3df378908>, 'additionalProperties': False, 'oneOf': [{'if': {'properties': {'compatible': {'contains': {'enum': ['mediatek,mt2701-smi-common']}}}}, 'then': {'properties': {'clock': {'items': {'maxItems': 3, 'minItems': 3}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}, {'const': 'async'}], 'maxItems': 3, 'minItems': 3, 'type': 'array'}}}}, {'else': {'properties': {'clock': {'items': {'maxItems': 2, 'minItems': 2}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}], 'maxItems': 2, 'minItems': 2, 'type': 'array'}}}, 'if': {'properties': {'compatible': {'additionalItems': False, 'items': [{'enum': ['mediatek,mt6779-smi-common', 'mediatek,mt8183-smi-common', 'mediatek,mt8192-smi-common']}], 'maxItems': 1, 'minItems': 1, 'type': 'array'}}}, 'then': {'properties': {'clock': {'items': {'maxItems': 4, 'minItems': 4}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}, {'const': 'gals0'}, {'const': 'gals1'}], 'maxItems': 4, 'minItems': 4, 'type': 'array'}}}}], 'patternProperties': {'pinctrl-[0-9]+': True}, 'properties': {'$nodename': True, 'clock-names': {'maxItems': 4, 'minItems': 2}, 'clocks': {'additionalItems': False, 'items': [{}, {}, {}, {}, {}], 'maxItems': 4, 'minItems': 2, 'type': 'array'}, 'compatible': {'oneOf': [{'additionalItems': False, 'items': [{'enum': ['mediatek,mt2701-smi-common', 'mediatek,mt2712-smi-common', 'mediatek,mt6779-smi-common', 'mediatek,mt8173-smi-common', 'mediatek,mt8183-smi-common', 'mediatek,mt8192-smi-common']}], 'maxItems': 1, 'minItems': 1, 'type': 'array'}, {'additionalItems': False, 'items': [{'const': 'mediatek,mt7623-smi-common'}, {'const': 'mediatek,mt2701-smi-common'}], 'maxItems': 2, 'minItems': 2, 'type': 'array'}]}, 'phandle': True, 'pinctrl-names': True, 'power-domains': {'maxItems': 1, 'minItems': 1}, 'reg': {'maxItems': 1, 'minItems': 1}, 'status': True}, 'required': ['compatible', 'reg', 'power-domains', 'clocks', 'clock-names'], 'select': {'properties': {'compatible': {'contains': {'enum': ['mediatek,mt2701-smi-common', 'mediatek,mt2712-smi-common', 'mediatek,mt6779-smi-common', 'mediatek,mt7623-smi-common', 'mediatek,mt8173-smi-common', 'mediatek,mt8183-smi-common', 'mediatek,mt8192-smi-common']}}}, 'required': ['compatible']}, 'title': 'SMI (Smart Multimedia Interface) Common'} From schema: .../Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.yaml ================================= Sorry, I didn't find a example of multi "if-then" in oneOf and don't know how to fix this. >From [1], "allOf" looks also is allowed... In this case, both allOf and anyOf have no warning when dt_binding_check. only oneOf has the warning log above. Any suggestion is appreciated. [1] https://elixir.bootlin.com/linux/v5.9-rc1/source/Documentation/devicetree/bindings/example-schema.yaml#L195 > > Best regards, > Krzysztof > > > - if: #gen1 hw > > then: > > use apb/smi/async clocks > > > > - if: #gen2 hw that has gals. > > then: > > use apb/smi/gals0/gals1 clocks > > else: # gen2 hw that doesn't have gals. > > use apb/smi clocks. _______________________________________________ Linux-mediatek mailing list Linux-mediatek@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-mediatek 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=-8.0 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY,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 2F44BC433E7 for ; Mon, 12 Oct 2020 12:13:55 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 C03D320757 for ; Mon, 12 Oct 2020 12:13:54 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="pMoLTatP"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=mediatek.com header.i=@mediatek.com header.b="Q83HusVU" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C03D320757 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=mediatek.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:Date:To:From: Subject:Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=3rWEUqS3skkTkvR8VxFnsLXFFo24VfD4gLZ6MRLtDlk=; b=pMoLTatPDGaIHsnpsGIue7ocE DZb3sSE29MFVs6X3dJI5T6m+q74eIpo91ub/AF7+Q1MjUphWxoqQFNaMsQZ+RjCcIvX21WNi7gUGv 0VqHXpMKA2qtv2oQ9VwdZHCjr+9n/MdmUAv5emfCpICEq2xjvultQVI4x+0Tx+AnoF56wDwuaHOfd mDT6RFLc7AaSUfFDvW9l2N3tL+9liFk3hN7YqTNLWHYNvWgRHmerMlayK25NUAxIaJ4xYlX6+PvRw XZN2LNsbLbD12DS9mEn3qwfmkWOpxSby6BkQC1uP1InxXlDQvYqPnjKOKjoI1OZrGzzgozJu5+twB KbnWw7GGg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kRwgi-0007hb-Mh; Mon, 12 Oct 2020 12:12:24 +0000 Received: from mailgw01.mediatek.com ([216.200.240.184]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kRwge-0007gj-Ej; Mon, 12 Oct 2020 12:12:22 +0000 X-UUID: e97ac839869f40e59bd9019c6b658804-20201012 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=mediatek.com; s=dk; h=Content-Transfer-Encoding:MIME-Version:Content-Type:References:In-Reply-To:Date:CC:To:From:Subject:Message-ID; bh=Bm1nD1Rni6NjuejkSKF6tuW+1OZNvmAl+g7ZgtGLPq8=; b=Q83HusVUoIPt4LmvIp0EA8lONkXn0OY0gdobVbwH/3zi8suo4ZoXxW9Zo1S+xJGOXbrRQ99JSDoys3ijdaufuACIg8gSwjZRMOnfKizKAwa4rD5pGY8DqwRU7fSG/16zp35lCZidwyWzhL/mQQpAnJvWAFv6piT3gm68QfktNRM=; X-UUID: e97ac839869f40e59bd9019c6b658804-20201012 Received: from mtkcas66.mediatek.inc [(172.29.193.44)] by mailgw01.mediatek.com (envelope-from ) (musrelay.mediatek.com ESMTP with TLSv1.2 ECDHE-RSA-AES256-SHA384 256/256) with ESMTP id 1401816764; Mon, 12 Oct 2020 04:12:06 -0800 Received: from MTKMBS31DR.mediatek.inc (172.27.6.102) by MTKMBS62DR.mediatek.inc (172.29.94.18) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Mon, 12 Oct 2020 05:02:03 -0700 Received: from MTKCAS32.mediatek.inc (172.27.4.184) by MTKMBS31DR.mediatek.inc (172.27.6.102) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Mon, 12 Oct 2020 20:02:00 +0800 Received: from [10.17.3.153] (10.17.3.153) by MTKCAS32.mediatek.inc (172.27.4.170) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Mon, 12 Oct 2020 20:01:58 +0800 Message-ID: <1602504119.26323.54.camel@mhfsdcap03> Subject: Re: [PATCH v3 02/24] dt-bindings: memory: mediatek: Convert SMI to DT schema From: Yong Wu To: Krzysztof Kozlowski , Rob Herring Date: Mon, 12 Oct 2020 20:01:59 +0800 In-Reply-To: <20201012071843.GA1889@pi3> References: <20200930070647.10188-1-yong.wu@mediatek.com> <20200930070647.10188-3-yong.wu@mediatek.com> <20201002110831.GD6888@pi3> <1601958428.26323.26.camel@mhfsdcap03> <1602310691.26323.39.camel@mhfsdcap03> <20201012071843.GA1889@pi3> X-Mailer: Evolution 3.10.4-0ubuntu2 MIME-Version: 1.0 X-TM-SNTS-SMTP: 1FA9AD3A565A7DE7F33FFEDBC7D6A606B4EFC4E07A07FFF66367021A06DDA1692000:8 X-MTK: N X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201012_081220_749112_9966A898 X-CRM114-Status: GOOD ( 24.85 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: youlin.pei@mediatek.com, devicetree@vger.kernel.org, kernel-team@android.com, Nicolas Boichat , srv_heupstream@mediatek.com, chao.hao@mediatek.com, Robin Murphy , Joerg Roedel , Greg Kroah-Hartman , "linux-kernel@vger.kernel.org" , Evan Green , Tomasz Figa , iommu@lists.linux-foundation.org, Rob Herring , linux-mediatek@lists.infradead.org, Matthias Brugger , ming-fan.chen@mediatek.com, anan.sun@mediatek.com, Will Deacon , linux-arm-kernel@lists.infradead.org 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 On Mon, 2020-10-12 at 09:18 +0200, Krzysztof Kozlowski wrote: > On Sat, Oct 10, 2020 at 02:18:11PM +0800, Yong Wu wrote: > > On Tue, 2020-10-06 at 09:15 +0200, Krzysztof Kozlowski wrote: > > > On Tue, 6 Oct 2020 at 06:27, Yong Wu wrote: > > > > > > > > On Fri, 2020-10-02 at 13:08 +0200, Krzysztof Kozlowski wrote: > > > > > On Wed, Sep 30, 2020 at 03:06:25PM +0800, Yong Wu wrote: > > > > > > Convert MediaTek SMI to DT schema. > > > > > > > > > > > > Signed-off-by: Yong Wu > > > > > > --- > > > > > > .../mediatek,smi-common.txt | 49 --------- > > > > > > .../mediatek,smi-common.yaml | 100 ++++++++++++++++++ > > > > > > .../memory-controllers/mediatek,smi-larb.txt | 49 --------- > > > > > > .../memory-controllers/mediatek,smi-larb.yaml | 91 ++++++++++++++++ > > > > > > 4 files changed, 191 insertions(+), 98 deletions(-) > > > > > > delete mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.txt > > > > > > create mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.yaml > > > > > > delete mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.txt > > > > > > create mode 100644 Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.yaml > > > > ... > > > > > > +properties: > > > > > > + compatible: > > > > > > + oneOf: > > > > > > + - enum: > > > > > > + - mediatek,mt2701-smi-common > > > > > > + - mediatek,mt2712-smi-common > > > > > > + - mediatek,mt6779-smi-common > > > > > > + - mediatek,mt8173-smi-common > > > > > > + - mediatek,mt8183-smi-common > > > > > > + > > > > > > + - description: for mt7623 > > > > > > + items: > > > > > > + - const: mediatek,mt7623-smi-common > > > > > > + - const: mediatek,mt2701-smi-common > > > > > > + > > > > > > + reg: > > > > > > + maxItems: 1 > > > > > > + > > > > > > + clocks: > > > > > > + description: | > > > > > > + apb and smi are mandatory. the async is only for generation 1 smi HW. > > > > > > + gals(global async local sync) also is optional, here is the list which > > > > > > + require gals: mt6779 and mt8183. > > > > > > + minItems: 2 > > > > > > + maxItems: 4 > > > > > > + items: > > > > > > + - description: apb is Advanced Peripheral Bus clock, It's the clock for > > > > > > + setting the register. > > > > > > + - description: smi is the clock for transfer data and command. > > > > > > + - description: async is asynchronous clock, it help transform the smi clock > > > > > > + into the emi clock domain. > > > > > > + - description: gals0 is the path0 clock of gals. > > > > > > + - description: gals1 is the path1 clock of gals. > > > > > > + > > > > > > + clock-names: > > > > > > + oneOf: > > > > > > + - items: > > > > > > + - const: apb > > > > > > + - const: smi > > > > > > + - items: > > > > > > + - const: apb > > > > > > + - const: smi > > > > > > + - const: async > > > > > > + - items: > > > > > > + - const: apb > > > > > > + - const: smi > > > > > > + - const: gals0 > > > > > > + - const: gals1 > > > > > > > > > > Similarly to my comment to other properties, this requirement per > > > > > compatible should be part of the schema within 'if-then'. > > > > > > > > I'm not so familiar with this format. Do this has "if-then-'else > > > > if'-then-else"? > > > > > > These are mutually exclusive conditions, so you can skip else: > > > - if-then > > > - if-then > > > - if-then > > > It will be more readable then stacking 'if' under 'else' > > > > Thanks. I will use something like this: > > > > anyOf: > > Then it should be oneOf as only one condition can be valid. I did do this at the beginning. But I get a warning log when dt_binding_check. Below is my schema and the detailed warning log: //======================================= clocks: description: | xxxxx minItems: 2 maxItems: 4 items: - description: apb is the clock for setting the register. - description: smi is the clock for transfer data and command. - description: async is asynchronous clock. - description: gals0 is the path0 clock of gals. - description: gals1 is the path1 clock of gals. clock-names: minItems: 2 maxItems: 4 required: - compatible - reg - power-domains - clocks - clock-names oneOf: - if: #only for gen1 HW properties: compatible: contains: enum: - mediatek,mt2701-smi-common then: properties: clock: items: minItems: 3 maxItems: 3 clock-names: items: - const: apb - const: smi - const: async - if: #for gen2 HW that have gals properties: compatible: enum: - mediatek,mt6779-smi-common - mediatek,mt8183-smi-common - mediatek,mt8192-smi-common then: properties: clock: items: minItems: 4 maxItems: 4 clock-names: items: - const: apb - const: smi - const: gals0 - const: gals1 else: #for gen2 HW that don't have gals properties: clock: items: minItems: 2 maxItems: 2 clock-names: items: - const: apb - const: smi additionalProperties: false //===========warning log===================== Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.example.dt.yaml .../Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.example.dt.yaml: smi@14022000: {'compatible': ['mediatek,mt8173-smi-common'], 'reg': [[335683584, 4096]], 'power-domains': [[4294967295, 3]], 'clocks': [[4294967295, 1], [4294967295, 1]], 'clock-names': ['apb', 'smi'], '$nodename': ['smi@14022000']} is valid under each of {'else': {'properties': {'clock': {'items': {'maxItems': 2, 'minItems': 2}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}], 'maxItems': 2, 'minItems': 2, 'type': 'array'}}}, 'if': {'properties': {'compatible': {'additionalItems': False, 'items': [{'enum': ['mediatek,mt6779-smi-common', 'mediatek,mt8183-smi-common', 'mediatek,mt8192-smi-common']}], 'maxItems': 1, 'minItems': 1, 'type': 'array'}}}, 'then': {'properties': {'clock': {'items': {'maxItems': 4, 'minItems': 4}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}, {'const' : 'gals0'}, {'const': 'gals1'}], 'maxItems': 4, 'minItems': 4, 'type': 'array'}}}}, {'if': {'properties': {'compatible': {'contains': {'enum': ['mediatek,mt2701-smi-common']}}}}, 'then': {'properties': {'clock': {'items': {'maxItems': 3, 'minItems': 3}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}, {'const': 'async'}], 'maxItems': 3, 'minItems': 3, 'type': 'array'}}}} {'$filename': '.../Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.yaml', '$id': 'http://devicetree.org/schemas/memory-controllers/mediatek,smi-common.yaml#', '$schema': 'http://devicetree.org/meta-schemas/core.yaml#', '$select_validator': .Validator object at 0x7fb3df378908>, 'additionalProperties': False, 'oneOf': [{'if': {'properties': {'compatible': {'contains': {'enum': ['mediatek,mt2701-smi-common']}}}}, 'then': {'properties': {'clock': {'items': {'maxItems': 3, 'minItems': 3}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}, {'const': 'async'}], 'maxItems': 3, 'minItems': 3, 'type': 'array'}}}}, {'else': {'properties': {'clock': {'items': {'maxItems': 2, 'minItems': 2}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}], 'maxItems': 2, 'minItems': 2, 'type': 'array'}}}, 'if': {'properties': {'compatible': {'additionalItems': False, 'items': [{'enum': ['mediatek,mt6779-smi-common', 'mediatek,mt8183-smi-common', 'mediatek,mt8192-smi-common']}], 'maxItems': 1, 'minItems': 1, 'type': 'array'}}}, 'then': {'properties': {'clock': {'items': {'maxItems': 4, 'minItems': 4}, 'type': 'array'}, 'clock-names': {'additionalItems': False, 'items': [{'const': 'apb'}, {'const': 'smi'}, {'const': 'gals0'}, {'const': 'gals1'}], 'maxItems': 4, 'minItems': 4, 'type': 'array'}}}}], 'patternProperties': {'pinctrl-[0-9]+': True}, 'properties': {'$nodename': True, 'clock-names': {'maxItems': 4, 'minItems': 2}, 'clocks': {'additionalItems': False, 'items': [{}, {}, {}, {}, {}], 'maxItems': 4, 'minItems': 2, 'type': 'array'}, 'compatible': {'oneOf': [{'additionalItems': False, 'items': [{'enum': ['mediatek,mt2701-smi-common', 'mediatek,mt2712-smi-common', 'mediatek,mt6779-smi-common', 'mediatek,mt8173-smi-common', 'mediatek,mt8183-smi-common', 'mediatek,mt8192-smi-common']}], 'maxItems': 1, 'minItems': 1, 'type': 'array'}, {'additionalItems': False, 'items': [{'const': 'mediatek,mt7623-smi-common'}, {'const': 'mediatek,mt2701-smi-common'}], 'maxItems': 2, 'minItems': 2, 'type': 'array'}]}, 'phandle': True, 'pinctrl-names': True, 'power-domains': {'maxItems': 1, 'minItems': 1}, 'reg': {'maxItems': 1, 'minItems': 1}, 'status': True}, 'required': ['compatible', 'reg', 'power-domains', 'clocks', 'clock-names'], 'select': {'properties': {'compatible': {'contains': {'enum': ['mediatek,mt2701-smi-common', 'mediatek,mt2712-smi-common', 'mediatek,mt6779-smi-common', 'mediatek,mt7623-smi-common', 'mediatek,mt8173-smi-common', 'mediatek,mt8183-smi-common', 'mediatek,mt8192-smi-common']}}}, 'required': ['compatible']}, 'title': 'SMI (Smart Multimedia Interface) Common'} From schema: .../Documentation/devicetree/bindings/memory-controllers/mediatek,smi-common.yaml ================================= Sorry, I didn't find a example of multi "if-then" in oneOf and don't know how to fix this. >From [1], "allOf" looks also is allowed... In this case, both allOf and anyOf have no warning when dt_binding_check. only oneOf has the warning log above. Any suggestion is appreciated. [1] https://elixir.bootlin.com/linux/v5.9-rc1/source/Documentation/devicetree/bindings/example-schema.yaml#L195 > > Best regards, > Krzysztof > > > - if: #gen1 hw > > then: > > use apb/smi/async clocks > > > > - if: #gen2 hw that has gals. > > then: > > use apb/smi/gals0/gals1 clocks > > else: # gen2 hw that doesn't have gals. > > use apb/smi clocks. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel