diff options
38 files changed, 102 insertions, 103 deletions
diff --git a/drivers/acpi/acpi_dbg.c b/drivers/acpi/acpi_dbg.c index 15e4604efba7..1f4128487dd4 100644 --- a/drivers/acpi/acpi_dbg.c +++ b/drivers/acpi/acpi_dbg.c | |||
| @@ -265,7 +265,7 @@ static int acpi_aml_write_kern(const char *buf, int len) | |||
| 265 | char *p; | 265 | char *p; |
| 266 | 266 | ||
| 267 | ret = acpi_aml_lock_write(crc, ACPI_AML_OUT_KERN); | 267 | ret = acpi_aml_lock_write(crc, ACPI_AML_OUT_KERN); |
| 268 | if (IS_ERR_VALUE(ret)) | 268 | if (ret < 0) |
| 269 | return ret; | 269 | return ret; |
| 270 | /* sync tail before inserting logs */ | 270 | /* sync tail before inserting logs */ |
| 271 | smp_mb(); | 271 | smp_mb(); |
| @@ -286,7 +286,7 @@ static int acpi_aml_readb_kern(void) | |||
| 286 | char *p; | 286 | char *p; |
| 287 | 287 | ||
| 288 | ret = acpi_aml_lock_read(crc, ACPI_AML_IN_KERN); | 288 | ret = acpi_aml_lock_read(crc, ACPI_AML_IN_KERN); |
| 289 | if (IS_ERR_VALUE(ret)) | 289 | if (ret < 0) |
| 290 | return ret; | 290 | return ret; |
| 291 | /* sync head before removing cmds */ | 291 | /* sync head before removing cmds */ |
| 292 | smp_rmb(); | 292 | smp_rmb(); |
| @@ -330,7 +330,7 @@ again: | |||
| 330 | goto again; | 330 | goto again; |
| 331 | break; | 331 | break; |
| 332 | } | 332 | } |
| 333 | if (IS_ERR_VALUE(ret)) | 333 | if (ret < 0) |
| 334 | break; | 334 | break; |
| 335 | size += ret; | 335 | size += ret; |
| 336 | count -= ret; | 336 | count -= ret; |
| @@ -373,7 +373,7 @@ again: | |||
| 373 | if (ret == 0) | 373 | if (ret == 0) |
| 374 | goto again; | 374 | goto again; |
| 375 | } | 375 | } |
| 376 | if (IS_ERR_VALUE(ret)) | 376 | if (ret < 0) |
| 377 | break; | 377 | break; |
| 378 | *(msg + size) = (char)ret; | 378 | *(msg + size) = (char)ret; |
| 379 | size++; | 379 | size++; |
| @@ -526,7 +526,7 @@ static int acpi_aml_open(struct inode *inode, struct file *file) | |||
| 526 | } | 526 | } |
| 527 | acpi_aml_io.users++; | 527 | acpi_aml_io.users++; |
| 528 | err_lock: | 528 | err_lock: |
| 529 | if (IS_ERR_VALUE(ret)) { | 529 | if (ret < 0) { |
| 530 | if (acpi_aml_active_reader == file) | 530 | if (acpi_aml_active_reader == file) |
| 531 | acpi_aml_active_reader = NULL; | 531 | acpi_aml_active_reader = NULL; |
| 532 | } | 532 | } |
| @@ -587,7 +587,7 @@ static int acpi_aml_read_user(char __user *buf, int len) | |||
| 587 | char *p; | 587 | char *p; |
| 588 | 588 | ||
| 589 | ret = acpi_aml_lock_read(crc, ACPI_AML_OUT_USER); | 589 | ret = acpi_aml_lock_read(crc, ACPI_AML_OUT_USER); |
| 590 | if (IS_ERR_VALUE(ret)) | 590 | if (ret < 0) |
| 591 | return ret; | 591 | return ret; |
| 592 | /* sync head before removing logs */ | 592 | /* sync head before removing logs */ |
| 593 | smp_rmb(); | 593 | smp_rmb(); |
| @@ -602,7 +602,7 @@ static int acpi_aml_read_user(char __user *buf, int len) | |||
| 602 | crc->tail = (crc->tail + n) & (ACPI_AML_BUF_SIZE - 1); | 602 | crc->tail = (crc->tail + n) & (ACPI_AML_BUF_SIZE - 1); |
| 603 | ret = n; | 603 | ret = n; |
| 604 | out: | 604 | out: |
| 605 | acpi_aml_unlock_fifo(ACPI_AML_OUT_USER, !IS_ERR_VALUE(ret)); | 605 | acpi_aml_unlock_fifo(ACPI_AML_OUT_USER, !ret); |
| 606 | return ret; | 606 | return ret; |
| 607 | } | 607 | } |
| 608 | 608 | ||
| @@ -634,7 +634,7 @@ again: | |||
| 634 | goto again; | 634 | goto again; |
| 635 | } | 635 | } |
| 636 | } | 636 | } |
| 637 | if (IS_ERR_VALUE(ret)) { | 637 | if (ret < 0) { |
| 638 | if (!acpi_aml_running()) | 638 | if (!acpi_aml_running()) |
| 639 | ret = 0; | 639 | ret = 0; |
| 640 | break; | 640 | break; |
| @@ -657,7 +657,7 @@ static int acpi_aml_write_user(const char __user *buf, int len) | |||
| 657 | char *p; | 657 | char *p; |
| 658 | 658 | ||
| 659 | ret = acpi_aml_lock_write(crc, ACPI_AML_IN_USER); | 659 | ret = acpi_aml_lock_write(crc, ACPI_AML_IN_USER); |
| 660 | if (IS_ERR_VALUE(ret)) | 660 | if (ret < 0) |
| 661 | return ret; | 661 | return ret; |
| 662 | /* sync tail before inserting cmds */ | 662 | /* sync tail before inserting cmds */ |
| 663 | smp_mb(); | 663 | smp_mb(); |
| @@ -672,7 +672,7 @@ static int acpi_aml_write_user(const char __user *buf, int len) | |||
| 672 | crc->head = (crc->head + n) & (ACPI_AML_BUF_SIZE - 1); | 672 | crc->head = (crc->head + n) & (ACPI_AML_BUF_SIZE - 1); |
| 673 | ret = n; | 673 | ret = n; |
| 674 | out: | 674 | out: |
| 675 | acpi_aml_unlock_fifo(ACPI_AML_IN_USER, !IS_ERR_VALUE(ret)); | 675 | acpi_aml_unlock_fifo(ACPI_AML_IN_USER, !ret); |
| 676 | return n; | 676 | return n; |
| 677 | } | 677 | } |
| 678 | 678 | ||
| @@ -704,7 +704,7 @@ again: | |||
| 704 | goto again; | 704 | goto again; |
| 705 | } | 705 | } |
| 706 | } | 706 | } |
| 707 | if (IS_ERR_VALUE(ret)) { | 707 | if (ret < 0) { |
| 708 | if (!acpi_aml_running()) | 708 | if (!acpi_aml_running()) |
| 709 | ret = 0; | 709 | ret = 0; |
| 710 | break; | 710 | break; |
diff --git a/drivers/ata/sata_highbank.c b/drivers/ata/sata_highbank.c index 8638d575b2b9..aafb8cc03523 100644 --- a/drivers/ata/sata_highbank.c +++ b/drivers/ata/sata_highbank.c | |||
| @@ -197,7 +197,7 @@ static void highbank_set_em_messages(struct device *dev, | |||
| 197 | 197 | ||
| 198 | for (i = 0; i < SGPIO_PINS; i++) { | 198 | for (i = 0; i < SGPIO_PINS; i++) { |
| 199 | err = of_get_named_gpio(np, "calxeda,sgpio-gpio", i); | 199 | err = of_get_named_gpio(np, "calxeda,sgpio-gpio", i); |
| 200 | if (IS_ERR_VALUE(err)) | 200 | if (err < 0) |
| 201 | return; | 201 | return; |
| 202 | 202 | ||
| 203 | pdata->sgpio_gpio[i] = err; | 203 | pdata->sgpio_gpio[i] = err; |
diff --git a/drivers/clk/tegra/clk-tegra210.c b/drivers/clk/tegra/clk-tegra210.c index b8551813ec43..456cf586d2c2 100644 --- a/drivers/clk/tegra/clk-tegra210.c +++ b/drivers/clk/tegra/clk-tegra210.c | |||
| @@ -1221,7 +1221,7 @@ static int tegra210_pll_fixed_mdiv_cfg(struct clk_hw *hw, | |||
| 1221 | p = rate >= params->vco_min ? 1 : -EINVAL; | 1221 | p = rate >= params->vco_min ? 1 : -EINVAL; |
| 1222 | } | 1222 | } |
| 1223 | 1223 | ||
| 1224 | if (IS_ERR_VALUE(p)) | 1224 | if (p < 0) |
| 1225 | return -EINVAL; | 1225 | return -EINVAL; |
| 1226 | 1226 | ||
| 1227 | cfg->m = tegra_pll_get_fixed_mdiv(hw, input_rate); | 1227 | cfg->m = tegra_pll_get_fixed_mdiv(hw, input_rate); |
diff --git a/drivers/cpufreq/omap-cpufreq.c b/drivers/cpufreq/omap-cpufreq.c index cead9bec4843..376e63ca94e8 100644 --- a/drivers/cpufreq/omap-cpufreq.c +++ b/drivers/cpufreq/omap-cpufreq.c | |||
| @@ -54,7 +54,7 @@ static int omap_target(struct cpufreq_policy *policy, unsigned int index) | |||
| 54 | 54 | ||
| 55 | freq = new_freq * 1000; | 55 | freq = new_freq * 1000; |
| 56 | ret = clk_round_rate(policy->clk, freq); | 56 | ret = clk_round_rate(policy->clk, freq); |
| 57 | if (IS_ERR_VALUE(ret)) { | 57 | if (ret < 0) { |
| 58 | dev_warn(mpu_dev, | 58 | dev_warn(mpu_dev, |
| 59 | "CPUfreq: Cannot find matching frequency for %lu\n", | 59 | "CPUfreq: Cannot find matching frequency for %lu\n", |
| 60 | freq); | 60 | freq); |
diff --git a/drivers/crypto/caam/ctrl.c b/drivers/crypto/caam/ctrl.c index 44d30b45f3cc..5ad5f3009ae0 100644 --- a/drivers/crypto/caam/ctrl.c +++ b/drivers/crypto/caam/ctrl.c | |||
| @@ -402,7 +402,7 @@ int caam_get_era(void) | |||
| 402 | ret = of_property_read_u32(caam_node, "fsl,sec-era", &prop); | 402 | ret = of_property_read_u32(caam_node, "fsl,sec-era", &prop); |
| 403 | of_node_put(caam_node); | 403 | of_node_put(caam_node); |
| 404 | 404 | ||
| 405 | return IS_ERR_VALUE(ret) ? -ENOTSUPP : prop; | 405 | return ret ? -ENOTSUPP : prop; |
| 406 | } | 406 | } |
| 407 | EXPORT_SYMBOL(caam_get_era); | 407 | EXPORT_SYMBOL(caam_get_era); |
| 408 | 408 | ||
diff --git a/drivers/dma/sun4i-dma.c b/drivers/dma/sun4i-dma.c index e0df233dde92..57aa227bfadb 100644 --- a/drivers/dma/sun4i-dma.c +++ b/drivers/dma/sun4i-dma.c | |||
| @@ -461,25 +461,25 @@ generate_ndma_promise(struct dma_chan *chan, dma_addr_t src, dma_addr_t dest, | |||
| 461 | 461 | ||
| 462 | /* Source burst */ | 462 | /* Source burst */ |
| 463 | ret = convert_burst(sconfig->src_maxburst); | 463 | ret = convert_burst(sconfig->src_maxburst); |
| 464 | if (IS_ERR_VALUE(ret)) | 464 | if (ret < 0) |
| 465 | goto fail; | 465 | goto fail; |
| 466 | promise->cfg |= SUN4I_DMA_CFG_SRC_BURST_LENGTH(ret); | 466 | promise->cfg |= SUN4I_DMA_CFG_SRC_BURST_LENGTH(ret); |
| 467 | 467 | ||
| 468 | /* Destination burst */ | 468 | /* Destination burst */ |
| 469 | ret = convert_burst(sconfig->dst_maxburst); | 469 | ret = convert_burst(sconfig->dst_maxburst); |
| 470 | if (IS_ERR_VALUE(ret)) | 470 | if (ret < 0) |
| 471 | goto fail; | 471 | goto fail; |
| 472 | promise->cfg |= SUN4I_DMA_CFG_DST_BURST_LENGTH(ret); | 472 | promise->cfg |= SUN4I_DMA_CFG_DST_BURST_LENGTH(ret); |
| 473 | 473 | ||
| 474 | /* Source bus width */ | 474 | /* Source bus width */ |
| 475 | ret = convert_buswidth(sconfig->src_addr_width); | 475 | ret = convert_buswidth(sconfig->src_addr_width); |
| 476 | if (IS_ERR_VALUE(ret)) | 476 | if (ret < 0) |
| 477 | goto fail; | 477 | goto fail; |
| 478 | promise->cfg |= SUN4I_DMA_CFG_SRC_DATA_WIDTH(ret); | 478 | promise->cfg |= SUN4I_DMA_CFG_SRC_DATA_WIDTH(ret); |
| 479 | 479 | ||
| 480 | /* Destination bus width */ | 480 | /* Destination bus width */ |
| 481 | ret = convert_buswidth(sconfig->dst_addr_width); | 481 | ret = convert_buswidth(sconfig->dst_addr_width); |
| 482 | if (IS_ERR_VALUE(ret)) | 482 | if (ret < 0) |
| 483 | goto fail; | 483 | goto fail; |
| 484 | promise->cfg |= SUN4I_DMA_CFG_DST_DATA_WIDTH(ret); | 484 | promise->cfg |= SUN4I_DMA_CFG_DST_DATA_WIDTH(ret); |
| 485 | 485 | ||
| @@ -518,25 +518,25 @@ generate_ddma_promise(struct dma_chan *chan, dma_addr_t src, dma_addr_t dest, | |||
| 518 | 518 | ||
| 519 | /* Source burst */ | 519 | /* Source burst */ |
| 520 | ret = convert_burst(sconfig->src_maxburst); | 520 | ret = convert_burst(sconfig->src_maxburst); |
| 521 | if (IS_ERR_VALUE(ret)) | 521 | if (ret < 0) |
| 522 | goto fail; | 522 | goto fail; |
| 523 | promise->cfg |= SUN4I_DMA_CFG_SRC_BURST_LENGTH(ret); | 523 | promise->cfg |= SUN4I_DMA_CFG_SRC_BURST_LENGTH(ret); |
| 524 | 524 | ||
| 525 | /* Destination burst */ | 525 | /* Destination burst */ |
| 526 | ret = convert_burst(sconfig->dst_maxburst); | 526 | ret = convert_burst(sconfig->dst_maxburst); |
| 527 | if (IS_ERR_VALUE(ret)) | 527 | if (ret < 0) |
| 528 | goto fail; | 528 | goto fail; |
| 529 | promise->cfg |= SUN4I_DMA_CFG_DST_BURST_LENGTH(ret); | 529 | promise->cfg |= SUN4I_DMA_CFG_DST_BURST_LENGTH(ret); |
| 530 | 530 | ||
| 531 | /* Source bus width */ | 531 | /* Source bus width */ |
| 532 | ret = convert_buswidth(sconfig->src_addr_width); | 532 | ret = convert_buswidth(sconfig->src_addr_width); |
| 533 | if (IS_ERR_VALUE(ret)) | 533 | if (ret < 0) |
| 534 | goto fail; | 534 | goto fail; |
| 535 | promise->cfg |= SUN4I_DMA_CFG_SRC_DATA_WIDTH(ret); | 535 | promise->cfg |= SUN4I_DMA_CFG_SRC_DATA_WIDTH(ret); |
| 536 | 536 | ||
| 537 | /* Destination bus width */ | 537 | /* Destination bus width */ |
| 538 | ret = convert_buswidth(sconfig->dst_addr_width); | 538 | ret = convert_buswidth(sconfig->dst_addr_width); |
| 539 | if (IS_ERR_VALUE(ret)) | 539 | if (ret < 0) |
| 540 | goto fail; | 540 | goto fail; |
| 541 | promise->cfg |= SUN4I_DMA_CFG_DST_DATA_WIDTH(ret); | 541 | promise->cfg |= SUN4I_DMA_CFG_DST_DATA_WIDTH(ret); |
| 542 | 542 | ||
diff --git a/drivers/gpio/gpio-xlp.c b/drivers/gpio/gpio-xlp.c index 08897dc11915..1a33a19d95b9 100644 --- a/drivers/gpio/gpio-xlp.c +++ b/drivers/gpio/gpio-xlp.c | |||
| @@ -393,7 +393,7 @@ static int xlp_gpio_probe(struct platform_device *pdev) | |||
| 393 | irq_base = irq_alloc_descs(-1, 0, gc->ngpio, 0); | 393 | irq_base = irq_alloc_descs(-1, 0, gc->ngpio, 0); |
| 394 | else | 394 | else |
| 395 | irq_base = irq_alloc_descs(-1, XLP_GPIO_IRQ_BASE, gc->ngpio, 0); | 395 | irq_base = irq_alloc_descs(-1, XLP_GPIO_IRQ_BASE, gc->ngpio, 0); |
| 396 | if (IS_ERR_VALUE(irq_base)) { | 396 | if (irq_base < 0) { |
| 397 | dev_err(&pdev->dev, "Failed to allocate IRQ numbers\n"); | 397 | dev_err(&pdev->dev, "Failed to allocate IRQ numbers\n"); |
| 398 | return irq_base; | 398 | return irq_base; |
| 399 | } | 399 | } |
diff --git a/drivers/gpu/drm/sti/sti_vtg.c b/drivers/gpu/drm/sti/sti_vtg.c index 32c7986b63ab..6bf4ce466d20 100644 --- a/drivers/gpu/drm/sti/sti_vtg.c +++ b/drivers/gpu/drm/sti/sti_vtg.c | |||
| @@ -437,7 +437,7 @@ static int vtg_probe(struct platform_device *pdev) | |||
| 437 | return -EPROBE_DEFER; | 437 | return -EPROBE_DEFER; |
| 438 | } else { | 438 | } else { |
| 439 | vtg->irq = platform_get_irq(pdev, 0); | 439 | vtg->irq = platform_get_irq(pdev, 0); |
| 440 | if (IS_ERR_VALUE(vtg->irq)) { | 440 | if (vtg->irq < 0) { |
| 441 | DRM_ERROR("Failed to get VTG interrupt\n"); | 441 | DRM_ERROR("Failed to get VTG interrupt\n"); |
| 442 | return vtg->irq; | 442 | return vtg->irq; |
| 443 | } | 443 | } |
| @@ -447,7 +447,7 @@ static int vtg_probe(struct platform_device *pdev) | |||
| 447 | ret = devm_request_threaded_irq(dev, vtg->irq, vtg_irq, | 447 | ret = devm_request_threaded_irq(dev, vtg->irq, vtg_irq, |
| 448 | vtg_irq_thread, IRQF_ONESHOT, | 448 | vtg_irq_thread, IRQF_ONESHOT, |
| 449 | dev_name(dev), vtg); | 449 | dev_name(dev), vtg); |
| 450 | if (IS_ERR_VALUE(ret)) { | 450 | if (ret < 0) { |
| 451 | DRM_ERROR("Failed to register VTG interrupt\n"); | 451 | DRM_ERROR("Failed to register VTG interrupt\n"); |
| 452 | return ret; | 452 | return ret; |
| 453 | } | 453 | } |
diff --git a/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c b/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c index 7716f42f8aab..6b8c5b3bf588 100644 --- a/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c +++ b/drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | |||
| @@ -342,7 +342,7 @@ static int tfp410_probe(struct platform_device *pdev) | |||
| 342 | 342 | ||
| 343 | tfp410_mod->gpio = of_get_named_gpio_flags(node, "powerdn-gpio", | 343 | tfp410_mod->gpio = of_get_named_gpio_flags(node, "powerdn-gpio", |
| 344 | 0, NULL); | 344 | 0, NULL); |
| 345 | if (IS_ERR_VALUE(tfp410_mod->gpio)) { | 345 | if (tfp410_mod->gpio < 0) { |
| 346 | dev_warn(&pdev->dev, "No power down GPIO\n"); | 346 | dev_warn(&pdev->dev, "No power down GPIO\n"); |
| 347 | } else { | 347 | } else { |
| 348 | ret = gpio_request(tfp410_mod->gpio, "DVI_PDn"); | 348 | ret = gpio_request(tfp410_mod->gpio, "DVI_PDn"); |
diff --git a/drivers/gpu/host1x/hw/intr_hw.c b/drivers/gpu/host1x/hw/intr_hw.c index 498b37e39058..e1e31e9e67cd 100644 --- a/drivers/gpu/host1x/hw/intr_hw.c +++ b/drivers/gpu/host1x/hw/intr_hw.c | |||
| @@ -85,7 +85,7 @@ static int _host1x_intr_init_host_sync(struct host1x *host, u32 cpm, | |||
| 85 | err = devm_request_irq(host->dev, host->intr_syncpt_irq, | 85 | err = devm_request_irq(host->dev, host->intr_syncpt_irq, |
| 86 | syncpt_thresh_isr, IRQF_SHARED, | 86 | syncpt_thresh_isr, IRQF_SHARED, |
| 87 | "host1x_syncpt", host); | 87 | "host1x_syncpt", host); |
| 88 | if (IS_ERR_VALUE(err)) { | 88 | if (err < 0) { |
| 89 | WARN_ON(1); | 89 | WARN_ON(1); |
| 90 | return err; | 90 | return err; |
| 91 | } | 91 | } |
diff --git a/drivers/iommu/arm-smmu-v3.c b/drivers/iommu/arm-smmu-v3.c index ebab33e77d67..94b68213c50d 100644 --- a/drivers/iommu/arm-smmu-v3.c +++ b/drivers/iommu/arm-smmu-v3.c | |||
| @@ -1477,7 +1477,7 @@ static int arm_smmu_domain_finalise_s1(struct arm_smmu_domain *smmu_domain, | |||
| 1477 | struct arm_smmu_s1_cfg *cfg = &smmu_domain->s1_cfg; | 1477 | struct arm_smmu_s1_cfg *cfg = &smmu_domain->s1_cfg; |
| 1478 | 1478 | ||
| 1479 | asid = arm_smmu_bitmap_alloc(smmu->asid_map, smmu->asid_bits); | 1479 | asid = arm_smmu_bitmap_alloc(smmu->asid_map, smmu->asid_bits); |
| 1480 | if (IS_ERR_VALUE(asid)) | 1480 | if (asid < 0) |
| 1481 | return asid; | 1481 | return asid; |
| 1482 | 1482 | ||
| 1483 | cfg->cdptr = dmam_alloc_coherent(smmu->dev, CTXDESC_CD_DWORDS << 3, | 1483 | cfg->cdptr = dmam_alloc_coherent(smmu->dev, CTXDESC_CD_DWORDS << 3, |
| @@ -1508,7 +1508,7 @@ static int arm_smmu_domain_finalise_s2(struct arm_smmu_domain *smmu_domain, | |||
| 1508 | struct arm_smmu_s2_cfg *cfg = &smmu_domain->s2_cfg; | 1508 | struct arm_smmu_s2_cfg *cfg = &smmu_domain->s2_cfg; |
| 1509 | 1509 | ||
| 1510 | vmid = arm_smmu_bitmap_alloc(smmu->vmid_map, smmu->vmid_bits); | 1510 | vmid = arm_smmu_bitmap_alloc(smmu->vmid_map, smmu->vmid_bits); |
| 1511 | if (IS_ERR_VALUE(vmid)) | 1511 | if (vmid < 0) |
| 1512 | return vmid; | 1512 | return vmid; |
| 1513 | 1513 | ||
| 1514 | cfg->vmid = (u16)vmid; | 1514 | cfg->vmid = (u16)vmid; |
| @@ -1569,7 +1569,7 @@ static int arm_smmu_domain_finalise(struct iommu_domain *domain) | |||
| 1569 | smmu_domain->pgtbl_ops = pgtbl_ops; | 1569 | smmu_domain->pgtbl_ops = pgtbl_ops; |
| 1570 | 1570 | ||
| 1571 | ret = finalise_stage_fn(smmu_domain, &pgtbl_cfg); | 1571 | ret = finalise_stage_fn(smmu_domain, &pgtbl_cfg); |
| 1572 | if (IS_ERR_VALUE(ret)) | 1572 | if (ret < 0) |
| 1573 | free_io_pgtable_ops(pgtbl_ops); | 1573 | free_io_pgtable_ops(pgtbl_ops); |
| 1574 | 1574 | ||
| 1575 | return ret; | 1575 | return ret; |
| @@ -1642,7 +1642,7 @@ static void arm_smmu_detach_dev(struct device *dev) | |||
| 1642 | struct arm_smmu_group *smmu_group = arm_smmu_group_get(dev); | 1642 | struct arm_smmu_group *smmu_group = arm_smmu_group_get(dev); |
| 1643 | 1643 | ||
| 1644 | smmu_group->ste.bypass = true; | 1644 | smmu_group->ste.bypass = true; |
| 1645 | if (IS_ERR_VALUE(arm_smmu_install_ste_for_group(smmu_group))) | 1645 | if (arm_smmu_install_ste_for_group(smmu_group) < 0) |
| 1646 | dev_warn(dev, "failed to install bypass STE\n"); | 1646 | dev_warn(dev, "failed to install bypass STE\n"); |
| 1647 | 1647 | ||
| 1648 | smmu_group->domain = NULL; | 1648 | smmu_group->domain = NULL; |
| @@ -1694,7 +1694,7 @@ static int arm_smmu_attach_dev(struct iommu_domain *domain, struct device *dev) | |||
| 1694 | smmu_group->ste.bypass = domain->type == IOMMU_DOMAIN_DMA; | 1694 | smmu_group->ste.bypass = domain->type == IOMMU_DOMAIN_DMA; |
| 1695 | 1695 | ||
| 1696 | ret = arm_smmu_install_ste_for_group(smmu_group); | 1696 | ret = arm_smmu_install_ste_for_group(smmu_group); |
| 1697 | if (IS_ERR_VALUE(ret)) | 1697 | if (ret < 0) |
| 1698 | smmu_group->domain = NULL; | 1698 | smmu_group->domain = NULL; |
| 1699 | 1699 | ||
| 1700 | out_unlock: | 1700 | out_unlock: |
| @@ -2235,7 +2235,7 @@ static int arm_smmu_setup_irqs(struct arm_smmu_device *smmu) | |||
| 2235 | arm_smmu_evtq_handler, | 2235 | arm_smmu_evtq_handler, |
| 2236 | arm_smmu_evtq_thread, | 2236 | arm_smmu_evtq_thread, |
| 2237 | 0, "arm-smmu-v3-evtq", smmu); | 2237 | 0, "arm-smmu-v3-evtq", smmu); |
| 2238 | if (IS_ERR_VALUE(ret)) | 2238 | if (ret < 0) |
| 2239 | dev_warn(smmu->dev, "failed to enable evtq irq\n"); | 2239 | dev_warn(smmu->dev, "failed to enable evtq irq\n"); |
| 2240 | } | 2240 | } |
| 2241 | 2241 | ||
| @@ -2244,7 +2244,7 @@ static int arm_smmu_setup_irqs(struct arm_smmu_device *smmu) | |||
| 2244 | ret = devm_request_irq(smmu->dev, irq, | 2244 | ret = devm_request_irq(smmu->dev, irq, |
| 2245 | arm_smmu_cmdq_sync_handler, 0, | 2245 | arm_smmu_cmdq_sync_handler, 0, |
| 2246 | "arm-smmu-v3-cmdq-sync", smmu); | 2246 | "arm-smmu-v3-cmdq-sync", smmu); |
| 2247 | if (IS_ERR_VALUE(ret)) | 2247 | if (ret < 0) |
| 2248 | dev_warn(smmu->dev, "failed to enable cmdq-sync irq\n"); | 2248 | dev_warn(smmu->dev, "failed to enable cmdq-sync irq\n"); |
| 2249 | } | 2249 | } |
| 2250 | 2250 | ||
| @@ -2252,7 +2252,7 @@ static int arm_smmu_setup_irqs(struct arm_smmu_device *smmu) | |||
| 2252 | if (irq) { | 2252 | if (irq) { |
| 2253 | ret = devm_request_irq(smmu->dev, irq, arm_smmu_gerror_handler, | 2253 | ret = devm_request_irq(smmu->dev, irq, arm_smmu_gerror_handler, |
| 2254 | 0, "arm-smmu-v3-gerror", smmu); | 2254 | 0, "arm-smmu-v3-gerror", smmu); |
| 2255 | if (IS_ERR_VALUE(ret)) | 2255 | if (ret < 0) |
| 2256 | dev_warn(smmu->dev, "failed to enable gerror irq\n"); | 2256 | dev_warn(smmu->dev, "failed to enable gerror irq\n"); |
| 2257 | } | 2257 | } |
| 2258 | 2258 | ||
| @@ -2264,7 +2264,7 @@ static int arm_smmu_setup_irqs(struct arm_smmu_device *smmu) | |||
| 2264 | arm_smmu_priq_thread, | 2264 | arm_smmu_priq_thread, |
| 2265 | 0, "arm-smmu-v3-priq", | 2265 | 0, "arm-smmu-v3-priq", |
| 2266 | smmu); | 2266 | smmu); |
| 2267 | if (IS_ERR_VALUE(ret)) | 2267 | if (ret < 0) |
| 2268 | dev_warn(smmu->dev, | 2268 | dev_warn(smmu->dev, |
| 2269 | "failed to enable priq irq\n"); | 2269 | "failed to enable priq irq\n"); |
| 2270 | else | 2270 | else |
diff --git a/drivers/iommu/arm-smmu.c b/drivers/iommu/arm-smmu.c index e206ce7a4e4b..9345a3fcb706 100644 --- a/drivers/iommu/arm-smmu.c +++ b/drivers/iommu/arm-smmu.c | |||
| @@ -950,7 +950,7 @@ static int arm_smmu_init_domain_context(struct iommu_domain *domain, | |||
| 950 | 950 | ||
| 951 | ret = __arm_smmu_alloc_bitmap(smmu->context_map, start, | 951 | ret = __arm_smmu_alloc_bitmap(smmu->context_map, start, |
| 952 | smmu->num_context_banks); | 952 | smmu->num_context_banks); |
| 953 | if (IS_ERR_VALUE(ret)) | 953 | if (ret < 0) |
| 954 | goto out_unlock; | 954 | goto out_unlock; |
| 955 | 955 | ||
| 956 | cfg->cbndx = ret; | 956 | cfg->cbndx = ret; |
| @@ -989,7 +989,7 @@ static int arm_smmu_init_domain_context(struct iommu_domain *domain, | |||
| 989 | irq = smmu->irqs[smmu->num_global_irqs + cfg->irptndx]; | 989 | irq = smmu->irqs[smmu->num_global_irqs + cfg->irptndx]; |
| 990 | ret = request_irq(irq, arm_smmu_context_fault, IRQF_SHARED, | 990 | ret = request_irq(irq, arm_smmu_context_fault, IRQF_SHARED, |
| 991 | "arm-smmu-context-fault", domain); | 991 | "arm-smmu-context-fault", domain); |
| 992 | if (IS_ERR_VALUE(ret)) { | 992 | if (ret < 0) { |
| 993 | dev_err(smmu->dev, "failed to request context IRQ %d (%u)\n", | 993 | dev_err(smmu->dev, "failed to request context IRQ %d (%u)\n", |
| 994 | cfg->irptndx, irq); | 994 | cfg->irptndx, irq); |
| 995 | cfg->irptndx = INVALID_IRPTNDX; | 995 | cfg->irptndx = INVALID_IRPTNDX; |
| @@ -1099,7 +1099,7 @@ static int arm_smmu_master_configure_smrs(struct arm_smmu_device *smmu, | |||
| 1099 | for (i = 0; i < cfg->num_streamids; ++i) { | 1099 | for (i = 0; i < cfg->num_streamids; ++i) { |
| 1100 | int idx = __arm_smmu_alloc_bitmap(smmu->smr_map, 0, | 1100 | int idx = __arm_smmu_alloc_bitmap(smmu->smr_map, 0, |
| 1101 | smmu->num_mapping_groups); | 1101 | smmu->num_mapping_groups); |
| 1102 | if (IS_ERR_VALUE(idx)) { | 1102 | if (idx < 0) { |
| 1103 | dev_err(smmu->dev, "failed to allocate free SMR\n"); | 1103 | dev_err(smmu->dev, "failed to allocate free SMR\n"); |
| 1104 | goto err_free_smrs; | 1104 | goto err_free_smrs; |
| 1105 | } | 1105 | } |
| @@ -1233,7 +1233,7 @@ static int arm_smmu_attach_dev(struct iommu_domain *domain, struct device *dev) | |||
| 1233 | 1233 | ||
| 1234 | /* Ensure that the domain is finalised */ | 1234 | /* Ensure that the domain is finalised */ |
| 1235 | ret = arm_smmu_init_domain_context(domain, smmu); | 1235 | ret = arm_smmu_init_domain_context(domain, smmu); |
| 1236 | if (IS_ERR_VALUE(ret)) | 1236 | if (ret < 0) |
| 1237 | return ret; | 1237 | return ret; |
| 1238 | 1238 | ||
| 1239 | /* | 1239 | /* |
diff --git a/drivers/irqchip/irq-clps711x.c b/drivers/irqchip/irq-clps711x.c index eb5eb0cd414d..2223b3f15d68 100644 --- a/drivers/irqchip/irq-clps711x.c +++ b/drivers/irqchip/irq-clps711x.c | |||
| @@ -182,7 +182,7 @@ static int __init _clps711x_intc_init(struct device_node *np, | |||
| 182 | writel_relaxed(0, clps711x_intc->intmr[2]); | 182 | writel_relaxed(0, clps711x_intc->intmr[2]); |
| 183 | 183 | ||
| 184 | err = irq_alloc_descs(-1, 0, ARRAY_SIZE(clps711x_irqs), numa_node_id()); | 184 | err = irq_alloc_descs(-1, 0, ARRAY_SIZE(clps711x_irqs), numa_node_id()); |
| 185 | if (IS_ERR_VALUE(err)) | 185 | if (err < 0) |
| 186 | goto out_iounmap; | 186 | goto out_iounmap; |
| 187 | 187 | ||
| 188 | clps711x_intc->ops.map = clps711x_intc_irq_map; | 188 | clps711x_intc->ops.map = clps711x_intc_irq_map; |
diff --git a/drivers/irqchip/irq-gic.c b/drivers/irqchip/irq-gic.c index b4e647179346..fbc4ae2afd29 100644 --- a/drivers/irqchip/irq-gic.c +++ b/drivers/irqchip/irq-gic.c | |||
| @@ -1123,7 +1123,7 @@ static int __init __gic_init_bases(struct gic_chip_data *gic, int irq_start, | |||
| 1123 | 1123 | ||
| 1124 | irq_base = irq_alloc_descs(irq_start, 16, gic_irqs, | 1124 | irq_base = irq_alloc_descs(irq_start, 16, gic_irqs, |
| 1125 | numa_node_id()); | 1125 | numa_node_id()); |
| 1126 | if (IS_ERR_VALUE(irq_base)) { | 1126 | if (irq_base < 0) { |
| 1127 | WARN(1, "Cannot allocate irq_descs @ IRQ%d, assuming pre-allocated\n", | 1127 | WARN(1, "Cannot allocate irq_descs @ IRQ%d, assuming pre-allocated\n", |
| 1128 | irq_start); | 1128 | irq_start); |
| 1129 | irq_base = irq_start; | 1129 | irq_base = irq_start; |
diff --git a/drivers/irqchip/irq-hip04.c b/drivers/irqchip/irq-hip04.c index 9688d2e2a636..9e25d8ce08e5 100644 --- a/drivers/irqchip/irq-hip04.c +++ b/drivers/irqchip/irq-hip04.c | |||
| @@ -402,7 +402,7 @@ hip04_of_init(struct device_node *node, struct device_node *parent) | |||
| 402 | nr_irqs -= hwirq_base; /* calculate # of irqs to allocate */ | 402 | nr_irqs -= hwirq_base; /* calculate # of irqs to allocate */ |
| 403 | 403 | ||
| 404 | irq_base = irq_alloc_descs(-1, hwirq_base, nr_irqs, numa_node_id()); | 404 | irq_base = irq_alloc_descs(-1, hwirq_base, nr_irqs, numa_node_id()); |
| 405 | if (IS_ERR_VALUE(irq_base)) { | 405 | if (irq_base < 0) { |
| 406 | pr_err("failed to allocate IRQ numbers\n"); | 406 | pr_err("failed to allocate IRQ numbers\n"); |
| 407 | return -EINVAL; | 407 | return -EINVAL; |
| 408 | } | 408 | } |
diff --git a/drivers/irqchip/spear-shirq.c b/drivers/irqchip/spear-shirq.c index 1ccd2abed65f..1518ba31a80c 100644 --- a/drivers/irqchip/spear-shirq.c +++ b/drivers/irqchip/spear-shirq.c | |||
| @@ -232,7 +232,7 @@ static int __init shirq_init(struct spear_shirq **shirq_blocks, int block_nr, | |||
| 232 | nr_irqs += shirq_blocks[i]->nr_irqs; | 232 | nr_irqs += shirq_blocks[i]->nr_irqs; |
| 233 | 233 | ||
| 234 | virq_base = irq_alloc_descs(-1, 0, nr_irqs, 0); | 234 | virq_base = irq_alloc_descs(-1, 0, nr_irqs, 0); |
| 235 | if (IS_ERR_VALUE(virq_base)) { | 235 | if (virq_base < 0) { |
| 236 | pr_err("%s: irq desc alloc failed\n", __func__); | 236 | pr_err("%s: irq desc alloc failed\n", __func__); |
| 237 | goto err_unmap; | 237 | goto err_unmap; |
| 238 | } | 238 | } |
diff --git a/drivers/media/i2c/adp1653.c b/drivers/media/i2c/adp1653.c index 9e1731c565e7..e191e295c951 100644 --- a/drivers/media/i2c/adp1653.c +++ b/drivers/media/i2c/adp1653.c | |||
| @@ -95,7 +95,7 @@ static int adp1653_get_fault(struct adp1653_flash *flash) | |||
| 95 | int rval; | 95 | int rval; |
| 96 | 96 | ||
| 97 | fault = i2c_smbus_read_byte_data(client, ADP1653_REG_FAULT); | 97 | fault = i2c_smbus_read_byte_data(client, ADP1653_REG_FAULT); |
| 98 | if (IS_ERR_VALUE(fault)) | 98 | if (fault < 0) |
| 99 | return fault; | 99 | return fault; |
| 100 | 100 | ||
| 101 | flash->fault |= fault; | 101 | flash->fault |= fault; |
| @@ -105,13 +105,13 @@ static int adp1653_get_fault(struct adp1653_flash *flash) | |||
| 105 | 105 | ||
| 106 | /* Clear faults. */ | 106 | /* Clear faults. */ |
| 107 | rval = i2c_smbus_write_byte_data(client, ADP1653_REG_OUT_SEL, 0); | 107 | rval = i2c_smbus_write_byte_data(client, ADP1653_REG_OUT_SEL, 0); |
| 108 | if (IS_ERR_VALUE(rval)) | 108 | if (rval < 0) |
| 109 | return rval; | 109 | return rval; |
| 110 | 110 | ||
| 111 | flash->led_mode->val = V4L2_FLASH_LED_MODE_NONE; | 111 | flash->led_mode->val = V4L2_FLASH_LED_MODE_NONE; |
| 112 | 112 | ||
| 113 | rval = adp1653_update_hw(flash); | 113 | rval = adp1653_update_hw(flash); |
| 114 | if (IS_ERR_VALUE(rval)) | 114 | if (rval) |
| 115 | return rval; | 115 | return rval; |
| 116 | 116 | ||
| 117 | return flash->fault; | 117 | return flash->fault; |
| @@ -158,7 +158,7 @@ static int adp1653_get_ctrl(struct v4l2_ctrl *ctrl) | |||
| 158 | int rval; | 158 | int rval; |
| 159 | 159 | ||
| 160 | rval = adp1653_get_fault(flash); | 160 | rval = adp1653_get_fault(flash); |
| 161 | if (IS_ERR_VALUE(rval)) | 161 | if (rval) |
| 162 | return rval; | 162 | return rval; |
| 163 | 163 | ||
| 164 | ctrl->cur.val = 0; | 164 | ctrl->cur.val = 0; |
| @@ -184,7 +184,7 @@ static int adp1653_set_ctrl(struct v4l2_ctrl *ctrl) | |||
| 184 | int rval; | 184 | int rval; |
| 185 | 185 | ||
| 186 | rval = adp1653_get_fault(flash); | 186 | rval = adp1653_get_fault(flash); |
| 187 | if (IS_ERR_VALUE(rval)) | 187 | if (rval) |
| 188 | return rval; | 188 | return rval; |
| 189 | if ((rval & (ADP1653_REG_FAULT_FLT_SCP | | 189 | if ((rval & (ADP1653_REG_FAULT_FLT_SCP | |
| 190 | ADP1653_REG_FAULT_FLT_OT | | 190 | ADP1653_REG_FAULT_FLT_OT | |
diff --git a/drivers/media/platform/s5p-tv/mixer_drv.c b/drivers/media/platform/s5p-tv/mixer_drv.c index 5ef67774971d..8a5d19469ddc 100644 --- a/drivers/media/platform/s5p-tv/mixer_drv.c +++ b/drivers/media/platform/s5p-tv/mixer_drv.c | |||
| @@ -146,7 +146,7 @@ int mxr_power_get(struct mxr_device *mdev) | |||
| 146 | 146 | ||
| 147 | /* returning 1 means that power is already enabled, | 147 | /* returning 1 means that power is already enabled, |
| 148 | * so zero success be returned */ | 148 | * so zero success be returned */ |
| 149 | if (IS_ERR_VALUE(ret)) | 149 | if (ret < 0) |
| 150 | return ret; | 150 | return ret; |
| 151 | return 0; | 151 | return 0; |
| 152 | } | 152 | } |
diff --git a/drivers/mfd/twl4030-irq.c b/drivers/mfd/twl4030-irq.c index 40e51b0baa46..b46c0cfc27d9 100644 --- a/drivers/mfd/twl4030-irq.c +++ b/drivers/mfd/twl4030-irq.c | |||
| @@ -696,7 +696,7 @@ int twl4030_init_irq(struct device *dev, int irq_num) | |||
| 696 | nr_irqs = TWL4030_PWR_NR_IRQS + TWL4030_CORE_NR_IRQS; | 696 | nr_irqs = TWL4030_PWR_NR_IRQS + TWL4030_CORE_NR_IRQS; |
| 697 | 697 | ||
| 698 | irq_base = irq_alloc_descs(-1, 0, nr_irqs, 0); | 698 | irq_base = irq_alloc_descs(-1, 0, nr_irqs, 0); |
| 699 | if (IS_ERR_VALUE(irq_base)) { | 699 | if (irq_base < 0) { |
| 700 | dev_err(dev, "Fail to allocate IRQ descs\n"); | 700 | dev_err(dev, "Fail to allocate IRQ descs\n"); |
| 701 | return irq_base; | 701 | return irq_base; |
| 702 | } | 702 | } |
diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c index b81b08f81325..c984321d1881 100644 --- a/drivers/mmc/core/mmc.c +++ b/drivers/mmc/core/mmc.c | |||
| @@ -1276,7 +1276,7 @@ static int mmc_select_hs200(struct mmc_card *card) | |||
| 1276 | * switch to HS200 mode if bus width is set successfully. | 1276 | * switch to HS200 mode if bus width is set successfully. |
| 1277 | */ | 1277 | */ |
| 1278 | err = mmc_select_bus_width(card); | 1278 | err = mmc_select_bus_width(card); |
| 1279 | if (!IS_ERR_VALUE(err)) { | 1279 | if (!err) { |
| 1280 | val = EXT_CSD_TIMING_HS200 | | 1280 | val = EXT_CSD_TIMING_HS200 | |
| 1281 | card->drive_strength << EXT_CSD_DRV_STR_SHIFT; | 1281 | card->drive_strength << EXT_CSD_DRV_STR_SHIFT; |
| 1282 | err = __mmc_switch(card, EXT_CSD_CMD_SET_NORMAL, | 1282 | err = __mmc_switch(card, EXT_CSD_CMD_SET_NORMAL, |
| @@ -1583,7 +1583,7 @@ static int mmc_init_card(struct mmc_host *host, u32 ocr, | |||
| 1583 | } else if (mmc_card_hs(card)) { | 1583 | } else if (mmc_card_hs(card)) { |
| 1584 | /* Select the desired bus width optionally */ | 1584 | /* Select the desired bus width optionally */ |
| 1585 | err = mmc_select_bus_width(card); | 1585 | err = mmc_select_bus_width(card); |
| 1586 | if (!IS_ERR_VALUE(err)) { | 1586 | if (!err) { |
| 1587 | err = mmc_select_hs_ddr(card); | 1587 | err = mmc_select_hs_ddr(card); |
| 1588 | if (err) | 1588 | if (err) |
| 1589 | goto free_card; | 1589 | goto free_card; |
diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c index 829a6eebcdce..2cc6123b1df9 100644 --- a/drivers/mmc/host/dw_mmc.c +++ b/drivers/mmc/host/dw_mmc.c | |||
| @@ -1431,7 +1431,7 @@ static int dw_mci_get_ro(struct mmc_host *mmc) | |||
| 1431 | int gpio_ro = mmc_gpio_get_ro(mmc); | 1431 | int gpio_ro = mmc_gpio_get_ro(mmc); |
| 1432 | 1432 | ||
| 1433 | /* Use platform get_ro function, else try on board write protect */ | 1433 | /* Use platform get_ro function, else try on board write protect */ |
| 1434 | if (!IS_ERR_VALUE(gpio_ro)) | 1434 | if (gpio_ro >= 0) |
| 1435 | read_only = gpio_ro; | 1435 | read_only = gpio_ro; |
| 1436 | else | 1436 | else |
| 1437 | read_only = | 1437 | read_only = |
| @@ -1454,7 +1454,7 @@ static int dw_mci_get_cd(struct mmc_host *mmc) | |||
| 1454 | if ((mmc->caps & MMC_CAP_NEEDS_POLL) || | 1454 | if ((mmc->caps & MMC_CAP_NEEDS_POLL) || |
| 1455 | (mmc->caps & MMC_CAP_NONREMOVABLE)) | 1455 | (mmc->caps & MMC_CAP_NONREMOVABLE)) |
| 1456 | present = 1; | 1456 | present = 1; |
| 1457 | else if (!IS_ERR_VALUE(gpio_cd)) | 1457 | else if (gpio_cd >= 0) |
| 1458 | present = gpio_cd; | 1458 | present = gpio_cd; |
| 1459 | else | 1459 | else |
| 1460 | present = (mci_readl(slot->host, CDETECT) & (1 << slot->id)) | 1460 | present = (mci_readl(slot->host, CDETECT) & (1 << slot->id)) |
| @@ -2927,7 +2927,7 @@ static void dw_mci_enable_cd(struct dw_mci *host) | |||
| 2927 | if (slot->mmc->caps & MMC_CAP_NEEDS_POLL) | 2927 | if (slot->mmc->caps & MMC_CAP_NEEDS_POLL) |
| 2928 | return; | 2928 | return; |
| 2929 | 2929 | ||
| 2930 | if (IS_ERR_VALUE(mmc_gpio_get_cd(slot->mmc))) | 2930 | if (mmc_gpio_get_cd(slot->mmc) < 0) |
| 2931 | break; | 2931 | break; |
| 2932 | } | 2932 | } |
| 2933 | if (i == host->num_slots) | 2933 | if (i == host->num_slots) |
diff --git a/drivers/mmc/host/sdhci-esdhc-imx.c b/drivers/mmc/host/sdhci-esdhc-imx.c index 2d300d87cda8..9d3ae1f4bd3c 100644 --- a/drivers/mmc/host/sdhci-esdhc-imx.c +++ b/drivers/mmc/host/sdhci-esdhc-imx.c | |||
| @@ -1011,7 +1011,7 @@ sdhci_esdhc_imx_probe_dt(struct platform_device *pdev, | |||
| 1011 | if (ret) | 1011 | if (ret) |
| 1012 | return ret; | 1012 | return ret; |
| 1013 | 1013 | ||
| 1014 | if (!IS_ERR_VALUE(mmc_gpio_get_cd(host->mmc))) | 1014 | if (mmc_gpio_get_cd(host->mmc) >= 0) |
| 1015 | host->quirks &= ~SDHCI_QUIRK_BROKEN_CARD_DETECTION; | 1015 | host->quirks &= ~SDHCI_QUIRK_BROKEN_CARD_DETECTION; |
| 1016 | 1016 | ||
| 1017 | return 0; | 1017 | return 0; |
diff --git a/drivers/mmc/host/sdhci-of-at91.c b/drivers/mmc/host/sdhci-of-at91.c index 25f779e09d8e..d4cef713d246 100644 --- a/drivers/mmc/host/sdhci-of-at91.c +++ b/drivers/mmc/host/sdhci-of-at91.c | |||
| @@ -289,7 +289,7 @@ static int sdhci_at91_probe(struct platform_device *pdev) | |||
| 289 | * to enable polling via device tree with broken-cd property. | 289 | * to enable polling via device tree with broken-cd property. |
| 290 | */ | 290 | */ |
| 291 | if (!(host->mmc->caps & MMC_CAP_NONREMOVABLE) && | 291 | if (!(host->mmc->caps & MMC_CAP_NONREMOVABLE) && |
| 292 | IS_ERR_VALUE(mmc_gpio_get_cd(host->mmc))) { | 292 | mmc_gpio_get_cd(host->mmc) < 0) { |
| 293 | host->mmc->caps |= MMC_CAP_NEEDS_POLL; | 293 | host->mmc->caps |= MMC_CAP_NEEDS_POLL; |
| 294 | host->quirks &= ~SDHCI_QUIRK_BROKEN_CARD_DETECTION; | 294 | host->quirks &= ~SDHCI_QUIRK_BROKEN_CARD_DETECTION; |
| 295 | } | 295 | } |
diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c index e010ea4eb6f5..0e3d7c056cb1 100644 --- a/drivers/mmc/host/sdhci.c +++ b/drivers/mmc/host/sdhci.c | |||
| @@ -1624,7 +1624,7 @@ static int sdhci_get_cd(struct mmc_host *mmc) | |||
| 1624 | * Try slot gpio detect, if defined it take precedence | 1624 | * Try slot gpio detect, if defined it take precedence |
| 1625 | * over build in controller functionality | 1625 | * over build in controller functionality |
| 1626 | */ | 1626 | */ |
| 1627 | if (!IS_ERR_VALUE(gpio_cd)) | 1627 | if (gpio_cd >= 0) |
| 1628 | return !!gpio_cd; | 1628 | return !!gpio_cd; |
| 1629 | 1629 | ||
| 1630 | /* If polling, assume that the card is always present. */ | 1630 | /* If polling, assume that the card is always present. */ |
| @@ -3077,7 +3077,7 @@ int sdhci_add_host(struct sdhci_host *host) | |||
| 3077 | 3077 | ||
| 3078 | if ((host->quirks & SDHCI_QUIRK_BROKEN_CARD_DETECTION) && | 3078 | if ((host->quirks & SDHCI_QUIRK_BROKEN_CARD_DETECTION) && |
| 3079 | !(mmc->caps & MMC_CAP_NONREMOVABLE) && | 3079 | !(mmc->caps & MMC_CAP_NONREMOVABLE) && |
| 3080 | IS_ERR_VALUE(mmc_gpio_get_cd(host->mmc))) | 3080 | mmc_gpio_get_cd(host->mmc) < 0) |
| 3081 | mmc->caps |= MMC_CAP_NEEDS_POLL; | 3081 | mmc->caps |= MMC_CAP_NEEDS_POLL; |
| 3082 | 3082 | ||
| 3083 | /* If there are external regulators, get them */ | 3083 | /* If there are external regulators, get them */ |
diff --git a/drivers/net/ethernet/freescale/fman/fman.c b/drivers/net/ethernet/freescale/fman/fman.c index bcb9dccada4d..1de2e1e51c2b 100644 --- a/drivers/net/ethernet/freescale/fman/fman.c +++ b/drivers/net/ethernet/freescale/fman/fman.c | |||
| @@ -615,7 +615,7 @@ struct fman { | |||
| 615 | struct fman_cfg *cfg; | 615 | struct fman_cfg *cfg; |
| 616 | struct muram_info *muram; | 616 | struct muram_info *muram; |
| 617 | /* cam section in muram */ | 617 | /* cam section in muram */ |
| 618 | int cam_offset; | 618 | unsigned long cam_offset; |
| 619 | size_t cam_size; | 619 | size_t cam_size; |
| 620 | /* Fifo in MURAM */ | 620 | /* Fifo in MURAM */ |
| 621 | int fifo_offset; | 621 | int fifo_offset; |
diff --git a/drivers/net/ethernet/freescale/fman/fman_muram.c b/drivers/net/ethernet/freescale/fman/fman_muram.c index 4eb0e9ac7182..47394c45b6e8 100644 --- a/drivers/net/ethernet/freescale/fman/fman_muram.c +++ b/drivers/net/ethernet/freescale/fman/fman_muram.c | |||
| @@ -129,7 +129,7 @@ unsigned long fman_muram_offset_to_vbase(struct muram_info *muram, | |||
| 129 | * | 129 | * |
| 130 | * Return: address of the allocated memory; NULL otherwise. | 130 | * Return: address of the allocated memory; NULL otherwise. |
| 131 | */ | 131 | */ |
| 132 | int fman_muram_alloc(struct muram_info *muram, size_t size) | 132 | unsigned long fman_muram_alloc(struct muram_info *muram, size_t size) |
| 133 | { | 133 | { |
| 134 | unsigned long vaddr; | 134 | unsigned long vaddr; |
| 135 | 135 | ||
| @@ -150,7 +150,7 @@ int fman_muram_alloc(struct muram_info *muram, size_t size) | |||
| 150 | * | 150 | * |
| 151 | * Free an allocated memory from FM-MURAM partition. | 151 | * Free an allocated memory from FM-MURAM partition. |
| 152 | */ | 152 | */ |
| 153 | void fman_muram_free_mem(struct muram_info *muram, u32 offset, size_t size) | 153 | void fman_muram_free_mem(struct muram_info *muram, unsigned long offset, size_t size) |
| 154 | { | 154 | { |
| 155 | unsigned long addr = fman_muram_offset_to_vbase(muram, offset); | 155 | unsigned long addr = fman_muram_offset_to_vbase(muram, offset); |
| 156 | 156 | ||
diff --git a/drivers/net/ethernet/freescale/fman/fman_muram.h b/drivers/net/ethernet/freescale/fman/fman_muram.h index dbf0af9e5bb5..889649ad8931 100644 --- a/drivers/net/ethernet/freescale/fman/fman_muram.h +++ b/drivers/net/ethernet/freescale/fman/fman_muram.h | |||
| @@ -44,8 +44,8 @@ struct muram_info *fman_muram_init(phys_addr_t base, size_t size); | |||
| 44 | unsigned long fman_muram_offset_to_vbase(struct muram_info *muram, | 44 | unsigned long fman_muram_offset_to_vbase(struct muram_info *muram, |
| 45 | unsigned long offset); | 45 | unsigned long offset); |
| 46 | 46 | ||
| 47 | int fman_muram_alloc(struct muram_info *muram, size_t size); | 47 | unsigned long fman_muram_alloc(struct muram_info *muram, size_t size); |
| 48 | 48 | ||
| 49 | void fman_muram_free_mem(struct muram_info *muram, u32 offset, size_t size); | 49 | void fman_muram_free_mem(struct muram_info *muram, unsigned long offset, size_t size); |
| 50 | 50 | ||
| 51 | #endif /* __FM_MURAM_EXT */ | 51 | #endif /* __FM_MURAM_EXT */ |
diff --git a/drivers/net/wireless/ti/wlcore/spi.c b/drivers/net/wireless/ti/wlcore/spi.c index 020ac1a4b408..cea9443c22a6 100644 --- a/drivers/net/wireless/ti/wlcore/spi.c +++ b/drivers/net/wireless/ti/wlcore/spi.c | |||
| @@ -382,7 +382,7 @@ static int wlcore_probe_of(struct spi_device *spi, struct wl12xx_spi_glue *glue, | |||
| 382 | 382 | ||
| 383 | ret = of_property_read_u32(dt_node, "ref-clock-frequency", | 383 | ret = of_property_read_u32(dt_node, "ref-clock-frequency", |
| 384 | &pdev_data->ref_clock_freq); | 384 | &pdev_data->ref_clock_freq); |
| 385 | if (IS_ERR_VALUE(ret)) { | 385 | if (ret) { |
| 386 | dev_err(glue->dev, | 386 | dev_err(glue->dev, |
| 387 | "can't get reference clock frequency (%d)\n", ret); | 387 | "can't get reference clock frequency (%d)\n", ret); |
| 388 | return ret; | 388 | return ret; |
| @@ -425,7 +425,7 @@ static int wl1271_probe(struct spi_device *spi) | |||
| 425 | } | 425 | } |
| 426 | 426 | ||
| 427 | ret = wlcore_probe_of(spi, glue, &pdev_data); | 427 | ret = wlcore_probe_of(spi, glue, &pdev_data); |
| 428 | if (IS_ERR_VALUE(ret)) { | 428 | if (ret) { |
| 429 | dev_err(glue->dev, | 429 | dev_err(glue->dev, |
| 430 | "can't get device tree parameters (%d)\n", ret); | 430 | "can't get device tree parameters (%d)\n", ret); |
| 431 | return ret; | 431 | return ret; |
diff --git a/drivers/nvmem/core.c b/drivers/nvmem/core.c index bb4ea123547f..965911d9b36a 100644 --- a/drivers/nvmem/core.c +++ b/drivers/nvmem/core.c | |||
| @@ -113,7 +113,7 @@ static ssize_t bin_attr_nvmem_read(struct file *filp, struct kobject *kobj, | |||
| 113 | 113 | ||
| 114 | rc = nvmem_reg_read(nvmem, pos, buf, count); | 114 | rc = nvmem_reg_read(nvmem, pos, buf, count); |
| 115 | 115 | ||
| 116 | if (IS_ERR_VALUE(rc)) | 116 | if (rc) |
| 117 | return rc; | 117 | return rc; |
| 118 | 118 | ||
| 119 | return count; | 119 | return count; |
| @@ -147,7 +147,7 @@ static ssize_t bin_attr_nvmem_write(struct file *filp, struct kobject *kobj, | |||
| 147 | 147 | ||
| 148 | rc = nvmem_reg_write(nvmem, pos, buf, count); | 148 | rc = nvmem_reg_write(nvmem, pos, buf, count); |
| 149 | 149 | ||
| 150 | if (IS_ERR_VALUE(rc)) | 150 | if (rc) |
| 151 | return rc; | 151 | return rc; |
| 152 | 152 | ||
| 153 | return count; | 153 | return count; |
| @@ -366,7 +366,7 @@ static int nvmem_add_cells(struct nvmem_device *nvmem, | |||
| 366 | } | 366 | } |
| 367 | 367 | ||
| 368 | rval = nvmem_cell_info_to_nvmem_cell(nvmem, &info[i], cells[i]); | 368 | rval = nvmem_cell_info_to_nvmem_cell(nvmem, &info[i], cells[i]); |
| 369 | if (IS_ERR_VALUE(rval)) { | 369 | if (rval) { |
| 370 | kfree(cells[i]); | 370 | kfree(cells[i]); |
| 371 | goto err; | 371 | goto err; |
| 372 | } | 372 | } |
| @@ -963,7 +963,7 @@ static int __nvmem_cell_read(struct nvmem_device *nvmem, | |||
| 963 | 963 | ||
| 964 | rc = nvmem_reg_read(nvmem, cell->offset, buf, cell->bytes); | 964 | rc = nvmem_reg_read(nvmem, cell->offset, buf, cell->bytes); |
| 965 | 965 | ||
| 966 | if (IS_ERR_VALUE(rc)) | 966 | if (rc) |
| 967 | return rc; | 967 | return rc; |
| 968 | 968 | ||
| 969 | /* shift bits in-place */ | 969 | /* shift bits in-place */ |
| @@ -998,7 +998,7 @@ void *nvmem_cell_read(struct nvmem_cell *cell, size_t *len) | |||
| 998 | return ERR_PTR(-ENOMEM); | 998 | return ERR_PTR(-ENOMEM); |
| 999 | 999 | ||
| 1000 | rc = __nvmem_cell_read(nvmem, cell, buf, len); | 1000 | rc = __nvmem_cell_read(nvmem, cell, buf, len); |
| 1001 | if (IS_ERR_VALUE(rc)) { | 1001 | if (rc) { |
| 1002 | kfree(buf); | 1002 | kfree(buf); |
| 1003 | return ERR_PTR(rc); | 1003 | return ERR_PTR(rc); |
| 1004 | } | 1004 | } |
| @@ -1083,7 +1083,7 @@ int nvmem_cell_write(struct nvmem_cell *cell, void *buf, size_t len) | |||
| 1083 | if (cell->bit_offset || cell->nbits) | 1083 | if (cell->bit_offset || cell->nbits) |
| 1084 | kfree(buf); | 1084 | kfree(buf); |
| 1085 | 1085 | ||
| 1086 | if (IS_ERR_VALUE(rc)) | 1086 | if (rc) |
| 1087 | return rc; | 1087 | return rc; |
| 1088 | 1088 | ||
| 1089 | return len; | 1089 | return len; |
| @@ -1111,11 +1111,11 @@ ssize_t nvmem_device_cell_read(struct nvmem_device *nvmem, | |||
| 1111 | return -EINVAL; | 1111 | return -EINVAL; |
| 1112 | 1112 | ||
| 1113 | rc = nvmem_cell_info_to_nvmem_cell(nvmem, info, &cell); | 1113 | rc = nvmem_cell_info_to_nvmem_cell(nvmem, info, &cell); |
| 1114 | if (IS_ERR_VALUE(rc)) | 1114 | if (rc) |
| 1115 | return rc; | 1115 | return rc; |
| 1116 | 1116 | ||
| 1117 | rc = __nvmem_cell_read(nvmem, &cell, buf, &len); | 1117 | rc = __nvmem_cell_read(nvmem, &cell, buf, &len); |
| 1118 | if (IS_ERR_VALUE(rc)) | 1118 | if (rc) |
| 1119 | return rc; | 1119 | return rc; |
| 1120 | 1120 | ||
| 1121 | return len; | 1121 | return len; |
| @@ -1141,7 +1141,7 @@ int nvmem_device_cell_write(struct nvmem_device *nvmem, | |||
| 1141 | return -EINVAL; | 1141 | return -EINVAL; |
| 1142 | 1142 | ||
| 1143 | rc = nvmem_cell_info_to_nvmem_cell(nvmem, info, &cell); | 1143 | rc = nvmem_cell_info_to_nvmem_cell(nvmem, info, &cell); |
| 1144 | if (IS_ERR_VALUE(rc)) | 1144 | if (rc) |
| 1145 | return rc; | 1145 | return rc; |
| 1146 | 1146 | ||
| 1147 | return nvmem_cell_write(&cell, buf, cell.bytes); | 1147 | return nvmem_cell_write(&cell, buf, cell.bytes); |
| @@ -1170,7 +1170,7 @@ int nvmem_device_read(struct nvmem_device *nvmem, | |||
| 1170 | 1170 | ||
| 1171 | rc = nvmem_reg_read(nvmem, offset, buf, bytes); | 1171 | rc = nvmem_reg_read(nvmem, offset, buf, bytes); |
| 1172 | 1172 | ||
| 1173 | if (IS_ERR_VALUE(rc)) | 1173 | if (rc) |
| 1174 | return rc; | 1174 | return rc; |
| 1175 | 1175 | ||
| 1176 | return bytes; | 1176 | return bytes; |
| @@ -1198,7 +1198,7 @@ int nvmem_device_write(struct nvmem_device *nvmem, | |||
| 1198 | 1198 | ||
| 1199 | rc = nvmem_reg_write(nvmem, offset, buf, bytes); | 1199 | rc = nvmem_reg_write(nvmem, offset, buf, bytes); |
| 1200 | 1200 | ||
| 1201 | if (IS_ERR_VALUE(rc)) | 1201 | if (rc) |
| 1202 | return rc; | 1202 | return rc; |
| 1203 | 1203 | ||
| 1204 | 1204 | ||
diff --git a/drivers/tty/serial/amba-pl011.c b/drivers/tty/serial/amba-pl011.c index a2aa655f56c4..1b7331e40d79 100644 --- a/drivers/tty/serial/amba-pl011.c +++ b/drivers/tty/serial/amba-pl011.c | |||
| @@ -2360,7 +2360,7 @@ static int pl011_probe_dt_alias(int index, struct device *dev) | |||
| 2360 | return ret; | 2360 | return ret; |
| 2361 | 2361 | ||
| 2362 | ret = of_alias_get_id(np, "serial"); | 2362 | ret = of_alias_get_id(np, "serial"); |
| 2363 | if (IS_ERR_VALUE(ret)) { | 2363 | if (ret < 0) { |
| 2364 | seen_dev_without_alias = true; | 2364 | seen_dev_without_alias = true; |
| 2365 | ret = index; | 2365 | ret = index; |
| 2366 | } else { | 2366 | } else { |
diff --git a/drivers/tty/serial/sprd_serial.c b/drivers/tty/serial/sprd_serial.c index 18971063f95f..699447aa8b43 100644 --- a/drivers/tty/serial/sprd_serial.c +++ b/drivers/tty/serial/sprd_serial.c | |||
| @@ -654,7 +654,7 @@ static int sprd_probe_dt_alias(int index, struct device *dev) | |||
| 654 | return ret; | 654 | return ret; |
| 655 | 655 | ||
| 656 | ret = of_alias_get_id(np, "serial"); | 656 | ret = of_alias_get_id(np, "serial"); |
| 657 | if (IS_ERR_VALUE(ret)) | 657 | if (ret < 0) |
| 658 | ret = index; | 658 | ret = index; |
| 659 | else if (ret >= ARRAY_SIZE(sprd_port) || sprd_port[ret] != NULL) { | 659 | else if (ret >= ARRAY_SIZE(sprd_port) || sprd_port[ret] != NULL) { |
| 660 | dev_warn(dev, "requested serial port %d not available.\n", ret); | 660 | dev_warn(dev, "requested serial port %d not available.\n", ret); |
diff --git a/drivers/video/fbdev/da8xx-fb.c b/drivers/video/fbdev/da8xx-fb.c index d8d583d32a37..c229b1a0d13b 100644 --- a/drivers/video/fbdev/da8xx-fb.c +++ b/drivers/video/fbdev/da8xx-fb.c | |||
| @@ -713,7 +713,7 @@ static int da8xx_fb_config_clk_divider(struct da8xx_fb_par *par, | |||
| 713 | 713 | ||
| 714 | if (par->lcdc_clk_rate != lcdc_clk_rate) { | 714 | if (par->lcdc_clk_rate != lcdc_clk_rate) { |
| 715 | ret = clk_set_rate(par->lcdc_clk, lcdc_clk_rate); | 715 | ret = clk_set_rate(par->lcdc_clk, lcdc_clk_rate); |
| 716 | if (IS_ERR_VALUE(ret)) { | 716 | if (ret) { |
| 717 | dev_err(par->dev, | 717 | dev_err(par->dev, |
| 718 | "unable to set clock rate at %u\n", | 718 | "unable to set clock rate at %u\n", |
| 719 | lcdc_clk_rate); | 719 | lcdc_clk_rate); |
| @@ -784,7 +784,7 @@ static int lcd_init(struct da8xx_fb_par *par, const struct lcd_ctrl_config *cfg, | |||
| 784 | int ret = 0; | 784 | int ret = 0; |
| 785 | 785 | ||
| 786 | ret = da8xx_fb_calc_config_clk_divider(par, panel); | 786 | ret = da8xx_fb_calc_config_clk_divider(par, panel); |
| 787 | if (IS_ERR_VALUE(ret)) { | 787 | if (ret) { |
| 788 | dev_err(par->dev, "unable to configure clock\n"); | 788 | dev_err(par->dev, "unable to configure clock\n"); |
| 789 | return ret; | 789 | return ret; |
| 790 | } | 790 | } |
diff --git a/fs/afs/write.c b/fs/afs/write.c index 65de439bdc4f..14d506efd1aa 100644 --- a/fs/afs/write.c +++ b/fs/afs/write.c | |||
| @@ -643,10 +643,6 @@ ssize_t afs_file_write(struct kiocb *iocb, struct iov_iter *from) | |||
| 643 | return 0; | 643 | return 0; |
| 644 | 644 | ||
| 645 | result = generic_file_write_iter(iocb, from); | 645 | result = generic_file_write_iter(iocb, from); |
| 646 | if (IS_ERR_VALUE(result)) { | ||
| 647 | _leave(" = %zd", result); | ||
| 648 | return result; | ||
| 649 | } | ||
| 650 | 646 | ||
| 651 | _leave(" = %zd", result); | 647 | _leave(" = %zd", result); |
| 652 | return result; | 648 | return result; |
diff --git a/fs/binfmt_flat.c b/fs/binfmt_flat.c index f723cd3a455c..caf9e39bb82b 100644 --- a/fs/binfmt_flat.c +++ b/fs/binfmt_flat.c | |||
| @@ -337,7 +337,7 @@ calc_reloc(unsigned long r, struct lib_info *p, int curid, int internalp) | |||
| 337 | "(%d != %d)", (unsigned) r, curid, id); | 337 | "(%d != %d)", (unsigned) r, curid, id); |
| 338 | goto failed; | 338 | goto failed; |
| 339 | } else if ( ! p->lib_list[id].loaded && | 339 | } else if ( ! p->lib_list[id].loaded && |
| 340 | IS_ERR_VALUE(load_flat_shared_library(id, p))) { | 340 | load_flat_shared_library(id, p) < 0) { |
| 341 | printk("BINFMT_FLAT: failed to load library %d", id); | 341 | printk("BINFMT_FLAT: failed to load library %d", id); |
| 342 | goto failed; | 342 | goto failed; |
| 343 | } | 343 | } |
| @@ -837,7 +837,7 @@ static int load_flat_shared_library(int id, struct lib_info *libs) | |||
| 837 | 837 | ||
| 838 | res = prepare_binprm(&bprm); | 838 | res = prepare_binprm(&bprm); |
| 839 | 839 | ||
| 840 | if (!IS_ERR_VALUE(res)) | 840 | if (!res) |
| 841 | res = load_flat_file(&bprm, libs, id, NULL); | 841 | res = load_flat_file(&bprm, libs, id, NULL); |
| 842 | 842 | ||
| 843 | abort_creds(bprm.cred); | 843 | abort_creds(bprm.cred); |
| @@ -883,7 +883,7 @@ static int load_flat_binary(struct linux_binprm * bprm) | |||
| 883 | stack_len += FLAT_STACK_ALIGN - 1; /* reserve for upcoming alignment */ | 883 | stack_len += FLAT_STACK_ALIGN - 1; /* reserve for upcoming alignment */ |
| 884 | 884 | ||
| 885 | res = load_flat_file(bprm, &libinfo, 0, &stack_len); | 885 | res = load_flat_file(bprm, &libinfo, 0, &stack_len); |
| 886 | if (IS_ERR_VALUE(res)) | 886 | if (res < 0) |
| 887 | return res; | 887 | return res; |
| 888 | 888 | ||
| 889 | /* Update data segment pointers for all libraries */ | 889 | /* Update data segment pointers for all libraries */ |
diff --git a/fs/gfs2/dir.c b/fs/gfs2/dir.c index 4a01f30e9995..271d93905bac 100644 --- a/fs/gfs2/dir.c +++ b/fs/gfs2/dir.c | |||
| @@ -783,12 +783,15 @@ static int get_leaf_nr(struct gfs2_inode *dip, u32 index, | |||
| 783 | u64 *leaf_out) | 783 | u64 *leaf_out) |
| 784 | { | 784 | { |
| 785 | __be64 *hash; | 785 | __be64 *hash; |
| 786 | int error; | ||
| 786 | 787 | ||
| 787 | hash = gfs2_dir_get_hash_table(dip); | 788 | hash = gfs2_dir_get_hash_table(dip); |
| 788 | if (IS_ERR(hash)) | 789 | error = PTR_ERR_OR_ZERO(hash); |
| 789 | return PTR_ERR(hash); | 790 | |
| 790 | *leaf_out = be64_to_cpu(*(hash + index)); | 791 | if (!error) |
| 791 | return 0; | 792 | *leaf_out = be64_to_cpu(*(hash + index)); |
| 793 | |||
| 794 | return error; | ||
| 792 | } | 795 | } |
| 793 | 796 | ||
| 794 | static int get_first_leaf(struct gfs2_inode *dip, u32 index, | 797 | static int get_first_leaf(struct gfs2_inode *dip, u32 index, |
| @@ -798,7 +801,7 @@ static int get_first_leaf(struct gfs2_inode *dip, u32 index, | |||
| 798 | int error; | 801 | int error; |
| 799 | 802 | ||
| 800 | error = get_leaf_nr(dip, index, &leaf_no); | 803 | error = get_leaf_nr(dip, index, &leaf_no); |
| 801 | if (!IS_ERR_VALUE(error)) | 804 | if (!error) |
| 802 | error = get_leaf(dip, leaf_no, bh_out); | 805 | error = get_leaf(dip, leaf_no, bh_out); |
| 803 | 806 | ||
| 804 | return error; | 807 | return error; |
| @@ -1014,7 +1017,7 @@ static int dir_split_leaf(struct inode *inode, const struct qstr *name) | |||
| 1014 | 1017 | ||
| 1015 | index = name->hash >> (32 - dip->i_depth); | 1018 | index = name->hash >> (32 - dip->i_depth); |
| 1016 | error = get_leaf_nr(dip, index, &leaf_no); | 1019 | error = get_leaf_nr(dip, index, &leaf_no); |
| 1017 | if (IS_ERR_VALUE(error)) | 1020 | if (error) |
| 1018 | return error; | 1021 | return error; |
| 1019 | 1022 | ||
| 1020 | /* Get the old leaf block */ | 1023 | /* Get the old leaf block */ |
diff --git a/kernel/pid.c b/kernel/pid.c index 4d73a834c7e6..f66162f2359b 100644 --- a/kernel/pid.c +++ b/kernel/pid.c | |||
| @@ -311,7 +311,7 @@ struct pid *alloc_pid(struct pid_namespace *ns) | |||
| 311 | pid->level = ns->level; | 311 | pid->level = ns->level; |
| 312 | for (i = ns->level; i >= 0; i--) { | 312 | for (i = ns->level; i >= 0; i--) { |
| 313 | nr = alloc_pidmap(tmp); | 313 | nr = alloc_pidmap(tmp); |
| 314 | if (IS_ERR_VALUE(nr)) { | 314 | if (nr < 0) { |
| 315 | retval = nr; | 315 | retval = nr; |
| 316 | goto out_free; | 316 | goto out_free; |
| 317 | } | 317 | } |
diff --git a/net/9p/client.c b/net/9p/client.c index ea79ee9a7348..3fc94a49ccd5 100644 --- a/net/9p/client.c +++ b/net/9p/client.c | |||
| @@ -518,10 +518,10 @@ static int p9_check_errors(struct p9_client *c, struct p9_req_t *req) | |||
| 518 | if (err) | 518 | if (err) |
| 519 | goto out_err; | 519 | goto out_err; |
| 520 | 520 | ||
| 521 | if (p9_is_proto_dotu(c)) | 521 | if (p9_is_proto_dotu(c) && ecode < 512) |
| 522 | err = -ecode; | 522 | err = -ecode; |
| 523 | 523 | ||
| 524 | if (!err || !IS_ERR_VALUE(err)) { | 524 | if (!err) { |
| 525 | err = p9_errstr2errno(ename, strlen(ename)); | 525 | err = p9_errstr2errno(ename, strlen(ename)); |
| 526 | 526 | ||
| 527 | p9_debug(P9_DEBUG_9P, "<<< RERROR (%d) %s\n", | 527 | p9_debug(P9_DEBUG_9P, "<<< RERROR (%d) %s\n", |
| @@ -605,10 +605,10 @@ static int p9_check_zc_errors(struct p9_client *c, struct p9_req_t *req, | |||
| 605 | if (err) | 605 | if (err) |
| 606 | goto out_err; | 606 | goto out_err; |
| 607 | 607 | ||
| 608 | if (p9_is_proto_dotu(c)) | 608 | if (p9_is_proto_dotu(c) && ecode < 512) |
| 609 | err = -ecode; | 609 | err = -ecode; |
| 610 | 610 | ||
| 611 | if (!err || !IS_ERR_VALUE(err)) { | 611 | if (!err) { |
| 612 | err = p9_errstr2errno(ename, strlen(ename)); | 612 | err = p9_errstr2errno(ename, strlen(ename)); |
| 613 | 613 | ||
| 614 | p9_debug(P9_DEBUG_9P, "<<< RERROR (%d) %s\n", | 614 | p9_debug(P9_DEBUG_9P, "<<< RERROR (%d) %s\n", |
diff --git a/sound/soc/qcom/lpass-platform.c b/sound/soc/qcom/lpass-platform.c index 6e8665430bd5..ddfe34434765 100644 --- a/sound/soc/qcom/lpass-platform.c +++ b/sound/soc/qcom/lpass-platform.c | |||
| @@ -491,7 +491,7 @@ static int lpass_platform_pcm_new(struct snd_soc_pcm_runtime *soc_runtime) | |||
| 491 | data->rdma_ch = v->alloc_dma_channel(drvdata, | 491 | data->rdma_ch = v->alloc_dma_channel(drvdata, |
| 492 | SNDRV_PCM_STREAM_PLAYBACK); | 492 | SNDRV_PCM_STREAM_PLAYBACK); |
| 493 | 493 | ||
| 494 | if (IS_ERR_VALUE(data->rdma_ch)) | 494 | if (data->rdma_ch < 0) |
| 495 | return data->rdma_ch; | 495 | return data->rdma_ch; |
| 496 | 496 | ||
| 497 | drvdata->substream[data->rdma_ch] = psubstream; | 497 | drvdata->substream[data->rdma_ch] = psubstream; |
| @@ -518,7 +518,7 @@ static int lpass_platform_pcm_new(struct snd_soc_pcm_runtime *soc_runtime) | |||
| 518 | data->wrdma_ch = v->alloc_dma_channel(drvdata, | 518 | data->wrdma_ch = v->alloc_dma_channel(drvdata, |
| 519 | SNDRV_PCM_STREAM_CAPTURE); | 519 | SNDRV_PCM_STREAM_CAPTURE); |
| 520 | 520 | ||
| 521 | if (IS_ERR_VALUE(data->wrdma_ch)) | 521 | if (data->wrdma_ch < 0) |
| 522 | goto capture_alloc_err; | 522 | goto capture_alloc_err; |
| 523 | 523 | ||
| 524 | drvdata->substream[data->wrdma_ch] = csubstream; | 524 | drvdata->substream[data->wrdma_ch] = csubstream; |
