linux-toradex.git/sound/soc/hisilicon, branch v4.19 Linux kernel for Apalis and Colibri modules ASoC: hisilicon: fix spelling mistake: "uknown" -> "unknown" 2018-05-13T01:08:11+00:00 Colin Ian King colin.king@canonical.com 2018-05-11T13:28:18+00:00 2b8eae795580ca13ce6f035cac599f6a85c81bbf Trivial fix to spelling mistake in dev_err message text Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Mark Brown <broonie@kernel.org> 
Trivial fix to spelling mistake in dev_err message text

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
ASoC: hisilicon: remove duplicate includes 2017-12-06T17:48:23+00:00 Pravin Shedge pravin.shedge4linux@gmail.com 2017-12-06T17:39:00+00:00 769e40f0c17df53bac8999939993a280c1ea3dbd These duplicate includes have been found with scripts/checkincludes.pl but they have been removed manually to avoid removing false positives. Signed-off-by: Pravin Shedge <pravin.shedge4linux@gmail.com> Signed-off-by: Mark Brown <broonie@kernel.org> 
These duplicate includes have been found with scripts/checkincludes.pl but
they have been removed manually to avoid removing false positives.

Signed-off-by: Pravin Shedge <pravin.shedge4linux@gmail.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
ASoC: hisilicon: constify snd_soc_dai_ops structure 2017-07-17T15:16:58+00:00 Gustavo A. R. Silva garsilva@embeddedor.com 2017-07-13T07:08:53+00:00 94372fceb2fb5d3d05d6db61c9658865d5e27ee4 This structure is only stored in the ops field of a snd_soc_dai_driver structure. That field is declared const, so snd_soc_dai_ops structures that have this property can be declared as const also. Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com> Signed-off-by: Mark Brown <broonie@kernel.org> 
This structure is only stored in the ops field of a snd_soc_dai_driver
structure. That field is declared const, so snd_soc_dai_ops structures
that have this property can be declared as const also.

Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
ASoC: hisilicon: localize functions without external linkage 2017-05-14T10:00:18+00:00 Takashi Sakamoto o-takashi@sakamocchi.jp 2017-05-02T13:33:02+00:00 2a54e845f6e5069666e1749bd952abdc0413910d A driver for hi6210 sound interface on hi6220 boards includes some functions which has no external linkage. These functions should have static qualifier. This commit adds the qualifier to localize the functions. This issue is detected by sparse: hi6210-i2s.c:100:5: warning: symbol 'hi6210_i2s_startup' was not declared. Should it be static? hi6210-i2s.c:178:6: warning: symbol 'hi6210_i2s_shutdown' was not declared. Should it be static? hi6210-i2s.c:527:27: warning: symbol 'hi6210_i2s_dai_init' was not declared. Should it be static? Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp> Signed-off-by: Mark Brown <broonie@kernel.org> 
A driver for hi6210 sound interface on hi6220 boards includes some
functions which has no external linkage. These functions should have
static qualifier.

This commit adds the qualifier to localize the functions. This issue is
detected by sparse:

hi6210-i2s.c:100:5: warning: symbol 'hi6210_i2s_startup' was not declared. Should it be static?
hi6210-i2s.c:178:6: warning: symbol 'hi6210_i2s_shutdown' was not declared. Should it be static?
hi6210-i2s.c:527:27: warning: symbol 'hi6210_i2s_dai_init' was not declared. Should it be static?

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Mark Brown <broonie@kernel.org>
ASoC: hisilicon: Use devm_snd_soc_register_component 2017-04-11T20:20:55+00:00 John Stultz john.stultz@linaro.org 2017-04-11T19:15:18+00:00 e6d56d21bdcfd8968c1859bc1892608acad3e6d8 Per feedback from Mark Brown, this patch updates the hi6210-i2s driver to use devm_snd_soc_register_component which simplifies the logic a bit. Signed-off-by: John Stultz <john.stultz@linaro.org> Signed-off-by: Mark Brown <broonie@kernel.org> 
Per feedback from Mark Brown, this patch updates the hi6210-i2s
driver to use devm_snd_soc_register_component which simplifies
the logic a bit.

Signed-off-by: John Stultz <john.stultz@linaro.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
ASoC: hisilicon: Address style nit to use break in final default of switch statement 2017-04-11T20:20:51+00:00 John Stultz john.stultz@linaro.org 2017-04-11T19:15:17+00:00 16c1c089b23aea681600d504102db950c1bd6ee4 Mark Brown suggested a style change to use break in the final default of a switch statement, so this patch addresses that. Signed-off-by: John Stultz <john.stultz@linaro.org> Signed-off-by: Mark Brown <broonie@kernel.org> 
Mark Brown suggested a style change to use break in the final
default of a switch statement, so this patch addresses that.

Signed-off-by: John Stultz <john.stultz@linaro.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
ASoC: hisilicon: Add error returns even for cases that shouldn't happen. 2017-04-11T20:20:44+00:00 John Stultz john.stultz@linaro.org 2017-04-11T19:15:16+00:00 da13d7462b013ab58129fe20bfb3acb3aa73e07e This patch addresses feedback from Mark Brown, adding a few extra error returns in cases that shouldn't happen Signed-off-by: John Stultz <john.stultz@linaro.org> Signed-off-by: Mark Brown <broonie@kernel.org> 
This patch addresses feedback from Mark Brown, adding a few
extra error returns in cases that shouldn't happen

Signed-off-by: John Stultz <john.stultz@linaro.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
ASoC: hisilicon: Add hi6210 i2s audio driver 2017-04-10T18:40:13+00:00 Andy Green andy.green@linaro.org 2017-03-31T22:06:00+00:00 0bf750f4cbe1406d8819cbec2a3ff6beeb824617 Add driver for hi6210 i2s controller found on hi6220 boards. Signed-off-by: Andy Green <andy.green@linaro.org> [jstultz: Forward ported to mainline, fairly major rework based on suggestions from Mark Brown] Signed-off-by: John Stultz <john.stultz@linaro.org> Signed-off-by: Mark Brown <broonie@kernel.org> 
Add driver for hi6210 i2s controller found on hi6220 boards.

Signed-off-by: Andy Green <andy.green@linaro.org>
[jstultz: Forward ported to mainline, fairly major rework
 based on suggestions from Mark Brown]
Signed-off-by: John Stultz <john.stultz@linaro.org>
Signed-off-by: Mark Brown <broonie@kernel.org>