On 01/07/2014 04:11 PM, Rafael J. Wysocki wrote: >> >> Ok, I spoke with other developers and I apparently misunderstood the context >> here. Distro's enable these features and this is too detailed for them to know >> what to do with it. How about simply "Required to enable platform specific power >> managemnet features on Baytrail"? >> >> KISS is easier said than done. > > Well, I personally think that this code should go into arch/x86/ as library code > needed to access IOSF Sideband on some platforms. I probably would make modules > depending on it select it, so for example if the RAPL driver is going to be > built, your driver should be build either and rather unconditionally in that > case. > > So the rule should be "if something *may* need it, build it" in my opinion. > I thought we were targeting this for drivers/x86? However, perhaps with power management tied in that doesn't make too much sense. -hpa -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html