media: media si2168: fully initialize si2168 on resume only when necessary

At connection time (or boot) in si2168_probe(), the firmware is not
loaded to the device and the device is not fully activated. It is
not useful or sensible to do this full initialization on resume in
case it has not been previously initialized and is expected to be
in this initialized state.
Calling si2168_init() and therefore reading the firmware file for
the first time during resume leads to problems and should be avoided.
It is however safe to read the firmware file once it has already been
read outside of a suspend/resume situation.

Add a staus flag 'initialized' to store whether si2168_init() has
successfully been called. If initialization fails (e.g. due to missing
firmware file), the flag is not set.
Register a separate si2168_resume callback which only calls
si2168_init() once the 'initialized' flag has been set and it is safe
to load the firmware at resume.
The first call to si2168_init() will now always happen when the device
is actually used for the first time and never during resume.
This avoids the unsafe firmware file reading and should also speed up
resume by skipping unnecessary device initialization.

Link: https://lore.kernel.org/linux-media/20210418001204.7453-3-kernel@tuxforce.de
[mchehab: fix several Coding Style issues]
Cc: Antti Palosaari <crope@iki.fi>
Signed-off-by: Lukas Middendorf <kernel@tuxforce.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>

diff --git a/drivers/media/dvb-frontends/si2168.c b/drivers/media/dvb-frontends/si2168.c
index 7fae5b1b8adfeaed83631b6862472198d0864b34..8a3c41a80c03a8fa6228ba64c56c7c387264e266 100644 (file)
--- a/drivers/media/dvb-frontends/si2168.c
+++ b/drivers/media/dvb-frontends/si2168.c
@@ -514,6 +514,7 @@ static int si2168_init(struct dvb_frontend *fe)
                goto err;
 
        dev->warm = true;
+       dev->initialized = true;
 warm:
        /* Init stats here to indicate which stats are supported */
        c->cnr.len = 1;
@@ -535,6 +536,26 @@ err:
        return ret;
 }
 
+static int si2168_resume(struct dvb_frontend *fe)
+{
+       struct i2c_client *client = fe->demodulator_priv;
+       struct si2168_dev *dev = i2c_get_clientdata(client);
+
+       /*
+        * check whether si2168_init() has been called successfully
+        * outside of a resume cycle. Only call it (and load firmware)
+        * in this case. si2168_init() is only called during resume
+        * once the device has actually been used. Otherwise, leave the
+        * device untouched.
+        */
+       if (dev->initialized) {
+               dev_dbg(&client->dev, "previsously initialized, call si2168_init()\n");
+               return si2168_init(fe);
+       }
+       dev_dbg(&client->dev, "not initialized yet, skipping init on resume\n");
+       return 0;
+}
+
 static int si2168_sleep(struct dvb_frontend *fe)
 {
        struct i2c_client *client = fe->demodulator_priv;
@@ -644,6 +665,7 @@ static const struct dvb_frontend_ops si2168_ops = {
 
        .init = si2168_init,
        .sleep = si2168_sleep,
+       .resume = si2168_resume,
 
        .set_frontend = si2168_set_frontend,
 

diff --git a/drivers/media/dvb-frontends/si2168_priv.h b/drivers/media/dvb-frontends/si2168_priv.h
index 4f4d122e396377415a1937cf9475980507b5f625..3a8976229a4a62f14a61f71f700d6a70b4f3a8ad 100644 (file)
--- a/drivers/media/dvb-frontends/si2168_priv.h
+++ b/drivers/media/dvb-frontends/si2168_priv.h
@@ -36,6 +36,7 @@ struct si2168_dev {
        u8 ts_mode;
        unsigned int active:1;
        unsigned int warm:1;
+       unsigned int initialized:1;
        unsigned int ts_clock_inv:1;
        unsigned int ts_clock_gapped:1;
        unsigned int spectral_inversion:1;