Skip to content

Commit 0850ee8

Browse files
committed
samples: drivers: video: capture: logging improvements
Remove trailing spaces at the end of log string. Move arguments on the next line for readability. Align arguments to first line. Signed-off-by: Josuah Demangeon <me@josuah.net>
1 parent 55eb71f commit 0850ee8

File tree

1 file changed

+14
-14
lines changed
  • samples/drivers/video/capture/src

1 file changed

+14
-14
lines changed

samples/drivers/video/capture/src/main.c

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -37,10 +37,10 @@ static inline int display_setup(const struct device *const display_dev, const ui
3737

3838
LOG_INF("- Capabilities:");
3939
LOG_INF(" x_resolution = %u, y_resolution = %u, supported_pixel_formats = %u"
40-
" current_pixel_format = %u, current_orientation = %u",
41-
capabilities.x_resolution, capabilities.y_resolution,
42-
capabilities.supported_pixel_formats, capabilities.current_pixel_format,
43-
capabilities.current_orientation);
40+
" current_pixel_format = %u, current_orientation = %u",
41+
capabilities.x_resolution, capabilities.y_resolution,
42+
capabilities.supported_pixel_formats, capabilities.current_pixel_format,
43+
capabilities.current_orientation);
4444

4545
/* Set display pixel format to match the one in use by the camera */
4646
switch (pixfmt) {
@@ -132,9 +132,9 @@ int main(void)
132132
const struct video_format_cap *fcap = &caps.format_caps[i];
133133
/* fourcc to string */
134134
LOG_INF(" %s width [%u; %u; %u] height [%u; %u; %u]",
135-
VIDEO_FOURCC_TO_STR(fcap->pixelformat),
136-
fcap->width_min, fcap->width_max, fcap->width_step,
137-
fcap->height_min, fcap->height_max, fcap->height_step);
135+
VIDEO_FOURCC_TO_STR(fcap->pixelformat),
136+
fcap->width_min, fcap->width_max, fcap->width_step,
137+
fcap->height_min, fcap->height_max, fcap->height_step);
138138
i++;
139139
}
140140

@@ -168,20 +168,20 @@ int main(void)
168168

169169
if (!video_get_frmival(video_dev, &frmival)) {
170170
LOG_INF("- Default frame rate : %f fps",
171-
1.0 * frmival.denominator / frmival.numerator);
171+
1.0 * frmival.denominator / frmival.numerator);
172172
}
173173

174174
LOG_INF("- Supported frame intervals for the default format:");
175175
memset(&fie, 0, sizeof(fie));
176176
fie.format = &fmt;
177177
while (video_enum_frmival(video_dev, &fie) == 0) {
178178
if (fie.type == VIDEO_FRMIVAL_TYPE_DISCRETE) {
179-
LOG_INF(" %u/%u ", fie.discrete.numerator, fie.discrete.denominator);
179+
LOG_INF(" %u/%u", fie.discrete.numerator, fie.discrete.denominator);
180180
} else {
181181
LOG_INF(" [min = %u/%u; max = %u/%u; step = %u/%u]",
182-
fie.stepwise.min.numerator, fie.stepwise.min.denominator,
183-
fie.stepwise.max.numerator, fie.stepwise.max.denominator,
184-
fie.stepwise.step.numerator, fie.stepwise.step.denominator);
182+
fie.stepwise.min.numerator, fie.stepwise.min.denominator,
183+
fie.stepwise.max.numerator, fie.stepwise.max.denominator,
184+
fie.stepwise.step.numerator, fie.stepwise.step.denominator);
185185
}
186186
fie.index++;
187187
}
@@ -263,8 +263,8 @@ int main(void)
263263
return 0;
264264
}
265265

266-
LOG_DBG("Got frame %u! size: %u; timestamp %u ms", frame++, vbuf->bytesused,
267-
vbuf->timestamp);
266+
LOG_DBG("Got frame %u! size: %u; timestamp %u ms",
267+
frame++, vbuf->bytesused, vbuf->timestamp);
268268

269269
#ifdef CONFIG_TEST
270270
if (is_colorbar_ok(vbuf->buffer, fmt)) {

0 commit comments

Comments
 (0)