Add INA220 monitor to RTM_PWR line

It may be desirable to monitor RTM power consumption

I agree, see no harm on this.

Maybe we should consider using a multi-channel measurement IC. INA3221[1] is very similar to the current one, and measures 3 channels at once. Any suggestions?

[1] http://www.ti.com/product/INA3221

I like this new IC. Apart from having 3 channels, it also has programmable alert outputs.
Maybe these outputs will be useful for MMC firmware?

To use these alert outputs, we may need to free some pins in the MMC. There will be some freeing due to the SCANSTA and other architectural changes, let us see if there are any left.

OTOH, polling is OK for the openMMC architecture and maybe this is not that necessary. Do you agree is more of a nice-to-have than a demanded feature??

As Gustavo said, polling is no problem at all, since the INA sensor task runs independently on openMMC (maybe for other firmwares, this could be useful).
Those critical/warning signals are all open-drain and can be tied together to a single pin, allowing us to use a hybrid interruption-polling scheme.
This scheme is already been used in the LM75 Temperature sensors, in which all Alert signals are tied in the OVERTEMP net.

I agree that these alarm outputs are just a nice to have feature.
I think the final decision should be left to Henrique as he's the guru of openMMC.

Added in commit:3a03dbe6

assigned to @tprzywoz

changed milestone to %AFC v4.0

Same as #36 (closed) - I used INA3221 and added RTM PWR measurement.

Solved in c961a747

closed