<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
Simon Horman wrote:
<blockquote cite="mid:20091222045540.GB4411@verge.net.au" type="cite">
<blockquote type="cite">
<pre wrap="">Actually, most ARM platforms didn't need PCI domains.
So, original patch is
+config PCI_DOMAINS
+ def_bool y
+ depends on PCI && ARCH_CXXXXXX
+
But I removed the "&& ARCH_CXXXXXXX" before I provided the patch.
Because ARCH_CXXXXXXX is not exist in ARM Linux kernel now (maybe it
would be committed at someday)
</pre>
</blockquote>
<pre wrap=""><!---->
Ok, now I am completely confused. Nothing currently in-tree should
use PCI_DOMAINS, but it defaults to y and nothing switches it to n?
Perhaps the following would be more appropriate:
config PCI_DOMAINS
bool
And then platforms can "select PCI_DOMAINS" as needed.
</pre>
</blockquote>
You are right, <br>
Don't use def_bool is right choice. <br>
<br>
I only maintain one arch, so I didn't think too much. <br>
<br>
Should I regenerate a new patch, or you will handle it ?<br>
<br>
<br>
</body>
</html>