[flashrom-gerrit] Change in flashrom[master]: hwaccess: replace macros by C code

[Thomas Heijligen (Code Review)]

Attention is currently required from: Felix Singer, Nico Huber, Edward O'Callaghan, Angel Pons, Nikolai Artemiev, Anastasia Klimchuk.
Thomas Heijligen has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/62898 )

Change subject: hwaccess: replace macros by C code
......................................................................

Patch Set 6:

(5 comments)

Commit Message:

https://review.coreboot.org/c/flashrom/+/62898/comment/48cfa498_d73abedf 
PS5, Line 9: seperate
> sep*a*rate

Done

https://review.coreboot.org/c/flashrom/+/62898/comment/f8770b71_d8756fcc 
PS5, Line 11: endiannes
> one more `s`

Done

https://review.coreboot.org/c/flashrom/+/62898/comment/1cc594c3_455d5d9f 
PS5, Line 12: swaped
> another `p`, i.e. `swapped` also a lot in the code.

Done

Patchset:

PS6: 
Patch set 6: mostly cosmetic fixes.

File hwaccess.h:

https://review.coreboot.org/c/flashrom/+/62898/comment/f602b9ed_d4215d17 
PS5, Line 30: 	return ((value & (uint16_t)0x00ffU) << 8) |
> Maybe add another space after `return` for alignment?

Done, corrected also the code of the function. This was the code from swap16, not swap8