2008/7/25 Pavel Roskin <proski@xxxxxxx>: > On Sun, 2008-07-20 at 06:32 +0300, Nick Kossifidis wrote: >> * Restore saved initial value after POST > ... >> @@ -170,6 +172,10 @@ static int ath5k_hw_post(struct ath5k_hw *ah) >> var_pattern = 0x003b080f; >> ath5k_hw_reg_write(ah, var_pattern, cur_reg); >> } >> + >> + /* Restore previous value */ >> + ath5k_hw_reg_write(ah, init_val, cur_reg); >> + >> } > > Perhaps "at the end of POST"? Anyway, the code looks good to me. > We are not restoring at the end of POST but after we end with each of the two registers, so i guess it's ok this way -- GPG ID: 0xD21DB2DB As you read this post global entropy rises. Have Fun ;-) Nick -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html