| Message ID | 20200804214711.177645-8-kieran.bingham@ideasonboard.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series |
|
| Related | show |
Hi Kieran, sorry missed this in v2 review On Tue, Aug 04, 2020 at 10:47:05PM +0100, Kieran Bingham wrote: > If an error is generated, try to be verbose about it in the libcamera logs. > > Reviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se> > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com> Reviewed-by: Jacopo Mondi <jacopo@jmondi.org> Thanks j > > --- > v2: > - Add todo to note possible debug info expansion > - Fix typo > - Print .toString() formatted pixel format > > v3: > - Remove colon from todo: > --- > src/android/camera_device.cpp | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp > index 84d8c32a90c2..e4dce680d46f 100644 > --- a/src/android/camera_device.cpp > +++ b/src/android/camera_device.cpp > @@ -1258,6 +1258,13 @@ void CameraDevice::notifyError(uint32_t frameNumber, camera3_stream_t *stream) > { > camera3_notify_msg_t notify = {}; > > + /* > + * \todo Report and identify the stream number or configuration to > + * clarify the stream that failed. > + */ > + LOG(HAL, Error) << "Error occurred on frame " << frameNumber << " (" > + << toPixelFormat(stream->format).toString() << ")"; > + > notify.type = CAMERA3_MSG_ERROR; > notify.message.error.error_stream = stream; > notify.message.error.frame_number = frameNumber; > -- > 2.25.1 > > _______________________________________________ > libcamera-devel mailing list > libcamera-devel@lists.libcamera.org > https://lists.libcamera.org/listinfo/libcamera-devel
diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp index 84d8c32a90c2..e4dce680d46f 100644 --- a/src/android/camera_device.cpp +++ b/src/android/camera_device.cpp @@ -1258,6 +1258,13 @@ void CameraDevice::notifyError(uint32_t frameNumber, camera3_stream_t *stream) { camera3_notify_msg_t notify = {}; + /* + * \todo Report and identify the stream number or configuration to + * clarify the stream that failed. + */ + LOG(HAL, Error) << "Error occurred on frame " << frameNumber << " (" + << toPixelFormat(stream->format).toString() << ")"; + notify.type = CAMERA3_MSG_ERROR; notify.message.error.error_stream = stream; notify.message.error.frame_number = frameNumber;