From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3CCC7C433EF for ; Mon, 13 Sep 2021 23:53:45 +0000 (UTC) Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 85AC66112E for ; Mon, 13 Sep 2021 23:53:44 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 85AC66112E Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id E2DD8838A6; Tue, 14 Sep 2021 01:53:42 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Received: by phobos.denx.de (Postfix, from userid 109) id 546B382D74; Tue, 14 Sep 2021 01:53:39 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by phobos.denx.de (Postfix) with ESMTP id 0837083903 for ; Tue, 14 Sep 2021 01:53:28 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=andre.przywara@arm.com Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 0582D6D; Mon, 13 Sep 2021 16:53:28 -0700 (PDT) Received: from slackpad.fritz.box (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 54F0B3F719; Mon, 13 Sep 2021 16:53:27 -0700 (PDT) Date: Tue, 14 Sep 2021 00:53:24 +0100 From: Andre Przywara To: Samuel Holland Cc: u-boot@lists.denx.de, Jagan Teki Subject: Re: [PATCH 2/3] sunxi: gpio: Remove name_to_gpio macro Message-ID: <20210914005324.69d2ed5a@slackpad.fritz.box> In-Reply-To: <20210911215049.5840-3-samuel@sholland.org> References: <20210911215049.5840-1-samuel@sholland.org> <20210911215049.5840-3-samuel@sholland.org> Organization: Arm Ltd. X-Mailer: Claws Mail 3.17.1 (GTK+ 2.24.31; x86_64-slackware-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean On Sat, 11 Sep 2021 16:50:48 -0500 Samuel Holland wrote: > This clarifies which callers must be updated to complete the DM_GPIO > conversion. > > The only remaining caller of name_to_gpio in generic code is inside the > !DM_GPIO block in cmd/gpio.c. DM_GPIO is always selected on sunxi, so > that code cannot be reached. And after this commit, there are only two > remaining implementations of name_to_gpio. > > Signed-off-by: Samuel Holland Compile tested on all sunxi boards. Acked-by: Andre Przywara Cheers, Andre > --- > > arch/arm/include/asm/arch-sunxi/gpio.h | 1 - > drivers/spi/spi-sunxi.c | 2 +- > drivers/video/Kconfig | 10 +++++----- > drivers/video/hitachi_tx18d42vm_lcd.c | 6 +++--- > drivers/video/sunxi/sunxi_display.c | 10 +++++----- > 5 files changed, 14 insertions(+), 15 deletions(-) > > diff --git a/arch/arm/include/asm/arch-sunxi/gpio.h b/arch/arm/include/asm/arch-sunxi/gpio.h > index 2969a530ae1..c93c051a19f 100644 > --- a/arch/arm/include/asm/arch-sunxi/gpio.h > +++ b/arch/arm/include/asm/arch-sunxi/gpio.h > @@ -244,7 +244,6 @@ int sunxi_gpio_set_drv(u32 pin, u32 val); > int sunxi_gpio_set_pull(u32 pin, u32 val); > int sunxi_name_to_gpio_bank(const char *name); > int sunxi_name_to_gpio(const char *name); > -#define name_to_gpio(name) sunxi_name_to_gpio(name) > > #if !defined CONFIG_SPL_BUILD && defined CONFIG_AXP_GPIO > int axp_gpio_init(void); > diff --git a/drivers/spi/spi-sunxi.c b/drivers/spi/spi-sunxi.c > index 4ca5d3a93ac..bc2f544e863 100644 > --- a/drivers/spi/spi-sunxi.c > +++ b/drivers/spi/spi-sunxi.c > @@ -245,7 +245,7 @@ static int sun4i_spi_parse_pins(struct udevice *dev) > break; > } > > - pin = name_to_gpio(pin_name); > + pin = sunxi_name_to_gpio(pin_name); > if (pin < 0) > break; > > diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig > index b1f8a9c1e62..b0c52a500df 100644 > --- a/drivers/video/Kconfig > +++ b/drivers/video/Kconfig > @@ -452,7 +452,7 @@ config VIDEO_LCD_SSD2828_RESET > default "" > ---help--- > The reset pin of SSD2828 chip. This takes a string in the format > - understood by 'name_to_gpio' function, e.g. PH1 for pin 1 of port H. > + understood by 'sunxi_name_to_gpio' function, e.g. PH1 for pin 1 of port H. > > config VIDEO_LCD_TDO_TL070WSH30 > bool "TDO TL070WSH30 DSI LCD panel support" > @@ -477,7 +477,7 @@ config VIDEO_LCD_SPI_CS > This is one of the SPI communication pins, involved in setting up a > working LCD configuration. The exact role of SPI may differ for > different hardware setups. The option takes a string in the format > - understood by 'name_to_gpio' function, e.g. PH1 for pin 1 of port H. > + understood by 'sunxi_name_to_gpio' function, e.g. PH1 for pin 1 of port H. > > config VIDEO_LCD_SPI_SCLK > string "SPI SCLK pin for LCD related config job" > @@ -487,7 +487,7 @@ config VIDEO_LCD_SPI_SCLK > This is one of the SPI communication pins, involved in setting up a > working LCD configuration. The exact role of SPI may differ for > different hardware setups. The option takes a string in the format > - understood by 'name_to_gpio' function, e.g. PH1 for pin 1 of port H. > + understood by 'sunxi_name_to_gpio' function, e.g. PH1 for pin 1 of port H. > > config VIDEO_LCD_SPI_MOSI > string "SPI MOSI pin for LCD related config job" > @@ -497,7 +497,7 @@ config VIDEO_LCD_SPI_MOSI > This is one of the SPI communication pins, involved in setting up a > working LCD configuration. The exact role of SPI may differ for > different hardware setups. The option takes a string in the format > - understood by 'name_to_gpio' function, e.g. PH1 for pin 1 of port H. > + understood by 'sunxi_name_to_gpio' function, e.g. PH1 for pin 1 of port H. > > config VIDEO_LCD_SPI_MISO > string "SPI MISO pin for LCD related config job (optional)" > @@ -509,7 +509,7 @@ config VIDEO_LCD_SPI_MISO > different hardware setups. If wired up, this pin may provide additional > useful functionality. Such as bi-directional communication with the > hardware and LCD panel id retrieval (if the panel can report it). The > - option takes a string in the format understood by 'name_to_gpio' > + option takes a string in the format understood by 'sunxi_name_to_gpio' > function, e.g. PH1 for pin 1 of port H. > > source "drivers/video/meson/Kconfig" > diff --git a/drivers/video/hitachi_tx18d42vm_lcd.c b/drivers/video/hitachi_tx18d42vm_lcd.c > index c6c8df6a96e..87c4d27438a 100644 > --- a/drivers/video/hitachi_tx18d42vm_lcd.c > +++ b/drivers/video/hitachi_tx18d42vm_lcd.c > @@ -49,9 +49,9 @@ int hitachi_tx18d42vm_init(void) > }; > int i, cs, clk, mosi, ret = 0; > > - cs = name_to_gpio(CONFIG_VIDEO_LCD_SPI_CS); > - clk = name_to_gpio(CONFIG_VIDEO_LCD_SPI_SCLK); > - mosi = name_to_gpio(CONFIG_VIDEO_LCD_SPI_MOSI); > + cs = sunxi_name_to_gpio(CONFIG_VIDEO_LCD_SPI_CS); > + clk = sunxi_name_to_gpio(CONFIG_VIDEO_LCD_SPI_SCLK); > + mosi = sunxi_name_to_gpio(CONFIG_VIDEO_LCD_SPI_MOSI); > > if (cs == -1 || clk == -1 || mosi == 1) { > printf("Error tx18d42vm spi gpio config is invalid\n"); > diff --git a/drivers/video/sunxi/sunxi_display.c b/drivers/video/sunxi/sunxi_display.c > index 128888f7af1..da3e8988296 100644 > --- a/drivers/video/sunxi/sunxi_display.c > +++ b/drivers/video/sunxi/sunxi_display.c > @@ -871,11 +871,11 @@ static void sunxi_vga_external_dac_enable(void) > static int sunxi_ssd2828_init(const struct ctfb_res_modes *mode) > { > struct ssd2828_config cfg = { > - .csx_pin = name_to_gpio(CONFIG_VIDEO_LCD_SPI_CS), > - .sck_pin = name_to_gpio(CONFIG_VIDEO_LCD_SPI_SCLK), > - .sdi_pin = name_to_gpio(CONFIG_VIDEO_LCD_SPI_MOSI), > - .sdo_pin = name_to_gpio(CONFIG_VIDEO_LCD_SPI_MISO), > - .reset_pin = name_to_gpio(CONFIG_VIDEO_LCD_SSD2828_RESET), > + .csx_pin = sunxi_name_to_gpio(CONFIG_VIDEO_LCD_SPI_CS), > + .sck_pin = sunxi_name_to_gpio(CONFIG_VIDEO_LCD_SPI_SCLK), > + .sdi_pin = sunxi_name_to_gpio(CONFIG_VIDEO_LCD_SPI_MOSI), > + .sdo_pin = sunxi_name_to_gpio(CONFIG_VIDEO_LCD_SPI_MISO), > + .reset_pin = sunxi_name_to_gpio(CONFIG_VIDEO_LCD_SSD2828_RESET), > .ssd2828_tx_clk_khz = CONFIG_VIDEO_LCD_SSD2828_TX_CLK * 1000, > .ssd2828_color_depth = 24, > #ifdef CONFIG_VIDEO_LCD_PANEL_MIPI_4_LANE_513_MBPS_VIA_SSD2828