Hi Matthieu, On Fri, Jul 12, 2013 at 03:56:02AM -0600, mathieu.poirier@xxxxxxxxxx wrote: > + > + np = of_find_node_by_path("/chosen"); > + if (!np) > + goto out; > + > + np = of_find_node_by_name(np, "linux,sysrq-reset-seq"); > + if (!np) { > + pr_debug("No sysrq node found"); > + goto out; > + } Can if be folded into: np = of_find_node_by_path("/chosen/linux,sysrq-reset-seq") ? Thanks. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html