| From foo@baz Fri Aug 8 09:26:33 PDT 2014 |
| From: Andrey Utkin <andrey.krieger.utkin@gmail.com> |
| Date: Mon, 4 Aug 2014 23:47:41 +0300 |
| Subject: arch/sparc/math-emu/math_32.c: drop stray break operator |
| |
| From: Andrey Utkin <andrey.krieger.utkin@gmail.com> |
| |
| [ Upstream commit 093758e3daede29cb4ce6aedb111becf9d4bfc57 ] |
| |
| 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@hotmail.com> |
| Signed-off-by: Andrey Utkin <andrey.krieger.utkin@gmail.com> |
| Signed-off-by: David S. Miller <davem@davemloft.net> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| arch/sparc/math-emu/math_32.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/arch/sparc/math-emu/math_32.c |
| +++ b/arch/sparc/math-emu/math_32.c |
| @@ -499,7 +499,7 @@ static int do_one_mathemu(u32 insn, unsi |
| case 0: fsr = *pfsr; |
| if (IR == -1) IR = 2; |
| /* fcc is always fcc0 */ |
| - fsr &= ~0xc00; fsr |= (IR << 10); break; |
| + fsr &= ~0xc00; fsr |= (IR << 10); |
| *pfsr = fsr; |
| break; |
| case 1: rd->s = IR; break; |