nv-tegra-mirror/linux-nvgpu
1
0
Fork 0
mirror of git://nv-tegra.nvidia.com/linux-nvgpu.git synced 2025-12-22 17:36:20 +03:00
Code Issues Packages Projects Releases Wiki Activity

gpu: nvgpu: remove duplicated busy in debugger

Browse Source 
On the past, we had separate calls for platform and channel busy, but
those got removed. The result is that in the debugger code we have
essentially a double busy call int the powergating enable/disable.

This change removes it

bug 200277762
JIRA: EVLR-1023

Change-Id: Iba70b81700f27b847e1d0222fb69ed1a7a883342
Signed-off-by: David Nieto <dmartineznie@nvidia.com>
Reviewed-on: http://git-master/r/1323220
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: svccoveritychecker <svccoveritychecker@nvidia.com>
Reviewed-by: Terje Bergstrom <tbergstrom@nvidia.com>
GVS: Gerrit_Virtual_Submit
Reviewed-on: http://git-master/r/1327753
Reviewed-by: Sumeet Gupta <sumeetg@nvidia.com>
This commit is contained in:
David Nieto
2017-03-17 09:47:17 -07:00
committed by mobile promotions
parent 280a628a67
commit ecd71ed447
1 changed files with 1 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/gpu/nvgpu/gk20a/dbg_gpu_gk20a.c
View File

@@ -1275,10 +1275,6 @@ static int dbg_set_powergate(struct dbg_session_gk20a *dbg_s, u32 powermode)
if (err) if (err)
return err; return err;
err = gk20a_busy(dbg_s->dev);
if (err)
return -EPERM;
/*do elpg disable before clock gating */ /*do elpg disable before clock gating */
gk20a_pmu_pg_global_enable(g, false); gk20a_pmu_pg_global_enable(g, false);
@@ -1330,7 +1326,6 @@ static int dbg_set_powergate(struct dbg_session_gk20a *dbg_s, u32 powermode)
gk20a_pmu_pg_global_enable(g, true); gk20a_pmu_pg_global_enable(g, true);
gk20a_dbg(gpu_dbg_gpu_dbg | gpu_dbg_fn, "module idle"); gk20a_dbg(gpu_dbg_gpu_dbg | gpu_dbg_fn, "module idle");
gk20a_idle(dbg_s->dev);
gk20a_idle(g->dev); gk20a_idle(g->dev);
} }
@@ -1426,6 +1421,7 @@ static int nvgpu_dbg_gpu_ioctl_hwpm_ctxsw_mode(struct dbg_session_gk20a *dbg_s,
"session doesn't have a valid reservation"); "session doesn't have a valid reservation");
} }
err = gk20a_busy(g->dev); err = gk20a_busy(g->dev);
if (err) { if (err) {
gk20a_err(dev_from_gk20a(g), "failed to poweron"); gk20a_err(dev_from_gk20a(g), "failed to poweron");