Changeset View
Changeset View
Standalone View
Standalone View
sys/arm64/include/reg.h
Show First 20 Lines • Show All 54 Lines • ▼ Show 20 Lines | struct fpreg { | ||||
uint32_t fp_cr; | uint32_t fp_cr; | ||||
}; | }; | ||||
struct fpreg32 { | struct fpreg32 { | ||||
int dummy; | int dummy; | ||||
}; | }; | ||||
struct dbreg { | struct dbreg { | ||||
uint32_t db_info; | uint8_t db_debug_ver; | ||||
uint32_t db_pad; | uint8_t db_nbkpts; | ||||
uint8_t db_nwtpts; | |||||
uint8_t db_pad[5]; | |||||
struct { | struct { | ||||
uint64_t dbr_addr; | uint64_t dbr_addr; | ||||
markj: Is there any reason not to provide these as separate fields? If so, perhaps it would be worth… | |||||
uint32_t dbr_ctrl; | uint32_t dbr_ctrl; | ||||
uint32_t dbr_pad; | uint32_t dbr_pad; | ||||
} db_regs[16]; | } db_breakregs[16]; | ||||
struct { | |||||
uint64_t dbw_addr; | |||||
uint32_t dbw_ctrl; | |||||
uint32_t dbw_pad; | |||||
} db_watchregs[16]; | |||||
Not Done Inline ActionsThis looks like it should be called db_breakregs - is it a problem to rename since we're already adding fields and changing the value of db_info? markj: This looks like it should be called db_breakregs - is it a problem to rename since we're… | |||||
}; | }; | ||||
struct dbreg32 { | struct dbreg32 { | ||||
int dummy; | int dummy; | ||||
}; | }; | ||||
#define __HAVE_REG32 | #define __HAVE_REG32 | ||||
Show All 21 Lines |
Is there any reason not to provide these as separate fields? If so, perhaps it would be worth providing accessors?