> Tested-by: Kevin Hilman <khil...@baylibre.com> > > I've been using some local trace_printk() to do exactly this, so I fully > support having some official tracepoints here. > > For my local hacks, I was trackin the state index as well as the state > value since for quick debug, I find the index to more human readable > than the state value, which I have to compare with the > arm,psci-suspend-pararm from the DT. > > Anyways, thanks for submitting this! > > Kevin
Thank you so much for the endorsement. Showing the index sounds like a good idea, although it should be done in a different layer (probably in the GenPD governor or cpuidle-psci driver?) because idx here should be the different index from the one we're looking for. I will keep my patch as it is just for now. Thank you for the idea!
