From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 0/6] clk: Initial feedback for off-SoC slow bus clocks
Date: Mon, 11 Jul 2011 11:43:06 +0000 [thread overview]
Message-ID: <20110711114304.GJ5092@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <20110711104625.GH26347@pengutronix.de>
On Mon, Jul 11, 2011 at 12:46:25PM +0200, Sascha Hauer wrote:
> On Mon, Jul 11, 2011 at 11:28:56AM +0100, Russell King - ARM Linux wrote:
> > If that's all its for, then can't some other solution for debugfs names
> > (lets stop calling them clock names to avoid confusion) be found -
> > such as
> > sprintf(debugfsname, "%s@%p", clk->debugfsprefix, clk);
> Sounds good. During my tests on i.MX I found it very convenient to have
> speaking debugfs names, so I vote for this suggestion instead the
> following ones.
FWIW that's pretty much exactly what I added except I used a - instead
of @.
WARNING: multiple messages have this Message-ID (diff)
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Russell King - ARM Linux <linux@arm.linux.org.uk>,
Grant Likely <grant@secretlab.ca>,
patches@opensource.wolfsonmicro.com, linux-sh@vger.kernel.org,
linux-kernel@vger.kernel.org,
Jeremy Kerr <jeremy.kerr@canonical.com>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 0/6] clk: Initial feedback for off-SoC slow bus clocks
Date: Mon, 11 Jul 2011 20:43:06 +0900 [thread overview]
Message-ID: <20110711114304.GJ5092@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <20110711104625.GH26347@pengutronix.de>
On Mon, Jul 11, 2011 at 12:46:25PM +0200, Sascha Hauer wrote:
> On Mon, Jul 11, 2011 at 11:28:56AM +0100, Russell King - ARM Linux wrote:
> > If that's all its for, then can't some other solution for debugfs names
> > (lets stop calling them clock names to avoid confusion) be found -
> > such as
> > sprintf(debugfsname, "%s@%p", clk->debugfsprefix, clk);
> Sounds good. During my tests on i.MX I found it very convenient to have
> speaking debugfs names, so I vote for this suggestion instead the
> following ones.
FWIW that's pretty much exactly what I added except I used a - instead
of @.
WARNING: multiple messages have this Message-ID (diff)
From: broonie@opensource.wolfsonmicro.com (Mark Brown)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/6] clk: Initial feedback for off-SoC slow bus clocks
Date: Mon, 11 Jul 2011 20:43:06 +0900 [thread overview]
Message-ID: <20110711114304.GJ5092@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <20110711104625.GH26347@pengutronix.de>
On Mon, Jul 11, 2011 at 12:46:25PM +0200, Sascha Hauer wrote:
> On Mon, Jul 11, 2011 at 11:28:56AM +0100, Russell King - ARM Linux wrote:
> > If that's all its for, then can't some other solution for debugfs names
> > (lets stop calling them clock names to avoid confusion) be found -
> > such as
> > sprintf(debugfsname, "%s@%p", clk->debugfsprefix, clk);
> Sounds good. During my tests on i.MX I found it very convenient to have
> speaking debugfs names, so I vote for this suggestion instead the
> following ones.
FWIW that's pretty much exactly what I added except I used a - instead
of @.
next prev parent reply other threads:[~2011-07-11 11:43 UTC|newest]
Thread overview: 72+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-11 2:53 [PATCH 0/6] clk: Initial feedback for off-SoC slow bus clocks Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-11 2:53 ` [PATCH 1/6] clk: Prototype and document clk_register() Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-11 2:53 ` [PATCH 2/6] clk: Provide a dummy clk_unregister() Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-11 2:53 ` [PATCH 3/6] clk: Constify struct clk_hw_ops Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-11 2:53 ` [PATCH 4/6] clk: Add Kconfig option to build all generic clk drivers Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-11 2:53 ` [PATCH 5/6] clk: Support multiple instances of the same clock provider Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-11 9:34 ` [PATCH 5/6] clk: Support multiple instances of the same clock Russell King - ARM Linux
2011-07-11 9:34 ` [PATCH 5/6] clk: Support multiple instances of the same clock provider Russell King - ARM Linux
2011-07-11 9:34 ` Russell King - ARM Linux
2011-07-11 10:53 ` [PATCH 5/6] clk: Support multiple instances of the same clock Mark Brown
2011-07-11 10:53 ` [PATCH 5/6] clk: Support multiple instances of the same clock provider Mark Brown
2011-07-11 10:53 ` Mark Brown
2011-07-11 11:11 ` [PATCH 5/6] clk: Support multiple instances of the same clock Russell King - ARM Linux
2011-07-11 11:11 ` [PATCH 5/6] clk: Support multiple instances of the same clock provider Russell King - ARM Linux
2011-07-11 11:11 ` Russell King - ARM Linux
2011-07-11 11:41 ` [PATCH 5/6] clk: Support multiple instances of the same clock Mark Brown
2011-07-11 11:41 ` [PATCH 5/6] clk: Support multiple instances of the same clock provider Mark Brown
2011-07-11 11:41 ` Mark Brown
2011-07-11 2:53 ` [PATCH 6/6] clk: Add initial WM831x clock driver Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-11 2:53 ` Mark Brown
2011-07-15 2:53 ` Grant Likely
2011-07-15 2:53 ` Grant Likely
2011-07-15 2:53 ` Grant Likely
2011-07-15 5:05 ` Mark Brown
2011-07-15 5:05 ` Mark Brown
2011-07-15 5:05 ` Mark Brown
2011-07-15 5:14 ` Ryan Mallon
2011-07-15 5:14 ` Ryan Mallon
2011-07-15 5:14 ` Ryan Mallon
2011-07-15 2:53 ` [PATCH 1/6] clk: Prototype and document clk_register() Grant Likely
2011-07-15 2:53 ` Grant Likely
2011-07-15 2:53 ` Grant Likely
2011-07-11 3:57 ` [PATCH 0/6] clk: Initial feedback for off-SoC slow bus clocks Mark Brown
2011-07-11 3:57 ` Mark Brown
2011-07-11 3:57 ` Mark Brown
2011-07-11 4:30 ` Mike Frysinger
2011-07-11 4:30 ` Mike Frysinger
2011-07-11 4:30 ` Mike Frysinger
2011-07-11 4:56 ` Barry Song
2011-07-11 4:56 ` Barry Song
2011-07-11 4:56 ` Barry Song
2011-07-11 5:01 ` [uclinux-dist-devel] [PATCH 0/6] clk: Initial feedback for Mike Frysinger
2011-07-11 5:01 ` [uclinux-dist-devel] [PATCH 0/6] clk: Initial feedback for off-SoC slow bus clocks Mike Frysinger
2011-07-11 5:01 ` Mike Frysinger
2011-07-11 9:31 ` Russell King - ARM Linux
2011-07-11 9:31 ` Russell King - ARM Linux
2011-07-11 9:31 ` Russell King - ARM Linux
2011-07-11 10:07 ` Sascha Hauer
2011-07-11 10:07 ` Sascha Hauer
2011-07-11 10:07 ` Sascha Hauer
2011-07-11 10:28 ` Russell King - ARM Linux
2011-07-11 10:28 ` Russell King - ARM Linux
2011-07-11 10:28 ` Russell King - ARM Linux
2011-07-11 10:46 ` Sascha Hauer
2011-07-11 10:46 ` Sascha Hauer
2011-07-11 10:46 ` Sascha Hauer
2011-07-11 11:43 ` Mark Brown [this message]
2011-07-11 11:43 ` Mark Brown
2011-07-11 11:43 ` Mark Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110711114304.GJ5092@opensource.wolfsonmicro.com \
--to=broonie@opensource.wolfsonmicro.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.