cron job: media_tree daily build: WARNINGS

This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Sun Aug 23 04:00:15 CEST 2015 git branch: test git hash: 3a6b0605c73d1d695f6d4e49289deaa3fa3e73ee gcc versi

[PATCH] [media] dvbdev: document most of the functions/data structs

Document the most relevant functions and data structs for developers that are working with the DVB subsystem. Signed-off-by: Mauro Carvalho Chehab diff --git a/Documentation/DocBook/device-drivers.tmpl b/Documentation/DocBook/device-drivers.tmpl index 5c9375b98303..93b74884ae24 100644 --- a/Doc

Re: [PATCH] lib: scatterlist: add sg splitting function

Robert Jarzmik writes: ...zip... top posting in [1] ... zip ... Hi Andrew, There was no review for a couple of weeks I'm afraid on this patch. Could I know what I can do to push it up ? And another question : do you want another patch to add a MAINTAINERS entry for this sg_split ? Cheers. --

Re: [PATCH 33/39] fixup: dvb_tuner_info

Em Sat, 22 Aug 2015 14:28:18 -0300 Mauro Carvalho Chehab escreveu: > Signed-off-by: Mauro Carvalho Chehab > > diff --git a/drivers/media/dvb-core/dvb_frontend.h > b/drivers/media/dvb-core/dvb_frontend.h > index afcd02932a92..8fc8d3a98382 100644 > --- a/drivers/media/dvb-core/dvb_frontend.h > +

[PATCH 01/39] [media] DocBook: fix an unbalanced tag

The got lost on some change at the DVB docbook, making the tag unbalanced and causing a warning. Fix it. Signed-off-by: Mauro Carvalho Chehab diff --git a/Documentation/DocBook/media/dvb/intro.xml b/Documentation/DocBook/media/dvb/intro.xml index d30751338493..51db15648099 100644 --- a/Documen

[PATCH 23/39] [media] v4l2-event.h: fix comments and add to DocBook

The comments there are good enough for DocBook, however they're using a wrong format. Fix and add to device-drivers.xml. Signed-off-by: Mauro Carvalho Chehab diff --git a/Documentation/DocBook/device-drivers.tmpl b/Documentation/DocBook/device-drivers.tmpl index 5f01f7ad15dc..46e6818b95ce 10064

[PATCH 02/39] [media] DocBook/device-drivers: Add drivers/media core stuff

There are lots of docbook marks at the media subsystem, but those aren't used. Add the core headers/code in order to start generating docs. Signed-off-by: Mauro Carvalho Chehab diff --git a/Documentation/DocBook/device-drivers.tmpl b/Documentation/DocBook/device-drivers.tmpl index faf09d4a0ea8

[PATCH 04/39] [media] DocBook/media/Makefile: Avoid make htmldocs to fail

If make is called twice like that: make V=1 DOCBOOKS=device-drivers.xml htmldocs Make will fail with: make -f ./scripts/Makefile.build obj=scripts/basic rm -f .tmp_quiet_recordmcount make -f ./scripts/Makefile.build obj=scripts build_docproc make -f ./scrip

[PATCH 00/39] Document the kABI for the media subsystem

Despite the media system being there for years and several files have already doc-nano tags, we never actually added a Media drivers section at device-drivers. Add one and add an initial set of documentation there for the kABI. The kAPI is already documented on a separate DocBook. Jon, As most o

[PATCH 03/39] [media] Docbook: Fix description of struct media_devnode

