From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Wed, 23 Jan 2013 12:39:42 +0100 Subject: [U-Boot] [PATCH 9/9] mx23: Use PIO mode support for MMC In-Reply-To: <201301231226.58545.marex@denx.de> References: <1358819250-31625-1-git-send-email-otavio@ossystems.com.br> <50FFA7B8.1090709@denx.de> <201301231226.58545.marex@denx.de> Message-ID: <50FFCBFE.3040908@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 23/01/2013 12:26, Marek Vasut wrote: >>> 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. >> >> So you'd like to have this patch ported to Marek's patchset so the >> support for PIO mode is kept available? Instead of enabling it for >> MX23 we can have a config option which allows force PIO mode and >> disable DMA. This is what you want? > > Forcing PIO is nonsense, why would it be of any use? No, I agree with Marek - if DMA is then working also for MX23, we do not need PIO at all. 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 =====================================================================