This actually fixes a bug.
The struct prototype is defined at the beginning of the code as
"si2157_ops" but the real struct is called "si2157_tuner_ops".
This is causing the name to be empty on this info msg: si2157 16-0060:
si2157: found a '' in cold state
Luis
On Fri, Jul 18, 2014 at 6:41 AM, Olli Salonen <olli.salonen@xxxxxx> wrote:
> Signed-off-by: Olli Salonen <olli.salonen@xxxxxx>
> ---
> drivers/media/tuners/si2157.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/media/tuners/si2157.c b/drivers/media/tuners/si2157.c
> index 329004f..4730f69 100644
> --- a/drivers/media/tuners/si2157.c
> +++ b/drivers/media/tuners/si2157.c
> @@ -277,7 +277,7 @@ err:
> return ret;
> }
>
> -static const struct dvb_tuner_ops si2157_tuner_ops = {
> +static const struct dvb_tuner_ops si2157_ops = {
> .info = {
> .name = "Silicon Labs Si2157/Si2158",
> .frequency_min = 110000000,
> @@ -317,7 +317,7 @@ static int si2157_probe(struct i2c_client *client,
> goto err;
>
> fe->tuner_priv = s;
> - memcpy(&fe->ops.tuner_ops, &si2157_tuner_ops,
> + memcpy(&fe->ops.tuner_ops, &si2157_ops,
> sizeof(struct dvb_tuner_ops));
>
> i2c_set_clientdata(client, s);
> --
> 1.9.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
