diff options
| author | Dmitry Osipenko <[email protected]> | 2019-11-05 00:55:59 +0300 |
|---|---|---|
| committer | Chanwoo Choi <[email protected]> | 2019-11-06 12:04:00 +0900 |
| commit | dccdea01adf3b1413fbbb342128702741069cb6c (patch) | |
| tree | 5fcdaae9c4c21abbe73d5303bcd200dfb00a2302 | |
| parent | 1f125dee4feda21bca39ed7f1165198d96fca233 (diff) | |
PM / devfreq: tegra30: Change irq type to unsigned int
IRQ numbers are always positive, hence the corresponding variable should
be unsigned to keep types consistent. This is a minor change that cleans
up code a tad more.
Suggested-by: Thierry Reding <[email protected]>
Acked-by: MyungJoo Ham <[email protected]>
Reviewed-by: Chanwoo Choi <[email protected]>
Tested-by: Peter Geis <[email protected]>
Signed-off-by: Dmitry Osipenko <[email protected]>
Signed-off-by: Chanwoo Choi <[email protected]>
| -rw-r--r-- | drivers/devfreq/tegra30-devfreq.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/devfreq/tegra30-devfreq.c b/drivers/devfreq/tegra30-devfreq.c index a6ba75f4106d..a27300f40b0b 100644 --- a/drivers/devfreq/tegra30-devfreq.c +++ b/drivers/devfreq/tegra30-devfreq.c @@ -160,7 +160,7 @@ struct tegra_devfreq { struct tegra_devfreq_device devices[ARRAY_SIZE(actmon_device_configs)]; - int irq; + unsigned int irq; }; struct tegra_actmon_emc_ratio { @@ -618,12 +618,12 @@ static int tegra_devfreq_probe(struct platform_device *pdev) return PTR_ERR(tegra->emc_clock); } - tegra->irq = platform_get_irq(pdev, 0); - if (tegra->irq < 0) { - err = tegra->irq; + err = platform_get_irq(pdev, 0); + if (err < 0) { dev_err(&pdev->dev, "Failed to get IRQ: %d\n", err); return err; } + tegra->irq = err; reset_control_assert(tegra->reset); |