From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=100.103.45.18 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1792773917; b=Yb0TFhdc86aIIeOeB0Jmt/4GSggo5/P6O6TYjunHcbcG8hW/Oly1Dxo+gNrejfBfe0rv Kj22jjnY6gyN9OMrakOM5EigAWlWKosIfIm6Cw3kajLl0vJTOjVpa/G1PhuzLGfhrz1+J 9JFtb+sLHuKeqVRzTHvHUr7RuZlgmGNk016bFXUIgw+5a2bOOgSgLOY6aMJYg6MpGy+9B u3Q3SfE5zBUh79gtAOppWhS0qbMJA2fpVs81vGPIdbOnw7qigLSrpqBRabaFv0dwE77uN +ZdKwsWt1vkmgFdpPa77zYUa2IVNamofsGGnpP+G6L/ZZGh1bUxr13DAKD+d/zAitBQ== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1792773917; h=Received:DKIM-Signature:From:To:Cc:Subject:Date:Message-ID:X-Mailer: In-Reply-To:References:User-Agent:X-stable:X-Patchwork-Hint: MIME-Version:Content-Transfer-Encoding; bh=FaQi0oP6v8S8xrWXJbTwkUSqvD8UvRwZ30tX2eLCLDM=; b=CDKVVlbHLJWRE/srn0VSGO+6njYmLAt7+fWFHTR5N1obDBgg2EJfB/ZQGLuc0L42gQhZ 6/Y7Zn2H4PzcwnXbYtxKDrZElkIyEp3HpztIF79cD2dXmqANO1U9ccfWFyDcODq0W/PUJ 5mxOBGmm84L7u8h+0ZIbMPwL4uOxuh9gRoniWn40HM3iQwkWnOrpzXRZrtkFP+1egWRkb +/GJI+LJPrZ54rGG9JTyk58QiF2T9XvxCXmus2tXBE1pFzAdVEain6PGdO6MYI10Ss6J+ JmXvUdItc1fk//DmnrSUpVOhcNVQkZVaKC+dafnFpIe7znMcJCUnxx1/elbLkBRcMTA== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass header.d=linuxfoundation.org header.i=@linuxfoundation.org header.a=rsa-sha256 header.s=korg header.b=AXUQIEtA; arc=none smtp.remote-ip=100.103.45.18 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A7AF1606A31A; Fri, 23 Oct 2026 16:45:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key, unprotected) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.a=rsa-sha256 header.s=korg header.b=AXUQIEtA Received: by smtp.kernel.org (Postfix) with ESMTPSA id A2FCC1F01825; Fri, 23 Oct 2026 16:45:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=korg; t=1792773916; bh=FaQi0oP6v8S8xrWXJbTwkUSqvD8UvRwZ30tX2eLCLDM=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=AXUQIEtAj5nlsRRCzI/O+bX2DSC8Oo1hPlzRl5kr9opmEAYeWUn014FlDQz8NZzfA /y7+7eT8fVyUad3w005SWBHzmhmXwUqzpRdvcE4tqFJ4Gg4FleBRY+j9VoHBQgFPFF /X2dAGNIQC1KEybRkkv7biHGfV89/r88RgKEtaI4= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, David Oberhollenzer , "Kevin Hilman (TI)" , Sasha Levin Subject: [PATCH 6.18 010/193] ARM: dts: ti/omap: dra7: fix PCIe PHY clock divider definition Date: Fri, 23 Oct 2026 18:34:47 +0200 Message-ID: <20261023163714.448195855@linuxfoundation.org> X-Mailer: git-send-email 2.56.0 In-Reply-To: <20261023163714.201276352@linuxfoundation.org> References: <20261023163714.201276352@linuxfoundation.org> User-Agent: quilt/0.69 X-stable: review X-Patchwork-Hint: ignore X-Mailing-List: patches@lists.linux.dev List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 6.18-stable review patch. If anyone has any objections, please let me know. ------------------ From: David Oberhollenzer [ Upstream commit 2fabd2f406d0cf787be47b3bbeea99b30004033d ] Commit d0bdd8bb7f35 ("ARM: dts: ti/omap: dra7: fix redundant clock divider definition") removes the clock divider definition from optfclk_pciephy_div, reasoning that the table is redundant, as it contains the values 1 and 2, while also setting ti,max-div to 2 while ti,min-div is implicitly initialized to 1. What that commit missed was the fact that the values in the table are in reverse order, i.e. first 2, then 1. As is clearly described in the documentation, the ordering is signifficant, the actual values for the register setting are derived from the index. If no table is specified, the driver assumes that the register value is divider - 1 (see also `_get_val` in `drivers/clk/ti/divider.c`). What is in fact redundant is the value of ti,max-div that is derived from the table if none is specified. To satisfy device tree semantic linters, it is removed instead. Fixes: d0bdd8bb7f35 ("ARM: dts: ti/omap: dra7: fix redundant clock divider definition") Signed-off-by: David Oberhollenzer Link: https://patch.msgid.link/20260716113947.92513-1-david.oberhollenzer@sigma-star.at Signed-off-by: Kevin Hilman (TI) Signed-off-by: Sasha Levin --- arch/arm/boot/dts/ti/omap/dra7xx-clocks.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/ti/omap/dra7xx-clocks.dtsi b/arch/arm/boot/dts/ti/omap/dra7xx-clocks.dtsi index 0de16ee262cf4..615a8fb477761 100644 --- a/arch/arm/boot/dts/ti/omap/dra7xx-clocks.dtsi +++ b/arch/arm/boot/dts/ti/omap/dra7xx-clocks.dtsi @@ -1376,8 +1376,8 @@ optfclk_pciephy_div: clock-optfclk-pciephy-div-8@4a00821c { clocks = <&apll_pcie_ck>; #clock-cells = <0>; reg = <0x021c>; + ti,dividers = <2>, <1>; ti,bit-shift = <8>; - ti,max-div = <2>; }; apll_pcie_clkvcoldo: clock-apll-pcie-clkvcoldo { -- 2.53.0