Commit 55b1c39b authored by Johan Hovold's avatar Johan Hovold Committed by Vinod Koul

phy: qcom-qmp-combo: drop redundant clock allocation

Since the QMP driver split, there is no reason to allocate the
fixed-rate pipe clock structure separately from the driver data.
Reviewed-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: default avatarJohan Hovold <johan+linaro@kernel.org>
Link: https://lore.kernel.org/r/20221121085058.31213-8-johan+linaro@kernel.orgSigned-off-by: default avatarVinod Koul <vkoul@kernel.org>
parent ee81f2eb
...@@ -895,6 +895,7 @@ struct qmp_combo { ...@@ -895,6 +895,7 @@ struct qmp_combo {
unsigned int dp_aux_cfg; unsigned int dp_aux_cfg;
struct phy_configure_opts_dp dp_opts; struct phy_configure_opts_dp dp_opts;
struct clk_fixed_rate pipe_clk_fixed;
struct clk_hw dp_link_hw; struct clk_hw dp_link_hw;
struct clk_hw dp_pixel_hw; struct clk_hw dp_pixel_hw;
}; };
...@@ -2239,15 +2240,11 @@ static void phy_clk_release_provider(void *res) ...@@ -2239,15 +2240,11 @@ static void phy_clk_release_provider(void *res)
*/ */
static int phy_pipe_clk_register(struct qmp_combo *qmp, struct device_node *np) static int phy_pipe_clk_register(struct qmp_combo *qmp, struct device_node *np)
{ {
struct clk_fixed_rate *fixed; struct clk_fixed_rate *fixed = &qmp->pipe_clk_fixed;
struct clk_init_data init = { }; struct clk_init_data init = { };
char name[64]; char name[64];
int ret; int ret;
fixed = devm_kzalloc(qmp->dev, sizeof(*fixed), GFP_KERNEL);
if (!fixed)
return -ENOMEM;
snprintf(name, sizeof(name), "%s::pipe_clk", dev_name(qmp->dev)); snprintf(name, sizeof(name), "%s::pipe_clk", dev_name(qmp->dev));
init.name = name; init.name = name;
init.ops = &clk_fixed_rate_ops; init.ops = &clk_fixed_rate_ops;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment