Francois Romieu wrote: > Janitoring - error handling during attach > - av7110_arm_sync(): small helper to factor out some code; > - av7110_attach() does not check the status code returned by all the > functions is uses; > - balance the error path in av7110_attach and have it easy to check. > Please check it; > - if everything is correctly balanced, device_initialized is not needed > anymore in struct av7110; > - av7110_detach(): no need to cast a void * pointer; > - av7110_detach(): die #ifdef, die ! > - change the returned value of av7110_av_exit() as it can't fail; > - change the returned value of av7110_ca_init() as it can fail. Removed > extraneous casts while are it; > - check for failure of vmalloc() in ci_ll_init(). > > Signed-off-by: Francois Romieu <romieu@xxxxxxxxxxxxx> Patch applied. Thanks, Johannes