Warning(.//include/media/media-devnode.h:80): No description found for parameter 'fops' Warning(.//include/media/media-devnode.h:80): No description found for parameter 'dev' Warning(.//include/media/media-devnode.h:80): No description found for parameter 'cdev' Warning(.//include/media/media-de

[PATCH 05/39] [media] Docbook: Fix comments at v4l2-async.h

Warning(.//include/media/v4l2-async.h:62): No description found for parameter 'match_type' Warning(.//include/media/v4l2-async.h:62): Excess struct/union/enum/typedef member 'bus_type' description in 'v4l2_async_subdev' Warning(.//include/media/v4l2-async.h:76): cannot understand function protot

[PATCH 26/39] [media] videobuf2-memops.h: add to device-drivers DocBook

The comment metadata was wrong: Warning(.//include/media/videobuf2-memops.h:25): cannot understand function prototype: 'struct vb2_vmarea_handler ' Warning(.//include/media/videobuf2-memops.h): no structured comments found Fix and add to DocBook. Signed-off-by: Mauro Carvalho Chehab diff --gi

[PATCH 27/39] [media] v4l2-mediabus: Add to DocBook

Fix the format of the comments and add to DocBook. Signed-off-by: Mauro Carvalho Chehab diff --git a/Documentation/DocBook/device-drivers.tmpl b/Documentation/DocBook/device-drivers.tmpl index 8d1e04c94dce..53a7c5d8b996 100644 --- a/Documentation/DocBook/device-drivers.tmpl +++ b/Documentation/

[PATCH 32/39] [media] Docbook: Document struct analog_parameters

That struct inside dvb-frontend.h stores some parameters from V4L2 API (videodev2.h), in order to be used by the hybrid analog/digital TV tuners. Document it. Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/dvb-core/dvb_frontend.h b/drivers/media/dvb-core/dvb_frontend.h index 1

[PATCH 24/39] [media] v4l-dv-timings.h: Add to device-drivers DocBook

There are already markups for documentation at v4l-dv-timings.h, however, they're not properly formatted. Convert them to the right format and add this file to the device-drivers DocBook. Signed-off-by: Mauro Carvalho Chehab diff --git a/Documentation/DocBook/device-drivers.tmpl b/Documentatio

[PATCH 18/39] [media] DocBook: add dvb_math.h to documentation

There are already some comments at dvb_math.h that are ready for DocBook, although not properly formatted. Convert them, fix some issues and add this file to the device-drivers DocBook. Signed-off-by: Mauro Carvalho Chehab diff --git a/Documentation/DocBook/device-drivers.tmpl b/Documentation/

[PATCH 29/39] [media] dvb_frontend: document dvb_frontend_tune_settings

Add Documentation for dvb_frontend_tune_settings struct. Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/dvb-core/dvb_frontend.h b/drivers/media/dvb-core/dvb_frontend.h index d20d3da97201..ea68ba61f0de 100644 --- a/drivers/media/dvb-core/dvb_frontend.h +++ b/drivers/media/dvb-co

[PATCH 25/39] [media] videobuf2-core: Add it to device-drivers DocBook

Most of the stuff at videobuf2-core are ok for adding it to DocBook. Two notes here: 1) As videobuf2-core will be soon be changed, better to not spend too much efforts right now, as things will change soon; 2) struct vb2_queue has a number of private elements that are documented. As Kern

[PATCH 21/39] [media] v4l2-ctls: don't document v4l2_ctrl_fill()

This is an obsolete function that has several missing arguments: Warning(.//include/media/v4l2-ctrls.h:340): No description found for parameter 'id' Warning(.//include/media/v4l2-ctrls.h:340): No description found for parameter 'name' Warning(.//include/media/v4l2-ctrls.h:

[PATCH 10/39] [media] v4l2-subdev: Add description for core ioctl handlers

Warning(.//include/media/v4l2-subdev.h:183): No description found for parameter 'queryctrl' Warning(.//include/media/v4l2-subdev.h:183): No description found for parameter 'g_ctrl' Warning(.//include/media/v4l2-subdev.h:183): No description found for parameter 's_ctrl' Warning(.//include/media/v

[PATCH 28/39] [media] DocBook: Better organize media devices

Instead of putting all media devices on a flat structure, split them on 4 types, just like we do with the userspace API: Video2Linux devices Digital TV (DVB) devices Remote Controller devices Media Controller devices Signed-off-by: Mauro Carvalho Chehab diff --git

[PATCH 20/39] [media] v4l2-ctrls.h: add to device-drivers DocBook

The comments there are using a wrong format. Due to that, DocBook were unable to parse it. Fix the tags format, and add it to device-drivers.xml. Signed-off-by: Mauro Carvalho Chehab diff --git a/Documentation/DocBook/device-drivers.tmpl b/Documentation/DocBook/device-drivers.tmpl index 030b38

[PATCH 07/39] [media] Docbook: fix comments at v4l2-flash-led-class.h

Warning(.//include/media/v4l2-flash-led-class.h:51): bad line: unique in the system Warning(.//include/media/v4l2-flash-led-class.h:56): bad line: definitions are available in the header file Warning(.//include/media/v4l2-flash-led-class.h:57): ba

[PATCH 36/39] [media] dvb: Use DVBFE_ALGO_HW where applicable

The dvb_frontend.c core defines a FE_ALGO_HW symbol that it is never used. Also, both cx24123 returns 1 to get_algo() callback instead of using DVBFE_ALGO_HW. Probably, those are some left overs from some code cleanup. Let's stop returning magic numbers and use the proper macro value. Signed-off

[PATCH 33/39] fixup: dvb_tuner_info

Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/dvb-core/dvb_frontend.h b/drivers/media/dvb-core/dvb_frontend.h index afcd02932a92..8fc8d3a98382 100644 --- a/drivers/media/dvb-core/dvb_frontend.h +++ b/drivers/media/dvb-core/dvb_frontend.h @@ -76,7 +76,8 @@ struct dvb_frontend;

[PATCH 16/39] [media] DocBook: add dvb_ca_en50221.h to documentation

There are already some tags at dvb_ca_en50221.h, but using a different format. Convert them, fix a few entries and add to the device-drivers DocBook. Signed-off-by: Mauro Carvalho Chehab diff --git a/Documentation/DocBook/device-drivers.tmpl b/Documentation/DocBook/device-drivers.tmpl index e3e

[PATCH 35/39] [media] dvb_frontend.h: document struct analog_demod_ops

Add documentation for struct analog_demod_info and struct analog_demod_ops. Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/dvb-core/dvb_frontend.h b/drivers/media/dvb-core/dvb_frontend.h index 991a6e0265d2..c6c85e6e9874 100644 --- a/drivers/media/dvb-core/dvb_frontend.h +++ b/d

[PATCH 13/39] [media] v4l2_subdev: describe ioctl parms at v4l2_subdev_video_ops

Warning(.//include/media/v4l2-subdev.h:381): No description found for parameter 'g_std' Warning(.//include/media/v4l2-subdev.h:381): No description found for parameter 's_std' Warning(.//include/media/v4l2-subdev.h:381): No description found for parameter 'querystd' Warning(.//include/media/v4l2

[PATCH 34/39] [media] dvb_frontend.h: Document struct dvb_tuner_ops

The struct dvb_tuner_ops contains lots of callbacks used by tuners. Document them. Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/dvb-core/dvb_frontend.h b/drivers/media/dvb-core/dvb_frontend.h index 8fc8d3a98382..991a6e0265d2 100644 --- a/drivers/media/dvb-core/dvb_frontend.h

[PATCH 37/39] [media] dvb-frontend.h: document struct dvb_frontend_ops

This is one of the most important functions of the DVB frontend, containing the logic needed to set the parameters at the demux and to send commands via SEC. Document it. Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/dvb-core/dvb_frontend.h b/drivers/media/dvb-core/dvb_fronte

[PATCH 39/39] [media] dvb_frontend.h: document the struct dvb_frontend

That struct is used on every DVB Front End driver, as it contains what's needed to register/use a frontend at the Kernel. Document it. Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/dvb-core/dvb_frontend.h b/drivers/media/dvb-core/dvb_frontend.h index 796cc5dca819..97661b2f247

[PATCH 38/39] [media] dvb-frontend.h: document struct dtv_frontend_properties

Most of the parameters here are already well defined at the userspace documentation. Yet, it is good to add some documentation, for the developers to be sure that they are the same as the ones at userspace API. Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/dvb-core/dvb_frontend

[PATCH 17/39] [media] DocBook: add dvb_frontend.h to documentation

There are already some comments at dvb_frontend.h that are ready for DocBook, although not properly formatted. Convert them, and add this file to the device-drivers DocBook. Signed-off-by: Mauro Carvalho Chehab diff --git a/Documentation/DocBook/device-drivers.tmpl b/Documentation/DocBook/devi

[PATCH 08/39] [media] Docbook: Fix comments at v4l2-mem2mem.h

Warning(.//include/media/v4l2-mem2mem.h:50): No description found for parameter 'lock' Warning(.//include/media/v4l2-mem2mem.h:50): No description found for parameter 'unlock' Warning(.//include/media/v4l2-mem2mem.h:167): No description found for parameter 'm2m_ctx' Warning(.//include/media/v4l2

[PATCH 22/39] [media] v4l2-ctrls.h: Document a few missing arguments

Warning(.//include/media/v4l2-ctrls.h:217): No description found for parameter 'p_new' Warning(.//include/media/v4l2-ctrls.h:217): No description found for parameter 'p_cur' Warning(.//include/media/v4l2-ctrls.h:217): Excess struct/union/enum/typedef member 'val64' description in 'v4l2_ctrl' War

[PATCH 30/39] [media] add documentation for struct dvb_tuner_info

Despite being used everywhere at DVB frontends, the struct dvb_tuner_info were never documented. Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/dvb-core/dvb_frontend.h b/drivers/media/dvb-core/dvb_frontend.h index ea68ba61f0de..e6df79fe8a71 100644 --- a/drivers/media/dvb-core/d

[PATCH 11/39] [media] v4l2-subdev: Add description for radio ioctl handlers

Warning(.//include/media/v4l2-subdev.h:224): No description found for parameter 's_radio' Warning(.//include/media/v4l2-subdev.h:224): No description found for parameter 's_frequency' Warning(.//include/media/v4l2-subdev.h:224): No description found for parameter 'enum_freq_bands' Warning(.//inc

[PATCH 09/39] [media] v4l2-subdev: convert documentation to the right format

struct v4l2_subdev_core_ops has some kernel-doc-nano documentation using a wrong format, with affects the number of stuff reported at the DocBook device-drivers.xml. Properly mark the such comment blocks using the right notation. Signed-off-by: Mauro Carvalho Chehab diff --git a/include/media/v

[PATCH 14/39] [media] v4l2_subdev: describe ioctl parms at the remaining structs

Fixes the following warnings: Warning(.//include/media/v4l2-subdev.h:445): No description found for parameter 'g_sliced_vbi_cap' Warning(.//include/media/v4l2-subdev.h:589): No description found for parameter 'enum_mbus_code' Warning(.//include/media/v4l2-subdev.h:589): No description found for

[PATCH 12/39] [media] v4l2-subdev: reorder the v4l2_subdev_video_ops comments

The comments for struct v4l2_subdev_video_ops are out of the order as the parameter would appear at struct. This is not a problem for DocBook, but humans find harder to mentally reorder ;) So, put them at the right order. That makes easier to check what's missing, and to put the comments in the ri

[PATCH 15/39] [media] v4l2-subdev: add remaining argument descriptions

Warning(.//include/media/v4l2-subdev.h:203): No description found for parameter 'ioctl' Warning(.//include/media/v4l2-subdev.h:203): No description found for parameter 'compat_ioctl32' Warning(.//include/media/v4l2-subdev.h:203): No description found for parameter 'subscribe_event' Warning(.//in

[PATCH 19/39] [media] DocBook: add dvb_ringbuffer.h to documentation

There are already some comments at dvb_ringbuffer.h that are ready for DocBook, although not properly formatted. Convert them, fix some issues and add this file to the device-drivers DocBook. While here, put multi-line comments on the right format. Signed-off-by: Mauro Carvalho Chehab diff --g

[PATCH 31/39] [media] dvb_frontend.h: get rid of dvbfe_modcod

This enum is not used anymore, as drivers use the modulation definitions from the public API. It is probably a left over from some DVB core cleanup. Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/dvb-core/dvb_frontend.h b/drivers/media/dvb-core/dvb_frontend.h index e6df79fe8a71

[PATCH 06/39] [media] Docbook: Fix s_rx_carrier_range parameter description

Warning(.//include/media/rc-core.h:178): No description found for parameter 's_rx_carrier_range' Warning(.//include/media/rc-core.h:178): Excess struct/union/enum/typedef member 's_rx_carrier' description in 'rc_dev' Signed-off-by: Mauro Carvalho Chehab diff --git a/include/media/rc-core.h b/i

Re: [PATCH] saa7164: convert to the control framework

On 08/22/2015 02:06 PM, Steven Toth wrote: > On Sat, Aug 22, 2015 at 6:47 AM, Hans Verkuil wrote: >> On 08/22/2015 09:24 AM, Ricardo Ribalda Delgado wrote: >>> Hello Hans >>> >>> With this patch I guess two of my previous patches are not needed. >>> Shall i resend the patchset or you just cherry p

Re: [PATCH v2 09/21] ARM: pxa: magician: Add OV9640 camera support

Petr Cvek writes: >> So Guennadi, is it possible to add a gpio through platform data to ov9640 >> driver, does it make sense, and would you accept to have the reset handled >> there >> ? And if yes, would you, Petr, accept to revamp your patch to have the reset >> and >> power handled in ov9640

Re: [PATCH] saa7164: convert to the control framework

On Sat, Aug 22, 2015 at 6:47 AM, Hans Verkuil wrote: > On 08/22/2015 09:24 AM, Ricardo Ribalda Delgado wrote: >> Hello Hans >> >> With this patch I guess two of my previous patches are not needed. >> Shall i resend the patchset or you just cherry pick the appropriate >> ones? > > Let's see how lon

halo

let us talk -- 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

Re: [PATCH] saa7164: convert to the control framework

On 08/22/2015 09:24 AM, Ricardo Ribalda Delgado wrote: > Hello Hans > > With this patch I guess two of my previous patches are not needed. > Shall i resend the patchset or you just cherry pick the appropriate > ones? Let's see how long it takes before I get an Ack (or not) from Steve. If that's q

[PATCH v3 07/10] media/usb/pvrusb2: Support for V4L2_CTRL_WHICH_DEF_VAL

This driver does not use the control infrastructure. Add support for the new field which on structure v4l2_ext_controls Acked-By: Mike Isely Signed-off-by: Ricardo Ribalda Delgado --- changes v3: By Mike Isely elevate the call to pvr2_hdw_get_ctrl_v4l() out of the if-statement drivers/media

Re: [PATCH] saa7164: convert to the control framework

Hello Hans With this patch I guess two of my previous patches are not needed. Shall i resend the patchset or you just cherry pick the appropriate ones? Thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More maj