Hi Balaji, > -----Original Message----- > From: Krishnamoorthy, Balaji T > Sent: Thursday, June 27, 2013 11:27 AM > To: Cousson, Benoit; Fernandes, Joel > Cc: lars@xxxxxxxxxx; Tony Lindgren; Nori, Sekhar; Matt Porter; Grant Likely; > Rob Herring; Vinod Koul; Mark Brown; Russell King; Rob Landley; Andrew > Morton; Jason Kridner; Koen Kooi; Chris Ball; Devicetree Discuss; Linux OMAP > List; Linux ARM Kernel List; Linux DaVinci Kernel List; Linux Kernel Mailing List; > Linux MMC List; Arnd Bergmann > Subject: Re: [PATCH 0/5] AM33xx: MMC resources from DT without HWMOD > data > > On Thursday 27 June 2013 07:56 PM, Joel A Fernandes wrote: > > Hi Benoit, > > > > On Thu, Jun 27, 2013 at 7:03 AM, Benoit Cousson <b-cousson@xxxxxx> wrote: > >> Hi Joel, > >> > >> On 06/26/2013 05:28 AM, Joel A Fernandes wrote: > >>> On Tue, Jun 25, 2013 at 8:23 PM, Joel A Fernandes <joelagnel@xxxxxx> > wrote: > >>>> From: Joel A Fernandes <agnel.joel@xxxxxxxxx> > >>>> > >>>> This series is fixes to get MMC working on AM33XX without HWMOD > data. > >>>> On removal of HWMOD data, interrupt and register properties need to > >>>> be provided for the driver to function correctly. > >>> > >>> Please don't pull patches authored by me in this series. I have > >>> posted them with my wrong email address by mistake. I will correct > >>> my email during the next repost, after a round of review. Matt's > >>> patches in the series are still good though. > >> > >> In that case, please repost these in two series to avoid pulling the > >> DTS files in the MMC tree. > > > > The series has been replaced with the single patch: > > "[PATCH v2] ARM: dts: add AM33XX MMC support" > > http://marc.info/?l=linux-omap&m=137230403815943&w=2 > > > > Some patches were squashed and others dropped in the series resulting > > in the single patch above. This patch should be good to apply > Hi Joel, > > Before pushing mmc dts support for am335x, Can you please let me > know if below 2 patches [1] are merged/accepted, > IIRC these patches are needed for MMC to work properly. > > I suggest to hold the dts patch until slave_sg_limit gets merged > otherwise it will cause regression in am335x mmc. > > [PATCH v4 1/3] dmaengine: add dma_get_slave_sg_limits() > [PATCH v4 2/3] dma: edma: add device_slave_sg_limits() support Actually this is an independent regression and is currently worked on by Lars-Peter Clausen [1]. Currently not having the MMC DTS merged is also a regression, so I suggest lets take it one step at a time and get the DTS merged in first. Also there will be overhead of reposting this DTS patch later again which will not change. What do you say? Thanks, -Joel [1] https://patchwork.kernel.org/patch/2228041/ -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
