Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b8d67ab6 authored by Antti Palosaari's avatar Antti Palosaari Committed by Mauro Carvalho Chehab
Browse files

[media] m88ts2022: clean up logging



There is no need to print module name nor function name as those
are done by kernel logging system when dev_xxx logging is used and
driver is proper I2C driver.

Signed-off-by: default avatarAntti Palosaari <crope@iki.fi>
Signed-off-by: default avatarMauro Carvalho Chehab <m.chehab@samsung.com>
parent eb773df5
Loading
Loading
Loading
Loading
+24 −27
Original line number Diff line number Diff line
@@ -46,8 +46,8 @@ static int m88ts2022_wr_regs(struct m88ts2022_dev *dev,
		ret = 0;
	} else {
		dev_warn(&dev->client->dev,
				"%s: i2c wr failed=%d reg=%02x len=%d\n",
				KBUILD_MODNAME, ret, reg, len);
				"i2c wr failed=%d reg=%02x len=%d\n",
				ret, reg, len);
		ret = -EREMOTEIO;
	}

@@ -85,8 +85,8 @@ static int m88ts2022_rd_regs(struct m88ts2022_dev *dev, u8 reg,
		ret = 0;
	} else {
		dev_warn(&dev->client->dev,
				"%s: i2c rd failed=%d reg=%02x len=%d\n",
				KBUILD_MODNAME, ret, reg, len);
				"i2c rd failed=%d reg=%02x len=%d\n",
				ret, reg, len);
		ret = -EREMOTEIO;
	}

