On Wed, Aug 28, 2013 at 09:55:00AM +0800, Chen Gang wrote: > So in my opnion, it will be better to let all level 1 related functions > within "if (level >= 1) {...}" code block, that will make code clearer > to readers (or some members may doubt about this code). I'm not going to apply your patch, otherwise I would have done that already. > BTW: could you provide more information about why need stsi_15_1_x() be > in "level 1" ? That's what the architecture says. You can download it here: publibfi.boulder.ibm.com/epubs/pdf/dz9zr009.pdf -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html