[media] mn88472: check if firmware is already running before loading it

Signed-off-by: Benjamin Larsson <benjamin@southpole.se>
Signed-off-by: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>

diff --git a/drivers/staging/media/mn88472/mn88472.c b/drivers/staging/media/mn88472/mn88472.c
index 85ef4f694c8b26e9eb8149ecb9cb367df8d3fb16..bfb3e447773afd862907cb4926f1cdab52d2df71 100644 (file)
--- a/drivers/staging/media/mn88472/mn88472.c
+++ b/drivers/staging/media/mn88472/mn88472.c
@@ -258,7 +258,7 @@ static int mn88472_init(struct dvb_frontend *fe)
        int ret, len, remaining;
        const struct firmware *fw = NULL;
        u8 *fw_file = MN88472_FIRMWARE;
-       unsigned int csum;
+       unsigned int tmp;
 
        dev_dbg(&client->dev, "\n");
 
@@ -274,6 +274,17 @@ static int mn88472_init(struct dvb_frontend *fe)
        if (ret)
                goto err;
 
+       /* check if firmware is already running */
+       ret = regmap_read(dev->regmap[0], 0xf5, &tmp);
+       if (ret)
+               goto err;
+
+       if (!(tmp & 0x1)) {
+               dev_info(&client->dev, "firmware already running\n");
+               dev->warm = true;
+               return 0;
+       }
+
        /* request the firmware, this will block and timeout */
        ret = request_firmware(&fw, fw_file, &client->dev);
        if (ret) {
@@ -305,18 +316,18 @@ static int mn88472_init(struct dvb_frontend *fe)
        }
 
        /* parity check of firmware */
-       ret = regmap_read(dev->regmap[0], 0xf8, &csum);
+       ret = regmap_read(dev->regmap[0], 0xf8, &tmp);
        if (ret) {
                dev_err(&client->dev,
                                "parity reg read failed=%d\n", ret);
                goto err;
        }
-       if (csum & 0x10) {
+       if (tmp & 0x10) {
                dev_err(&client->dev,
-                               "firmware parity check failed=0x%x\n", csum);
+                               "firmware parity check failed=0x%x\n", tmp);
                goto err;
        }
-       dev_err(&client->dev, "firmware parity check succeeded=0x%x\n", csum);
+       dev_err(&client->dev, "firmware parity check succeeded=0x%x\n", tmp);
 
        ret = regmap_write(dev->regmap[0], 0xf5, 0x00);
        if (ret)