From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932631AbcKVMuS (ORCPT ); Tue, 22 Nov 2016 07:50:18 -0500 Received: from mail-qt0-f176.google.com ([209.85.216.176]:36639 "EHLO mail-qt0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932369AbcKVMuQ (ORCPT ); Tue, 22 Nov 2016 07:50:16 -0500 MIME-Version: 1.0 In-Reply-To: <51a13cb8-082c-c6c8-c1a1-c39de252d022@ti.com> References: <20161121161541.27048-1-ahaslam@baylibre.com> <20161121161541.27048-4-ahaslam@baylibre.com> <51a13cb8-082c-c6c8-c1a1-c39de252d022@ti.com> From: Axel Haslam Date: Tue, 22 Nov 2016 13:49:35 +0100 Message-ID: Subject: Re: [PATCH 3/3] ARM: davinci: hawk: use gpio descriptor for card detect To: Sekhar Nori Cc: Kevin Hilman , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Nov 22, 2016 at 11:26 AM, Sekhar Nori wrote: > On Monday 21 November 2016 09:45 PM, Axel Haslam wrote: >> Currently the mmc driver is polling the gpio to know if the >> card was removed. >> >> By using a gpio descriptor instead of the platform callbacks, the >> driver will be able to register the gpio with the mmc core with API's >> designed for this purpose. >> >> This has the advantage that an irq will be registered, >> and polling is no longer needed. Also, platform callbacks can be removed. >> >> Signed-off-by: Axel Haslam > > This patch looks good, provided it is not based of 1/3 and 2/3. There > are other boards in mach-davinci using the card detect and wp callbacks. > And some like board-dm365-evm.c have those pins routed through a CPLD. > So I guess there is more work to be done before platform callbacks can > completely be removed from MMC/SD driver. But the closer we get, the > better it is :) > Yes, i saw the platform using CPLD for the pins. that might be tricky to remove. we can flag the mmc "broken card detect" which would force polling on the pin. im not sure if it is feasible to move the cpld pin handling to a driver.. (maybe a gpio driver?) > Thanks, > Sekhar >