Skip to content

Commit 55b1c39

Browse files
jhovoldvinodkoul
authored andcommitted
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: Dmitry Baryshkov <[email protected]> Signed-off-by: Johan Hovold <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Vinod Koul <[email protected]>
1 parent ee81f2e commit 55b1c39

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

drivers/phy/qualcomm/phy-qcom-qmp-combo.c

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -895,6 +895,7 @@ struct qmp_combo {
895895
unsigned int dp_aux_cfg;
896896
struct phy_configure_opts_dp dp_opts;
897897

898+
struct clk_fixed_rate pipe_clk_fixed;
898899
struct clk_hw dp_link_hw;
899900
struct clk_hw dp_pixel_hw;
900901
};
@@ -2239,15 +2240,11 @@ static void phy_clk_release_provider(void *res)
22392240
*/
22402241
static int phy_pipe_clk_register(struct qmp_combo *qmp, struct device_node *np)
22412242
{
2242-
struct clk_fixed_rate *fixed;
2243+
struct clk_fixed_rate *fixed = &qmp->pipe_clk_fixed;
22432244
struct clk_init_data init = { };
22442245
char name[64];
22452246
int ret;
22462247

2247-
fixed = devm_kzalloc(qmp->dev, sizeof(*fixed), GFP_KERNEL);
2248-
if (!fixed)
2249-
return -ENOMEM;
2250-
22512248
snprintf(name, sizeof(name), "%s::pipe_clk", dev_name(qmp->dev));
22522249
init.name = name;
22532250
init.ops = &clk_fixed_rate_ops;

0 commit comments

Comments
 (0)