Commit 6f9c8a13 authored by Krzysztof Kozlowski's avatar Krzysztof Kozlowski Committed by Sebastian Reichel

power: reset: syscon-poweroff: simplify pdev->dev usage

Make the probe() code a bit simpler and shorter by storing all the
'&pdev->dev' as 'dev'.
Signed-off-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/r/20230901120057.47018-2-krzysztof.kozlowski@linaro.orgSigned-off-by: default avatarSebastian Reichel <sebastian.reichel@collabora.com>
parent 71629338
...@@ -32,23 +32,24 @@ static void syscon_poweroff(void) ...@@ -32,23 +32,24 @@ static void syscon_poweroff(void)
static int syscon_poweroff_probe(struct platform_device *pdev) static int syscon_poweroff_probe(struct platform_device *pdev)
{ {
struct device *dev = &pdev->dev;
int mask_err, value_err; int mask_err, value_err;
map = syscon_regmap_lookup_by_phandle(pdev->dev.of_node, "regmap"); map = syscon_regmap_lookup_by_phandle(dev->of_node, "regmap");
if (IS_ERR(map)) { if (IS_ERR(map)) {
dev_err(&pdev->dev, "unable to get syscon"); dev_err(dev, "unable to get syscon");
return PTR_ERR(map); return PTR_ERR(map);
} }
if (of_property_read_u32(pdev->dev.of_node, "offset", &offset)) { if (of_property_read_u32(dev->of_node, "offset", &offset)) {
dev_err(&pdev->dev, "unable to read 'offset'"); dev_err(dev, "unable to read 'offset'");
return -EINVAL; return -EINVAL;
} }
value_err = of_property_read_u32(pdev->dev.of_node, "value", &value); value_err = of_property_read_u32(dev->of_node, "value", &value);
mask_err = of_property_read_u32(pdev->dev.of_node, "mask", &mask); mask_err = of_property_read_u32(dev->of_node, "mask", &mask);
if (value_err && mask_err) { if (value_err && mask_err) {
dev_err(&pdev->dev, "unable to read 'value' and 'mask'"); dev_err(dev, "unable to read 'value' and 'mask'");
return -EINVAL; return -EINVAL;
} }
...@@ -62,7 +63,7 @@ static int syscon_poweroff_probe(struct platform_device *pdev) ...@@ -62,7 +63,7 @@ static int syscon_poweroff_probe(struct platform_device *pdev)
} }
if (pm_power_off) { if (pm_power_off) {
dev_err(&pdev->dev, "pm_power_off already claimed for %ps", dev_err(dev, "pm_power_off already claimed for %ps",
pm_power_off); pm_power_off);
return -EBUSY; return -EBUSY;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment