What do you think about using the same *type instance* for all the values below, e.g. "fan{n}"? That might make it easier to see that "fanspeed-fan0" and "temperature-fan-vrm0" (then "temperature-fam0") belong to the same fan. Also, "voltage-fan-voltage0" is kind of redundant (dito: current).