Hi, The comment of struct ab8500_regulator_info says the delay means "startup/set voltage delay in us". I'm confused by the meaning, does it mean enable_time or set_voltage_time_sel time or both?
* @enable_time: Time taken for the regulator voltage output voltage to * stabilise after being enabled, in microseconds. * @set_voltage_time_sel: Time taken for the regulator voltage output voltage * to stabilise after being set to a new value, in microseconds. * The function provides the from and to voltage selector, the * function should return the worst case. Current code only has delay setting for AB8500_LDO_TVOUT, it looks like it means enable_time rather than the delay for set_voltage_time_sel. (well, AB8500_LDO_TVOUT is fixed voltage, it does not implement set_voltage_time_sel) ab8500_regulator_set_voltage_time_sel() returns info->delay, but all it's users (AB8500_LDO_AUX1, AB8500_LDO_AUX2, AB8500_LDO_AUX3, AB8500_LDO_INTCORE) do not has delay setting. ( So now ab8500_regulator_set_voltage_time_sel returns 0. ) Axel -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
