From mboxrd@z Thu Jan 1 00:00:00 1970 From: Balaji T K Subject: Re: [PATCH v11 4/7] mmc: omap_hsmmc: Extend debugfs by SDIO IRQ handling, runtime state Date: Fri, 9 May 2014 22:40:34 +0530 Message-ID: <536D0C0A.1000207@ti.com> References: <1399591018-12930-1-git-send-email-afenkart@gmail.com> <1399591234-13089-1-git-send-email-afenkart@gmail.com> <1399591234-13089-4-git-send-email-afenkart@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from comal.ext.ti.com ([198.47.26.152]:52984 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757256AbaEIRKo (ORCPT ); Fri, 9 May 2014 13:10:44 -0400 In-Reply-To: <1399591234-13089-4-git-send-email-afenkart@gmail.com> Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Andreas Fenkart Cc: Tony Lindgren , Chris Ball , Grant Likely , Felipe Balbi , Andreas Mueller , zonque@gmail.com, galak@codeaurora.org, linux-doc@vger.kernel.org, linux-mmc@vger.kernel.org, linux-omap@vger.kernel.org On Friday 09 May 2014 04:50 AM, Andreas Fenkart wrote: > Add SDIO IRQ entries to debugfs entry. Note that PSTATE shows current > state of data lines, incl. SDIO IRQ pending > > Signed-off-by: Andreas Fenkart > Looks good to me Acked-by: Balaji T K > diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c > index f76462d..14857d7 100644 > --- a/drivers/mmc/host/omap_hsmmc.c > +++ b/drivers/mmc/host/omap_hsmmc.c > @@ -83,6 +83,7 @@ static void apply_clk_hack(struct device *dev) > #define OMAP_HSMMC_RSP54 0x0118 > #define OMAP_HSMMC_RSP76 0x011C > #define OMAP_HSMMC_DATA 0x0120 > +#define OMAP_HSMMC_PSTATE 0x0124 > #define OMAP_HSMMC_HCTL 0x0128 > #define OMAP_HSMMC_SYSCTL 0x012C > #define OMAP_HSMMC_STAT 0x0130 > @@ -1854,14 +1855,29 @@ static int omap_hsmmc_regs_show(struct seq_file *s, void *data) > { > struct mmc_host *mmc = s->private; > struct omap_hsmmc_host *host = mmc_priv(mmc); > + bool suspended; > > - seq_printf(s, "mmc%d:\n ctx_loss:\t%d\n\nregs:\n", > - mmc->index, host->context_loss); > + seq_printf(s, "mmc%d:\n", mmc->index); > + seq_printf(s, "sdio irq mode\t%s\n", > + (mmc->caps & MMC_CAP_SDIO_IRQ) ? "interrupt" : "polling"); > > - pm_runtime_get_sync(host->dev); > + if (mmc->caps & MMC_CAP_SDIO_IRQ) { > + seq_printf(s, "sdio irq \t%s\n", > + (host->flags & HSMMC_SDIO_IRQ_ENABLED) ? "enabled" > + : "disabled"); > + } > + > + suspended = host->dev->power.runtime_status != RPM_ACTIVE; > + seq_printf(s, "runtime state\t%s\n", (suspended ? "idle" : "active")); > > + seq_printf(s, "ctx_loss:\t%d\n", host->context_loss); > + > + pm_runtime_get_sync(host->dev); > + seq_puts(s, "\nregs:\n"); > seq_printf(s, "CON:\t\t0x%08x\n", > OMAP_HSMMC_READ(host->base, CON)); > + seq_printf(s, "PSTATE:\t\t0x%08x\n", > + OMAP_HSMMC_READ(host->base, PSTATE)); > seq_printf(s, "HCTL:\t\t0x%08x\n", > OMAP_HSMMC_READ(host->base, HCTL)); > seq_printf(s, "SYSCTL:\t\t0x%08x\n", >