* [PATCH 2/5] ASoC: cs42l73: Add missing regmap use_single config
2021-05-11 17:57 [PATCH 1/5] ASoC: cs53l30: Add missing regmap use_single config Charles Keepax
@ 2021-05-11 17:57 ` Charles Keepax
2021-05-11 17:57 ` [PATCH 3/5] ASoC: cs35l34: " Charles Keepax
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Charles Keepax @ 2021-05-11 17:57 UTC (permalink / raw)
To: broonie
Cc: pierre-louis.bossart, alsa-devel, patches, lgirdwood,
james.schulman, david.rhodes
This device requires single register transactions, this will
definely cause problems with the new device ID parsing which uses
regmap_bulk_read but might also show up in the cache sync sometimes.
Add the missing flags to the regmap_config.
Fixes: 26495252fe0d ("ASoC: cs42l73: Minor error paths fixups")
Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
---
sound/soc/codecs/cs42l73.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/sound/soc/codecs/cs42l73.c b/sound/soc/codecs/cs42l73.c
index da5d77a5f55bb..018463f34e127 100644
--- a/sound/soc/codecs/cs42l73.c
+++ b/sound/soc/codecs/cs42l73.c
@@ -1269,6 +1269,9 @@ static const struct regmap_config cs42l73_regmap = {
.volatile_reg = cs42l73_volatile_register,
.readable_reg = cs42l73_readable_register,
.cache_type = REGCACHE_RBTREE,
+
+ .use_single_read = true,
+ .use_single_write = true,
};
static int cs42l73_i2c_probe(struct i2c_client *i2c_client,
--
2.11.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 3/5] ASoC: cs35l34: Add missing regmap use_single config
2021-05-11 17:57 [PATCH 1/5] ASoC: cs53l30: Add missing regmap use_single config Charles Keepax
2021-05-11 17:57 ` [PATCH 2/5] ASoC: cs42l73: " Charles Keepax
@ 2021-05-11 17:57 ` Charles Keepax
2021-05-11 17:57 ` [PATCH 4/5] ASoC: cs35l32: " Charles Keepax
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Charles Keepax @ 2021-05-11 17:57 UTC (permalink / raw)
To: broonie
Cc: pierre-louis.bossart, alsa-devel, patches, lgirdwood,
james.schulman, david.rhodes
This device requires single register transactions, this will
definely cause problems with the new device ID parsing which uses
regmap_bulk_read but might also show up in the cache sync sometimes.
Add the missing flags to the regmap_config.
Fixes: 8cb9b001635c ("ASoC: cs35l34: Minor error paths fixups")
Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
---
sound/soc/codecs/cs35l34.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/sound/soc/codecs/cs35l34.c b/sound/soc/codecs/cs35l34.c
index 6657cc5db3e87..ed678241c22bc 100644
--- a/sound/soc/codecs/cs35l34.c
+++ b/sound/soc/codecs/cs35l34.c
@@ -801,6 +801,9 @@ static struct regmap_config cs35l34_regmap = {
.readable_reg = cs35l34_readable_register,
.precious_reg = cs35l34_precious_register,
.cache_type = REGCACHE_RBTREE,
+
+ .use_single_read = true,
+ .use_single_write = true,
};
static int cs35l34_handle_of_data(struct i2c_client *i2c_client,
--
2.11.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 4/5] ASoC: cs35l32: Add missing regmap use_single config
2021-05-11 17:57 [PATCH 1/5] ASoC: cs53l30: Add missing regmap use_single config Charles Keepax
2021-05-11 17:57 ` [PATCH 2/5] ASoC: cs42l73: " Charles Keepax
2021-05-11 17:57 ` [PATCH 3/5] ASoC: cs35l34: " Charles Keepax
@ 2021-05-11 17:57 ` Charles Keepax
2021-05-11 17:57 ` [PATCH 5/5] ASoC: cs42l52: Minor tidy up of error paths Charles Keepax
2021-05-11 18:21 ` [PATCH 1/5] ASoC: cs53l30: Add missing regmap use_single config Mark Brown
4 siblings, 0 replies; 6+ messages in thread
From: Charles Keepax @ 2021-05-11 17:57 UTC (permalink / raw)
To: broonie
Cc: pierre-louis.bossart, alsa-devel, patches, lgirdwood,
james.schulman, david.rhodes
This device requires single register transactions, this will
definely cause problems with the new device ID parsing which uses
regmap_bulk_read but might also show up in the cache sync sometimes.
Add the missing flags to the regmap_config.
Fixes: 283160f1419d ("ASoC: cs35l32: Minor error paths fixups")
Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
---
sound/soc/codecs/cs35l32.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/sound/soc/codecs/cs35l32.c b/sound/soc/codecs/cs35l32.c
index 7e1047362a901..933e3d627e5f8 100644
--- a/sound/soc/codecs/cs35l32.c
+++ b/sound/soc/codecs/cs35l32.c
@@ -262,6 +262,9 @@ static const struct regmap_config cs35l32_regmap = {
.readable_reg = cs35l32_readable_register,
.precious_reg = cs35l32_precious_register,
.cache_type = REGCACHE_RBTREE,
+
+ .use_single_read = true,
+ .use_single_write = true,
};
static int cs35l32_handle_of_data(struct i2c_client *i2c_client,
--
2.11.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 5/5] ASoC: cs42l52: Minor tidy up of error paths
2021-05-11 17:57 [PATCH 1/5] ASoC: cs53l30: Add missing regmap use_single config Charles Keepax
` (2 preceding siblings ...)
2021-05-11 17:57 ` [PATCH 4/5] ASoC: cs35l32: " Charles Keepax
@ 2021-05-11 17:57 ` Charles Keepax
2021-05-11 18:21 ` [PATCH 1/5] ASoC: cs53l30: Add missing regmap use_single config Mark Brown
4 siblings, 0 replies; 6+ messages in thread
From: Charles Keepax @ 2021-05-11 17:57 UTC (permalink / raw)
To: broonie
Cc: pierre-louis.bossart, alsa-devel, patches, lgirdwood,
james.schulman, david.rhodes
Fixup a needlessly initialised variable and an unchecked return
value.
Reported-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
---
sound/soc/codecs/cs42l56.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/sound/soc/codecs/cs42l56.c b/sound/soc/codecs/cs42l56.c
index c44a5cdb796ec..7cdffdf6b8cf0 100644
--- a/sound/soc/codecs/cs42l56.c
+++ b/sound/soc/codecs/cs42l56.c
@@ -1175,7 +1175,7 @@ static int cs42l56_i2c_probe(struct i2c_client *i2c_client,
struct cs42l56_platform_data *pdata =
dev_get_platdata(&i2c_client->dev);
int ret, i;
- unsigned int devid = 0;
+ unsigned int devid;
unsigned int alpha_rev, metal_rev;
unsigned int reg;
@@ -1245,6 +1245,11 @@ static int cs42l56_i2c_probe(struct i2c_client *i2c_client,
}
ret = regmap_read(cs42l56->regmap, CS42L56_CHIP_ID_1, ®);
+ if (ret) {
+ dev_err(&i2c_client->dev, "Failed to read chip ID: %d\n", ret);
+ return ret;
+ }
+
devid = reg & CS42L56_CHIP_ID_MASK;
if (devid != CS42L56_DEVID) {
dev_err(&i2c_client->dev,
--
2.11.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/5] ASoC: cs53l30: Add missing regmap use_single config
2021-05-11 17:57 [PATCH 1/5] ASoC: cs53l30: Add missing regmap use_single config Charles Keepax
` (3 preceding siblings ...)
2021-05-11 17:57 ` [PATCH 5/5] ASoC: cs42l52: Minor tidy up of error paths Charles Keepax
@ 2021-05-11 18:21 ` Mark Brown
4 siblings, 0 replies; 6+ messages in thread
From: Mark Brown @ 2021-05-11 18:21 UTC (permalink / raw)
To: Charles Keepax
Cc: alsa-devel, patches, lgirdwood, pierre-louis.bossart, Mark Brown,
james.schulman, david.rhodes
On Tue, 11 May 2021 18:57:14 +0100, Charles Keepax wrote:
> This device requires single register transactions, this will
> definely cause problems with the new device ID parsing which
> uses regmap_bulk_read but might also show up in the cache sync
> sometimes. Add the missing flags to the regmap_config.
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
Thanks!
[1/5] ASoC: cs53l30: Add missing regmap use_single config
(no commit info)
[2/5] ASoC: cs42l73: Add missing regmap use_single config
(no commit info)
[3/5] ASoC: cs35l34: Add missing regmap use_single config
(no commit info)
[4/5] ASoC: cs35l32: Add missing regmap use_single config
(no commit info)
[5/5] ASoC: cs42l52: Minor tidy up of error paths
commit: 4ac9b48adf4d561d0e33419d548278f205dd70b5
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
^ permalink raw reply [flat|nested] 6+ messages in thread