[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20220613080706.jqilerx6jc4m2pxh@vireshk-i7>
Date: Mon, 13 Jun 2022 13:37:06 +0530
From: Viresh Kumar <viresh.kumar@...aro.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
Viresh Kumar <vireshk@...nel.org>, Nishanth Menon <nm@...com>,
Stephen Boyd <sboyd@...nel.org>,
"Rafael J. Wysocki" <rafael@...nel.org>
Cc: linux-pm@...r.kernel.org,
Vincent Guittot <vincent.guittot@...aro.org>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 5/8] OPP: Allow multiple clocks for a device
On 10-06-22, 13:50, Viresh Kumar wrote:
> @@ -1594,24 +1601,28 @@ EXPORT_SYMBOL_GPL(dev_pm_opp_remove_all_dynamic);
> struct dev_pm_opp *_opp_allocate(struct opp_table *opp_table)
> {
> struct dev_pm_opp *opp;
> - int supply_count, supply_size, icc_size;
> + int supply_count, supply_size, icc_size, clk_size;
>
> /* Allocate space for at least one supply */
> supply_count = opp_table->regulator_count > 0 ?
> opp_table->regulator_count : 1;
> supply_size = sizeof(*opp->supplies) * supply_count;
> + clk_size = sizeof(*opp->rates) * opp_table->clk_count;
> icc_size = sizeof(*opp->bandwidth) * opp_table->path_count;
>
> /* allocate new OPP node and supplies structures */
> opp = kzalloc(sizeof(*opp) + supply_size + icc_size, GFP_KERNEL);
The change for this line was lost in rebase I think. I have fixed my branch with
this Krzysztof, so testing over it should be fine. Thanks.
--
viresh
Powered by blists - more mailing lists