martinayotte Posted April 5, 2019 Author Share Posted April 5, 2019 15 minutes ago, dreddit said: The legacy fix involves the PH07 but find no reference on the board unless it refers to a gpio pin? Yes, but this PH7 is the SPDIF-OUT for Audio, which is not even connect on OPi3. I doubt this fix has anything to do with reboot issue, especially that this FEX attached this PH7 to "recovery" button ... 0 Quote Link to comment Share on other sites More sharing options...
martinayotte Posted April 5, 2019 Author Share Posted April 5, 2019 15 minutes ago, dreddit said: The legacy fix involves the PH07 but find no reference on the board unless it refers to a gpio pin? Yes, but this PH7 is the SPDIF-OUT for Audio, which is not even connect on OPi3. I doubt this fix has anything to do with reboot issue, especially that this FEX attached this PH7 to "recovery" button ... 0 Quote Link to comment Share on other sites More sharing options...
martinayotte Posted April 5, 2019 Author Share Posted April 5, 2019 15 minutes ago, dreddit said: The legacy fix involves the PH07 but find no reference on the board unless it refers to a gpio pin? Yes, but this PH7 is the SPDIF-OUT for Audio, which is not even connect on neither OPi3, OPiOnePlus or OPiLite2. I doubt this fix has anything to do with reboot issue, especially that this FEX attached this PH7 to "recovery" button ... 0 Quote Link to comment Share on other sites More sharing options...
frozeus Posted April 7, 2019 Share Posted April 7, 2019 SPIDF-OUT is PL9. I think the legacy version doesn't use the watchdog or not the same to reboot the board. EDIT: I think it uses the ARISC co-processor does someone with the h64 can test if the WDOG works ? /* This should reset the system after 0.5s */ # devmem $((0x030090a0 + 0x0014)) 0x00000001 # devmem $((0x030090a0 + 0x0018)) 0x00000000 # devmem $((0x030090a0 + 0x0018)) 32 0x1 /* This should reset the system after 1s (as recommended in the User Manual) */ # devmem $((0x030090a0 + 0x0014)) 0x00000001 # devmem $((0x030090a0 + 0x0018)) 0x00000000 # devmem $((0x030090a0 + 0x0018)) 32 $((0x1 << 4)) # devmem $((0x030090a0 + 0x0018)) 0x00000010 # devmem $((0x030090a0 + 0x0018)) 32 $((0x1 << 4 | 0x1)) 0 Quote Link to comment Share on other sites More sharing options...
frozeus Posted April 7, 2019 Share Posted April 7, 2019 Changing to the R_WDOG instead of WDOG in ATF fix the issue. 0001-HACK-plat-allwinner-common-Use-R_WDOG.patch 0 Quote Link to comment Share on other sites More sharing options...
martinayotte Posted April 7, 2019 Author Share Posted April 7, 2019 5 hours ago, froezus said: SPIDF-OUT is PL9. You are better looking at the schematic of any H6 (OPi3, OPiOnePlus, OPILite2) before arguing, PL9 is S-IR-RX, PH7 is really SPDIF-OUT ... 0 Quote Link to comment Share on other sites More sharing options...
martinayotte Posted April 7, 2019 Author Share Posted April 7, 2019 47 minutes ago, froezus said: Changing to the R_WDOG instead of WDOG in ATF fix the issue. Where did you find this patch ? We are not seeing it on linux-sunxi ML ... Did you tried out ? 0 Quote Link to comment Share on other sites More sharing options...
frozeus Posted April 7, 2019 Share Posted April 7, 2019 12 minutes ago, martinayotte said: You are better looking at the schematic of any H6 (OPi3, OPiOnePlus, OPILite2) before arguing, PL9 is S-IR-RX, PH7 is really SPDIF-OUT ... Oups indeed, My brain switch between IR and SPDIF. The patch is from me and yes I tested it. Still not posted it because it's more a hack. 0 Quote Link to comment Share on other sites More sharing options...
martinayotte Posted April 7, 2019 Author Share Posted April 7, 2019 6 minutes ago, froezus said: The patch is from me and yes I tested it. Still not posted it because it's more a hack. Oh ! So, @froezus is in fact Clément Péron ? Alors, bienvenue sur Armbian ... Je preśume que tu es basé en France, selon l'heure de ta patch ? Moi, je suis au Québec ... Je vais donc essayer celle-ci sous peu ! Salutation ! 0 Quote Link to comment Share on other sites More sharing options...
frozeus Posted April 7, 2019 Share Posted April 7, 2019 12 minutes ago, martinayotte said: Oh ! So, @froezus is in fact Clément Péron ? Alors, bienvenue sur Armbian ... Je preśume que tu es basé en France, selon l'heure de ta patch ? Moi, je suis au Québec ... Je vais donc essayer celle-ci sous peu ! Salutation ! Oui, salutation à toi aussi ! 0 Quote Link to comment Share on other sites More sharing options...
martinayotte Posted April 8, 2019 Author Share Posted April 8, 2019 3 hours ago, froezus said: The patch is from me and yes I tested it. Still not posted it because it's more a hack. Did you got chance to figure out if the changes breaks or works for H3/H5/A64 ? If not, we will need to build different BL31 for H6 differently from H3/H5/A64, since I'm not fluent with ATF, I'm not even sure that Armbian scripts distinguish between those ATF flavors accordingly ... Did you exchange discussions with @megi and @Icenowy and any other about the issue ? Maybe the Official ML will still be a good place ... Thanks for your work, since we struggling about the issue since at least 3 months ... 0 Quote Link to comment Share on other sites More sharing options...
megi Posted April 8, 2019 Share Posted April 8, 2019 @froezus Nice! Thank you. I've changed your patch, so that it uses R_WDOG only on H6. https://megous.com/dl/tmp/0001-Fix-reset-issue-on-H6-by-using-R_WDOG.patch It's still a hack, but at least it doesn't touch other SoCs. U-boot will also need patching, because it doesn't use ATF for reset. I tested it on Opi3, and it works. :)) 1 Quote Link to comment Share on other sites More sharing options...
frozeus Posted April 8, 2019 Share Posted April 8, 2019 Before trying any patch it would better to first understand, how it works on PineH64. 0 Quote Link to comment Share on other sites More sharing options...
ilchenko Evgeny Posted April 8, 2019 Share Posted April 8, 2019 1 hour ago, froezus said: Before trying any patch it would better to first understand, how it works on PineH64. please tell us how to install this patch 0 Quote Link to comment Share on other sites More sharing options...
frozeus Posted April 8, 2019 Share Posted April 8, 2019 You just need to put it in build/patch/atf/atf-sunxi64/ 0 Quote Link to comment Share on other sites More sharing options...
martinayotte Posted April 8, 2019 Author Share Posted April 8, 2019 13 hours ago, megi said: U-boot will also need patching, because it doesn't use ATF for reset. What would be this second patch for the U-Boot "reset" ? EDIT : for the "reboot" patch, I'm confirming : it works ! Thanks a lot to both @froezus and @megi !!! EDIT2 : tested on all my H6, OPiOne+/OPiLite2/OPi3 ... And patch committed in Armbian ! 4 Quote Link to comment Share on other sites More sharing options...
frozeus Posted April 8, 2019 Share Posted April 8, 2019 My UART is broken could you test this patch ? https://github.com/clementperon/u-boot/commit/dc1abccb3628aefa6df3d44e027f33d8afc3fc86 Thanks 0 Quote Link to comment Share on other sites More sharing options...
martinayotte Posted April 8, 2019 Author Share Posted April 8, 2019 46 minutes ago, froezus said: My UART is broken You should do like me : I have more than a dozen of USB-TTL around, almost every boards have one, so I don't have to waste time unplugging/plugging them while testing. I've even placed a new order on eBay to get more ... 46 minutes ago, froezus said: could you test this patch ? Tested on OPiOnePlus, now doing the OPiLite2 and OPi3 ... Patch is already committed to Armbian ! Thanks a lot for the help ! EDIT : OPiLite2 and OPi3 tested too ... 1 Quote Link to comment Share on other sites More sharing options...
frozeus Posted April 8, 2019 Share Posted April 8, 2019 I made a short circuit on my board and the TX pin is no more working... 0 Quote Link to comment Share on other sites More sharing options...
martinayotte Posted April 8, 2019 Author Share Posted April 8, 2019 2 minutes ago, froezus said: I made a short circuit on my board and the TX pin is no more working... Oh ! On the board side ... Next time, it is better to short the USB-TTL side, they are less expensive ... 0 Quote Link to comment Share on other sites More sharing options...
frozeus Posted April 8, 2019 Share Posted April 8, 2019 4 minutes ago, martinayotte said: Oh ! On the board side ... Next time, it is better to short the USB-TTL side, they are less expensive ... Yes, at least the RX is still working. BTW i have sent the patch for U-Boot if you could add your "Tested-by" tag for the reset patch. Thanks 0 Quote Link to comment Share on other sites More sharing options...
ilchenko Evgeny Posted April 8, 2019 Share Posted April 8, 2019 2 hours ago, froezus said: You just need to put it in build/patch/atf/atf-sunxi64/ i cant find this path( 0 Quote Link to comment Share on other sites More sharing options...
frozeus Posted April 8, 2019 Share Posted April 8, 2019 It has been merged so no need to add : https://github.com/armbian/build/tree/master/patch/atf/atf-sunxi64 0 Quote Link to comment Share on other sites More sharing options...
ilchenko Evgeny Posted April 8, 2019 Share Posted April 8, 2019 2 hours ago, froezus said: It has been merged so no need to add : https://github.com/armbian/build/tree/master/patch/atf/atf-sunxi64 hmmm, orange pi 3 does not support? Only supported boards? how i can fix reboot on orange pi 3 ? 0 Quote Link to comment Share on other sites More sharing options...
martinayotte Posted April 8, 2019 Author Share Posted April 8, 2019 3 minutes ago, ilchenko Evgeny said: how i can fix reboot on orange pi 3 ? I've committed both Megi and Froezus patches earlier today in Armbian build scripts, so you simply need to recompile a new U-Boot and install it. 0 Quote Link to comment Share on other sites More sharing options...
frozeus Posted April 8, 2019 Share Posted April 8, 2019 I have proposed a proper patch to ATF hope it will be accepted soon https://github.com/ARM-software/arm-trusted-firmware/pull/1926 0 Quote Link to comment Share on other sites More sharing options...
martinayotte Posted April 9, 2019 Author Share Posted April 9, 2019 2 hours ago, froezus said: I have proposed a proper patch to ATF hope it will be accepted soon I'm a bit confused ... @megi patch seems to be H6 specific, but yours seen to affect all Allwinners SoC ... Can you details ? ... 0 Quote Link to comment Share on other sites More sharing options...
frozeus Posted April 9, 2019 Share Posted April 9, 2019 no the SUNXI_WDT_QUIRK is only set to 1 in sun50i_h6/platform.mk not for all platforms. 0 Quote Link to comment Share on other sites More sharing options...
Icenowy Posted April 9, 2019 Share Posted April 9, 2019 I tested on a Rongpin RP-H6B development board, and it also resets properly with normal WDOG. So maybe only OPi's are affected? 0 Quote Link to comment Share on other sites More sharing options...
dolphs Posted April 9, 2019 Share Posted April 9, 2019 just compiled fresh full OS image ( Armbian_5.78_Orangepioneplus_Debian_stretch_dev_5.0.7.img ) but alas " reboot " hangs, so therefore my guess is I was too early and will wait another few days building from scratch, thanks for your efforts! :~/armbian/patch/atf/atf-sunxi64$ ls -la total 36 drwxrwxr-x 4 dolphs dolphs 4096 Apr 9 16:32 . drwxrwxr-x 5 dolphs dolphs 4096 Mar 11 00:03 .. -rw-r--r-- 1 root root 902 Apr 9 16:32 0001-Fix-reset-issue-on-H6-by-using-R_WDOG.patch -rw-rw-r-- 1 dolphs dolphs 1112 Mar 11 00:03 add-SRAM-mapping-for-SCPI.patch.disabled drwxrwxr-x 2 dolphs dolphs 4096 Apr 9 16:32 board_pine64so drwxrwxr-x 2 dolphs dolphs 4096 Mar 11 00:03 board_pinebook-a64 -rw-rw-r-- 1 dolphs dolphs 1079 Mar 11 00:03 enable-a53-errata-workaround.patch.disabled -rw-rw-r-- 1 dolphs dolphs 1315 Mar 11 00:03 enable-additional-regulators.patch -rw-rw-r-- 1 dolphs dolphs 525 Mar 11 00:03 set-rsb-to-nonsec.patch Just noticed the patch is added with root user instead of mine, perhaps need to chown it and give it another try? 0 Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.