aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndrzej Hajda <a.hajda@samsung.com>2015-12-14 06:12:42 -0500
committerChanwoo Choi <cw00.choi@samsung.com>2015-12-14 20:02:23 -0500
commitcbc46603df56b5933aef6765fc50d3aa18f96ced (patch)
treeab564e12eb2c31170cb4323fcaeb32a5ce319937
parentc2957563ad0965c8bfb393dcd22ddb683dd1ff1c (diff)
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.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/extcon/extcon-max77693.c b/drivers/extcon/extcon-max77693.c
index 44c499e1beee..fdf8f5d4d4e9 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)
1127 /* Support irq domain for MAX77693 MUIC device */ 1127 /* Support irq domain for MAX77693 MUIC device */
1128 for (i = 0; i < ARRAY_SIZE(muic_irqs); i++) { 1128 for (i = 0; i < ARRAY_SIZE(muic_irqs); i++) {
1129 struct max77693_muic_irq *muic_irq = &muic_irqs[i]; 1129 struct max77693_muic_irq *muic_irq = &muic_irqs[i];
1130 unsigned int virq = 0; 1130 int virq;
1131 1131
1132 virq = regmap_irq_get_virq(max77693->irq_data_muic, 1132 virq = regmap_irq_get_virq(max77693->irq_data_muic,
1133 muic_irq->irq); 1133 muic_irq->irq);
1134 if (!virq) 1134 if (virq <= 0)
1135 return -EINVAL; 1135 return -EINVAL;
1136 muic_irq->virq = virq; 1136 muic_irq->virq = virq;
1137 1137