2008/7/22 Jiri Slaby <jirislaby@xxxxxxxxx>: > On 07/20/2008 05:52 AM, Nick Kossifidis wrote: >> Last minute fix... >> >> AR5K_CPCORN is changed to AR5K_CPCOVF in reg list update so we have to update debug.c in order to compile >> >> Changes-licensed-under: ISC >> Signed-off-by: Nick Kossifidis <mickflemm@xxxxxxxxx> >> >> --- >> diff --git a/drivers/net/wireless/ath5k/debug.c b/drivers/net/wireless/ath5k/debug.c >> index 41d5fa3..6fa6c8e 100644 >> --- a/drivers/net/wireless/ath5k/debug.c >> +++ b/drivers/net/wireless/ath5k/debug.c >> @@ -129,7 +129,7 @@ static struct reg regs[] = { >> REG_STRUCT_INIT(AR5K_CPC1), >> REG_STRUCT_INIT(AR5K_CPC2), >> REG_STRUCT_INIT(AR5K_CPC3), >> - REG_STRUCT_INIT(AR5K_CPCORN), >> + REG_STRUCT_INIT(AR5K_CPCOVF), >> REG_STRUCT_INIT(AR5K_RESET_CTL), >> REG_STRUCT_INIT(AR5K_SLEEP_CTL), >> REG_STRUCT_INIT(AR5K_INTPEND), > > Please amend this to the patch 1 to not break bisectability. I see your point but this series is meant to be applied as a whole, i didn't notice this bug until i tested the code with debugging enabled (which is off by default), that's why it's "last minute fix". Is it really that much needed to repost the first patch ? I mean the patch series works and doesn't introduce any problems... -- 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