[coreboot-gerrit] Change in coreboot[master]: cbfstool: Fix offset calculation for aligned files

[Tim Wawrzynczak (Code Review)]

Attention is currently required from: Julius Werner, Tim Wawrzynczak.
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59877 )

Change subject: cbfstool: Fix offset calculation for aligned files
......................................................................

Patch Set 2: Code-Review+1

(1 comment)

File util/cbfstool/cbfstool.c:

https://review.coreboot.org/c/coreboot/+/59877/comment/49cddeaa_9e0c39c3 
PS2, Line 1085: *offset;
> I don't really understand this x86-specific address space stuff well, so I hope someone can correct  […]

As far as I understand too, there are only the 2 address spaces at play here, host and flash and basically yeah the host address space is for convenience when dealing with addresses where what matters in the end is the resulting memory-mapped address on x86.

and I agree that this looks like it is preserving existing behavior.