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

[WIP] Software fixes for Darjeeling DV #26504

Draft
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

pamaury
Copy link
Contributor

@pamaury pamaury commented Mar 4, 2025

This PR recreates the SW fixes identified by @alees24 in #26088 in a way that is compatible with all tops. Not everything has been converted yet.

@pamaury pamaury requested a review from andreaskurth March 4, 2025 16:56
@pamaury pamaury force-pushed the dj_sw_fixes branch 2 times, most recently from 45b349b to 9491808 Compare March 5, 2025 10:18
pamaury added 6 commits March 5, 2025 14:23
Compile all DJ tests, not just the UART smoketest.

Signed-off-by: Amaury Pouly <[email protected]>
The template was still using earlgrey types. This requires to pass
top information to the generator and templates.

Signed-off-by: Amaury Pouly <[email protected]>
On Darjeeling, the MMIO is not open by default, therefore the ePMP
needs to be configured prior to doing any register access.

Signed-off-by: Amaury Pouly <[email protected]>
@pamaury pamaury force-pushed the dj_sw_fixes branch 2 times, most recently from 52d57f5 to d5bb8e8 Compare March 6, 2025 15:28
alees24 and others added 7 commits March 6, 2025 18:59
The RAM was temporarily put at the beginning of the CTN region
to workaround the software using the wrong linking address.

Signed-off-by: Amaury Pouly <[email protected]>
The test code is stored in a RAM inside the CTN space at a fixed
offset.

Signed-off-by: Amaury Pouly <[email protected]>
Instead of duplicating the logic to find the manifest, rely on the
fact that the OTTF already needs to know the address to link.
Therefore we can reuse the OTTF top config to create a symbol for
the manifest address.

Signed-off-by: Amaury Pouly <[email protected]>
This higher-level function provides a convenient way to connect a
peripheral I/O to a pad without needing to know if it's an MIO/DIO:
it will do the natural thing in all cases. This is useful for multitop
because Earlgrey and Darjeeling have different kinds of pads, e.g.
the GPIOs are MIOs on EG but DIOs on DJ.

Signed-off-by: Amaury Pouly <[email protected]>
Ideally part of this code should be moved to a top-specific
library but in the mean time, provide a common solution for
EG and DJ.

Signed-off-by: Amaury Pouly <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants