On Fri, Aug 17, 2012 at 06:17:38PM -0700, H Hartley Sweeten wrote: > + if (this_board->n_diochan) { > + s = dev->subdevices + subdev; This pointer math sucks still... I feel like the unreadable code is part of what caused this bug. We don't have to change these all at once. If we fix only one line, then at least one line will be readable and that is one more than before so *improvement*. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel