diff options
| author | Nils Lehnen <nils.lehnen@proton.me> | 2026-07-18 00:42:40 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2026-07-28 09:23:33 +0200 |
| commit | 96ea99e3f9e7b90eca6f0122441deefd6364a048 (patch) | |
| tree | ab2d788e31fdaacc13b3aa6f5f88f714218ca242 | |
| parent | d205dfa8cb825f1954ca1cfa474fc50bf06ee4aa (diff) | |
| download | linux-96ea99e3f9e7b90eca6f0122441deefd6364a048.tar.gz linux-96ea99e3f9e7b90eca6f0122441deefd6364a048.zip | |
staging: most: video: replace pr_err with dev_err
The MOST core and the already converted net component log with device
context, while the video component still uses bare pr_err(), which
leaves its messages without any hint of the originating interface.
Both affected functions have the most_interface at hand, so log through
dev_err(iface->dev, ...) exactly like drivers/most/core.c does.
All English text in this patch (commit message and code comments) was
translated from a German draft with the assistance of Claude Fable 5.
Assisted-by: Claude:claude-fable-5 sparse
Signed-off-by: Nils Lehnen <nils.lehnen@proton.me>
Link: https://patch.msgid.link/20260718004223.54943-1-nils.lehnen@proton.me
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/most/video/video.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/staging/most/video/video.c b/drivers/staging/most/video/video.c index 78234620d48d..88892b5db28b 100644 --- a/drivers/staging/most/video/video.c +++ b/drivers/staging/most/video/video.c @@ -464,18 +464,18 @@ static int comp_probe_channel(struct most_interface *iface, int channel_idx, struct most_video_dev *mdev = get_comp_dev(iface, channel_idx); if (mdev) { - pr_err("Channel already linked\n"); + dev_err(iface->dev, "Channel already linked\n"); return -EEXIST; } if (ccfg->direction != MOST_CH_RX) { - pr_err("Wrong direction, expected rx\n"); + dev_err(iface->dev, "Wrong direction, expected rx\n"); return -EINVAL; } if (ccfg->data_type != MOST_CH_SYNC && ccfg->data_type != MOST_CH_ISOC) { - pr_err("Wrong channel type, expected sync or isoc\n"); + dev_err(iface->dev, "Wrong channel type, expected sync or isoc\n"); return -EINVAL; } @@ -495,7 +495,7 @@ static int comp_probe_channel(struct most_interface *iface, int channel_idx, strscpy(mdev->v4l2_dev.name, name, sizeof(mdev->v4l2_dev.name)); ret = v4l2_device_register(NULL, &mdev->v4l2_dev); if (ret) { - pr_err("v4l2_device_register() failed\n"); + dev_err(iface->dev, "v4l2_device_register() failed\n"); kfree(mdev); return ret; } @@ -521,7 +521,7 @@ static int comp_disconnect_channel(struct most_interface *iface, struct most_video_dev *mdev = get_comp_dev(iface, channel_idx); if (!mdev) { - pr_err("no such channel is linked\n"); + dev_err(iface->dev, "no such channel is linked\n"); return -ENOENT; } |