On Monday, September 02, 2013 10:54:17 AM Daniel Lezcano wrote: > On 08/30/2013 12:21 PM, Bartlomiej Zolnierkiewicz wrote: > > Add "cpuidle-exynos.max_states=" parameter to allow user to specify > > the maximum of allowed CPU idle states for ARM EXYNOS cpuidle driver. > > > > This change is needed because C1 state (AFTR mode) is often not able > > to work properly due to incompatibility with some bootloader versions. > > > > Usage examples: > > > > "cpuidle-exynos.max_states=1" disables C1 state (AFTR mode). > > > > "cpuidle-exynos.max_states=0" disables the driver completely. > > > > Signed-off-by: Bartlomiej Zolnierkiewicz <[email protected]> > > Signed-off-by: Kyungmin Park <[email protected]> > > Cc: Tomasz Figa <[email protected]> > > Cc: Amit Daniel Kachhap <[email protected]> > > There is a max_cstate option for acpi and intel idle. There is also the > cpuidle.off=1 option. As the semantic is the same, I think adding a > common cpuidle option usable for all the drivers is better.
I thought about making the option common for all cpuidle drivers first but due to support for multiple cpuidle drivers on one machine (i.e. big.LITTLE), per-driver option looked like a better approach. Should I make the option common and not worry about multiple drivers on one machine support? Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
