Installation adventures.

[issalig]

I create this not to mix installation problems with other stuff.

[issalig]

@cristianoag It asks for a repository

$ sudo apt-get build-dep openmsx
Leyendo lista de paquetes... Hecho
E: Debe poner algún URI «deb-src» en su «sources.list»

[cristianoag]

openmsx.org/manual/compile.html Get Outlook for iOS<https://aka.ms/o0ukef>

…

________________________________ From: issalig ***@***.***> Sent: Tuesday, February 8, 2022 3:49:37 PM To: S0urceror/MSX-USB ***@***.***> Cc: Subscribed ***@***.***> Subject: Re: [S0urceror/MSX-USB] Installation adventures. (Issue #11) @crsitianoag It asks for a repository $ sudo apt-get build-dep openmsx Leyendo lista de paquetes... Hecho E: Debe poner algún URI «deb-src» en su «sources.list» — Reply to this email directly, view it on GitHub<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FS0urceror%2FMSX-USB%2Fissues%2F11%23issuecomment-1032947057&data=04%7C01%7C%7Ce9cb50ca8d4b48f0e79b08d9eb33c203%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637799429807528016%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=eYBOiwHsCy0ZHCADBnBrCP%2FXkbSdB68Tuz33KIoYxTg%3D&reserved=0>, or unsubscribe<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALSAPKWQVQQFCOIMZWJUZQ3U2FQUDANCNFSM5N3JYU6Q&data=04%7C01%7C%7Ce9cb50ca8d4b48f0e79b08d9eb33c203%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637799429807528016%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=PrtLvN3Pl48tt3%2BuAHiEvVJ86HANzDySg8VohTUkG6I%3D&reserved=0>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[issalig]

Well, I finally compiled S0urceror's OpenMSX , in my system I needed sudo apt-get install tcl-dev not libtclcl1-dev
I still have to setup download machine roms and more config stuff, but this will be tomorrow.
Thanks @cristianoag for your help.

[issalig]

Finally, I have everything working.
I started a document not to loose all the steps https://github.com/issalig/MSX-USB/blob/master/installation.md
It is still work in progress.

[issalig]

I found a bug while I was checking my ch376s with https://github.com/S0urceror/MSX-USB/blob/master/test/usb_via_arduino/src/main.parallel.cpp

main_paralell crashed when printing iProd due to a small buffer

MSX-USB/test/usb_via_arduino/src/main.parallel.cpp

Line 1527 in 2de667e

printf (" iProduct\t\t\t%d\t%s\n",device->iProduct,get_string2 (device_address,device->iProduct).c_str());

In get_string2

MSX-USB/test/usb_via_arduino/src/main.parallel.cpp

Line 1084 in 2de667e

uint8_t* data=(uint8_t*) malloc (max_packet_size);

data has size max_packet_size which is 64

But then in data_in_transfer, length is 255 and when being bzeroed it corrupts memory

MSX-USB/test/usb_via_arduino/src/main.parallel.cpp

Line 799 in 2de667e

bzero(result, length);

I fixed it by changing this line

MSX-USB/test/usb_via_arduino/src/main.parallel.cpp

Line 1084 in 2de667e

uint8_t* data=(uint8_t*) malloc (max_packet_size);

to uint8_t* data=(uint8_t*) malloc (255)

Tell me if you are experiencing the same and I will propose a pull request.

[cristianoag]

I believe that bus just a test program. You should focus on the drivers folder. Get Outlook for iOS<https://aka.ms/o0ukef>

…

________________________________ From: issalig ***@***.***> Sent: Wednesday, February 9, 2022 10:24:32 AM To: S0urceror/MSX-USB ***@***.***> Cc: Cristiano Goncalves ***@***.***>; Mention ***@***.***> Subject: Re: [S0urceror/MSX-USB] Installation adventures. (Issue #11) I found a bug while I was checking my ch376s with https://github.com/S0urceror/MSX-USB/blob/master/test/usb_via_arduino/src/main.parallel.cpp<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FS0urceror%2FMSX-USB%2Fblob%2Fmaster%2Ftest%2Fusb_via_arduino%2Fsrc%2Fmain.parallel.cpp&data=04%7C01%7C%7C9ae290889480474a132008d9ebcf8285%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637800098742589029%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=erjA0z7eaBr1c2lu3R1bMcsIR3nJT0ZWuWo%2By%2FiV7d4%3D&reserved=0> main_paralell crashed when printing iProd due to a small buffer https://github.com/S0urceror/MSX-USB/blob/2de667e102a2785ea67408f9beaa6fe030070103/test/usb_via_arduino/src/main.parallel.cpp#L1527<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FS0urceror%2FMSX-USB%2Fblob%2F2de667e102a2785ea67408f9beaa6fe030070103%2Ftest%2Fusb_via_arduino%2Fsrc%2Fmain.parallel.cpp%23L1527&data=04%7C01%7C%7C9ae290889480474a132008d9ebcf8285%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637800098742589029%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=PDxHPYlZcR8%2BleWyguBxu%2BtvYWEWs5VPucQD6oKm%2Bk8%3D&reserved=0> In get_string2 https://github.com/S0urceror/MSX-USB/blob/2de667e102a2785ea67408f9beaa6fe030070103/test/usb_via_arduino/src/main.parallel.cpp#L1084<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FS0urceror%2FMSX-USB%2Fblob%2F2de667e102a2785ea67408f9beaa6fe030070103%2Ftest%2Fusb_via_arduino%2Fsrc%2Fmain.parallel.cpp%23L1084&data=04%7C01%7C%7C9ae290889480474a132008d9ebcf8285%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637800098742589029%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=U99ODtETcrZ%2BXzLp2fk5ElHRE%2B2dGgnn%2FnWOyiRWw8o%3D&reserved=0> data has size max_packet_size which is 64 But then in https://github.com/S0urceror/MSX-USB/blob/master/test/usb_via_arduino/src/main.parallel.cpp#L799<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FS0urceror%2FMSX-USB%2Fblob%2Fmaster%2Ftest%2Fusb_via_arduino%2Fsrc%2Fmain.parallel.cpp%23L799&data=04%7C01%7C%7C9ae290889480474a132008d9ebcf8285%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637800098742589029%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=3cu%2F%2BuuVBFE94UdXzC4OD6IMKvXrb321X%2BalylmFr1Y%3D&reserved=0> length is 255 and when being bzeroed it corrupts memory. I fixed it by changing https://github.com/S0urceror/MSX-USB/blob/2de667e102a2785ea67408f9beaa6fe030070103/test/usb_via_arduino/src/main.parallel.cpp#L1084<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FS0urceror%2FMSX-USB%2Fblob%2F2de667e102a2785ea67408f9beaa6fe030070103%2Ftest%2Fusb_via_arduino%2Fsrc%2Fmain.parallel.cpp%23L1084&data=04%7C01%7C%7C9ae290889480474a132008d9ebcf8285%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637800098742589029%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=U99ODtETcrZ%2BXzLp2fk5ElHRE%2B2dGgnn%2FnWOyiRWw8o%3D&reserved=0> to uint8_t* data=(uint8_t*) malloc (255) Tell me if you are experiencing the same. — Reply to this email directly, view it on GitHub<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FS0urceror%2FMSX-USB%2Fissues%2F11%23issuecomment-1033758036&data=04%7C01%7C%7C9ae290889480474a132008d9ebcf8285%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637800098742589029%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=rUyctXB8KscAnrK8xXth0sCndpF%2F0UBOq6RcT3aLbl4%3D&reserved=0>, or unsubscribe<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALSAPKRR5AA2KL7PDUDIOSTU2JTJBANCNFSM5N3JYU6Q&data=04%7C01%7C%7C9ae290889480474a132008d9ebcf8285%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637800098742589029%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=BHSSfeW9vrqXfpQJK4NRskmkImOHqr6PWzZam3Oze5g%3D&reserved=0>. You are receiving this because you were mentioned.Message ID: ***@***.***>

[issalig]

Yes, it is only to check wiring and if ch376s is responding and to get properties from the devices.

[S0urceror]

I found a bug while I was checking my ch376s with https://github.com/S0urceror/MSX-USB/blob/master/test/usb_via_arduino/src/main.parallel.cpp

Good catch. Never happened here because the USB strings are usually not so long. I have now changed the setup package to request maximum 64 bytes as well.

And pushed the change back to GitHub.

[S0urceror]

Finally, I have everything working.
I started a document not to loose all the steps https://github.com/issalig/MSX-USB/blob/master/installation.md
It is still work in progress.

Nice write up. Would you be okay to merge all your updated instructions with the main once you consider them finished?

[issalig]

Yes of course, that's the idea. I will tell you when It is finished. Also i wrote info about programming the cpld and the flash so everything is documented.

[issalig]

I had some problems with openMSX, for some reason it did not communicate with arduino.
I compared sources of main.parallel which works correctly and ch376s.cc in openMSX and the main difference is that it does not call tcgetattr, thus I removed tcgetattr from ch376s.cc and now it works good for me.

Have you experienced this behaviour?

https://github.com/S0urceror/openMSX/blob/dd40674cb36805b95b444d464765004d234c7d4e/src/usb/ch376s.cc#L38

[S0urceror]

Hi Issalig, You can indeed safely remove this. Initially I retrieved the serial attributes, modified only the ones that need changing, and set them. Now that we have bzero afterwards all gets reset. No need to first retrieve the attributes. Regards, Mario From: issalig ***@***.***> Date: Thursday, 10 February 2022 at 16:00 To: S0urceror/MSX-USB ***@***.***> Cc: S0urceror ***@***.***>, Comment ***@***.***> Subject: Re: [S0urceror/MSX-USB] Installation adventures. (Issue #11) I had some problems with openMSX, for some reason it did not communicate with arduino. I compared sources of main.parallel which works correctly and ch376s.cc in openMSX and the main difference is that it does not call tcgetattr, thus I removed tcgetattr from ch376s.cc and now it works good for me. Have you experienced this behaviour? https://github.com/S0urceror/openMSX/blob/dd40674cb36805b95b444d464765004d234c7d4e/src/usb/ch376s.cc#L38 — Reply to this email directly, view it on GitHub<#11 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABGC6KBBBYJ6JAEGZKW3TCDU2PHJTANCNFSM5N3JYU6Q>. Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you commented.Message ID: ***@***.***>

[issalig]

I just write here as a reminder to change instructions (mainly paths and few other things) to make it work with recent @cristianoag linux support.