Commit 084b6790 authored by Jae Hyun Yoo's avatar Jae Hyun Yoo Committed by Mauro Carvalho Chehab

media: aspeed: reduce noisy log printing outs

Currently, this driver prints out too much log messages when a
mode change happens, video turned off by screen saver and etc.
Actually, all cases are reported to user space properly. Also,
these are not critical errors but recoverable things, so this
commit changes the log level of some noisy printing outs.
Signed-off-by: default avatarJae Hyun Yoo <jae.hyun.yoo@linux.intel.com>
Reviewed-by: default avatarEddie James <eajames@linux.ibm.com>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
parent c8b996ca
...@@ -441,7 +441,7 @@ static int aspeed_video_start_frame(struct aspeed_video *video) ...@@ -441,7 +441,7 @@ static int aspeed_video_start_frame(struct aspeed_video *video)
if (!(seq_ctrl & VE_SEQ_CTRL_COMP_BUSY) || if (!(seq_ctrl & VE_SEQ_CTRL_COMP_BUSY) ||
!(seq_ctrl & VE_SEQ_CTRL_CAP_BUSY)) { !(seq_ctrl & VE_SEQ_CTRL_CAP_BUSY)) {
dev_err(video->dev, "Engine busy; don't start frame\n"); dev_dbg(video->dev, "Engine busy; don't start frame\n");
return -EBUSY; return -EBUSY;
} }
...@@ -769,7 +769,7 @@ static void aspeed_video_get_resolution(struct aspeed_video *video) ...@@ -769,7 +769,7 @@ static void aspeed_video_get_resolution(struct aspeed_video *video)
res_check(video), res_check(video),
MODE_DETECT_TIMEOUT); MODE_DETECT_TIMEOUT);
if (!rc) { if (!rc) {
dev_err(video->dev, "Timed out; first mode detect\n"); dev_dbg(video->dev, "Timed out; first mode detect\n");
clear_bit(VIDEO_RES_DETECT, &video->flags); clear_bit(VIDEO_RES_DETECT, &video->flags);
return; return;
} }
...@@ -787,7 +787,7 @@ static void aspeed_video_get_resolution(struct aspeed_video *video) ...@@ -787,7 +787,7 @@ static void aspeed_video_get_resolution(struct aspeed_video *video)
MODE_DETECT_TIMEOUT); MODE_DETECT_TIMEOUT);
clear_bit(VIDEO_RES_DETECT, &video->flags); clear_bit(VIDEO_RES_DETECT, &video->flags);
if (!rc) { if (!rc) {
dev_err(video->dev, "Timed out; second mode detect\n"); dev_dbg(video->dev, "Timed out; second mode detect\n");
return; return;
} }
...@@ -821,7 +821,7 @@ static void aspeed_video_get_resolution(struct aspeed_video *video) ...@@ -821,7 +821,7 @@ static void aspeed_video_get_resolution(struct aspeed_video *video)
} while (invalid_resolution && (tries++ < INVALID_RESOLUTION_RETRIES)); } while (invalid_resolution && (tries++ < INVALID_RESOLUTION_RETRIES));
if (invalid_resolution) { if (invalid_resolution) {
dev_err(video->dev, "Invalid resolution detected\n"); dev_dbg(video->dev, "Invalid resolution detected\n");
return; return;
} }
...@@ -1456,7 +1456,7 @@ static void aspeed_video_stop_streaming(struct vb2_queue *q) ...@@ -1456,7 +1456,7 @@ static void aspeed_video_stop_streaming(struct vb2_queue *q)
!test_bit(VIDEO_FRAME_INPRG, &video->flags), !test_bit(VIDEO_FRAME_INPRG, &video->flags),
STOP_TIMEOUT); STOP_TIMEOUT);
if (!rc) { if (!rc) {
dev_err(video->dev, "Timed out when stopping streaming\n"); dev_dbg(video->dev, "Timed out when stopping streaming\n");
/* /*
* Need to force stop any DMA and try and get HW into a good * Need to force stop any DMA and try and get HW into a good
......
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