Commit 74f5edbf authored by Weili Qian's avatar Weili Qian Committed by Herbert Xu

crypto: hisilicon - change parameter passing of debugfs function

To avoid repeatedly obtaining 'qm' from 'filp', parameter passing of
debugfs function directly use 'qm' instead of 'filp'.
Signed-off-by: default avatarWeili Qian <qianweili@huawei.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 607c191b
...@@ -1203,16 +1203,13 @@ static struct hisi_qm *file_to_qm(struct debugfs_file *file) ...@@ -1203,16 +1203,13 @@ static struct hisi_qm *file_to_qm(struct debugfs_file *file)
return container_of(debug, struct hisi_qm, debug); return container_of(debug, struct hisi_qm, debug);
} }
static u32 current_q_read(struct debugfs_file *file) static u32 current_q_read(struct hisi_qm *qm)
{ {
struct hisi_qm *qm = file_to_qm(file);
return readl(qm->io_base + QM_DFX_SQE_CNT_VF_SQN) >> QM_DFX_QN_SHIFT; return readl(qm->io_base + QM_DFX_SQE_CNT_VF_SQN) >> QM_DFX_QN_SHIFT;
} }
static int current_q_write(struct debugfs_file *file, u32 val) static int current_q_write(struct hisi_qm *qm, u32 val)
{ {
struct hisi_qm *qm = file_to_qm(file);
u32 tmp; u32 tmp;
if (val >= qm->debug.curr_qm_qp_num) if (val >= qm->debug.curr_qm_qp_num)
...@@ -1229,18 +1226,14 @@ static int current_q_write(struct debugfs_file *file, u32 val) ...@@ -1229,18 +1226,14 @@ static int current_q_write(struct debugfs_file *file, u32 val)
return 0; return 0;
} }
static u32 clear_enable_read(struct debugfs_file *file) static u32 clear_enable_read(struct hisi_qm *qm)
{ {
struct hisi_qm *qm = file_to_qm(file);
return readl(qm->io_base + QM_DFX_CNT_CLR_CE); return readl(qm->io_base + QM_DFX_CNT_CLR_CE);
} }
/* rd_clr_ctrl 1 enable read clear, otherwise 0 disable it */ /* rd_clr_ctrl 1 enable read clear, otherwise 0 disable it */
static int clear_enable_write(struct debugfs_file *file, u32 rd_clr_ctrl) static int clear_enable_write(struct hisi_qm *qm, u32 rd_clr_ctrl)
{ {
struct hisi_qm *qm = file_to_qm(file);
if (rd_clr_ctrl > 1) if (rd_clr_ctrl > 1)
return -EINVAL; return -EINVAL;
...@@ -1249,16 +1242,13 @@ static int clear_enable_write(struct debugfs_file *file, u32 rd_clr_ctrl) ...@@ -1249,16 +1242,13 @@ static int clear_enable_write(struct debugfs_file *file, u32 rd_clr_ctrl)
return 0; return 0;
} }
static u32 current_qm_read(struct debugfs_file *file) static u32 current_qm_read(struct hisi_qm *qm)
{ {
struct hisi_qm *qm = file_to_qm(file);
return readl(qm->io_base + QM_DFX_MB_CNT_VF); return readl(qm->io_base + QM_DFX_MB_CNT_VF);
} }
static int current_qm_write(struct debugfs_file *file, u32 val) static int current_qm_write(struct hisi_qm *qm, u32 val)
{ {
struct hisi_qm *qm = file_to_qm(file);
u32 tmp; u32 tmp;
if (val > qm->vfs_num) if (val > qm->vfs_num)
...@@ -1301,13 +1291,13 @@ static ssize_t qm_debug_read(struct file *filp, char __user *buf, ...@@ -1301,13 +1291,13 @@ static ssize_t qm_debug_read(struct file *filp, char __user *buf,
mutex_lock(&file->lock); mutex_lock(&file->lock);
switch (index) { switch (index) {
case CURRENT_QM: case CURRENT_QM:
val = current_qm_read(file); val = current_qm_read(qm);
break; break;
case CURRENT_Q: case CURRENT_Q:
val = current_q_read(file); val = current_q_read(qm);
break; break;
case CLEAR_ENABLE: case CLEAR_ENABLE:
val = clear_enable_read(file); val = clear_enable_read(qm);
break; break;
default: default:
goto err_input; goto err_input;
...@@ -1356,13 +1346,13 @@ static ssize_t qm_debug_write(struct file *filp, const char __user *buf, ...@@ -1356,13 +1346,13 @@ static ssize_t qm_debug_write(struct file *filp, const char __user *buf,
mutex_lock(&file->lock); mutex_lock(&file->lock);
switch (index) { switch (index) {
case CURRENT_QM: case CURRENT_QM:
ret = current_qm_write(file, val); ret = current_qm_write(qm, val);
break; break;
case CURRENT_Q: case CURRENT_Q:
ret = current_q_write(file, val); ret = current_q_write(qm, val);
break; break;
case CLEAR_ENABLE: case CLEAR_ENABLE:
ret = clear_enable_write(file, val); ret = clear_enable_write(qm, val);
break; break;
default: default:
ret = -EINVAL; ret = -EINVAL;
......
...@@ -557,17 +557,14 @@ static void sec_hw_error_disable(struct hisi_qm *qm) ...@@ -557,17 +557,14 @@ static void sec_hw_error_disable(struct hisi_qm *qm)
writel(SEC_RAS_DISABLE, qm->io_base + SEC_RAS_NFE_REG); writel(SEC_RAS_DISABLE, qm->io_base + SEC_RAS_NFE_REG);
} }
static u32 sec_clear_enable_read(struct sec_debug_file *file) static u32 sec_clear_enable_read(struct hisi_qm *qm)
{ {
struct hisi_qm *qm = file->qm;
return readl(qm->io_base + SEC_CTRL_CNT_CLR_CE) & return readl(qm->io_base + SEC_CTRL_CNT_CLR_CE) &
SEC_CTRL_CNT_CLR_CE_BIT; SEC_CTRL_CNT_CLR_CE_BIT;
} }
static int sec_clear_enable_write(struct sec_debug_file *file, u32 val) static int sec_clear_enable_write(struct hisi_qm *qm, u32 val)
{ {
struct hisi_qm *qm = file->qm;
u32 tmp; u32 tmp;
if (val != 1 && val) if (val != 1 && val)
...@@ -597,7 +594,7 @@ static ssize_t sec_debug_read(struct file *filp, char __user *buf, ...@@ -597,7 +594,7 @@ static ssize_t sec_debug_read(struct file *filp, char __user *buf,
switch (file->index) { switch (file->index) {
case SEC_CLEAR_ENABLE: case SEC_CLEAR_ENABLE:
val = sec_clear_enable_read(file); val = sec_clear_enable_read(qm);
break; break;
default: default:
goto err_input; goto err_input;
...@@ -647,7 +644,7 @@ static ssize_t sec_debug_write(struct file *filp, const char __user *buf, ...@@ -647,7 +644,7 @@ static ssize_t sec_debug_write(struct file *filp, const char __user *buf,
switch (file->index) { switch (file->index) {
case SEC_CLEAR_ENABLE: case SEC_CLEAR_ENABLE:
ret = sec_clear_enable_write(file, val); ret = sec_clear_enable_write(qm, val);
if (ret) if (ret)
goto err_input; goto err_input;
break; break;
......
...@@ -450,17 +450,14 @@ static inline struct hisi_qm *file_to_qm(struct ctrl_debug_file *file) ...@@ -450,17 +450,14 @@ static inline struct hisi_qm *file_to_qm(struct ctrl_debug_file *file)
return &hisi_zip->qm; return &hisi_zip->qm;
} }
static u32 clear_enable_read(struct ctrl_debug_file *file) static u32 clear_enable_read(struct hisi_qm *qm)
{ {
struct hisi_qm *qm = file_to_qm(file);
return readl(qm->io_base + HZIP_SOFT_CTRL_CNT_CLR_CE) & return readl(qm->io_base + HZIP_SOFT_CTRL_CNT_CLR_CE) &
HZIP_SOFT_CTRL_CNT_CLR_CE_BIT; HZIP_SOFT_CTRL_CNT_CLR_CE_BIT;
} }
static int clear_enable_write(struct ctrl_debug_file *file, u32 val) static int clear_enable_write(struct hisi_qm *qm, u32 val)
{ {
struct hisi_qm *qm = file_to_qm(file);
u32 tmp; u32 tmp;
if (val != 1 && val != 0) if (val != 1 && val != 0)
...@@ -489,7 +486,7 @@ static ssize_t hisi_zip_ctrl_debug_read(struct file *filp, char __user *buf, ...@@ -489,7 +486,7 @@ static ssize_t hisi_zip_ctrl_debug_read(struct file *filp, char __user *buf,
spin_lock_irq(&file->lock); spin_lock_irq(&file->lock);
switch (file->index) { switch (file->index) {
case HZIP_CLEAR_ENABLE: case HZIP_CLEAR_ENABLE:
val = clear_enable_read(file); val = clear_enable_read(qm);
break; break;
default: default:
goto err_input; goto err_input;
...@@ -537,7 +534,7 @@ static ssize_t hisi_zip_ctrl_debug_write(struct file *filp, ...@@ -537,7 +534,7 @@ static ssize_t hisi_zip_ctrl_debug_write(struct file *filp,
spin_lock_irq(&file->lock); spin_lock_irq(&file->lock);
switch (file->index) { switch (file->index) {
case HZIP_CLEAR_ENABLE: case HZIP_CLEAR_ENABLE:
ret = clear_enable_write(file, val); ret = clear_enable_write(qm, val);
if (ret) if (ret)
goto err_input; goto err_input;
break; break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment