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 297CE39B976 for ; Sat, 28 Feb 2026 18:16:52 +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=1772302612; cv=none; b=PjtdTDCXjU6kPO7nIy4ewr1odrZlpBiSYBkYBdpXDEWz6ybLW5MTYEhz7lzO8A3Jay1udVIX3ekLgL6rqjMtOG9zEMk8TbDWu2pNAoAbQGx42DwuQ7XJ1d22tiyN/P5cK2ceeD6AUP1nBe02jA2YEkmbrDhDEB2d7Qp5fJux8xQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772302612; c=relaxed/simple; bh=4lkz5QtqazdIxX2cZN66Jk0z3fss+lyB1L6jK0pFnrI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=cfv0yM29ZQ/KRhbGWzgTXFFz1+7NyLwrN3mKBcSA4e3ZygeUZJLAaKQJe4f3TlFhdid0QSJa8qwnb1NiesrLomU3Awp/YLS4POHLsBYq63ftmpaevxiIRnW+VmbC1XFZZnkuiBBl9FKPJ9JYSaGlkBWWy9eDM7uYCOBAtIArjq8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=m70ZcX4M; 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="m70ZcX4M" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8DFE1C19424; Sat, 28 Feb 2026 18:16:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1772302612; bh=4lkz5QtqazdIxX2cZN66Jk0z3fss+lyB1L6jK0pFnrI=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=m70ZcX4MPEBNItRI/YHIXg0p1bHOv6viswouArCKfGhuEksaVDGZ2FizzEHjkk6X0 5/97D6GUILvnUFZ5hQH2RheH4voDkswQLbF4Gy+fcsqmX5Bxt/o5vM7Drek+7FPRSG W03cqO0VVSek5bwl/UesawqH65e0uoQu9k+0da4Ljc62AZd9pq1rmaQZe7cSLWLO6C 0pdT4YrdcgqsAC1hcDT4/DIhg/GCyq/5qrJM5u3mCEqHbMhJbYVHLNA+WaXKPoeSEI +djvPOOvSKVjl3y61JRowWw2cR3fO1ebTWuPGZU9IMcRGinaHF2sGNCEggjRP8V1Ox QcHrNAT0T5XuA== From: Sasha Levin To: patches@lists.linux.dev Cc: Martin Blumenstingl , Jerome Brunet , Sasha Levin Subject: [PATCH 5.15 128/164] clk: meson: gxbb: Limit the HDMI PLL OD to /4 on GXL/GXM SoCs Date: Sat, 28 Feb 2026 13:14:27 -0500 Message-ID: <20260228181505.1600663-128-sashal@kernel.org> X-Mailer: git-send-email 2.51.0 In-Reply-To: <20260228181505.1600663-1-sashal@kernel.org> References: <20260228181505.1600663-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