On Monday 08 February 2016 21:42:29 Tony Lindgren wrote: > * Pali Rohár <pali.rohar@xxxxxxxxx> [160208 12:37]: > > On Monday 08 February 2016 21:22:10 Tony Lindgren wrote: > > > > > > I know, but how many users and developers are looking into > > > > > > git *history* if they need to know what is missing in > > > > > > *current* kernel? > > > > > > > > > > > > I think that removing board-rx51.c file would mean that > > > > > > table is lost and everybody forget that this table needs > > > > > > to be ported to nokia n900 DTS file. > > > > > > > > > > How about send a patch to copy those comments to > > > > > board-n900.dts too? > > > > > > > > Ok, I'm fine with it. But I would like to see cpuidle omap > > > > support in DT. > > > > > > Hmm but we're already using arch/arm/mach-omap2/cpuidle34xx.c > > > to hit deeper idle states for DT based booting too? Or maybe > > > you mean something else? > > > > And this is our problem. Nokia N900 needs different cpu idle > > parameters as hardcoded in cpuidle34xx.c file. And because they > > are not in DT, but in kernel source code, we cannot overwrite them > > from n900 DTS file. > > Oh. Can we set different parameters in omap3_idle_init() > based on the soc_is_omap34xx()? We had it and it was removed, see: https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=231900afba52d6faddfb480cde4132d4edc089bc > You'd assume these parameters are SoC specific, not board > specific? Looks like board specific. > Regards, > > Tony This ATAGS thread is too big now and dicussion about cpuidle I already started in different thread named "Re: Nokia N900: Proper C-states". You are CCed, so better reply here. Sorry for interrupting :-) -- Pali Rohár pali.rohar@xxxxxxxxx
Attachment:
signature.asc
Description: This is a digitally signed message part.
