
<div> </div><div> </div><div>29.06.2014, 20:34, "Сергей Василюгин" <vasilugin@yandex.ru>:</div><blockquote type="cite"><div> </div><div> </div><div>29.06.2014, 18:33, "Roman Yeryomin" <<a href="mailto:leroi.lists@gmail.com">leroi.lists@gmail.com</a>>:</div><blockquote type="cite"><p>On 28 June 2014 19:17, Сергей Василюгин <<a href="mailto:vasilugin@yandex.ru">vasilugin@yandex.ru</a>> wrote:</p><blockquote> 26.06.2014, 06:03, "Daniel" <<a href="mailto:daniel@makrotopia.org">daniel@makrotopia.org</a>>:<br /><br /> Hi Roman,<br /><br /> On 04/04/2014 04:39 PM, Roman Yeryomin wrote:<br /><br />  I worked on other things lately but plan to return to rt2x00 soon and<br />  maybe try ralink driver on trunk again.<br /><br /> I started looking into your patches and started to see things moving as far<br /> as<br /> you got.<br /> I suggest to define RT6352 and set chip.rt to that instead of checking for<br /> chip.rf == RF7620 in cases which were meant for RT5390.<br /> Is there any more recent version of your work on rt2x00 than the<br /> 630-rt2x00-add-mt<span><span>7620-20140131</span></span>.patch included in your first post? If so,<br /> please<br /> share it, I'd like to re-use what ever there is and try to botch things up a<br /> bit ;)<br /><br /> Cheers<br /><br /><br /> Daniel<br /><br /> After some attempts tonight I make working version of the patch. But my<br /> trunk revision really very old and I need time to rebase it.<br /> The key error was in rt2800_rfcsr_read for RF7620 (wrong bitmask). So<br /> *value = rt2x00_get_field32(reg, RF_CSR_CFG_DATA);<br /> instead of<br /> *value = rt2x00_get_field32(reg, RF_CSR_CFG_DATA_MT7620);<br /> broke all rf reading.</blockquote><p>Honestly that sounds weird because unless you change all the other<br />masks for mt7620 you will have them overlapped.<br />Also this is how that register is described in datasheet (the fields<br />are in reverse order comparing to all other socs).<br />If you say you got it working can you send at least binary image to<br />test (while you are trying to rebase it)?<br /><br />Regards,<br />Roman</p></blockquote><div>1. As my friends say - let me show :)</div><div>Your recently sent patch:</div><div> </div><div>static void rt2800_rfcsr_read(struct rt2x00_dev *rt2x00dev,<br />                              const unsigned int word, u8 *value)<br /> {<br />@@ -182,15 +221,31 @@<br />         * doesn't become available in time, reg will be 0xffffffff<br />         * which means we return 0xff to the caller.<br />         */<br />-       if (WAIT_FOR_RFCSR(rt2x00dev, &reg)) {<br />-               reg = 0;<br />-               rt2x00_set_field32(&reg, RF_CSR_CFG_REGNUM, word);<br />-               rt2x00_set_field32(&reg, RF_CSR_CFG_WRITE, 0);<br />-               rt2x00_set_field32(&reg, RF_CSR_CFG_BUSY, 1);<br />+       switch (rt2x00dev->chip.rf) {<br />+       case RF7620:<br />+               if (WAIT_FOR_RFCSR_MT7620(rt2x00dev, &reg)) {<br />+                       reg = 0;<br />+                       rt2x00_set_field32(&reg, RF_CSR_CFG_REGNUM_MT7620, word);<br />+                       rt2x00_set_field32(&reg, RF_CSR_CFG_WRITE_MT7620, 0);<br />+                       rt2x00_set_field32(&reg, RF_CSR_CFG_BUSY_MT7620, 1);<br />+<br />+                       rt2800_register_write_lock(rt2x00dev, RF_CSR_CFG, reg);<br />+<br />+                       WAIT_FOR_RFCSR_MT7620(rt2x00dev, &reg);<br />+               }<br />+               break;<br />+       default:<br />+               if (WAIT_FOR_RFCSR(rt2x00dev, &reg)) {<br />+                       reg = 0;<br />+                       rt2x00_set_field32(&reg, RF_CSR_CFG_REGNUM, word);<br />+                       rt2x00_set_field32(&reg, RF_CSR_CFG_WRITE, 0);<br />+                       rt2x00_set_field32(&reg, RF_CSR_CFG_BUSY, 1);<br /> <br />-               rt2800_register_write_lock(rt2x00dev, RF_CSR_CFG, reg);<br />+                       rt2800_register_write_lock(rt2x00dev, RF_CSR_CFG, reg);<br /> <br />-               WAIT_FOR_RFCSR(rt2x00dev, &reg);<br />+                       WAIT_FOR_RFCSR(rt2x00dev, &reg);<br />+               }<br />+               break;<br />        }<br /> <br />        *value = rt2x00_get_field32(reg, RF_CSR_CFG_DATA);<br />@@ -198,6 +253,12 @@<br />        mutex_unlock(&rt2x00dev->csr_mutex);<br /> }</div><div> </div><div>use call</div><div>*value = rt2x00_get_field32(reg, RF_CSR_CFG_DATA);</div><div>for both case - default and mt7620. I call</div><div>*value = rt2x00_get_field32(reg, RF_CSR_CFG_DATA_MT7620);</div><div>for mt7620 :)</div><div> </div><div>2. Binary images: <a href="https://yadi.sk/d/OeTxrigZVMg8g">https://yadi.sk/d/OeTxrigZVMg8g</a></div><div>Dlink dir620f1 is my test board (pkg_id=0, chip_ver=2, eco_num=3 - auto detect not working yet). AFAIK asus rt-n14u use the same.</div><div> </div></blockquote><div>Updated.</div><div> </div><div>---<br />serge</div><div> </div>