tegra: hwpm: redraft regops ioctl return status

- As regops ioctl status is an output from kernel, the status cannot be
invalid. So, remove TEGRA_SOC_HWPM_REG_OP_STATUS_INVALID define.
- Redefine regops ioctl status to make SUCCESS equivalent to value 0.
- Add TEGRA_SOC_HWPM_REG_OP_STATUS_WR_FAILED to indicate failure in
regops write command.

Bug 3335825

Change-Id: I31152f1ce2558fdf4c8829dd19fbcb9c87e20572
Signed-off-by: Vedashree Vidwans <vvidwans@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2556234
Reviewed-by: svcacv <svcacv@nvidia.com>
Reviewed-by: Seema Khowala <seemaj@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
GVS: Gerrit_Virtual_Submit
This commit is contained in:
Vedashree Vidwans
2021-07-08 13:55:59 -07:00
committed by mobile promotions
parent 492dd01e04
commit ae2a1e7dfc

View File

@@ -904,7 +904,7 @@ static int exec_reg_ops_ioctl(struct tegra_soc_hwpm *hwpm,
reg_op->phys_addr, reg_op->mask_lo, reg_op->phys_addr, reg_op->mask_lo,
reg_op->reg_val_lo, true, aperture->is_ip); reg_op->reg_val_lo, true, aperture->is_ip);
if (ret < 0) { if (ret < 0) {
REG_OP_FAIL(INVALID, REG_OP_FAIL(WR_FAILED,
"WR32 REGOP failed for register(0x%llx)", "WR32 REGOP failed for register(0x%llx)",
reg_op->phys_addr); reg_op->phys_addr);
} else { } else {
@@ -919,7 +919,7 @@ static int exec_reg_ops_ioctl(struct tegra_soc_hwpm *hwpm,
reg_op->phys_addr, reg_op->mask_lo, reg_op->phys_addr, reg_op->mask_lo,
reg_op->reg_val_lo, true, aperture->is_ip); reg_op->reg_val_lo, true, aperture->is_ip);
if (ret < 0) { if (ret < 0) {
REG_OP_FAIL(INVALID, REG_OP_FAIL(WR_FAILED,
"WR64 REGOP failed for register(0x%llx)", "WR64 REGOP failed for register(0x%llx)",
reg_op->phys_addr); reg_op->phys_addr);
continue; continue;
@@ -930,7 +930,7 @@ static int exec_reg_ops_ioctl(struct tegra_soc_hwpm *hwpm,
reg_op->phys_addr + 4, reg_op->mask_hi, reg_op->phys_addr + 4, reg_op->mask_hi,
reg_op->reg_val_hi, true, aperture->is_ip); reg_op->reg_val_hi, true, aperture->is_ip);
if (ret < 0) { if (ret < 0) {
REG_OP_FAIL(INVALID, REG_OP_FAIL(WR_FAILED,
"WR64 REGOP failed for register(0x%llx)", "WR64 REGOP failed for register(0x%llx)",
reg_op->phys_addr + 4); reg_op->phys_addr + 4);
} else { } else {