Thread (7 messages) 7 messages, 3 authors, 2012-07-05
STALE5201d

[PATCH 2/2] ARM: mxs: defconfig: Enable CONFIG_COMMON_CLK_DEBUG

From: s.hauer@pengutronix.de (Sascha Hauer)
Date: 2012-07-05 06:25:40

On Thu, Jul 05, 2012 at 08:55:06AM +0800, Shawn Guo wrote:
On Mon, Jul 02, 2012 at 05:35:51PM -0300, Fabio Estevam wrote:
quoted
From: Fabio Estevam <redacted>

With the common clock framework in place, it is useful to enable
CONFIG_COMMON_CLK_DEBUG option, in order to easily inspect the clock tree.
I'm ambiguous on this.  It does look useful, but on the other hand,
I'm wondering why it hasn't been enabled by framework by default.

Sascha, what's your take on this?
I also like to have debugfs support enabled in the defconfigs. I don't
know if it's desirable to enable this unconditionally when
CONFIG_DEBUGFS is enabled because I think the clk debugfs support can
have some bad influence on performance.

I think Fabios patches are fine for nor. If we decide to enable clk
debugfs support later unconditionally than all we have is two unused
lines in the defconfigs.

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help