On 09/06/16 02:04, Laurence Oberman wrote:
> 
> 
> ----- Original Message -----
>> From: "Luis de Bethencourt" <lui...@osg.samsung.com>
>> To: linux-ker...@vger.kernel.org
>> Cc: h...@suse.com, j...@linux.vnet.ibm.com, "martin petersen" 
>> <martin.peter...@oracle.com>,
>> linux-scsi@vger.kernel.org, jav...@osg.samsung.com, "Luis de Bethencourt" 
>> <lui...@osg.samsung.com>
>> Sent: Wednesday, June 8, 2016 6:23:02 PM
>> Subject: [PATCH] aic7xxx: fix wrong return values
>>
>> Convention of error codes says to return them as negative values.
>>
>> Signed-off-by: Luis de Bethencourt <lui...@osg.samsung.com>
>> ---
>>  drivers/scsi/aic7xxx/aic7770_osm.c     |  6 +++---
>>  drivers/scsi/aic7xxx/aic79xx_core.c    | 24 ++++++++++++------------
>>  drivers/scsi/aic7xxx/aic79xx_osm.c     |  8 ++++----
>>  drivers/scsi/aic7xxx/aic79xx_osm_pci.c | 16 ++++++++--------
>>  drivers/scsi/aic7xxx/aic79xx_pci.c     |  2 +-
>>  drivers/scsi/aic7xxx/aic7xxx_core.c    | 26 +++++++++++++-------------
>>  drivers/scsi/aic7xxx/aic7xxx_osm.c     |  8 ++++----
>>  drivers/scsi/aic7xxx/aic7xxx_osm_pci.c | 12 ++++++------
>>  drivers/scsi/aic7xxx/aic7xxx_pci.c     |  4 ++--
>>  9 files changed, 53 insertions(+), 53 deletions(-)
>>
>> diff --git a/drivers/scsi/aic7xxx/aic7770_osm.c
>> b/drivers/scsi/aic7xxx/aic7770_osm.c
>> index 3d401d0..50f030d 100644
>> --- a/drivers/scsi/aic7xxx/aic7770_osm.c
>> +++ b/drivers/scsi/aic7xxx/aic7770_osm.c
>> @@ -51,7 +51,7 @@ aic7770_map_registers(struct ahc_softc *ahc, u_int port)
>>       * Lock out other contenders for our i/o space.
>>       */
>>      if (!request_region(port, AHC_EISA_IOSIZE, "aic7xxx"))
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      ahc->tag = BUS_SPACE_PIO;
>>      ahc->bsh.ioport = port;
>>      return (0);
>> @@ -87,10 +87,10 @@ aic7770_probe(struct device *dev)
>>      sprintf(buf, "ahc_eisa:%d", eisaBase >> 12);
>>      name = kstrdup(buf, GFP_ATOMIC);
>>      if (name == NULL)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      ahc = ahc_alloc(&aic7xxx_driver_template, name);
>>      if (ahc == NULL)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      error = aic7770_config(ahc, aic7770_ident_table + edev->id.driver_data,
>>                             eisaBase);
>>      if (error != 0) {
>> diff --git a/drivers/scsi/aic7xxx/aic79xx_core.c
>> b/drivers/scsi/aic7xxx/aic79xx_core.c
>> index 109e2c9..bf850d8 100644
>> --- a/drivers/scsi/aic7xxx/aic79xx_core.c
>> +++ b/drivers/scsi/aic7xxx/aic79xx_core.c
>> @@ -6422,7 +6422,7 @@ ahd_init_scbdata(struct ahd_softc *ahd)
>>      scb_data->maxhscbs = ahd_probe_scbs(ahd);
>>      if (scb_data->maxhscbs == 0) {
>>              printk("%s: No SCB space found\n", ahd_name(ahd));
>> -            return (ENXIO);
>> +            return -ENXIO;
>>      }
>>  
>>      ahd_initialize_hscbs(ahd);
>> @@ -6501,7 +6501,7 @@ ahd_init_scbdata(struct ahd_softc *ahd)
>>  
>>  error_exit:
>>  
>> -    return (ENOMEM);
>> +    return -ENOMEM;
>>  }
>>  
>>  static struct scb *
>> @@ -7076,7 +7076,7 @@ ahd_init(struct ahd_softc *ahd)
>>      ahd->stack_size = ahd_probe_stack_size(ahd);
>>      ahd->saved_stack = kmalloc(ahd->stack_size * sizeof(uint16_t), 
>> GFP_ATOMIC);
>>      if (ahd->saved_stack == NULL)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>  
>>      /*
>>       * Verify that the compiler hasn't over-aggressively
>> @@ -7115,7 +7115,7 @@ ahd_init(struct ahd_softc *ahd)
>>                             /*maxsegsz*/AHD_MAXTRANSFER_SIZE,
>>                             /*flags*/BUS_DMA_ALLOCNOW,
>>                             &ahd->buffer_dmat) != 0) {
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      }
>>  #endif
>>  
>> @@ -7143,7 +7143,7 @@ ahd_init(struct ahd_softc *ahd)
>>                             /*nsegments*/1,
>>                             /*maxsegsz*/BUS_SPACE_MAXSIZE_32BIT,
>>                             /*flags*/0, &ahd->shared_data_dmat) != 0) {
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      }
>>  
>>      ahd->init_level++;
>> @@ -7153,7 +7153,7 @@ ahd_init(struct ahd_softc *ahd)
>>                           (void **)&ahd->shared_data_map.vaddr,
>>                           BUS_DMA_NOWAIT,
>>                           &ahd->shared_data_map.dmamap) != 0) {
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      }
>>  
>>      ahd->init_level++;
>> @@ -7194,7 +7194,7 @@ ahd_init(struct ahd_softc *ahd)
>>  
>>      /* Allocate SCB data now that buffer_dmat is initialized */
>>      if (ahd_init_scbdata(ahd) != 0)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>  
>>      if ((ahd->flags & AHD_INITIATORROLE) == 0)
>>              ahd->flags &= ~AHD_RESET_BUS_A;
>> @@ -7644,7 +7644,7 @@ ahd_default_config(struct ahd_softc *ahd)
>>      if (ahd_alloc_tstate(ahd, ahd->our_id, 'A') == NULL) {
>>              printk("%s: unable to allocate ahd_tmode_tstate.  "
>>                     "Failing attach\n", ahd_name(ahd));
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      }
>>  
>>      for (targ = 0; targ < AHD_NUM_TARGETS; targ++) {
>> @@ -7723,7 +7723,7 @@ ahd_parse_cfgdata(struct ahd_softc *ahd, struct
>> seeprom_config *sc)
>>      if (ahd_alloc_tstate(ahd, ahd->our_id, 'A') == NULL) {
>>              printk("%s: unable to allocate ahd_tmode_tstate.  "
>>                     "Failing attach\n", ahd_name(ahd));
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      }
>>  
>>      for (targ = 0; targ < max_targ; targ++) {
>> @@ -7967,7 +7967,7 @@ ahd_suspend(struct ahd_softc *ahd)
>>  
>>      if (LIST_FIRST(&ahd->pending_scbs) != NULL) {
>>              ahd_unpause(ahd);
>> -            return (EBUSY);
>> +            return -EBUSY;
>>      }
>>      ahd_shutdown(ahd);
>>      return (0);
>> @@ -10126,7 +10126,7 @@ ahd_wait_seeprom(struct ahd_softc *ahd)
>>              ahd_delay(5);
>>  
>>      if (cnt == 0)
>> -            return (ETIMEDOUT);
>> +            return -ETIMEDOUT;
>>      return (0);
>>  }
>>  
>> @@ -10227,7 +10227,7 @@ ahd_wait_flexport(struct ahd_softc *ahd)
>>              ahd_delay(5);
>>  
>>      if (cnt == 0)
>> -            return (ETIMEDOUT);
>> +            return -ETIMEDOUT;
>>      return (0);
>>  }
>>  
>> diff --git a/drivers/scsi/aic7xxx/aic79xx_osm.c
>> b/drivers/scsi/aic7xxx/aic79xx_osm.c
>> index 2588b8f..1dc4977 100644
>> --- a/drivers/scsi/aic7xxx/aic79xx_osm.c
>> +++ b/drivers/scsi/aic7xxx/aic79xx_osm.c
>> @@ -940,7 +940,7 @@ ahd_dma_tag_create(struct ahd_softc *ahd, bus_dma_tag_t
>> parent,
>>  
>>      dmat = kmalloc(sizeof(*dmat), GFP_ATOMIC);
>>      if (dmat == NULL)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>  
>>      /*
>>       * Linux is very simplistic about DMA memory.  For now don't
>> @@ -969,7 +969,7 @@ ahd_dmamem_alloc(struct ahd_softc *ahd, bus_dma_tag_t
>> dmat, void** vaddr,
>>      *vaddr = pci_alloc_consistent(ahd->dev_softc,
>>                                    dmat->maxsize, mapp);
>>      if (*vaddr == NULL)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      return(0);
>>  }
>>  
>> @@ -1232,7 +1232,7 @@ ahd_linux_register_host(struct ahd_softc *ahd, struct
>> scsi_host_template *templa
>>      template->name = ahd->description;
>>      host = scsi_host_alloc(template, sizeof(struct ahd_softc *));
>>      if (host == NULL)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>  
>>      *((struct ahd_softc **)host->hostdata) = ahd;
>>      ahd->platform_data->host = host;
>> @@ -1327,7 +1327,7 @@ ahd_platform_alloc(struct ahd_softc *ahd, void
>> *platform_arg)
>>      ahd->platform_data =
>>          kzalloc(sizeof(struct ahd_platform_data), GFP_ATOMIC);
>>      if (ahd->platform_data == NULL)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      ahd->platform_data->irq = AHD_LINUX_NOIRQ;
>>      ahd_lockinit(ahd);
>>      ahd->seltime = (aic79xx_seltime & 0x3) << 4;
>> diff --git a/drivers/scsi/aic7xxx/aic79xx_osm_pci.c
>> b/drivers/scsi/aic7xxx/aic79xx_osm_pci.c
>> index 8466aa7..db58ad9 100644
>> --- a/drivers/scsi/aic7xxx/aic79xx_osm_pci.c
>> +++ b/drivers/scsi/aic7xxx/aic79xx_osm_pci.c
>> @@ -259,12 +259,12 @@ ahd_linux_pci_reserve_io_regions(struct ahd_softc *ahd,
>> resource_size_t *base,
>>       */
>>      *base2 = pci_resource_start(ahd->dev_softc, 3);
>>      if (*base == 0 || *base2 == 0)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      if (!request_region(*base, 256, "aic79xx"))
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      if (!request_region(*base2, 256, "aic79xx")) {
>>              release_region(*base, 256);
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      }
>>      return (0);
>>  }
>> @@ -280,10 +280,10 @@ ahd_linux_pci_reserve_mem_region(struct ahd_softc *ahd,
>>      int     error = 0;
>>  
>>      if (aic79xx_allow_memio == 0)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>  
>>      if ((ahd->bugs & AHD_PCIX_MMAPIO_BUG) != 0)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>  
>>      start = pci_resource_start(ahd->dev_softc, 1);
>>      base_page = start & PAGE_MASK;
>> @@ -291,17 +291,17 @@ ahd_linux_pci_reserve_mem_region(struct ahd_softc *ahd,
>>      if (start != 0) {
>>              *bus_addr = start;
>>              if (!request_mem_region(start, 0x1000, "aic79xx"))
>> -                    error = ENOMEM;
>> +                    error = -ENOMEM;
>>              if (!error) {
>>                      *maddr = ioremap_nocache(base_page, base_offset + 512);
>>                      if (*maddr == NULL) {
>> -                            error = ENOMEM;
>> +                            error = -ENOMEM;
>>                              release_mem_region(start, 0x1000);
>>                      } else
>>                              *maddr += base_offset;
>>              }
>>      } else
>> -            error = ENOMEM;
>> +            error = -ENOMEM;
>>      return (error);
>>  }
>>  
>> diff --git a/drivers/scsi/aic7xxx/aic79xx_pci.c
>> b/drivers/scsi/aic7xxx/aic79xx_pci.c
>> index cc9bd26..d597dc9 100644
>> --- a/drivers/scsi/aic7xxx/aic79xx_pci.c
>> +++ b/drivers/scsi/aic7xxx/aic79xx_pci.c
>> @@ -360,7 +360,7 @@ ahd_pci_config(struct ahd_softc *ahd, const struct
>> ahd_pci_identity *entry)
>>  
>>      error = ahd_reset(ahd, /*reinit*/FALSE);
>>      if (error != 0)
>> -            return (ENXIO);
>> +            return -ENXIO;
>>  
>>      ahd->pci_cachesize =
>>          ahd_pci_read_config(ahd->dev_softc, CSIZE_LATTIME,
>> diff --git a/drivers/scsi/aic7xxx/aic7xxx_core.c
>> b/drivers/scsi/aic7xxx/aic7xxx_core.c
>> index 64ab9ea..a08179d 100644
>> --- a/drivers/scsi/aic7xxx/aic7xxx_core.c
>> +++ b/drivers/scsi/aic7xxx/aic7xxx_core.c
>> @@ -4466,7 +4466,7 @@ ahc_softc_init(struct ahc_softc *ahc)
>>      if (ahc->scb_data == NULL) {
>>              ahc->scb_data = kzalloc(sizeof(*ahc->scb_data), GFP_ATOMIC);
>>              if (ahc->scb_data == NULL)
>> -                    return (ENOMEM);
>> +                    return -ENOMEM;
>>      }
>>  
>>      return (0);
>> @@ -4782,14 +4782,14 @@ ahc_init_scbdata(struct ahc_softc *ahc)
>>      scb_data->scbarray = kzalloc(sizeof(struct scb) * AHC_SCB_MAX_ALLOC,
>>                              GFP_ATOMIC);
>>      if (scb_data->scbarray == NULL)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>  
>>      /* Determine the number of hardware SCBs and initialize them */
>>  
>>      scb_data->maxhscbs = ahc_probe_scbs(ahc);
>>      if (ahc->scb_data->maxhscbs == 0) {
>>              printk("%s: No SCB space found\n", ahc_name(ahc));
>> -            return (ENXIO);
>> +            return -ENXIO;
>>      }
>>  
>>      /*
>> @@ -4904,7 +4904,7 @@ ahc_init_scbdata(struct ahc_softc *ahc)
>>  
>>  error_exit:
>>  
>> -    return (ENOMEM);
>> +    return -ENOMEM;
>>  }
>>  
>>  static void
>> @@ -5339,7 +5339,7 @@ ahc_init(struct ahc_softc *ahc)
>>                             /*maxsegsz*/AHC_MAXTRANSFER_SIZE,
>>                             /*flags*/BUS_DMA_ALLOCNOW,
>>                             &ahc->buffer_dmat) != 0) {
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      }
>>  #endif
>>  
>> @@ -5367,7 +5367,7 @@ ahc_init(struct ahc_softc *ahc)
>>                             /*nsegments*/1,
>>                             /*maxsegsz*/BUS_SPACE_MAXSIZE_32BIT,
>>                             /*flags*/0, &ahc->shared_data_dmat) != 0) {
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      }
>>  
>>      ahc->init_level++;
>> @@ -5376,7 +5376,7 @@ ahc_init(struct ahc_softc *ahc)
>>      if (ahc_dmamem_alloc(ahc, ahc->shared_data_dmat,
>>                           (void **)&ahc->qoutfifo,
>>                           BUS_DMA_NOWAIT, &ahc->shared_data_dmamap) != 0) {
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      }
>>  
>>      ahc->init_level++;
>> @@ -5404,7 +5404,7 @@ ahc_init(struct ahc_softc *ahc)
>>      /* Allocate SCB data now that buffer_dmat is initialized */
>>      if (ahc->scb_data->maxhscbs == 0)
>>              if (ahc_init_scbdata(ahc) != 0)
>> -                    return (ENOMEM);
>> +                    return -ENOMEM;
>>  
>>      /*
>>       * Allocate a tstate to house information for our
>> @@ -5414,14 +5414,14 @@ ahc_init(struct ahc_softc *ahc)
>>      if (ahc_alloc_tstate(ahc, ahc->our_id, 'A') == NULL) {
>>              printk("%s: unable to allocate ahc_tmode_tstate.  "
>>                     "Failing attach\n", ahc_name(ahc));
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      }
>>  
>>      if ((ahc->features & AHC_TWIN) != 0) {
>>              if (ahc_alloc_tstate(ahc, ahc->our_id_b, 'B') == NULL) {
>>                      printk("%s: unable to allocate ahc_tmode_tstate.  "
>>                             "Failing attach\n", ahc_name(ahc));
>> -                    return (ENOMEM);
>> +                    return -ENOMEM;
>>              }
>>      }
>>  
>> @@ -5660,7 +5660,7 @@ ahc_suspend(struct ahc_softc *ahc)
>>  
>>      if (LIST_FIRST(&ahc->pending_scbs) != NULL) {
>>              ahc_unpause(ahc);
>> -            return (EBUSY);
>> +            return -EBUSY;
>>      }
>>  
>>  #ifdef AHC_TARGET_MODE
>> @@ -5671,7 +5671,7 @@ ahc_suspend(struct ahc_softc *ahc)
>>       */
>>      if (ahc->pending_device != NULL) {
>>              ahc_unpause(ahc);
>> -            return (EBUSY);
>> +            return -EBUSY;
>>      }
>>  #endif
>>      ahc_shutdown(ahc);
>> @@ -6908,7 +6908,7 @@ ahc_loadseq(struct ahc_softc *ahc)
>>                      printk("\n%s: Program too large for instruction memory "
>>                             "size of %d!\n", ahc_name(ahc),
>>                             ahc->instruction_ram_size);
>> -                    return (ENOMEM);
>> +                    return -ENOMEM;
>>              }
>>  
>>              /*
>> diff --git a/drivers/scsi/aic7xxx/aic7xxx_osm.c
>> b/drivers/scsi/aic7xxx/aic7xxx_osm.c
>> index fc6a831..78433f6 100644
>> --- a/drivers/scsi/aic7xxx/aic7xxx_osm.c
>> +++ b/drivers/scsi/aic7xxx/aic7xxx_osm.c
>> @@ -835,7 +835,7 @@ ahc_dma_tag_create(struct ahc_softc *ahc, bus_dma_tag_t
>> parent,
>>  
>>      dmat = kmalloc(sizeof(*dmat), GFP_ATOMIC);
>>      if (dmat == NULL)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>  
>>      /*
>>       * Linux is very simplistic about DMA memory.  For now don't
>> @@ -864,7 +864,7 @@ ahc_dmamem_alloc(struct ahc_softc *ahc, bus_dma_tag_t
>> dmat, void** vaddr,
>>      *vaddr = pci_alloc_consistent(ahc->dev_softc,
>>                                    dmat->maxsize, mapp);
>>      if (*vaddr == NULL)
>> -            return ENOMEM;
>> +            return -ENOMEM;
>>      return 0;
>>  }
>>  
>> @@ -1096,7 +1096,7 @@ ahc_linux_register_host(struct ahc_softc *ahc, struct
>> scsi_host_template *templa
>>      template->name = ahc->description;
>>      host = scsi_host_alloc(template, sizeof(struct ahc_softc *));
>>      if (host == NULL)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>  
>>      *((struct ahc_softc **)host->hostdata) = ahc;
>>      ahc->platform_data->host = host;
>> @@ -1215,7 +1215,7 @@ ahc_platform_alloc(struct ahc_softc *ahc, void
>> *platform_arg)
>>      ahc->platform_data =
>>          kzalloc(sizeof(struct ahc_platform_data), GFP_ATOMIC);
>>      if (ahc->platform_data == NULL)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      ahc->platform_data->irq = AHC_LINUX_NOIRQ;
>>      ahc_lockinit(ahc);
>>      ahc->seltime = (aic7xxx_seltime & 0x3) << 4;
>> diff --git a/drivers/scsi/aic7xxx/aic7xxx_osm_pci.c
>> b/drivers/scsi/aic7xxx/aic7xxx_osm_pci.c
>> index 0fc14da..8bca7f4 100644
>> --- a/drivers/scsi/aic7xxx/aic7xxx_osm_pci.c
>> +++ b/drivers/scsi/aic7xxx/aic7xxx_osm_pci.c
>> @@ -346,13 +346,13 @@ static int
>>  ahc_linux_pci_reserve_io_region(struct ahc_softc *ahc, resource_size_t
>>  *base)
>>  {
>>      if (aic7xxx_allow_memio == 0)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>  
>>      *base = pci_resource_start(ahc->dev_softc, 0);
>>      if (*base == 0)
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      if (!request_region(*base, 256, "aic7xxx"))
>> -            return (ENOMEM);
>> +            return -ENOMEM;
>>      return (0);
>>  }
>>  
>> @@ -369,16 +369,16 @@ ahc_linux_pci_reserve_mem_region(struct ahc_softc *ahc,
>>      if (start != 0) {
>>              *bus_addr = start;
>>              if (!request_mem_region(start, 0x1000, "aic7xxx"))
>> -                    error = ENOMEM;
>> +                    error = -ENOMEM;
>>              if (error == 0) {
>>                      *maddr = ioremap_nocache(start, 256);
>>                      if (*maddr == NULL) {
>> -                            error = ENOMEM;
>> +                            error = -ENOMEM;
>>                              release_mem_region(start, 0x1000);
>>                      }
>>              }
>>      } else
>> -            error = ENOMEM;
>> +            error = -ENOMEM;
>>      return (error);
>>  }
>>  
>> diff --git a/drivers/scsi/aic7xxx/aic7xxx_pci.c
>> b/drivers/scsi/aic7xxx/aic7xxx_pci.c
>> index 22d5a94..40e1c9b 100644
>> --- a/drivers/scsi/aic7xxx/aic7xxx_pci.c
>> +++ b/drivers/scsi/aic7xxx/aic7xxx_pci.c
>> @@ -806,7 +806,7 @@ ahc_pci_config(struct ahc_softc *ahc, const struct
>> ahc_pci_identity *entry)
>>  
>>      error = ahc_reset(ahc, /*reinit*/FALSE);
>>      if (error != 0)
>> -            return (ENXIO);
>> +            return -ENXIO;
>>  
>>      if ((ahc->features & AHC_DT) != 0) {
>>              u_int sfunct;
>> @@ -2387,7 +2387,7 @@ static int
>>  ahc_raid_setup(struct ahc_softc *ahc)
>>  {
>>      printk("RAID functionality unsupported\n");
>> -    return (ENXIO);
>> +    return -ENXIO;
>>  }
>>  
>>  static int
>> --
>> 2.5.1
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
>> the body of a message to majord...@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
> 
> Patch looks simple as the change is straightforward.
> However can you make the code consistent, some have parenthesis in return, 
> some not.

I only changed lines that contain positive error returns. These files are 
already inconsistent
with returning with or without parenthesis. I can resend this patch with a 
second one that fixes
those inconsistencies, if the maintainer would accept that.

> How did this work before though if it was returning non-negative to the 
> caller or upper layer
> Has this been tested to work with the changes
> 
> Reviewed-by Laurence Oberman <lober...@redhat.com>
> 

This worked because all returned value checks are the usual
if (function_that_might_err())
        goto error;

So any return that isn't 0 is treated as an error.

Thanks for the review,
Luis
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to