[RFC PATCH] bq27000 battery mutex protection

Werner Almesberger werner at openmoko.org
Mon Oct 12 18:37:06 CEST 2009


Michael Trimarchi wrote:
> Protect the data using a mutex. Fix a race that can happen when
> the user read from the sysfs and the worker execute in the middle.

Looks good, but I'd suggest to put the inner part of
bq27000_battery_get_property into a separate function and have a
small wrapper that does just the locking. This way, you can still
just return (keeps the code simpler) and there's no risk of ever
missing an unlock path.

- Werner



More information about the openmoko-kernel mailing list