Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756516Ab3HMBHU (ORCPT ); Mon, 12 Aug 2013 21:07:20 -0400 Received: from arroyo.ext.ti.com ([192.94.94.40]:43952 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756326Ab3HMBHT (ORCPT ); Mon, 12 Aug 2013 21:07:19 -0400 Message-ID: <5209869C.4020407@ti.com> Date: Mon, 12 Aug 2013 20:06:36 -0500 From: Joel Fernandes Reply-To: User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130623 Thunderbird/17.0.7 MIME-Version: 1.0 To: Sekhar Nori CC: Santosh Shilimkar , Sricharan R , Rajendra Nayak , Lokesh Vutla , Matt Porter , Vinod Koul , Dan Williams , Russell King , Linux OMAP List , Linux ARM Kernel List , Linux DaVinci Kernel List , Linux Kernel Mailing List , Linux MMC List Subject: Re: [PATCH v3 07/12] dma: edma: Add function to dump a PaRAM set from PaRAM References: <1375719297-12871-1-git-send-email-joelf@ti.com> <1375719297-12871-8-git-send-email-joelf@ti.com> <52087824.60002@ti.com> In-Reply-To: <52087824.60002@ti.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2560 Lines: 82 On 08/12/2013 12:52 AM, Sekhar Nori wrote: > On Monday 05 August 2013 09:44 PM, Joel Fernandes wrote: >> Previously, such a dump function was used but it wasn't reading >> from the PaRAM, rather just from a edmacc_param structure, we >> add a helpful function for debugging that directly reads from >> the PaRAM and gives the current state correctly (including links >> and interrupt information). >> >> Signed-off-by: Joel Fernandes > > You should convert existing instances of PaRAM set dump to use this new > function along with introducing it. Well, the old dump callers were thrown out with completely rewritten code. This rewritten code already contains the dump calls. Are you saying pull out those dump pset calls into a separate patch? Thanks, -Joel > Sekhar > >> --- >> drivers/dma/edma.c | 31 +++++++++++++++++++++++++++++++ >> 1 file changed, 31 insertions(+) >> >> diff --git a/drivers/dma/edma.c b/drivers/dma/edma.c >> index 080d669..a242269 100644 >> --- a/drivers/dma/edma.c >> +++ b/drivers/dma/edma.c >> @@ -102,6 +102,37 @@ static void edma_desc_free(struct virt_dma_desc *vdesc) >> kfree(container_of(vdesc, struct edma_desc, vdesc)); >> } >> >> +static inline void dump_pset(struct edma_chan *echan, int slot, >> + struct edmacc_param *pset_unused, int pset_idx) >> +{ >> + struct edmacc_param pset_local, *pset; >> + pset = &pset_local; >> + >> + edma_read_slot(slot, pset); >> + >> + dev_dbg(echan->vchan.chan.device->dev, >> + "\n pset[%d]:\n" >> + " chnum\t%d\n" >> + " slot\t%d\n" >> + " opt\t%08x\n" >> + " src\t%08x\n" >> + " dst\t%08x\n" >> + " abcnt\t%08x\n" >> + " ccnt\t%08x\n" >> + " bidx\t%08x\n" >> + " cidx\t%08x\n" >> + " lkrld\t%08x\n", >> + pset_idx, echan->ch_num, slot, >> + pset[0].opt, >> + pset[0].src, >> + pset[0].dst, >> + pset[0].a_b_cnt, >> + pset[0].ccnt, >> + pset[0].src_dst_bidx, >> + pset[0].src_dst_cidx, >> + pset[0].link_bcntrld); >> +} >> + >> /* Dispatch a queued descriptor to the controller (caller holds lock) */ >> static void edma_execute(struct edma_chan *echan) >> { >> > > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/