From: kbuild test robot <lkp@intel.com>
To: Meghana Madhyastha <meghana.madhyastha@gmail.com>
Cc: outreachy-kernel@googlegroups.com,
dri-devel@lists.freedesktop.org, kbuild-all@01.org
Subject: Re: [PATCH v4 2/3] drm/tinydrm: Add devres versions of drm_of_find_backlight
Date: Sat, 30 Sep 2017 13:18:38 +0800 [thread overview]
Message-ID: <201709301353.dtkVCsoW%fengguang.wu@intel.com> (raw)
In-Reply-To: <d9f46a0f5e14a0ff4386b64037d6abeed60ebb0c.1506698677.git.meghana.madhyastha@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 3446 bytes --]
Hi Meghana,
[auto build test ERROR on drm/drm-next]
[also build test ERROR on v4.14-rc2 next-20170929]
[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/Meghana-Madhyastha/drm-tinydrm-drm_of_find_backlight-helper/20170930-122931
base: git://people.freedesktop.org/~airlied/linux.git drm-next
config: x86_64-randconfig-x017-201739 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64
Note: the linux-review/Meghana-Madhyastha/drm-tinydrm-drm_of_find_backlight-helper/20170930-122931 HEAD 4dc8fd20ad41e923832dc78cf4d8e7f98c0fedb9 builds fine.
It only hurts bisectibility.
All errors (new ones prefixed by >>):
drivers/gpu/drm/tinydrm/mi0283qt.c: In function 'mi0283qt_probe':
>> drivers/gpu/drm/tinydrm/mi0283qt.c:192:20: error: implicit declaration of function 'devm_drm_of_find_backlight' [-Werror=implicit-function-declaration]
mipi->backlight = devm_drm_of_find_backlight(dev);
^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/tinydrm/mi0283qt.c:192:18: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
mipi->backlight = devm_drm_of_find_backlight(dev);
^
cc1: some warnings being treated as errors
vim +/devm_drm_of_find_backlight +192 drivers/gpu/drm/tinydrm/mi0283qt.c
163
164 static int mi0283qt_probe(struct spi_device *spi)
165 {
166 struct device *dev = &spi->dev;
167 struct mipi_dbi *mipi;
168 struct gpio_desc *dc;
169 u32 rotation = 0;
170 int ret;
171
172 mipi = devm_kzalloc(dev, sizeof(*mipi), GFP_KERNEL);
173 if (!mipi)
174 return -ENOMEM;
175
176 mipi->reset = devm_gpiod_get_optional(dev, "reset", GPIOD_OUT_HIGH);
177 if (IS_ERR(mipi->reset)) {
178 dev_err(dev, "Failed to get gpio 'reset'\n");
179 return PTR_ERR(mipi->reset);
180 }
181
182 dc = devm_gpiod_get_optional(dev, "dc", GPIOD_OUT_LOW);
183 if (IS_ERR(dc)) {
184 dev_err(dev, "Failed to get gpio 'dc'\n");
185 return PTR_ERR(dc);
186 }
187
188 mipi->regulator = devm_regulator_get(dev, "power");
189 if (IS_ERR(mipi->regulator))
190 return PTR_ERR(mipi->regulator);
191
> 192 mipi->backlight = devm_drm_of_find_backlight(dev);
193 if (IS_ERR(mipi->backlight))
194 return PTR_ERR(mipi->backlight);
195
196 device_property_read_u32(dev, "rotation", &rotation);
197
198 ret = mipi_dbi_spi_init(spi, mipi, dc);
199 if (ret)
200 return ret;
201
202 ret = mipi_dbi_init(&spi->dev, mipi, &mi0283qt_pipe_funcs,
203 &mi0283qt_driver, &mi0283qt_mode, rotation);
204 if (ret)
205 return ret;
206
207 ret = mi0283qt_init(mipi);
208 if (ret)
209 return ret;
210
211 /* use devres to fini after drm unregister (drv->remove is before) */
212 ret = devm_add_action(dev, mi0283qt_fini, mipi);
213 if (ret) {
214 mi0283qt_fini(mipi);
215 return ret;
216 }
217
218 spi_set_drvdata(spi, mipi);
219
220 return devm_tinydrm_register(&mipi->tinydrm);
221 }
222
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 29255 bytes --]
[-- Attachment #3: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2017-09-30 5:18 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-29 15:28 [PATCH v4 0/3] drm/tinydrm: drm_of_find_backlight helper Meghana Madhyastha
2017-09-29 15:30 ` [PATCH v4 1/3] drm/tinydrm: Move tinydrm_of_find_backlight into drm_of.c Meghana Madhyastha
2017-09-29 15:31 ` [PATCH v4 2/3] drm/tinydrm: Add devres versions of drm_of_find_backlight Meghana Madhyastha
2017-09-30 5:18 ` kbuild test robot [this message]
2017-09-29 15:32 ` [PATCH v4 3/3] drm/tinydrm: Add the dummy versions of drm_of_find_backlight functions Meghana Madhyastha
2017-09-30 5:18 ` kbuild test robot
2017-10-02 7:56 ` Daniel Vetter
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=201709301353.dtkVCsoW%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=kbuild-all@01.org \
--cc=meghana.madhyastha@gmail.com \
--cc=outreachy-kernel@googlegroups.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