[PATCH 0/3] Documentation: media: camera_sensor: Document blankings handling

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello,
   this series updates the camera-sensor documentation to indicate that
vertical and horizontal blankings should be updated when a new image format
is applied on the sensor.

1/3 is more for discussion than inclusion, as I noticed the documentation and
the CCS driver use the analogue crop rectangle sizes to compute blanking limits
while most driver use the visible size.

Which ones are correct ?

As I interpret "Figure 47 Line and Frame Blanking Definitions" in CSI-2 spec (I
have version 1.01.00 r0.04 2-Apr-2009) it seems clear to me that blankings
surround the visible size ("valid data"). However it is also correct to consider
blanking limits are a property of the pixel array usually represented by a
minx/max limits of some "total output size" register.

I wonder if it makes any real difference in practice...

3/3 also documents that exposure limits should be updated when a new vertical
blankings are applied.

I haven't documented if the actual control value should be reset to some
know value as the init-time default or to its minimum value, as described in
https://lore.kernel.org/linux-media/20221019065801.4n7alfivhffbvgzo@uno.localdomain/T/

A question for Dave: should the registration of events handler callbacks be
documented too as clarified in:
https://www.spinics.net/lists/linux-media/msg220715.html

Thanks!
  j

Jacopo Mondi (3):
  Documentation: media: camera-sensor: Correct frame interval
  documentation: media: camera_sensor: Document blankings handling
  documentation: media: camera_sensor: Update exposure on blanking
    change

 .../driver-api/media/camera-sensor.rst        | 69 +++++++++++++++++--
 1 file changed, 65 insertions(+), 4 deletions(-)

--
2.38.1




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux