Stephen Rothwell wrote: >> + ep8248e_bcsr = of_iomap(ep8248e_bcsr_node, 0); > > of_node_put(ep8248e_bcsr_node; No, because we hold on to the node for later use in ep8248e_mdio_probe(). BTW, is there any actual node removal done that justifies dealing with refcounts everywhere? -Scot