Re: [PATCH Resend] PM: Remove unnecessary !!

From: Rafael J. Wysocki
Date: Mon Mar 10 2014 - 20:46:56 EST


On Monday, March 10, 2014 07:31:51 PM Viresh Kumar wrote:
> Double ! or !! are normally required to get 0 or 1 out of a expression. A
> comparision always returns 0 or 1 and hence there is no need to apply double !
> over it again.
>
> Signed-off-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>

Queued up for 3.15, thanks!

> ---
> kernel/power/suspend.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/kernel/power/suspend.c b/kernel/power/suspend.c
> index 62ee437..90b3d93 100644
> --- a/kernel/power/suspend.c
> +++ b/kernel/power/suspend.c
> @@ -39,7 +39,7 @@ static const struct platform_suspend_ops *suspend_ops;
>
> static bool need_suspend_ops(suspend_state_t state)
> {
> - return !!(state > PM_SUSPEND_FREEZE);
> + return state > PM_SUSPEND_FREEZE;
> }
>
> static DECLARE_WAIT_QUEUE_HEAD(suspend_freeze_wait_head);
>

--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/