The first patch contains some white-space and formatting fixes that I made while I was looking at the following two TODO items: * the loop in kp2000_probe_cores() that uses probe_core_uio() also probably needs error handling. * probe_core_uio() probably needs error handling The second and third patches contain fixes for some sparse warnings. The last two patches contain the actual error-handling fixes. Jeremy Sowden (5): staging: kpc2000: inverted conditional in order to reduce indentation. staging: kpc2000: declare two functions as static. staging: kpc2000: added designated initializers to two structs. staging: kpc2000: added missing clean-up to probe_core_uio. staging: kpc2000: clean up after probe failure. drivers/staging/kpc2000/TODO | 2 - drivers/staging/kpc2000/kpc2000/cell_probe.c | 66 ++++++++++++-------- 2 files changed, 39 insertions(+), 29 deletions(-) Since v1: * in patch 3 removed the assignment of the struct-members initialized by the designated initializers. -- 2.20.1 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel