Commit d9350f21 authored by Aditya Pakki's avatar Aditya Pakki Committed by Sudeep Holla

firmware: arm_scmi: replace of_match_device->data with of_device_get_match_data()

of_match_device can return NULL if no matching device is found though
it's highly unlikely to happen in scmi_probe as it's called only if
a valid match is found.

However we can use of_device_get_match_data() instead of
of_match_device()->data to handle NULL pointer checks and return -EINVAL
in such a scenario.
Reviewed-by: default avatarSteven Price <steven.price@arm.com>
Signed-off-by: default avatarAditya Pakki <pakki001@umn.edu>
Signed-off-by: default avatarSudeep Holla <sudeep.holla@arm.com>
parent fa7fe29a
...@@ -796,7 +796,9 @@ static int scmi_probe(struct platform_device *pdev) ...@@ -796,7 +796,9 @@ static int scmi_probe(struct platform_device *pdev)
return -EINVAL; return -EINVAL;
} }
desc = of_match_device(scmi_of_match, dev)->data; desc = of_device_get_match_data(dev);
if (!desc)
return -EINVAL;
info = devm_kzalloc(dev, sizeof(*info), GFP_KERNEL); info = devm_kzalloc(dev, sizeof(*info), GFP_KERNEL);
if (!info) if (!info)
......
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