On 31-05-22, 14:05, Chanwoo Choi wrote:
> > diff --git a/drivers/devfreq/exynos-bus.c b/drivers/devfreq/exynos-bus.c
> > index 780e525eb92a..8fca24565e7d 100644
> > --- a/drivers/devfreq/exynos-bus.c
> > +++ b/drivers/devfreq/exynos-bus.c
> > @@ -161,8 +161,11 @@ static void exynos_bus_exit(struct device *dev)
> >
> > dev_pm_opp_of_remove_table(dev);
> > clk_disable_unprepare(bus->clk);
> > - dev_pm_opp_clear_config(bus->opp_table);
> > - bus->opp_table = NULL;
> > +
> > + if (bus->opp_table) {
> > + dev_pm_opp_clear_config(bus->opp_table);
> > + bus->opp_table = NULL;
> > + }
> > }
> >
> > static void exynos_bus_passive_exit(struct device *dev)
> > @@ -463,8 +466,10 @@ static int exynos_bus_probe(struct platform_device *pdev)
> > dev_pm_opp_of_remove_table(dev);
> > clk_disable_unprepare(bus->clk);
> > err_reg:
> > - dev_pm_opp_clear_config(bus->opp_table);
> > - bus->opp_table = NULL;
> > + if (bus->opp_table) {
> > + dev_pm_opp_clear_config(bus->opp_table);
> > + bus->opp_table = NULL;
> > + }
> >
> > return ret;
> > }
> >
>
> This change is enough to remove the null pointer error. Thanks.
Pushed this and WARN_ON() in OPP core. Thanks.
--
viresh
