tegra-vic: Make pdata->falcon_data NULL.

In case firmware load failed 15340000.vic
for whatever reason, we need to reassign
pdata->falcon_data NULL else based on check
if (pdata->falcon_data), non NULL , code will
assume that firmware is loaded properly last time
and will go ahead submitting the job, which may result
in SMMU-fault. We need that when it tries next time
it sees pdata->falcon_data is NULL and does allocation
and firmware loading before job submission,

Bug 200764796

Change-Id: Ia949ceb19b6b07db94b8a310a6c00363f8e58a6d
Signed-off-by: Achal Verma <achalv@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2598619
Reviewed-by: svcacv <svcacv@nvidia.com>
Reviewed-by: Dmitry Pervushin <dpervushin@nvidia.com>
Reviewed-by: Amrita Deshmukh <amritad@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2606002
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
Reviewed-by: Phoenix Jung <pjung@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>

1 files changed, 5 insertions, 3 deletions

diff --git a/drivers/video/tegra/host/flcn/flcn.c b/drivers/video/tegra/host/flcn/flcn.c
index 9a11c4619..e7832fbec 100644
--- a/drivers/video/tegra/host/flcn/flcn.c
+++ b/drivers/video/tegra/host/flcn/flcn.c
@@ -1,7 +1,7 @@
 /*
 * Tegra flcn common driver
 *
-* Copyright (c) 2011-2018, NVIDIA CORPORATION.  All rights reserved.
+* Copyright (c) 2011-2021, NVIDIA CORPORATION.  All rights reserved.
 *
 * This program is free software; you can redistribute it and/or modify it
 * under the terms and conditions of the GNU General Public License,
@@ -553,9 +553,11 @@ static int nvhost_flcn_init_sw(struct platform_device *dev)
         set_flcn(dev, v);
         nvhost_dbg_fn("primed dev:%p v:%p", dev, v);
         err = flcn_read_ucode(dev, pdata->firmware_name, v);
-        if (err || !v->valid)
+        if (err || !v->valid) {
+                kfree(v);
+                set_flcn(dev, NULL);
                 goto clean_up;
-
+        }
         return 0;
 
  clean_up: