From 67d50156170b17e5bca460ab6e5648e2b11f061c Mon Sep 17 00:00:00 2001 From: Edward O'Callaghan Date: Wed, 7 Sep 2022 22:21:39 +1000 Subject: drivers/: Make 'fallback_{un}map' the default unless defined Drop the explicit need to specify the default 'fallback_{un}map' callback function pointer from the 'programmer_entry' struct. This is a reasonable default for every other driver in the tree with only a select few exceptions [atavia, serprog, dummyflasher and internal]. Thus this simplifies driver development and paves way to remove the 'programmer' global handle. Change-Id: I5ea7bd68f7ae2cd4af9902ef07255ab6ce0bfdb3 Signed-off-by: Edward O'Callaghan Reviewed-on: https://review.coreboot.org/c/flashrom/+/67404 Tested-by: build bot (Jenkins) Reviewed-by: Felix Singer Reviewed-by: Anastasia Klimchuk Reviewed-by: Angel Pons --- serprog.c | 1 - 1 file changed, 1 deletion(-) (limited to 'serprog.c') diff --git a/serprog.c b/serprog.c index 57196a98..12d15d74 100644 --- a/serprog.c +++ b/serprog.c @@ -967,6 +967,5 @@ const struct programmer_entry programmer_serprog = { .devs.note = "All programmer devices speaking the serprog protocol\n", .init = serprog_init, .map_flash_region = serprog_map, - .unmap_flash_region = fallback_unmap, .delay = serprog_delay, }; -- cgit v1.2.3