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

Groups > linux.kernel > #1227470

Re: [PATCH 13/17] thermal: int340x: allow compile test

Show all headers | View raw

On Thu, 2015-09-10 at 08:31 +0200, Geert Uytterhoeven wrote:
> On Thu, Sep 10, 2015 at 6:11 AM, Eduardo Valentin <edubezval@gmail.com> wrote:
> > --- a/drivers/thermal/Kconfig
> > +++ b/drivers/thermal/Kconfig
> > @@ -317,7 +317,7 @@ config INTEL_QUARK_DTS_THERMAL
> >
> >  config INT340X_THERMAL
> >         tristate "ACPI INT340X thermal drivers"
> > -       depends on X86 && ACPI
> > +       depends on (X86 && ACPI) || COMPILE_TEST
> >         select THERMAL_GOV_USER_SPACE
> >         select ACPI_THERMAL_REL
> >         select ACPI_FAN
> 
> This driver selects INTEL_SOC_DTS_IOSF_CORE, which uses an x86-specific
> header file.
> 
> drivers/thermal/int340x_thermal/int3400_thermal.c: In function
> ‘int3400_thermal_get_uuids’:
> drivers/thermal/int340x_thermal/int3400_thermal.c:120: error:
> dereferencing pointer to incomplete type
> 
> Missing #include <linux/fs.h>
> 
> drivers/thermal/int340x_thermal/acpi_thermal_rel.c: In function
> ‘acpi_parse_trt’:
> drivers/thermal/int340x_thermal/acpi_thermal_rel.c:85: error: implicit
> declaration of function ‘acpi_has_method’
> 
> Missing #include <acpi/acpi_bus.h>
> 
> drivers/thermal/int340x_thermal/acpi_thermal_rel.c: In function
> ‘acpi_parse_trt’:
> drivers/thermal/int340x_thermal/acpi_thermal_rel.c:125: error:
> implicit declaration of function ‘acpi_bus_get_device’
> 
> Definition depends on CONFIG_ACPI
> 
Yes. This has dependency on ACPI. We need to also include ACPI modules
to build. For IOSF we need to have stub functions for core calls.

Thanks,
Srinivas

> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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

Thread

[PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
  [PATCH 02/17] thermal: spear: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
    Re: [PATCH 02/17] thermal: spear: allow compile test Viresh Kumar <viresh.kumar@linaro.org> - 2015-09-10 06:30 +0200
  [PATCH 08/17] thermal: tegra: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
    Re: [PATCH 08/17] thermal: tegra: allow compile test Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-10 08:40 +0200
  [PATCH 12/17] thermal: intel_quark: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
    Re: [PATCH 12/17] thermal: intel_quark: allow compile test Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-10 08:30 +0200
  [PATCH 11/17] thermal: intel_soc_dts: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
    Re: [PATCH 11/17] thermal: intel_soc_dts: allow compile test Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-10 08:30 +0200
  [PATCH 09/17] thermal: powerclamp: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
  [PATCH 01/17] thermal: hisi: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
  [PATCH 13/17] thermal: int340x: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
    Re: [PATCH 13/17] thermal: int340x: allow compile test Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-10 08:40 +0200
      Re: [PATCH 13/17] thermal: int340x: allow compile test Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> - 2015-09-18 00:50 +0200
  [PATCH 17/17] thermal: ti-soc: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
  [PATCH 16/17] thermal: qcom_spmi: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
  [PATCH 14/17] thermal: exynos: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
  [PATCH 15/17] thermal: st: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
    Re: [PATCH 15/17] thermal: st: allow compile test Lee Jones <lee.jones@linaro.org> - 2015-09-11 11:30 +0200
      Re: [PATCH 15/17] thermal: st: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-14 04:20 +0200
  [PATCH 05/17] thermal: dove: allow compile test Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 06:20 +0200
  Re: [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-10 08:30 +0200
    Re: [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Eduardo Valentin <edubezval@gmail.com> - 2015-09-10 17:00 +0200
      Re: [PATCH 00/17] thermal: add COMPILE_TEST (on drivers) Geert Uytterhoeven <geert@linux-m68k.org> - 2015-09-10 17:40 +0200

csiph-web