lkml.org 
[lkml]   [2020]   [Sep]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.8 047/255] ASoC: Intel: sof_sdw_rt711: remove properties in card remove
    Date
    From: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>

    [ Upstream commit cf0418cd06ce42fcf35beb33e315b5a77e596926 ]

    The rt711 jack detection properties are set from the machine drivers
    during the card probe, as done in other ASoC examples.

    KASAN reports a use-after-free error when unbinding drivers due to a
    confusing sequence between the ACPI core, the device core and the
    SoundWire device cleanups.

    Rather than fixing this sequence, follow the recommendation to have
    the same caller add and remove properties, add an explicit
    device_remove_properties() in the card .remove() callback.

    In future patches the use of device_add/remove_properties will be
    replaced by a direct handling of a swnode, but the sequence will
    remain the same.

    Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
    Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
    Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
    Link: https://lore.kernel.org/r/20200717211337.31956-3-pierre-louis.bossart@linux.intel.com
    Signed-off-by: Mark Brown <broonie@kernel.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    sound/soc/intel/boards/sof_sdw.c | 1 +
    sound/soc/intel/boards/sof_sdw_common.h | 1 +
    sound/soc/intel/boards/sof_sdw_rt711.c | 15 +++++++++++++++
    3 files changed, 17 insertions(+)

    diff --git a/sound/soc/intel/boards/sof_sdw.c b/sound/soc/intel/boards/sof_sdw.c
    index 1bfd9613449e9..95a119a2d354e 100644
    --- a/sound/soc/intel/boards/sof_sdw.c
    +++ b/sound/soc/intel/boards/sof_sdw.c
    @@ -184,6 +184,7 @@ static struct sof_sdw_codec_info codec_info_list[] = {
    .direction = {true, true},
    .dai_name = "rt711-aif1",
    .init = sof_sdw_rt711_init,
    + .exit = sof_sdw_rt711_exit,
    },
    {
    .id = 0x1308,
    diff --git a/sound/soc/intel/boards/sof_sdw_common.h b/sound/soc/intel/boards/sof_sdw_common.h
    index 69b363b8a6869..fdd2385049e1e 100644
    --- a/sound/soc/intel/boards/sof_sdw_common.h
    +++ b/sound/soc/intel/boards/sof_sdw_common.h
    @@ -84,6 +84,7 @@ int sof_sdw_rt711_init(const struct snd_soc_acpi_link_adr *link,
    struct snd_soc_dai_link *dai_links,
    struct sof_sdw_codec_info *info,
    bool playback);
    +int sof_sdw_rt711_exit(struct device *dev, struct snd_soc_dai_link *dai_link);

    /* RT700 support */
    int sof_sdw_rt700_init(const struct snd_soc_acpi_link_adr *link,
    diff --git a/sound/soc/intel/boards/sof_sdw_rt711.c b/sound/soc/intel/boards/sof_sdw_rt711.c
    index d4d75c8dc6b78..0cb9f1c1f8676 100644
    --- a/sound/soc/intel/boards/sof_sdw_rt711.c
    +++ b/sound/soc/intel/boards/sof_sdw_rt711.c
    @@ -133,6 +133,21 @@ static int rt711_rtd_init(struct snd_soc_pcm_runtime *rtd)
    return ret;
    }

    +int sof_sdw_rt711_exit(struct device *dev, struct snd_soc_dai_link *dai_link)
    +{
    + struct device *sdw_dev;
    +
    + sdw_dev = bus_find_device_by_name(&sdw_bus_type, NULL,
    + dai_link->codecs[0].name);
    + if (!sdw_dev)
    + return -EINVAL;
    +
    + device_remove_properties(sdw_dev);
    + put_device(sdw_dev);
    +
    + return 0;
    +}
    +
    int sof_sdw_rt711_init(const struct snd_soc_acpi_link_adr *link,
    struct snd_soc_dai_link *dai_links,
    struct sof_sdw_codec_info *info,
    --
    2.25.1


    \
     
     \ /
      Last update: 2020-09-01 17:40    [W:4.080 / U:0.236 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site