Message ID | 20190904125427.13943-7-jacopo@jmondi.org |
---|---|
State | Superseded |
Headers | show |
Series |
|
Related | show |
Hi Jacopo, Thank you for the patch. On Wed, Sep 04, 2019 at 02:54:26PM +0200, Jacopo Mondi wrote: > Use more opportune sizes, manually calculated, for the generated request > template. > > Signed-off-by: Jacopo Mondi <jacopo@jmondi.org> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > src/android/camera_device.cpp | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp > index e96821ad523b..dee6bcc3d6f2 100644 > --- a/src/android/camera_device.cpp > +++ b/src/android/camera_device.cpp > @@ -557,11 +557,11 @@ const camera_metadata_t *CameraDevice::constructDefaultRequestSettings(int type) > return requestTemplate_; > } > > - /* \todo Use correct sizes */ > - #define REQUEST_TEMPLATE_ENTRIES 30 > - #define REQUEST_TEMPLATE_DATA 2048 > - requestTemplate_ = allocate_camera_metadata(REQUEST_TEMPLATE_ENTRIES, > - REQUEST_TEMPLATE_DATA); > + /* > + * \todo Keep this in sync with the actual number of entries. > + * Currently: 12 entries, 15 bytes > + */ > + requestTemplate_ = allocate_camera_metadata(15, 20); > if (!requestTemplate_) { > LOG(HAL, Error) << "Failed to allocate template metadata"; > return nullptr;
diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp index e96821ad523b..dee6bcc3d6f2 100644 --- a/src/android/camera_device.cpp +++ b/src/android/camera_device.cpp @@ -557,11 +557,11 @@ const camera_metadata_t *CameraDevice::constructDefaultRequestSettings(int type) return requestTemplate_; } - /* \todo Use correct sizes */ - #define REQUEST_TEMPLATE_ENTRIES 30 - #define REQUEST_TEMPLATE_DATA 2048 - requestTemplate_ = allocate_camera_metadata(REQUEST_TEMPLATE_ENTRIES, - REQUEST_TEMPLATE_DATA); + /* + * \todo Keep this in sync with the actual number of entries. + * Currently: 12 entries, 15 bytes + */ + requestTemplate_ = allocate_camera_metadata(15, 20); if (!requestTemplate_) { LOG(HAL, Error) << "Failed to allocate template metadata"; return nullptr;
Use more opportune sizes, manually calculated, for the generated request template. Signed-off-by: Jacopo Mondi <jacopo@jmondi.org> --- src/android/camera_device.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)