From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jagan Teki Date: Mon, 28 Aug 2017 14:44:24 +0530 Subject: [U-Boot] [PATCH 2/2] sunxi: Enable MMC new mode for A83T In-Reply-To: <20170828091022.ghqmb4bvnuzppble@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> <20170828091022.ghqmb4bvnuzppble@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:40 PM, Maxime Ripard wrote: > On Mon, Aug 28, 2017 at 02:30:23PM +0530, Jagan Teki wrote: >> 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. > > That argument doesn't really stand, any user is able to modify the > configuration themself, and experience that breakage. > > The fact that this is not enabled by default just make it less > exposed, but it's still a bug that can be encountered today in a > real-life situation. OK, will apply these 3 for the release. thanks! -- Jagan Teki Free Software Engineer | www.openedev.com U-Boot, Linux | Upstream Maintainer Hyderabad, India.