lkml.org 
[lkml]   [2015]   [Jul]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH 7/7] cpufreq: Separate CPU device removal from CPU online
    From
    Hi Viresh,

    On Tue, Jul 28, 2015 at 4:06 AM, Viresh Kumar <viresh.kumar@linaro.org> wrote:
    > On 27-07-15, 23:56, Rafael J. Wysocki wrote:
    >> OK, I've just seen that patch, but it doesn't modify bus_probe_device() AFAICS.
    >
    > Why is bus_probe_device() required to be modified?

    Because that's a different place where sif->add() is called and if it
    can fail, we need to fail the probing there too.

    In other words, failing things in one place and not doing that in
    another analogous place is inconsistent and incorrect.

    > I wrote this patch to propagate -EPROBE_DEFER from the ->init()
    > callback, which is called from cpufreq_register_driver().
    >
    > And that worked after my patch..
    >
    >> Plus we also ignore the return value of cpufreq_add_dev() in the
    >> hotplug notifier callback.
    >
    > My use case needed it for the subsys callback.

    That case is a failing cpufreq driver registration, right?

    What if a CPU is registered after the cpufreq driver has been
    registered and *that* fails? Shouldn't it fail in the same way in
    both cases?

    Thanks,
    Rafael


    \
     
     \ /
      Last update: 2015-07-28 16:41    [W:2.224 / U:1.388 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site