From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jagan Teki Date: Mon, 28 Aug 2017 14:30:23 +0530 Subject: [U-Boot] [PATCH 2/2] sunxi: Enable MMC new mode for A83T In-Reply-To: <20170828084502.qxh36hnofxahuwja@flea.lan> References: <20170823100342.26148-1-maxime.ripard@free-electrons.com> <20170823100342.26148-2-maxime.ripard@free-electrons.com> <20170828065945.7yaffhlbdgxkshh4@flea.lan> <20170828084502.qxh36hnofxahuwja@flea.lan> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Mon, Aug 28, 2017 at 2:15 PM, Maxime Ripard wrote: > On Mon, Aug 28, 2017 at 04:12:15PM +0800, Chen-Yu Tsai wrote: >> On Mon, Aug 28, 2017 at 4:05 PM, Jagan Teki wrote: >> > On Mon, Aug 28, 2017 at 12:29 PM, Maxime Ripard >> > wrote: >> >> Hi Jagan, >> >> >> >> On Sat, Aug 26, 2017 at 12:11:23PM +0530, Jagan Teki wrote: >> >>> On Wed, Aug 23, 2017 at 3:33 PM, Maxime Ripard >> >>> wrote: >> >>> > The eMMC controller for the A83T uses the new operating mode. Enable it. >> >>> > >> >>> > Signed-off-by: Maxime Ripard >> >>> >> >>> Reviewed-by: Jagan Teki >> >> >> >> I think both these patches should be merged in the upcoming >> >> release. It's a fix for a usecase that already exists (we already >> >> enable the eMMC on A83T), and it's unusable on some boards without it. >> > >> > But none of A83T boards were not enable eMMC yet? even sun8i-a83t.dtsi >> > still need to add node. May be we can wait once all add? >> >> The sunxi-mmc driver does not use the device tree. >> >> One only needs to set CONFIG_MMC_SUNXI_SLOT_EXTRA=2 to use the eMMC. >> And yes, eMMC is broken on some boards unless the new timing mode is >> used. > > And the support is broken until "mmc: sunxi: fix legacy MMC > initialisation" in my other serie is applied. With these patches, eMMC detected BPI-M3. But none of A83T boards have CONFIG_MMC_SUNXI_SLOT_EXTRA=2 So it's better to take this once boards added. thanks! -- Jagan Teki Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream Maintainer Hyderabad, India.