Thread (12 messages) flat view 12 messages, 5 authors, 2008-01-02

Re: [PATCH 2/3] mpc82xx: Embedded Planet EP8248E support

From: Stephen Rothwell <hidden>
Date: 2007-12-13 00:56:09

On Wed, 12 Dec 2007 16:48:24 -0600 Scott Wood [off-list ref] wrote:
Stephen Rothwell wrote:
quoted
quoted
+	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().
Sorry, you are right, I didn't notice that it was global ...
BTW, is there any actual node removal done that justifies dealing with 
refcounts everywhere?
In the hotplug code.  So I guess it is not very relevant to you, but it
is a pain to retrofit this stuff if it does become important more widely.

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help