arch, board: squash lines for immediate return

Remove unneeded variables and assignments.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Reviewed-by: Minkyu Kang <mk7.kang@samsung.com>
Reviewed-by: Angelo Dureghello <angelo@sysam.it>

diff --git a/arch/arm/cpu/arm920t/imx/timer.c b/arch/arm/cpu/arm920t/imx/timer.c
index b62558f92f47ae7b6a87c43955cd5c027379de15..178422a9bf366e0bf81294481fc7c61ab3f76855 100644 (file)
--- 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 52a6824cf592187dbb74d66bd8c41c17798f61b2..f42eee166b6c44634089378ee0c64b1ce5488c5c 100644 (file)
--- 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 0a0006b426323d4162ea73900137cad3541051c2..90e2128b22ee3ca95d3cc0b169dd9bc7fdd04492 100644 (file)
--- 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 914b1feb6833310144d229c307b8130218d00f02..ba9171ebe9f94c1ba455178ccc7030e70d027358 100644 (file)
--- 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 45c92c3d3e703a227d53b8c8fde648038c88b211..abb7dc1d1ddfb0693ae5e33b6a01da0b61d161bf 100644 (file)
--- 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 31633543e093143bb1dd2ee95d3f96bb2d35b0ae..cb90c83f4e4ab84e2f581df047606e8a4b520b18 100644 (file)
--- 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 8508e8db40eeacb4afc636f9398739c6522cf441..4ee91e16f9c689efadf5c82d7760f290a817322c 100644 (file)
--- 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 38093097e47984b52df44c8fb9e47c4f5b8bb25c..c87f0fdd29f58e0be1b222b13623f3f050c40bdc 100644 (file)
--- 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 1332072ffe3d01f0761170f1a28e2aa3bdf83996..915eb5185b6c7e75578b42571d7212734c46c2a5 100644 (file)
--- 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 c8d09636abceba8cf25e1ddc36af3df43423fbd7..2838f9a1edd39ab708c459e15ad020d2ab1b0a49 100644 (file)
--- 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 5c1e0717a54544e2c133cb79f3e059d901135dc1..9043de62d36d8d7c26b9d471850b629494e44ce6 100644 (file)
--- 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 dc2e3ba3a08d0d7b3b9c73c2791a23fc52370a90..80b77398c467eeccd254b4310a03b3af15ef1ff1 100644 (file)
--- 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 340a009c895af92133c764dd13c6c84a34666260..aeacd6a253a51f2f804bbb5074f00ad2228c4e5c 100644 (file)
--- 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 be5953ad2d8000f306e0f2fa9c4d6196e48bd266..a6c3e699b811badd629774a55badf9b299b7fe12 100644 (file)
--- 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 1600568193ab194ac6e88ea8a8ed26799e5271f6..e8329bba6cbbfbe2b10d97e80373a38dde22918f 100644 (file)
--- 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 50da4109bb8bd4c08f30029ddfa1dabccfcb669f..b5d5ba9bdf9b6aabedada02236a710fd70fa5dc5 100644 (file)
--- 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)