Re: [PATCH 8/28] rt2x00: optimize mac/bssid writing

[Michael Buesch <mb@xxxxxxxxx>]

On Wednesday 28 February 2007 15:07, Ivo van Doorn wrote:
> Handling the mac and bssid configuration can be done much easier
> by writing the passed data directly into the register instead
> of moving it to a local variable first.
> 
> Signed-off-by: Ivo van Doorn <IvDoorn@xxxxxxxxx>
> 
> ---
> 
> diff --git a/drivers/net/wireless/mac80211/rt2x00/rt2400pci.c b/drivers/net/wireless/mac80211/rt2x00/rt2400pci.c
> index 27e151d..b6bf9f3 100644
> --- a/drivers/net/wireless/mac80211/rt2x00/rt2400pci.c
> +++ b/drivers/net/wireless/mac80211/rt2x00/rt2400pci.c
> @@ -319,14 +319,11 @@ static inline void rt2400pci_close_debugfs(struct rt2x00_dev *rt2x00dev){}
>   */
>  static void rt2400pci_config_bssid(struct rt2x00_dev *rt2x00dev, u8 *bssid)
>  {
> -	u32 reg[2] = { 0, 0 };
> -
>  	/*
>  	 * The BSSID is passed to us as an array of bytes,
>  	 * that array is little endian, so no need for byte ordering.
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
>  	 */
> -	memcpy(&reg, bssid, ETH_ALEN);
> -	rt2x00_register_multiwrite(rt2x00dev, CSR5, &reg[0], sizeof(reg));
> +	rt2x00_register_multiwrite(rt2x00dev, CSR5, (u32*)bssid, ETH_ALEN);
                                                    ^^^^^^

This doesn't break on BE machines?

[Snipped the rest of the patch. Same comment applies there, too]

-- 
Greetings Michael.
-
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