mirror of
				https://review.coreboot.org/flashrom.git
				synced 2025-10-31 05:10:41 +01:00 
			
		
		
		
	tests: add init_shutdown test for realtek_mst_i2c_spi
This can catch regressions like the earlier one in this programmer that caused initialization to always fail. Requires support for mocking POSIX file I/O functions because the programmer does ioctls on the opened file. TEST=ninja test Signed-off-by: Peter Marheine <pmarheine@chromium.org> Change-Id: I5a5c617d1ec35d2a3bbe622e5add82a65eb396f0 Reviewed-on: https://review.coreboot.org/c/flashrom/+/56911 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Anastasia Klimchuk <aklm@chromium.org> Reviewed-by: Edward O'Callaghan <quasisec@chromium.org>
This commit is contained in:
		 Peter Marheine
					Peter Marheine
				
			
				
					committed by
					
						 Edward O'Callaghan
						Edward O'Callaghan
					
				
			
			
				
	
			
			
			 Edward O'Callaghan
						Edward O'Callaghan
					
				
			
						parent
						
							b261bec1ee
						
					
				
				
					commit
					47f4c18260
				
			| @@ -209,3 +209,57 @@ void linux_spi_init_and_shutdown_test_success(void **state) | ||||
| 	skip(); | ||||
| #endif | ||||
| } | ||||
|  | ||||
| #define REALTEK_MST_MOCK_FD 0x10ec | ||||
|  | ||||
| static int realtek_mst_open(void *state, const char *pathname, int flags) | ||||
| { | ||||
| 	assert_string_equal(pathname, "/dev/i2c-254"); | ||||
| 	assert_int_equal(flags & O_RDWR, O_RDWR); | ||||
| 	return REALTEK_MST_MOCK_FD; | ||||
| } | ||||
|  | ||||
| static int realtek_mst_ioctl(void *state, int fd, unsigned long request, va_list args) | ||||
| { | ||||
| 	assert_int_equal(fd, REALTEK_MST_MOCK_FD); | ||||
| 	assert_int_equal(request, I2C_SLAVE); | ||||
| 	/* Only access to I2C address 0x4a is expected */ | ||||
| 	unsigned long addr = va_arg(args, unsigned long); | ||||
| 	assert_int_equal(addr, 0x4a); | ||||
|  | ||||
| 	return 0; | ||||
| } | ||||
|  | ||||
| static int realtek_mst_read(void *state, int fd, void *buf, size_t sz) | ||||
| { | ||||
| 	assert_int_equal(fd, REALTEK_MST_MOCK_FD); | ||||
| 	assert_int_equal(sz, 1); | ||||
| 	return sz; | ||||
| } | ||||
|  | ||||
| static int realtek_mst_write(void *state, int fd, const void *buf, size_t sz) | ||||
| { | ||||
| 	assert_int_equal(fd, REALTEK_MST_MOCK_FD); | ||||
| 	const LargestIntegralType accepted_sizes[] = {1, 2}; | ||||
| 	assert_in_set(sz, accepted_sizes, ARRAY_SIZE(accepted_sizes)); | ||||
| 	return sz; | ||||
| } | ||||
|  | ||||
| void realtek_mst_init_and_shutdown_test_success(void **state) | ||||
| { | ||||
| #if CONFIG_REALTEK_MST_I2C_SPI == 1 | ||||
| 	const struct io_mock realtek_mst_io = { | ||||
| 		.open = realtek_mst_open, | ||||
| 		.ioctl = realtek_mst_ioctl, | ||||
| 		.read = realtek_mst_read, | ||||
| 		.write = realtek_mst_write, | ||||
| 	}; | ||||
| 	io_mock_register(&realtek_mst_io); | ||||
|  | ||||
| 	run_lifecycle(state, &programmer_realtek_mst_i2c_spi, "bus=254,enter-isp=0"); | ||||
|  | ||||
| 	io_mock_register(NULL); | ||||
| #else | ||||
| 	skip(); | ||||
| #endif /* CONFIG_REALTEK_I2C_SPI */ | ||||
| } | ||||
|   | ||||
| @@ -42,6 +42,14 @@ struct pci_dev { | ||||
| 	unsigned int device_id; | ||||
| }; | ||||
|  | ||||
| /* POSIX open() flags, avoiding dependency on fcntl.h */ | ||||
| #define O_RDONLY 0 | ||||
| #define O_WRONLY 1 | ||||
| #define O_RDWR 2 | ||||
|  | ||||
| /* Linux I2C interface constants, avoiding linux/i2c-dev.h */ | ||||
| #define I2C_SLAVE 0x0703 | ||||
|  | ||||
| struct io_mock { | ||||
| 	void *state; | ||||
|  | ||||
| @@ -65,6 +73,12 @@ struct io_mock { | ||||
| 					unsigned char *data, | ||||
| 					uint16_t wLength, | ||||
| 					unsigned int timeout); | ||||
|  | ||||
| 	/* POSIX File I/O */ | ||||
| 	int (*open)(void *state, const char *pathname, int flags); | ||||
| 	int (*ioctl)(void *state, int fd, unsigned long request, va_list args); | ||||
| 	int (*read)(void *state, int fd, void *buf, size_t sz); | ||||
| 	int (*write)(void *state, int fd, const void *buf, size_t sz); | ||||
| }; | ||||
|  | ||||
| void io_mock_register(const struct io_mock *io); | ||||
|   | ||||
| @@ -34,6 +34,8 @@ mocks = [ | ||||
|   '-Wl,--wrap=open', | ||||
|   '-Wl,--wrap=open64', | ||||
|   '-Wl,--wrap=ioctl', | ||||
|   '-Wl,--wrap=read', | ||||
|   '-Wl,--wrap=write', | ||||
|   '-Wl,--wrap=fopen', | ||||
|   '-Wl,--wrap=fopen64', | ||||
|   '-Wl,--wrap=rget_io_perms', | ||||
|   | ||||
| @@ -83,21 +83,49 @@ uint8_t __wrap_sio_read(uint16_t port, uint8_t reg) | ||||
| int __wrap_open(const char *pathname, int flags) | ||||
| { | ||||
| 	LOG_ME; | ||||
| 	if (current_io && current_io->open) | ||||
| 		return current_io->open(current_io->state, pathname, flags); | ||||
| 	return MOCK_HANDLE; | ||||
| } | ||||
|  | ||||
| int __wrap_open64(const char *pathname, int flags) | ||||
| { | ||||
| 	LOG_ME; | ||||
| 	if (current_io && current_io->open) | ||||
| 		return current_io->open(current_io->state, pathname, flags); | ||||
| 	return MOCK_HANDLE; | ||||
| } | ||||
|  | ||||
| int __wrap_ioctl(int fd, unsigned long int request, ...) | ||||
| { | ||||
| 	LOG_ME; | ||||
| 	if (current_io && current_io->ioctl) { | ||||
| 		va_list args; | ||||
| 		int out; | ||||
| 		va_start(args, request); | ||||
| 		out = current_io->ioctl(current_io->state, fd, request, args); | ||||
| 		va_end(args); | ||||
| 		return out; | ||||
| 	} | ||||
| 	return MOCK_HANDLE; | ||||
| } | ||||
|  | ||||
| int __wrap_write(int fd, const void *buf, size_t sz) | ||||
| { | ||||
| 	LOG_ME; | ||||
| 	if (current_io && current_io->write) | ||||
| 		return current_io->write(current_io->state, fd, buf, sz); | ||||
| 	return sz; | ||||
| } | ||||
|  | ||||
| int __wrap_read(int fd, void *buf, size_t sz) | ||||
| { | ||||
| 	LOG_ME; | ||||
| 	if (current_io && current_io->read) | ||||
| 		return current_io->read(current_io->state, fd, buf, sz); | ||||
| 	return sz; | ||||
| } | ||||
|  | ||||
| FILE *__wrap_fopen(const char *pathname, const char *mode) | ||||
| { | ||||
| 	LOG_ME; | ||||
| @@ -250,6 +278,7 @@ int main(void) | ||||
| 		cmocka_unit_test(dediprog_init_and_shutdown_test_success), | ||||
| 		cmocka_unit_test(ene_lpc_init_and_shutdown_test_success), | ||||
| 		cmocka_unit_test(linux_spi_init_and_shutdown_test_success), | ||||
| 		cmocka_unit_test(realtek_mst_init_and_shutdown_test_success), | ||||
| 	}; | ||||
| 	ret |= cmocka_run_group_tests_name("init_shutdown.c tests", init_shutdown_tests, NULL, NULL); | ||||
|  | ||||
|   | ||||
| @@ -47,6 +47,7 @@ void nicrealtek_init_and_shutdown_test_success(void **state); | ||||
| void dediprog_init_and_shutdown_test_success(void **state); | ||||
| void ene_lpc_init_and_shutdown_test_success(void **state); | ||||
| void linux_spi_init_and_shutdown_test_success(void **state); | ||||
| void realtek_mst_init_and_shutdown_test_success(void **state); | ||||
|  | ||||
| /* layout.c */ | ||||
| void included_regions_dont_overlap_test_success(void **state); | ||||
|   | ||||
		Reference in New Issue
	
	Block a user