Hi Sakari On Fri, Oct 11, 2024 at 10:55:33AM +0300, Sakari Ailus wrote: > The sensor drivers do not configure the output size of the sensors but the > entire internal pipeline. Reflect this in the documentation. > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > --- > .../userspace-api/media/drivers/camera-sensor.rst | 10 ++++++---- > 1 file changed, 6 insertions(+), 4 deletions(-) > > diff --git a/Documentation/userspace-api/media/drivers/camera-sensor.rst b/Documentation/userspace-api/media/drivers/camera-sensor.rst > index d82cd803e337..ad4049ff7eec 100644 > --- a/Documentation/userspace-api/media/drivers/camera-sensor.rst > +++ b/Documentation/userspace-api/media/drivers/camera-sensor.rst > @@ -10,11 +10,13 @@ used to control the camera sensor drivers. > > You may also find :ref:`media_writing_camera_sensor_drivers` useful. > > -Frame size > ----------- > +Sensor internal pipeline configuration > +-------------------------------------- > > -There are two distinct ways to configure the frame size produced by camera > -sensors. > +The camera sensors have an internal processing pipeline including cropping and As a non native speaker I'm probably wrong, but "The" followed by a plural name to identify a category sounds weird to me. What about just "Camera sensors have" > +binning functionality. The sensor drivers belong to two distinct classes, freely functionalities ? Same question for "The sensor drivers", just "Sensor drivers" ? > +configurable and register list based drivers, depending on how the driver s/drivers/ones ? > +configures this functionality. configures the pipeline. ? > > Freely configurable camera sensor drivers > ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > -- > 2.39.5 > >
