@@ -1066,9 +1066,9 @@ int CameraCapabilities::initializeStaticMetadata()
/* Sensor static metadata. */
std::array<int32_t, 2> pixelArraySize;
{
- const Size &size = properties.get(properties::PixelArraySize).value_or(utils::defopt);
- pixelArraySize[0] = size.width;
- pixelArraySize[1] = size.height;
+ const Rectangle &area = properties.get(properties::PixelArrayArea).value_or(utils::defopt);
+ pixelArraySize[0] = area.width;
+ pixelArraySize[1] = area.height;
staticMetadata_->addEntry(ANDROID_SENSOR_INFO_PIXEL_ARRAY_SIZE,
pixelArraySize);
}
@@ -597,7 +597,7 @@ int UVCCameraData::init(std::shared_ptr<MediaDevice> media)
properties_.set(properties::Location, location);
- properties_.set(properties::PixelArraySize, resolution);
+ properties_.set(properties::PixelArrayArea, Rectangle(resolution));
properties_.set(properties::PixelArrayActiveAreas, { Rectangle(resolution) });
/* Initialise the supported controls. */
@@ -424,16 +424,30 @@ controls:
The property can be used to calculate the physical size of the sensor's
pixel array area and for calibration purposes.
- - PixelArraySize:
- type: Size
+ - PixelArrayArea:
+ type: Rectangle
description: |
- The camera sensor pixel array readable area vertical and horizontal
- sizes, in pixels.
+ The camera sensor pixel array area, in pixels.
- The PixelArraySize property defines the size in pixel units of the
- readable part of full pixel array matrix, including optical black
+ The PixelArrayArea property defines the size in pixel units of the full
+ pixel array matrix including readable and non-readable pixels. The top
+ left corner of this rectangle is always at (0,0). For the largest
+ readable area \see PixelArrayReadableArea.
+
+ All other properties that describe portions of the pixel array such as
+ PixelArrayActiveAreas and PixelArrayReadableArea are defined relative to
+ this rectangle.
+
+ - PixelArrayReadableArea:
+ type: Rectangle
+ description: |
+ The camera sensor pixel array readable area.
+
+ The PixelArrayReadableArea property defines the rectangle in pixel units
+ of the readable part of full pixel array matrix, including optical black
pixels used for calibration, pixels which are not considered valid for
- capture and active pixels containing valid image data.
+ capture and active pixels containing valid image data. It is defined
+ relative to the PixelArrayArea rectangle.
The property describes the maximum size of the raw data captured by the
camera, which might not correspond to the physical size of the sensor
@@ -442,23 +456,25 @@ controls:
For example, let's consider a pixel array matrix assembled as follows
- +--------------------------------------------------+
- |xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx|
- |xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx|
- |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx|
- |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx|
- |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx|
- |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx|
- |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx|
- |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx|
+ PixelArrayArea.width
+ /--------------------------------------------------/
+ +--------------------------------------------------+ /
+ |xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx| |
+ |xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx| |
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| |
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| |
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| |
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| |
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| |
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| | PixelArrayArea.height
... ... ... ... ...
... ... ... ... ...
- |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx|
- |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx|
- |xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx|
- |xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx|
- +--------------------------------------------------+
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| |
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| |
+ |xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx| |
+ |xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx| |
+ +--------------------------------------------------+ /
starting with two lines of non-readable pixels (x), followed by N lines
of readable data (D) surrounded by two columns of non-readable pixels on
@@ -467,35 +483,32 @@ controls:
sizes of the largest possible buffer of raw data that can be presented
to applications.
- PixelArraySize.width
- /----------------------------------------------/
- +----------------------------------------------+ /
- |DDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDD| |
- |DDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDD| |
- |DDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDD| |
- |DDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDD| |
- |DDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDD| |
- |DDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDD| | PixelArraySize.height
- ... ... ... ... ...
- ... ... ... ... ...
- |DDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDD| |
- |DDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDD| |
- +----------------------------------------------+ /
+ PixelArrayReadableArea.width
+ /--------------------------------------------/
+ PixelArrayReadableArea.x
+ +--o-----------------------------------------------+
+ |xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx|
+ |xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx|
+ oxxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| / PixelArrayReadableArea.y
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| |
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| |
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| |
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| |
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| | PixelArrayReadableArea.height
+ ... ... ... ... ...
- This defines a rectangle whose top-left corner is placed in position (0,
- 0) and whose vertical and horizontal sizes are defined by this property.
- All other rectangles that describe portions of the pixel array, such as
- the optical black pixels rectangles and active pixel areas, are defined
- relatively to this rectangle.
+ ... ... ... ... ...
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| |
+ |xxDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDxx| /
+ |xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx|
+ |xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx|
+ +--------------------------------------------------+
All the coordinates are expressed relative to the default sensor readout
direction, without any transformation (such as horizontal and vertical
flipping) applied. When mapping them to the raw pixel buffer,
applications shall take any configured transformation into account.
- \todo Rename this property to Size once we will have property
- categories (i.e. Properties::PixelArray::Size)
-
- PixelArrayActiveAreas:
type: Rectangle
size: [n]
@@ -507,7 +520,7 @@ controls:
This property describes an arbitrary number of overlapping rectangles,
with each rectangle representing the maximum image size that the camera
sensor can produce for a particular aspect ratio. They are defined
- relatively to the PixelArraySize rectangle.
+ relative to the PixelArrayArea rectangle.
When multiple rectangles are reported, they shall be ordered from the
tallest to the shortest.
@@ -519,13 +532,13 @@ controls:
and/or applying pixel sub-sampling techniques such as pixel skipping or
binning.
- PixelArraySize.width
+ PixelArrayArea.width
/----------------/
x1 x2
(0,0)-> +-o------------o-+ /
y1 o +------------+ | |
| |////////////| | |
- | |////////////| | | PixelArraySize.height
+ | |////////////| | | PixelArrayArea.height
| |////////////| | |
y2 o +------------+ | |
+----------------+ /
@@ -539,14 +552,14 @@ controls:
resolutions will report several overlapping rectangles, one for each
natively supported resolution.
- PixelArraySize.width
+ PixelArrayArea.width
/------------------/
x1 x2 x3 x4
(0,0)-> +o---o------o---o+ /
y1 o +------+ | |
| |//////| | |
y2 o+---+------+---+| |
- ||///|//////|///|| | PixelArraySize.height
+ ||///|//////|///|| | PixelArrayArea.height
y3 o+---+------+---+| |
| |//////| | |
y4 o +------+ | |
@@ -131,7 +131,8 @@ private:
std::vector<controls::draft::TestPatternModeEnum> testPatternModes_;
controls::draft::TestPatternModeEnum testPatternMode_;
- Size pixelArraySize_;
+ Rectangle pixelArrayArea_;
+ Rectangle readableArea_;
Rectangle activeArea_;
const BayerFormat *bayerFormat_;
bool supportHFlips_;
@@ -396,33 +397,40 @@ int CameraSensorLegacy::validateSensorDriver()
* \todo Make support for selection targets mandatory as soon as all
* test platforms have been updated.
*/
- Rectangle rect;
- int ret = subdev_->getSelection(pad_, V4L2_SEL_TGT_CROP_BOUNDS, &rect);
+ int ret = subdev_->getSelection(pad_, V4L2_SEL_TGT_NATIVE_SIZE, &pixelArrayArea_);
if (ret) {
/*
* Default the pixel array size to the largest size supported
* by the sensor. The sizes_ vector is sorted in ascending
* order, the largest size is thus the last element.
*/
- pixelArraySize_ = sizes_.back();
+ pixelArrayArea_ = Rectangle(sizes_.back());
LOG(CameraSensor, Warning)
- << "The PixelArraySize property has been defaulted to "
- << pixelArraySize_;
+ << "The PixelArrayArea property has been defaulted to "
+ << pixelArrayArea_;
+ err = -EINVAL;
+ }
+
+ ret = subdev_->getSelection(pad_, V4L2_SEL_TGT_CROP_BOUNDS, &readableArea_);
+ if (ret) {
+ readableArea_ = pixelArrayArea_;
+ LOG(CameraSensor, Warning)
+ << "The PixelArrayReadableArea property has been defaulted to "
+ << readableArea_;
err = -EINVAL;
- } else {
- pixelArraySize_ = rect.size();
}
ret = subdev_->getSelection(pad_, V4L2_SEL_TGT_CROP_DEFAULT, &activeArea_);
if (ret) {
- activeArea_ = Rectangle(pixelArraySize_);
+ activeArea_ = readableArea_;
LOG(CameraSensor, Warning)
<< "The PixelArrayActiveAreas property has been defaulted to "
<< activeArea_;
err = -EINVAL;
}
+ Rectangle rect;
ret = subdev_->getSelection(pad_, V4L2_SEL_TGT_CROP, &rect);
if (ret) {
LOG(CameraSensor, Warning)
@@ -476,8 +484,9 @@ int CameraSensorLegacy::validateSensorDriver()
void CameraSensorLegacy::initVimcDefaultProperties()
{
/* Use the largest supported size. */
- pixelArraySize_ = sizes_.back();
- activeArea_ = Rectangle(pixelArraySize_);
+ pixelArrayArea_ = Rectangle(sizes_.back());
+ readableArea_ = pixelArrayArea_;
+ activeArea_ = pixelArrayArea_;
}
void CameraSensorLegacy::initStaticProperties()
@@ -627,7 +636,8 @@ int CameraSensorLegacy::initProperties()
mountingOrientation_ = Orientation::Rotate0;
}
- properties_.set(properties::PixelArraySize, pixelArraySize_);
+ properties_.set(properties::PixelArrayArea, pixelArrayArea_);
+ properties_.set(properties::PixelArrayReadableArea, readableArea_);
properties_.set(properties::PixelArrayActiveAreas, { activeArea_ });
/* Color filter array pattern, register only for RAW sensors. */
@@ -141,7 +141,8 @@ private:
std::vector<controls::draft::TestPatternModeEnum> testPatternModes_;
controls::draft::TestPatternModeEnum testPatternMode_;
- Size pixelArraySize_;
+ Rectangle pixelArrayArea_;
+ Rectangle readableArea_;
Rectangle activeArea_;
BayerFormat::Order cfaPattern_;
bool supportHFlips_;
@@ -431,7 +432,17 @@ std::optional<int> CameraSensorRaw::init()
return { ret };
}
- pixelArraySize_ = rect.size();
+ /*
+ * \todo Implement querying the physical sensor size based on pad 1,
+ * stream 0. See
+ * https://lore.kernel.org/linux-media/20260409201501.975242-23-sakari.ailus@linux.intel.com/
+ */
+ LOG(CameraSensor, Warning) << "PixelArrayArea might be incorrect";
+ pixelArrayArea_ = Rectangle(0, 0,
+ rect.x + rect.width,
+ rect.y + rect.height);
+
+ readableArea_ = rect;
ret = subdev_->getSelection(streams_.image.sink, V4L2_SEL_TGT_CROP_DEFAULT,
&activeArea_);
@@ -634,7 +645,8 @@ int CameraSensorRaw::initProperties()
mountingOrientation_ = Orientation::Rotate0;
}
- properties_.set(properties::PixelArraySize, pixelArraySize_);
+ properties_.set(properties::PixelArrayArea, pixelArrayArea_);
+ properties_.set(properties::PixelArrayReadableArea, readableArea_);
properties_.set(properties::PixelArrayActiveAreas, { activeArea_ });
/* Color filter array pattern. */