From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756742Ab3KHJ1B (ORCPT ); Fri, 8 Nov 2013 04:27:01 -0500 Received: from mail-pd0-f173.google.com ([209.85.192.173]:38830 "EHLO mail-pd0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752822Ab3KHJ05 (ORCPT ); Fri, 8 Nov 2013 04:26:57 -0500 Date: Fri, 8 Nov 2013 09:26:51 +0000 From: Lee Jones To: Linus Walleij Cc: Ulf Hansson , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 3/7] ARM: ux500: Remove ATAG support for SDI (MMC) Message-ID: <20131108092651.GT30901@lee--X1> References: <1383732348-18576-1-git-send-email-lee.jones@linaro.org> <1383732348-18576-4-git-send-email-lee.jones@linaro.org> <20131107120742.GP30901@lee--X1> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 08 Nov 2013, Linus Walleij wrote: > On Thu, Nov 7, 2013 at 1:07 PM, Lee Jones wrote: > > >> > -struct mmci_platform_data mop500_sdi0_data = { > >> > - .f_max = 100000000, > >> > - .capabilities = MMC_CAP_4_BIT_DATA | > >> > - MMC_CAP_SD_HIGHSPEED | > >> > - MMC_CAP_MMC_HIGHSPEED | > >> > - MMC_CAP_ERASE | > >> > - MMC_CAP_UHS_SDR12 | > >> > - MMC_CAP_UHS_SDR25, > >> > >> I guess all the above caps is having binders in DT then? > > > > They're not there yet, no. But this stuff is not used anymore in > > Mainline hence the patch. I attempted to supply bindings for them, but > > they were rejected. Would you like to have a stab at that? > > Yeah but we basically regressed the ux500 with the patch now > queued for v3.13: > commit 49c129519a7a8840767321c38d2eaf84a263529b > "ARM: ux500: Stop passing MMC's platform data for Device Tree boots" > > Which deleted the passing of this non-yet-DT:ed platform > data. > > Since it's a regression we need to do a fixup patch for the > v3.13 series adding it back. Okay, do you want me to do that? > Since this patch probably needs to be rewritten on top of > that I don't merge this patch as of now. Sure. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog