From patchwork Tue Sep 6 13:17:38 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Masahiro Yamada X-Patchwork-Id: 75516 Delivered-To: patch@linaro.org Received: by 10.140.106.11 with SMTP id d11csp545322qgf; Tue, 6 Sep 2016 06:20:01 -0700 (PDT) X-Received: by 10.194.223.73 with SMTP id qs9mr35618001wjc.188.1473168001478; Tue, 06 Sep 2016 06:20:01 -0700 (PDT) Return-Path: Received: from theia.denx.de (theia.denx.de. [85.214.87.163]) by mx.google.com with ESMTP id b195si1371557wmg.64.2016.09.06.06.20.00; Tue, 06 Sep 2016 06:20:01 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of u-boot-bounces@lists.denx.de designates 85.214.87.163 as permitted sender) client-ip=85.214.87.163; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@nifty.com; spf=pass (google.com: best guess record for domain of u-boot-bounces@lists.denx.de designates 85.214.87.163 as permitted sender) smtp.mailfrom=u-boot-bounces@lists.denx.de Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 315574BA5C; Tue, 6 Sep 2016 15:19:59 +0200 (CEST) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id g7AMMwQojpmO; Tue, 6 Sep 2016 15:19:59 +0200 (CEST) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 0A84BB383A; Tue, 6 Sep 2016 15:17:55 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id D503BB380E for ; Tue, 6 Sep 2016 15:17:03 +0200 (CEST) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id qejwSSd448ZP for ; Tue, 6 Sep 2016 15:17:03 +0200 (CEST) X-policyd-weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5 NOT_IN_BL_NJABL=-1.5 (only DNSBL check requested) Received: from conuserg-10.nifty.com (conuserg-10.nifty.com [210.131.2.77]) by theia.denx.de (Postfix) with ESMTPS id B64994BB1A for ; Tue, 6 Sep 2016 15:16:40 +0200 (CEST) Received: from beagle.diag.org (p14092-ipngnfx01kyoto.kyoto.ocn.ne.jp [153.142.97.92]) (authenticated) by conuserg-10.nifty.com with ESMTP id u86DFbiw013304; Tue, 6 Sep 2016 22:15:45 +0900 DKIM-Filter: OpenDKIM Filter v2.10.3 conuserg-10.nifty.com u86DFbiw013304 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nifty.com; s=dec2015msa; t=1473167746; bh=EN9X1aQeXurmL6pV8xHIfLeyslEqPXlTalJ0s0A6CCQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=yxkoazBCNfiYK17TCN3oik3FKoStOWBpnmilsn2XvdzJVRA/DpxIAL+M+rqLNNnKJ 8PuxcuUEuRDxF0xJQHtbkht93glP+2ooub1nOwG6lzUH6a5/TQCkEYMpGnVcUR36Xj FEbvcJVV46CdTCsVK5L1WL7ohu8dDA732ATFXtjXj66lCyCII3KVdMrjtH4Fwp0MO6 Ku0a1h/xTp+3s1Nn6OkDL0C8c3IaVqzeMmIL0zVV/sBS32lcR5oRPzUc5odt4WOP4s WcJCCc3v67wAzrYkJatpmq0KG0Km+h+MjyhHDI6BxpQpA9A5JuRIQxlzMlg8RNhmC0 tONRT8DYTjOPA== X-Nifty-SrcIP: [153.142.97.92] From: Masahiro Yamada To: u-boot@lists.denx.de Date: Tue, 6 Sep 2016 22:17:38 +0900 Message-Id: <1473167860-27465-8-git-send-email-yamada.masahiro@socionext.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1473167860-27465-1-git-send-email-yamada.masahiro@socionext.com> References: <1473167860-27465-1-git-send-email-yamada.masahiro@socionext.com> Cc: Marek Vasut , Chris Zankel , Stephen Warren , Huan Wang , Thomas Weber , Robert Baldyga , Angelo Dureghello , Stefan Roese Subject: [U-Boot] [PATCH v3 7/8] arch, board: squash lines for immediate return X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.15 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" Remove unneeded variables and assignments. Signed-off-by: Masahiro Yamada --- Changes in v3: - More fixes arch/arm/cpu/arm920t/imx/timer.c | 6 +----- arch/arm/cpu/armv7/am33xx/sys_info.c | 4 +--- arch/arm/cpu/sa1100/timer.c | 5 +---- arch/arm/mach-zynq/cpu.c | 8 ++------ arch/blackfin/cpu/interrupts.c | 5 +---- arch/m68k/lib/time.c | 4 +--- arch/powerpc/cpu/mpc512x/cpu.c | 6 +----- arch/powerpc/cpu/mpc83xx/cpu.c | 6 +----- arch/xtensa/lib/time.c | 5 +---- board/amcc/bamboo/bamboo.c | 6 +----- board/amcc/bubinga/bubinga.c | 5 +---- board/amcc/canyonlands/canyonlands.c | 6 +----- board/corscience/tricorder/tricorder-eeprom.c | 20 +++++--------------- board/freescale/common/zm7300.c | 4 +--- board/samsung/goni/goni.c | 8 +------- board/ti/omap5_uevm/evm.c | 5 +---- 16 files changed, 21 insertions(+), 82 deletions(-) -- 1.9.1 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot diff --git a/arch/arm/cpu/arm920t/imx/timer.c b/arch/arm/cpu/arm920t/imx/timer.c index b62558f..178422a 100644 --- a/arch/arm/cpu/arm920t/imx/timer.c +++ b/arch/arm/cpu/arm920t/imx/timer.c @@ -78,11 +78,7 @@ unsigned long long get_ticks(void) */ ulong get_tbclk (void) { - ulong tbclk; - - tbclk = CONFIG_SYS_HZ; - - return tbclk; + return CONFIG_SYS_HZ; } /* diff --git a/arch/arm/cpu/armv7/am33xx/sys_info.c b/arch/arm/cpu/armv7/am33xx/sys_info.c index 52a6824..f42eee1 100644 --- a/arch/arm/cpu/armv7/am33xx/sys_info.c +++ b/arch/arm/cpu/armv7/am33xx/sys_info.c @@ -65,9 +65,7 @@ u32 get_device_type(void) */ u32 get_sysboot_value(void) { - int mode; - mode = readl(&cstat->statusreg) & (SYSBOOT_MASK); - return mode; + return readl(&cstat->statusreg) & SYSBOOT_MASK; } #ifdef CONFIG_DISPLAY_CPUINFO diff --git a/arch/arm/cpu/sa1100/timer.c b/arch/arm/cpu/sa1100/timer.c index 0a0006b..90e2128 100644 --- a/arch/arm/cpu/sa1100/timer.c +++ b/arch/arm/cpu/sa1100/timer.c @@ -66,8 +66,5 @@ unsigned long long get_ticks(void) */ ulong get_tbclk (void) { - ulong tbclk; - - tbclk = CONFIG_SYS_HZ; - return tbclk; + return CONFIG_SYS_HZ; } diff --git a/arch/arm/mach-zynq/cpu.c b/arch/arm/mach-zynq/cpu.c index 914b1fe..ba9171e 100644 --- a/arch/arm/mach-zynq/cpu.c +++ b/arch/arm/mach-zynq/cpu.c @@ -43,12 +43,8 @@ int arch_cpu_init(void) unsigned int zynq_get_silicon_version(void) { - unsigned int ver; - - ver = (readl(&devcfg_base->mctrl) & - ZYNQ_SILICON_VER_MASK) >> ZYNQ_SILICON_VER_SHIFT; - - return ver; + return (readl(&devcfg_base->mctrl) & ZYNQ_SILICON_VER_MASK) + >> ZYNQ_SILICON_VER_SHIFT; } void reset_cpu(ulong addr) diff --git a/arch/blackfin/cpu/interrupts.c b/arch/blackfin/cpu/interrupts.c index 45c92c3..abb7dc1 100644 --- a/arch/blackfin/cpu/interrupts.c +++ b/arch/blackfin/cpu/interrupts.c @@ -47,10 +47,7 @@ unsigned long long get_ticks(void) */ ulong get_tbclk(void) { - ulong tbclk; - - tbclk = CONFIG_SYS_HZ; - return tbclk; + return CONFIG_SYS_HZ; } void enable_interrupts(void) diff --git a/arch/m68k/lib/time.c b/arch/m68k/lib/time.c index 3163354..cb90c83 100644 --- a/arch/m68k/lib/time.c +++ b/arch/m68k/lib/time.c @@ -192,7 +192,5 @@ unsigned long usec2ticks(unsigned long usec) */ ulong get_tbclk(void) { - ulong tbclk; - tbclk = CONFIG_SYS_HZ; - return tbclk; + return CONFIG_SYS_HZ; } diff --git a/arch/powerpc/cpu/mpc512x/cpu.c b/arch/powerpc/cpu/mpc512x/cpu.c index 8508e8d..4ee91e1 100644 --- a/arch/powerpc/cpu/mpc512x/cpu.c +++ b/arch/powerpc/cpu/mpc512x/cpu.c @@ -95,11 +95,7 @@ do_reset (cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[]) */ unsigned long get_tbclk (void) { - ulong tbclk; - - tbclk = (gd->bus_clk + 3L) / 4L; - - return tbclk; + return (gd->bus_clk + 3L) / 4L; } diff --git a/arch/powerpc/cpu/mpc83xx/cpu.c b/arch/powerpc/cpu/mpc83xx/cpu.c index 3809309..c87f0fd 100644 --- a/arch/powerpc/cpu/mpc83xx/cpu.c +++ b/arch/powerpc/cpu/mpc83xx/cpu.c @@ -173,11 +173,7 @@ do_reset (cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[]) unsigned long get_tbclk(void) { - ulong tbclk; - - tbclk = (gd->bus_clk + 3L) / 4L; - - return tbclk; + return (gd->bus_clk + 3L) / 4L; } diff --git a/arch/xtensa/lib/time.c b/arch/xtensa/lib/time.c index 1332072..915eb51 100644 --- a/arch/xtensa/lib/time.c +++ b/arch/xtensa/lib/time.c @@ -104,10 +104,7 @@ unsigned long long get_ticks(void) */ ulong get_tbclk(void) { - ulong tbclk; - - tbclk = CONFIG_SYS_HZ; - return tbclk; + return CONFIG_SYS_HZ; } #if XCHAL_HAVE_CCOUNT diff --git a/board/amcc/bamboo/bamboo.c b/board/amcc/bamboo/bamboo.c index c8d0963..2838f9a 100644 --- a/board/amcc/bamboo/bamboo.c +++ b/board/amcc/bamboo/bamboo.c @@ -438,11 +438,7 @@ int checkboard(void) phys_size_t initdram (int board_type) { - long dram_size; - - dram_size = spd_sdram(); - - return dram_size; + return spd_sdram(); } /*----------------------------------------------------------------------------+ diff --git a/board/amcc/bubinga/bubinga.c b/board/amcc/bubinga/bubinga.c index 5c1e071..9043de6 100644 --- a/board/amcc/bubinga/bubinga.c +++ b/board/amcc/bubinga/bubinga.c @@ -57,8 +57,5 @@ int checkboard(void) ------------------------------------------------------------------------- */ phys_size_t initdram(int board_type) { - long int ret; - - ret = spd_sdram(); - return ret; + return spd_sdram(); } diff --git a/board/amcc/canyonlands/canyonlands.c b/board/amcc/canyonlands/canyonlands.c index dc2e3ba..80b7739 100644 --- a/board/amcc/canyonlands/canyonlands.c +++ b/board/amcc/canyonlands/canyonlands.c @@ -63,11 +63,7 @@ u32 ddr_clktr(u32 default_val) { */ static inline int board_fpga_read(int offset) { - int data; - - data = in_8((void *)(CONFIG_SYS_FPGA_BASE + offset)); - - return data; + return in_8((void *)(CONFIG_SYS_FPGA_BASE + offset)); } static inline void board_fpga_write(int offset, int data) diff --git a/board/corscience/tricorder/tricorder-eeprom.c b/board/corscience/tricorder/tricorder-eeprom.c index 340a009..aeacd6a 100644 --- a/board/corscience/tricorder/tricorder-eeprom.c +++ b/board/corscience/tricorder/tricorder-eeprom.c @@ -190,13 +190,8 @@ int do_tricorder_eeprom(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[]) if (argc == 3) { ulong dev_addr = simple_strtoul(argv[2], NULL, 16); - if (strcmp(argv[1], "read") == 0) { - int rcode; - - rcode = tricorder_eeprom_read(dev_addr); - - return rcode; - } + if (strcmp(argv[1], "read") == 0) + return tricorder_eeprom_read(dev_addr); } else if (argc == 6 || argc == 7) { ulong dev_addr = simple_strtoul(argv[2], NULL, 16); char *name = argv[3]; @@ -207,14 +202,9 @@ int do_tricorder_eeprom(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[]) if (argc == 7) interface = argv[6]; - if (strcmp(argv[1], "write") == 0) { - int rcode; - - rcode = tricorder_eeprom_write(dev_addr, name, version, - serial, interface); - - return rcode; - } + if (strcmp(argv[1], "write") == 0) + return tricorder_eeprom_write(dev_addr, name, version, + serial, interface); } return CMD_RET_USAGE; diff --git a/board/freescale/common/zm7300.c b/board/freescale/common/zm7300.c index be5953a..a6c3e69 100644 --- a/board/freescale/common/zm7300.c +++ b/board/freescale/common/zm7300.c @@ -140,9 +140,7 @@ int dpm_wrp(u8 r, u8 d) /* Uses the DPM command RRP */ u8 zm_read(uchar reg) { - u8 d; - d = dpm_rrp(reg); - return d; + return dpm_rrp(reg); } /* ZM_write -- diff --git a/board/samsung/goni/goni.c b/board/samsung/goni/goni.c index 1600568..e8329bb 100644 --- a/board/samsung/goni/goni.c +++ b/board/samsung/goni/goni.c @@ -45,17 +45,11 @@ void i2c_init_board(void) int power_init_board(void) { - int ret; - /* * For PMIC the I2C bus is named as I2C5, but it is connected * to logical I2C adapter 0 */ - ret = pmic_init(I2C_0); - if (ret) - return ret; - - return 0; + return pmic_init(I2C_0); } int dram_init(void) diff --git a/board/ti/omap5_uevm/evm.c b/board/ti/omap5_uevm/evm.c index 50da410..b5d5ba9 100644 --- a/board/ti/omap5_uevm/evm.c +++ b/board/ti/omap5_uevm/evm.c @@ -245,10 +245,7 @@ int ehci_hcd_init(int index, enum usb_init_type init, int ehci_hcd_stop(void) { - int ret; - - ret = omap_ehci_hcd_stop(); - return ret; + return omap_ehci_hcd_stop(); } void usb_hub_reset_devices(int port)