On Wed, Jan 22, 2014 at 04:23:14PM +0000, Lorenzo Pieralisi wrote: > On Wed, Jan 22, 2014 at 11:52:14AM +0000, Mark Brown wrote: > > Actually looking at the OPP binding I do wonder if it might not be > > better to have a v2/rich binding for them which is extensible - the fact > > that it's not possible to add additional information seems like an > > issue, this can't be the only thing anyone might want to add and lining > > up multiple tables is never fun. > On one hand OPP bindings need improvement and that's on the cards. I am not > really following what you mean by "extensible", I only want to make sure > that the C-state bindings do not become too complex. By extensible I guess I mostly mean not just a list of numbers we can't add additional information to without breaking compatibility. Having to look through tables and make sure they all use the same indexes gets error prone and generally miserable. > Do you mean extending OPP bindings to add eg C-state information there > (or whatever piece of information that is OPP dependent) ? > It seems a bit of a stretch but I can think about that. > I think that C-state properties are better defined in the C-state bindings > that was the idea but I am open to suggestions. I think defining in these bindings makes total sense, it's just providing an extension to the OPP binding (which is already being looked at anyway). The exensibility issue I see is that with the current OPP binding it's inelegant for another binding to add extra information about an operating point relevant to that binding.
Attachment:
signature.asc
Description: Digital signature