public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Lucas Stach <l.stach@pengutronix.de>
To: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Cc: linux-media@vger.kernel.org, kernel@pengutronix.de,
	patchwork-lst@pengutronix.de, kbuild test robot <lkp@intel.com>
Subject: Re: [PATCH 2/9] [media] tvp5150: add userspace subdev API
Date: Thu, 19 Nov 2015 09:59:15 +0100	[thread overview]
Message-ID: <1447923555.3144.67.camel@pengutronix.de> (raw)
In-Reply-To: <201511190158.nOgnlwG4%fengguang.wu@intel.com>

Am Donnerstag, den 19.11.2015, 01:06 +0800 schrieb kbuild test robot:
> Hi Philipp,
> 
> [auto build test ERROR on: v4.4-rc1]
> [also build test ERROR on: next-20151118]
> [cannot apply to: linuxtv-media/master]
> 
> url:    https://github.com/0day-ci/linux/commits/Lucas-Stach/tvp5150-convert-register-access-to-regmap/20151119-005732
> config: x86_64-randconfig-x018-11181928 (attached as .config)
> reproduce:
>         # save the attached .config to linux build tree
>         make ARCH=x86_64 
> 
> All error/warnings (new ones prefixed by >>):
> 
>    drivers/media/i2c/tvp5150.c: In function 'tvp5150_get_pad_format':
> >> drivers/media/i2c/tvp5150.c:1062:10: error: implicit declaration of function 'v4l2_subdev_get_try_format' [-Werror=implicit-function-declaration]
>       return v4l2_subdev_get_try_format(sd, cfg, pad);
>              ^
> >> drivers/media/i2c/tvp5150.c:1062:10: warning: return makes pointer from integer without a cast [-Wint-conversion]
>    drivers/media/i2c/tvp5150.c: In function 'tvp5150_get_pad_crop':
> >> drivers/media/i2c/tvp5150.c:1077:10: error: implicit declaration of function 'v4l2_subdev_get_try_crop' [-Werror=implicit-function-declaration]
>       return v4l2_subdev_get_try_crop(sd, cfg, pad);
>              ^
>    drivers/media/i2c/tvp5150.c:1077:10: warning: return makes pointer from integer without a cast [-Wint-conversion]
>    drivers/media/i2c/tvp5150.c: In function 'tvp5150_open':
> >> drivers/media/i2c/tvp5150.c:1180:27: warning: passing argument 2 of 'tvp5150_set_default' makes pointer from integer without a cast [-Wint-conversion]
>      tvp5150_set_default(std, v4l2_subdev_get_try_crop(fh, 0),
>                               ^
>    drivers/media/i2c/tvp5150.c:1152:13: note: expected 'struct v4l2_rect *' but argument is of type 'int'
>     static void tvp5150_set_default(v4l2_std_id std, struct v4l2_rect *crop,
>                 ^
>    drivers/media/i2c/tvp5150.c:1181:6: warning: passing argument 3 of 'tvp5150_set_default' makes pointer from integer without a cast [-Wint-conversion]
>          v4l2_subdev_get_try_format(fh, 0));
>          ^
>    drivers/media/i2c/tvp5150.c:1152:13: note: expected 'struct v4l2_mbus_framefmt *' but argument is of type 'int'
>     static void tvp5150_set_default(v4l2_std_id std, struct v4l2_rect *crop,
>                 ^
>    drivers/media/i2c/tvp5150.c: In function 'tvp5150_probe':
> >> drivers/media/i2c/tvp5150.c:1340:4: error: 'struct v4l2_subdev' has no member named 'entity'
>      sd->entity.flags |= MEDIA_ENT_T_V4L2_SUBDEV_DECODER;
>        ^
>    drivers/media/i2c/tvp5150.c:1342:29: error: 'struct v4l2_subdev' has no member named 'entity'
>      res = media_entity_init(&sd->entity, 1, &core->pad, 0);
>                                 ^
>    cc1: some warnings being treated as errors

Ok, this is just a missing depends on VIDEO_V4L2_SUBDEV_API. I'll wait
for other feedback before resending with that fixed.

Regards,
Lucas

-- 
Pengutronix e.K.             | Lucas Stach                 |
Industrial Linux Solutions   | http://www.pengutronix.de/  |


  reply	other threads:[~2015-11-19  8:59 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-18 16:55 [PATCH 1/9] [media] tvp5150: convert register access to regmap Lucas Stach
2015-11-18 16:55 ` [PATCH 2/9] [media] tvp5150: add userspace subdev API Lucas Stach
2015-11-18 17:06   ` kbuild test robot
2015-11-19  8:59     ` Lucas Stach [this message]
2015-11-19  9:27       ` Mauro Carvalho Chehab
2015-11-18 17:08   ` kbuild test robot
2015-11-18 16:55 ` [PATCH 3/9] [media] tvp5150: determine BT.656 or YUV 4:2:2 mode from device tree Lucas Stach
2015-11-18 16:55 ` [PATCH 4/9] [media] tvp5150: fix standard autodetection Lucas Stach
2015-11-18 16:55 ` [PATCH 5/9] [media] tvp5150: split reset/enable routine Lucas Stach
2015-11-18 16:55 ` [PATCH 6/9] [media] tvp5150: trigger autodetection on subdev open to reset cropping Lucas Stach
2015-11-18 16:55 ` [PATCH 7/9] [media] tvp5150: remove pin configuration from initialization tables Lucas Stach
2015-11-18 16:55 ` [PATCH 8/9] [media] tvp5150: Add sync lock interrupt handling Lucas Stach
2015-11-18 16:55 ` [PATCH 9/9] [media] tvp5150: disable output while signal not locked Lucas Stach

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1447923555.3144.67.camel@pengutronix.de \
    --to=l.stach@pengutronix.de \
    --cc=kernel@pengutronix.de \
    --cc=linux-media@vger.kernel.org \
    --cc=lkp@intel.com \
    --cc=mchehab@osg.samsung.com \
    --cc=patchwork-lst@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox