diff options
author | Wolfram Sang <wsa+renesas@sang-engineering.com> | 2016-03-01 19:38:31 +0300 |
---|---|---|
committer | Eduardo Valentin <edubezval@gmail.com> | 2016-03-08 22:53:12 +0300 |
commit | 67bc3f4226c1d29ee820eff98fcaf8a9283938fe (patch) | |
tree | 2b7d9a4a0b84f7e296d99960c74736178ce0fd92 | |
parent | 527860fe4684bf69281d7479c5034243dd6c36c4 (diff) | |
download | linux-67bc3f4226c1d29ee820eff98fcaf8a9283938fe.tar.xz |
thermal: rcar_thermal: don't open code of_device_get_match_data()
This change will also make Coverity happy by avoiding a theoretical NULL
pointer dereference; yet another reason is to use the above helper function
to tighten the code and make it more readable.
Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
-rw-r--r-- | drivers/thermal/rcar_thermal.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/thermal/rcar_thermal.c b/drivers/thermal/rcar_thermal.c index 0e735acea33a..82daba09e150 100644 --- a/drivers/thermal/rcar_thermal.c +++ b/drivers/thermal/rcar_thermal.c @@ -430,8 +430,7 @@ static int rcar_thermal_probe(struct platform_device *pdev) struct rcar_thermal_priv *priv; struct device *dev = &pdev->dev; struct resource *res, *irq; - const struct of_device_id *of_id = of_match_device(rcar_thermal_dt_ids, dev); - unsigned long of_data = (unsigned long)of_id->data; + unsigned long of_data = (unsigned long)of_device_get_match_data(dev); int mres = 0; int i; int ret = -ENODEV; |