From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932928AbdBHJZ4 (ORCPT ); Wed, 8 Feb 2017 04:25:56 -0500 Received: from mail-wm0-f52.google.com ([74.125.82.52]:36523 "EHLO mail-wm0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753941AbdBHJY3 (ORCPT ); Wed, 8 Feb 2017 04:24:29 -0500 Date: Wed, 8 Feb 2017 09:24:25 +0000 From: Lee Jones To: Stephen Rothwell , Greg KH Cc: Linus Walleij , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Boris Brezillon , Alexander Stein , Andy Shevchenko Subject: [PATCH] serial: st-asc: Use new GPIOD API to obtain RTS pin Message-ID: <20170208092425.azqkx3nwf4jhsfz5@dell> References: <20170208161113.1b597c01@canb.auug.org.au> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170208161113.1b597c01@canb.auug.org.au> User-Agent: Mutt/1.6.2-neo (2016-08-21) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The commits mentioned below adapt the GPIO API to allow more information to be passed directly through devm_get_gpiod_from_child() in the first instance. This facilitates the removal of subsequent calls, such as gpiod_direction_output(). This patch firstly moves to utilise the new API and secondly removes the now superfluous call do set the direction. Fixes: a264d10ff45c ("gpiolib: Convert fwnode_get_named_gpiod() to configure GPIO") Fixes: b2987d7438e0 ("gpio: Pass GPIO label down to gpiod_request") Fixes: 4b0947974e59 ("gpio: Rename devm_get_gpiod_from_child()") Reported-by: Stephen Rothwell Suggested-by: Boris Brezillon Signed-off-by: Lee Jones --- drivers/tty/serial/st-asc.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/tty/serial/st-asc.c b/drivers/tty/serial/st-asc.c index bcf1d33..c334bcc 100644 --- a/drivers/tty/serial/st-asc.c +++ b/drivers/tty/serial/st-asc.c @@ -575,12 +575,13 @@ static void asc_set_termios(struct uart_port *port, struct ktermios *termios, pinctrl_select_state(ascport->pinctrl, ascport->states[NO_HW_FLOWCTRL]); - gpiod = devm_get_gpiod_from_child(port->dev, "rts", - &np->fwnode); - if (!IS_ERR(gpiod)) { - gpiod_direction_output(gpiod, 0); + gpiod = devm_fwnode_get_gpiod_from_child(port->dev, + "rts", + &np->fwnode, + GPIOD_OUT_LOW, + np->name); + if (!IS_ERR(gpiod)) ascport->rts = gpiod; - } } } -- 2.9.3