From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 07B20359A9E for ; Sat, 28 Feb 2026 18:14:01 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772302441; cv=none; b=DwepaShKIYvfqmi2198RRN1HYKH2D8u76CD5miam80L6ssC8sGyyF6xRuiGRQdInbR18YK9QWCJrSA5Cd1Nhy8bNJhuLK7DtF3yUngi5b3zxjU6VfVJLNJfA/jNxFkBYsglzW4MU3GRW7EYoVkPSuhJhEASJjRFtSdgrynqruR8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772302441; c=relaxed/simple; bh=4lkz5QtqazdIxX2cZN66Jk0z3fss+lyB1L6jK0pFnrI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=EWJ9ZDXFwhm2r1Zbx0Jg3GnLXW01p85708o1aNhIiMLZRiDsnA/DgkbBAtz51dZT4FXwMz2mai2uCDUcZ+Dn8P0cstsBbpw1Fo+7JAQmeVUXoI4dE+WKb/AXIR8zf01tbryIvlsEqC8TsFYrSo8ERkJNPOubzK/39x179kkr6rw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=Gik7ifDU; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="Gik7ifDU" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 66587C116D0; Sat, 28 Feb 2026 18:14:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1772302440; bh=4lkz5QtqazdIxX2cZN66Jk0z3fss+lyB1L6jK0pFnrI=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=Gik7ifDU0OJXmylp4Z6357sq379MrXIWMBrJNmhl2IeFrDXBnpK77h9nGBr/egBmR Exk2rjXFDzBXaLjoevrQ1GF9JUxM/boIrzeEKF2PGg101QseTv047KimMIXhKYvSg8 BYAQr/i5cSZP8PLMwgjKQPbOCwyk7Cy/cMKIsgXbZ+TxJfeXaOeL8dA1dd5O6lmgSq rrGVa4bBSNa62faF9jfkh4X9z2unGu2nr81gTPLNX90pLIJT1hr6JRBZ9abivpOA9O ZD8LvQWKLUl1YwIGcebMWxtDDEiKkkTLomFNfoTbVvy6XvFbhBQbd6AsdUUYmqmeJh EHXaJKI3ya9jA== From: Sasha Levin To: patches@lists.linux.dev Cc: Martin Blumenstingl , Jerome Brunet , Sasha Levin Subject: [PATCH 6.1 179/232] clk: meson: gxbb: Limit the HDMI PLL OD to /4 on GXL/GXM SoCs Date: Sat, 28 Feb 2026 13:10:32 -0500 Message-ID: <20260228181127.1592657-179-sashal@kernel.org> X-Mailer: git-send-email 2.51.0 In-Reply-To: <20260228181127.1592657-1-sashal@kernel.org> References: <20260228181127.1592657-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore Content-Transfer-Encoding: 8bit From: Martin Blumenstingl [ Upstream commit 5b1a43950fd3162af0ce52b13c14a2d29b179d4f ] GXBB has the HDMI PLL OD in the HHI_HDMI_PLL_CNTL2 register while for GXL/GXM the OD has moved to HHI_HDMI_PLL_CNTL3. At first glance the rest of the OD setup seems identical. However, looking at the downstream kernel sources as well as testing shows that GXL only supports three OD values: - register value 0 means: divide by 1 - register value 1 means: divide by 2 - register value 2 means: divide by 4 Using register value 3 (which on GXBB means: divide by 8) still divides by 4 as verified using meson-clk-measure. Downstream sources are also only using OD register values 0, 1 and 2 for GXL (while for GXBB the downstream kernel sources are also using value 3). Add clk_div_table and have it replace the CLK_DIVIDER_POWER_OF_TWO flag to make the kernel's view of this register match with how the hardware actually works. Fixes: 69d92293274b ("clk: meson: add the gxl hdmi pll") Signed-off-by: Martin Blumenstingl Link: https://lore.kernel.org/r/20260105204710.447779-2-martin.blumenstingl@googlemail.com Signed-off-by: Jerome Brunet Signed-off-by: Sasha Levin --- drivers/clk/meson/gxbb.c | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/drivers/clk/meson/gxbb.c b/drivers/clk/meson/gxbb.c index 35bc13e73c0dd..6f3918f0a7826 100644 --- a/drivers/clk/meson/gxbb.c +++ b/drivers/clk/meson/gxbb.c @@ -316,12 +316,23 @@ static struct clk_regmap gxbb_hdmi_pll = { }, }; +/* + * GXL hdmi OD dividers are POWER_OF_TWO dividers but limited to /4. + * A divider value of 3 should map to /8 but instead map /4 so ignore it. + */ +static const struct clk_div_table gxl_hdmi_pll_od_div_table[] = { + { .val = 0, .div = 1 }, + { .val = 1, .div = 2 }, + { .val = 2, .div = 4 }, + { /* sentinel */ } +}; + static struct clk_regmap gxl_hdmi_pll_od = { .data = &(struct clk_regmap_div_data){ .offset = HHI_HDMI_PLL_CNTL + 8, .shift = 21, .width = 2, - .flags = CLK_DIVIDER_POWER_OF_TWO, + .table = gxl_hdmi_pll_od_div_table, }, .hw.init = &(struct clk_init_data){ .name = "hdmi_pll_od", @@ -339,7 +350,7 @@ static struct clk_regmap gxl_hdmi_pll_od2 = { .offset = HHI_HDMI_PLL_CNTL + 8, .shift = 23, .width = 2, - .flags = CLK_DIVIDER_POWER_OF_TWO, + .table = gxl_hdmi_pll_od_div_table, }, .hw.init = &(struct clk_init_data){ .name = "hdmi_pll_od2", @@ -357,7 +368,7 @@ static struct clk_regmap gxl_hdmi_pll = { .offset = HHI_HDMI_PLL_CNTL + 8, .shift = 19, .width = 2, - .flags = CLK_DIVIDER_POWER_OF_TWO, + .table = gxl_hdmi_pll_od_div_table, }, .hw.init = &(struct clk_init_data){ .name = "hdmi_pll", -- 2.51.0