diff options
| author | Andrzej Hajda <a.hajda@samsung.com> | 2015-12-14 12:12:42 +0100 |
|---|---|---|
| committer | Chanwoo Choi <cw00.choi@samsung.com> | 2015-12-15 10:02:23 +0900 |
| commit | cbc46603df56b5933aef6765fc50d3aa18f96ced (patch) | |
| tree | ab564e12eb2c31170cb4323fcaeb32a5ce319937 | |
| parent | c2957563ad0965c8bfb393dcd22ddb683dd1ff1c (diff) | |
| download | op-kernel-dev-cbc46603df56b5933aef6765fc50d3aa18f96ced.zip op-kernel-dev-cbc46603df56b5933aef6765fc50d3aa18f96ced.tar.gz | |
extcon: max77693: fix handling return value of regmap_irq_get_virq
The function can return negative values, so its result should
be assigned to signed variable.
Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
Suggested-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
| -rw-r--r-- | drivers/extcon/extcon-max77693.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/extcon/extcon-max77693.c b/drivers/extcon/extcon-max77693.c index 44c499e..fdf8f5d 100644 --- a/drivers/extcon/extcon-max77693.c +++ b/drivers/extcon/extcon-max77693.c @@ -1127,11 +1127,11 @@ static int max77693_muic_probe(struct platform_device *pdev) /* Support irq domain for MAX77693 MUIC device */ for (i = 0; i < ARRAY_SIZE(muic_irqs); i++) { struct max77693_muic_irq *muic_irq = &muic_irqs[i]; - unsigned int virq = 0; + int virq; virq = regmap_irq_get_virq(max77693->irq_data_muic, muic_irq->irq); - if (!virq) + if (virq <= 0) return -EINVAL; muic_irq->virq = virq; |
