From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Wed, 23 Jan 2013 10:04:56 +0100 Subject: [U-Boot] [PATCH 9/9] mx23: Use PIO mode support for MMC In-Reply-To: References: <1358819250-31625-1-git-send-email-otavio@ossystems.com.br> <1358819250-31625-10-git-send-email-otavio@ossystems.com.br> <201301220250.47441.marex@denx.de> Message-ID: <50FFA7B8.1090709@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 22/01/2013 02:56, Otavio Salvador wrote: > On Mon, Jan 21, 2013 at 11:50 PM, Marek Vasut wrote: >> Dear Otavio Salvador, >> >> NAK. DMA works on MX28 and works in Linux on MX23, therefore the DMA is not >> broken on either. Thus there's no reason to introduce such "force_pio" stuff. > > Lack of time to work on it is one reason. I won't be able to work on > it for some days and I prefer a working solution than skip it for > another release. > > -- > Otavio Salvador O.S. Systems > E-mail: otavio at ossystems.com.br http://www.ossystems.com.br > Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br > Hi Otavio, hi Marek, let me say some words about this discussion. I agree that if there is no support for DMA for MX23, we could add support with PIO mode. However, there should be a very good reason, because as far as I understand, added support for DMA is valid for mxs platform. The solution introduced with this patchset lets also the door open for people that can use MX28 in PIO mode. I think (I have already seen in ML Marek's patchset for it) the way to fix DMA for mx23 and then add support for MMC is the rught way to do. Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================