mirror of
				https://xff.cz/git/u-boot/
				synced 2025-10-31 18:35:42 +01:00 
			
		
		
		
	tee: optee: discover services dependent on tee-supplicant
Makes OP-TEE to enumerate also services depending on tee-supplicant support in U-Boot. This change allows OP-TEE services like fTPM TA to be discovered and get a TPM device registered in U-Boot. Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org> Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org> Signed-off-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
This commit is contained in:
		
				
					committed by
					
						 Ilias Apalodimas
						Ilias Apalodimas
					
				
			
			
				
	
			
			
			
						parent
						
							476a3d58df
						
					
				
				
					commit
					fe8a4ed011
				
			| @@ -102,13 +102,14 @@ static int bind_service_list(struct udevice *dev, struct tee_shm *service_list, | |||||||
| 	return 0; | 	return 0; | ||||||
| } | } | ||||||
|  |  | ||||||
| static int __enum_services(struct udevice *dev, struct tee_shm *shm, size_t *shm_size, u32 tee_sess) | static int __enum_services(struct udevice *dev, struct tee_shm *shm, size_t *shm_size, u32 tee_sess, | ||||||
|  | 			   unsigned int pta_cmd) | ||||||
| { | { | ||||||
| 	struct tee_invoke_arg arg = { }; | 	struct tee_invoke_arg arg = { }; | ||||||
| 	struct tee_param param = { }; | 	struct tee_param param = { }; | ||||||
| 	int ret = 0; | 	int ret = 0; | ||||||
|  |  | ||||||
| 	arg.func = PTA_CMD_GET_DEVICES; | 	arg.func = pta_cmd; | ||||||
| 	arg.session = tee_sess; | 	arg.session = tee_sess; | ||||||
|  |  | ||||||
| 	/* Fill invoke cmd params */ | 	/* Fill invoke cmd params */ | ||||||
| @@ -118,7 +119,7 @@ static int __enum_services(struct udevice *dev, struct tee_shm *shm, size_t *shm | |||||||
|  |  | ||||||
| 	ret = tee_invoke_func(dev, &arg, 1, ¶m); | 	ret = tee_invoke_func(dev, &arg, 1, ¶m); | ||||||
| 	if (ret || (arg.ret && arg.ret != TEE_ERROR_SHORT_BUFFER)) { | 	if (ret || (arg.ret && arg.ret != TEE_ERROR_SHORT_BUFFER)) { | ||||||
| 		dev_err(dev, "PTA_CMD_GET_DEVICES invoke function err: 0x%x\n", arg.ret); | 		dev_err(dev, "Enumeration command 0x%x failed: 0x%x\n", pta_cmd, arg.ret); | ||||||
| 		return -EINVAL; | 		return -EINVAL; | ||||||
| 	} | 	} | ||||||
|  |  | ||||||
| @@ -127,12 +128,13 @@ static int __enum_services(struct udevice *dev, struct tee_shm *shm, size_t *shm | |||||||
| 	return 0; | 	return 0; | ||||||
| } | } | ||||||
|  |  | ||||||
| static int enum_services(struct udevice *dev, struct tee_shm **shm, size_t *count, u32 tee_sess) | static int enum_services(struct udevice *dev, struct tee_shm **shm, size_t *count, u32 tee_sess, | ||||||
|  | 			 unsigned int pta_cmd) | ||||||
| { | { | ||||||
| 	size_t shm_size = 0; | 	size_t shm_size = 0; | ||||||
| 	int ret; | 	int ret; | ||||||
|  |  | ||||||
| 	ret = __enum_services(dev, NULL, &shm_size, tee_sess); | 	ret = __enum_services(dev, NULL, &shm_size, tee_sess, pta_cmd); | ||||||
| 	if (ret) | 	if (ret) | ||||||
| 		return ret; | 		return ret; | ||||||
|  |  | ||||||
| @@ -142,7 +144,7 @@ static int enum_services(struct udevice *dev, struct tee_shm **shm, size_t *coun | |||||||
| 		return ret; | 		return ret; | ||||||
| 	} | 	} | ||||||
|  |  | ||||||
| 	ret = __enum_services(dev, *shm, &shm_size, tee_sess); | 	ret = __enum_services(dev, *shm, &shm_size, tee_sess, pta_cmd); | ||||||
| 	if (!ret) | 	if (!ret) | ||||||
| 		*count = shm_size / sizeof(struct tee_optee_ta_uuid); | 		*count = shm_size / sizeof(struct tee_optee_ta_uuid); | ||||||
|  |  | ||||||
| @@ -174,20 +176,32 @@ static int bind_service_drivers(struct udevice *dev) | |||||||
| 	struct tee_shm *service_list = NULL; | 	struct tee_shm *service_list = NULL; | ||||||
| 	size_t service_count; | 	size_t service_count; | ||||||
| 	u32 tee_sess; | 	u32 tee_sess; | ||||||
| 	int ret; | 	int ret, ret2; | ||||||
|  |  | ||||||
| 	ret = open_enum_session(dev, &tee_sess); | 	ret = open_enum_session(dev, &tee_sess); | ||||||
| 	if (ret) | 	if (ret) | ||||||
| 		return ret; | 		return ret; | ||||||
|  |  | ||||||
| 	ret = enum_services(dev, &service_list, &service_count, tee_sess); | 	ret = enum_services(dev, &service_list, &service_count, tee_sess, | ||||||
|  | 			    PTA_CMD_GET_DEVICES); | ||||||
| 	if (!ret) | 	if (!ret) | ||||||
| 		ret = bind_service_list(dev, service_list, service_count); | 		ret = bind_service_list(dev, service_list, service_count); | ||||||
|  |  | ||||||
| 	tee_shm_free(service_list); | 	tee_shm_free(service_list); | ||||||
|  |  | ||||||
|  | 	ret2 = enum_services(dev, &service_list, &service_count, tee_sess, | ||||||
|  | 			     PTA_CMD_GET_DEVICES_SUPP); | ||||||
|  | 	if (!ret2) | ||||||
|  | 		ret2 = bind_service_list(dev, service_list, service_count); | ||||||
|  |  | ||||||
|  | 	tee_shm_free(service_list); | ||||||
|  |  | ||||||
| 	tee_close_session(dev, tee_sess); | 	tee_close_session(dev, tee_sess); | ||||||
|  |  | ||||||
| 	return ret; | 	if (ret) | ||||||
|  | 		return ret; | ||||||
|  |  | ||||||
|  | 	return ret2; | ||||||
| } | } | ||||||
|  |  | ||||||
| /** | /** | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user