From patchwork Tue Sep 25 11:19:04 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sachin Kamat X-Patchwork-Id: 1503941 Return-Path: X-Original-To: patchwork-linux-media@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id 7C36B3FE65 for ; Tue, 25 Sep 2012 11:23:01 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755539Ab2IYLWp (ORCPT ); Tue, 25 Sep 2012 07:22:45 -0400 Received: from mail-pa0-f46.google.com ([209.85.220.46]:56770 "EHLO mail-pa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754810Ab2IYLWo (ORCPT ); Tue, 25 Sep 2012 07:22:44 -0400 Received: by padhz1 with SMTP id hz1so1591877pad.19 for ; Tue, 25 Sep 2012 04:22:43 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=from:to:cc:subject:date:message-id:x-mailer:x-gm-message-state; bh=iwXK/ENJbNSuF1XrIuDvWHGNpuX30TEM0t+iuFaxOso=; b=bxuC9CYlzlAbvZqiUSZ3mbMO7ZYM4i/o0gXm8UFg4wLi031m5AAiV4LyyRodjvoZDp WQpErRVgWfoaWld2s6nOeRLB826o/fQsHY/KQYjfYj9j6DE4s6Wsrq5Omj2i6fxCYceF 2DleppF2vffyZceGTHudF3iunHfW1f57gieyqkF1n1D7OdQhPWT2PhvArGXMR1CMNrYw 1lqbnY7vO9oSttIZW9qteepEHy1RbC9I4r6O2gNd9/UCsQj20lqv1RXfaLySUcv9LSNY 1ui/a3FZRlmOe9rF+awEHqyIIAmtIcfWfKw5SQgwIWPfebNf4aoLh/PnHdqbfVcsgwdP mESw== Received: by 10.68.136.229 with SMTP id qd5mr12733776pbb.154.1348572163551; Tue, 25 Sep 2012 04:22:43 -0700 (PDT) Received: from localhost.localdomain ([115.113.119.130]) by mx.google.com with ESMTPS id uh7sm197087pbc.35.2012.09.25.04.22.40 (version=TLSv1/SSLv3 cipher=OTHER); Tue, 25 Sep 2012 04:22:42 -0700 (PDT) From: Sachin Kamat To: linux-media@vger.kernel.org Cc: mchehab@infradead.org, s.nawrocki@samsung.com, sachin.kamat@linaro.org, patches@linaro.org Subject: [PATCH] [media] s5p-fimc: Fix incorrect condition in fimc_lite_reqbufs() Date: Tue, 25 Sep 2012 16:49:04 +0530 Message-Id: <1348571944-7139-1-git-send-email-sachin.kamat@linaro.org> X-Mailer: git-send-email 1.7.4.1 X-Gm-Message-State: ALoCoQljUJTXMz/RGFXUz/lho2kU0CrhFwYJuw+dQ+6FC35TKH/Ivqbi4gTHPES70eZ49niZmhRk Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org When precedence rules are applied, the condition always evaluates to be false which was not the intention. Adding the missing braces for correct evaluation of the expression and subsequent functionality. Signed-off-by: Sachin Kamat --- drivers/media/platform/s5p-fimc/fimc-lite.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/media/platform/s5p-fimc/fimc-lite.c b/drivers/media/platform/s5p-fimc/fimc-lite.c index 9289008..20e5e24 100644 --- a/drivers/media/platform/s5p-fimc/fimc-lite.c +++ b/drivers/media/platform/s5p-fimc/fimc-lite.c @@ -825,7 +825,7 @@ static int fimc_lite_reqbufs(struct file *file, void *priv, reqbufs->count = max_t(u32, FLITE_REQ_BUFS_MIN, reqbufs->count); ret = vb2_reqbufs(&fimc->vb_queue, reqbufs); - if (!ret < 0) + if (!(ret < 0)) fimc->reqbufs_count = reqbufs->count; return ret;