diff options
| author | Michal Simek <michal.simek@xilinx.com> | 2020-11-06 13:53:01 +0100 |
|---|---|---|
| committer | Michal Simek <michal.simek@xilinx.com> | 2020-11-20 10:42:54 +0100 |
| commit | 5fb093f471d2ded8f8ad58a85f247b84631ca8dc (patch) | |
| tree | f3cfe7f8b9a264a30350b4c5496a25765f44e434 | |
| parent | 072dbc7ac551ff4e316ac66988a8b915f48cc06f (diff) | |
| download | u-boot-5fb093f471d2ded8f8ad58a85f247b84631ca8dc.tar.gz | |
fru: common: Switch capture variable with the rest
capture variable is bool which is just one byte and it is just causing
unaligned accesses. Better to have it as last entry in the structure.
It also simplify offset calculation for initial header copy.
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
| -rw-r--r-- | board/xilinx/common/fru.h | 2 | ||||
| -rw-r--r-- | board/xilinx/common/fru_ops.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/board/xilinx/common/fru.h b/board/xilinx/common/fru.h index a3e6520257..e85dde4531 100644 --- a/board/xilinx/common/fru.h +++ b/board/xilinx/common/fru.h @@ -53,9 +53,9 @@ struct fru_board_data { }; struct fru_table { - bool captured; struct fru_common_hdr hdr; struct fru_board_data brd; + bool captured; }; #define FRU_TYPELEN_CODE_MASK 0xC0 diff --git a/board/xilinx/common/fru_ops.c b/board/xilinx/common/fru_ops.c index fc3add7d93..affcb121aa 100644 --- a/board/xilinx/common/fru_ops.c +++ b/board/xilinx/common/fru_ops.c @@ -217,7 +217,7 @@ int fru_capture(unsigned long addr) hdr = (struct fru_common_hdr *)addr; - memcpy((void *)&fru_data.hdr, (void *)hdr, + memcpy((void *)&fru_data, (void *)hdr, sizeof(struct fru_common_hdr)); fru_data.captured = true; |