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 576F3C61DC2 for ; Thu, 27 Aug 2026 13:07:18 +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=FmJk1bI2daWaJTjZ/KQDHONp7CitHX5WVwmk9KLzZmE=; b=NZm/QKpo1Xm70s 25kXpRHvLPbg4iG7oQxH8OEK5SizSiG/8hvXip/WhzeS+EaKOFZtDlhXrGlq9cQky9p3QdV22gSgm x2kv8h1QtrLmXcD+ygMz5tied/IGpDKTdGLVtRHjMEg9dTObAHDwss133GLUgd/3DrB1zylJI7McY sTtXFbV6zg/BptZjcKFcKDQ335dN4Xom4CSerAQxoB7pIhQLiyQYxBqxtw/e9aQidSQLqoHOvSWJO fnxmgLM+LfRC3MPq+WhA72Oe2pp4Ry4OAtkUjXwVDaGSYASpTpCeX1er/HG3KaUapZG5knIaRfUVa 2v3AGAPif86e64vIen7g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wzZp7-000000041dB-3asO; Thu, 27 Aug 2026 13:07:17 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wzZp6-000000041ck-2Z7S; Thu, 27 Aug 2026 13:07:16 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 2CB8B40914; Thu, 27 Aug 2026 13:07:16 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 119E81F000E9; Thu, 27 Aug 2026 13:07:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1787836036; bh=E7idqHOncmG7mF6Fij9tBltdrtulZ+mN4mIw16QKjBY=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=HbkNkqofgca0N5Jj0aw+z4giRuz2xH3T4sWKZErKe9iV4Rla0Ul7qRVW3meU6hTXL RzzC9URElHLfzSvuCUSJYk3m1rOoSCwWZF9jw7cIi3PZc2OncHlWQ+h4B8Y5z6Ddiw n77PfXAVcTjuX5X9CAcdV7kvHjURMrI4WdBes/IYYEwo4uXXFeoFb4SfCnK71sp965 30BGGHUNDgnXCTK7aoRz3uiKhnspaGD/wTywqrV4qixFUcFlswhKBVUbRn4tVS+DEd KJQ7cLa5lGntEhy47ujyEYXAFoaD8QVFk6JXT5hbfLWbh+7/oCVqr6vR334kaCWYGd VfEUHF4prbYhA== Date: Thu, 27 Aug 2026 15:07:11 +0200 From: Krzysztof Kozlowski To: Marek Vasut Cc: linux-usb@vger.kernel.org, fabrice.gasnier@foss.st.com, Alexandre Torgue , Christian Bruel , Conor Dooley , Greg Kroah-Hartman , Krzysztof Kozlowski , Maxime Coquelin , Neil Armstrong , Pankaj Dev , Rahul Kumar , Rob Herring , Rosen Penev , Thinh Nguyen , Vinod Koul , devicetree@vger.kernel.org, kernel@dh-electronics.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com Subject: Re: [PATCH v3 7/9] dt-bindings: arm: stm32: Switch st,stm32mp23/25-syscfg into simple-mfd Message-ID: <20260827-skilled-optimal-shellfish-c548e5@quoll> References: <20260822074816.548662-1-marex@nabladev.com> <20260822074816.548662-8-marex@nabladev.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20260822074816.548662-8-marex@nabladev.com> X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org On Sat, Aug 22, 2026 at 09:46:40AM +0200, Marek Vasut wrote: > The st,stm32mp23-syscfg and st,stm32mp25-syscfg can have subnodes, which > represent the USB 2.0 FEMTO-PHY. Turn the syscfg into "simple-mfd" so the > PHYs would get populated by the OS. Sashiko comment is partially valid - you change the ABI in incompatible way, by requiring a new compatible. Commit msg should briefly explain the impact, e.g. that there is no impact because phy was not implemented by Linux and was not described in the binding. OTOH, adding simple-mfd means children have no dependency on the parent, so this should be considered or even explicitly mentioned in commit msg (parent clock is irrelevant to child, right?) OTOH2, you have now patchset dependency which is nowwhere explained. Merging constraints are the most important part of cover letter so must be clearly stated. IMO, should be stated also here, because many maintainers do not read cover letters. Or just decouple dependencies with using 'compatibles' pattern (see Qualcomm msm/mdss display bindings). Best regards, Krzysztof -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy