coja/lbmk
1
0
Fork 0
mirror of https://codeberg.org/libreboot/lbmk.git synced 2026-07-12 22:42:54 +02:00
Code Issues Packages Projects Releases Wiki Activity

util/nvmutil: make word/setWord a function

Browse Source 
having this as a macro makes the code quite brittle.

better to have it as a function.

Signed-off-by: Leah Rowe <leah@libreboot.org>
This commit is contained in:
Leah Rowe
2026-03-02 17:33:34 +00:00
parent 8707695184
commit 4e7d48b5c5
1 changed files with 17 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
util/nvmutil/nvmutil.c
+17 -5
View File
@@ -20,9 +20,10 @@ void cmd_setchecksum(void), cmd_brick(void), swap(int partnum), writeGbe(void),
parseMacString(const char *strMac, uint16_t *mac), cmd_swap(void),
openFiles(const char *path), cmd_copy(void), writeGbe_part(int),
readGbe_part(int), usage(char*), set_io_flags(int, char **),
set_cmd(int, char **);
set_cmd(int, char **), setWord(int, int, uint16_t);
int goodChecksum(int partnum);
uint8_t hextonum(char chs), rhex(void);
uint16_t word(int, int);
#ifdef __OpenBSD__
void block_unveil(void);
@@ -72,10 +73,6 @@ void (*cmd)(void) = NULL;
err(SET_ERR(ECANCELED), "%s", l); \
if (fstat(f, &st) == -1) err(SET_ERR(ECANCELED), "%s", l)
#define word(pos16, partnum) ((uint16_t *) gbe[partnum])[pos16]
#define setWord(pos16, p, val16) if (word(pos16, p) != val16) \
nvmPartChanged[p] = 1 | (word(pos16, p) = val16)
int
main(int argc, char *argv[])
{
@@ -405,6 +402,21 @@ goodChecksum(int partnum)
return 0;
}
uint16_t
word(int pos16, int p)
{
return ((uint16_t *) gbe[p])[pos16];
}
void
setWord(int pos16, int p, uint16_t val16)
{
if (((uint16_t *) gbe[p])[pos16] != val16) {
nvmPartChanged[p] = 1;
((uint16_t *) gbe[p])[pos16] = val16;
}
}
void
writeGbe(void)
{