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 97B35C6FD1D for ; Thu, 23 Mar 2023 11:38:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=TTY0qNJLNNwteAi0MDN8tpqQgo+hG+ry+FOM4QmeYsk=; b=REZPR1NmrHG79Z x5DzVq6w1vJVu+UPi+YBRsTbmvX+ItQKUeQ8eJu2ws1Tq94mwijji/XNP95QklRsO1y15JQj3cj5N TRUHYIxaeFatkOorRlYleN9q2bZCQy7t3HOz64DOVpjlKDQi98jAGFbEd/WghQnjU8w+/Hm/J/Sdw AyZShKLM3i9TgMcB6Ydo/YUi+MGOIyIRcr5s/9hP/3P+It92x34Hxsj699igwnElphrscp9g7UhQh C8oYBYbLZBYnip1mgsvu82rlXCuXN1rghT3WLfxMIalrH4ePDzi42taY/o/ND75/oE/yS88SPQZ8G sq9fxZsRmmrUi+Feejaw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1pfJGY-001l41-0v; Thu, 23 Mar 2023 11:37:58 +0000 Received: from mail-lf1-x12a.google.com ([2a00:1450:4864:20::12a]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1pfJGU-001l2s-1h for linux-arm-kernel@lists.infradead.org; Thu, 23 Mar 2023 11:37:56 +0000 Received: by mail-lf1-x12a.google.com with SMTP id t11so27283107lfr.1 for ; Thu, 23 Mar 2023 04:37:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; t=1679571472; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=9wqfCTQu6TpSkQBuufTN+FDVnW3u9zukJQMSdPwg2XI=; b=Op0RbaHTvg7xYxUMJ5x37/ssOSmmCk314Ar6tZMsow7cQf4VpukR5WbwpoKvNPlu2w qpLueWiBUoznP+bidlh55yerR8HD4zeejtChefwtTSe1QazWFMbdyGnY7mUCCXIN+C5p UtnU1MO4sn6fjSneBxTpdUUWbf/OTjbDO2fQyhJFMTedWk4Q8Hz0fa2g4qKvUuPB0R0L pOA5cNoFm5zLcVhgdJ0m8/KbohJNPJzBwmiyVYWwPrA/h+CvdZpyS1elXEd1H4aTKx2K M5V9cImzOfpBDW+UvHUuaXJGB+wAOK6qAO5qwuxxINZy8nyW6mJW1jCqvnH/gYAcr4fB 81Lw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1679571472; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=9wqfCTQu6TpSkQBuufTN+FDVnW3u9zukJQMSdPwg2XI=; b=nD7QBtZJ4FUDPPVpI69H0yJbNql4GzW+umHCUmr7nKm/dhF3UnRc7naDo6Kpa0nPym SebampsoHu4Oyoc8CGvqvn9wxESOxQt41os6MLOuHzI2EPKz6NtSJRibHrtfb32qJYMn xJ1TUW7jUpoZaUwWWA0Y3pKhzhXAIobU2iuKUfIAd/0MQmcasz5MJQSKlA836w1N/XDn X1S6ECh4egl3GAMB5hHYJ1n3H55EQ+UN7ShEQrWNGQHEPuce5OrtFAQJJ4cgQyl+WEuy DBAVmpp0FpI9+1oLeMa9PDcGpTp+FVCUK/brNI7dU/BrAaOcA+drhra8MnlUpyPqlGek 8ZHg== X-Gm-Message-State: AO0yUKUX+haq6ughEqIY/oPWQdavu7yPRtarEDmEFKZfFR2cIU+5q67A TVTu75GpaFDxbl+yro/yUUA= X-Google-Smtp-Source: AK7set/GwotgsUGGJrT8NBvp53IUC4qj6qO7Ly+8+IGCS8e+OHcK+s+ODAxaZ68ZV4PenA/B0IUpqw== X-Received: by 2002:a05:6512:102d:b0:4e7:fa8a:886e with SMTP id r13-20020a056512102d00b004e7fa8a886emr2704920lfr.51.1679571472098; Thu, 23 Mar 2023 04:37:52 -0700 (PDT) Received: from mobilestation ([95.79.133.202]) by smtp.gmail.com with ESMTPSA id w9-20020ac254a9000000b004e845b49d81sm2898087lfk.140.2023.03.23.04.37.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 23 Mar 2023 04:37:51 -0700 (PDT) Date: Thu, 23 Mar 2023 14:37:48 +0300 From: Serge Semin To: Krzysztof Kozlowski Cc: Serge Semin , Giuseppe Cavallaro , Alexandre Torgue , Jose Abreu , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Maxime Coquelin , Rob Herring , Krzysztof Kozlowski , Alexey Malahov , Pavel Parkhomenko , Christian Marangi , Richard Cochran , Biao Huang , Yang Yingliang , devicetree@vger.kernel.org, netdev@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH net-next 09/16] dt-bindings: net: dwmac: Prohibit additional props in AXI-config Message-ID: <20230323113748.bj45qbvut2cthvyr@mobilestation> References: <20230313225103.30512-1-Sergey.Semin@baikalelectronics.ru> <20230313225103.30512-10-Sergey.Semin@baikalelectronics.ru> <78224241-00a3-2e8e-4763-603b27ac3b83@linaro.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <78224241-00a3-2e8e-4763-603b27ac3b83@linaro.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230323_043754_577345_B15BB221 X-CRM114-Status: GOOD ( 18.81 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, Mar 16, 2023 at 09:06:04AM +0100, Krzysztof Kozlowski wrote: > On 13/03/2023 23:50, Serge Semin wrote: > > Currently DT-schema of the AXI-bus config sub-node prohibits to have > > unknown properties by using the unevaluatedProperties property. It's > > overkill for the sub-node which doesn't use any combining schemas > > keywords (allOf, anyOf, etc). Instead more natural is to use > > additionalProperties to prohibit for that. > > > > Signed-off-by: Serge Semin > > --- > > Documentation/devicetree/bindings/net/snps,dwmac.yaml | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/net/snps,dwmac.yaml b/Documentation/devicetree/bindings/net/snps,dwmac.yaml > > index 89be67e55c3e..d1b2910b799b 100644 > > --- a/Documentation/devicetree/bindings/net/snps,dwmac.yaml > > +++ b/Documentation/devicetree/bindings/net/snps,dwmac.yaml > > @@ -466,7 +466,6 @@ properties: > > > > stmmac-axi-config: > > type: object > > - unevaluatedProperties: false > > description: > > AXI BUS Mode parameters. > > > > @@ -518,6 +517,8 @@ properties: > > description: > > rebuild INCRx Burst > > > > + additionalProperties: false > > But why moving it? Keep the same placement. No firm justification except that vast majority of DT bindings have that keyword placed at the tail of the schema body. Anyway I'll get the it back to the original line. -Serge(y) > > Best regards, > Krzysztof > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel