[alsa-devel] [PATCH 1/1] ALSA: rme9652: Remove redundant break

Takashi Iwai tiwai at suse.de
Fri Sep 13 14:40:05 CEST 2013


At Fri, 13 Sep 2013 16:18:59 +0530,
Sachin Kamat wrote:
> 
> 'break' after return statement is not necessary.
> 
> Signed-off-by: Sachin Kamat <sachin.kamat at linaro.org>

Applied this one, too.  Thanks.


Takashi

> ---
>  sound/pci/rme9652/hdspm.c |    5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/sound/pci/rme9652/hdspm.c b/sound/pci/rme9652/hdspm.c
> index 3cde55b..2907e68 100644
> --- a/sound/pci/rme9652/hdspm.c
> +++ b/sound/pci/rme9652/hdspm.c
> @@ -3996,7 +3996,6 @@ static int hdspm_tco_sync_check(struct hdspm *hdspm)
>  					return 1;
>  			}
>  			return 0;
> -			break;
>  		case AES32:
>  			status = hdspm_read(hdspm, HDSPM_statusRegister);
>  			if (status & HDSPM_tcoLockAes) {
> @@ -4006,9 +4005,6 @@ static int hdspm_tco_sync_check(struct hdspm *hdspm)
>  					return 1;
>  			}
>  			return 0;
> -
> -			break;
> -
>  		case RayDAT:
>  		case AIO:
>  			status = hdspm_read(hdspm, HDSPM_RD_STATUS_1);
> @@ -4018,7 +4014,6 @@ static int hdspm_tco_sync_check(struct hdspm *hdspm)
>  			if (status & 0x4000000)
>  				return 1; /* Lock */
>  			return 0; /* No signal */
> -			break;
>  
>  		default:
>  			break;
> -- 
> 1.7.9.5
> 


More information about the Alsa-devel mailing list