On Fri, 2008-07-25 at 00:36 +0300, Nick Kossifidis wrote: > 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 OK, fine with me. -- Regards, Pavel Roskin -- 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
