Acked-by.
--
Dipl.-Ing.
Jan-Simon Möller
jansimon.moeller@xxxxxx
On Thursday 12 September 2013 09:50:29 Sachin Kamat wrote:
> 'break' after return is redundant. Remove it.
>
> Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx>
> ---
> drivers/leds/leds-blinkm.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/drivers/leds/leds-blinkm.c b/drivers/leds/leds-blinkm.c
> index a502678..66d0a57 100644
> --- a/drivers/leds/leds-blinkm.c
> +++ b/drivers/leds/leds-blinkm.c
> @@ -161,13 +161,10 @@ static ssize_t show_color_common(struct device *dev,
> char *buf, int color) switch (color) {
> case RED:
> return scnprintf(buf, PAGE_SIZE, "%02X\n", data->red);
> - break;
> case GREEN:
> return scnprintf(buf, PAGE_SIZE, "%02X\n", data->green);
> - break;
> case BLUE:
> return scnprintf(buf, PAGE_SIZE, "%02X\n", data->blue);
> - break;
> default:
> return -EINVAL;
> }
--
To unsubscribe from this list: send the line "unsubscribe linux-leds" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
