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

@FIR-433: Made changes to remove peripheral accesses. #6

Merged
merged 1 commit into from
Dec 23, 2024

Conversation

atrivedi-tsavoritesi
Copy link
Collaborator

The G0208 SKYP branch has peripheral accesses of UART and SPI disabled Removing the peripheral block from the DTS file

The G0208 SKYP branch has peripheral accesses of UART and SPI disabled
Removing the peripheral block from the DTS file
@atrivedi-tsavoritesi atrivedi-tsavoritesi merged commit 582e2b7 into main Dec 23, 2024
6 of 9 checks passed
@atrivedi-tsavoritesi atrivedi-tsavoritesi deleted the FIR-433 branch December 23, 2024 20:32
@@ -89,15 +89,15 @@
zephyr,memory-region = "SRAM1";
};

soc {
/*soc {
peripheral@71000000 {
#address-cells = <1>;
#size-cells = <1>;
ranges = <0x0 0x71000000 0x1effffff>;

#include "tsi_skyp_m85-common.dtsi"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this dtsi file be included (moved out of this block)? Your comment block hides this currently.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This inclusion of the dtsi file was only for this soc block. Not including it does not have major impact till we enable another IO.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok. I am trying out these changes in my view. I will update you either way.

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.

4 participants