On 2/8/21 11:34 PM, Pali Rohár wrote:
On Monday 08 February 2021 23:21:38 Pali Rohár wrote:
On Monday 08 February 2021 23:15:33 Lukasz Majewski wrote:
Hi Pali,
Resended v2 patch series with fixed commit messagesThis patch series fix usbtty code (serial console via USB peripheral mode), fix underlying musb peripheral code, fix compilation of CONFIG_USB_DEVICE (used by usbtty), remove unused Nokia RX-51 code to decrease size of U-Boot binary and finally enable usbtty serial console for Nokia RX-51.
With this patch series debugging of Nokia RX-51 can be done also via USB serial console.
It fixes also stability of musb code and allows usage of file transfers via Kermit protocol on Nokia RX-51. Kermit file transfer via U-Boot loadb command is stable on Nokia N900 and gives about 52kB/s transfer rate.
On computer this serial console is accessible via /dev/ttyACM0 device.
I've integrated your patchset and now it turns out that the u-boot size is too big: https://dev.azure.com/lukma633/U-Boot/_build/results?buildId=24&view=res... https://dev.azure.com/lukma633/U-Boot/_build/results?buildId=24&view=log...
Ah :-( In November when I sent these patches, U-Boot binary was smaller.
The branch is https://github.com/lmajewski/u-boot-dfu/tree/testing
Have you experienced similar issues?
Yes, memory for U-Boot is limited. It is needed to decrease size of U-Boot binary and then it will work.
I will try to look at it later and find some dead code which can be commented or removed to decrease binary size...
Just a quick test, could you try to add
CONFIG_OPTIMIZE_INLINING=y
into configs/nokia_rx51_defconfig file, if it helps?
Surely you can try build-testing the patches yourself ? :-)