On 07/10/2011 05:28 AM, Joe Perches wrote:
> Signed-off-by: Joe Perches<j...@perches.com>

Acked-by: Pavel Roskin <pro...@gnu.org>

> ---
>   drivers/net/wireless/ath/ath5k/desc.c |    3 +++
>   1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath5k/desc.c 
> b/drivers/net/wireless/ath/ath5k/desc.c
> index 62172d5..f82383b 100644
> --- a/drivers/net/wireless/ath/ath5k/desc.c
> +++ b/drivers/net/wireless/ath/ath5k/desc.c
> @@ -107,10 +107,13 @@ ath5k_hw_setup_2word_tx_desc(struct ath5k_hw *ah, 
> struct ath5k_desc *desc,
>               case AR5K_PKT_TYPE_BEACON:
>               case AR5K_PKT_TYPE_PROBE_RESP:
>                       frame_type = AR5K_AR5210_TX_DESC_FRAME_TYPE_NO_DELAY;
> +                     break;
>               case AR5K_PKT_TYPE_PIFS:
>                       frame_type = AR5K_AR5210_TX_DESC_FRAME_TYPE_PIFS;
> +                     break;
>               default:
>                       frame_type = type;
> +                     break;
>               }

The intention here is to replace frame types from enum ath5k_pkt_type 
with their AR5210-specific counterparts.  So the intention is definitely 
to have breaks here.

Unfortunately, AR5210 cards are extremely rare these days.  I have one, 
but it only works with old motherboards.  It would take me half a day to 
dust off that system, compile the kernel and check the patch.  But I 
assume your patch is fine.  At least it's very unlikely to break anything.

-- 
Regards,
Pavel Roskin
_______________________________________________
ath5k-devel mailing list
ath5k-devel@lists.ath5k.org
https://lists.ath5k.org/mailman/listinfo/ath5k-devel

Reply via email to