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

Groups > linux.kernel > #1225890 > unrolled thread

[PATCH 1/2] mmc: wbsd: Remove unneded semicolon

Started byJavier Martinez Canillas <javier@osg.samsung.com>
First post2015-09-16 11:40 +0200
Last post2015-09-17 11:00 +0200
Articles 4 — 2 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH 1/2] mmc: wbsd: Remove unneded semicolon Javier Martinez Canillas <javier@osg.samsung.com> - 2015-09-16 11:40 +0200
    [PATCH 2/2] mmc: vub300: Remove unneded semicolons Javier Martinez Canillas <javier@osg.samsung.com> - 2015-09-16 11:40 +0200
      Re: [PATCH 2/2] mmc: vub300: Remove unneded semicolons Ulf Hansson <ulf.hansson@linaro.org> - 2015-09-17 10:50 +0200
    Re: [PATCH 1/2] mmc: wbsd: Remove unneded semicolon Ulf Hansson <ulf.hansson@linaro.org> - 2015-09-17 11:00 +0200

#1225890 — [PATCH 1/2] mmc: wbsd: Remove unneded semicolon

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2015-09-16 11:40 +0200
Subject[PATCH 1/2] mmc: wbsd: Remove unneded semicolon
Message-ID<q9hm3-uy-17@gated-at.bofh.it>
It's not needed an is just creating a null statement, so remove it.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 drivers/mmc/host/wbsd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mmc/host/wbsd.c b/drivers/mmc/host/wbsd.c
index ca183ea767b3..c3fd16d997ca 100644
--- a/drivers/mmc/host/wbsd.c
+++ b/drivers/mmc/host/wbsd.c
@@ -809,7 +809,7 @@ static void wbsd_request(struct mmc_host *mmc, struct mmc_request *mrq)
 			cmd->error = -EINVAL;
 
 			goto done;
-		};
+		}
 	}
 
 	/*
-- 
2.4.3

--
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]

#1225891 — [PATCH 2/2] mmc: vub300: Remove unneded semicolons

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2015-09-16 11:40 +0200
Subject[PATCH 2/2] mmc: vub300: Remove unneded semicolons
Message-ID<q9hm3-uy-15@gated-at.bofh.it>
In reply to#1225890
They aren't needed and are just creating null statements so remove it.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

---

 drivers/mmc/host/vub300.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/mmc/host/vub300.c b/drivers/mmc/host/vub300.c
index fbabbb82b354..1e819f98b94f 100644
--- a/drivers/mmc/host/vub300.c
+++ b/drivers/mmc/host/vub300.c
@@ -563,7 +563,7 @@ static void add_offloaded_reg(struct vub300_mmc_host *vub300,
 			i += 1;
 			continue;
 		}
-	};
+	}
 	__add_offloaded_reg_to_fifo(vub300, register_access, func);
 }
 
@@ -1372,7 +1372,7 @@ static void download_offload_pseudocode(struct vub300_mmc_host *vub300)
 		l += snprintf(vub300->vub_name + l,
 			      sizeof(vub300->vub_name) - l, "_%04X%04X",
 			      sf->vendor, sf->device);
-	};
+	}
 	snprintf(vub300->vub_name + l, sizeof(vub300->vub_name) - l, ".bin");
 	dev_info(&vub300->udev->dev, "requesting offload firmware %s\n",
 		 vub300->vub_name);
@@ -1893,7 +1893,7 @@ static int satisfy_request_from_offloaded_data(struct vub300_mmc_host *vub300,
 			i += 1;
 			continue;
 		}
-	};
+	}
 	if (vub300->total_offload_count == 0)
 		return 0;
 	else if (vub300->fn[func].offload_count == 0)
-- 
2.4.3

--
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]

#1226774 — Re: [PATCH 2/2] mmc: vub300: Remove unneded semicolons

FromUlf Hansson <ulf.hansson@linaro.org>
Date2015-09-17 10:50 +0200
SubjectRe: [PATCH 2/2] mmc: vub300: Remove unneded semicolons
Message-ID<q9D3e-6EA-41@gated-at.bofh.it>
In reply to#1225891
On 16 September 2015 at 11:30, Javier Martinez Canillas
<javier@osg.samsung.com> wrote:
> They aren't needed and are just creating null statements so remove it.
>
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

Thanks, applied for next!

Kind regards
Uffe


>
> ---
>
>  drivers/mmc/host/vub300.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/mmc/host/vub300.c b/drivers/mmc/host/vub300.c
> index fbabbb82b354..1e819f98b94f 100644
> --- a/drivers/mmc/host/vub300.c
> +++ b/drivers/mmc/host/vub300.c
> @@ -563,7 +563,7 @@ static void add_offloaded_reg(struct vub300_mmc_host *vub300,
>                         i += 1;
>                         continue;
>                 }
> -       };
> +       }
>         __add_offloaded_reg_to_fifo(vub300, register_access, func);
>  }
>
> @@ -1372,7 +1372,7 @@ static void download_offload_pseudocode(struct vub300_mmc_host *vub300)
>                 l += snprintf(vub300->vub_name + l,
>                               sizeof(vub300->vub_name) - l, "_%04X%04X",
>                               sf->vendor, sf->device);
> -       };
> +       }
>         snprintf(vub300->vub_name + l, sizeof(vub300->vub_name) - l, ".bin");
>         dev_info(&vub300->udev->dev, "requesting offload firmware %s\n",
>                  vub300->vub_name);
> @@ -1893,7 +1893,7 @@ static int satisfy_request_from_offloaded_data(struct vub300_mmc_host *vub300,
>                         i += 1;
>                         continue;
>                 }
> -       };
> +       }
>         if (vub300->total_offload_count == 0)
>                 return 0;
>         else if (vub300->fn[func].offload_count == 0)
> --
> 2.4.3
>
--
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]

#1226776

FromUlf Hansson <ulf.hansson@linaro.org>
Date2015-09-17 11:00 +0200
Message-ID<q9DcS-6QA-3@gated-at.bofh.it>
In reply to#1225890
On 16 September 2015 at 11:30, Javier Martinez Canillas
<javier@osg.samsung.com> wrote:
> It's not needed an is just creating a null statement, so remove it.
>
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

Thanks, applied for next!

Kind regards
Uffe


> ---
>
>  drivers/mmc/host/wbsd.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/host/wbsd.c b/drivers/mmc/host/wbsd.c
> index ca183ea767b3..c3fd16d997ca 100644
> --- a/drivers/mmc/host/wbsd.c
> +++ b/drivers/mmc/host/wbsd.c
> @@ -809,7 +809,7 @@ static void wbsd_request(struct mmc_host *mmc, struct mmc_request *mrq)
>                         cmd->error = -EINVAL;
>
>                         goto done;
> -               };
> +               }
>         }
>
>         /*
> --
> 2.4.3
>
--
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