Re: [PATCH] scsi: qedf: Remove always false 'tmp_prio < 0' statement
2019년 9월 24일 (화) 오전 11:32, Martin K. Petersen 님이 작성: > > > Austin, [...] > > So remove 'always false' statement. > > Applied to 5.4/scsi-fixes, thanks! > Good, thanks for information. > -- > Martin K. Petersen Oracle Linux Engineering
Re: [PATCH] scsi: qedf: Remove always false 'tmp_prio < 0' statement
Austin, > Since tmp_prio is declared as u8, the following statement is always false. >tmp_prio < 0 > > So remove 'always false' statement. Applied to 5.4/scsi-fixes, thanks! -- Martin K. Petersen Oracle Linux Engineering
RE: [PATCH] scsi: qedf: Remove always false 'tmp_prio < 0' statement
> -Original Message- > From: linux-scsi-ow...@vger.kernel.org On > Behalf Of Austin Kim > Sent: Thursday, September 19, 2019 1:26 PM > To: j...@linux.ibm.com; martin.peter...@oracle.com > Cc: linux-s...@vger.kernel.org; linux-kernel@vger.kernel.org; QLogic-Storage- > upstr...@cavium.com; austindh@gmail.com > Subject: [PATCH] scsi: qedf: Remove always false 'tmp_prio < 0' statement > > Since tmp_prio is declared as u8, the following statement is always false. >tmp_prio < 0 > > So remove 'always false' statement. > > Signed-off-by: Austin Kim > --- > drivers/scsi/qedf/qedf_main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/scsi/qedf/qedf_main.c b/drivers/scsi/qedf/qedf_main.c > index > 1659d35..59ca98f 100644 > --- a/drivers/scsi/qedf/qedf_main.c > +++ b/drivers/scsi/qedf/qedf_main.c > @@ -596,7 +596,7 @@ static void qedf_dcbx_handler(void *dev, struct > qed_dcbx_get *get, u32 mib_type) > tmp_prio = get->operational.app_prio.fcoe; > if (qedf_default_prio > -1) > qedf->prio = qedf_default_prio; > - else if (tmp_prio < 0 || tmp_prio > 7) { > + else if (tmp_prio > 7) { > QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_DISC, > "FIP/FCoE prio %d out of range, setting to %d.\n", > tmp_prio, QEDF_DEFAULT_PRIO); > -- > 2.6.2 Hi Austin, Thanks for the patch. Acked-by: Saurav Kashyap
[PATCH] scsi: qedf: Remove always false 'tmp_prio < 0' statement
Since tmp_prio is declared as u8, the following statement is always false. tmp_prio < 0 So remove 'always false' statement. Signed-off-by: Austin Kim --- drivers/scsi/qedf/qedf_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/qedf/qedf_main.c b/drivers/scsi/qedf/qedf_main.c index 1659d35..59ca98f 100644 --- a/drivers/scsi/qedf/qedf_main.c +++ b/drivers/scsi/qedf/qedf_main.c @@ -596,7 +596,7 @@ static void qedf_dcbx_handler(void *dev, struct qed_dcbx_get *get, u32 mib_type) tmp_prio = get->operational.app_prio.fcoe; if (qedf_default_prio > -1) qedf->prio = qedf_default_prio; - else if (tmp_prio < 0 || tmp_prio > 7) { + else if (tmp_prio > 7) { QEDF_INFO(&(qedf->dbg_ctx), QEDF_LOG_DISC, "FIP/FCoE prio %d out of range, setting to %d.\n", tmp_prio, QEDF_DEFAULT_PRIO); -- 2.6.2