mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-29 13:53:33 +00:00
soc: qcom: rpmh: Update rpmh_invalidate function to return void
Currently rpmh_invalidate() always returns success. Update its return type to void. Reviewed-by: Lina Iyer <ilina@codeaurora.org> Reviewed-by: Stephen Boyd <swboyd@chromium.org> Suggested-by: Stephen Boyd <swboyd@chromium.org> Signed-off-by: Maulik Shah <mkshah@codeaurora.org> Link: https://lore.kernel.org/r/1592485553-29163-1-git-send-email-mkshah@codeaurora.org Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
This commit is contained in:
parent
681b091238
commit
73edcd38d7
3 changed files with 6 additions and 11 deletions
|
@ -266,11 +266,7 @@ int qcom_icc_bcm_voter_commit(struct bcm_voter *voter)
|
||||||
if (!commit_idx[0])
|
if (!commit_idx[0])
|
||||||
goto out;
|
goto out;
|
||||||
|
|
||||||
ret = rpmh_invalidate(voter->dev);
|
rpmh_invalidate(voter->dev);
|
||||||
if (ret) {
|
|
||||||
pr_err("Error invalidating RPMH client (%d)\n", ret);
|
|
||||||
goto out;
|
|
||||||
}
|
|
||||||
|
|
||||||
ret = rpmh_write_batch(voter->dev, RPMH_ACTIVE_ONLY_STATE,
|
ret = rpmh_write_batch(voter->dev, RPMH_ACTIVE_ONLY_STATE,
|
||||||
cmds, commit_idx);
|
cmds, commit_idx);
|
||||||
|
|
|
@ -497,7 +497,7 @@ int rpmh_flush(struct rpmh_ctrlr *ctrlr)
|
||||||
*
|
*
|
||||||
* Invalidate the sleep and wake values in batch_cache.
|
* Invalidate the sleep and wake values in batch_cache.
|
||||||
*/
|
*/
|
||||||
int rpmh_invalidate(const struct device *dev)
|
void rpmh_invalidate(const struct device *dev)
|
||||||
{
|
{
|
||||||
struct rpmh_ctrlr *ctrlr = get_rpmh_ctrlr(dev);
|
struct rpmh_ctrlr *ctrlr = get_rpmh_ctrlr(dev);
|
||||||
struct batch_cache_req *req, *tmp;
|
struct batch_cache_req *req, *tmp;
|
||||||
|
@ -509,7 +509,5 @@ int rpmh_invalidate(const struct device *dev)
|
||||||
INIT_LIST_HEAD(&ctrlr->batch_cache);
|
INIT_LIST_HEAD(&ctrlr->batch_cache);
|
||||||
ctrlr->dirty = true;
|
ctrlr->dirty = true;
|
||||||
spin_unlock_irqrestore(&ctrlr->cache_lock, flags);
|
spin_unlock_irqrestore(&ctrlr->cache_lock, flags);
|
||||||
|
|
||||||
return 0;
|
|
||||||
}
|
}
|
||||||
EXPORT_SYMBOL(rpmh_invalidate);
|
EXPORT_SYMBOL(rpmh_invalidate);
|
||||||
|
|
|
@ -20,7 +20,7 @@ int rpmh_write_async(const struct device *dev, enum rpmh_state state,
|
||||||
int rpmh_write_batch(const struct device *dev, enum rpmh_state state,
|
int rpmh_write_batch(const struct device *dev, enum rpmh_state state,
|
||||||
const struct tcs_cmd *cmd, u32 *n);
|
const struct tcs_cmd *cmd, u32 *n);
|
||||||
|
|
||||||
int rpmh_invalidate(const struct device *dev);
|
void rpmh_invalidate(const struct device *dev);
|
||||||
|
|
||||||
#else
|
#else
|
||||||
|
|
||||||
|
@ -38,8 +38,9 @@ static inline int rpmh_write_batch(const struct device *dev,
|
||||||
const struct tcs_cmd *cmd, u32 *n)
|
const struct tcs_cmd *cmd, u32 *n)
|
||||||
{ return -ENODEV; }
|
{ return -ENODEV; }
|
||||||
|
|
||||||
static inline int rpmh_invalidate(const struct device *dev)
|
static inline void rpmh_invalidate(const struct device *dev)
|
||||||
{ return -ENODEV; }
|
{
|
||||||
|
}
|
||||||
|
|
||||||
#endif /* CONFIG_QCOM_RPMH */
|
#endif /* CONFIG_QCOM_RPMH */
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue