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 2B7D7C19F32 for ; Fri, 7 Mar 2025 14:08:13 +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:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=lskUXuU0lR7L0bPpAN9tCgMdyYTowH0nWizaVa0jYvQ=; b=AiSx22/bVzXpNd xoFFjumQ9vxmM/kAuI/6uVWpU0+quOTHQQlW8NIijHEt8WCj50m9xABb5vGOHVEhZ49Vj7wesq3JY nutalyd1U+hdDZL9teL+nC3WUX73n/Pd7N5iFcpgSKkz9b3HnBvcUWjStgnJNt8wR1ijmOoGJH2cm azlzjntOuqEsqdqFLGilr8ySogdQrwfo/j5i0CszXRKipQUzyjIB0Roorie9+FdTrBf0UgGj6YvyL whra0qNjJXwrzUynouTt4S/SpDY4GU0Cf4Kxr+qNBL8z+HK+JwWeLEWoDSsrBVMwGV9Pwr77XmoLg eunBv01AMXd4JPeGybbQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tqYMx-0000000EPwn-2QW5; Fri, 07 Mar 2025 14:08:07 +0000 Received: from sender4-pp-f112.zoho.com ([136.143.188.112]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tqYLK-0000000EPdb-0Lmy; Fri, 07 Mar 2025 14:06:27 +0000 ARC-Seal: i=1; a=rsa-sha256; t=1741356358; cv=none; d=zohomail.com; s=zohoarc; b=RUSqeTjX0/f+vSCIJFG7Ivi/H9ExZZOTIGsH8b/7NgnDNY+UPVZ7ECjWzwOuDI3RzpTfQqmbZSAsfna/nJrzgxqNHi9CItEF9nS40fA67K5OyjyabwmfpK9um35EBQjHtXoOmHIdnjUaDBCCDMC8yAzHg++9XIkIh6wtiClIxik= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1741356358; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=FP+ifasHniIC8neSq6VwSq5ZsIenCwU36aCvQ3EbV9k=; b=M4EErEyApvaCAnFL5c92n1M0wMYZ4/LNdYQL/JjFfVIWr1+oWZ7LWHG91M7LH/wdakw0O6B/Z6DdW1CFhUYSU8YrcdcDdi/S/B33oM+ZFqVbEURJgvl8Ve4JwYi+DVhf5OMbdwCpJiS1Sj2d8rpDnz9ZN6y3eCQFi/Cx2aen930= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=nicolas.frattaroli@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1741356358; s=zohomail; d=collabora.com; i=nicolas.frattaroli@collabora.com; h=From:From:To:To:Cc:Cc:Subject:Subject:Date:Date:Message-ID:In-Reply-To:References:MIME-Version:Content-Transfer-Encoding:Content-Type:Message-Id:Reply-To; bh=FP+ifasHniIC8neSq6VwSq5ZsIenCwU36aCvQ3EbV9k=; b=k4LLpH1xLhH5D3H7h2upAramGsfkMWDuPKZ9FZ0XGVS9QKMQsdQP0bKN4azcuUGk pMfBCbFEZtyQaxyMl/XEFyBtH0id4McTIUQZz/KeSYcNbJQotbnjO7wP5JrTi+oGXgB 95fO8bj46v0ezvai2ZHpniitwdl6VAtoA17B70u4= Received: by mx.zohomail.com with SMTPS id 1741356356876241.9419093370691; Fri, 7 Mar 2025 06:05:56 -0800 (PST) From: Nicolas Frattaroli To: Daniel Golle , Aurelien Jarno , Olivia Mackall , Herbert Xu , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , linux-crypto@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, Krzysztof Kozlowski Cc: Krzysztof Kozlowski Subject: Re: [PATCH v2] dt-bindings: rng: rockchip,rk3588-rng: Drop unnecessary status from example Date: Fri, 07 Mar 2025 15:05:50 +0100 Message-ID: <5417098.31r3eYUQgx@workhorse> In-Reply-To: <20250307093309.44950-1-krzysztof.kozlowski@linaro.org> References: <20250307093309.44950-1-krzysztof.kozlowski@linaro.org> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250307_060626_184686_2D9C8ACC X-CRM114-Status: GOOD ( 17.48 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org On Friday, 7 March 2025 10:33:09 Central European Standard Time Krzysztof Kozlowski wrote: > Device nodes are enabled by default, so no need for 'status = "okay"' in > the DTS example. > > Reviewed-by: Heiko Stuebner > Signed-off-by: Krzysztof Kozlowski > > --- > > Changes in v2: > 1. Drop unnecessary full stop in subject prefix after ':'. > 2. Add Rb tag. > --- > Documentation/devicetree/bindings/rng/rockchip,rk3588-rng.yaml | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/rng/rockchip,rk3588-rng.yaml > b/Documentation/devicetree/bindings/rng/rockchip,rk3588-rng.yaml index > 757967212f55..ca71b400bcae 100644 > --- a/Documentation/devicetree/bindings/rng/rockchip,rk3588-rng.yaml > +++ b/Documentation/devicetree/bindings/rng/rockchip,rk3588-rng.yaml > @@ -53,7 +53,6 @@ examples: > interrupts = ; > clocks = <&scmi_clk SCMI_HCLK_SECURE_NS>; > resets = <&scmi_reset SCMI_SRST_H_TRNG_NS>; > - status = "okay"; > }; > }; Hi, is there the possibility we could make dtschema as invoked by `make dt_binding_check W=1` (or W=2) add a warning for examples that have disabled or explicitly listed status properties when not needed? Or is this something better handled in, say, checkpatch.pl? The question arises because dumb mistakes by me like this should ideally be caught before they waste precious maintainer time. If it's best handled in dtschema, I can look into working on that so you guys don't have to do even more work due to me. Cheers, Nicolas Frattaroli _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip