• Nishanth Menon's avatar
    ARM: OMAP2+: OPP: Fix to ensure check of right oppdef after bad one · b110547e
    Nishanth Menon authored
    Commit 9fa2df6b
    (ARM: OMAP2+: OPP: allow OPP enumeration to continue if device is not present)
    makes the logic:
    for (i = 0; i < opp_def_size; i++) {
    	<snip>
    	if (!oh || !oh->od) {
    		<snip>
    		continue;
    	}
    <snip>
    opp_def++;
    }
    
    In short, the moment we hit a "Bad OPP", we end up looping the list
    comparing against the bad opp definition pointer for the rest of the
    iteration count. Instead, increment opp_def in the for loop itself
    and allow continue to be used in code without much thought so that
    we check the next set of OPP definition pointers :)
    
    Cc: Steve Sakoman <steve@sakoman.com>
    Cc: Tony Lindgren <tony@atomide.com>
    Cc: stable@vger.kernel.org
    Signed-off-by: default avatarNishanth Menon <nm@ti.com>
    Signed-off-by: default avatarKevin Hilman <khilman@ti.com>
    b110547e
opp.c 2.49 KB