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

Couldn't Build HSS for Debugging ( Reason could be recent changes in BootProcess - by hugh breslin) #68

Open
1 of 5 tasks
krishnaraj-Murugaiyan opened this issue Jun 11, 2024 · 0 comments

Comments

@krishnaraj-Murugaiyan
Copy link

krishnaraj-Murugaiyan commented Jun 11, 2024

Microchip PolarFire SoC Issue Template

Checklist

Please complete this checklist before filing an issue:

  • Are you using the latest releases of our deliverables?
    • If not please update to the latest version of the each deliverable needed (reference design, HSS, Linux / bare metal example) and re-test as this issue may already have been resolved.
  • Is this something you can debug and fix?
    • Send a pull request! Bug fixes and documentation fixes are welcome.
  • Have a usage question?
    • Ask your question on our discussion forum. We use the discussion forum for usage questions.
  • Have an idea for a feature?
    • Post the feature request on the discussion forum. This will allow us to assess and plan for the feature without having an open issue until it is released. GitHub issues should only be used for code bugs.
    • Is this issue directly related to Microchip code? I.e have you found an upstream bug?
      • If so please open an issue against the upstream repository and not the Microchip fork.

None of the above, create an issue

Please make sure to add all the information needed to understand the bug so that someone can help. If the info is missing we'll add the 'Needs more information' label and close the issue until there is enough information.

  • [ x ] Provide a minimal code snippet example that reproduces the bug where appropriate.

Refer: https://github.com/orgs/polarfire-soc/discussions/100#discussion-4167312
To Reproduce : pick latest hss and build as per above mentioned link.

  • With CONFIG_SERVICE_BOOT=n I get
    D:\PFSOC\WORKSPACE\hart-software-services/init/hss_boot_init.c:421: undefined reference to 'HSS_Register_Boot_Image'

it seems I have to disable CONFIG_SERVICE_MMC too if i do that

 CC        build/init/hss_boot_init.o
init/hss_boot_init.c:159:6: error: #error Unable to determine boot mechanism
      error Unable to determine boot mechanism
      ^~~~~
init/hss_boot_init.c: In function 'HSS_BootListStorageProviders':
init/hss_boot_init.c:176:28: error: comparison is always false due to limited range of data type [-Werror=type-limits]
     for (uint32_t i = 0; i < ARRAY_SIZE(pStorages); i++) {
                            ^
cc1.exe: all warnings being treated as errors
make: *** [application/rules.mk:184: build/init/hss_boot_init.o] Error 1
"make all" terminated with exit code 2. Build might be incomplete.

Since i don't need this part of the code i commented these line

Now,

<artificial>:(.rodata.globalInitFunctions+0x38): undefined reference to 'HSS_PMP_Init'

in hss_registry.c HSS_PMP_Init is referenced without checking IS_ENABLED(IS_ENABLED(CONFIG_SERVICE_BOOT))

With these changes I am able to build and debug the HSS, But I am not sure what are the other parts are actually affected.
So it needs a further lookup and proper fix.

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

No branches or pull requests

1 participant