Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1530320

Re: [PATCH v17 08/15] clocksource/drivers/arm_arch_timer: move arch_timer_needs_of_probing into DT init call

Show all headers | View raw

Hi ,

On 25 November 2016 at 22:32, kbuild test robot <lkp@intel.com> wrote:
> Hi Fu,
>
> [auto build test ERROR on pm/linux-next]
> [also build test ERROR on v4.9-rc6]
> [cannot apply to tip/timers/core next-20161125]
> [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/fu-wei-linaro-org/acpi-clocksource-add-GTDT-driver-and-GTDT-support-in-arm_arch_timer/20161125-171111
> base:   https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git linux-next
> config: arm64-defconfig (attached as .config)
> compiler: aarch64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
> reproduce:
>         wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # save the attached .config to linux build tree
>         make.cross ARCH=arm64
>
> Note: the linux-review/fu-wei-linaro-org/acpi-clocksource-add-GTDT-driver-and-GTDT-support-in-arm_arch_timer/20161125-171111 HEAD 498f1f2503da21841b0e7679ddbdb86a40451bdb builds fine.
>       It only hurts bisectibility.
>
> All errors (new ones prefixed by >>):
>
>    drivers/clocksource/arm_arch_timer.c: In function 'arch_timer_acpi_init':

Sorry, again,

a "+ int ret;" should be move from [12/15] to here, I have fix the
problem in my repo, it would happen in next patchset

https://git.linaro.org/people/fu.wei/linux.git/log/?h=topic-gtdt-wakeup-timer_upstream_v18_devel

>>> drivers/clocksource/arm_arch_timer.c:1071:2: error: 'ret' undeclared (first use in this function)
>      ret = arch_timer_register();
>      ^~~
>    drivers/clocksource/arm_arch_timer.c:1071:2: note: each undeclared identifier is reported only once for each function it appears in
>
> vim +/ret +1071 drivers/clocksource/arm_arch_timer.c
>
>   1065                  return -EINVAL;
>   1066          }
>   1067
>   1068          /* Always-on capability */
>   1069          arch_timer_c3stop = !(gtdt->non_secure_el1_flags & ACPI_GTDT_ALWAYS_ON);
>   1070
>> 1071          ret = arch_timer_register();
>   1072          if (ret)
>   1073                  return ret;
>   1074
>
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation



-- 
Best regards,

Fu Wei
Software Engineer
Red Hat

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v17 00/15] acpi, clocksource: add GTDT driver and GTDT support in arm_arch_timer fu.wei@linaro.org - 2016-11-25 09:50 +0100
  [PATCH v17 13/15] acpi/arm64: Add memory-mapped timer support in GTDT driver fu.wei@linaro.org - 2016-11-25 09:50 +0100
  [PATCH v17 04/15] clocksource/drivers/arm_arch_timer: rename some enums and defines. fu.wei@linaro.org - 2016-11-25 09:50 +0100
  [PATCH v17 11/15] acpi/arm64: Add GTDT table parse driver fu.wei@linaro.org - 2016-11-25 09:50 +0100
  [PATCH v17 10/15] clocksource/drivers/arm_arch_timer: Refactor the timer init code to prepare for GTDT fu.wei@linaro.org - 2016-11-25 09:50 +0100
  [PATCH v17 14/15] clocksource/drivers/arm_arch_timer: Add GTDT support for memory-mapped timer fu.wei@linaro.org - 2016-11-25 09:50 +0100
  [PATCH v17 05/15] clocksource/drivers/arm_arch_timer: rework PPI determination fu.wei@linaro.org - 2016-11-25 09:50 +0100
  [PATCH v17 03/15] clocksource/drivers/arm_arch_timer: Improve printk relevant code fu.wei@linaro.org - 2016-11-25 09:50 +0100
  [PATCH v17 09/15] clocksource/drivers/arm_arch_timer: Introduce some new structs to prepare for GTDT fu.wei@linaro.org - 2016-11-25 09:50 +0100
  [PATCH v17 01/15] clocksource/drivers/arm_arch_timer: Move enums and defines to header file fu.wei@linaro.org - 2016-11-25 09:50 +0100
  [PATCH v17 06/15] clocksource/drivers/arm_arch_timer: Rework counter frequency detection. fu.wei@linaro.org - 2016-11-25 09:50 +0100
  [PATCH v17 07/15] clocksource/drivers/arm_arch_timer: Refactor arch_timer_needs_probing fu.wei@linaro.org - 2016-11-25 09:50 +0100
  [PATCH v17 02/15] clocksource/drivers/arm_arch_timer: Add a new enum for spi type fu.wei@linaro.org - 2016-11-25 09:50 +0100
  [PATCH v17 08/15] clocksource/drivers/arm_arch_timer: move arch_timer_needs_of_probing into DT init call fu.wei@linaro.org - 2016-11-25 10:00 +0100
    Re: [PATCH v17 08/15] clocksource/drivers/arm_arch_timer: move  arch_timer_needs_of_probing into DT init call Fu Wei <fu.wei@linaro.org> - 2016-11-25 16:10 +0100
      Re: [Linaro-acpi] [PATCH v17 08/15] clocksource/drivers/arm_arch_timer:  move arch_timer_needs_of_probing into DT init call Timur Tabi <timur@codeaurora.org> - 2016-12-07 18:30 +0100
        Re: [Linaro-acpi] [PATCH v17 08/15] clocksource/drivers/arm_arch_timer:  move arch_timer_needs_of_probing into DT init call Fu Wei <fu.wei@linaro.org> - 2016-12-08 04:20 +0100
          Re: [Linaro-acpi] [PATCH v17 08/15]  clocksource/drivers/arm_arch_timer: move arch_timer_needs_of_probing into DT  init call Mark Rutland <mark.rutland@arm.com> - 2016-12-08 12:10 +0100
            Re: [Linaro-acpi] [PATCH v17 08/15] clocksource/drivers/arm_arch_timer:  move arch_timer_needs_of_probing into DT init call Fu Wei <fu.wei@linaro.org> - 2016-12-08 12:30 +0100
  [PATCH v17 12/15] clocksource/drivers/arm_arch_timer: Simplify ACPI support code. fu.wei@linaro.org - 2016-11-25 10:00 +0100
  [PATCH v17 15/15] acpi/arm64: Add SBSA Generic Watchdog support in GTDT driver fu.wei@linaro.org - 2016-11-25 10:00 +0100

csiph-web