Thread (7 messages) flat view 7 messages, 3 authors, 2015-09-22

[PATCH v4 1/4] edac: Export edac_debugfs

From: bp@alien8.de (Borislav Petkov)
Date: 2015-09-22 16:34:05
Also in: linux-devicetree

Possibly related (same subject, not in this thread)

On Fri, Aug 14, 2015 at 12:46:06AM -0600, Loc Ho wrote:
This patch exports and expose the edac_debugfs file node. This allows
EDAC driver to create debugfs node under the EDAC debugfs node.
...
-#ifdef CONFIG_EDAC_DEBUG
-static struct dentry *edac_debugfs;
+struct dentry *edac_debugfs;
+EXPORT_SYMBOL_GPL(edac_debugfs);
Hmm, so Arnd was right about this being the wrong direction we're
chasing. We should rather be exporting a bunch of EDAC debugfs wrappers
which EDAC drivers can call instead of exposing edac_debugfs itself.

And this is much better and the right(tm) approach IMO. So I went
and hacked it in, it builds on those !x86 targets (as far as my
cross-compilers are concerned, build succeeds :-)).

So here it is as a follow-up to this message.

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help