From: Andrey Utkin <andrey.krieger.utkin@xxxxxxxxx> Date: Mon, 4 Aug 2014 23:47:41 +0300 > This commit is a guesswork, but it seems to make sense to drop this > break, as otherwise the following line is never executed and becomes > dead code. And that following line actually saves the result of > local calculation by the pointer given in function argument. So the > proposed change makes sense if this code in the whole makes sense (but I > am unable to analyze it in the whole). > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=81641 > Reported-by: David Binderman <dcb314@xxxxxxxxxxx> > Signed-off-by: Andrey Utkin <andrey.krieger.utkin@xxxxxxxxx> Yeah, not putting the condition code result into the register might cause problems :-) Thanks for catching this, applied and queued up for -stable. -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
