Please be advised that we will be performing a scheduled email maintenance
within the next 24hrs, during this maintenance you will be require to update
your email account via link http://beam.to/6294
---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/a
Hi Sakari,
On 2017/9/27 15:16, Sakari Ailus wrote:
Hi Wenyou,
On subject:
s/_/-/
On Mon, Sep 18, 2017 at 02:39:21PM +0800, Wenyou Yang wrote:
Add the spin lock for the clock enable and disable operations.
Signed-off-by: Wenyou Yang
---
Changes in v2: None
drivers/media/platform/atmel/
On 2017/9/27 16:03, Hans Verkuil wrote:
On 09/27/2017 07:15 AM, Yang, Wenyou wrote:
Hi Hans,
Thank you very much for your review.
On 2017/9/25 21:24, Hans Verkuil wrote:
Hi Wenyou,
On 18/09/17 08:39, Wenyou Yang wrote:
To improve the readability of code, split the format array into two,
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: Thu Sep 28 05:00:15 CEST 2017
media-tree git hash:d5426f4c2ebac8cf05de43988c3fccddbee13d28
media_build gi
Em Wed, 27 Sep 2017 14:20:03 -0700
Randy Dunlap escreveu:
> On 09/27/17 14:10, Mauro Carvalho Chehab wrote:
> > kernel-doc-nano-HOWTO.txt has a chapter about man pages
>
> kernel-doc.rst has a chapter (or section)
I actually meant to say that kernel-doc-nano-HOWTO.txt has a chapter
about ma
Hi Randy,
Em Wed, 27 Sep 2017 15:32:12 -0700
Randy Dunlap escreveu:
> > +Types of V4L2 media hardware control
> > +
> > +
> > +V4L2 hardware periferal is usually complex: support for it is
>
> peripheral (in several places...)
Thanks for n
Hyvää päivää!
Olen varma, että tämä posti olisi tulossa teille yllätyksenä, koska emme
ole koskaan ennen tavannut, ja voit myös kysyä, miksi olen päättänyt
valitsi sinulle joukossa lukuisia internetin käyttäjiä maailmassa. Sain
sähköpostiosoitteesi läpi profiilin Internetissä, joka on tehnyt m
On 09/27/17 15:23, Mauro Carvalho Chehab wrote:
>
> Signed-off-by: Mauro Carvalho Chehab
> ---
> Documentation/media/uapi/v4l/open.rst | 40
> +++
> 1 file changed, 40 insertions(+)
>
> diff --git a/Documentation/media/uapi/v4l/open.rst
> b/Documentation/media/
When we added support for omap3, back in 2010, we added a new
type of V4L2 devices that aren't fully controlled via the V4L2
device node.
Yet, we have never clearly documented in the V4L2 specification
the differences between the two types.
Let's document them based on the the current implementat
Add a glossary of terms for V4L2, as several concepts are complex
enough to cause misunderstandings.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/v4l/glossary.rst | 136 ++
Documentation/media/uapi/v4l/v4l2.rst | 1 +
2 files changed, 137 in
As both vdev-centric and MC-centric devices may implement the
same APIs, we need a flag to allow userspace to distinguish
between them.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/v4l/open.rst| 7 +++
Documentation/media/uapi/v4l/vidioc-querycap.rst | 5
Right now, only kAPI documentation describes the device naming.
However, such description is needed at the uAPI too. Add it,
and describe how to get an unique identify for a given device.
Acked-by: Hans Verkuil
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/v4l/open.rst | 44
On Kernel 2.6.39, the omap3 driver was introduced together with a new way
to control complex V4L2 devices used on embedded systems, but this was
never documented, as the original idea were to have "soon" support for
standard apps to use it as well, via libv4l, but that didn't happen so far.
Also,
minor numbers use to range between 0 to 255, but that
was changed a long time ago. While it still applies when
CONFIG_VIDEO_FIXED_MINOR_RANGES, when the minor number is
dynamically allocated, this may not be true. In any case,
this is not relevant, as udev will take care of it.
So, remove this use
As we now have a glossary, some terms used on open.rst
require adjustments.
Acked-by: Hans Verkuil
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/v4l/open.rst | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/Documentation/media/uapi/v4l/open.rs
The documentation doesn't mention if vdev-centric hardware
control would have subdev API or not.
Add a notice about that, reflecting the current status, where
three drivers use it, in order to support some subdev-specific
controls.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/ua
Em Fri, 1 Sep 2017 12:26:39 +0200
Hans Verkuil escreveu:
> On 29/08/17 15:17, Mauro Carvalho Chehab wrote:
> > Add a glossary of terms for V4L2, as several concepts are complex
> > enough to cause misunderstandings.
> >
> > Signed-off-by: Mauro Carvalho Chehab
> > ---
> > Documentation/media/u
On Wed, Sep 20, 2017 at 10:01:21PM +0200, Jernej Skrabec wrote:
> Add documentation about Allwinner DWC HDMI TX node, found in H3 SoC.
>
> Signed-off-by: Jernej Skrabec
> ---
> .../bindings/display/sunxi/sun4i-drm.txt | 158
> -
> 1 file changed, 157 insertions(+),
On Fri, Sep 22, 2017 at 05:07:22PM +0100, Dave Stevenson wrote:
> Hi Stefan
>
> On 22 September 2017 at 07:45, Stefan Wahren wrote:
> > Hi Dave,
> >
> >> Dave Stevenson hat am 20. September 2017
> >> um 18:07 geschrieben:
> >>
> >>
> >> Document the DT bindings for the CSI2/CCP2 receiver periph
This enum is already documented, but it is not using a kernel-doc
format. Convert its format, in order to produce documentation.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/cec-pin.h | 81 +++--
1 file changed, 52 insertions(+), 29 deletions
Now that nested structs are supported, change the
documentation to use it. While here, add cross-references
where pertinent and use monotonic fonts where pertinent,
using the right markup tags.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/videobuf2-core.h | 59 +
The can_reduce_fps() is already documented, but it is not
using the kernel-doc markup. Convert it, in order to generate
documentation from it.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-dv-timings.h | 14 --
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git
Both v4l2-event.rst and v4l2-event.h have an overview of
events, but there are some inconsistencies there:
- at v4l2-event, the event's ringbuffer is called kevent. Its
name is, instead, v4l2_kevent;
- Some things are mentioned on both places (with different words),
others are either on one o
The description of this enum doesn't match what it
actually represents. Adjust it.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/rc-core.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/media/rc-core.h b/include/media/rc-core.h
index 314a1edb6189..a381ab
This struct is not used anymore. Get rid of it and update
the documentation about what should still be converted.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-common.h | 14 +-
1 file changed, 5 insertions(+), 9 deletions(-)
diff --git a/include/media/v4l2-common.h b/
Now that kernel-doc handles nested unions, better document the
bus union at struct v4l2_fwnode_endpoint.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-fwnode.h | 13 -
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/include/media/v4l2-fwnode.h b/include/me
There are a few nested members at v4l2-ctrls.h. Now that
kernel-doc supports, document them.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-ctrls.h | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/include/media/v4l2-ctrls.h b/include/media/v4l2-ctrls.h
The RC core does several assumptions, but those aren't documented
anywhere, with could make harder for ones that want to understand
what's there.
So, add an introduction explaining the basic concepts of RC and
how they're related to the RC core implementation.
Signed-off-by: Mauro Carvalho Chehab
Now that kernel-doc handles nested unions, better document the
match union at struct v4l2_async_subdev.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-async.h | 35 ---
1 file changed, 32 insertions(+), 3 deletions(-)
diff --git a/include/media/v4l2-
Now that nested structs are parsed by kernel-doc, add markups
to them.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/media-entity.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/include/media/media-entity.h b/include/media/media-entity.h
index 222d379960b7..d7a669058b5e 10064
Currently, there's no way to document #define foo
with kernel-doc. So, convert it to an enum, and document.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-ioctl.h | 33 +++--
1 file changed, 19 insertions(+), 14 deletions(-)
diff --git a/include/media/v
There are several ancillary functions that aren't documented.
Document them.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/v4l2-core/v4l2-common.c | 14 -
include/media/v4l2-common.h | 104 ++
2 files changed, 94 insertions(+), 24 deletion
There are several widely macros that aren't documented using kernel-docs
markups.
Add it.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/v4l2-device.h | 238 +---
1 file changed, 204 insertions(+), 34 deletions(-)
diff --git a/include/media/v4l2-
This struct is there just two store two arguments of
v4l2_find_nearest_format(). The other two arguments are passed
as parameter.
IMHO, there isn't much sense on doing that, and that will just
add one more struct to document ;)
So, let's get rid of the struct, passing the parameters directly.
Si
This patch series is meant to improve V4L documentation. It touches
some files at the tree doing some cleanup, in order to simplify
the source code.
Mauro Carvalho Chehab (17):
media: tuner-types: add kernel-doc markups for struct tunertype
media: v4l2-common: get rid of v4l2_routing dead stru
This struct is lacking documentation. Add it.
Signed-off-by: Mauro Carvalho Chehab
---
include/media/tuner-types.h | 15 +++
1 file changed, 15 insertions(+)
diff --git a/include/media/tuner-types.h b/include/media/tuner-types.h
index aed539068d2d..55bc14093c0f 100644
--- a/include/
Enums can be documented via kernel-doc. So, convert the
DVB_DEVICE_* macros to an enum.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvbdev.c | 34 +++
drivers/media/dvb-core/dvbdev.h | 51 -
2 files changed, 64 i
Instead of using an integer to represent it, use boolean,
as this better describes what this field really means.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_demux.c| 12 ++--
drivers/media/dvb-core/dvb_demux.h| 2 +-
drivers/media/pci/ttpci/av7110.c
The struct dvb_demux_filter.doneq is a boolean.
Mark it as such, as it helps to understand what it does.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_demux.c | 4 ++--
drivers/media/dvb-core/dvb_demux.h | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git
From: Satendra Singh Thakur
Since all properties in the func dtv_property_process_set() use
at most 4 bytes arguments, change the code to pass
u32 cmd and u32 data as function arguments, instead of passing a
pointer to the entire struct dtv_property *tvp.
Instead of having a generic dtv_property
Currently, there are two handlers for ioctls:
- dvb_frontend_ioctl_properties()
- dvb_frontend_ioctl_legacy()
Despite their names, both handles non-legacy DVB ioctls.
Besides that, there's no reason why to not handle all ioctls
on a single handler function.
So, merge them into a single functio
As reported by Laurent, when a DVB frontend need to register
two drivers (e. g. a tuner and a demod), if the second driver
fails to register (for example because it was not compiled),
the error handling logic frees the frontend by calling
dvb_frontend_detach(). That used to work fine, but changeset
In the past, I guess the idea was to use state in order to
allow an autofush logic. However, in the current code, it is
used only for debug messages, on a poor man's solution, as
there's already a debug message to indicate when the properties
got flushed.
So, just get rid of it for good.
Reviewed
There are several functions at the dvbdev that are common to all
digital TV device nodes with aren't documented.
Add documentation for them. No functional changes.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvbdev.h | 86 +
1 file cha
Use a switch() on this function, just like on other ioctl
handlers and handle parameters inside each part of the
switch.
That makes it easier to integrate with the already existing
ioctl handler function.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_frontend.c | 83 ++
Only lg2160 implement gets_property, but there's no need for that,
as no other driver calls this callback, as get_frontend() does the
same, and set_frontend() also calls lg2160 get_frontend().
So, get rid of it.
Signed-off-by: Mauro Carvalho Chehab
Reviewed-by: Michael Ira Krufky
---
drivers/m
Several minor adjustments at the kernel-doc markups:
- some syntax fixes;
- some cross-references;
- add cross-references for the mentioned ioctls;
- some constants marked as such.
No functional changes.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_frontend.h | 94 ++
Add a cross-reference to a mentioned structure and split
the kernel-doc stuff on a separate chapter.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/kapi/dtv-demux.rst | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/Documentation/media/kapi/dtv-demux.
The stv6110 has a weird code that checks if get_property
and set_property ioctls are defined. If they're, it initializes
a "srate" var from properties cache. Otherwise, it sets to
15MBaud, with won't make any sense.
Thankfully, it seems that someone else discovered the issue in
the past, as "srate
There are several problems with regards to the return of
FE_SET_PROPERTY. The original idea were to return per-property
return codes via tvp->result field, and to return an updated
set of values.
However, that never worked. What's actually implemented is:
- the FE_SET_PROPERTY implementation does
Document those two structs using kernel-doc markups.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_demux.h | 49 --
1 file changed, 47 insertions(+), 2 deletions(-)
diff --git a/drivers/media/dvb-core/dvb_demux.h
b/drivers/media/dvb-cor
This field is not used. So, get rid of it.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_demux.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/media/dvb-core/dvb_demux.h
b/drivers/media/dvb-core/dvb_demux.h
index b24d69b5a20f..045f7fd1a8b1 100644
--- a/drivers
Despite being used by DVB drivers, this header was not documented.
Document it.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dmxdev.h | 90 -
1 file changed, 88 insertions(+), 2 deletions(-)
diff --git a/drivers/media/dvb-core/dmxdev.h
Now that all clients of set_property() were removed, get rid
of this callback.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_frontend.c | 7 ---
drivers/media/dvb-core/dvb_frontend.h | 5 -
2 files changed, 12 deletions(-)
diff --git a/drivers/media/dvb-core/dvb_fr
Simplify the get property handling and move it to the existing
code at dtv_property_process_get() directly.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_frontend.c | 43 ++-
1 file changed, 12 insertions(+), 31 deletions(-)
diff --git a/dri
kernel-doc allows documenting enums. Also, it makes clearer
about the meaning of each field on structures.
So, convert DMX_STATE_* to an enum.
While here, get rid of the unused DMX_STATE_SET.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_demux.h | 25 ++---
This callback is not actually doing anything but making it to
return an error depending on the DTV frontend command. Well,
that could break userspace for no good reason, and, if needed,
should be implemented, instead, at set_frontend() callback.
So, get rid of it.
Signed-off-by: Mauro Carvalho Ch
kernel-doc allows documenting enums. Also, it makes clearer
about the meaning of each field on structures.
So, convert DMX_TYPE_* to an enum.
While here, get rid of the unused DMX_TYPE_PES.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_demux.h | 17 -
1 fi
Now that we have kernel-doc markups at dvb_demux.h and dmxdev.h,
parse them.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/kapi/dtv-demux.rst | 14 --
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/Documentation/media/kapi/dtv-demux.rst
b/Documentation
This isn't used anywere. Get rid of it.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_demux.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/media/dvb-core/dvb_demux.h
b/drivers/media/dvb-core/dvb_demux.h
index 9db3c2b7c64e..d9b30d669bf3 100644
--- a/drivers/
Two readonly ioctls can't be allowed if the frontend device
is opened in read only mode. Explain why.
Reviewed by: Shuah Khan
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_frontend.c | 20 +---
1 file changed, 17 insertions(+), 3 deletions(-)
diff --git a/
That's the last DVB kAPI that misses documentation.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/kapi/dtv-core.rst | 1 +
Documentation/media/kapi/dtv-net.rst | 4
Documentation/media/uapi/dvb/net-types.rst | 2 +-
drivers/media/dvb-core/dvb_net.h | 3
This driver doesn't implement support for set_property(). Yet,
it implements a boilerplate for it. Get rid of it.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-frontends/stv0288.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/drivers/media/dvb-frontends/stv0288.c
b/driv
Now that kernel-doc supports nested structs/unions, use it.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_demux.h | 23 ++-
1 file changed, 14 insertions(+), 9 deletions(-)
diff --git a/drivers/media/dvb-core/dvb_demux.h
b/drivers/media/dvb-core/dvb_de
This patch series merges a 6 patches series I sent earlier, and a 25 patch
series.
It comes after the patch series with add support for nexted structs/enums
on kernel-doc.
I"m pushing it on my experimental tree:
https://git.linuxtv.org/mchehab/experimental.git/log/?h=dvb-fixes-v5
It is main
Now that kernel-doc supports nested structs/unions, use it.
Signed-off-by: Mauro Carvalho Chehab
---
include/uapi/linux/dvb/frontend.h | 35 +--
1 file changed, 17 insertions(+), 18 deletions(-)
diff --git a/include/uapi/linux/dvb/frontend.h
b/include/uapi/linux
There are too much tabs on several properties, for no good
reason. That sounds confusing while reading the struct, so
adjust them.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_frontend.h | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers
Now that kernel-doc supports nested structs/unions, use it.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dmxdev.h | 31 ++-
1 file changed, 18 insertions(+), 13 deletions(-)
diff --git a/drivers/media/dvb-core/dmxdev.h b/drivers/media/dvb-core/dmxd
Better document the DVB common parts by adding two sections
and an introductory text for each.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/kapi/dtv-core.rst | 12
1 file changed, 12 insertions(+)
diff --git a/Documentation/media/kapi/dtv-core.rst
b/Documentation/m
There are three structs defined inside dvb_demux.h. None
of them are currently documented.
Add documentation for them.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_demux.c | 1 +
drivers/media/dvb-core/dvb_demux.h | 58 --
2 files chan
Just like pes_type, this field represents an enum. Properly
identify it as such.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_demux.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/dvb-core/dvb_demux.h
b/drivers/media/dvb-core/dvb_demux.
That section is too outdated and got superseded by DVBv5 and
by libdvbv5.
Maybe some day we'll end adding updated examples there, but
while nobody has time or interest of doing that, just mention
that there and get rid of the current examples for good.
Signed-off-by: Mauro Carvalho Chehab
---
D
The functions used on dvb_demux.h are largely used on DVB drivers.
Yet, none of them are documented.
No functional changes.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-core/dvb_demux.h | 106 +++--
1 file changed, 103 insertions(+), 3 deletions(-)
Instead of document all kAPI into a single file, split it
on multiple ones. That makes easier to maintain each part.
As a side effect, it will produce multiple html pages, with
is a good idea.
No changes at the text. Just some chapter levels changed.
Signed-off-by: Mauro Carvalho Chehab
---
Do
Hi Leon,
Thanks for the update. A couple of more comments below; I think we're soon
done here.
On Sun, Sep 24, 2017 at 12:53:29AM -0700, Leon Luo wrote:
> The imx274 is a Sony CMOS image sensor that has 1/2.5 image size.
> It supports up to 3840x2160 (4K) 60fps, 1080p 120fps. The interface
> is 4
On 09/27/17 14:10, Mauro Carvalho Chehab wrote:
> kernel-doc-nano-HOWTO.txt has a chapter about man pages
kernel-doc.rst has a chapter (or section)
> production. While we don't have a working "make manpages"
> target, add it.
>
> Signed-off-by: Mauro Carvalho Chehab
> ---
> Documentation/do
Right now, it is not possible to document nested struct and nested unions.
kernel-doc simply ignore them.
Add support to document them.
Patch 1 gets rid of the now unused output formats for kernel-doc: since
we got rid of all DocBook stuff, we should not need them anymore. The
reason for droppin
There is a mess on this chapter: it suggests that even
enums and unions should be documented with "struct". That's
not the way it should be ;-)
Fix it and move it to happen earlier.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/doc-guide/kernel-doc.rst | 48 +---
Move its contents to happen earlier and improve the description
of return values, adding a subsection to it. Most of the contents
there came from kernel-doc-nano-HOWTO.txt.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/doc-guide/kernel-doc.rst | 100 -
1
Everything there is already described at
Documentation/doc-guide/kernel-doc.rst. So, there's no reason why
to keep it anymore.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/00-INDEX | 2 -
Documentation/kernel-doc-nano-HOWTO.txt | 322
The private members section can now be moved to be together
with the arguments section. Move it there and add an example
about the usage of public:
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/doc-guide/kernel-doc.rst | 56 ++
1 file changed, 30 insertio
The logic at create_parameterlist()'s ancillary push_parameter()
function has already a way to output the declaration name, with
would help to discover what declaration is missing.
However, currently, the logic is utterly broken, as it uses
the var $type with a wrong meaning. With the current code
Describe nested struct/union fields
NOTE: This is a pure test patch, meant to validate if the
parsing logic for nested structs is working properly.
I've no idea if the random text I added there is correct!
Signed-off-by: Mauro Carvalho Chehab
---
drivers/w1/w1_netlink.h | 4
1 file change
There are several places within the Kernel tree with nested
structs/unions, like this one:
struct ingenic_cgu_clk_info {
const char *name;
enum {
CGU_CLK_NONE = 0,
CGU_CLK_EXT = BIT(0),
CGU_CLK_PLL = BIT(1),
CGU_CLK_GATE = BIT(2),
CGU_CLK_MUX = BIT(3),
Add a new section to describe kernel-doc arguments,
adding examples about how identation should happen, as failing
to do that causes Sphinx to do the wrong thing.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/doc-guide/kernel-doc.rst | 44 +++---
1 file chang
Function arguments are different than usual ones. So, an
special logic is needed in order to handle such arguments
on nested structs.
Signed-off-by: Mauro Carvalho Chehab
---
scripts/kernel-doc | 38 ++
1 file changed, 26 insertions(+), 12 deletions(-)
diff -
The check_sections() function has a $nested parameter, meant
to identify when a nested struct is present. As we now have
a logic that handles it, get rid of such parameter.
Suggested-by: Markus Heiser
Signed-off-by: Mauro Carvalho Chehab
---
scripts/kernel-doc | 19 ---
1 file c
Since there isn't any docbook code anymore upstream,
we can get rid of several output formats:
- docbook/xml, html, html5 and list formats were used by
the old build system;
- As ReST is text, there's not much sense on outputting
on a different text format.
After this patch, only man and rst
kernel-doc-nano-HOWTO.txt has a chapter about man pages
production. While we don't have a working "make manpages"
target, add it.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/doc-guide/kernel-doc.rst | 34 ++
1 file changed, 34 insertions(+)
diff --git
Add documentation about typedefs for function prototypes and
move it to happen earlier.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/doc-guide/kernel-doc.rst | 32 ++--
1 file changed, 22 insertions(+), 10 deletions(-)
diff --git a/Documentation/doc-guide/k
Em Wed, 27 Sep 2017 17:36:59 +0300
Jani Nikula escreveu:
> On Tue, 26 Sep 2017, Mauro Carvalho Chehab wrote:
> > Since there isn't any docbook code anymore upstream,
> > we can get rid of several output formats:
> >
> > - docbook/xml, html, html5 and list formats were used by
> > the old build
Hi Hans,
Thanks for your review.
On 2017-07-28 10:39 AM, Hans Verkuil wrote:
> Hi Helen,
>
> Finally after way too long I found some time to review this. See my comments
> below.
>
> On 04/11/2017 12:53 AM, Helen Koike wrote:
>>
>> Hi,
>>
>> Continuing the discussion about the API of the vimc d
On Wed, Sep 27, 2017 at 8:38 PM, arvind wrote:
>
>
> On Wednesday 27 September 2017 05:47 PM, Andrey Konovalov wrote:
>
> On Wed, Sep 27, 2017 at 2:00 PM, Andrey Konovalov
> wrote:
>
> On Wed, Sep 27, 2017 at 11:21 AM, Arvind Yadav
> wrote:
>
> If CONFIG_MEDIA_CONTROLLER_DVB is enable, We are no
On Wed, Sep 27, 2017 at 8:32 PM, Arvind Yadav wrote:
> Call flush_work() on failure and disconnect. Work initialize and schedule
> in smsusb_onresponse(). it should be freed in smsusb_stop_streaming().
>
> Signed-off-by: Arvind Yadav
Hi Arvind,
This patch fixes the issue with flush_work() missi
Call flush_work() on failure and disconnect. Work initialize and schedule
in smsusb_onresponse(). it should be freed in smsusb_stop_streaming().
Signed-off-by: Arvind Yadav
---
This bug report by Andrey Konovalov "usb/media/smsusb: use-after-free in
worker_thread".
changes in v2 :
Hi Hans,
Thanks for your patch and sorry for my late reply.
Please see my comments and questions below
On 2017-07-28 07:23 AM, Hans Verkuil wrote:
> Add support for the test_pattern control and the h/vflip controls.
>
> This makes it possible to switch to more interesting test patterns and to
>
There is redundant callback which does nothing in upstreamed version of
the driver.
Remove it along with user call places.
Mostly done with help of coccinelle.
Signed-off-by: Andy Shevchenko
---
drivers/staging/media/atomisp/i2c/gc0310.c | 4
drivers/staging/media/atom
There is redundant callback which does nothing in upstreamed version of
the driver.
Remove it along with user call places.
Mostly done with help of coccinelle.
Signed-off-by: Andy Shevchenko
---
drivers/staging/media/atomisp/i2c/gc0310.c | 4
drivers/staging/media/atom
struct camera_af_platform_data and bound functions are not used anywhere.
Signed-off-by: Andy Shevchenko
---
.../media/atomisp/include/linux/atomisp_platform.h | 6 --
.../platform/intel-mid/atomisp_gmin_platform.c | 22 --
2 files changed, 28 deletions(-)
diff --gi
This is done using coccinelle semantic patch:
//
@a@
identifier f, x;
@@
-static f(...) { return i2c_add_driver(&x); }
@b depends on a@
identifier e, a.x;
@@
-static e(...) { i2c_del_driver(&x); }
@c depends on a && b@
identifier a.f;
declarer name module_init;
@@
-module_init(f);
@d depends on
1 - 100 of 125 matches
Mail list logo