Received: by 2002:a05:7412:d1aa:b0:fc:a2b0:25d7 with SMTP id ba42csp1923222rdb; Wed, 31 Jan 2024 13:23:13 -0800 (PST) X-Google-Smtp-Source: AGHT+IHgkEQxZ8bj6FVkrgUzqV/SoL2dddhnIls4yZY9N590AsaLZdkWkPPf0OyOaxoFKgpgFqz4 X-Received: by 2002:a05:6a00:26c7:b0:6df:dadf:280e with SMTP id p7-20020a056a0026c700b006dfdadf280emr217517pfw.20.1706736193021; Wed, 31 Jan 2024 13:23:13 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1706736193; cv=pass; d=google.com; s=arc-20160816; b=fLxnmn+DOF07RXqDSAh/t0IMfPeuNI7PzZQ97FgCjqtupj8f22UKC1qXz9nwU+RLL9 Bx4MjJM1jIwgx1ek7WQeEAnLgbSNstaD63rdYuo1HSEEJGQ5TXkfwy1qlrbQgAGFXbNI Vzt7aNJIgUg7lKoe9/2au8RyR1tW01AHyoJinu+ofGzb8iJjas/7n3omOtdh9PY1JJxS W9DsmGMAhhi545Lgk+SB4PY3Ri1uG8qMTX/osC6tq8s/TAtos+VIjgslatdmMflixLHe fepmSI7eUzj8PgRJY3B8vyHyVLNMMxj5ONJBVqxC4qs1rorai0ip1NBu/VaDj9ZPQBHS awwA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:autocrypt:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:in-reply-to:message-id :date:subject:cc:to:from; bh=7cqrCygcCHzogycsG5LssvwUWfW9UNYvLlHJCVhnSqM=; fh=0yh+3UotA/CKY38h1XtW2lc7ryIeVybX9thja+3iodw=; b=chbrCU0d31TF7mJA03KzZeIcW7w42x1qqmSsdAiIQPPgwX/AeA1nGK/uC+FfJ8Bgvl qDo42dluyz28gsktzN2OU1JNf1Ks4bphPHAEV2eWBbrUdPs8OHAYRS8+Gx80nKanllp/ ZEi4oLkb8AoWJJ5U7eIPX5r5asU7ZLHE5iLLuK6HQUEa9NXm6+5PJr59U1ButCUX3pr6 bXSEfsZb3BRGCkPQflx481EphwmTgmtEgyCGjn60d9Y4RPTFZQhTE6B3yfJSkVlBE+cx wEEEPw0vsyDRGOslDI0fZThv8hPS98nqpvv2RNJnXSF7nofPh/q7qaflbnKr/LoWxNvB V8kQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; arc=pass (i=1 spf=pass spfdomain=skole.hr); spf=pass (google.com: domain of linux-kernel+bounces-47145-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-47145-linux.lists.archive=gmail.com@vger.kernel.org" X-Forwarded-Encrypted: i=1; AJvYcCWUOLlBHKrLpxULz4T1/HQ4h2vKKQU4MTGBRaHQOLgchztX7yhTCv3nstTOwn1SP7bAO/LpMk6RaI4DcHEvt2mBCyzp5GxPnOQzXxl7JA== Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id u3-20020aa78483000000b006dda2014f31si10316833pfn.172.2024.01.31.13.23.12 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 31 Jan 2024 13:23:12 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-47145-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; arc=pass (i=1 spf=pass spfdomain=skole.hr); spf=pass (google.com: domain of linux-kernel+bounces-47145-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-47145-linux.lists.archive=gmail.com@vger.kernel.org" Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 1D9D52808E7 for ; Wed, 31 Jan 2024 21:22:14 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id D939F39AEE; Wed, 31 Jan 2024 21:16:38 +0000 (UTC) Received: from mx.skole.hr (mx2.hosting.skole.hr [161.53.165.186]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EC94C39AD1; Wed, 31 Jan 2024 21:16:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=161.53.165.186 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706735798; cv=none; b=EGDJQDCC36cxRHGMWcxss7Yz1He67aPiraP0z7c2LZc9RptqrDLEITx4uJ1mi2/bkNjYUxvdvNKRjDFucOYSzM1Y9jJ71/TqoSU0vraD+XbcldE5LEobrYblIKQy5TDBWZGCQElZJxsK8D2kbobKXlw1WknigZNV1LpYWp54bRE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706735798; c=relaxed/simple; bh=JjwjKb8sW4Z+jJBf9W2XGorWIH0Ek1LmoLb1Pq5zm9Y=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=ONGLd5r3KUQagaifgP2RmnK2421RQHCNN+mTTH15V80OZ5qOj5o6VjZpztMlWthmj6h1UzTPHM7bMDNNXwt8rAf0V+mjKPedI6YeRG97/PO4JI5Da6G0fHVw1SpZxMp87S2TIkvM3d/CjoMGiYBbj1UastdXd4FerNf9b+Z/ttg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=skole.hr; spf=pass smtp.mailfrom=skole.hr; arc=none smtp.client-ip=161.53.165.186 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=skole.hr Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=skole.hr Received: from mx2.hosting.skole.hr (localhost.localdomain [127.0.0.1]) by mx.skole.hr (mx.skole.hr) with ESMTP id 804D685438; Wed, 31 Jan 2024 22:08:52 +0100 (CET) From: Duje =?utf-8?B?TWloYW5vdmnEhw==?= To: Rob Herring Cc: Conor Dooley , Vinod Koul , Krzysztof Kozlowski , dmaengine@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] dt-bindings: mmp-dma: convert to YAML Date: Wed, 31 Jan 2024 22:08:07 +0100 Message-ID: <4894227.31r3eYUQgx@radijator> In-Reply-To: <20240131171857.GA1531631-robh@kernel.org> References: <20240127-pxa-dma-yaml-v1-1-573bafe86454@skole.hr> <2924724.e9J7NaK4W3@radijator> <20240131171857.GA1531631-robh@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Autocrypt: addr=duje.mihanovic@skole.hr; keydata= mQINBGBhuA8BEACtpIbYNfUtQkpVqgHMPlcQR/vZhB7VUh5S32uSyerG28gUxFs2be//GOhSHv+ DilYp3N3pnTdu1NPGD/D1bzxpSuCz6lylansMzpP21Idn3ydqFydDTduQlvY6nqR2p5hndQg6II pmVvNZXLyP2B3EE1ypdLIm6dJJIZzLm6uJywAePCyncRDJY0J7mn7q8Nwzd6LG74D8+6+fKptFS QYI8Ira7rLtGZHsbfO9MLQI/dSL6xe8ZTnEMjQMAmFvsd2M2rAm8YIV57h/B8oP5V0U4/CkHVho m+a2p0nGRmyDeluQ3rQmX1/m6M5W0yBnEcz5yWgVV63zoZp9EJu3NcZWs22LD6SQjTV1X8Eo999 LtviIj2rIeCliozdsHwv3lN0BzTg9ST9klnDgY0eYeSY1lstwCXrApZCSBKnz98nX9CuuZeGx0b PHelxzHW/+VtWu1IH5679wcZ7J/kQYUxhhk+cIpadRiRaXgZffxd3Fkv4sJ8gP0mTU8g6UEresg lm9kZKYIeKpaKreM7f/WadUbtpkxby8Tl1qp24jS1XcFTdnjTo3YB2i2Rm9mAL2Bun9rNSwvDjE fjMt5D5I+CIpIshaQwAXwRTBJHHAfeEt62C1FQRQEMAksp4Kk1s2UpZkekZzNn48BnwWq75+kEj tuOtJIQGWTEHBgMG9dBO6OwARAQABtClEdWplIE1paGFub3ZpxIcgPGR1amUubWloYW5vdmljQH Nrb2xlLmhyPokCTgQTAQgAOAIbAwULCQgHAgYVCgkICwIEFgIDAQIeAQIXgBYhBFPfnU2cP+EQ+ zYteJoRnrBCLZbhBQJg01LLAAoJEJoRnrBCLZbhMwoQAJBNKdxLxUBUYjLR3dEePkIXmY27++cI DHGmoSSTu5BWqlw9rKyDK8dGxTOdc9Pd4968hskWhLSwmb8vTgNPRf1qOg2PROdeXG34pYc2DEC 0qfzs19jGE+fGE4QnvPCHBe5fkT2FPCBmNShxZc1YSkhHjpTIKHPAtX1/eIYveNK2AS/jpl23Uh hG9wsR2+tlySPNjAtYOnXxWDIUex8Vsj2a2PBXNVS3bRDeKmtSHuYo7JrQZdDc0IJiRm0BiLEOI ehTtcYqYr1Ztw7VNN2Mop/JG2nlxXNaQmyaV6kF/tuaqn1DJQcb0OxjAXEUMaICYJOwS9HSt26n uwo8dUiUPLQTih/wm6tyu2xrgMwqVT5jiKIssSS+7QNTsmldubRSYjFT49vwkVoUQ6Z3UO6BVdd f3OG4meE0S5uQc7Moebq67ILxfQ8XsDvdvEliVuHh89GAlQOttTpc6lNk8gCWQ+LFLvS66/6LFz mK1X4zC7K/V6B2xlP4ZIa3IC9QIGuQaRsVBbbiGB3CNgh0Sabsfs4cDJ7zzG1jE7Y4R9uYvdSFj Liq5SFlaswQ+LRl9sgzukEBTmNjdDVhufMY2jxtcMtck978E1W1zrg94iVl5E0HQZcpFHCZjRZX Fa42yPsvVkFwy4IEht9UJacMW9Hkq5BFHsdToWmg7RY8Mh04rszTiQJUBBMBCAA+AhsDBQsJCAc CBhUKCQgLAgQWAgMBAh4BAheAFiEEU9+dTZw/4RD7Ni14mhGesEItluEFAmCVBxAFCQXW6YEACg kQmhGesEItluFXIg//QnqY5RrQ1pLw2J51UwFec4hFMFJ6MixI9/YgizsRd2QLM7Cyi+ljkaHFQ mO4O5p0RsbF/2cc4u1D+MhQJGl6Ch6bdHoiWFrNUexgBUmflr4ekpI+GIFzikl6JTYHcRfkjobj 0Tmr8zWoxzcdFhrzGn5/6AH3GxudpUr6WQD5iDSe43T7ZcY8zHfD+9zcsZ2LHhRhpHU0q+ERQw+ Rnh7C3urXlrAlFzuKuPh2tHT76glRaledJ8cK34vHNi73TYpsFy4tfhAPhHwBogtjBf63jBOd/E S6wuYpKwcfNXo9EuEpJzJOitFwOvAra5AbCE+N/C/IOu2aFeOyu2SbHro06+Eyf/jy1A2t+LgLb E5cZu5ETyicfpN8L7m7wTTXTSx0NhETNWfgV95RUI6WIW5N4OCOVo8d/GOMVEYqMoDZndQin9B3 lDgojyagdzhXljP2BqavKdnPWbcKQ+JViR+e7EjLWVifgZkAvEhyirbTKYsgKkaRxoQP68U0bEy ukygDZRdzBmWaZPqBOzA5AH+OYiYVzzFqdBAHr2+z4mTN6W0td7CFDRAS2RzQApO3B1QH408Ke9 Oy69HwG+gdlfwloN6JTvgr5vQc8T6e3iC3Be/guLyW5UbLPxyFHimznVOizDYbZO1QSZMqk4G9I gA8e05P8dxEQJUsdZFtDdNPOYm5Ag0EYGG4DwEQAMD0bO0u9apmI1WOk41IdU1Hc76HLUA9jsiB ffA9yZ1OpnFEIAwSeUO8PFK7W5YPdRreNsUvMmBiLJid9y0tW5sACjSrH+amCQl0hJ3KlEkr+Vu Wga1a+Ye0qzg87bQae769RhwzEPvQvvNoTxTtvT5Alg2p3JSv5d/wC2Tu9IoFKkDAIoCFsvytuZ r2LuH3oK57oThhbEogYXR7YJ0JIwVg7nOQXnqpUTzxkh/73FKN6Bx01m37pB3wTe8w3w8r8WOip oRU+aPWhafDNFrdyBfSVOAw3fmX9yAfFfZo4w9OTdkrLLdK6SmX7mqiMstoZnvZIpLRk/L0ZNrJ 8fAVD+fEcpUiCoKwiiY0QFCWumMXITeD4zlo/Y6lQKhUp6EY0kcjG1D7n5sBR5oQcsC9PlH9a12 L+tNIfljayiEVobmkPwGf5p3sxOqeks6WWoB9+ZIk888kQdI/b7VA/86QvsTqubpJtr5uVNtyyj ZYTBHFnEGcA5+Rs2K/8TWFYDEBZiybfpCxrYT2RdTF7ef2wQZAiNZhzaEwxr7S4YTFuCwwqaKLt vckGv2fsFUy3qe28tw93oCNQxSqgOq6RD0HfblViXeioyP1nWVLAx6paS7d38TT6cz0HJCtOMFn S+UpJDv2x3gReCPBoqRx7LV4aYMyGy4pzwes+yO87hxULtw/ABEBAAGJAjYEGAEIACAWIQRT351 NnD/hEPs2LXiaEZ6wQi2W4QUCYGG4DwIbDAAKCRCaEZ6wQi2W4de4D/0aCxE4dTmO1xQ6BDXlKp DCegk8dIqrxK8Edbdq9/WGSO6Js0QfIL50IHAR739FbScT4+oSObeg0ap9kCGfW0AXGZaU82Ed1 5u+MzgksHE+t8cgULTKjqqt+PXq0yxZfLwI9itTa3zE2d6Uxd4Vzq77jjQuDL6o3zM6BQTJGYxx S6mELElcnMlo9lIZKzCAHaIkkMlMNBfvm8Q92aCuQ75xjWhis9K9lyV9cQZfu8AyP4zMGFk50Z5 tEF2UFylqKu+v8FZiezviwu9NsZegIY4DRaPWF5GWmFhYU4e9gBFG5xhEoIlO+etu1nSE1UJk+r mvJL20uKNUPnhXTJaQTzACpA1/2FqDnOUUx8qOYqmHMlFuy2qUh/QHShjc2AtngTFZrzAnGz6ni lRl32b7p8N+KaO4u2UGmGOwd/CuCzr2DxGomUSyCwOta7vOxator+NPK48roa417gBZ6ZFRplma ExicLFSnwBdGC3NnDa+yoRHKXHVSDfkb/FEhWuN/1tTZ96uxVYtHcln+snB2N6/hwmrOon2cHNu UeTLcrVyqI0Qz8JT4ksGxkxziO2L/e0O/xUp9mLAswixWt8+BMz/3sIJbdAPBVyt5QbHzWR6aID B5cQ1aQwZB8n7yt8B0sd/uIQItYu2urJ9gVAJkaEDms8+vbtOM4totXk5swwGxRg== Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="UTF-8" On Wednesday, January 31, 2024 6:18:57 PM CET Rob Herring wrote: > On Sun, Jan 28, 2024 at 07:01:36PM +0100, Duje Mihanovi=C4=87 wrote: > > On Sunday, January 28, 2024 6:28:03 PM CET Conor Dooley wrote: > > > That said, for properties that are only usable on some platforms, ple= ase > > > define them at the top level and conditionally permit/constrain them. > >=20 > > Could you please point me to how to do so if this if/then does not do it > > properly? >=20 > Negate the if and then: >=20 > then: > properties: > asram: false >=20 > There are lots of examples in the tree. This works as expected, thanks. > > > > +unevaluatedProperties: false > > > > + > > > > +examples: > > > > + # Peripheral controller > > > > + - | > > > > + pdma0: dma-controller@d4000000 { > > >=20 > > > The label is not needed here or below. > > > In fact, I'd probably delete the second example as it shows nothing t= hat > > > the first one does not. > >=20 > > I'd rather add the asram property in the second node (adding onto the=20 above > > comment, I now see that it shouldn't have even passed dt_binding_check > > because of the missing asram, but it did). >=20 > It passed because 'required' is what checks for property presence and > nowhere is asram required. It is missing a type definition which should > have warned, but may not since it is under an 'if'. I thought specifying minItems would have the same effect, but that turned o= ut=20 not to be the case. Also, the 'if' did indeed suppress the missing type=20 warning. Regards, =2D-=20 Duje