Skip to content

Align IOVIRT_BLOCK to the correct length #7

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.

Sign up for GitHub

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

Closed
ownia wants to merge 1 commit into from
Closed

Align IOVIRT_BLOCK to the correct length #7

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion pal/baremetal/base/include/pal_common_support.h
View file
Open in desktop
Copy link
Contributor

@chetan-rathore chetan-rathore Jun 25, 2025

Choose a reason for hiding this comment

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

Hi @ownia,

The PR description correctly mentions to make all IOVIRT_LENGTH to 150, but the changes in PR is changing them to 256.
We are internally raising PR to fix length in one file. pal/uefi_dt/include/pal_uefi.h

Copy link
Author

@ownia ownia Jun 25, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

We are internally raising PR to fix length in one file. pal/uefi_dt/include/pal_uefi.h

good! :)

Original file line number Diff line number Diff line change
Expand Up @@ -598,7 +598,7 @@ typedef union {
ID_MAP map;
}NODE_DATA_MAP;

#define MAX_NAMED_COMP_LENGTH 150
#define MAX_NAMED_COMP_LENGTH 256

typedef struct {
uint64_t smmu_base; /* SMMU base to which component is attached, else NULL */
Expand Down
2 changes: 1 addition & 1 deletion pal/baremetal/target/RDN2/include/platform_override_struct.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ typedef struct {
uint64_t smmu_base;
} PLATFORM_OVERRIDE_IOVIRT_PMCG_INFO_BLOCK;

#define MAX_NAMED_COMP_LENGTH 150
#define MAX_NAMED_COMP_LENGTH 256
typedef struct {
uint32_t its_count;
uint32_t identifiers[1]; /* GIC ITS identifier arrary */
Expand Down
2 changes: 1 addition & 1 deletion pal/uefi_acpi/include/pal_uefi.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -334,7 +334,7 @@ typedef union {
ID_MAP map;
}NODE_DATA_MAP;

#define MAX_NAMED_COMP_LENGTH 150
#define MAX_NAMED_COMP_LENGTH 256

typedef struct {
UINT64 smmu_base; /* SMMU base to which component is attached, else NULL */
Expand Down
2 changes: 1 addition & 1 deletion val/include/pal_interface.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
#include <linux/slab.h>
#endif

#define MAX_NAMED_COMP_LENGTH 150
#define MAX_NAMED_COMP_LENGTH 256

#ifdef TARGET_BM_BOOT

Expand Down