2023-02-13 09:37:35

by wangweidong.a

[permalink] [raw]
Subject: [PATCH 0/3][next] ASoC: codecs: Modify error implicit declaration of function and misspelling

From: Weidong Wang <[email protected]>

Change the function name aw88395_plack_event to aw88395_playback_event
Change hdrlen to hdr_len
Add select GPIOLIB to the sound/soc/codecs/Kconfig file

Weidong Wang (3):
ASoC: codecs: Fixed a spelling error in the function name
ASoC: codecs: Modify the log print statement
ASoC: codecs: Modify error: implicit declaration of function
'gpiod_set_value_cansleep'

sound/soc/codecs/Kconfig | 1 +
sound/soc/codecs/aw88395/aw88395.c | 4 ++--
sound/soc/codecs/aw88395/aw88395_lib.c | 2 +-
3 files changed, 4 insertions(+), 3 deletions(-)


base-commit: 6ba8a227fd19d19779005fb66ad7562608e1df83
--
2.39.1



2023-02-13 09:37:38

by wangweidong.a

[permalink] [raw]
Subject: [PATCH 1/3][next] ASoC: codecs: Fixed a spelling error in the function name

From: Weidong Wang <[email protected]>

Change the function name aw88395_plack_event to aw88395_playback_event

Signed-off-by: Ben Yi <[email protected]>
Signed-off-by: Weidong Wang <[email protected]>
---
sound/soc/codecs/aw88395/aw88395.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sound/soc/codecs/aw88395/aw88395.c b/sound/soc/codecs/aw88395/aw88395.c
index 16ca7ac5ad28..afdce6b7fa26 100644
--- a/sound/soc/codecs/aw88395/aw88395.c
+++ b/sound/soc/codecs/aw88395/aw88395.c
@@ -363,7 +363,7 @@ static const struct snd_kcontrol_new aw88395_controls[] = {
aw88395_profile_get, aw88395_profile_set),
};

-static int aw88395_plack_event(struct snd_soc_dapm_widget *w,
+static int aw88395_playback_event(struct snd_soc_dapm_widget *w,
struct snd_kcontrol *k, int event)
{
struct snd_soc_component *component = snd_soc_dapm_to_component(w->dapm);
@@ -388,7 +388,7 @@ static int aw88395_plack_event(struct snd_soc_dapm_widget *w,
static const struct snd_soc_dapm_widget aw88395_dapm_widgets[] = {
/* playback */
SND_SOC_DAPM_AIF_IN_E("AIF_RX", "Speaker_Playback", 0, 0, 0, 0,
- aw88395_plack_event,
+ aw88395_playback_event,
SND_SOC_DAPM_PRE_PMU | SND_SOC_DAPM_POST_PMD),
SND_SOC_DAPM_OUTPUT("DAC Output"),

--
2.39.1


2023-02-13 09:37:41

by wangweidong.a

[permalink] [raw]
Subject: [PATCH 2/3][next] ASoC: codecs: Modify the log print statement

From: Weidong Wang <[email protected]>

Change hdrlen to hdr_len

Signed-off-by: Ben Yi <[email protected]>
Signed-off-by: Weidong Wang <[email protected]>
---
sound/soc/codecs/aw88395/aw88395_lib.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/aw88395/aw88395_lib.c b/sound/soc/codecs/aw88395/aw88395_lib.c
index d7c31a202adc..05bcf49da857 100644
--- a/sound/soc/codecs/aw88395/aw88395_lib.c
+++ b/sound/soc/codecs/aw88395/aw88395_lib.c
@@ -932,7 +932,7 @@ static int aw_dev_check_cfg_by_hdr(struct aw_device *aw_dev, struct aw_container

hdr_ddt_len = cfg_hdr->hdr_offset + cfg_hdr->ddt_size;
if (hdr_ddt_len > aw_cfg->len) {
- dev_err(aw_dev->dev, "hdrlen with ddt_len [%d] overflow file size[%d]",
+ dev_err(aw_dev->dev, "hdr_len with ddt_len [%d] overflow file size[%d]",
cfg_hdr->hdr_offset, aw_cfg->len);
return -EINVAL;
}
--
2.39.1


2023-02-13 09:37:43

by wangweidong.a

[permalink] [raw]
Subject: [PATCH 3/3][next] ASoC: codecs: Modify error implicit declaration of function 'gpiod_set_value_cansleep'

From: Weidong Wang <[email protected]>

Add select GPIOLIB to the sound/soc/codecs/Kconfig file

Signed-off-by: Ben Yi <[email protected]>
Signed-off-by: Weidong Wang <[email protected]>
---
sound/soc/codecs/Kconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig
index ab7b0696d01d..4621674e68bf 100644
--- a/sound/soc/codecs/Kconfig
+++ b/sound/soc/codecs/Kconfig
@@ -614,6 +614,7 @@ config SND_SOC_AW88395
select CRC8
select CRC32
select REGMAP_I2C
+ select GPIOLIB
select SND_SOC_AW88395_LIB
help
this option enables support for aw88395 Smart PA.
--
2.39.1


2023-02-13 16:19:09

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH 0/3][next] ASoC: codecs: Modify error implicit declaration of function and misspelling

On Mon, 13 Feb 2023 17:36:46 +0800, [email protected] wrote:
> Change the function name aw88395_plack_event to aw88395_playback_event
> Change hdrlen to hdr_len
> Add select GPIOLIB to the sound/soc/codecs/Kconfig file
>
> Weidong Wang (3):
> ASoC: codecs: Fixed a spelling error in the function name
> ASoC: codecs: Modify the log print statement
> ASoC: codecs: Modify error: implicit declaration of function
> 'gpiod_set_value_cansleep'
>
> [...]

Applied to

broonie/sound.git for-next

Thanks!

[1/3] ASoC: codecs: Fixed a spelling error in the function name
commit: 27db41a1f73ad18ed6540e1f8a7083422e5b53d6
[2/3] ASoC: codecs: Modify the log print statement
commit: 14d5d454547adae64a76731c90c37fe33f7d1d44
[3/3] ASoC: codecs: Modify error implicit declaration of function 'gpiod_set_value_cansleep'
commit: 4680716d6a33f50c96eb7ce6ae9cb868baa04438

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark