09.04.2020 20:52, Thierry Reding пишет:
...
> +static long tegra210_clk_emc_round_rate(struct clk_hw *hw, unsigned long rate,
> + unsigned long *prate)
> +{
> + struct tegra210_clk_emc *emc = to_tegra210_clk_emc(hw);
> + struct tegra210_clk_emc_provider *provider = emc->provider;
> + unsigned int i;
> +
> + if (!provider || !provider->configs || provider->num_configs == 0)
> + return clk_hw_get_rate(hw);
This still looks wrong to me. Nobody should be able to get EMC clock
until provider is registered.
This is troublesome, especially given that you're allowing the EMC
driver to be compiled as a loadable module. For example, this won't work
with the current ACTMON driver because it builds OPP table based on the
clk-rate rounding during the driver's probe, so it won't be able to do
it properly if provider is "temporarily" missing.
... I think that in a longer run we should stop manually building the
ACTMON's OPP table and instead define a proper OPP table (per-HW Speedo
ID, with voltages) in a device-tree. But this is just a vague plans for
the future for now.
