Re: [PATCH v3 2/3] cpufreq: brcmstb-avs-cpufreq: AVS CPUfreq driver for Broadcom STB SoCs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 11 October 2016 at 22:38, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> On 07-10-16, 16:21, Markus Mayer wrote:
>> +static int brcm_avs_cpufreq_exit(struct cpufreq_policy *policy)
>> +{
>> +     struct private_data *priv = policy->driver_data;
>> +
>> +     iounmap(priv->base);
>> +     iounmap(priv->avs_intr_base);
>> +
>> +     return 0;
>> +}
>
> Shouldn't the above be done in the below routine instead ?

Good point. I'll move it.

>> +static int brcm_avs_cpufreq_remove(struct platform_device *pdev)
>> +{
>> +     platform_set_drvdata(pdev, NULL);
>> +
>> +     return cpufreq_unregister_driver(&brcm_avs_driver);
>> +}
>
> --
> viresh
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux