Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issues with BCM serial driver #171

Open
Ivan-Velickovic opened this issue Oct 18, 2023 · 2 comments
Open

Issues with BCM serial driver #171

Ivan-Velickovic opened this issue Oct 18, 2023 · 2 comments
Labels
CI related to continuous integration and testing help wanted hw-test sel4test hardware builds + runs

Comments

@Ivan-Velickovic
Copy link
Contributor

A while ago I tried to get the RPi4 into the seL4 CI due to how fragile the platform is. After some fixes/work I thought it was ready to go but then it got blocked on not working in release mode (seL4/ci-actions#278).

seL4test would just hang in release mode. What seems to be causing this is the BCM serial driver as applying this patch fixes everything:

diff --git a/libplatsupport/src/mach/bcm/serial.c b/libplatsupport/src/mach/bcm/serial.c
index 0091c15..ec77a7d 100644
--- a/libplatsupport/src/mach/bcm/serial.c
+++ b/libplatsupport/src/mach/bcm/serial.c
@@ -185,7 +185,7 @@ int uart_init(const struct dev_defn *defn, const ps_io_ops_t *ops, ps_chardevice
 
     }
 
-    uart_gpio_configure(defn->id, ops);
+    // uart_gpio_configure(defn->id, ops);
 
     uart_funcs.uart_init(defn, ops, dev);

I don't know anything about GPIO and don't really have the chance to understand it right now so I don't know why this works. In fact, I don't even remember why I tried commenting out this.

Debug mode of sel4test does not make use of the actual serial driver (which seems weird in my opinion now that I think about it) which is why this issue only came up in release mode.

If someone more familiar with GPIO could take a look at this that would be great.

@axel-h
Copy link
Member

axel-h commented Nov 14, 2023

I wonder, should the seL4 release 13.0 (https://github.com/orgs/seL4/projects/2) include proper RPi4 support also?

@Ivan-Velickovic
Copy link
Contributor Author

If someone volunteers to solve the issue, than maybe it could be included yes.

@axel-h axel-h added help wanted hw-test sel4test hardware builds + runs CI related to continuous integration and testing labels Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI related to continuous integration and testing help wanted hw-test sel4test hardware builds + runs
Projects
None yet
Development

No branches or pull requests

2 participants