On Fri, 2012-05-25 at 10:30 +0200, Alexander Graf wrote: > > + while (count--) { > > + switch (esize) { > > + case 0: tmp = ldub_phys(src); > > I'm surprised checkpatch didn't complain here. Please do > > case x: > foo(); > break(); > > > break; > > + case 1: tmp = lduw_phys(src); break; > > + case 2: tmp = ldl_phys(src); break; > > + case 3: tmp = ldq_phys(src); break; > > + default: > > + return H_PARAMETER; Checkpatch absolutely complained and I decided to ignore it, seriously, you really want to replace a nice & readable piece of code with something that takes 3 pages and is generally gross & ugly ? Some times, you have to ignore check patch and let sanity prevail. Ben. > Indentation? Not sure what's up with identation, I had it all fixed up to please checkpatch, maybe I screwed up the sending of the patch itself. Oh well, I'm off to hospital on monday so that will have to wait til I'm back (I regret you didn't make those comments on the previous iteration of the patch though). Ben. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html