Changeset View
Changeset View
Standalone View
Standalone View
sys/sys/efi.h
Show First 20 Lines • Show All 87 Lines • ▼ Show 20 Lines | |||||
#define EFI_MD_ATTR_NV 0x0000000000008000UL | #define EFI_MD_ATTR_NV 0x0000000000008000UL | ||||
#define EFI_MD_ATTR_MORE_RELIABLE \ | #define EFI_MD_ATTR_MORE_RELIABLE \ | ||||
0x0000000000010000UL | 0x0000000000010000UL | ||||
#define EFI_MD_ATTR_RO 0x0000000000020000UL | #define EFI_MD_ATTR_RO 0x0000000000020000UL | ||||
#define EFI_MD_ATTR_RT 0x8000000000000000UL | #define EFI_MD_ATTR_RT 0x8000000000000000UL | ||||
}; | }; | ||||
#define efi_next_descriptor(ptr, size) \ | #define efi_next_descriptor(ptr, size) \ | ||||
((struct efi_md *)(((uint8_t *)(ptr)) + (size))) | ((struct efi_md *)(void *)(((uint8_t *)(ptr)) + (size))) | ||||
cem: The second `struct efi_md *` cast is redundant here as well, unless something is directly… | |||||
Done Inline ActionsI think I'd rather leave it for now, quick grep shows nobody uses it like that, but it is a valid way to call it. trasz: I think I'd rather leave it for now, quick grep shows nobody uses it like that, but it is a… | |||||
struct efi_tm { | struct efi_tm { | ||||
uint16_t tm_year; /* 1998 - 20XX */ | uint16_t tm_year; /* 1998 - 20XX */ | ||||
uint8_t tm_mon; /* 1 - 12 */ | uint8_t tm_mon; /* 1 - 12 */ | ||||
uint8_t tm_mday; /* 1 - 31 */ | uint8_t tm_mday; /* 1 - 31 */ | ||||
uint8_t tm_hour; /* 0 - 23 */ | uint8_t tm_hour; /* 0 - 23 */ | ||||
uint8_t tm_min; /* 0 - 59 */ | uint8_t tm_min; /* 0 - 59 */ | ||||
uint8_t tm_sec; /* 0 - 59 */ | uint8_t tm_sec; /* 0 - 59 */ | ||||
▲ Show 20 Lines • Show All 94 Lines • Show Last 20 Lines |
The second struct efi_md * cast is redundant here as well, unless something is directly acting on this macro like: efi_next_descriptor(...)->md_phys. Which, I hope not.