David Brownell wrote: > On Thursday 28 December 2006 12:02 am, Dirk Behme wrote: > >> static void >>-isp1301_work(void *data) >>+isp1301_work(struct work_struct *data) >> { >>- struct isp1301 *isp = data; >>+ struct isp1301 *isp = (struct isp1301 *)data; >> int stop; > > > Clearly incorrect. It should use container_of(). The work_struct > is NOT the first member of that struct... this will be oopsing. Yes, I see. Thanks! So, next attempt ;) Below both, ISP1301 and board-h3. Again, only checked for compilation. Dirk