Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754253AbcK1IPN (ORCPT ); Mon, 28 Nov 2016 03:15:13 -0500 Received: from lelnx193.ext.ti.com ([198.47.27.77]:16861 "EHLO lelnx193.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751725AbcK1IPG (ORCPT ); Mon, 28 Nov 2016 03:15:06 -0500 Subject: Re: [PATCH v2 1/3] ARM: davinci: hawk: use gpio descriptor for mmc pins To: Axel Haslam , , References: <20161124150454.23899-1-ahaslam@baylibre.com> <20161124150454.23899-2-ahaslam@baylibre.com> CC: , From: Sekhar Nori Message-ID: <71d110ec-486b-5f26-1566-0fb9de93227e@ti.com> Date: Mon, 28 Nov 2016 13:44:38 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <20161124150454.23899-2-ahaslam@baylibre.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1179 Lines: 36 On Thursday 24 November 2016 08:34 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 using the mmc core > API's designed for this purpose. s/API's/APIs > > This has the advantage that an irq will be registered, and > polling is no longer needed. Also, a dependency on platform > callbacks is removed for this board. > > Signed-off-by: Axel Haslam > --- > arch/arm/mach-davinci/board-omapl138-hawk.c | 42 ++++++++--------------------- > 1 file changed, 11 insertions(+), 31 deletions(-) > > diff --git a/arch/arm/mach-davinci/board-omapl138-hawk.c b/arch/arm/mach-davinci/board-omapl138-hawk.c > index a4e8726..a2966d3 100644 > --- a/arch/arm/mach-davinci/board-omapl138-hawk.c > +++ b/arch/arm/mach-davinci/board-omapl138-hawk.c > @@ -15,6 +15,7 @@ > #include > #include > #include > +#include Moved this include to below linux/gpio.h Applied to v4.10/soc Thanks, Sekhar