Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1199640 > unrolled thread

[PATCH 1/3] Staging: lustre: Replace comma with a semicolon

Started byShraddha Barke <shraddha.6596@gmail.com>
First post2015-08-04 12:30 +0200
Last post2015-08-04 14:10 +0200
Articles 4 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 1/3] Staging: lustre: Replace comma with a semicolon Shraddha Barke <shraddha.6596@gmail.com> - 2015-08-04 12:30 +0200
    [PATCH 3/3] Staging: netlogic: Replace comma with a semicolon Shraddha Barke <shraddha.6596@gmail.com> - 2015-08-04 12:30 +0200
    Re: [PATCH 1/3] Staging: lustre: Replace comma with a semicolon Julia Lawall <julia.lawall@lip6.fr> - 2015-08-04 13:40 +0200
      Re: [PATCH 1/3] Staging: lustre: Replace comma with a semicolon Julia Lawall <julia.lawall@lip6.fr> - 2015-08-04 14:10 +0200

#1199640 — [PATCH 1/3] Staging: lustre: Replace comma with a semicolon

FromShraddha Barke <shraddha.6596@gmail.com>
Date2015-08-04 12:30 +0200
Subject[PATCH 1/3] Staging: lustre: Replace comma with a semicolon
Message-ID<pTHDP-7JR-3@gated-at.bofh.it>
Replace comma between expression statements by a semicolon.
The semantic patch used is as follows:

@@
expression e1,e2;
@@
e1
- ,
+ ;
e2;
---
 drivers/staging/lustre/lustre/libcfs/libcfs_string.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/lustre/lustre/libcfs/libcfs_string.c b/drivers/staging/lustre/lustre/libcfs/libcfs_string.c
index 4dde8e0..efe5e66 100644
--- a/drivers/staging/lustre/lustre/libcfs/libcfs_string.c
+++ b/drivers/staging/lustre/lustre/libcfs/libcfs_string.c
@@ -400,7 +400,7 @@ cfs_expr_list_free(struct cfs_expr_list *expr_list)
 		struct cfs_range_expr *expr;
 
 		expr = list_entry(expr_list->el_exprs.next,
-				      struct cfs_range_expr, re_link),
+				      struct cfs_range_expr, re_link);
 		list_del(&expr->re_link);
 		LIBCFS_FREE(expr, sizeof(*expr));
 	}
-- 
2.1.0

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1199641 — [PATCH 3/3] Staging: netlogic: Replace comma with a semicolon

FromShraddha Barke <shraddha.6596@gmail.com>
Date2015-08-04 12:30 +0200
Subject[PATCH 3/3] Staging: netlogic: Replace comma with a semicolon
Message-ID<pTHDP-7JR-11@gated-at.bofh.it>
In reply to#1199640
Replace comma between expression statements by a semicolon.
The semantic patch used is as follows:

@@
expression e1,e2;
@@
e1
- ,
+ ;
e2;
---
 drivers/staging/netlogic/platform_net.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/netlogic/platform_net.c b/drivers/staging/netlogic/platform_net.c
index 77c3c35..e914147 100644
--- a/drivers/staging/netlogic/platform_net.c
+++ b/drivers/staging/netlogic/platform_net.c
@@ -163,7 +163,7 @@ static void xls_gmac_init(void)
 	switch (nlm_prom_info.board_major_version) {
 	case 12:
 		/* first block RGMII or XAUI, use RGMII */
-		ndata0.phy_interface = PHY_INTERFACE_MODE_RGMII,
+		ndata0.phy_interface = PHY_INTERFACE_MODE_RGMII;
 		ndata0.tx_stnid[0] = FMN_STNID_GMAC0_TX0;
 		ndata0.phy_addr[0] = 0;
 
-- 
2.1.0

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1199696

FromJulia Lawall <julia.lawall@lip6.fr>
Date2015-08-04 13:40 +0200
Message-ID<pTIJA-OZ-5@gated-at.bofh.it>
In reply to#1199640
1 and 2 need to have different subjects (the [PATCH] part is not enough,
because it will go away).

julia

On Tue, 4 Aug 2015, Shraddha Barke wrote:

> Replace comma between expression statements by a semicolon.
> The semantic patch used is as follows:
>
> @@
> expression e1,e2;
> @@
> e1
> - ,
> + ;
> e2;
> ---
>  drivers/staging/lustre/lustre/libcfs/libcfs_string.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/lustre/lustre/libcfs/libcfs_string.c b/drivers/staging/lustre/lustre/libcfs/libcfs_string.c
> index 4dde8e0..efe5e66 100644
> --- a/drivers/staging/lustre/lustre/libcfs/libcfs_string.c
> +++ b/drivers/staging/lustre/lustre/libcfs/libcfs_string.c
> @@ -400,7 +400,7 @@ cfs_expr_list_free(struct cfs_expr_list *expr_list)
>  		struct cfs_range_expr *expr;
>
>  		expr = list_entry(expr_list->el_exprs.next,
> -				      struct cfs_range_expr, re_link),
> +				      struct cfs_range_expr, re_link);
>  		list_del(&expr->re_link);
>  		LIBCFS_FREE(expr, sizeof(*expr));
>  	}
> --
> 2.1.0
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1199736

FromJulia Lawall <julia.lawall@lip6.fr>
Date2015-08-04 14:10 +0200
Message-ID<pTJcF-1D3-63@gated-at.bofh.it>
In reply to#1199696
On Tue, 4 Aug 2015, Shraddha Barke wrote:

>
> Should I resend the entire patch series as v2 ? Or only the first two
> patches?

The entire series.  You can say under the --- that there is no change in
the cases where there is no change.

julia
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web