@@ -141,8 +141,8 @@ static int m88ts2022_cmd(struct dvb_frontend *fe,

	for (i = 0; i < 2; i++) {
		dev_dbg(&dev->client->dev,
				"%s: i=%d op=%02x reg=%02x mask=%02x val=%02x\n",
				__func__, i, op, reg, mask, val);
				"i=%d op=%02x reg=%02x mask=%02x val=%02x\n",
				i, op, reg, mask, val);

		for (i = 0; i < ARRAY_SIZE(reg_vals); i++) {
			ret = m88ts2022_wr_reg(dev, reg_vals[i].reg,
@@ -178,8 +178,8 @@ static int m88ts2022_set_params(struct dvb_frontend *fe)
	u16 u16tmp;

	dev_dbg(&dev->client->dev,
			"%s: frequency=%d symbol_rate=%d rolloff=%d\n",
			__func__, c->frequency, c->symbol_rate, c->rolloff);
			"frequency=%d symbol_rate=%d rolloff=%d\n",
			c->frequency, c->symbol_rate, c->rolloff);
	/*
	 * Integer-N PLL synthesizer
	 * kHz is used for all calculations to keep calculations within 32-bit
@@ -228,10 +228,9 @@ static int m88ts2022_set_params(struct dvb_frontend *fe)
		goto err;

	dev_dbg(&dev->client->dev,
			"%s: frequency=%u offset=%d f_vco_khz=%u pll_n=%u div_ref=%u div_out=%u\n",
			__func__, dev->frequency_khz,
			dev->frequency_khz - c->frequency, f_vco_khz, pll_n,
			div_ref, div_out);
			"frequency=%u offset=%d f_vco_khz=%u pll_n=%u div_ref=%u div_out=%u\n",
			dev->frequency_khz, dev->frequency_khz - c->frequency,
			f_vco_khz, pll_n, div_ref, div_out);

	ret = m88ts2022_cmd(fe, 0x10, 5, 0x15, 0x40, 0x00, NULL);
	if (ret)
@@ -371,7 +370,7 @@ static int m88ts2022_set_params(struct dvb_frontend *fe)
		goto err;
err:
	if (ret)
		dev_dbg(&dev->client->dev, "%s: failed=%d\n", __func__, ret);
		dev_dbg(&dev->client->dev, "failed=%d\n", ret);

	return ret;
}
@@ -395,7 +394,7 @@ static int m88ts2022_init(struct dvb_frontend *fe)
		{0x12, 0xa0},
	};

	dev_dbg(&dev->client->dev, "%s:\n", __func__);
	dev_dbg(&dev->client->dev, "\n");

	ret = m88ts2022_wr_reg(dev, 0x00, 0x01);
	if (ret)
@@ -442,7 +441,7 @@ static int m88ts2022_init(struct dvb_frontend *fe)
	}
err:
	if (ret)
		dev_dbg(&dev->client->dev, "%s: failed=%d\n", __func__, ret);
		dev_dbg(&dev->client->dev, "failed=%d\n", ret);
	return ret;
}

@@ -451,14 +450,14 @@ static int m88ts2022_sleep(struct dvb_frontend *fe)
	struct m88ts2022_dev *dev = fe->tuner_priv;
	int ret;

	dev_dbg(&dev->client->dev, "%s:\n", __func__);
	dev_dbg(&dev->client->dev, "\n");

	ret = m88ts2022_wr_reg(dev, 0x00, 0x00);
	if (ret)
		goto err;
err:
	if (ret)
		dev_dbg(&dev->client->dev, "%s: failed=%d\n", __func__, ret);
		dev_dbg(&dev->client->dev, "failed=%d\n", ret);
	return ret;
}

@@ -466,7 +465,7 @@ static int m88ts2022_get_frequency(struct dvb_frontend *fe, u32 *frequency)
{
	struct m88ts2022_dev *dev = fe->tuner_priv;

	dev_dbg(&dev->client->dev, "%s:\n", __func__);
	dev_dbg(&dev->client->dev, "\n");

	*frequency = dev->frequency_khz;
	return 0;
@@ -476,7 +475,7 @@ static int m88ts2022_get_if_frequency(struct dvb_frontend *fe, u32 *frequency)
{
	struct m88ts2022_dev *dev = fe->tuner_priv;

	dev_dbg(&dev->client->dev, "%s:\n", __func__);
	dev_dbg(&dev->client->dev, "\n");

	*frequency = 0; /* Zero-IF */
	return 0;
@@ -520,7 +519,7 @@ static int m88ts2022_get_rf_strength(struct dvb_frontend *fe, u16 *strength)
	*strength = (u16tmp - 59000) * 0xffff / (61500 - 59000);
err:
	if (ret)
		dev_dbg(&dev->client->dev, "%s: failed=%d\n", __func__, ret);
		dev_dbg(&dev->client->dev, "failed=%d\n", ret);
	return ret;
}

@@ -552,7 +551,7 @@ static int m88ts2022_probe(struct i2c_client *client,
	dev = kzalloc(sizeof(*dev), GFP_KERNEL);
	if (!dev) {
		ret = -ENOMEM;
		dev_err(&client->dev, "%s: kzalloc() failed\n", KBUILD_MODNAME);
		dev_err(&client->dev, "kzalloc() failed\n");
		goto err;
	}

@@ -582,7 +581,7 @@ static int m88ts2022_probe(struct i2c_client *client,
	if (ret)
		goto err;

	dev_dbg(&dev->client->dev, "%s: chip_id=%02x\n", __func__, chip_id);
	dev_dbg(&dev->client->dev, "chip_id=%02x\n", chip_id);

	switch (chip_id) {
	case 0xc3:
@@ -627,9 +626,7 @@ static int m88ts2022_probe(struct i2c_client *client,
	if (ret)
		goto err;

	dev_info(&dev->client->dev,
			"%s: Montage M88TS2022 successfully identified\n",
			KBUILD_MODNAME);
	dev_info(&dev->client->dev, "Montage M88TS2022 successfully identified\n");

	fe->tuner_priv = dev;
	memcpy(&fe->ops.tuner_ops, &m88ts2022_tuner_ops,
@@ -638,7 +635,7 @@ static int m88ts2022_probe(struct i2c_client *client,
	i2c_set_clientdata(client, dev);
	return 0;
err:
	dev_dbg(&client->dev, "%s: failed=%d\n", __func__, ret);
	dev_dbg(&client->dev, "failed=%d\n", ret);
	kfree(dev);
	return ret;
}
@@ -648,7 +645,7 @@ static int m88ts2022_remove(struct i2c_client *client)
	struct m88ts2022_dev *dev = i2c_get_clientdata(client);
	struct dvb_frontend *fe = dev->cfg.fe;

	dev_dbg(&client->dev, "%s:\n", __func__);
	dev_dbg(&client->dev, "\n");

	memset(&fe->ops.tuner_ops, 0, sizeof(struct dvb_tuner_ops));
	fe->tuner_priv = NULL;