11 comments:
File tests/include/lib/fmap/fmap_config.h:
Always put a license line, even on generated files, just to be safe.
Done
Patch Set #2, Line 7: * FMAP_OFFSET was changed to zero for testing purposes.
Doesn't look like it's zero below?
Done
Patch Set #2, Line 84: //#define FMAP_SECTION_UNUSED_HOLE_START 0xfff000
??
Done
File tests/include/lib/fmap/fmap_data.h:
Patch Set #2, Line 3: #ifndef TESTS_INCLUDE_LIB_FMAP_FMAP_DATA_H_
I don't think you should use tests/include/lib for this, because then all the headers in there clash […]
Done
nit: just use sizeof()?
Done
Patch Set #2, Line 18: static void *local_mmap(const struct region_device *rdev, size_t offset, size_t size)
I think you could just use a mem_region_device instead of all of this?
Done
Patch Set #2, Line 38: size = MIN(size, flash_buffer_size - offset);
Doesn't the check above make this unnecessary?
Done
Patch Set #2, Line 84: rdev_root = malloc(sizeof(rdev_new_root));
I think you can just write […]
Done
Patch Set #2, Line 146: assert_int_not_equal(-1, fmap_locate_area_as_rdev("RO_VPD", rdev));
Please test for actual offsets, not just != -1.
Done
Test the returned rdev, too? Make sure you cannot rdev_writeat() the RO version?
Done
Patch Set #2, Line 261: FMAP_SECTION_FW_MAIN_A_SIZE
Maybe make it smaller so you can also test that the remaining part of the region gets zeroed on over […]
Done
To view, visit change 48557. To unsubscribe, or for help writing mail filters, visit settings.