lkml.org 
[lkml]   [2022]   [May]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v4 11/17] ASoC: cs35l41: Add common cs35l41 enter hibernate function
    Date
    From: Stefan Binding <sbinding@opensource.cirrus.com>

    Since the CS35L41 HDA driver also support hibernation, it
    makes sense to move code from the ASoC driver to enter
    hibernation into common code.

    Since HDA must support laptops which do not support hibernation
    due to lack of external boost GPIO it is necessary to
    ensure the function returns an error when an unsupported
    boost type is in use.

    Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>

    Signed-off-by: Stefan Binding <sbinding@opensource.cirrus.com>
    Signed-off-by: Vitaly Rodionov <vitalyr@opensource.cirrus.com>
    ---

    Changes since v2:
    - No change

    include/sound/cs35l41.h | 2 ++
    sound/soc/codecs/cs35l41-lib.c | 19 +++++++++++++++++++
    sound/soc/codecs/cs35l41.c | 10 +---------
    3 files changed, 22 insertions(+), 9 deletions(-)

    diff --git a/include/sound/cs35l41.h b/include/sound/cs35l41.h
    index f848ba1e75b3..9ac5918269a5 100644
    --- a/include/sound/cs35l41.h
    +++ b/include/sound/cs35l41.h
    @@ -885,6 +885,8 @@ void cs35l41_configure_cs_dsp(struct device *dev, struct regmap *reg, struct cs_
    int cs35l41_set_cspl_mbox_cmd(struct device *dev, struct regmap *regmap,
    enum cs35l41_cspl_mbox_cmd cmd);
    int cs35l41_write_fs_errata(struct device *dev, struct regmap *regmap);
    +int cs35l41_enter_hibernate(struct device *dev, struct regmap *regmap,
    + enum cs35l41_boost_type b_type);
    int cs35l41_exit_hibernate(struct device *dev, struct regmap *regmap);
    int cs35l41_init_boost(struct device *dev, struct regmap *regmap,
    struct cs35l41_hw_cfg *hw_cfg);
    diff --git a/sound/soc/codecs/cs35l41-lib.c b/sound/soc/codecs/cs35l41-lib.c
    index e726a38f1997..0c7d1c791279 100644
    --- a/sound/soc/codecs/cs35l41-lib.c
    +++ b/sound/soc/codecs/cs35l41-lib.c
    @@ -1322,6 +1322,25 @@ int cs35l41_write_fs_errata(struct device *dev, struct regmap *regmap)
    }
    EXPORT_SYMBOL_GPL(cs35l41_write_fs_errata);

    +int cs35l41_enter_hibernate(struct device *dev, struct regmap *regmap,
    + enum cs35l41_boost_type b_type)
    +{
    + if (!cs35l41_safe_reset(regmap, b_type)) {
    + dev_dbg(dev, "System does not support Suspend\n");
    + return -EINVAL;
    + }
    +
    + dev_dbg(dev, "Enter hibernate\n");
    + regmap_write(regmap, CS35L41_WAKESRC_CTL, 0x0088);
    + regmap_write(regmap, CS35L41_WAKESRC_CTL, 0x0188);
    +
    + // Don't wait for ACK since bus activity would wake the device
    + regmap_write(regmap, CS35L41_DSP_VIRT1_MBOX_1, CSPL_MBOX_CMD_HIBERNATE);
    +
    + return 0;
    +}
    +EXPORT_SYMBOL_GPL(cs35l41_enter_hibernate);
    +
    static void cs35l41_wait_for_pwrmgt_sts(struct device *dev, struct regmap *regmap)
    {
    const int pwrmgt_retries = 10;
    diff --git a/sound/soc/codecs/cs35l41.c b/sound/soc/codecs/cs35l41.c
    index be7d02517739..a115ea35b92d 100644
    --- a/sound/soc/codecs/cs35l41.c
    +++ b/sound/soc/codecs/cs35l41.c
    @@ -1335,15 +1335,7 @@ static int __maybe_unused cs35l41_runtime_suspend(struct device *dev)
    if (!cs35l41->dsp.preloaded || !cs35l41->dsp.cs_dsp.running)
    return 0;

    - dev_dbg(cs35l41->dev, "Enter hibernate\n");
    -
    - cs35l41_safe_reset(cs35l41->regmap, cs35l41->hw_cfg.bst_type);
    - regmap_write(cs35l41->regmap, CS35L41_WAKESRC_CTL, 0x0088);
    - regmap_write(cs35l41->regmap, CS35L41_WAKESRC_CTL, 0x0188);
    -
    - // Don't wait for ACK since bus activity would wake the device
    - regmap_write(cs35l41->regmap, CS35L41_DSP_VIRT1_MBOX_1,
    - CSPL_MBOX_CMD_HIBERNATE);
    + cs35l41_enter_hibernate(dev, cs35l41->regmap, cs35l41->hw_cfg.bst_type);

    regcache_cache_only(cs35l41->regmap, true);
    regcache_mark_dirty(cs35l41->regmap);
    --
    2.34.1
    \
     
     \ /
      Last update: 2022-05-25 15:19    [W:4.154 / U:0.292 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site