Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755183AbdFXTwj (ORCPT ); Sat, 24 Jun 2017 15:52:39 -0400 Received: from mail.kernel.org ([198.145.29.99]:33090 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751482AbdFXTwi (ORCPT ); Sat, 24 Jun 2017 15:52:38 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D6EAE22B69 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=jic23@kernel.org Date: Sat, 24 Jun 2017 20:52:32 +0100 From: Jonathan Cameron To: Eugen Hristev Cc: kbuild test robot , , , , , Subject: Re: [PATCH] iio: adc: at91-sama5d2_adc: add support for suspend/resume functionality Message-ID: <20170624205232.71e9ec38@kernel.org> In-Reply-To: References: <201706230509.6BVT6qtS%fengguang.wu@intel.com> X-Mailer: Claws Mail 3.15.0-dirty (GTK+ 2.24.31; x86_64-unknown-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4290 Lines: 96 On Fri, 23 Jun 2017 12:25:35 +0300 Eugen Hristev wrote: > On 23.06.2017 01:07, kbuild test robot wrote: > > Hi Eugen, > > > > [auto build test ERROR on iio/togreg] > > [also build test ERROR on v4.12-rc6 next-20170622] > > [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/Eugen-Hristev/iio-adc-at91-sama5d2_adc-add-support-for-suspend-resume-functionality/20170623-043330 > > base: https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg > > config: sparc64-allyesconfig (attached as .config) > > compiler: sparc64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705 > > reproduce: > > wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > chmod +x ~/bin/make.cross > > # save the attached .config to linux build tree > > make.cross ARCH=sparc64 > > > > All errors (new ones prefixed by >>): > > > > drivers/iio/adc/at91-sama5d2_adc.c: In function 'at91_adc_suspend': > >>> drivers/iio/adc/at91-sama5d2_adc.c:559:9: error: implicit declaration of function 'pinctrl_pm_select_sleep_state' [-Werror=implicit-function-declaration] > > return pinctrl_pm_select_sleep_state(dev); > > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > Will send a revised patch to fix this warning on SPARC architecture. > > > drivers/iio/adc/at91-sama5d2_adc.c: In function 'at91_adc_resume': > >>> drivers/iio/adc/at91-sama5d2_adc.c:569:8: error: implicit declaration of function 'pinctrl_pm_select_default_state' [-Werror=implicit-function-declaration] > > ret = pinctrl_pm_select_default_state(dev); > > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > >>> drivers/iio/adc/at91-sama5d2_adc.c:589:3: error: implicit declaration of function 'at91_adc_configure_trigger' [-Werror=implicit-function-declaration] > > at91_adc_configure_trigger(st->trig, true); > > ^~~~~~~~~~~~~~~~~~~~~~~~~~ > >>> drivers/iio/adc/at91-sama5d2_adc.c:589:32: error: 'struct at91_adc_state' has no member named 'trig'; did you mean 'reg'? > > at91_adc_configure_trigger(st->trig, true); > These two errors appears because this patch is on top of my other patch > for hardware trigger and buffer which is on the way upstream, and not > yet in linux-next. > Building over the tree with the included patch will work. Ah that explains the cryptic comment in the v2 of this patch. Good to have that cleared up! > > Regards, > Eugen > > ^~ > > cc1: some warnings being treated as errors > > > > vim +/pinctrl_pm_select_sleep_state +559 drivers/iio/adc/at91-sama5d2_adc.c > > > > 553 at91_adc_writel(st, AT91_SAMA5D2_CR, AT91_SAMA5D2_CR_SWRST); > > 554 > > 555 clk_disable_unprepare(st->per_clk); > > 556 regulator_disable(st->vref); > > 557 regulator_disable(st->reg); > > 558 > > > 559 return pinctrl_pm_select_sleep_state(dev); > > 560 } > > 561 > > 562 static int at91_adc_resume(struct device *dev) > > 563 { > > 564 struct iio_dev *indio_dev = > > 565 platform_get_drvdata(to_platform_device(dev)); > > 566 struct at91_adc_state *st = iio_priv(indio_dev); > > 567 int ret; > > 568 > > > 569 ret = pinctrl_pm_select_default_state(dev); > > 570 if (ret) > > 571 goto resume_failed; > > 572 > > 573 ret = regulator_enable(st->reg); > > 574 if (ret) > > 575 goto resume_failed; > > 576 > > 577 ret = regulator_enable(st->vref); > > 578 if (ret) > > 579 goto reg_disable_resume; > > 580 > > 581 ret = clk_prepare_enable(st->per_clk); > > 582 if (ret) > > 583 goto vref_disable_resume; > > 584 > > 585 at91_adc_hw_init(st); > > 586 > > 587 /* reconfiguring trigger hardware state */ > > 588 if (iio_buffer_enabled(indio_dev)) > > > 589 at91_adc_configure_trigger(st->trig, true); > > 590 > > 591 return 0; > > 592 > > > > --- > > 0-DAY kernel test infrastructure Open Source Technology Center > > https://lists.01.org/pipermail/kbuild-all Intel Corporation > >