On 21/07/18 19:51, Tom aan de Wiel wrote:
> Hi!
>
> As of now, the version of the document I have locally is pretty much
> identical to the one you already linked to.
> I haven't had the time to shape it into a more documentation kind of document
> instead of the report format it has now.
> However, the report does contain all details and therefore could suffice as
> documentation for the time being?
OK, then I'll ask Mauro to put up the document I have on linuxtv.org so I can
refer to it.
Thanks!
Hans
>
> Tom
>
> tor. 19. jul. 2018 kl. 17:52 skrev Hans Verkuil <hverk...@xs4all.nl
> <mailto:hverk...@xs4all.nl>>:
>
> On 07/19/2018 05:39 PM, Sakari Ailus wrote:
> > On Thu, Jul 19, 2018 at 03:20:22PM +0200, Hans Verkuil wrote:
> >> On 07/19/18 15:15, sakari.ai...@iki.fi <mailto:sakari.ai...@iki.fi>
> wrote:
> >>> On Thu, Jul 19, 2018 at 02:13:50PM +0200, Hans Verkuil wrote:
> >>>> From: Hans Verkuil <hansv...@cisco.com <mailto:hansv...@cisco.com>>
> >>>>
> >>>> Add a new pixelformat for the vicodec software codec using the
> >>>> Fast Walsh Hadamard Transform.
> >>>>
> >>>> Signed-off-by: Hans Verkuil <hansv...@cisco.com
> <mailto:hansv...@cisco.com>>
> >>>
> >>> Could you add documentation for this format, please?
> >>>
> >>
> >> ??? It's part of the patch:
> >>
> >> diff --git a/Documentation/media/uapi/v4l/pixfmt-compressed.rst
> b/Documentation/media/uapi/v4l/pixfmt-compressed.rst
> >> index abec03937bb3..e5419f046b59 100644
> >> --- a/Documentation/media/uapi/v4l/pixfmt-compressed.rst
> >> +++ b/Documentation/media/uapi/v4l/pixfmt-compressed.rst
> >> @@ -95,3 +95,10 @@ Compressed Formats
> >> - ``V4L2_PIX_FMT_HEVC``
> >> - 'HEVC'
> >> - HEVC/H.265 video elementary stream.
> >> + * .. _V4L2-PIX-FMT-FWHT:
> >> +
> >> + - ``V4L2_PIX_FMT_FWHT``
> >> + - 'FWHT'
> >> + - Video elementary stream using a codec based on the Fast Walsh
> Hadamard
> >> + Transform. This codec is implemented by the vicodec ('Virtual
> Codec')
> >> + driver.
> >>
> >> Since the whole codec is implemented in the vicodec source I didn't
> think it
> >> necessary to say more about it.
> >
> > Oh, well. The source is there but user space developers shouldn't need
> to
> > read it. OTOH it might be that they're also not the primary audience for
> > this driver either. If there's a Wikipedia article you could refer to or
> > such that'd be fine IMO, too.
> >
> > Up to you.
> >
>
> Tom,
>
> Do you have a newer version of your document? If not, can I refer to the
> version
> I have? If it can be put up at the linuxtv.org <http://linuxtv.org>
> website, then that takes care of
> the documentation.
>
> Regards,
>
> Hans
>
