<div dir="ltr"><div dir="ltr">Hi Jacopo,<div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 19 Jan 2021 at 15:15, Jacopo Mondi <<a href="mailto:jacopo@jmondi.org">jacopo@jmondi.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Naush,<br>
<br>
On Thu, Jan 14, 2021 at 07:34:20AM +0000, Naushir Patuck wrote:<br>
> Hi Laurent,<br>
><br>
><br>
> On Thu, 14 Jan 2021 at 06:04, Laurent Pinchart <<br>
> <a href="mailto:laurent.pinchart@ideasonboard.com" target="_blank">laurent.pinchart@ideasonboard.com</a>> wrote:<br>
><br>
> > Hi Naush,<br>
> ><br>
<br>
[snip]<br>
<br>
> > > ><br>
> > > > >            When reported in<br>
> > > > > +          metadata, the control expresses the minimum and maximum<br>
> > frame<br>
> > > > > +          durations used after being clipped to these limits.<br>
> > > > > +<br>
> > > ><br>
> > > > But this sounds weird. The previous part states that FrameDurations has<br>
> > > > higher priority than all other parameters, but then this sentence says<br>
> > > > it's clipped by "these limits".<br>
> > ><br>
> > > You are right, this does not read correct.  I wanted to express that the<br>
> > > frame durations provided may be further limited by what the sensor mode<br>
> > can<br>
> > > actually achieve.  How about replacing the above paragraph of text with<br>
> > the<br>
> > > following:<br>
> > ><br>
> > > When reported in metadata, the control expresses the minimum and maximum<br>
> > > frame durations used after being clipped to the sensor provided frame<br>
> > > duration limits.<br>
> ><br>
> > Sounds good to me.<br>
> ><br>
> > > > > +          \sa AeExposureMode<br>
> > > > > +          \sa ExposureTime<br>
> > > > > +<br>
> > > > > +          \todo Refer to the frame duration limits property to<br>
> > describe how<br>
> > > > > +          application-provided values gets clipped and reset.<br>
> > > ><br>
> > > > It hasn't been long, and the context is already starting to escape me.<br>
> > > > Would it be possible to expand this just a little bit so that we'll<br>
> > know<br>
> > > > what it means in 3 months time ?<br>
> > ><br>
> > > Perhaps this makes more sense given the rewording above?  Or maybe a<br>
> > reword<br>
> > > as follows:<br>
> > ><br>
> > > \todo Refer to the frame duration limits property to describe how<br>
> > > application-provided values get clipped to what is supported by the<br>
> > sensor<br>
> > > mode.<br>
> > ><br>
> > > Hopefully that makes things more readable?<br>
<br>
I know where this last statment came from, as I had a frame durations<br>
limit property on its way. But as it has not been finalized, and I'm<br>
currently questioning if it is really required or not, can we drop<br>
this last part so that the last obstacle for this series to be merged<br>
is removed ?<br></blockquote><div><br></div><div>Yes, I am fine with removing this todo statement if this property is not confirmed to be introduced.  I will post an update with it removed, as well as the previous wording changes as discussed with Laurent.  Hopefully it should then be good for merging.</div><div><br></div><div>Regards,</div><div>Naush</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks<br>
   j<br>
<br>
> ><br>
> > Not quite I'm afraid, but maybe it's just too early in the morning :-)<br>
> ><br>
> > Is this about documenting how other properties also get clipped by the<br>
> > sensor mode ? Or something else ?<br>
> ><br>
><br>
> It's about how the frame durations are clipped by the sensor mode limits -<br>
> as advertised by the sensor properties in the future.<br>
> We can remove this statement entirely if you do not think it's appropriate,<br>
> or a rewording as follows:<br>
><br>
> \todo Refer to the frame duration limits property (when available) to<br>
> obtain sensor<br>
> mode limits used for clipping the application-provided values.<br>
><br>
> Regards,<br>
> Naush<br>
><br>
><br>
><br>
> ><br>
> > > > > +<br>
> > > > > +          \todo Define how to calculate the capture frame rate by<br>
> > > > > +          defining controls to report additional delays introduced<br>
> > by<br>
> > > > > +          the capture pipeline or post-processing stages (ie JPEG<br>
> > > > > +          conversion, frame scaling).<br>
> > > > > +<br>
> > > > > +          \todo Provide an explicit definition of default control<br>
> > values, for<br>
> > > > > +          this and all other controls.<br>
> > > > > +      size: [2]<br>
> > > > > +<br>
> > > > >    #<br>
> > ----------------------------------------------------------------------------<br>
> > > > >    # Draft controls section<br>
> > > > ><br>
> ><br>
> > --<br>
> > Regards,<br>
> ><br>
> > Laurent Pinchart<br>
> ><br>
<br>
> _______________________________________________<br>
> libcamera-devel mailing list<br>
> <a href="mailto:libcamera-devel@lists.libcamera.org" target="_blank">libcamera-devel@lists.libcamera.org</a><br>
> <a href="https://lists.libcamera.org/listinfo/libcamera-devel" rel="noreferrer" target="_blank">https://lists.libcamera.org/listinfo/libcamera-devel</a><br>
<br>
</blockquote></div></div>