[REVIEW PATCH 34/86] msi3101: improve ADC config stream format selection

[Antti Palosaari]

Improve ADC config stream format selection. No functional changes.

Signed-off-by: Antti Palosaari <cr...@iki.fi>
---
 drivers/staging/media/msi3101/sdr-msi3101.c | 37 +++++++++++++----------------
 1 file changed, 17 insertions(+), 20 deletions(-)

diff --git a/drivers/staging/media/msi3101/sdr-msi3101.c 
b/drivers/staging/media/msi3101/sdr-msi3101.c
index 6b9f0da..ba37fce 100644
--- a/drivers/staging/media/msi3101/sdr-msi3101.c
+++ b/drivers/staging/media/msi3101/sdr-msi3101.c
@@ -1247,38 +1247,35 @@ static int msi3101_set_usb_adc(struct msi3101_state *s)
        f_sr = s->f_adc;
 
        /* select stream format */
-       if (f_sr < 6000000) {
-               s->convert_stream = msi3101_convert_stream_252;
-               reg7 = 0x00009407;
-       } else if (f_sr < 8000000) {
-               s->convert_stream = msi3101_convert_stream_336;
-               reg7 = 0x00008507;
-       } else if (f_sr < 9000000) {
-               s->convert_stream = msi3101_convert_stream_384;
-               reg7 = 0x0000a507;
-       } else {
-               s->convert_stream = msi3101_convert_stream_504;
-               reg7 = 0x000c9407;
-       }
-
-       if (s->pixelformat == V4L2_PIX_FMT_SDR_U8) {
+       switch (s->pixelformat) {
+       case V4L2_PIX_FMT_SDR_U8:
                s->convert_stream = msi3101_convert_stream_504_u8;
                reg7 = 0x000c9407;
-       } else if (s->pixelformat == V4L2_PIX_FMT_SDR_U16LE) {
+               break;
+       case V4L2_PIX_FMT_SDR_U16LE:
                s->convert_stream = msi3101_convert_stream_252_u16;
                reg7 = 0x00009407;
-       } else if (s->pixelformat == V4L2_PIX_FMT_SDR_S8) {
+               break;
+       case V4L2_PIX_FMT_SDR_S8:
                s->convert_stream = msi3101_convert_stream_504;
                reg7 = 0x000c9407;
-       } else if (s->pixelformat == V4L2_PIX_FMT_SDR_MSI2500_384) {
+               break;
+       case V4L2_PIX_FMT_SDR_MSI2500_384:
                s->convert_stream = msi3101_convert_stream_384;
                reg7 = 0x0000a507;
-       } else if (s->pixelformat == V4L2_PIX_FMT_SDR_S12) {
+               break;
+       case V4L2_PIX_FMT_SDR_S12:
                s->convert_stream = msi3101_convert_stream_336;
                reg7 = 0x00008507;
-       } else if (s->pixelformat == V4L2_PIX_FMT_SDR_S14) {
+               break;
+       case V4L2_PIX_FMT_SDR_S14:
                s->convert_stream = msi3101_convert_stream_252;
                reg7 = 0x00009407;
+               break;
+       default:
+               s->convert_stream = msi3101_convert_stream_504_u8;
+               reg7 = 0x000c9407;
+               break;
        }
 
        /*
-- 
1.8.5.3

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html