File ipxe.19d0fab40f07eeea7fe6b9e0d4e8d4b0c2de215f.p... of Package xen

Overview Repositories Revisions Requests Users Attributes Meta

File ipxe.19d0fab40f07eeea7fe6b9e0d4e8d4b0c2de215f.patch of Package xen (Revision 59c4c69f148b2bf9fd1136c2fe8ebb61)

Currently displaying revision 59c4c69f148b2bf9fd1136c2fe8ebb61 , Show latest

From: Bruce Rogers <brogers@suse.com>
Date: Tue, 16 Feb 2021 11:29:41 -0700
Subject: 19d0fab40f07eeea7fe6b9e0d4e8d4b0c2de215f

[ath5k] Add missing AR5K_EEPROM_READ in ath5k_eeprom_read_turbo_modes

The GCC11 compiler pointed out something that apparently no previous
compiler noticed: in ath5k_eeprom_pread_turbo_modes, local variable
val is used uninitialized. From what I can see, the code is just
missing an initial AR5K_EEPROM_READ. Add it right before the switch
statement.

Signed-off-by: Bruce Rogers <brogers@suse.com>
Signed-off-by: Michael Brown <mcb30@ipxe.org>
---
 src/drivers/net/ath/ath5k/ath5k_eeprom.c | 1 +
 1 file changed, 1 insertion(+)

--- a/src/drivers/net/ath/ath5k/ath5k_eeprom.c
+++ b/src/drivers/net/ath/ath5k/ath5k_eeprom.c
@@ -407,24 +407,25 @@ done:
 static int
 ath5k_eeprom_read_turbo_modes(struct ath5k_hw *ah,
 			      u32 *offset, unsigned int mode)
 {
 	struct ath5k_eeprom_info *ee = &ah->ah_capabilities.cap_eeprom;
 	u32 o = *offset;
 	u16 val;
 	int ret;
 
 	if (ee->ee_version < AR5K_EEPROM_VERSION_5_0)
 		return 0;
 
+	AR5K_EEPROM_READ(o++, val);
 	switch (mode){
 	case AR5K_EEPROM_MODE_11A:
 		ee->ee_switch_settling_turbo[mode] = (val >> 6) & 0x7f;
 
 		ee->ee_atn_tx_rx_turbo[mode] = (val >> 13) & 0x7;
 		AR5K_EEPROM_READ(o++, val);
 		ee->ee_atn_tx_rx_turbo[mode] |= (val & 0x7) << 3;
 		ee->ee_margin_tx_rx_turbo[mode] = (val >> 3) & 0x3f;
 
 		ee->ee_adc_desired_size_turbo[mode] = (val >> 9) & 0x7f;
 		AR5K_EEPROM_READ(o++, val);
 		ee->ee_adc_desired_size_turbo[mode] |= (val & 0x1) << 7;

Places

File ipxe.19d0fab40f07eeea7fe6b9e0d4e8d4b0c2de215f.patch of Package xen (Revision 59c4c69f148b2bf9fd1136c2fe8ebb61)

Places