Hi! On Wed, Feb 08, 2006 at 01:04:22PM +0000, Matthew Garrett wrote: > diff --git a/include/linux/pm.h b/include/linux/pm.h ... > +void pm_set_ac_status(int (*ac_status_function)(void)) > +{ > + down(&pm_sem); > + get_ac_status = ac_status_function; > + up(&pm_sem); > +} Why do you need a semaphore/mutex, when you only do one assignment, which is atomic by itself? BYtE Philipp -- / / (_)__ __ ____ __ Philipp Hahn / /__/ / _ \/ // /\ \/ / /____/_/_//_/\_,_/ /_/\_\ pmhahn@xxxxxxxxxxxxx