From patchwork Thu Dec 21 12:28:42 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mark Brown X-Patchwork-Id: 10127605 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 04F0C60390 for ; Thu, 21 Dec 2017 12:29:35 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 14DF129B84 for ; Thu, 21 Dec 2017 12:29:35 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 06A0229BD6; Thu, 21 Dec 2017 12:29:35 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_MED autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 819ED29B84 for ; Thu, 21 Dec 2017 12:29:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Date:Message-Id: In-Reply-To:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: References:List-Owner; bh=aZCrXnyIoQB7Tp2N6dRwm6fTHzJ6tBYkgJECxBiQXPk=; b=S1d JX2EUa6pbIylnESFHhI0KWZWnx4wDYlWr3RlWzbrLrXljcmB2WxWvYC9sxYjEreB31y+c1GFuQb2G Kt/p1fcB42P0O87zHp0i0s/aAI9OKDR3u++PimOsG4PwPjfGTik/2V9qmU8LnXoobC3UQ52APQW5h rIdFlbrpPM1F4QNcinsEkA7KJ/1KGm94cDzeula/tWh87AkWsweRzyrGlWLMoPl1v1B0o+Q0BZij0 6prtMF3aNFUHqK1xS77MYDt9Cwt/dIPiDZ/7khakURsMm3lI5xvsIL67p1ht+2hkEd3sQwyVJ/oU3 ixivnHNn7l18lF5N4ckVbTz5kR3iZLg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.89 #1 (Red Hat Linux)) id 1eRzyd-0005Du-W0; Thu, 21 Dec 2017 12:29:32 +0000 Received: from merlin.infradead.org ([2001:8b0:10b:1231::1]) by bombadil.infradead.org with esmtps (Exim 4.89 #1 (Red Hat Linux)) id 1eRzyc-0005Dc-A9 for linux-arm-kernel@bombadil.infradead.org; Thu, 21 Dec 2017 12:29:30 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=merlin.20170209; h=Date:Message-Id:In-Reply-To:Subject:Cc: To:From:Sender:Reply-To:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:References:List-Id:List-Help: List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=H2eoxLR4EaVSUoWIq0LhhoNUVQW1WVkD2mZD+DxmfYI=; b=Hup8BrFxsHJ8cXDwKXKL6OqmFo KywsTQgCGpEXk7xLnApa+au01SsINvGsDvTc2qbDJTiAA+jikGb7EkmErRhcTxFVFZA884cc+pPb5 lBNiTe0g33lUJnZWPo8ousxHD96OKD+3C1fxSs6jugT8J2fkhUVhBNWopH9YThGegDY4R7OlS6yQx E5T3eOWFUyJlzFo5EsYTdssx+P2ITYDWT89b9sxPWjFy1zwG3jr4n3HO/sAWXCxtCu0VPDfWIGcAH uKztrWKuHnOm0ZQXJeKFD/VjpJB68FoHABwWURmSCXcY0hLbOjRtaxeLKuCQdBotu0CfQVKdU73hH P/IEuanA==; Received: from heliosphere.sirena.org.uk ([2a01:7e01::f03c:91ff:fed4:a3b6]) by merlin.infradead.org with esmtps (Exim 4.89 #1 (Red Hat Linux)) id 1eRzyZ-0004b8-6U for linux-arm-kernel@lists.infradead.org; Thu, 21 Dec 2017 12:29:28 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sirena.org.uk; s=20170815-heliosphere; h=Date:Message-Id:In-Reply-To: Subject:Cc:To:From:Sender:Reply-To:MIME-Version:Content-Type: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:References: List-Id:List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner: List-Archive; bh=H2eoxLR4EaVSUoWIq0LhhoNUVQW1WVkD2mZD+DxmfYI=; b=Zbsixco06JL6 lyWeht5AZcwJXZYPwDIlcE8P54EG3RuaR60C26377XYCe5zZR1IHh0GzM7jJwfwT8Toch6EKKFRpR aJO+zkC86dMxjO7j1+RdsXsc+qa+Xx934Pl8Umc6KKg6HTgK+JPVbsHm6AA5cow93cwyJRK06bh9R QzdEc=; Received: from cpc102320-sgyl38-2-0-cust46.18-2.cable.virginm.net ([82.37.168.47] helo=finisterre.ee.mobilebroadband) by heliosphere.sirena.org.uk with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1eRzxq-0006KL-JZ; Thu, 21 Dec 2017 12:28:42 +0000 Received: by finisterre.ee.mobilebroadband (Postfix, from userid 1000) id 367A744006B; Thu, 21 Dec 2017 12:28:42 +0000 (GMT) From: Mark Brown To: Rasmus Villemoes Subject: Applied "spi: pxa2xx: avoid redundant gpio_to_desc(desc_to_gpio()) round-trip" to the spi tree In-Reply-To: <20171221003731.21633-1-linux@rasmusvillemoes.dk> Message-Id: <20171221122842.367A744006B@finisterre.ee.mobilebroadband> Date: Thu, 21 Dec 2017 12:28:42 +0000 (GMT) X-Bad-Reply: In-Reply-To but no 'Re:' in Subject. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-kernel@vger.kernel.org, Haojian Zhuang , linux-spi@vger.kernel.org, Mark Brown , Daniel Mack , Robert Jarzmik , linux-arm-kernel@lists.infradead.org MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP The patch spi: pxa2xx: avoid redundant gpio_to_desc(desc_to_gpio()) round-trip has been applied to the spi tree at https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark From 221886646f75964ca31cf60f1811b2c9c4e965a5 Mon Sep 17 00:00:00 2001 From: Rasmus Villemoes Date: Thu, 21 Dec 2017 01:37:31 +0100 Subject: [PATCH] spi: pxa2xx: avoid redundant gpio_to_desc(desc_to_gpio()) round-trip gpio_free(gpio) simply does gpiod_free(gpio_to_desc(gpio)), so it's simpler and cleaner to use gpiod_free directly. Signed-off-by: Rasmus Villemoes Signed-off-by: Mark Brown --- drivers/spi/spi-pxa2xx.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/spi/spi-pxa2xx.c b/drivers/spi/spi-pxa2xx.c index 4cb515a3104c..c209dc1047b5 100644 --- a/drivers/spi/spi-pxa2xx.c +++ b/drivers/spi/spi-pxa2xx.c @@ -1237,7 +1237,7 @@ static int setup_cs(struct spi_device *spi, struct chip_data *chip, * different chip_info, release previously requested GPIO */ if (chip->gpiod_cs) { - gpio_free(desc_to_gpio(chip->gpiod_cs)); + gpiod_free(chip->gpiod_cs); chip->gpiod_cs = NULL; } @@ -1417,7 +1417,7 @@ static void cleanup(struct spi_device *spi) if (drv_data->ssp_type != CE4100_SSP && !drv_data->cs_gpiods && chip->gpiod_cs) - gpio_free(desc_to_gpio(chip->gpiod_cs)); + gpiod_free(chip->gpiod_cs); kfree(chip); }