On Tue 30.10.07 12:45, Luis R. Rodriguez wrote:
> On 10/27/07, Nick Kossifidis <mick@xxxxxxxxxxx> wrote:
> > fill_tx_desc is used for fast frames operation, since
> > we don't support fast frames (and since fill_tx_desc
> > had a bug -thanx to Ulrich Meis for finding that out-)
> > these functions are not needed (+ they are misleading
> > because they don't "fill" any tx descriptor).
> >
> > I couldn't test this patch much so plz someone ACK it...
> >
> > It applies on top of my previous patches (i just thought
> > that [PATCH 8/7] won't look nice ;-) ).
> >
> >
> > Changes-licensed-under: ISC
> > Signed-off-by: Ulrich Meis <meis@xxxxxxxxxxxxxxxxxxx>
> > Signed-Off-by: Nick Kossifidis <mickflemm@xxxxxxxxx>
>
> Very nice, I've been testing this on 2.6.24-rc1 for a day without
> problems. I'll soon post results of of all your patches and your 1-7
> series on big endian. Compiling takes a while there, heh.. I guess I
> should be cross compiling huh.
People using the 5212 need the line below though if they wanna be able
to sent out frames :) It's missing in the 4word version. Besides that
the patches also seem to run fine here.
Uli
P.S.: Sry Nick, posted this to you yesterday but forgot to include the list.
diff --git a/drivers/net/wireless/ath5k/hw.c b/drivers/net/wireless/ath5k/hw.c
index 7b9920c..dc9b881 100644
--- a/drivers/net/wireless/ath5k/hw.c
+++ b/drivers/net/wireless/ath5k/hw.c
@@ -3641,6 +3641,8 @@ static int ath5k_hw_setup_4word_tx_desc(struct ath5k_hw *ah,
if (buff_len & ~AR5K_4W_TX_DESC_CTL1_BUF_LEN)
return -EINVAL;
+ tx_desc->tx_control_1 = buff_len & AR5K_2W_TX_DESC_CTL1_BUF_LEN;
+
tx_desc->tx_control_0 |=
AR5K_REG_SM(tx_power, AR5K_4W_TX_DESC_CTL0_XMIT_POWER) |
AR5K_REG_SM(antenna_mode, AR5K_4W_TX_DESC_CTL0_ANT_MODE_XMIT);
-
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
