From: Julia Lawall <julia.lawall@lip6.fr>
To: Maxime Ripard <maxime.ripard@free-electrons.com>
Cc: linux-media@vger.kernel.org, devicetree@vger.kernel.org,
"Cyprian Wronka" <cwronka@cadence.com>,
"Neil Webb" <neilw@cadence.com>,
"Richard Sproul" <sproul@cadence.com>,
"Alan Douglas" <adouglas@cadence.com>,
"Steve Creaney" <screaney@cadence.com>,
"Thomas Petazzoni" <thomas.petazzoni@free-electrons.com>,
"Boris Brezillon" <boris.brezillon@free-electrons.com>,
"Niklas Söderlund" <niklas.soderlund@ragnatech.se>,
"Laurent Pinchart" <laurent.pinchart@ideasonboard.com>,
"Hans Verkuil" <hans.verkuil@cisco.com>,
"Sakari Ailus" <sakari.ailus@linux.intel.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Mark Rutland" <mark.rutland@arm.com>,
"Rob Herring" <robh+dt@kernel.org>,
kbuild-all@01.org
Subject: Re: [PATCH 2/2] v4l: cadence: Add Cadence MIPI-CSI2 RX driver (fwd)
Date: Thu, 6 Jul 2017 08:04:46 +0200 (CEST) [thread overview]
Message-ID: <alpine.DEB.2.20.1707060803210.2028@hadrien> (raw)
Please check on this (line 199).
julia
---------- Forwarded message ----------
Date: Thu, 6 Jul 2017 13:58:29 +0800
From: kbuild test robot <fengguang.wu@intel.com>
To: kbuild@01.org
Cc: Julia Lawall <julia.lawall@lip6.fr>
Subject: Re: [PATCH 2/2] v4l: cadence: Add Cadence MIPI-CSI2 RX driver
Hi Maxime,
[auto build test WARNING on linuxtv-media/master]
[also build test WARNING on v4.12]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Maxime-Ripard/dt-bindings-media-Add-Cadence-MIPI-CSI2RX-Device-Tree-bindings/20170705-205643
base: git://linuxtv.org/media_tree.git master
:::::: branch date: 17 hours ago
:::::: commit date: 17 hours ago
>> drivers/media/platform/cadence/cdns-csi2rx.c:199:5-23: WARNING: Unsigned expression compared with zero: csi2rx -> sensor_pad < 0
git remote add linux-review https://github.com/0day-ci/linux
git remote update linux-review
git checkout fb3d2879ba0623bcdc83c99ba70229ec1713feaf
vim +199 drivers/media/platform/cadence/cdns-csi2rx.c
fb3d2879 Maxime Ripard 2017-07-03 183 return media_create_pad_link(&csi2rx->sensor_subdev->entity,
fb3d2879 Maxime Ripard 2017-07-03 184 csi2rx->sensor_pad,
fb3d2879 Maxime Ripard 2017-07-03 185 &csi2rx->subdev.entity, 0,
fb3d2879 Maxime Ripard 2017-07-03 186 MEDIA_LNK_FL_ENABLED |
fb3d2879 Maxime Ripard 2017-07-03 187 MEDIA_LNK_FL_IMMUTABLE);
fb3d2879 Maxime Ripard 2017-07-03 188 }
fb3d2879 Maxime Ripard 2017-07-03 189
fb3d2879 Maxime Ripard 2017-07-03 190 static int csi2rx_async_bound(struct v4l2_async_notifier *notifier,
fb3d2879 Maxime Ripard 2017-07-03 191 struct v4l2_subdev *subdev,
fb3d2879 Maxime Ripard 2017-07-03 192 struct v4l2_async_subdev *asd)
fb3d2879 Maxime Ripard 2017-07-03 193 {
fb3d2879 Maxime Ripard 2017-07-03 194 struct csi2rx_priv *csi2rx = v4l2_notifier_to_csi2rx(notifier);
fb3d2879 Maxime Ripard 2017-07-03 195
fb3d2879 Maxime Ripard 2017-07-03 196 csi2rx->sensor_pad = media_entity_get_fwnode_pad(&subdev->entity,
fb3d2879 Maxime Ripard 2017-07-03 197 &csi2rx->sensor_node->fwnode,
fb3d2879 Maxime Ripard 2017-07-03 198 MEDIA_PAD_FL_SOURCE);
fb3d2879 Maxime Ripard 2017-07-03 @199 if (csi2rx->sensor_pad < 0) {
fb3d2879 Maxime Ripard 2017-07-03 200 dev_err(csi2rx->dev, "Couldn't find output pad for subdev %s\n",
fb3d2879 Maxime Ripard 2017-07-03 201 subdev->name);
fb3d2879 Maxime Ripard 2017-07-03 202 return csi2rx->sensor_pad;
fb3d2879 Maxime Ripard 2017-07-03 203 }
fb3d2879 Maxime Ripard 2017-07-03 204
fb3d2879 Maxime Ripard 2017-07-03 205 csi2rx->sensor_subdev = subdev;
fb3d2879 Maxime Ripard 2017-07-03 206
fb3d2879 Maxime Ripard 2017-07-03 207 dev_dbg(csi2rx->dev, "Bound %s pad: %d\n", subdev->name,
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
reply other threads:[~2017-07-06 6:04 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=alpine.DEB.2.20.1707060803210.2028@hadrien \
--to=julia.lawall@lip6.fr \
--cc=adouglas@cadence.com \
--cc=boris.brezillon@free-electrons.com \
--cc=cwronka@cadence.com \
--cc=devicetree@vger.kernel.org \
--cc=hans.verkuil@cisco.com \
--cc=kbuild-all@01.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=maxime.ripard@free-electrons.com \
--cc=mchehab@kernel.org \
--cc=neilw@cadence.com \
--cc=niklas.soderlund@ragnatech.se \
--cc=robh+dt@kernel.org \
--cc=sakari.ailus@linux.intel.com \
--cc=screaney@cadence.com \
--cc=sproul@cadence.com \
--cc=thomas.petazzoni@free-electrons.com \
/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