Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752811Ab3HVB4E (ORCPT ); Wed, 21 Aug 2013 21:56:04 -0400 Received: from perches-mx.perches.com ([206.117.179.246]:34326 "EHLO labridge.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752120Ab3HVB4B (ORCPT ); Wed, 21 Aug 2013 21:56:01 -0400 Message-ID: <1377136559.12003.18.camel@joe-AO722> Subject: Re: [PATCH] scsi: fix the build warning From: Joe Perches To: "Martin K. Petersen" Cc: Zhi Yong Wu , linux-scsi@vger.kernel.org, James Bottomley , linux-kernel mlist , Zhi Yong Wu , akinobu.mita@gmail.com Date: Wed, 21 Aug 2013 18:55:59 -0700 In-Reply-To: References: <1377132262-15744-1-git-send-email-zwu.kernel@gmail.com> <1377133329.12003.9.camel@joe-AO722> <1377134271.12003.15.camel@joe-AO722> Content-Type: text/plain; charset="ISO-8859-1" X-Mailer: Evolution 3.6.4-0ubuntu1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1128 Lines: 34 On Wed, 2013-08-21 at 21:25 -0400, Martin K. Petersen wrote: > >>>>> "Joe" == Joe Perches writes: > > Joe> I don't get this build warning in the first place and I think the > Joe> scsi_debug file is quite old and probably doesn't need to be > Joe> changed at all. > > guard isn't a boolean, it selects the checksum algorithm used. OK, maybe this then... --- drivers/scsi/scsi_debug.c | 2 +- 1 file changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/scsi_debug.c b/drivers/scsi/scsi_debug.c index cb4fefa..6fc2831 100644 --- a/drivers/scsi/scsi_debug.c +++ b/drivers/scsi/scsi_debug.c @@ -3312,7 +3312,7 @@ static int __init scsi_debug_init(void) return -EINVAL; } - if (scsi_debug_guard > 1) { + if (scsi_debug_guard < 0 || scsi_debug_guard > 1) { printk(KERN_ERR "scsi_debug_init: guard must be 0 or 1\n"); return -EINVAL; } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/