From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754089Ab2LRBhQ (ORCPT ); Mon, 17 Dec 2012 20:37:16 -0500 Received: from mho-04-ewr.mailhop.org ([204.13.248.74]:61557 "EHLO mho-02-ewr.mailhop.org" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752817Ab2LRBhO (ORCPT ); Mon, 17 Dec 2012 20:37:14 -0500 X-Mail-Handler: Dyn Standard SMTP by Dyn X-Originating-IP: 72.84.113.162 X-Report-Abuse-To: abuse@dyndns.com (see http://www.dyndns.com/services/sendlabs/outbound_abuse.html for abuse reporting information) X-MHO-User: U2FsdGVkX1/700EHNIAhDNMm7DGTJ22Y8xjCrUoNsYo= Date: Mon, 17 Dec 2012 20:37:08 -0500 From: Jason Cooper To: Sebastian Hesselbarth Cc: Russell King , Andrew Lunn , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 3/3] ARM: Dove: add Cubox sdhci card detect gpio Message-ID: <20121218013708.GQ21694@titan.lakedaemon.net> References: <1355790269-10596-1-git-send-email-sebastian.hesselbarth@gmail.com> <1355790269-10596-3-git-send-email-sebastian.hesselbarth@gmail.com> <20121218003810.GP21694@titan.lakedaemon.net> <50CFBDC4.8040109@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <50CFBDC4.8040109@gmail.com> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Dec 18, 2012 at 01:50:12AM +0100, Sebastian Hesselbarth wrote: > On 12/18/2012 01:38 AM, Jason Cooper wrote: > > On Tue, Dec 18, 2012 at 01:24:29AM +0100, Sebastian Hesselbarth wrote: > >> Card detect for sdhci on Cubox is connected to the wrong pin > >> (sdio1_cd instead of sdio0_cd). With support for cd-gpios and > >> pinctrl add the corresponding properties to DT for Cubox. > >> > >> Signed-off-by: Sebastian Hesselbarth > >> ... > >> +&sdio0 { > >> + status = "okay"; > >> + /* sdio0 card detect is connected to wrong pin on CuBox */ > >> + cd-gpios =<&gpio0 12 1>; > > > > this looks like a fix that should be merged into v3.8, I'll cue it up > > for v3.8-rc2. > > Jason, > > it is more a feature than a fix, as gpio-cd support for sdhci-dove > moved in lately. You want me to split this patch from the other two > (that I'll reorder as requested)? I was referring to the code more than the physical feature itself. You appear to remove the default &sdio0, and replace it with a working version. Looked like a fix to me ;-) No need to send it separate, I'll apply it as is once v3.8-rc1 drops. thx, Jason.