Skip to contentSkip to conversation

fix a missing break in devivers/staging/iio

PATCH In review Jimmy Li 2 participants 2 messages Mar 22, 2014

Subsystems (4)
Staging IIO Analog Devices IIO drivers Staging - industrial io
lore
Share & tools
lore

Jimmy Li

author
#0 lore raw

PATCH 1/1 fix a missing break in devivers/staging/iio

fix a missing break.

Signed-off-by Jimmy Li <[email protected]>
---
 drivers/staging/iio/resolver/ad2s1200.c |    1 +
 1 file changed, 1 insertion(+)
drivers/staging/iio/resolver/ad2s1200.c +1 -0
index 36eedd8..e2b4820 100644
@@ -70,6 +70,7 @@ static int ad2s1200_read_raw(struct iio_dev *indio_dev,
 		vel = (((s16)(st->rx[0])) << 4) | ((st->rx[1] & 0xF0) >> 4);
 		vel = (vel << 4) >> 4;
 		*val = vel;
+		break;
 	default:
 		mutex_unlock(&st->lock);
 		return -EINVAL;
Signature · 1 line
1.7.9.5
maintainer
#1 lore raw
Quoted 3 lines · Jimmy Li

fix a missing break.

Signed-off-by Jimmy Li <[email protected]>

Thanks. I reworded the patch title to match with convention we have ended up with for iio patches and applied to the fixes-togreg branch of iio.git

This will go in shortly after the first rc.

Jonathan

Quoted 17 lines
---
  drivers/staging/iio/resolver/ad2s1200.c |    1 +
  1 file changed, 1 insertion(+)
drivers/staging/iio/resolver/ad2s1200.c +1 -0
index 36eedd8..e2b4820 100644
@@ -70,6 +70,7 @@ static int ad2s1200_read_raw(struct iio_dev *indio_dev,
  		vel = (((s16)(st->rx[0])) << 4) | ((st->rx[1] & 0xF0) >> 4);
  		vel = (vel << 4) >> 4;
  		*val = vel;
+		break;
  	default:
  		mutex_unlock(&st->lock);
  		return -EINVAL;