From patchwork Wed Jul 21 14:04:03 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Maxime Ripard X-Patchwork-Id: 12391201 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=-18.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT 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 4ABB1C12002 for ; Wed, 21 Jul 2021 14:39:51 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 1D3746120D for ; Wed, 21 Jul 2021 14:39:51 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1D3746120D Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=cerno.tech 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=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=OQToeWD4OwP6+iC1kb4NJGraJlMlz7FTyUTiZVPVe7I=; b=Ze0pQffjjGlpQO JlSGPdyjCsnsw/0kZB1CGsTRO33WyinTRYL7uChLBPx4p6TDeCdFc59BJWARX3WjuhdmgFG5Z5Nph D0AeWDWFewWFcLtMkrdi+m/8zdlehzJvS1LywXn/lR3PfDcBtn9lxWNLMRugS4+uABR4ij7+DJk5N 7MXkxw1nA7lR7tEfW5r1IibWTzVdj8U2jkIn7k2eMcVPn/1WXaksh9NdCoHIL3K9Ni2ujPEMf8CQM /EKu7feZIvKdADXbGryJWlzmJkOtEDjvdjQB19nURkNNlUDhCqT1Ux4LrxnFFa0oN1Gt8pLRb1PPR ioIx/3ZnPtQOXKx5Alzw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m6DLd-00G17O-Hs; Wed, 21 Jul 2021 14:37:22 +0000 Received: from new4-smtp.messagingengine.com ([66.111.4.230]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m6Cqu-00FqHX-Rx for linux-arm-kernel@lists.infradead.org; Wed, 21 Jul 2021 14:05:38 +0000 Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailnew.nyi.internal (Postfix) with ESMTP id 3AAE558046C; Wed, 21 Jul 2021 10:05:36 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute4.internal (MEProxy); Wed, 21 Jul 2021 10:05:36 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; s=fm3; bh=Uq8dukZV0FjfW wNFgyBbCaCvYElCG1N0o/DPMIuV7dc=; b=tiVNMJMBQn/YyRtI+o+265pG1Q2ge L1RlIprX0W+boGYM7O/JPajFwni1v3YeCYxAGNRPXbLJqvr/38IsgCEj1acreHJ1 eQPt1Zwqqzs2w5br+EW+IHDjId0hJLGWm+PwYWQbplTWZl3EkrungG1WX/v/Dz/+ u4YIBhehlalULWiwMbUmwxhfajkNPy+4fNRo4Zz9ZpVPRKqxfopAH2CoWAn5jRWX biCdVvqEI60y4wXVf3rdm2pHM2ief2Ro1QVoXXf//wFZdnS3Rv2DLdkEXeel2p59 wd/vyYOUbpTS0jkaDiCuFUktwvB5H+RYIUJDdrtOFlUEUJoIzygwUpq7A== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:date:from :in-reply-to:message-id:mime-version:references:subject:to :x-me-proxy:x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s= fm3; bh=Uq8dukZV0FjfWwNFgyBbCaCvYElCG1N0o/DPMIuV7dc=; b=k395fBlP QQcrTDQ762bdF6QdyyrkPN7yrn36MNfXRlQbRcs3P9JtgQAKJhx1y8iWb30xaJ+Q p0JhFP63+ZsFKHPnx8OW3vvwVkOcjg+Tp7ePFPXdMqLOtNLeoMye47vaI2pn6rUQ F0SEz4MeCVCnjf2oJjqXo6z5Iw95TagsLgxtlXHxNvsU3rrK7OpoHVPfF7kjr5lz vKWlkiBilZG7jtv8y/fhj3dhL48N6aVFoRJ4yLs+OnqxuCOyafJQZ5O/t6rvzBTR 5YbMlOipaAPv1rM8OdU3TeIBSi9zYsKqQ+aBwtsKiYA0ILuetBUHmHqVVNh4IOQF xfb863OMI3UsVw== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvtddrfeeggdeiiecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpefhvffufffkofgjfhgggfestdekredtredttdenucfhrhhomhepofgrgihimhgv ucftihhprghrugcuoehmrgigihhmvgestggvrhhnohdrthgvtghhqeenucggtffrrghtth gvrhhnpedvkeelveefffekjefhffeuleetleefudeifeehuddugffghffhffehveevheeh vdenucevlhhushhtvghrufhiiigvpedunecurfgrrhgrmhepmhgrihhlfhhrohhmpehmrg igihhmvgestggvrhhnohdrthgvtghh X-ME-Proxy: Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 21 Jul 2021 10:05:35 -0400 (EDT) From: Maxime Ripard To: Chen-Yu Tsai , Maxime Ripard , Jernej Skrabec , devicetree@vger.kernel.org, Rob Herring , Frank Rowand Cc: linux-arm-kernel@lists.infradead.org, linux-sunxi@googlegroups.com, Amit Kucheria , Daniel Lezcano , linux-pm@vger.kernel.org, Zhang Rui Subject: [PATCH 33/54] dt-bindings: thermal: Make trips node optional Date: Wed, 21 Jul 2021 16:04:03 +0200 Message-Id: <20210721140424.725744-34-maxime@cerno.tech> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20210721140424.725744-1-maxime@cerno.tech> References: <20210721140424.725744-1-maxime@cerno.tech> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210721_070537_052909_C663C596 X-CRM114-Status: GOOD ( 14.79 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Even though the previous binding made it a required child node, the implementation in Linux never made it mandatory and just ignored thermal zones without trip points. This was even effectively encouraged, since the thermal core wouldn't allow a thermal sensor to probe without a thermal zone. In the case where you had a thermal device that had multiple sensors but with enough knowledge to provide trip points for only a few of them, this meant that the only way to make that driver probe was to provide a thermal zone without the trips node required by the binding. This obviously led to a fair number of device trees doing exactly that, making the initial binding requirement ineffective. Let's make it clear by dropping that requirement. Cc: Amit Kucheria Cc: Daniel Lezcano Cc: linux-pm@vger.kernel.org Cc: Zhang Rui Signed-off-by: Maxime Ripard Reviewed-by: Rob Herring --- Documentation/devicetree/bindings/thermal/thermal-zones.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/thermal/thermal-zones.yaml b/Documentation/devicetree/bindings/thermal/thermal-zones.yaml index 164f71598c59..a07de5ed0ca6 100644 --- a/Documentation/devicetree/bindings/thermal/thermal-zones.yaml +++ b/Documentation/devicetree/bindings/thermal/thermal-zones.yaml @@ -215,7 +215,7 @@ patternProperties: - polling-delay - polling-delay-passive - thermal-sensors - - trips + additionalProperties: false additionalProperties: false