When setting the ARCSR registers we could use a better/more descriptive and already available variable name than plain "value".
Signed-off-by Ivo van Doorn <[EMAIL PROTECTED]> --- diff -rU3 wireless-dev-rt2x00-netif/drivers/net/wireless/d80211/rt2x00/rt2400pci.c wireless-dev-rt2x00-variable/drivers/net/wireless/d80211/rt2x00/rt2400pci.c --- wireless-dev-rt2x00-netif/drivers/net/wireless/d80211/rt2x00/rt2400pci.c 2006-08-27 16:32:51.000000000 +0200 +++ wireless-dev-rt2x00-variable/drivers/net/wireless/d80211/rt2x00/rt2400pci.c 2006-08-27 16:56:28.000000000 +0200 @@ -773,12 +773,12 @@ rt2x00_set_field32(®[0], TXCSR1_ACK_CONSUME_TIME, value); rt2x00_register_write(rt2x00dev, TXCSR1, reg[0]); - value = DEVICE_GET_RATE_FIELD(rate, PREAMBLE) ? 8 : 0; + preamble = DEVICE_GET_RATE_FIELD(rate, PREAMBLE) ? 8 : 0; - reg[0] = cpu_to_le32(0x00700400 | value); /* ARCSR2 */ - reg[1] = cpu_to_le32(0x00380401 | value); /* ARCSR3 */ - reg[2] = cpu_to_le32(0x00150402 | value); /* ARCSR4 */ - reg[3] = cpu_to_le32(0x000b8403 | value); /* ARCSR5 */ + reg[0] = cpu_to_le32(0x00700400 | preamble); /* ARCSR2 */ + reg[1] = cpu_to_le32(0x00380401 | preamble); /* ARCSR3 */ + reg[2] = cpu_to_le32(0x00150402 | preamble); /* ARCSR4 */ + reg[3] = cpu_to_le32(0x000b8403 | preamble); /* ARCSR5 */ rt2x00_register_multiwrite(rt2x00dev, ARCSR2, ®[0], sizeof(reg)); } diff -rU3 wireless-dev-rt2x00-netif/drivers/net/wireless/d80211/rt2x00/rt2500pci.c wireless-dev-rt2x00-variable/drivers/net/wireless/d80211/rt2x00/rt2500pci.c --- wireless-dev-rt2x00-netif/drivers/net/wireless/d80211/rt2x00/rt2500pci.c 2006-08-27 16:34:16.000000000 +0200 +++ wireless-dev-rt2x00-variable/drivers/net/wireless/d80211/rt2x00/rt2500pci.c 2006-08-27 16:56:34.000000000 +0200 @@ -839,12 +839,12 @@ rt2x00_set_field32(®[0], TXCSR1_ACK_CONSUME_TIME, value); rt2x00_register_write(rt2x00dev, TXCSR1, reg[0]); - value = DEVICE_GET_RATE_FIELD(rate, PREAMBLE) ? 8 : 0; + preamble = DEVICE_GET_RATE_FIELD(rate, PREAMBLE) ? 8 : 0; - reg[0] = cpu_to_le32(0x00700400 | value); /* ARCSR2 */ - reg[1] = cpu_to_le32(0x00380401 | value); /* ARCSR3 */ - reg[2] = cpu_to_le32(0x00150402 | value); /* ARCSR4 */ - reg[3] = cpu_to_le32(0x000b8403 | value); /* ARCSR5 */ + reg[0] = cpu_to_le32(0x00700400 | preamble); /* ARCSR2 */ + reg[1] = cpu_to_le32(0x00380401 | preamble); /* ARCSR3 */ + reg[2] = cpu_to_le32(0x00150402 | preamble); /* ARCSR4 */ + reg[3] = cpu_to_le32(0x000b8403 | preamble); /* ARCSR5 */ rt2x00_register_multiwrite(rt2x00dev, ARCSR2, ®[0], sizeof(reg)); } - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html