mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-08-31 04:46:30 -04:00
clk: ti: Make sure clk_init_data is fully initialized
The clk_init_data structure contains several mutually-exclusive members
for different methods to specify the possible parents of a clock,
prompting drivers to initialize only the members they need. However,
not initializing all members may cause subtle issues, which are only
exposed when CONFIG_INIT_STACK_ALL_PATTERN or CONFIG_INIT_STACK_NONE is
enabled.
_register_mux() fills in init.parent_data, and assumes that
init.parent_names is NULL. However, the latter is uninitialized, and
thus may cause a crash.
Make sure all members are fully initialized, to fix such bugs, and to
avoid future breakage when converting drivers to a different method for
specifying the parents.
Fixes: 667f420c09 ("clk: ti: mux: resolve parent clocks by DT index, not by name")
Closes: https://lore.kernel.org/CAMuHMdU3yVqoyHC4eNF2NuYo8wy+6ODLoYat4R71X99Mxc_=kw@mail.gmail.com
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Brian Masney <bmasney@redhat.com>
Reviewed-by: Mathieu Dubois-Briand <mathieu.dubois-briand@bootlin.com>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
This commit is contained in:
committed by
Stephen Boyd
parent
8bc7ceadce
commit
3ae1674975
@@ -483,7 +483,7 @@ static const struct clk_ops ti_adpll_ops = {
|
||||
|
||||
static int ti_adpll_init_dco(struct ti_adpll_data *d)
|
||||
{
|
||||
struct clk_init_data init;
|
||||
struct clk_init_data init = {};
|
||||
struct clk *clock;
|
||||
const char *postfix;
|
||||
int width, err;
|
||||
@@ -576,7 +576,7 @@ static int ti_adpll_init_clkout(struct ti_adpll_data *d,
|
||||
struct clk *clk1)
|
||||
{
|
||||
struct ti_adpll_clkout_data *co;
|
||||
struct clk_init_data init;
|
||||
struct clk_init_data init = {};
|
||||
struct clk_ops *ops;
|
||||
const char *parent_names[2];
|
||||
const char *child_name;
|
||||
|
||||
@@ -311,7 +311,7 @@ static struct clk *_register_divider(struct device_node *node,
|
||||
u32 flags,
|
||||
struct clk_omap_divider *div)
|
||||
{
|
||||
struct clk_init_data init;
|
||||
struct clk_init_data init = {};
|
||||
const char *parent_name;
|
||||
const char *name;
|
||||
|
||||
|
||||
@@ -124,9 +124,9 @@ static struct clk *_register_mux(struct device_node *node, const char *name,
|
||||
struct clk_omap_reg *reg, u8 shift, u32 mask,
|
||||
s8 latch, u8 clk_mux_flags, u32 *table)
|
||||
{
|
||||
struct clk_init_data init = {};
|
||||
struct clk_omap_mux *mux;
|
||||
struct clk *clk;
|
||||
struct clk_init_data init;
|
||||
|
||||
/* allocate the mux */
|
||||
mux = kzalloc_obj(*mux);
|
||||
|
||||
Reference in New Issue
Block a user