diff mbox series

clk: nomadik: Correct str_enabled_disabled() SXTALO case

Message ID 20250115081914.12841-1-krzysztof.kozlowski@linaro.org (mailing list archive)
State New
Headers show
Series clk: nomadik: Correct str_enabled_disabled() SXTALO case | expand

Commit Message

Krzysztof Kozlowski Jan. 15, 2025, 8:19 a.m. UTC
Commit 1cd1c8f400e9 ("clk: Use str_enable_disable-like helpers") should
use reversed check for SXTALO.

Reported-by: Stanislav Jakubek <stano.jakubek@gmail.com>
Closes: https://lore.kernel.org/all/Z4diEj__bbQl4ULS@standask-GA-A55M-S2HP/
Fixes: 1cd1c8f400e9 ("clk: Use str_enable_disable-like helpers")
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 drivers/clk/clk-nomadik.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Linus Walleij Jan. 15, 2025, 11:28 a.m. UTC | #1
On Wed, Jan 15, 2025 at 9:19 AM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:

> Commit 1cd1c8f400e9 ("clk: Use str_enable_disable-like helpers") should
> use reversed check for SXTALO.
>
> Reported-by: Stanislav Jakubek <stano.jakubek@gmail.com>
> Closes: https://lore.kernel.org/all/Z4diEj__bbQl4ULS@standask-GA-A55M-S2HP/
> Fixes: 1cd1c8f400e9 ("clk: Use str_enable_disable-like helpers")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Thanks a lot for fixing up this Krzysztof!
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij
diff mbox series

Patch

diff --git a/drivers/clk/clk-nomadik.c b/drivers/clk/clk-nomadik.c
index f3a73ac5a1b9..fc0aeb4247f2 100644
--- a/drivers/clk/clk-nomadik.c
+++ b/drivers/clk/clk-nomadik.c
@@ -117,7 +117,7 @@  static void __init nomadik_src_init(void)
 
 	val = readl(src_base + SRC_XTALCR);
 	pr_info("SXTALO is %s\n",
-		str_enabled_disabled(val & SRC_XTALCR_SXTALDIS));
+		str_disabled_enabled(val & SRC_XTALCR_SXTALDIS));
 	pr_info("MXTAL is %s\n",
 		str_enabled_disabled(val & SRC_XTALCR_MXTALSTAT));
 	if (of_property_read_bool(np, "disable-sxtalo")) {