diff options
| author | Xiang Chen <chenxiang66@hisilicon.com> | 2019-02-28 09:50:57 -0500 |
|---|---|---|
| committer | Martin K. Petersen <martin.petersen@oracle.com> | 2019-03-06 19:26:46 -0500 |
| commit | fba770c6682447f1998267f4143595e77be4f112 (patch) | |
| tree | 0e5bf3c5f789b3e8a34d29d70c87aecace644b15 | |
| parent | b8870ec63676aba1d823f0b36c5f7e9929e57d23 (diff) | |
scsi: hisi_sas: Change return variable type in phy_up_v3_hw()
According to the tool fortify, phy_up_v3_hw() returns signed value, while
it should return an unsigned value.
So change variable "res" from int to irq_return_t.
Signed-off-by: Xiang Chen <chenxiang66@hisilicon.com>
Signed-off-by: John Garry <john.garry@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
| -rw-r--r-- | drivers/scsi/hisi_sas/hisi_sas_v3_hw.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/scsi/hisi_sas/hisi_sas_v3_hw.c b/drivers/scsi/hisi_sas/hisi_sas_v3_hw.c index 00738d0673fe..151a102c4cc7 100644 --- a/drivers/scsi/hisi_sas/hisi_sas_v3_hw.c +++ b/drivers/scsi/hisi_sas/hisi_sas_v3_hw.c | |||
| @@ -1344,7 +1344,8 @@ static void prep_abort_v3_hw(struct hisi_hba *hisi_hba, | |||
| 1344 | 1344 | ||
| 1345 | static irqreturn_t phy_up_v3_hw(int phy_no, struct hisi_hba *hisi_hba) | 1345 | static irqreturn_t phy_up_v3_hw(int phy_no, struct hisi_hba *hisi_hba) |
| 1346 | { | 1346 | { |
| 1347 | int i, res; | 1347 | int i; |
| 1348 | irqreturn_t res; | ||
| 1348 | u32 context, port_id, link_rate; | 1349 | u32 context, port_id, link_rate; |
| 1349 | struct hisi_sas_phy *phy = &hisi_hba->phy[phy_no]; | 1350 | struct hisi_sas_phy *phy = &hisi_hba->phy[phy_no]; |
| 1350 | struct asd_sas_phy *sas_phy = &phy->sas_phy; | 1351 | struct asd_sas_phy *sas_phy = &phy->sas_phy; |
