Re: [PATCH] media: qcom: camss: clean up a check

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Dan Carpenter <dan.carpenter@xxxxxxxxxx>, Robert Foss <rfoss@xxxxxxxxxx>
Subject: Re: [PATCH] media: qcom: camss: clean up a check
From: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx>
Date: Thu, 12 Oct 2023 18:13:19 +0200
Cc: Todor Tomov <todor.too@xxxxxxxxx>, "Bryan O'Donoghue" <bryan.odonoghue@xxxxxxxxxx>, Andy Gross <agross@xxxxxxxxxx>, Bjorn Andersson <andersson@xxxxxxxxxx>, Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>, linux-media@xxxxxxxxxxxxxxx, linux-arm-msm@xxxxxxxxxxxxxxx, kernel-janitors@xxxxxxxxxxxxxxx
In-reply-to: <f11b1d6b-5800-4d75-9732-506be3f8458d@moroto.mountain>
References: <f11b1d6b-5800-4d75-9732-506be3f8458d@moroto.mountain>
User-agent: Mozilla Thunderbird



On 10/12/23 11:42, Dan Carpenter wrote:

Imagine that "->vfe_num" is zero, then the subtraction will underflow to
UINT_MAX.  Plus it's just cleaner to use >= instead.

Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
---

Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx>

Konrad

References:
- [PATCH] media: qcom: camss: clean up a check
  - From: Dan Carpenter

Prev by Date: [PATCH v2] soc: qcom: llcc: Fix LLCC_TRP_ATTR2_CFGn offset
Next by Date: Re: linux-next: Tree for Oct 12 (drivers/gpu/drm/msm/adreno/a6xx_gmu.o)
Previous by thread: Re: [PATCH] media: qcom: camss: clean up a check
Next by thread: [PATCH] coresight-tmc-etr: Fix busy error when enable multiple sources
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [Linux for Sparc] [IETF Annouce] [Security] [Bugtraq] [Linux MIPS] [ECOS] [Asterisk Internet PBX] [Linux API]

Powered by Linux