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 327E6CA0FE1 for ; Sat, 23 Aug 2025 06:30:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:CC:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=J0xEHhpDr5Ii0lNf94Y5NJ7HClkQVYYRlfEovqbGqyo=; b=aKaJ5hW4KvPeYdfn7Brl05tvNM edAfRG5gpMPp5USnlfs9jLelcl0RwFwMGc6SssL6Pmig4RRDJgYXtvbEpmd6lE4plXZ6+ltTl+itw 7ZRps6xiGhka1ZOAmU+EJ9HRHBPhCAW9UZnsmstJfp2ix1oERJNIMj/MZRWTUMEsiDmRPef6lVvpQ tTaBlh/M6pP2RlkeTynfWULRJuoxRYDQiMOT1FG8u6DVKvoK6uD/gr/GouMiot9dcLy7cNt7kUy31 FFKNmGp+Rn0cxfPBWk5UsDp3RT8n6CEFZu58FTk5dmqH/ULk4hgxM1L8WrEFgYnp3S8RIp5rp19uy BFjWX4zg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1uphlY-00000003yPk-2Aw8; Sat, 23 Aug 2025 06:30:16 +0000 Received: from mx08-00178001.pphosted.com ([91.207.212.93]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1upRHQ-00000002YKe-1xrb for linux-arm-kernel@lists.infradead.org; Fri, 22 Aug 2025 12:54:05 +0000 Received: from pps.filterd (m0369457.ppops.net [127.0.0.1]) by mx07-00178001.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 57MCc12j019648; Fri, 22 Aug 2025 14:53:50 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=foss.st.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=selector1; bh= J0xEHhpDr5Ii0lNf94Y5NJ7HClkQVYYRlfEovqbGqyo=; b=yrD/44iTpG6d7ZWH wgrtFXahyjJZvdUKlQCA4ddPj0sRoExjds+LljZRRBiQ0y5Vs/RqnX2RPKD2rdDX +GeUDKKbq7pQs/fLkoAEqA5Yeqe8/6eqSnCqTBNKQ0mtxwfBV0O+TdGXuQQy6i5T b7YzugGDY6bL8EsJ47ZHyg71QYyP5uHOWfV/rJUC6VK7Wkoe06e1YqwP9DtWCbba TrjmsxWD8z3bziyn0nc+6/2MiDX8U8Fz96kcXVgve5t7mje+bzwOoQRSQvKeQFWl dqXpLwcKhVex7DPC1lghryENCpzYxpQ52mfnxGufdt2SODr8pacM8ut7eUqqT787 RhXNaA== Received: from beta.dmz-ap.st.com (beta.dmz-ap.st.com [138.198.100.35]) by mx07-00178001.pphosted.com (PPS) with ESMTPS id 48n81wtrcs-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 22 Aug 2025 14:53:50 +0200 (MEST) Received: from euls16034.sgp.st.com (euls16034.sgp.st.com [10.75.44.20]) by beta.dmz-ap.st.com (STMicroelectronics) with ESMTP id 5314C40044; Fri, 22 Aug 2025 14:52:36 +0200 (CEST) Received: from Webmail-eu.st.com (shfdag1node3.st.com [10.75.129.71]) by euls16034.sgp.st.com (STMicroelectronics) with ESMTP id B0FF66C67BC; Fri, 22 Aug 2025 14:51:24 +0200 (CEST) Received: from [10.48.87.178] (10.48.87.178) by SHFDAG1NODE3.st.com (10.75.129.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Fri, 22 Aug 2025 14:51:23 +0200 Message-ID: <09c20af0-e33f-4325-9cf5-249cb2f31b38@foss.st.com> Date: Fri, 22 Aug 2025 14:51:23 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v4 06/13] dt-bindings: arm: stm32: add required #clock-cells property To: Raphael Gallais-Pou , Philippe Cornu , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Rob Herring , "Krzysztof Kozlowski" , Conor Dooley , "Maxime Coquelin" , Alexandre Torgue , Catalin Marinas , Will Deacon , Christophe Roullier CC: , , , , References: <20250821-drm-misc-next-v4-0-7060500f8fd3@foss.st.com> <20250821-drm-misc-next-v4-6-7060500f8fd3@foss.st.com> Content-Language: en-US From: Yannick FERTRE In-Reply-To: <20250821-drm-misc-next-v4-6-7060500f8fd3@foss.st.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.48.87.178] X-ClientProxiedBy: EQNCAS1NODE4.st.com (10.75.129.82) To SHFDAG1NODE3.st.com (10.75.129.71) X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1099,Hydra:6.1.9,FMLib:17.12.80.40 definitions=2025-08-22_04,2025-08-20_03,2025-03-28_01 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250822_055404_770406_DED6E535 X-CRM114-Status: GOOD ( 13.19 ) 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 Hi Raphael, Thanks for the patch. Reviewed-by: Yannick Fertre Le 21/08/2025 à 13:08, Raphael Gallais-Pou a écrit : > On STM32MP25 SoC, the syscfg peripheral provides a clock to the display > subsystem through a multiplexer. Since it only provides a single clock, > the cell value is 0. > > Doing so allows the clock consumers to reach the peripheral and gate the > clock accordingly. > > Reviewed-by: Rob Herring (Arm) > Signed-off-by: Raphael Gallais-Pou > --- > .../bindings/arm/stm32/st,stm32-syscon.yaml | 31 +++++++++++++++------- > 1 file changed, 21 insertions(+), 10 deletions(-) > > diff --git a/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml b/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml > index ed97652c84922813e94b1818c07fe8714891c089..95d2319afe235fa86974d80f89c9deeae2275232 100644 > --- a/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml > +++ b/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml > @@ -36,20 +36,31 @@ properties: > clocks: > maxItems: 1 > > + "#clock-cells": > + const: 0 > + > required: > - compatible > - reg > > -if: > - properties: > - compatible: > - contains: > - enum: > - - st,stm32mp157-syscfg > - - st,stm32f4-gcan > -then: > - required: > - - clocks > +allOf: > + - if: > + properties: > + compatible: > + contains: > + enum: > + - st,stm32mp157-syscfg > + - st,stm32f4-gcan > + then: > + required: > + - clocks > + - if: > + properties: > + compatible: > + const: st,stm32mp25-syscfg > + then: > + required: > + - "#clock-cells" > > additionalProperties: false > >