Commit 257e29f8 authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

[media] tvp5150: convert it to use dev_foo() macros

Instead of using v4l_foo(), use the dev_foo() macros, as
most modern media drivers.
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent ce8591ff
...@@ -74,11 +74,11 @@ static int tvp5150_read(struct v4l2_subdev *sd, unsigned char addr) ...@@ -74,11 +74,11 @@ static int tvp5150_read(struct v4l2_subdev *sd, unsigned char addr)
rc = i2c_smbus_read_byte_data(c, addr); rc = i2c_smbus_read_byte_data(c, addr);
if (rc < 0) { if (rc < 0) {
v4l2_err(sd, "i2c i/o error: rc == %d\n", rc); dev_err(sd->dev, "i2c i/o error: rc == %d\n", rc);
return rc; return rc;
} }
v4l2_dbg(2, debug, sd, "tvp5150: read 0x%02x = 0x%02x\n", addr, rc); dev_dbg_lvl(sd->dev, 2, debug, "tvp5150: read 0x%02x = %02x\n", addr, rc);
return rc; return rc;
} }
...@@ -89,10 +89,10 @@ static int tvp5150_write(struct v4l2_subdev *sd, unsigned char addr, ...@@ -89,10 +89,10 @@ static int tvp5150_write(struct v4l2_subdev *sd, unsigned char addr,
struct i2c_client *c = v4l2_get_subdevdata(sd); struct i2c_client *c = v4l2_get_subdevdata(sd);
int rc; int rc;
v4l2_dbg(2, debug, sd, "tvp5150: writing 0x%02x 0x%02x\n", addr, value); dev_dbg_lvl(sd->dev, 2, debug, "tvp5150: writing %02x %02x\n", addr, value);
rc = i2c_smbus_write_byte_data(c, addr, value); rc = i2c_smbus_write_byte_data(c, addr, value);
if (rc < 0) if (rc < 0)
v4l2_err(sd, "i2c i/o error: rc == %d\n", rc); dev_err(sd->dev, "i2c i/o error: rc == %d\n", rc);
return rc; return rc;
} }
...@@ -280,8 +280,7 @@ static inline void tvp5150_selmux(struct v4l2_subdev *sd) ...@@ -280,8 +280,7 @@ static inline void tvp5150_selmux(struct v4l2_subdev *sd)
break; break;
} }
v4l2_dbg(1, debug, sd, "Selecting video route: route input=%i, output=%i " dev_dbg_lvl(sd->dev, 1, debug, "Selecting video route: route input=%i, output=%i => tvp5150 input=%i, opmode=%i\n",
"=> tvp5150 input=%i, opmode=%i\n",
decoder->input, decoder->output, decoder->input, decoder->output,
input, opmode); input, opmode);
...@@ -293,7 +292,7 @@ static inline void tvp5150_selmux(struct v4l2_subdev *sd) ...@@ -293,7 +292,7 @@ static inline void tvp5150_selmux(struct v4l2_subdev *sd)
*/ */
val = tvp5150_read(sd, TVP5150_MISC_CTL); val = tvp5150_read(sd, TVP5150_MISC_CTL);
if (val < 0) { if (val < 0) {
v4l2_err(sd, "%s: failed with error = %d\n", __func__, val); dev_err(sd->dev, "%s: failed with error = %d\n", __func__, val);
return; return;
} }
...@@ -611,7 +610,7 @@ static int tvp5150_g_sliced_vbi_cap(struct v4l2_subdev *sd, ...@@ -611,7 +610,7 @@ static int tvp5150_g_sliced_vbi_cap(struct v4l2_subdev *sd,
const struct i2c_vbi_ram_value *regs = vbi_ram_default; const struct i2c_vbi_ram_value *regs = vbi_ram_default;
int line; int line;
v4l2_dbg(1, debug, sd, "g_sliced_vbi_cap\n"); dev_dbg_lvl(sd->dev, 1, debug, "g_sliced_vbi_cap\n");
memset(cap, 0, sizeof *cap); memset(cap, 0, sizeof *cap);
while (regs->reg != (u16)-1 ) { while (regs->reg != (u16)-1 ) {
...@@ -649,7 +648,7 @@ static int tvp5150_set_vbi(struct v4l2_subdev *sd, ...@@ -649,7 +648,7 @@ static int tvp5150_set_vbi(struct v4l2_subdev *sd,
int pos=0; int pos=0;
if (std == V4L2_STD_ALL) { if (std == V4L2_STD_ALL) {
v4l2_err(sd, "VBI can't be configured without knowing number of lines\n"); dev_err(sd->dev, "VBI can't be configured without knowing number of lines\n");
return 0; return 0;
} else if (std & V4L2_STD_625_50) { } else if (std & V4L2_STD_625_50) {
/* Don't follow NTSC Line number convension */ /* Don't follow NTSC Line number convension */
...@@ -697,7 +696,7 @@ static int tvp5150_get_vbi(struct v4l2_subdev *sd, ...@@ -697,7 +696,7 @@ static int tvp5150_get_vbi(struct v4l2_subdev *sd,
int i, ret = 0; int i, ret = 0;
if (std == V4L2_STD_ALL) { if (std == V4L2_STD_ALL) {
v4l2_err(sd, "VBI can't be configured without knowing number of lines\n"); dev_err(sd->dev, "VBI can't be configured without knowing number of lines\n");
return 0; return 0;
} else if (std & V4L2_STD_625_50) { } else if (std & V4L2_STD_625_50) {
/* Don't follow NTSC Line number convension */ /* Don't follow NTSC Line number convension */
...@@ -712,7 +711,7 @@ static int tvp5150_get_vbi(struct v4l2_subdev *sd, ...@@ -712,7 +711,7 @@ static int tvp5150_get_vbi(struct v4l2_subdev *sd,
for (i = 0; i <= 1; i++) { for (i = 0; i <= 1; i++) {
ret = tvp5150_read(sd, reg + i); ret = tvp5150_read(sd, reg + i);
if (ret < 0) { if (ret < 0) {
v4l2_err(sd, "%s: failed with error = %d\n", dev_err(sd->dev, "%s: failed with error = %d\n",
__func__, ret); __func__, ret);
return 0; return 0;
} }
...@@ -749,7 +748,7 @@ static int tvp5150_set_std(struct v4l2_subdev *sd, v4l2_std_id std) ...@@ -749,7 +748,7 @@ static int tvp5150_set_std(struct v4l2_subdev *sd, v4l2_std_id std)
fmt = VIDEO_STD_SECAM_BIT; fmt = VIDEO_STD_SECAM_BIT;
} }
v4l2_dbg(1, debug, sd, "Set video std register to %d.\n", fmt); dev_dbg_lvl(sd->dev, 1, debug, "Set video std register to %d.\n", fmt);
tvp5150_write(sd, TVP5150_VIDEO_STD, fmt); tvp5150_write(sd, TVP5150_VIDEO_STD, fmt);
return 0; return 0;
} }
...@@ -866,7 +865,7 @@ static int tvp5150_fill_fmt(struct v4l2_subdev *sd, ...@@ -866,7 +865,7 @@ static int tvp5150_fill_fmt(struct v4l2_subdev *sd,
f->field = V4L2_FIELD_ALTERNATE; f->field = V4L2_FIELD_ALTERNATE;
f->colorspace = V4L2_COLORSPACE_SMPTE170M; f->colorspace = V4L2_COLORSPACE_SMPTE170M;
v4l2_dbg(1, debug, sd, "width = %d, height = %d\n", f->width, dev_dbg_lvl(sd->dev, 1, debug, "width = %d, height = %d\n", f->width,
f->height); f->height);
return 0; return 0;
} }
...@@ -884,7 +883,7 @@ static int tvp5150_set_selection(struct v4l2_subdev *sd, ...@@ -884,7 +883,7 @@ static int tvp5150_set_selection(struct v4l2_subdev *sd,
sel->target != V4L2_SEL_TGT_CROP) sel->target != V4L2_SEL_TGT_CROP)
return -EINVAL; return -EINVAL;
v4l2_dbg(1, debug, sd, "%s left=%d, top=%d, width=%d, height=%d\n", dev_dbg_lvl(sd->dev, 1, debug, "%s left=%d, top=%d, width=%d, height=%d\n",
__func__, rect.left, rect.top, rect.width, rect.height); __func__, rect.left, rect.top, rect.width, rect.height);
/* tvp5150 has some special limits */ /* tvp5150 has some special limits */
...@@ -1148,7 +1147,7 @@ static int tvp5150_g_register(struct v4l2_subdev *sd, struct v4l2_dbg_register * ...@@ -1148,7 +1147,7 @@ static int tvp5150_g_register(struct v4l2_subdev *sd, struct v4l2_dbg_register *
res = tvp5150_read(sd, reg->reg & 0xff); res = tvp5150_read(sd, reg->reg & 0xff);
if (res < 0) { if (res < 0) {
v4l2_err(sd, "%s: failed with error = %d\n", __func__, res); dev_err(sd->dev, "%s: failed with error = %d\n", __func__, res);
return res; return res;
} }
...@@ -1288,21 +1287,21 @@ static int tvp5150_detect_version(struct tvp5150 *core) ...@@ -1288,21 +1287,21 @@ static int tvp5150_detect_version(struct tvp5150 *core)
core->dev_id = (regs[0] << 8) | regs[1]; core->dev_id = (regs[0] << 8) | regs[1];
core->rom_ver = (regs[2] << 8) | regs[3]; core->rom_ver = (regs[2] << 8) | regs[3];
v4l2_info(sd, "tvp%04x (%u.%u) chip found @ 0x%02x (%s)\n", dev_info(sd->dev, "tvp%04x (%u.%u) chip found @ 0x%02x (%s)\n",
core->dev_id, regs[2], regs[3], c->addr << 1, core->dev_id, regs[2], regs[3], c->addr << 1,
c->adapter->name); c->adapter->name);
if (core->dev_id == 0x5150 && core->rom_ver == 0x0321) { if (core->dev_id == 0x5150 && core->rom_ver == 0x0321) {
v4l2_info(sd, "tvp5150a detected.\n"); dev_info(sd->dev, "tvp5150a detected.\n");
} else if (core->dev_id == 0x5150 && core->rom_ver == 0x0400) { } else if (core->dev_id == 0x5150 && core->rom_ver == 0x0400) {
v4l2_info(sd, "tvp5150am1 detected.\n"); dev_info(sd->dev, "tvp5150am1 detected.\n");
/* ITU-T BT.656.4 timing */ /* ITU-T BT.656.4 timing */
tvp5150_write(sd, TVP5150_REV_SELECT, 0); tvp5150_write(sd, TVP5150_REV_SELECT, 0);
} else if (core->dev_id == 0x5151 && core->rom_ver == 0x0100) { } else if (core->dev_id == 0x5151 && core->rom_ver == 0x0100) {
v4l2_info(sd, "tvp5151 detected.\n"); dev_info(sd->dev, "tvp5151 detected.\n");
} else { } else {
v4l2_info(sd, "*** unknown tvp%04x chip detected.\n", dev_info(sd->dev, "*** unknown tvp%04x chip detected.\n",
core->dev_id); core->dev_id);
} }
...@@ -1381,7 +1380,7 @@ static int tvp5150_parse_dt(struct tvp5150 *decoder, struct device_node *np) ...@@ -1381,7 +1380,7 @@ static int tvp5150_parse_dt(struct tvp5150 *decoder, struct device_node *np)
for_each_available_child_of_node(connectors, child) { for_each_available_child_of_node(connectors, child) {
ret = of_property_read_u32(child, "input", &input_type); ret = of_property_read_u32(child, "input", &input_type);
if (ret) { if (ret) {
v4l2_err(&decoder->sd, dev_err(decoder->sd.dev,
"missing type property in node %s\n", "missing type property in node %s\n",
child->name); child->name);
goto err_connector; goto err_connector;
...@@ -1396,7 +1395,7 @@ static int tvp5150_parse_dt(struct tvp5150 *decoder, struct device_node *np) ...@@ -1396,7 +1395,7 @@ static int tvp5150_parse_dt(struct tvp5150 *decoder, struct device_node *np)
/* Each input connector can only be defined once */ /* Each input connector can only be defined once */
if (input->name) { if (input->name) {
v4l2_err(&decoder->sd, dev_err(decoder->sd.dev,
"input %s with same type already exists\n", "input %s with same type already exists\n",
input->name); input->name);
ret = -EINVAL; ret = -EINVAL;
...@@ -1417,7 +1416,7 @@ static int tvp5150_parse_dt(struct tvp5150 *decoder, struct device_node *np) ...@@ -1417,7 +1416,7 @@ static int tvp5150_parse_dt(struct tvp5150 *decoder, struct device_node *np)
ret = of_property_read_string(child, "label", &name); ret = of_property_read_string(child, "label", &name);
if (ret < 0) { if (ret < 0) {
v4l2_err(&decoder->sd, dev_err(decoder->sd.dev,
"missing label property in node %s\n", "missing label property in node %s\n",
child->name); child->name);
goto err_connector; goto err_connector;
...@@ -1465,7 +1464,7 @@ static int tvp5150_probe(struct i2c_client *c, ...@@ -1465,7 +1464,7 @@ static int tvp5150_probe(struct i2c_client *c,
if (IS_ENABLED(CONFIG_OF) && np) { if (IS_ENABLED(CONFIG_OF) && np) {
res = tvp5150_parse_dt(core, np); res = tvp5150_parse_dt(core, np);
if (res) { if (res) {
v4l2_err(sd, "DT parsing error: %d\n", res); dev_err(sd->dev, "DT parsing error: %d\n", res);
return res; return res;
} }
} else { } else {
...@@ -1549,7 +1548,7 @@ static int tvp5150_remove(struct i2c_client *c) ...@@ -1549,7 +1548,7 @@ static int tvp5150_remove(struct i2c_client *c)
struct v4l2_subdev *sd = i2c_get_clientdata(c); struct v4l2_subdev *sd = i2c_get_clientdata(c);
struct tvp5150 *decoder = to_tvp5150(sd); struct tvp5150 *decoder = to_tvp5150(sd);
v4l2_dbg(1, debug, sd, dev_dbg_lvl(sd->dev, 1, debug,
"tvp5150.c: removing tvp5150 adapter on address 0x%x\n", "tvp5150.c: removing tvp5150 adapter on address 0x%x\n",
c->addr << 1); c->addr << 1);
......
...@@ -259,8 +259,7 @@ static int snd_em28xx_capture_open(struct snd_pcm_substream *substream) ...@@ -259,8 +259,7 @@ static int snd_em28xx_capture_open(struct snd_pcm_substream *substream)
int nonblock, ret = 0; int nonblock, ret = 0;
if (!dev) { if (!dev) {
dev_err(&dev->udev->dev, pr_err("em28xx-audio: BUG: em28xx can't find device struct. Can't proceed with open\n");
"BUG: em28xx can't find device struct. Can't proceed with open\n");
return -ENODEV; return -ENODEV;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment