Re: [PATCH v2 1/3] media: atomisp: convert default struct values to use compound-literals with designated initializers.

On Sat, Dec 02, 2017 at 08:41:48PM +, Jeremy Sowden wrote: > On 2017-12-02, at 10:35:06 +, Jeremy Sowden wrote: > > On 2017-12-02, at 13:20:09 +0300, Dan Carpenter wrote: > > > On Fri, Dec 01, 2017 at 05:19:37PM +, Jeremy Sowden wrote: > > > > -#define DEFAULT_PIPE_INFO \ > > > > -{ \

cron job: media_tree daily build: ERRORS

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 Dec 3 05:00:16 CET 2017 media-tree git hash:781b045baefdabf7e0bc9f33672ca830d3db9f27 media_build

Re: [GIT PULL] SAA716x DVB driver

On 02.12.2017 20:49, Mauro Carvalho Chehab wrote: > Em Sat, 2 Dec 2017 18:51:16 + > Jemma Denson escreveu: > >> Hi Mauro, >> >> On 27/11/17 11:24, Mauro Carvalho Chehab wrote: >>> Em Fri, 24 Nov 2017 17:28:37 +0100 >>> Tycho Lürsen escreveu: >>>

Re: [GIT PULL] SAA716x DVB driver

Mauro, On 28.11.2017 19:35, Mauro Carvalho Chehab wrote: > Em Mon, 25 Sep 2017 00:17:00 +0200 > Soeren Moch escreveu: > >>> What I'm saying is that, >>> if we're adding it on staging, we need to have a plan to reimplement >>> it to whatever API replaces the DVB video API, as this

[PATCH v4 3/3] media: atomisp: delete empty default struct values.

Removing zero-valued struct-members left a number of the default struct-values empty. These values have now been removed. Signed-off-by: Jeremy Sowden --- .../atomisp/pci/atomisp2/css2400/ia_css_pipe.h | 1 - .../atomisp/pci/atomisp2/css2400/ia_css_types.h| 1 -

[PATCH v4 1/3] media: atomisp: convert default struct values to use compound-literals with designated initializers.

The CSS API uses a lot of nested anonymous structs defined in object macros to assign default values to its data-structures. These have been changed to use compound-literals and designated initializers to make them more comprehensible and less fragile. The compound-literals can also be used in

[PATCH v4 2/3] media: atomisp: delete zero-valued struct members.

A lot of the members of the default struct values used by the CSS API were explicitly initialized to zero values. Designated initializers have allowed these members to be removed. Signed-off-by: Jeremy Sowden --- .../hive_isp_css_common/input_formatter_global.h | 16 ---

[PATCH v4 0/3] media: atomisp: clean up of data-structure initialization in the CSS API

v4. Removed the parens around the compound-literals in the macro definitions. They were superfluous and the errors reported by checkpatch.pl appear to have been false positives resulting from a bug. v3. Rebased on to git://linuxtv.org/media_tree.git. Patch-set applies cleanly to

Re: [PATCH v2 1/3] media: atomisp: convert default struct values to use compound-literals with designated initializers.

On 2017-12-02, at 20:41:48 +, Jeremy Sowden wrote: > On 2017-12-02, at 10:35:06 +, Jeremy Sowden wrote: > > On 2017-12-02, at 13:20:09 +0300, Dan Carpenter wrote: > > > On Fri, Dec 01, 2017 at 05:19:37PM +, Jeremy Sowden wrote: > > > > -#define DEFAULT_PIPE_INFO \ > > > > -{ \ > > > >

Re: [PATCH v2 1/3] media: atomisp: convert default struct values to use compound-literals with designated initializers.

On 2017-12-02, at 10:35:06 +, Jeremy Sowden wrote: > On 2017-12-02, at 13:20:09 +0300, Dan Carpenter wrote: > > On Fri, Dec 01, 2017 at 05:19:37PM +, Jeremy Sowden wrote: > > > -#define DEFAULT_PIPE_INFO \ > > > -{ \ > > > - {IA_CSS_BINARY_DEFAULT_FRAME_INFO}, /* output_info */ \ > > >

Re: [GIT PULL] SAA716x DVB driver

Em Sat, 2 Dec 2017 18:51:16 + Jemma Denson escreveu: > Hi Mauro, > > On 27/11/17 11:24, Mauro Carvalho Chehab wrote: > > Em Fri, 24 Nov 2017 17:28:37 +0100 > > Tycho Lürsen escreveu: > > > >> Hi Mauro, > >> > >> afaik the last communication about

Re: [GIT PULL] SAA716x DVB driver

Hi Mauro, On 27/11/17 11:24, Mauro Carvalho Chehab wrote: Em Fri, 24 Nov 2017 17:28:37 +0100 Tycho Lürsen escreveu: Hi Mauro, afaik the last communication about submission of this driver was about two months ago. This driver is important to me, because I own several

Re: [PATCH v2 2/3] media: atomisp: delete zero-valued struct members.

On 2017-12-01, at 17:41:50 +, Alan Cox wrote: > > --- > > a/drivers/staging/media/atomisp/pci/atomisp2/css2400/ia_css_pipe_public.h > > +++ > > b/drivers/staging/media/atomisp/pci/atomisp2/css2400/ia_css_pipe_public.h > > @@ -152,14 +152,6 @@ struct ia_css_pipe_config { > > }; > > > > >

Re: [PATCH v12 2/2] rcar-csi2: add Renesas R-Car MIPI CSI-2 receiver driver

Hej Sakari, Thanks for your feedback. On 2017-12-02 16:05:08 +0200, Sakari Ailus wrote: > Hejssan, > > On Sat, Dec 02, 2017 at 12:08:21PM +0100, Niklas Söderlund wrote: > ... > > > > +static int rcar_csi2_parse_dt(struct rcar_csi2 *priv) > > > > +{ > > > > + struct device_node *ep; > > >

Re: [PATCH v12 2/2] rcar-csi2: add Renesas R-Car MIPI CSI-2 receiver driver

Hejssan, On Sat, Dec 02, 2017 at 12:08:21PM +0100, Niklas Söderlund wrote: ... > > > +static int rcar_csi2_parse_dt(struct rcar_csi2 *priv) > > > +{ > > > + struct device_node *ep; > > > + struct v4l2_fwnode_endpoint v4l2_ep; > > > + int ret; > > > + > > > + ep =

[PATCH] [media] vcodec: mediatek: Add MODULE_LICENSE to mtk_vcodec_util.c

This fixes the following warning in an allmodconfig build: WARNING: modpost: missing MODULE_LICENSE() in drivers/media/platform/mtk-vcodec/mtk-vcodec-common.o This matches the license at the top of the file. Signed-off-by: Daniel Axtens ---

Re: [PATCH v12 2/2] rcar-csi2: add Renesas R-Car MIPI CSI-2 receiver driver

Hej Sakari, On 2017-12-01 15:01:36 +0200, Sakari Ailus wrote: > Hej Niklas, > > Tack för dina nya lappar! Tack för dina kommentarer. > > On Wed, Nov 29, 2017 at 08:32:35PM +0100, Niklas Söderlund wrote: > > A V4L2 driver for Renesas R-Car MIPI CSI-2 receiver. The driver > > supports the

Re: [PATCH v2 1/3] media: atomisp: convert default struct values to use compound-literals with designated initializers.

On Fri, Dec 01, 2017 at 05:19:37PM +, Jeremy Sowden wrote: > -#define DEFAULT_PIPE_INFO \ > -{ \ > - {IA_CSS_BINARY_DEFAULT_FRAME_INFO}, /* output_info */ \ > - {IA_CSS_BINARY_DEFAULT_FRAME_INFO}, /* vf_output_info */ \ > - IA_CSS_BINARY_DEFAULT_FRAME_INFO, /*

Re: [PATCH v2 1/3] media: atomisp: convert default struct values to use compound-literals with designated initializers.

On 2017-12-02, at 13:20:09 +0300, Dan Carpenter wrote: > On Fri, Dec 01, 2017 at 05:19:37PM +, Jeremy Sowden wrote: > > -#define DEFAULT_PIPE_INFO \ > > -{ \ > > - {IA_CSS_BINARY_DEFAULT_FRAME_INFO}, /* output_info */ \ > > - {IA_CSS_BINARY_DEFAULT_FRAME_INFO}, /* vf_output_info */