Re: [PATCH RFC v6 2/5] Documentation: iio: add Open Sensor Fusion driver overview

[Jonathan Cameron]

On Tue, 30 Jun 2026 14:04:43 +0900
Kim Jinseob <kimjinseob88@xxxxxxxxx> wrote:

> I will remove "initial" there in the next revision.
> 
Hi Jinseob

A small process thing that a lot of people get 'wrong' is to reply when only
saying you are making the suggested change. That adds a lot of noise to the
mailing list. The correct place for this information is in the change log
for the next version.

Thanks,

Jonathan

> Thanks
> 
> Jinseob
> 
> 2026년 6월 30일 (화) 오전 8:03, Jonathan Cameron <jic23@xxxxxxxxxx>님이 작성:
> >
> > On Mon, 29 Jun 2026 04:13:34 +0900
> > Jinseob Kim <kimjinseob88@xxxxxxxxx> wrote:
> >  
> > > Document the Linux IIO mapping for Open Sensor Fusion devices.
> > >
> > > The overview explains that sensor channels are discovered at runtime
> > > from mandatory capability reports. It also documents that OSF0 is a
> > > wire-format detail and that protocol_major and protocol_minor carry
> > > protocol compatibility information.
> > >
> > > Signed-off-by: Jinseob Kim <kimjinseob88@xxxxxxxxx>  
> >
> > One small thing inline.  Otherwise looks good to me!
> >
> > Jonathan
> >  
> > > diff --git a/Documentation/iio/open-sensor-fusion.rst b/Documentation/iio/open-sensor-fusion.rst
> > > new file mode 100644
> > > index 000000000..832901f5e
> > > --- /dev/null
> > > +++ b/Documentation/iio/open-sensor-fusion.rst  
> >  
> > > +Timestamps
> > > +----------
> > > +
> > > +OSF frames include a device-side ``timestamp_us`` field. Buffered IIO samples use
> > > +an IIO timestamp captured on the host when samples are pushed to IIO buffers.
> > > +The initial driver does not correlate the device timestamp with the host IIO
> > > +clock.  
> >
> > Really small thing but I would avoid talking about 'initial' driver.
> > It can cause confusion about when something is true and when it is not
> > as we may have multiple non 'initial' drivers before you get to changing
> > this handling. So just remove that word.
> >