09:38 <krzk> PabloPL: Hi, Indeed I did not think about this... let me take a look...

09:49 <krzk> PabloPL: first, I do not see the search for smp_operaitons... the ARM generic driver indeed looks for registered cpuidle methods

09:49 <krzk> but not smp operations

09:49 <krzk> where do you have the smp_operations?

10:14 <PabloPL> krzk: in arm_cpuidle_init inside arch/arm/kernel/cpuidle.c

10:15 <PabloPL> krzk: it's needed for "enable-method" (which according to docs should be optional, but then cpuidle is not working)

10:23 <PabloPL> krzk: About cpu_operations, in few other places where i've found them registered (using CPU_METHOD_OF_DECLARE) i saw that there was always smp_operations passed to this

10:24 <PabloPL> krzk: so i'm not sure if this is required (and in s5pv210 case it'll be empty) or if i shouldn't use smp_operations, how it should look

11:41 <krzk> PabloPL: in arm_cpuidle_init() you need cpuidle_method, not cpu

11:41 <krzk> this is CPUIDLE_METHOD_OF_DECLARE

11:41 <krzk> not CPU_METHOD_OF_DECLARE

11:44 <krzk> but anyway I am not sure whether cpuidle-arm is good solution for you... it is a generic ARM driver which uses DT to choose the enable-method, exactly like you mentioned

11:44 <krzk> It came from ARMv8 where the enable-method is PSCI

11:45 <krzk> The other solution - maybe easier - is just to take the exynos driver and extend it

11:45 <krzk> It is a very simple driver which calls ARM mach code for entering deeper modes - exynos_enter_lowpower()

11:50 <krzk> PabloPL: the driver will call the mach-supplied callback for entering low power mode (supplied through platform_data). See rch/arm/mach-exynos/exynos.c (and skip the coupled case).

13:28 <PabloPL> krzk: Ok, thanks. will look at this

16:32 <wwilly> typically why I don't like hardkernel software, kernel panic with a fresh install of their last img...