From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933392AbbLSSo1 (ORCPT ); Sat, 19 Dec 2015 13:44:27 -0500 Received: from mout.web.de ([212.227.17.11]:49614 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932750AbbLSSoZ (ORCPT ); Sat, 19 Dec 2015 13:44:25 -0500 To: linux-scsi@vger.kernel.org, Dick Kennedy , "James E. J. Bottomley" , James Smart , "Martin K. Petersen" Cc: LKML , kernel-janitors@vger.kernel.org, Julia Lawall From: SF Markus Elfring Subject: [PATCH] SCSI-lpfc: Use a signed return type for two functions X-Enigmail-Draft-Status: N1110 Message-ID: <5675A576.8020008@users.sourceforge.net> Date: Sat, 19 Dec 2015 19:44:06 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:VM0em+RyeZQGFJNpzvvB1topxOrsyjIBV+irvqELVzwv1v9yY1n SB9ENTMOs0rM4p//ykjPoXcaKEYig4/Wn5mn0QC8Fxtj8a3ns+Cc+WsOF/xQtRVb5OHhdNW Wn3a/twwWVAhaEcPRm5tvCD3B4TF2RetMEAndwxeeIW1m3B5bq8FamKd789G/cgT+rlv09g MCReB09omoeI9A2yanknQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:Z9+DyFr+DSw=:wwYJ6PWHCAytXGuSm/msvr JE+bsWIIS/t2LjndnLhlsJRZ0WGRZV7F+xA3ZC3ufTLJlsChmmVo1xqTE6woSPFBFpVBlqbks tqbx+dhrZiZwGTLXJNQPgqsqS8U44w5ypz/78Q5LvKA8/FbKMoRayqqIJ4m1ma+2s1bd+B3on lPzzTg+octvbmY1NL4PfCbMsXQqqz9pzade5anWs2p66+I/E6XHwghTaNGuZQStfGz7VHPln5 dFgo53pgfHSIa2lXQzKuVnkMOCWPQN7yfaH7/NfbZ8eIwy3NhrvQVyX5Ld+rmC7mU/poRjNgP ekEff4QL0Y7p1ywPsnNUtJ3dON1SKqXgOTzQ4DmqnRZn6bYwmWb8Sj8KbyszRXDmvHLsVAySn Sne8+CE2FnNmGHX4zkWqqJp0ob5RB+W9c0NVknveCYCE4Qv8+zyCOdMPlXP7Nutls+DrLQAHW 9CsxE0Ij2UUKUH86nRmkjKldbpbmyd5dq5phitwyuOH8iFjD6W6zYhKzJSTgYVZOA+ckf8dZf LVbxhzV3wuT+7tyEOD+qzH569S/NxmQZbQJkQUcAN5BLdRc6bVbcF3+DtUx5IKSiyw/8mStDT vEZ+liCGXn/9PA0zEdZR0UcDTs4NW7bg6LwlBTYoBH6yIGsJY8C/AZPQRlu96Kr5o0bZld4YW 6WMzZUD6vx6Rzy9ini32Xhhf2ZAZ+1i/mGP4aXGs/5m+9O7oPXw3rbh81BlEd5108fwNwkUSF 8zIcPbTqvKbRAbOv3RhDraDyO+a+BHi7dwsPvLV2+EcB/zR17R9P40DsTtGtLOMF/7ZeuW109 XDmO4Og Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Sat, 19 Dec 2015 19:32:27 +0100 The return type "size_t" was used by the functions "lpfc_wwn_set" and "lpfc_oas_lun_state_set" despite of the aspect that they will eventually return a negative error code. Improve this implementation detail by using the type "ssize_t" instead. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/scsi/lpfc/lpfc_attr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/lpfc/lpfc_attr.c b/drivers/scsi/lpfc/lpfc_attr.c index f6446d7..777960e 100644 --- a/drivers/scsi/lpfc/lpfc_attr.c +++ b/drivers/scsi/lpfc/lpfc_attr.c @@ -2089,7 +2089,7 @@ static char *lpfc_soft_wwn_key = "C99G71SL8032A"; * -EINVAL if the buffer does not contain a valid wwn * 0 success **/ -static size_t +static ssize_t lpfc_wwn_set(const char *buf, size_t cnt, char wwn[]) { unsigned int i, j; @@ -2570,7 +2570,7 @@ static DEVICE_ATTR(lpfc_xlane_lun_status, S_IRUGO, * -EPERM OAS is not enabled or not supported by this port. * */ -static size_t +static ssize_t lpfc_oas_lun_state_set(struct lpfc_hba *phba, uint8_t vpt_wwpn[], uint8_t tgt_wwpn[], uint64_t lun, uint32_t oas_state) { -- 2.6.3