Linux MIPS Architecture development
 help / color / mirror / Atom feed
From: Ralf Baechle <ralf@linux-mips.org>
To: Markos Chandras <markos.chandras@imgtec.com>
Cc: linux-mips@linux-mips.org
Subject: Re: [PATCH v2] MIPS: math-emu: ieee754int.h: Fix indentation
Date: Fri, 17 Jul 2015 12:35:41 +0200	[thread overview]
Message-ID: <20150717103541.GE9084@linux-mips.org> (raw)
In-Reply-To: <1437061413-8385-1-git-send-email-markos.chandras@imgtec.com>

On Thu, Jul 16, 2015 at 04:43:33PM +0100, Markos Chandras wrote:

> diff --git a/arch/mips/math-emu/ieee754int.h b/arch/mips/math-emu/ieee754int.h
> index 05389d5e3a93..6383e2c5c1ad 100644
> --- a/arch/mips/math-emu/ieee754int.h
> +++ b/arch/mips/math-emu/ieee754int.h
> @@ -65,8 +65,8 @@ static inline int ieee754_class_nan(int xc)
>  			vc = IEEE754_CLASS_INF;				\
>  		else if (vm & SP_MBIT(SP_FBITS-1))			\
>  			vc = IEEE754_CLASS_SNAN;			\
> -	else								\
> -		vc = IEEE754_CLASS_QNAN;				\
> +		else							\
> +			vc = IEEE754_CLASS_QNAN;			\
>  	} else if (ve == SP_EMIN-1+SP_EBIAS) {				\
>  		if (vm) {						\
>  			ve = SP_EMIN;					\
> @@ -105,8 +105,8 @@ static inline int ieee754_class_nan(int xc)
>  		if (vm) {						\
>  			ve = DP_EMIN;					\
>  			vc = IEEE754_CLASS_DNORM;			\
> -	} else								\
> -		vc = IEEE754_CLASS_ZERO;				\
> +		} else							\
> +			vc = IEEE754_CLASS_ZERO;			\
>  	} else {							\
>  		ve -= DP_EBIAS;						\
>  		vm |= DP_HIDDEN_BIT;					\

Looks correct but the the first segment duplicates
https://patchwork.linux-mips.org/patch/10732/ so I'm applying only
the 2nd segment.

Thanks,

  Ralf

  parent reply	other threads:[~2015-07-17 10:35 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-16 12:27 [PATCH] MIPS: math-emu: ieee754int.h: Fix indentation Markos Chandras
2015-07-16 12:27 ` Markos Chandras
2015-07-16 15:43 ` [PATCH v2] " Markos Chandras
2015-07-16 15:43   ` Markos Chandras
2015-07-17 10:35   ` Ralf Baechle [this message]
2015-07-17  7:58 ` [PATCH] " Ralf Baechle

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150717103541.GE9084@linux-mips.org \
    --to=ralf@linux-mips.org \
    --cc=linux-mips@linux-mips.org \
    --cc=markos.chandras@imgtec.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox