@@ -720,13 +720,10 @@ static bool meson_hdmi_connector_is_available(struct device *dev)
/* If the endpoint node exists, consider it enabled */ remote = of_graph_get_remote_port(ep);
- if (remote) {
of_node_put(ep);
return true;
- }
- of_node_put(ep); of_node_put(remote);
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/dri...
Can the order of these put calls matter (because of processor caches)?
if (remote)
return true;
return false;
Would the use of a ternary operator be more succinct here?
+ return remote ? true : false;
Regards, Markus