Messages in this thread | | | Date | Wed, 16 May 2007 19:37:06 -0400 | From | Ed Sweetman <> | Subject | Re: [PATCH] Kconfig powernow-k8 driver should depend on ACPI P-States driver |
| |
Daniel Drake wrote: > Ed Sweetman wrote: >> Like i mentioned off list, the problem here is that cpu freq modules >> dont depend (Kconfig) on CONFIG_ACPI_PROCESSOR, yet they do. > > Not really. Firstly, some of the cpufreq modules *do* depend on > CONFIG_ACPI_PROCESSOR. Secondly, the ones that don't have an existing > dependency do not actually depend on ACPI_PROCESSOR in some/most > configurations. > > I'll send in a patch to fix the real problem soon. > > Daniel
The way i patched it was to just include a "select ACPI_PROCESSOR" in the X86_POWERNOW_K8 Kconfig entry.
Since the "driver" that the user sees in the Kconfig is X86_POWERNOW_K8 is actually not a driver at all, our actual driver behaves differently since the "pseudo" driver only depends is CPUFREQ. This is misleading, as the actual driver beneath it, depends on ACPI_PROCESSOR too. Now the driver beneath it responds as you'd think. It disables itself when it's depends lines are invalid. Which is good. But the menu entry that we see is for X86_POWERNOW_K8, and that isn't disabled or anything when those depends lines of the driver it actually represents fails.
This is easily fixed with the select line in the "pseudo" driver ...which i find a little more appropriate than a depends line.
As for actual module dependency issues, i haven't bothered looking into that. As far as the Kconfig shows it shouldn't be allowed to have ACPI_PROCESSOR as a module at all. So maybe that's intended.
- 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/
| |