From b068cc58d9fe95006cd78a68a8b26b8fcd5847b9 Mon Sep 17 00:00:00 2001 From: Bjorn Andersson Date: Fri, 4 Oct 2024 08:02:28 -0700 Subject: [PATCH] firehose: Allow bConfigDescrLock to be passed as 1 During the introduction of UFS provisioning, the author left behind an additional safety measure of forcing the user to explicitly modify the source and recompile QDL in order to be able to lock the provisioning, to avoid users accidentally doing so. While it does reduce the risk of irreversable mistakes, it's not suitable when QDL is distributed as a binary - and the user already need to set bConfigDescrLock both in provided XML files and on command line. So, let's remove the safety net... Signed-off-by: Bjorn Andersson --- firehose.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/firehose.c b/firehose.c index be43839..f69943d 100644 --- a/firehose.c +++ b/firehose.c @@ -613,7 +613,7 @@ int firehose_apply_ufs_common(struct qdl_device *qdl, struct ufs_common *ufs) xml_setpropf(node_to_send, "bSecureRemovalType", "%d", ufs->bSecureRemovalType); xml_setpropf(node_to_send, "bInitActiveICCLevel", "%d", ufs->bInitActiveICCLevel); xml_setpropf(node_to_send, "wPeriodicRTCUpdate", "%d", ufs->wPeriodicRTCUpdate); - xml_setpropf(node_to_send, "bConfigDescrLock", "%d", 0/*ufs->bConfigDescrLock*/); //Safety, remove before fly + xml_setpropf(node_to_send, "bConfigDescrLock", "%d", ufs->bConfigDescrLock); ret = firehose_send_single_tag(qdl, node_to_send); if (ret)