diff options
author | Dmitry Osipenko <digetx@gmail.com> | 2021-01-20 03:31:51 +0300 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2021-01-26 07:32:46 +0100 |
commit | 6755568ad111394da224c823e8baf31a05a1469c (patch) | |
tree | e00db3e6af39507cb1bf4a4a1f84235a77959390 /sound | |
parent | 87f0e46e7559beb6f1d1ff99f8f48b1b9d86db52 (diff) | |
download | lwn-6755568ad111394da224c823e8baf31a05a1469c.tar.gz lwn-6755568ad111394da224c823e8baf31a05a1469c.zip |
ALSA: hda/tegra: Remove unnecessary null-check from hda_tegra_runtime_resume()
The "chip" can't be NULL in hda_tegra_runtime_resume() because code would
crash otherwise. Let's remove the unnecessary check in order to clean up
code a tad.
Tested-by: Peter Geis <pgwipeout@gmail.com> # Ouya T30 audio works
Tested-by: Matt Merhar <mattmerhar@protonmail.com> # Ouya T30 boot-tested
Suggested-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Link: https://lore.kernel.org/r/20210120003154.26749-4-digetx@gmail.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound')
-rw-r--r-- | sound/pci/hda/hda_tegra.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/pci/hda/hda_tegra.c b/sound/pci/hda/hda_tegra.c index 04dcd4cdfd9e..6f2b743b9d75 100644 --- a/sound/pci/hda/hda_tegra.c +++ b/sound/pci/hda/hda_tegra.c @@ -178,7 +178,7 @@ static int __maybe_unused hda_tegra_runtime_resume(struct device *dev) rc = clk_bulk_prepare_enable(hda->nclocks, hda->clocks); if (rc != 0) return rc; - if (chip && chip->running) { + if (chip->running) { hda_tegra_init(hda); azx_init_chip(chip, 1); /* disable controller wake up event*/ |