On Mon, Jan 14, 2019 at 12:39:40PM -0700, Jerry Snitselaar wrote: > Alexey was following the example of tpm_tis_update_timeouts() which > returns true if the timeouts were updated, and otherwise returns > false. The bool here makes sense to me, but what rc would you suggest > in this case? Maybe the pattern used there is not that great then. The callback should simply be update_durations(chip), and it would do whatever updates needed and either return zero or -errno. And of course update durations_adjusted flag because that is needed in sysfs. /Jarkko
