[PATCH 2/3] libertas: fix memory alignment problems on the blackfin

Holger Schurig hs4233 at mail.mn-solutions.de
Fri Jan 25 02:57:16 EST 2008


> diff --git a/drivers/net/wireless/libertas/dev.h
> b/drivers/net/wireless/libertas/dev.h index 58d7ef6..5a69f2b
> 100644
> --- a/drivers/net/wireless/libertas/dev.h
> +++ b/drivers/net/wireless/libertas/dev.h
> @@ -349,7 +349,7 @@ struct assoc_request {
>         u8 channel;
>         u8 band;
>         u8 mode;
> -       u8 bssid[ETH_ALEN];
> +       u8 bssid[ETH_ALEN] __attribute__ ((aligned (2)));
>
>         /** WEP keys */
>         struct enc_key wep_keys[4];


This would go, but it would also work to simply re-arrange the 
fields. assoc_req doesn't have a direct hardware representation, 
so we can move the fields at will, e.g. the u8

What would you prefer?



More information about the libertas-dev mailing list