linux.git/drivers/clk/versatile, branch v6.12.80

clk: Use of_property_present()

2024-08-02T23:53:38+00:00

Use of_property_present() to test for property presence rather than
of_(find|get)_property(). This is part of a larger effort to remove
callers of of_find_property() and similar functions.
of_(find|get)_property() leak the DT struct property and data pointers
which is a problem for dynamically allocated nodes which may be freed.

Signed-off-by: Rob Herring (Arm) 
Link: https://lore.kernel.org/r/20240731191312.1710417-4-robh@kernel.org
Reviewed-by: Linus Walleij 
Acked-by: Geert Uytterhoeven  # clk-mstp.c
Reviewed-by: Geert Uytterhoeven 
Signed-off-by: Stephen Boyd

clk: vexpress-osc: add missing MODULE_DESCRIPTION() macro

2024-07-02T22:56:43+00:00

With ARCH=arm64, make allmodconfig && make W=1 C=1 reports:
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/clk/versatile/clk-vexpress-osc.o

Add the missing invocation of the MODULE_DESCRIPTION() macro.

Signed-off-by: Jeff Johnson 
Link: https://lore.kernel.org/r/20240613-md-arm64-drivers-clk-versatile-v1-1-6b8a5e5e00ef@quicinc.com
Acked-by: Sudeep Holla 
Reviewed-by: Linus Walleij 
Signed-off-by: Stephen Boyd

clk: versatile: sp810: Add a determine_rate hook

2023-06-09T01:39:31+00:00

The Versatile sp810 "timerclken" clock implements a mux with a
set_parent hook, but doesn't provide a determine_rate implementation.

This is a bit odd, since set_parent() is there to, as its name implies,
change the parent of a clock. However, the most likely candidates to
trigger that parent change are either the assigned-clock-parents device
tree property or a call to clk_set_rate(), with determine_rate()
figuring out which parent is the best suited for a given rate.

This mismatch is probably due to the fact that the driver introduction
predates the determine_rate introduction, and it was never revised since
then.

The default, implicit, behaviour that has been in use so far has thus
been to simply keep using the current parent in all cases. This is also
the behaviour of the new clk_hw_determine_rate_no_reparent() helper, so
we can simply use it to make our expectation explicit.

Cc: Linus Walleij 
Cc: Pawel Moll 
Cc: linux-arm-kernel@lists.infradead.org
Signed-off-by: Maxime Ripard 
Link: https://lore.kernel.org/r/20221018-clk-range-checks-fixes-v4-39-971d5077e7d2@cerno.tech
Acked-by: Linus Walleij 
Acked-by: Pawel Moll 
Signed-off-by: Stephen Boyd

clk: versatile: clk-icst: use after free on error path

2021-12-07T20:25:29+00:00

This frees "name" and then tries to display in as part of the error
message on the next line.  Swap the order.

Fixes: 1b2189f3aa50 ("clk: versatile: clk-icst: Ensure clock names are unique")
Signed-off-by: Dan Carpenter 
Link: https://lore.kernel.org/r/20211117072604.GC5237@kili
Reviewed-by: Linus Walleij 
Signed-off-by: Stephen Boyd

clk: versatile: clk-icst: Ensure clock names are unique

2021-11-12T04:27:16+00:00

Commit 2d3de197a818 ("ARM: dts: arm: Update ICST clock nodes 'reg' and
node names") moved to using generic node names. That results in trying
to register multiple clocks with the same name. Fix this by including
the unit-address in the clock name.

Fixes: 2d3de197a818 ("ARM: dts: arm: Update ICST clock nodes 'reg' and node names")
Cc: stable@vger.kernel.org
Cc: Linus Walleij 
Cc: Stephen Boyd 
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-clk@vger.kernel.org
Signed-off-by: Rob Herring 
Reviewed-by: Stephen Boyd 
Reviewed-by: Linus Walleij 
Link: https://lore.kernel.org/r/20211109164650.2233507-3-robh@kernel.org

clk: versatile: hide clock drivers from non-ARM users

2021-10-15T01:00:35+00:00

Commit 419b3ab6987f ("clk: versatile: remove dependency on ARCH_*")
made the whole menu of ARM reference clock drivers visible on all
architectures. I can't see how this is an improvement for non-ARM
users. Unless build-testing, there is no point on presenting
ARM-only clock drivers on other architectures.

Signed-off-by: Jean Delvare 
Cc: Peter Collingbourne 
Cc: Lee Jones 
Cc: Linus Walleij 
Cc: Stephen Boyd 
Link: https://lore.kernel.org/r/20210901180953.5bd2a994@endymion
Reviewed-by: Linus Walleij 
Signed-off-by: Stephen Boyd

clk: versatile: Rename ICST to CLK_ICST

2021-10-15T01:00:25+00:00

For consistency, prefix the ICST config option with CLK as all other
clock source drivers have.

Signed-off-by: Jean Delvare 
Cc: Linus Walleij 
Cc: Michael Turquette 
Cc: Stephen Boyd 
Reviewed-by: Linus Walleij 
Link: https://lore.kernel.org/r/20210901180833.4558932d@endymion
Signed-off-by: Stephen Boyd

clk: versatile: clk-icst: Support 'reg' in addition to 'vco-offset' for register address

2021-09-15T01:11:23+00:00

The ICST binding now also supports 'reg' in addition to 'vco-offset' for
the VCO register address. Add support to the driver to get the VCO
address from 'reg'.

Cc: Linus Walleij 
Cc: Stephen Boyd 
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-clk@vger.kernel.org
Signed-off-by: Rob Herring 
Link: https://lore.kernel.org/r/20210913192816.1225025-6-robh@kernel.org
Reviewed-by: Linus Walleij 
Signed-off-by: Stephen Boyd

clk: versatile: Depend on HAS_IOMEM

2021-06-04T19:23:28+00:00

kbuild robot reports that s390 fails to build this driver with
COMPILE_TEST. Let's depend on HAS_IOMEM so that s390 doesn't try to
build it.

Cc: Lee Jones 
Acked-by: Linus Walleij 
Reported-by: kernel test robot 
Fixes: 419b3ab6987f ("clk: versatile: remove dependency on ARCH_*")
Link: https://lore.kernel.org/r/20210604192321.2594519-1-sboyd@kernel.org
Signed-off-by: Stephen Boyd

clk: versatile: remove dependency on ARCH_*

2021-06-02T01:18:17+00:00

It is now possible to build a modular kernel for vexpress by
not setting CONFIG_ARCH_VEXPRESS=y and instead setting =m on the
drivers that it normally implies. This is with the exception of
CLK_VEXPRESS_OSC which is currently hidden behind a dependency on
one of several ARCH_* variables. Remove that dependency so that
CLK_VEXPRESS_OSC may be enabled without it.

Signed-off-by: Peter Collingbourne 
Link: https://linux-review.googlesource.com/id/I435a21e2e5f6187db54f4ef2079b60028ab2ea69
Signed-off-by: Lee Jones 
Link: https://lore.kernel.org/r/20210520161702.3746174-1-lee.jones@linaro.org
Reviewed-by: Linus Walleij 
Signed-off-by: Stephen Boyd