<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux-toradex.git/include/acpi/acpi_io.h, branch v3.14.41</title>
<subtitle>Linux kernel for Apalis and Colibri modules</subtitle>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/'/>
<entry>
<title>ACPI / i915: Fix incorrect &lt;acpi/acpi.h&gt; inclusions via &lt;linux/acpi_io.h&gt;</title>
<updated>2013-12-07T00:24:33+00:00</updated>
<author>
<name>Lv Zheng</name>
<email>lv.zheng@intel.com</email>
</author>
<published>2013-12-06T08:52:05+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=27d50c82714f6477ac690034b37d202f76eb4f70'/>
<id>27d50c82714f6477ac690034b37d202f76eb4f70</id>
<content type='text'>
To avoid build problems and breaking dependencies between ACPI header
files, &lt;acpi/acpi.h&gt; should not be included directly by code outside
of the ACPI core subsystem.  However, that is possible if
&lt;linux/acpi_io.h&gt; is included, because that file contains
a direct inclusion of &lt;acpi/acpi.h&gt;.

For this reason, remove the direct &lt;acpi/acpi.h&gt; inclusion from
&lt;linux/acpi_io.h&gt;, move that file from include/linux/ to include/acpi/
and make &lt;linux/acpi.h&gt; include it for CONFIG_ACPI set along with the
other ACPI header files.  Accordingly, Remove the inclusions of
&lt;linux/acpi_io.h&gt; from everywhere.

Of course, that causes the contents of the new &lt;acpi/acpi_io.h&gt; file
to be available for CONFIG_ACPI set only, so intel_opregion.o that
depends on it should also depend on CONFIG_ACPI (and it really should
not be compiled for CONFIG_ACPI unset anyway).

References: https://01.org/linuxgraphics/sites/default/files/documentation/acpi_igd_opregion_spec.pdf
Cc: Matthew Garrett &lt;mjg59@srcf.ucam.org&gt;
Signed-off-by: Lv Zheng &lt;lv.zheng@intel.com&gt;
Acked-by: Daniel Vetter &lt;daniel.vetter@ffwll.ch&gt;
[rjw: Subject and changelog]
Signed-off-by: Rafael J. Wysocki &lt;rafael.j.wysocki@intel.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
To avoid build problems and breaking dependencies between ACPI header
files, &lt;acpi/acpi.h&gt; should not be included directly by code outside
of the ACPI core subsystem.  However, that is possible if
&lt;linux/acpi_io.h&gt; is included, because that file contains
a direct inclusion of &lt;acpi/acpi.h&gt;.

For this reason, remove the direct &lt;acpi/acpi.h&gt; inclusion from
&lt;linux/acpi_io.h&gt;, move that file from include/linux/ to include/acpi/
and make &lt;linux/acpi.h&gt; include it for CONFIG_ACPI set along with the
other ACPI header files.  Accordingly, Remove the inclusions of
&lt;linux/acpi_io.h&gt; from everywhere.

Of course, that causes the contents of the new &lt;acpi/acpi_io.h&gt; file
to be available for CONFIG_ACPI set only, so intel_opregion.o that
depends on it should also depend on CONFIG_ACPI (and it really should
not be compiled for CONFIG_ACPI unset anyway).

References: https://01.org/linuxgraphics/sites/default/files/documentation/acpi_igd_opregion_spec.pdf
Cc: Matthew Garrett &lt;mjg59@srcf.ucam.org&gt;
Signed-off-by: Lv Zheng &lt;lv.zheng@intel.com&gt;
Acked-by: Daniel Vetter &lt;daniel.vetter@ffwll.ch&gt;
[rjw: Subject and changelog]
Signed-off-by: Rafael J. Wysocki &lt;rafael.j.wysocki@intel.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
