aboutsummaryrefslogtreecommitdiffstats
path: root/target/linux/bcm27xx/patches-5.15/950-0840-clk-tests-Add-tests-for-uncached-clock.patch
diff options
context:
space:
mode:
Diffstat (limited to 'target/linux/bcm27xx/patches-5.15/950-0840-clk-tests-Add-tests-for-uncached-clock.patch')
-rw-r--r--target/linux/bcm27xx/patches-5.15/950-0840-clk-tests-Add-tests-for-uncached-clock.patch124
1 files changed, 124 insertions, 0 deletions
diff --git a/target/linux/bcm27xx/patches-5.15/950-0840-clk-tests-Add-tests-for-uncached-clock.patch b/target/linux/bcm27xx/patches-5.15/950-0840-clk-tests-Add-tests-for-uncached-clock.patch
new file mode 100644
index 0000000000..3db0743490
--- /dev/null
+++ b/target/linux/bcm27xx/patches-5.15/950-0840-clk-tests-Add-tests-for-uncached-clock.patch
@@ -0,0 +1,124 @@
+From b28798f3ae1a63169dd57e5b5cb55848172aa4c6 Mon Sep 17 00:00:00 2001
+From: Maxime Ripard <maxime@cerno.tech>
+Date: Fri, 8 Apr 2022 09:54:39 +0200
+Subject: [PATCH] clk: tests: Add tests for uncached clock
+
+The clock framework supports clocks that can have their rate changed
+without the kernel knowing about it using the CLK_GET_RATE_NOCACHE flag.
+
+As its name suggests, this flag turns off the rate caching in the clock
+framework, reading out the rate from the hardware any time we need to
+read it.
+
+Let's add a couple of tests to make sure it works as intended.
+
+Tested-by: Alexander Stein <alexander.stein@ew.tq-group.com> # imx8mp
+Tested-by: Marek Szyprowski <m.szyprowski@samsung.com> # exynos4210, meson g12b
+Signed-off-by: Maxime Ripard <maxime@cerno.tech>
+---
+ drivers/clk/clk_test.c | 88 +++++++++++++++++++++++++++++++++++++++++-
+ 1 file changed, 87 insertions(+), 1 deletion(-)
+
+--- a/drivers/clk/clk_test.c
++++ b/drivers/clk/clk_test.c
+@@ -262,6 +262,91 @@ static struct kunit_suite clk_test_suite
+ .test_cases = clk_test_cases,
+ };
+
++static int clk_uncached_test_init(struct kunit *test)
++{
++ struct clk_dummy_context *ctx;
++ int ret;
++
++ ctx = kunit_kzalloc(test, sizeof(*ctx), GFP_KERNEL);
++ if (!ctx)
++ return -ENOMEM;
++ test->priv = ctx;
++
++ ctx->rate = DUMMY_CLOCK_INIT_RATE;
++ ctx->hw.init = CLK_HW_INIT_NO_PARENT("test-clk",
++ &clk_dummy_rate_ops,
++ CLK_GET_RATE_NOCACHE);
++
++ ret = clk_hw_register(NULL, &ctx->hw);
++ if (ret)
++ return ret;
++
++ return 0;
++}
++
++/*
++ * Test that for an uncached clock, the clock framework doesn't cache
++ * the rate and clk_get_rate() will return the underlying clock rate
++ * even if it changed.
++ */
++static void clk_test_uncached_get_rate(struct kunit *test)
++{
++ struct clk_dummy_context *ctx = test->priv;
++ struct clk_hw *hw = &ctx->hw;
++ struct clk *clk = hw->clk;
++ unsigned long rate;
++
++ rate = clk_get_rate(clk);
++ KUNIT_ASSERT_GT(test, rate, 0);
++ KUNIT_EXPECT_EQ(test, rate, DUMMY_CLOCK_INIT_RATE);
++
++ ctx->rate = DUMMY_CLOCK_RATE_1;
++ rate = clk_get_rate(clk);
++ KUNIT_ASSERT_GT(test, rate, 0);
++ KUNIT_EXPECT_EQ(test, rate, DUMMY_CLOCK_RATE_1);
++}
++
++/*
++ * Test that for an uncached clock, clk_set_rate_range() will work
++ * properly if the rate hasn't changed.
++ */
++static void clk_test_uncached_set_range(struct kunit *test)
++{
++ struct clk_dummy_context *ctx = test->priv;
++ struct clk_hw *hw = &ctx->hw;
++ struct clk *clk = hw->clk;
++ unsigned long rate;
++
++ KUNIT_ASSERT_EQ(test,
++ clk_set_rate_range(clk,
++ DUMMY_CLOCK_RATE_1,
++ DUMMY_CLOCK_RATE_2),
++ 0);
++
++ rate = clk_get_rate(clk);
++ KUNIT_ASSERT_GT(test, rate, 0);
++ KUNIT_EXPECT_GE(test, rate, DUMMY_CLOCK_RATE_1);
++ KUNIT_EXPECT_LE(test, rate, DUMMY_CLOCK_RATE_2);
++}
++
++static struct kunit_case clk_uncached_test_cases[] = {
++ KUNIT_CASE(clk_test_uncached_get_rate),
++ KUNIT_CASE(clk_test_uncached_set_range),
++ {}
++};
++
++/*
++ * Test suite for a basic, uncached, rate clock, without any parent.
++ *
++ * These tests are supposed to exercise the rate API with simple scenarios
++ */
++static struct kunit_suite clk_uncached_test_suite = {
++ .name = "clk-uncached-test",
++ .init = clk_uncached_test_init,
++ .exit = clk_test_exit,
++ .test_cases = clk_uncached_test_cases,
++};
++
+ struct clk_single_parent_ctx {
+ struct clk_dummy_context parent_ctx;
+ struct clk_hw hw;
+@@ -1042,6 +1127,7 @@ kunit_test_suites(
+ &clk_orphan_transparent_single_parent_test_suite,
+ &clk_range_test_suite,
+ &clk_range_maximize_test_suite,
+- &clk_range_minimize_test_suite
++ &clk_range_minimize_test_suite,
++ &clk_uncached_test_suite
+ );
+ MODULE_LICENSE("GPL v2");