Hi. On Mon, 2005-07-11 at 04:03, Pavel Machek wrote: > Hi! > > > +static void suspend2_suspend_2(void) > > +{ > > + if (!save_image_part1()) { > > + suspend_power_down(); > > + > > + if (suspend2_powerdown_method == 3) { > > + int temp_result; > > + > > + temp_result = read_pageset2(1); > > > Is that just me or do I see way too many numbers. suspend2_suspend_2 > is really funny name for a functions. powerdown_method should really > use some symbolic constants.
No, it's not just you. It's one of those hangovers from the original code that I hadn't gotten around to cleaning up. Symbolic constants now in place for the powerdown method too. Regards, Nigel -- Evolution. Enumerate the requirements. Consider the interdependencies. Calculate the probabilities. Be amazed that people believe it happened. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/