On Mon, Jan 16, 2017 at 04:17:58AM +0100, Daniel Golle wrote: > @@ -7131,6 +7236,12 @@ static int rt2800_validate_eeprom(struct rt2x00_dev *rt2x00dev) > rt2x00_set_field16(&word, EEPROM_NIC_CONF0_RF_TYPE, RF2820); > rt2800_eeprom_write(rt2x00dev, EEPROM_NIC_CONF0, word); > rt2x00_eeprom_dbg(rt2x00dev, "Antenna: 0x%04x\n", word); > + } else if (rt2x00_rt(rt2x00dev, RT5350)) { > + rt2x00_set_field16(&word, EEPROM_NIC_CONF0_RXPATH, 1); > + rt2x00_set_field16(&word, EEPROM_NIC_CONF0_TXPATH, 1); > + rt2x00_set_field16(&word, EEPROM_NIC_CONF0_RF_TYPE, RF3320); Here you set RF3320 .. > + rt2x00_eeprom_write(rt2x00dev, EEPROM_NIC_CONF0, word); > + rt2x00_eeprom_dbg(rt2x00dev, "Antenna: 0x%04x\n", word); > } else if (rt2x00_rt(rt2x00dev, RT2860) || > rt2x00_rt(rt2x00dev, RT2872)) { > /* > @@ -7265,6 +7376,8 @@ static int rt2800_init_eeprom(struct rt2x00_dev *rt2x00dev) > rt2800_eeprom_read(rt2x00dev, EEPROM_CHIP_ID, &rf); > else if (rt2x00_rt(rt2x00dev, RT3352)) > rf = RF3322; > + else if (rt2x00_rt(rt2x00dev, RT5350)) > + rf = RF5350; and here RF5350. This does not seems to be correct. Stanislaw