• Jonathan Marek's avatar
    opp: Don't skip freq update for different frequency · de04241a
    Jonathan Marek authored
    We skip the OPP update if the current and target OPPs are same. This is
    fine for the devices that don't support frequency but may cause issues
    for the ones that need to program frequency.
    
    An OPP entry doesn't really signify a single operating frequency but
    rather the highest frequency at which the other properties of the OPP
    entry apply. And we may reach here with different frequency values,
    while all of them would point to the same OPP entry in the OPP table.
    
    We just need to update the clock frequency in that case, though in order
    to not add special exit points we reuse the code flow from a normal
    path.
    
    While at it, rearrange the conditionals in the 'if' statement to check
    'enabled' flag at the end.
    
    Fixes: 81c4d8a3 ("opp: Keep track of currently programmed OPP")
    Signed-off-by: default avatarJonathan Marek <jonathan@marek.ca>
    [ Viresh: Improved commit log and subject, rename current_freq as
    	  current_rate, document it, remove local variable and rearrange
    	  code. ]
    Signed-off-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
    de04241a
opp.h 9.27 KB