Hi Laurent,

On Tue, Apr 07, 2015 at 01:11:34PM +0300, Laurent Pinchart wrote:
> Hi Sakari,
> 
> Thank you for the patch.
> 
> On Tuesday 07 April 2015 01:57:29 Sakari Ailus wrote:
> > The field is unused. Remove it.
> 
> Do you know what the field was added for in the first place ?

Frankly I have to admit I have no idea. It's part of the original patch
which adds V4L2 OF support:

---
commit 99fd133f907afdb430942d8d2ae53faa438adfe8
Author: Guennadi Liakhovetski <g.liakhovet...@gmx.de>
Date:   Wed Sep 26 05:24:03 2012 -0300

    [media] Add a V4L2 OF parser
    
    Add a V4L2 OF parser, implementing bindings documented in
    Documentation/devicetree/bindings/media/video-interfaces.txt.
    [s.nawro...@samsung.com: various corrections and improvements
    since the initial version]
    
    Signed-off-by: Guennadi Liakhovetski <g.liakhovet...@gmx.de>
    Signed-off-by: Sylwester Nawrocki <s.nawro...@samsung.com>
    Signed-off-by: Kyungmin Park <kyungmin.p...@samsung.com>
    Signed-off-by: Mauro Carvalho Chehab <mche...@redhat.com>
---

It looks like the intent has been that the field is used in order to keep a
list of structs of this kind, but no-one is using it for that purpose at the
moment.

-- 
Regards,

Sakari Ailus
e-mail: sakari.ai...@iki.fi     XMPP: sai...@retiisi.org.uk
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to