lkml.org 
[lkml]   [2016]   [Sep]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 12/21] cpufreq: Convert to hotplug state machine
    On 2016-09-06 23:27:46 [+0200], Rafael J. Wysocki wrote:
    > > + ret = cpuhp_setup_state_nocalls(CPUHP_AP_ONLINE_DYN, "cpufreq:online",
    > > + cpufreq_online,
    > > + cpufreq_offline);
    > > + if (ret < 0)
    > > + goto err_if_unreg;
    > > + hp_online = ret;
    >
    > hp_online is enum cpuhp_state (and we pass it to
    > cpuhp_remove_state_nocalls(() later on), but
    > cpuhp_setup_state_nocalls() returns an int (and that should be 0 if it
    > is not an error code AFAICS), so is this actually correct?

    Not sure what you are pointing out here. Let me try to cover it.
    cpuhp_setup_state_nocalls() return <0 for errors. Those are are not
    assigned to hp_online. It returns 0 for success on ID was !=
    CPUHP_AP_ONLINE_DYN and >= 0 for success if ID was CPUHP_AP_ONLINE_DYN.
    In the latter case the dynamic assigned ID is returned which should be
    used if you plan to remove the callbacks.
    Assigning an unsigned int to enum is okay because enumeration constants
    itself should be an int.

    > Thanks,
    > Rafael

    Sebastian

    \
     
     \ /
      Last update: 2016-09-17 09:58    [W:2.859 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site