linux-kernel - RE: [PATCH 2/2] ARM: dts: add AM33XX MMC support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <79CD15C6BA57404B839C016229A409A83EC213C8@DBDE01.ent.ti.com>
Date:	Fri, 8 Mar 2013 12:29:02 +0000
From:	"Hiremath, Vaibhav" <hvaibhav@...com>
To:	"Porter, Matt" <mporter@...com>
CC:	Linux OMAP List <linux-omap@...r.kernel.org>,
	Russell King <linux@....linux.org.uk>,
	"Krishnamoorthy, Balaji T" <balajitk@...com>,
	Devicetree Discuss <devicetree-discuss@...ts.ozlabs.org>,
	Linux MMC List <linux-mmc@...r.kernel.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Chris Ball <cjb@...top.org>,
	Linux ARM Kernel List <linux-arm-kernel@...ts.infradead.org>
Subject: RE: [PATCH 2/2] ARM: dts: add AM33XX MMC support

> -----Original Message-----
> From: Matt Porter [mailto:ohiomdp@...il.com] On Behalf Of Porter, Matt
> Sent: Thursday, March 07, 2013 9:33 PM
> To: Hiremath, Vaibhav
> Cc: Linux OMAP List; Russell King; Krishnamoorthy, Balaji T; Devicetree
> Discuss; Linux MMC List; Linux Kernel Mailing List; Chris Ball; Linux
> ARM Kernel List
> Subject: Re: [PATCH 2/2] ARM: dts: add AM33XX MMC support
> 
> On Thu, Mar 07, 2013 at 03:50:01PM +0000, Vaibhav Hiremath wrote:
> >
> > > -----Original Message-----
> > > From: Matt Porter [mailto:ohiomdp@...il.com] On Behalf Of Porter,
> Matt
> > > Sent: Thursday, March 07, 2013 8:34 PM
> > > To: Hiremath, Vaibhav
> > > Cc: Chris Ball; Russell King; Krishnamoorthy, Balaji T; Devicetree
> > > Discuss; Linux MMC List; Linux Kernel Mailing List; Linux OMAP
> List;
> > > Linux ARM Kernel List
> > > Subject: Re: [PATCH 2/2] ARM: dts: add AM33XX MMC support
> > >
> > > On Thu, Mar 07, 2013 at 02:59:42PM +0000, Vaibhav Hiremath wrote:
> > > >
> > > > > -----Original Message-----
> > > > > From: Hiremath, Vaibhav
> > > > > Sent: Thursday, March 07, 2013 8:24 PM
> > > > > To: Porter, Matt
> > > > > Cc: Linux OMAP List; Russell King; Krishnamoorthy, Balaji T;
> > > Devicetree
> > > > > Discuss; Linux MMC List; Linux Kernel Mailing List; Chris Ball;
> > > Linux
> > > > > ARM Kernel List
> > > > > Subject: RE: [PATCH 2/2] ARM: dts: add AM33XX MMC support
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: Matt Porter [mailto:ohiomdp@...il.com] On Behalf Of
> Porter,
> > > > > Matt
> > > > > > Sent: Thursday, March 07, 2013 8:17 PM
> > > > > > To: Hiremath, Vaibhav
> > > > > > Cc: Linux OMAP List; Russell King; Krishnamoorthy, Balaji T;
> > > > > Devicetree
> > > > > > Discuss; Linux MMC List; Linux Kernel Mailing List; Chris
> Ball;
> > > Linux
> > > > > > ARM Kernel List
> > > > > > Subject: Re: [PATCH 2/2] ARM: dts: add AM33XX MMC support
> > > > > >
> > > > > > On Thu, Mar 07, 2013 at 02:39:55PM +0000, Vaibhav Hiremath
> wrote:
> > > > > > >
> > > > > > > > -----Original Message-----
> > > > > > > > From: Matt Porter [mailto:ohiomdp@...il.com] On Behalf Of
> > > Porter,
> > > > > > Matt
> > > > > > > > Sent: Thursday, March 07, 2013 7:43 PM
> > > > > > > > To: Hiremath, Vaibhav
> > > > > > > > Cc: Krishnamoorthy, Balaji T; Chris Ball; Cousson,
> Benoit;
> > > Tony
> > > > > > > > Lindgren; Russell King; Devicetree Discuss; Linux ARM
> Kernel
> > > > > List;
> > > > > > > > Linux OMAP List; Linux Kernel Mailing List; Linux MMC
> List
> > > > > > > > Subject: Re: [PATCH 2/2] ARM: dts: add AM33XX MMC support
> > > > > > > >
> > > > > > > > On Thu, Mar 07, 2013 at 05:29:24AM +0000, Vaibhav
> Hiremath
> > > wrote:
> > > > > > > > > > -----Original Message-----
> > > > > > > > > > From: linux-omap-owner@...r.kernel.org [mailto:linux-
> > > omap-
> > > > > > > > > > owner@...r.kernel.org] On Behalf Of Porter, Matt
> > > > > > > > > > Sent: Thursday, March 07, 2013 9:47 AM
> > > > > > > > > > To: Krishnamoorthy, Balaji T; Chris Ball; Cousson,
> > > Benoit;
> > > > > Tony
> > > > > > > > > > Lindgren; Russell King
> > > > > > > > > > Cc: Devicetree Discuss; Linux ARM Kernel List; Linux
> OMAP
> > > > > List;
> > > > > > > > Linux
> > > > > > > > > > Kernel Mailing List; Linux MMC List
> > > > > > > > > > Subject: [PATCH 2/2] ARM: dts: add AM33XX MMC support
> > > > > > > > > >
> > > > <snip>
> > > > >
> > > > > I believe you meant "CONFIG_TI_EDMA" right?
> > > > >
> > > > > Yes, I just enabled it and the result is still same.
> > > > >
> > > > >
> > > > >
> > > > > [root@...go /]# dmesg | grep -ir mmc
> > > > > [    0.506844] vmmc: 1800 <--> 3300 mV at 3300 mV
> > > > > [    0.506970] vmmc: supplied by vbat
> > > > > [root@...go /]#
> > > > > [root@...go /]#
> > > > > [root@...go /]# dmesg | grep -ir dma
> > > > > [    0.217063] DMA: preallocated 256 KiB pool for atomic
> coherent
> > > > > allocations
> > > > > [    0.236321] platform 49000000.edma: alias fck already exists
> > > > > [    0.236360] platform 49000000.edma: alias fck already exists
> > > > > [    0.236381] platform 49000000.edma: alias fck already exists
> > > > > [    0.370705] edma-dma-engine edma-dma-engine.0: TI EDMA DMA
> > > engine
> > > > > driver
> > > > > [    0.445156] omap-dma-engine omap-dma-engine: OMAP DMA engine
> > > driver
> > > > > [root@...go /]#
> > > > > [root@...go /]#
> > > > >
> > > > >
> > > > I have applied below patches from your recent post
> > > >
> > > >
> > > > [2/2] ARM: dts: add AM33XX MMC support
> > > > [1/2] mmc: omap_hsmmc: set max_segs based on dma engine limits
> > > > [v4,3/3] mmc: davinci: get SG segment limits with
> > > dma_get_slave_sg_limits()
> > > > [v4,2/3] dma: edma: add device_slave_sg_limits() support
> > > > [v4,1/3] dmaengine: add dma_get_slave_sg_limits()
> > > > [v9,9/9] ARM: dts: add AM33XX SPI DMA support
> > > > [v9,8/9] spi: omap2-mcspi: add generic DMA request support to the
> DT
> > > binding
> > > > [v9,7/9] spi: omap2-mcspi: convert to
> > > dma_request_slave_channel_compat()
> > > > [v9,6/9] ARM: dts: add AM33XX EDMA support
> > > > [v9,5/9] dmaengine: edma: Add TI EDMA device tree binding
> > > > [v9,4/9] dmaengine: edma: enable build for AM33XX
> > > > [v9,3/9] ARM: edma: add AM33XX support to the private EDMA API
> > > > [v9,2/9] ARM: edma: remove unused transfer controller handlers
> > > > [v9,1/9] ARM: davinci: move private EDMA API to arm/common
> > > > [v3,2/2] mmc: omap_hsmmc: add generic DMA request support to the
> DT
> > > binding
> > > > [v3,1/2] mmc: omap_hsmmc: convert to
> > > dma_request_slave_channel_compat()
> > > >
> > > >
> > > >
> > > > Am I missing anything here?
> > >
> > > Yes, you missed the
> > > http://www.spinics.net/lists/arm-kernel/msg227886.html dependency
> > > mentioned first in the cover letter.
> > >
> >
> > Matt, I manually edited the file with above patch and result is still
> the same.
> > Can you point me to branch where you have tested MMC code?
> 
> git://github.com/ohporter/linux.git edma-dmaengine-am33xx-mmc-v1
> 
> omap2plus_defconfig + CONFIG_TI_EDMA
> 
> I just doublechecked MMC rootfs on bone and evmsk as it's the standard
> smoke test. My EVM is intermittent now so trying to coax it to power up
> to reverify.
> 

Matt,

Your branch is working for me, I tested it on EVM. Not sure what is wrong with manual rebasing
I did yesterday. I will try to nail-down root-cause here.

And you can submit the next version fixing the comments I have given.

Thanks,
Vaibhav
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/