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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id B10DCC433FE for ; Wed, 26 Oct 2022 04:54:29 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 9771584F46; Wed, 26 Oct 2022 06:54:27 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=denx.de Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1666760067; bh=nLmsS03iUnx+seslzgzTnIXEf7Y/RYWKyPYg7/Ep1Z0=; h=Date:Subject:To:Cc:References:From:In-Reply-To:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=x/V1Efo5qJFnYlsXdqu2djyPEb+cahVbySY5sLIGu7RTwXk/G/gP22DrN69fUk10d QnA5LBCcmGTE8+2l71MYUcGpCYb7SAH0RYfxtAjxmYWhhid6f4Plipw4PF86B6HN1X QLYMEM+yRvagLQkzDqhbm/aEvA5wngijlOMby+eR2Ht14ub5JQ+Ld/W7wNJPPrJI6L 6+5UMITHEmIiNIR/eyZGYjnoL2KBdO6KmWwGfLTQGB2YbqnZh+4xBt0oTcbC97uWiJ qm54xekQQ+/UPe+KKFh2kG95ZBgDLbPY9XnafI6i4h++Gy1cOFXNUG8bkwubRzzw7E WOXmfSagZ+FWg== Received: by phobos.denx.de (Postfix, from userid 109) id A218A84F8A; Wed, 26 Oct 2022 06:54:25 +0200 (CEST) Received: from mout-u-204.mailbox.org (mout-u-204.mailbox.org [IPv6:2001:67c:2050:101:465::204]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 463ED84F46 for ; Wed, 26 Oct 2022 06:54:22 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=denx.de Authentication-Results: phobos.denx.de; spf=fail smtp.mailfrom=sr@denx.de Received: from smtp202.mailbox.org (smtp202.mailbox.org [IPv6:2001:67c:2050:b231:465::202]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mout-u-204.mailbox.org (Postfix) with ESMTPS id 4MxxL41l97z9sX1; Wed, 26 Oct 2022 06:54:20 +0200 (CEST) Message-ID: Date: Wed, 26 Oct 2022 06:54:18 +0200 MIME-Version: 1.0 Subject: Re: [PATCH] treewide: Remove the unnecessary space before semicolon Content-Language: en-US To: Bin Meng , u-boot@lists.denx.de Cc: Ashok Reddy Soma , "Derald D. Woods" , Joe Hershberger , =?UTF-8?Q?Marek_Beh=c3=ban?= , Michal Simek , =?UTF-8?Q?Pali_Roh=c3=a1r?= , Priyanka Jain , Priyanka Singh , Ramon Fried , Simon Glass , Vladimir Oltean , Wolfgang Denk References: <20221026044007.2473420-1-bmeng@tinylab.org> From: Stefan Roese In-Reply-To: <20221026044007.2473420-1-bmeng@tinylab.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Rspamd-Queue-Id: 4MxxL41l97z9sX1 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 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.6 at phobos.denx.de X-Virus-Status: Clean On 26.10.22 06:40, Bin Meng wrote: > %s/return ;/return; > > Signed-off-by: Bin Meng > --- > > arch/powerpc/cpu/mpc85xx/cpu_init.c | 2 +- > arch/powerpc/cpu/mpc85xx/cpu_init_early.c | 2 +- > arch/powerpc/cpu/mpc85xx/fdt.c | 6 +++--- > arch/powerpc/cpu/mpc85xx/liodn.c | 4 ++-- > arch/powerpc/cpu/mpc85xx/mpc8548_serdes.c | 2 +- > arch/powerpc/cpu/mpc85xx/portals.c | 8 ++++---- > arch/powerpc/cpu/mpc85xx/tlb.c | 2 +- > arch/powerpc/cpu/mpc8xxx/fdt.c | 2 +- > arch/powerpc/cpu/mpc8xxx/law.c | 2 +- > arch/powerpc/lib/bootm.c | 4 ++-- > board/ti/evm/evm.c | 2 +- > drivers/ddr/fsl/util.c | 2 +- > drivers/net/fm/init.c | 4 ++-- > 13 files changed, 21 insertions(+), 21 deletions(-) Nice. Reviewed by: Stefan Roese Thanks, Stefan > diff --git a/arch/powerpc/cpu/mpc85xx/cpu_init.c b/arch/powerpc/cpu/mpc85xx/cpu_init.c > index 92a7e608ce..90f024d0a7 100644 > --- a/arch/powerpc/cpu/mpc85xx/cpu_init.c > +++ b/arch/powerpc/cpu/mpc85xx/cpu_init.c > @@ -446,7 +446,7 @@ ulong cpu_init_f(void) > /* Implement a dummy function for those platforms w/o SERDES */ > static void __fsl_serdes__init(void) > { > - return ; > + return; > } > __attribute__((weak, alias("__fsl_serdes__init"))) void fsl_serdes_init(void); > > diff --git a/arch/powerpc/cpu/mpc85xx/cpu_init_early.c b/arch/powerpc/cpu/mpc85xx/cpu_init_early.c > index 1bba216371..e6347cfd2d 100644 > --- a/arch/powerpc/cpu/mpc85xx/cpu_init_early.c > +++ b/arch/powerpc/cpu/mpc85xx/cpu_init_early.c > @@ -76,7 +76,7 @@ void setup_ifc(void) > ifc_out32(&(ifc_regs.gregs->csor_cs[0].csor), CONFIG_SYS_CSOR0); > ifc_out32(&(ifc_regs.gregs->amask_cs[0].amask), CONFIG_SYS_AMASK0); > > - return ; > + return; > } > #endif > > diff --git a/arch/powerpc/cpu/mpc85xx/fdt.c b/arch/powerpc/cpu/mpc85xx/fdt.c > index c8ad6a1b01..61d8aa7f4c 100644 > --- a/arch/powerpc/cpu/mpc85xx/fdt.c > +++ b/arch/powerpc/cpu/mpc85xx/fdt.c > @@ -271,13 +271,13 @@ static inline void ft_fixup_l2cache(void *blob) > > if (ph == NULL) { > debug("no next-level-cache property\n"); > - return ; > + return; > } > > off = fdt_node_offset_by_phandle(blob, *ph); > if (off < 0) { > printf("%s: %s\n", __func__, fdt_strerror(off)); > - return ; > + return; > } > > ft_fixup_l2cache_compatible(blob, off); > @@ -373,7 +373,7 @@ next: > l3_off = fdt_node_offset_by_phandle(blob, l3_off); > if (l3_off < 0) { > printf("%s: %s\n", __func__, fdt_strerror(off)); > - return ; > + return; > } > ft_fixup_l3cache(blob, l3_off); > } > diff --git a/arch/powerpc/cpu/mpc85xx/liodn.c b/arch/powerpc/cpu/mpc85xx/liodn.c > index a084002494..34974c92bd 100644 > --- a/arch/powerpc/cpu/mpc85xx/liodn.c > +++ b/arch/powerpc/cpu/mpc85xx/liodn.c > @@ -111,7 +111,7 @@ static void setup_fman_liodn_base(enum fsl_dpaa_dev dev, > #endif > default: > printf("Error: Invalid device type to %s\n", __FUNCTION__); > - return ; > + return; > } > > base = (liodn_bases[dev].id[0] << 16) | liodn_bases[dev].id[0]; > @@ -232,7 +232,7 @@ static void fdt_fixup_srio_liodn(void *blob, struct srio_liodn_id_table *tbl) > /* search for srio node, if doesn't exist just return - nothing todo */ > srio_off = fdt_node_offset_by_compatible(blob, -1, "fsl,srio"); > if (srio_off < 0) > - return ; > + return; > > for (i = 0; i < srio_liodn_tbl_sz; i++) { > int off, portid = tbl[i].portid; > diff --git a/arch/powerpc/cpu/mpc85xx/mpc8548_serdes.c b/arch/powerpc/cpu/mpc85xx/mpc8548_serdes.c > index 2a5c3e3200..38f30afdfc 100644 > --- a/arch/powerpc/cpu/mpc85xx/mpc8548_serdes.c > +++ b/arch/powerpc/cpu/mpc85xx/mpc8548_serdes.c > @@ -45,7 +45,7 @@ void fsl_serdes_init(void) > > if (srds1_cfg >= ARRAY_SIZE(serdes1_cfg_tbl)) { > printf("Invalid PORDEVSR[IO_SEL] = %d\n", srds1_cfg); > - return ; > + return; > } > > for (lane = 0; lane < SRDS1_MAX_LANES; lane++) { > diff --git a/arch/powerpc/cpu/mpc85xx/portals.c b/arch/powerpc/cpu/mpc85xx/portals.c > index 52e2124fb9..6b4cbddcdf 100644 > --- a/arch/powerpc/cpu/mpc85xx/portals.c > +++ b/arch/powerpc/cpu/mpc85xx/portals.c > @@ -21,7 +21,7 @@ void fdt_portal(void *blob, const char *compat, const char *container, > > off = fdt_node_offset_by_compatible(blob, -1, compat); > if (off < 0) > - return ; > + return; > > off = fdt_parent_offset(blob, off); > /* if non-zero assume we have a container */ > @@ -35,7 +35,7 @@ void fdt_portal(void *blob, const char *compat, const char *container, > range = fdt_getprop_w(blob, off, "ranges", &len); > if (range == NULL) { > printf("ERROR: container for %s has no ranges", compat); > - return ; > + return; > } > > range[0] = 0; > @@ -61,13 +61,13 @@ void fdt_portal(void *blob, const char *compat, const char *container, > if (container && (memcmp(container, name, len))) { > printf("WARNING: container names didn't match %s %s\n", > container, name); > - return ; > + return; > } > > memcpy(&buf, name, len); > len += sprintf(&buf[len], "@%llx", addr); > fdt_set_name(blob, off, buf); > - return ; > + return; > } > > printf("ERROR: %s isn't in a container. Not supported\n", compat); > diff --git a/arch/powerpc/cpu/mpc85xx/tlb.c b/arch/powerpc/cpu/mpc85xx/tlb.c > index e39fe14382..81e60722f9 100644 > --- a/arch/powerpc/cpu/mpc85xx/tlb.c > +++ b/arch/powerpc/cpu/mpc85xx/tlb.c > @@ -41,7 +41,7 @@ __weak void init_tlbs(void) > tlb_table[i].mas7); > } > > - return ; > + return; > } > > #if !defined(CONFIG_NAND_SPL) && \ > diff --git a/arch/powerpc/cpu/mpc8xxx/fdt.c b/arch/powerpc/cpu/mpc8xxx/fdt.c > index 871554a7f4..3004290248 100644 > --- a/arch/powerpc/cpu/mpc8xxx/fdt.c > +++ b/arch/powerpc/cpu/mpc8xxx/fdt.c > @@ -128,7 +128,7 @@ void ft_srio_setup(void *blob) > /* search for srio node, if doesn't exist just return - nothing todo */ > srio_off = fdt_node_offset_by_compatible(blob, -1, "fsl,srio"); > if (srio_off < 0) > - return ; > + return; > > #ifdef CONFIG_SRIO1 > if (is_serdes_configured(SRIO1)) > diff --git a/arch/powerpc/cpu/mpc8xxx/law.c b/arch/powerpc/cpu/mpc8xxx/law.c > index 198b5b10a5..24bbe3b45f 100644 > --- a/arch/powerpc/cpu/mpc8xxx/law.c > +++ b/arch/powerpc/cpu/mpc8xxx/law.c > @@ -353,5 +353,5 @@ void init_laws(void) > } > #endif > > - return ; > + return; > } > diff --git a/arch/powerpc/lib/bootm.c b/arch/powerpc/lib/bootm.c > index 512787854c..8ae8d8a3e7 100644 > --- a/arch/powerpc/lib/bootm.c > +++ b/arch/powerpc/lib/bootm.c > @@ -113,7 +113,7 @@ static void boot_jump_linux(struct bootm_headers *images) > cmd_start, cmd_end, 0, 0); > /* does not return */ > } > - return ; > + return; > } > > void arch_lmb_reserve(struct lmb *lmb) > @@ -148,7 +148,7 @@ void arch_lmb_reserve(struct lmb *lmb) > cpu_mp_lmb_reserve(lmb); > #endif > > - return ; > + return; > } > > static void boot_prep_linux(struct bootm_headers *images) > diff --git a/board/ti/evm/evm.c b/board/ti/evm/evm.c > index 39b5c706a9..a7f9a7eca1 100644 > --- a/board/ti/evm/evm.c > +++ b/board/ti/evm/evm.c > @@ -234,7 +234,7 @@ static void reset_net_chip(void) > ret = gpio_request(rst_gpio, ""); > if (ret < 0) { > printf("Unable to get GPIO %d\n", rst_gpio); > - return ; > + return; > } > > /* Configure as output */ > diff --git a/drivers/ddr/fsl/util.c b/drivers/ddr/fsl/util.c > index 43cb01804b..589d7df731 100644 > --- a/drivers/ddr/fsl/util.c > +++ b/drivers/ddr/fsl/util.c > @@ -147,7 +147,7 @@ __fsl_ddr_set_lawbar(const common_timing_params_t *memctl_common_params, > if (set_ddr_laws(base, size, law_memctl) < 0) { > printf("%s: ERROR (ctrl #%d, TRGT ID=%x)\n", __func__, ctrl_num, > law_memctl); > - return ; > + return; > } > debug("setup ddr law base = 0x%llx, size 0x%llx, TRGT_ID 0x%x\n", > base, size, law_memctl); > diff --git a/drivers/net/fm/init.c b/drivers/net/fm/init.c > index af94dabe29..43d2000d24 100644 > --- a/drivers/net/fm/init.c > +++ b/drivers/net/fm/init.c > @@ -149,7 +149,7 @@ void fman_enet_init(void) > } > } > > - return ; > + return; > } > > void fm_disable_port(enum fm_port port) > @@ -233,7 +233,7 @@ static void > __def_board_ft_fman_fixup_port(void *blob, char * prop, phys_addr_t pa, > enum fm_port port, int offset) > { > - return ; > + return; > } > > void board_ft_fman_fixup_port(void *blob, char * prop, phys_addr_t pa, Viele Grüße, Stefan Roese -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de