xen-troops / meta-xt-prod-devel-rpi5

Apache License 2.0
3 stars 5 forks source link

RPI5: add scmi and wifi support #41

Closed GrygiriiS closed 1 month ago

GrygiriiS commented 2 months ago

Both ENABLE_SCMI and ENABLE_WIFI are disabled by default.

NOTE1. if ENABLE_SCMI is enabled it will change SRAM mapping for Zephyr Dom0 application, so to run, the rpi_5.overlay has to be edited to correct "sram0: memory@10000000" node with proper values.

GrygiriiS commented 2 months ago

@oleksiimoisieiev @lorc @rshym

rshym commented 2 months ago

Acked-by: Ruslan Shymkevych <ruslan_shymkevych@epam.com>

GrygiriiS commented 2 months ago

This PR updated:

Test branch to be used for integration can be found here: https://github.com/GrygiriiS/meta-xt-prod-devel-rpi5/tree/rpi5_scmi_wifi_v1_dev

oleksiimoisieiev commented 2 months ago

I think README should be updated as well

rshym commented 1 month ago

After the repush, the first commit message still mentions AUTOREV. The second commit has 'No end-of-the-file" (Red crossed circle).

oleksiimoisieiev commented 1 month ago

This approach to add all scmi related stuff in meta-xt-prod-devel looks as the only acceptable solution. That's because there is a significant difference in Zephyr (Dom0) memory mapping with/w-o SCMI. So we can't make it completely optional (controlled by config param) without making 2 zephyr branches or using special snippets. In any case - every configuration will have it's specific memory-mapping. So when all configuration is on the top layer - it can be used as the reference for making project-specific top meta-layer with or without SCMI enabled. This approach looks good so Reviewed-by: Oleksii Moisieiev <oleksii_moisieiev@epam.com> with a small comments.

lorc commented 1 month ago

Thanks

Acked-by: Volodymyr Babchuk <volodymyr_babchuk@epam.com>