From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm2-f12.google.com (mail-wm2-f12.google.com [74.125.225.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EE8EC4B0CB2 for ; Fri, 25 Sep 2026 14:43:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.225.140 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790347439; cv=none; b=g3KmDciIa5bxhKPZqT1GDfeCkxE2f0gU7UGA2VaFJ0qyY9WhpWcMAFYiOzm2yobuIixrecEQNpq3xBzmsci2DHiaRO7bK1+bhIsjcXYLiY4sujCmTQYmgKTiDe+cBG6aU4BvxnaqZ11yYrYj+zWVkvN14yQGeltPIKOy+/LXb4g= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790347439; c=relaxed/simple; bh=s2tP4LuXG6DitbJlb4NU+9ieo9q10zgjsECRa7Nn+zo=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=EfyRMtYGFfS+zaQ4B+DQotsULqFnKKlnyKboITJGh+bl9zE0jB9vd2VDYO3ye8mrlCl+70VZOnu30vco+PYexL+L5kIdc7h3uS1YROLKGfVlfodMeQLnvg7SKD9S81Bl16LzbwtMXGgXUc+i11VCPMQbTL7+xy/CMGePfNMzvnk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre.com header.i=@baylibre.com header.b=IApszyH0; arc=none smtp.client-ip=74.125.225.140 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre.com header.i=@baylibre.com header.b="IApszyH0" Received: by mail-wm2-f12.google.com with SMTP id 5b1f17b1804b1-49b91369d18so6876155e9.0 for ; Fri, 25 Sep 2026 07:43:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre.com; s=google; t=1790347430; x=1790952230; darn=vger.kernel.org; h=content-transfer-encoding:content-type:mime-version:message-id:date :references:in-reply-to:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=DtEfZGf6pOZoZKp7BZybV8NV7US5hG1rb4KkghHs3uw=; b=IApszyH0rgyC4GkO6MAkId0rbPwMzvK/8y6/2cqShaoWCeQ8QT7rYGJq60/UFcOojd veasg+HsVcSBPzuke75fWJgfwHcws4zyhTx+TvNBRyZaPvgoy6nivXgU/YkQVCz64AUr w9qNn6HuCUZp4yfrxzV+9whjxBKi55GZjtlRQ469lTsecEwpyMMRHVTcpWHCZUrL5SiK 8CUB7Xn5pz8ZX3rdzJLqn8BZWg2Q9FLHC9ice5UoHnFN+Ns3Nbt02ZenjS5ljdm9AEkm I2BLVzXJ0YFg5sURuvZ4AUlcWhAAWLS6RnsGD/qHTtg6DHdxDFxmst+msC3pl9jzpyN8 sKMQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1790347430; x=1790952230; h=content-transfer-encoding:content-type:mime-version:message-id:date :references:in-reply-to:subject:cc:to:from:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=DtEfZGf6pOZoZKp7BZybV8NV7US5hG1rb4KkghHs3uw=; b=POt/mXR/HM+vXMLSKDZkg0Nc4ohmqcN7XORSMj3I8uY/GuUQxe3X7Jml6Cy0ESNAox g+cK5Sc+i1M0Oy8DRJx/7f9cCzcVdA/T5K9UgtVT38vliKQB2+/Vtk2IxqkgFd2DXWUS 9/L+Sq1tyz93wErzArkWuZmGt4VafQL4QRVVCwR7XPVpi03DKV+GKKugNMf/aDvhCZAN 7tzwNxa93wzlNJGtFNlwiOip255RSrJEqUxikbwZAhs3hIVvTVv9HRZrW++sQ6VD+K0K U1W420gtHv1hdGIWQuziCr3f+i+0VvrDEhhz/wgP+apwodGTff/pSF7KeLsbQFG5eCI/ Q3ig== X-Forwarded-Encrypted: i=1; AKwUvByWmH47xWZizEyOMwAiSWrQYiN2ZMLaoiN4xidybKCkBV8rkeGiY2P/IhIRdfOLku2OTGyGESj0pqA1@vger.kernel.org X-Gm-Message-State: AFuF++l8/FFcRwKTcRqlq6AeXF80TuqAQaG1BM3KYMpNFQOhpJeGGqxA cs7lCr7N/v3s+nZvLvMZTs5AH4cW+2jRYMmbdOD3zvkJD1TbjBHF7Aj0HveWXDGZa4s= X-Gm-Gg: AYBFou0u3lVZI7WiJy89r72MhSdF1m3HhggODtGJ1QMvZU/M1tfW5lKntX3liACISiS fRgLF6QZpdcDrV3ooYmZPPiuHCLnI5w4UmamB6nT4Z3b7dsQG8f41lzOBnZVL7+aG0v1LRQn6ix tAqP9r/hROAlrKQhT7QlaRkKfRWtna8vteUd6KzGLQc1cDqKCR65qejHKYpJOKlpqy8vsMZN8uo OCUlaXK3W/MsmBDE4YkCrrW/ApQZbQr3zshHBteqWIamnuc9PeiHK6zmwvW3d4ef9nM0733MA2P P50YzcwcIhKgmGADEqazqNdxpgZejLtNGmL2vVqiGZQVI6eEFzZgO0WvJCyPlacBEczfcVFbzrV 0DTEmLjV4zJhfjFO6yxdo+2CkVuAbBkNQjjwtyl1QP7sFa/wl3SI1tP2DlfgkIEY/FpvzZl1gtj Dk9W0Ye2HYrqAH477Gq5R5mouIXUYq6a+XXTSNSXMDpsCxiKbQ4N5oErpplujRgRqliG8i2unlk UmObBjDVQqN5eAVEA== X-Received: by 2002:a05:600c:c8f:b0:49c:dc14:d681 with SMTP id 5b1f17b1804b1-49fe66bdf3bmr105928135e9.3.1790347430286; Fri, 25 Sep 2026 07:43:50 -0700 (PDT) Received: from localhost (82-67-6-57.subs.proxad.net. [82.67.6.57]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-4887a35607bsm7104435f8f.16.2026.09.25.07.43.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 25 Sep 2026 07:43:47 -0700 (PDT) From: Jerome Brunet To: Cody Kang , David Airlie , Simona Vetter , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Yixun Lan , Vinod Koul , Neil Armstrong , Haylen Chu , Michael Turquette , Stephen Boyd , Brian Masney , Philipp Zabel , Paul Walmsley , Palmer Dabbelt , Albert Ou , Alexandre Ghiti , Yao Zi , Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= , Guodong Xu Cc: dri-devel@lists.freedesktop.org, linux-riscv@lists.infradead.org, devicetree@vger.kernel.org, spacemit@lists.linux.dev, linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org, linux-clk@vger.kernel.org, Cody Kang Subject: Re: [PATCH v2 06/17] clk: spacemit: k3: parent eDP/DP pixel clock to the PHY PLL In-Reply-To: <20260809-k3-display-v2-6-327d7910bf71@gmail.com> References: <20260809-k3-display-v2-0-327d7910bf71@gmail.com> <20260809-k3-display-v2-6-327d7910bf71@gmail.com> Date: Fri, 25 Sep 2026 16:43:46 +0200 Message-ID: <1jwls975nh.fsf@starbuckisacylon.baylibre.com> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On dim. 09 ao=C3=BBt 2026 at 21:14, Cody Kang wrote: > The APMU eDP/DP pixel-clock muxes were parented to a placeholder, because > their real parents (the DP PHY PLLs) did not exist as clocks yet. They do > now, so point each mux at its PLL and let the pixel clock reach the > controller through the clock framework. > > Signed-off-by: Cody Kang > > --- > v2: > - no change > --- > drivers/clk/spacemit/ccu-k3.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/clk/spacemit/ccu-k3.c b/drivers/clk/spacemit/ccu-k3.c > index cb0c4277f72a8..afbbbd797826d 100644 > --- a/drivers/clk/spacemit/ccu-k3.c > +++ b/drivers/clk/spacemit/ccu-k3.c > @@ -937,13 +937,13 @@ CCU_MUX_DIV_GATE_FC_DEFINE(ufs_aclk, ufs_aclk_paren= ts, APMU_UFS_CLK_RES_CTRL, 5, >=20=20 > static const struct clk_parent_data edp0_pclk_parents[] =3D { > CCU_PARENT_HW(lcd_pxclk), > - CCU_PARENT_NAME(external_clk), > + CCU_PARENT_NAME(edp0_pll_pxclk), hidden behind this macro is a .fw_name. Basically these are your DT input names. You can't change them at will like that because your bindings are supposed to be stable. You should not use placeholder name, ever. No need to make the name too specific either, from the beginning it should have been something like "pixel_pll" or whatever your doc refer it with. "external_clk" is not declared in your bindings AFAICT, so at least it should be fine to remove it. There need to be a dt-binding patch documenting the new name you need and then you can start using it in the driver. In the future, please try to document all the inputs of your controller in the bindings. Even if you do not use them yet, they exist. > }; > CCU_MUX_GATE_DEFINE(edp0_pxclk, edp0_pclk_parents, APMU_LCD_EDP_CTRL, 2,= 1, BIT(1), 0); >=20=20 > static const struct clk_parent_data edp1_pclk_parents[] =3D { > CCU_PARENT_HW(dsi4ln2_lcd_pxclk), > - CCU_PARENT_NAME(external_clk), > + CCU_PARENT_NAME(edp1_pll_pxclk), > }; > CCU_MUX_GATE_DEFINE(edp1_pxclk, edp1_pclk_parents, APMU_LCD_EDP_CTRL, 18= , 1, BIT(17), 0); >=20=20 > > --=20 > 2.43.0 > --=20 Jerome