linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: mpa@pengutronix.de (Markus Pargmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/8] ASoC: fsl-ssi: Move sysfs stats to debugfs
Date: Mon, 25 Nov 2013 10:45:50 +0100	[thread overview]
Message-ID: <20131125094550.GA29656@pengutronix.de> (raw)
In-Reply-To: <20131122183356.GQ14725@sirena.org.uk>

Hi,

On Fri, Nov 22, 2013 at 06:33:56PM +0000, Mark Brown wrote:
> On Fri, Nov 22, 2013 at 05:42:04PM +0100, Markus Pargmann wrote:
> > On Thu, Nov 21, 2013 at 02:22:08PM +0000, Mark Brown wrote:
> > > On Wed, Nov 20, 2013 at 10:04:16AM +0100, Markus Pargmann wrote:
> 
> > > > This patch moves the statistics to debugfs and disables interrupts
> > > > completely if the kernel is build without DEBUG_FS enabled.
> 
> > > We can still see the error counts through /proc/interrupts even without
> > > the debugfs file.
> 
> > The interrupts are visible without the debugfs file when enabled through
> > SIER register. But it does not show the specific reason for the interrupts.
> 
> > I am not sure what you suggest. To remove the stats file completely and
> > always disable interrupts?
> 
> No, I'm suggesting not disabling interrupts completely when debugfs is
> disabled.  Since we can still see the interrupts happening even if
> debugfs is not enabled we're still getting diagnostics.

Okay, I will change it.

Thanks,

Markus


-- 
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 |

  reply	other threads:[~2013-11-25  9:45 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-20  9:04 [PATCH 0/8] ASoC: fsl-ssi: offline/online configuration and cleanups Markus Pargmann
2013-11-20  9:04 ` [PATCH 1/8] ASoC: fsl-ssi: Move ac97 specific setup to seperate function Markus Pargmann
2013-11-21 14:24   ` Mark Brown
2013-11-20  9:04 ` [PATCH 2/8] ASoC: fsl-ssi: Move sysfs stats to debugfs Markus Pargmann
2013-11-21 14:22   ` Mark Brown
2013-11-22 16:42     ` Markus Pargmann
2013-11-22 18:33       ` Mark Brown
2013-11-25  9:45         ` Markus Pargmann [this message]
2013-11-20  9:04 ` [PATCH 3/8] ASoC: fsl-ssi: Add imx50-ssi and of_device_id matching Markus Pargmann
2013-11-20 12:45   ` Shawn Guo
2013-11-20 13:30     ` Markus Pargmann
2013-11-20  9:04 ` [PATCH 4/8] ASoC: fsl-ssi: Add offline_config flag Markus Pargmann
2013-11-20  9:04 ` [PATCH 5/8] ASoC: fsl-ssi: Add configuration helper functions Markus Pargmann
2013-11-20  9:04 ` [PATCH 6/8] ASoC: fsl-ssi: Move RX/TX configuration to seperate functions Markus Pargmann
2013-11-20  9:04 ` [PATCH 7/8] ASoC: fsl-ssi: Drop ac97 specific trigger function Markus Pargmann
2013-11-20  9:04 ` [PATCH 8/8] ARM: DTS: imx5* imx6*, use imx50-ssi Markus Pargmann

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=20131125094550.GA29656@pengutronix.de \
    --to=mpa@pengutronix.de \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).