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 1A1C3C77B75 for ; Wed, 17 May 2023 08:48:42 +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:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Content-ID:In-Reply-To: References:Message-ID:Date:Subject:To:From:Reply-To:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=A+gAz1r/aEYozJN+tfQRsYDrHy/hVYupSkftB3ewAPQ=; b=1IZ/x4sbSa17uR Zv4jUbzuUDjdmT7TUp3Z62n/bkiSZ97IaLnyxbzgyDzoam806djMsOLFfctOnJUULCXyrWnuYIw+G AkwkIMA6xpuVF2g2XBDYZcXZvoWoOstqY7PZ/MFwAYRNPJqefcT9Vd8pjtDoM7TqRdB55jek3WmAA 4DZtKhV71fzNtkJ0tTFEDPDfaCCINekr4gLucscR89PkvmmFfrvMZSQsGhQfChcocaoqRjRK6TQ7+ V9+EryAym1Doxj5dIZLyLKpCVeuyj6Gw0AIYxEYmuk+yMl8C4HktcA1uK+08uBbEyZCb+e4MuyzT+ wpSJ82TJKnZt0VywBfNQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1pzCpV-008vLs-1S; Wed, 17 May 2023 08:48:17 +0000 Received: from esa.microchip.iphmx.com ([68.232.154.123]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1pzCpR-008vKi-1J for linux-arm-kernel@lists.infradead.org; Wed, 17 May 2023 08:48:15 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1684313293; x=1715849293; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=ksGvY8CSGyG+wWhAiQl6PPnLJNHCXtj0zuKlFPq78yw=; b=xRRQynIgtWrT4Y7YIpfZ+hL6+f5i4+AOxb/ELIRbzkf7unqj6je8I0EO WXO5peZgyPrW7BCIjOfVWhj56WT6r1rIckh87p3Lowi0rm9mV7KxT9uJc As2CKe3POqJoK8tDcd0ImuY9j2vr+tN0YBO86pUIl+FoWC1jwFKqUR1ZE 8BPp5sTS8HE6wXiwa/uLiPrzNr30htSRCTDkpzaG4+oTlZCAbvS5le1f/ dMrVJNwqcy3tv6uujZ3Yeydf/Qppew6KI2tKRYDr9BNq4wgeMglxZreUY q+TX4aEjO6VnLHc56UfpizdXhsa4g9hR2sLHIgaeRlD0cFeHOs/xn2uqP w==; X-IronPort-AV: E=Sophos;i="5.99,281,1677567600"; d="scan'208";a="215845908" X-Amp-Result: SKIPPED(no attachment in message) Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa2.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 17 May 2023 01:48:06 -0700 Received: from chn-vm-ex01.mchp-main.com (10.10.85.143) by chn-vm-ex01.mchp-main.com (10.10.85.143) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.21; Wed, 17 May 2023 01:48:06 -0700 Received: from NAM12-MW2-obe.outbound.protection.outlook.com (10.10.215.89) by email.microchip.com (10.10.87.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.21 via Frontend Transport; Wed, 17 May 2023 01:48:06 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=JxBjaJVpJLGOcVLKb0mAzYceI4xg16JVPDpJ9n6Gfme2J5ubeteH090xqCCVY8iI9eoq4P3IEcPOLXv/oDh2KW79f6oAtticJPzyPG0d00ZsXM1X5BpCEZVfGBRuzAEdmdFJfyYPAALKq4h4asolgPbx+8JzeH6KzPLvSpLTx/HTwYYC7n+uYw/vTSuy0akb4F6Jm3pMfOs6Y+aQ9vy0ATqSbNhrmjty/kOSWYzXbo5AdZRCzKRjywDzPJ3XGwX0x4QB7t7t66cLwC6rp0RM9GKchp78xaFfCAxJPHGXO8jfetP8rX7jMYKxBy+oyUGmPoFuMO2D1AuNysgtE8cjvA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=ksGvY8CSGyG+wWhAiQl6PPnLJNHCXtj0zuKlFPq78yw=; b=JeihitNA6lP8on/3X++C3F/hDokfSMTEPzEAQc2DnG7nu0Q69kV67/KIdBmNzyB7uFugT8wzO3G6iBi6miBNB4mVVx+7XsXhNXH+mKR11w8nAyrEjFde4Gh95HY3rU6QGlAFWJ3NJ6Gx3fulkSqlQlpHG+hWQgJDnCKEnwHyPMWb06xhY6WvXUK4uM0hDHrEzc3WrS3HgrgcqnKxdWen5CN6q62/bMa1q0dtLY+5cLv8r30nz7OZQY9ucf5yRvkEN70+p+bSAKycmHl/vNLYjPw3sg3AVXUkCsEQ07fEr6qxbK3Knny2R55DAeFplGiPNQ39HYV3rGi3ckoJvJzwUQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microchip.com; dmarc=pass action=none header.from=microchip.com; dkim=pass header.d=microchip.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microchiptechnology.onmicrosoft.com; s=selector2-microchiptechnology-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ksGvY8CSGyG+wWhAiQl6PPnLJNHCXtj0zuKlFPq78yw=; b=l5pKDjpTv0zG+CiwmapMKTdgSwlBLnH3FvVwFGXNb5kkcOexqyaxnSJryb9dy0A6BK/HSoKJynJ1Dds+62QE36n5Q1M42XDckf4SsBPCICARODB9KtSEqrTlafzWzlZzUt2nTehKUN/JhrbefR/Km3Mpy688WqcPCy5P9biwuR8= Received: from SJ2PR11MB7648.namprd11.prod.outlook.com (2603:10b6:a03:4c3::17) by CO6PR11MB5635.namprd11.prod.outlook.com (2603:10b6:5:35f::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6411.17; Wed, 17 May 2023 08:48:04 +0000 Received: from SJ2PR11MB7648.namprd11.prod.outlook.com ([fe80::27bf:a69f:806f:67be]) by SJ2PR11MB7648.namprd11.prod.outlook.com ([fe80::27bf:a69f:806f:67be%5]) with mapi id 15.20.6387.030; Wed, 17 May 2023 08:48:04 +0000 From: To: Subject: Re: [PATCH v4 2/5] dt-bindings: clocks: atmel,at91rm9200-pmc: convert to yaml Thread-Topic: [PATCH v4 2/5] dt-bindings: clocks: atmel,at91rm9200-pmc: convert to yaml Thread-Index: AQHZiJxLlS7j96kN5EubResGVVnG+Q== Date: Wed, 17 May 2023 08:48:04 +0000 Message-ID: <131f3527-8ad8-bdd2-036d-6c8c61f69142@microchip.com> References: <20230516051836.2511149-1-claudiu.beznea@microchip.com> <20230516051836.2511149-3-claudiu.beznea@microchip.com> <20230516-modulator-reason-1d3a754c6dd7@wendy> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microchip.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: SJ2PR11MB7648:EE_|CO6PR11MB5635:EE_ x-ms-office365-filtering-correlation-id: 6ffd77e0-4ac6-427e-466d-08db56b36dc1 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: eBQdL2JZKy85/4gGrTTJZGKKRbMlyx3Jb7F/uRVtLDsaEREtpnkSyV2tL1ZllA52k2yu4LwLapHhEjZp540yMzkQzfI5nEKXZdXemfK0N97MfWxr+qUmXt7TFHnlGAjkw6YWJE/Uq1/o3kjS7Lqv0/Z7t056xLb1nxxPC0vNbN7eVkn+g3oukTaq5meS6ZgrTt2zc3kR97jJXNz3szO6dQVPZds/DIql/pi2jkXS64Vk+nxRoa5w27NSu/PukF1dxnVSB5A2yd+ICdMloJfUy9GJUp1V7tVhmPzH4I/AbdA3tiLTtZgcgtOUfV0UKS/hEOYKsPDIfQs6oTM5DlJnVmzpmXAeq2ayS55x+935iAlZrCxRrNZiHHVBFg5FMSPm2ZwDO2ROAmKglOY8WaF2ukGdeoAMrwvCg11GHOSWkexe2i36/rba24X75bQsUr3R5lGqpeBLfHdcj8s/dtgsyktBJlaCQly0rDUKysoqbZfAGOCvPytMrwNfU1WS45nzCqFzXD1S6XndU7GsHgevNMEDe7K4wagH7vnochrULZpwopA6yyCXsZ572TquayEjhT9H4HmvkD1UE1Ag30M/wgBEU1YbIkAnxiYKyTKJE8zB6h5V5O8sGBMGxoJGyXeCEmspzIwzsQ3gy3DpdDtQkvggi8EkxiVmDnzf4WL6A7JxbBknoKeAGmi1MGBGSXyU4IHDcdRjze7yKS4iqLbGPw== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SJ2PR11MB7648.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(136003)(346002)(366004)(376002)(396003)(39860400002)(451199021)(31686004)(966005)(316002)(4326008)(6636002)(76116006)(66946007)(66556008)(66476007)(66446008)(64756008)(478600001)(37006003)(91956017)(54906003)(36756003)(38070700005)(31696002)(86362001)(83380400001)(26005)(6506007)(6512007)(2616005)(186003)(53546011)(41300700001)(5660300002)(8936002)(6862004)(8676002)(6486002)(2906002)(71200400001)(122000001)(38100700002)(43740500002)(45980500001);DIR:OUT;SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?dXIweHFkdW9ud0Y4RG5rOUFza2JtYkZPY0FUZVI5V3FIS1U0MDZhK0pvMzRs?= =?utf-8?B?L1ZMZUZ0R052UDIxRjg3V0hVeUNDVHlZRFlYWC83ZzNVM0k3RWFVSFdTRE1Y?= =?utf-8?B?c1d3cVdPamNuaWd6a2tVem9DdTBUL2xZbmE3MFRld0NBbDRKV2h5aVA1MmZk?= =?utf-8?B?ekpTUkZPNjAxcTFmQVdvejM4bGVQVWNoMGsxVEJEcS9USTFQNjZxaWxIcHdo?= =?utf-8?B?VXRpdUd0bFFCTUFrUkdHOVhiU0Jlc09GN3Iya1ZjM1R5MVZpOW5pYkh3SFd5?= =?utf-8?B?aitpbzhZWlloM0lRQkRPb2ZnKzFkdGdJajVTaDNjMTh6RWhtRDhLUENaOFB1?= =?utf-8?B?bzh3RWhJWHpDWUVORmdwN1dqL2V2enpXUmtSVTZ4c0QrZlUrQTJvVjhWWmpN?= =?utf-8?B?TFgzWnZyMmRSdmw2ZlpPVk9DSVd2RllFb1BsVWtXNmw2ci90NWZ0bnJuMkRK?= =?utf-8?B?SmQ3WU9rMzVCY3hobmxmMHUxNDRVZUNnSHhLOHNOMDZFY1ZyZmM4Y2Z1Rjlk?= =?utf-8?B?MW1Rd3lLNjFHZERqUWhUUjIzWHFiNURzTUJFYWkxSC82dks1cmxUTFhXeHYy?= =?utf-8?B?Y3FoOUl5VUVzSzFPVWZHa0xaWUlBSHErOXRIZytGRUxDeXJIT3J4YVhPeW1h?= =?utf-8?B?dloxaEZIcmxPZFBWUitvbVRnMzR0UC9XaVNlVjZJOGdBaVdMNmhwTkQzT3ZN?= =?utf-8?B?UlQ1Y052V3cwUXllL0VjbWQ3dktreVhuTXZ5WEFvYmZzcDhVTS9xTEVUZEFN?= =?utf-8?B?bGs1RVdZZjQ0dXJROHphQy9XT0pkVDFKVWR6WWlQV1dWK3FTMUJRcXgydEhF?= =?utf-8?B?YzM0d2pUYzBNRmQwRGJMZHpsMVNndjIvbVpSQkIvcEZjK2VXK3pUeEwrZFox?= =?utf-8?B?K1VXbTkxY3dUODA0YllkSVhYQXM1YkdDM1VXK2t1NlcyVE9GV0VjeFRibWl1?= =?utf-8?B?NW5jcGVGaWMwU2RIaEVEVkdDNDVaVElNeS9Ubm1GTHZtbE9rS2owODJGd1pC?= =?utf-8?B?ZEk3aVY4dmdDOEtWWmREM2IvTnQzT1VxUXJNUEVnTDFsalJoTHhXOCtaMEJn?= =?utf-8?B?SmxiVXJmRmZ1UGJhNmVrVGovdUswTGZqeUlwdndaZ0VhOTZEZGM4TUUyemVM?= =?utf-8?B?UXNzV2lHa0tXS3orLzVXRGU1WDlYQWNBZm83TnBGUFZScDVKR2VDeStDWWhT?= =?utf-8?B?aEs3YnpFbzMvakpUZmducTNUT1YyOWs2MmhycEpWM3FyZGpRRnlzQUQwT0tX?= =?utf-8?B?NFhGOWd4SDRXdGROVHZmazhOZFUxOGhmNnZQREtmSW14cHd2Nkd5T2d0Kys3?= =?utf-8?B?TVhtQmk5bkY2cHVuTnlzTGhnV0FHSTNYT01RZFpCcVJBb2V2VHVXalJnZVBj?= =?utf-8?B?SElFK2pVTndRNThIRmR0RVdCeUF2OVQ5Z2I5alE0R1hKaFBtU3k5VWxMeWN5?= =?utf-8?B?Z05YNXlKanFvaUF5eFkzeVdUSXRrUlFjZTdJUThFZUsvWS85MFZ6MHpGV0wr?= =?utf-8?B?TUptdTByMWprcENJOGJibC9tWW16enJ1WlRtU3NXMit3bXF4OW5MKzBTTDJB?= =?utf-8?B?ZUExTXNwRmVqdStUK21UVlgvbTE4NU12RjJDcnFoMk1KcCtWNkFUTXhLYWJy?= =?utf-8?B?Q2NhODlCdmRmWHpoam96b28wODZ5MmloQ1JBSjNYeTY1RndoeGxFZlZjbGpL?= =?utf-8?B?djRYdFc1bmN3SFk2YzNFVURVUWNKZWxSZ1pLOFB2dHJLQmVBWjFteW5ZVWdV?= =?utf-8?B?cWk2c2VzV0FYajJIV0RCWndzYVhoNFk3dVU5NnhLRTJLWGxDUmZhY0dTN2FR?= =?utf-8?B?N1JvQUlJZ3BzM2hsVkZYamRZSjhtdHpiODkzVGNQRXdTYnFEelNFM29mMTRB?= =?utf-8?B?MFEzc2U3Mkl5VVpDWjdncXZjVUUrSTdoWStDM3dNemhPR0VFUmR3TWRIRTdL?= =?utf-8?B?K0c4aXgrYkhmZWZkWEZpUzBIMTVrMGVjeW1qVzMwZkt4RER3Nm5pV1NmdGgw?= =?utf-8?B?UFg3TzdZcTFRMnBnVGgzOUluQ2FCWEZDay94OEZxLytRTFhSYTUrenpYZkVu?= =?utf-8?B?bWRaRG03ZzRuWnpaTUQzaVpLa0IyNHViMG1hb01maUorRzc5amswWFBrMThD?= =?utf-8?Q?rXbJjcUQ90H0d2Vac4e0pOJFs?= Content-ID: MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7648.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6ffd77e0-4ac6-427e-466d-08db56b36dc1 X-MS-Exchange-CrossTenant-originalarrivaltime: 17 May 2023 08:48:04.3465 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 3f4057f3-b418-4d4e-ba84-d55b4e897d88 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: U2BSbKIv8A04OYw7Pqp3kuOxgnSYAlqPIicI5U95eWPdxQA8qBGuros139VyW/GkAfhA4Fx50WGQDkOFBWCZ4KyXQ2Fw0kmJxNDgdnO3E9A= X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO6PR11MB5635 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230517_014813_542606_3D1FE39A X-CRM114-Status: GOOD ( 19.50 ) 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: , Cc: devicetree@vger.kernel.org, alexandre.belloni@bootlin.com, sboyd@kernel.org, mturquette@baylibre.com, linux-kernel@vger.kernel.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, linux-clk@vger.kernel.org, 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 16.05.2023 15:58, Claudiu Beznea - M18063 wrote: > Hi, Conor, > > On 16.05.2023 15:00, Conor Dooley wrote: >> Hey Claudiu, >> >> On Tue, May 16, 2023 at 08:18:33AM +0300, Claudiu Beznea wrote: >>> Convert Atmel PMC documentation to yaml. Along with it clock names >>> were adapted according to the current available device trees as >>> different controller versions accept different clock (some of them >>> have 3 clocks as input, some has 2 clocks as inputs and some with 2 >>> input clocks uses different clock names). >>> diff --git a/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >>> new file mode 100644 >>> index 000000000000..e5f514bc4bf7 >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >>> @@ -0,0 +1,153 @@ >>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >>> +%YAML 1.2 >>> +--- >>> +$id: http://devicetree.org/schemas/clock/atmel,at91rm9200-pmc.yaml# >>> +$schema: http://devicetree.org/meta-schemas/core.yaml# >>> + >>> +title: Atmel Power Management Controller (PMC) >>> + >>> +maintainers: >>> + - Claudiu Beznea >>> + >>> +description: >>> + The power management controller optimizes power consumption by controlling all >>> + system and user peripheral clocks. The PMC enables/disables the clock inputs >>> + to many of the peripherals and to the processor. >>> + >>> +properties: >>> + compatible: >>> + oneOf: >>> + - items: >>> + - const: atmel,at91sam9g20-pmc >>> + - const: atmel,at91sam9260-pmc >>> + - const: syscon >>> + - items: >>> + - enum: >>> + - atmel,at91sam9g15-pmc >>> + - atmel,at91sam9g25-pmc >>> + - atmel,at91sam9g35-pmc >>> + - atmel,at91sam9x25-pmc >>> + - atmel,at91sam9x35-pmc >>> + - const: atmel,at91sam9x5-pmc >> Yet another combinations question for you... >> With this binding the following is not possible: >> >> "atmel,at91sam9x5-pmc", "syscon" >> >> Is that intended? > > No, I've just missed it. Same for the above. I'll have a new round and fix it. Though... shouldn't this have been detected by make dtbs_check? > > Thanks for having a look at this, > Claudiu > >> I notice "atmel,at91sam9260-pmc" is able to appear as: >> >> "atmel,at91sam9260-pmc", "syscon" >> >> So the inconsistency stands out. >> >>> + - const: syscon >>> + - items: >>> + - enum: >>> + - atmel,at91rm9200-pmc >>> + - atmel,at91sam9260-pmc >>> + - atmel,at91sam9g45-pmc >>> + - atmel,at91sam9n12-pmc >>> + - atmel,at91sam9rl-pmc >>> + - atmel,sama5d2-pmc >>> + - atmel,sama5d3-pmc >>> + - atmel,sama5d4-pmc >>> + - microchip,sam9x60-pmc >>> + - microchip,sama7g5-pmc >>> + - const: syscon >> Otherwise, this looks grand to me. >> >> Cheers, >> Conor. > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel