[PATCH 00/10] ASoC: Use maple tree for Cirrus Logic devices

[PATCH 00/10] ASoC: Use maple tree for Cirrus Logic devices

[Mark Brown]

A lot of the Cirrus Logic devices only support single register
read/write operations so they get no benefit from using the rbtree cache
over the more modern maple tree cache, convert them to use maple tree.

Signed-off-by: Mark Brown <broonie@kernel.org>
---
Mark Brown (10):
      ASoC: cs35l32: Use maple tree register cache
      ASoC: cs35l33: Use maple tree register cache
      ASoC: cs35l34: Use maple tree register cache
      ASoC: cs35l35: Use maple tree register cache
      ASoC: cs4234: Use maple tree register cache
      ASoC: cs42l42: Use maple tree register cache
      ASoC: cs42l73: Use maple tree register cache
      ASoC: cs42l83: Use maple tree register cache
      ASoC: cs43130: Use maple tree register cache
      ASoC: cs35l30: Use maple tree register cache

 sound/soc/codecs/cs35l32.c     | 2 +-
 sound/soc/codecs/cs35l33.c     | 2 +-
 sound/soc/codecs/cs35l34.c     | 2 +-
 sound/soc/codecs/cs35l35.c     | 2 +-
 sound/soc/codecs/cs4234.c      | 2 +-
 sound/soc/codecs/cs42l42.c     | 2 +-
 sound/soc/codecs/cs42l73.c     | 2 +-
 sound/soc/codecs/cs42l83-i2c.c | 2 +-
 sound/soc/codecs/cs43130.c     | 2 +-
 sound/soc/codecs/cs53l30.c     | 2 +-
 10 files changed, 10 insertions(+), 10 deletions(-)
---
base-commit: 9561de3a55bed6bdd44a12820ba81ec416e705a7
change-id: 20230609-asoc-cirrus-maple-0aba1f5c18b8

Best regards,
-- 
Mark Brown <broonie@kernel.org>

[PATCH 01/10] ASoC: cs35l32: Use maple tree register cache

[Mark Brown]

The cs35l32 can only support single register read and write operations
so does not benefit from block writes. This means it gets no benefit from
using the rbtree register cache over the maple tree register cache so
convert it to use maple trees instead, it is more modern.

Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/codecs/cs35l32.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/cs35l32.c b/sound/soc/codecs/cs35l32.c
index dc7a58d68076..8421599f1de7 100644
--- a/sound/soc/codecs/cs35l32.c
+++ b/sound/soc/codecs/cs35l32.c
@@ -260,7 +260,7 @@ static const struct regmap_config cs35l32_regmap = {
 	.volatile_reg = cs35l32_volatile_register,
 	.readable_reg = cs35l32_readable_register,
 	.precious_reg = cs35l32_precious_register,
-	.cache_type = REGCACHE_RBTREE,
+	.cache_type = REGCACHE_MAPLE,
 
 	.use_single_read = true,
 	.use_single_write = true,

-- 
2.30.2

[PATCH 02/10] ASoC: cs35l33: Use maple tree register cache

[Mark Brown]

The cs35l33 can only support single register read and write operations
so does not benefit from block writes. This means it gets no benefit from
using the rbtree register cache over the maple tree register cache so
convert it to use maple trees instead, it is more modern.

Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/codecs/cs35l33.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/cs35l33.c b/sound/soc/codecs/cs35l33.c
index 15e79168d256..fde3196bfe1c 100644
--- a/sound/soc/codecs/cs35l33.c
+++ b/sound/soc/codecs/cs35l33.c
@@ -852,7 +852,7 @@ static const struct regmap_config cs35l33_regmap = {
 	.volatile_reg = cs35l33_volatile_register,
 	.readable_reg = cs35l33_readable_register,
 	.writeable_reg = cs35l33_writeable_register,
-	.cache_type = REGCACHE_RBTREE,
+	.cache_type = REGCACHE_MAPLE,
 	.use_single_read = true,
 	.use_single_write = true,
 };

-- 
2.30.2

[PATCH 03/10] ASoC: cs35l34: Use maple tree register cache

[Mark Brown]

The cs35l34 can only support single register read and write operations
so does not benefit from block writes. This means it gets no benefit from
using the rbtree register cache over the maple tree register cache so
convert it to use maple trees instead, it is more modern.

Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/codecs/cs35l34.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/cs35l34.c b/sound/soc/codecs/cs35l34.c
index b3f98023e6a7..471d9e421fc1 100644
--- a/sound/soc/codecs/cs35l34.c
+++ b/sound/soc/codecs/cs35l34.c
@@ -799,7 +799,7 @@ static struct regmap_config cs35l34_regmap = {
 	.volatile_reg = cs35l34_volatile_register,
 	.readable_reg = cs35l34_readable_register,
 	.precious_reg = cs35l34_precious_register,
-	.cache_type = REGCACHE_RBTREE,
+	.cache_type = REGCACHE_MAPLE,
 
 	.use_single_read = true,
 	.use_single_write = true,

-- 
2.30.2

[PATCH 04/10] ASoC: cs35l35: Use maple tree register cache

[Mark Brown]

The cs35l35 can only support single register read and write operations
so does not benefit from block writes. This means it gets no benefit from
using the rbtree register cache over the maple tree register cache so
convert it to use maple trees instead, it is more modern.

Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/codecs/cs35l35.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/cs35l35.c b/sound/soc/codecs/cs35l35.c
index 947a440a3a47..d37a49b372ff 100644
--- a/sound/soc/codecs/cs35l35.c
+++ b/sound/soc/codecs/cs35l35.c
@@ -1099,7 +1099,7 @@ static struct regmap_config cs35l35_regmap = {
 	.volatile_reg = cs35l35_volatile_register,
 	.readable_reg = cs35l35_readable_register,
 	.precious_reg = cs35l35_precious_register,
-	.cache_type = REGCACHE_RBTREE,
+	.cache_type = REGCACHE_MAPLE,
 	.use_single_read = true,
 	.use_single_write = true,
 };

-- 
2.30.2

[PATCH 05/10] ASoC: cs4234: Use maple tree register cache

[Mark Brown]

The cs4234 can only support single register read and write operations
so does not benefit from block writes. This means it gets no benefit from
using the rbtree register cache over the maple tree register cache so
convert it to use maple trees instead, it is more modern.

Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/codecs/cs4234.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/cs4234.c b/sound/soc/codecs/cs4234.c
index dee1a6662c2e..e1d64e0f28fe 100644
--- a/sound/soc/codecs/cs4234.c
+++ b/sound/soc/codecs/cs4234.c
@@ -675,7 +675,7 @@ static const struct regmap_config cs4234_regmap = {
 	.writeable_reg = cs4234_writeable_register,
 	.reg_defaults = cs4234_default_reg,
 	.num_reg_defaults = ARRAY_SIZE(cs4234_default_reg),
-	.cache_type = REGCACHE_RBTREE,
+	.cache_type = REGCACHE_MAPLE,
 	.use_single_read = true,
 	.use_single_write = true,
 };

-- 
2.30.2

[PATCH 06/10] ASoC: cs42l42: Use maple tree register cache

[Mark Brown]

The cs42l42 can only support single register read and write operations
so does not benefit from block writes. This means it gets no benefit from
using the rbtree register cache over the maple tree register cache so
convert it to use maple trees instead, it is more modern.

Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/codecs/cs42l42.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/cs42l42.c b/sound/soc/codecs/cs42l42.c
index e3edaa1a2761..1c8cc4021232 100644
--- a/sound/soc/codecs/cs42l42.c
+++ b/sound/soc/codecs/cs42l42.c
@@ -393,7 +393,7 @@ const struct regmap_config cs42l42_regmap = {
 	.max_register = CS42L42_MAX_REGISTER,
 	.reg_defaults = cs42l42_reg_defaults,
 	.num_reg_defaults = ARRAY_SIZE(cs42l42_reg_defaults),
-	.cache_type = REGCACHE_RBTREE,
+	.cache_type = REGCACHE_MAPLE,
 
 	.use_single_read = true,
 	.use_single_write = true,

-- 
2.30.2

[PATCH 07/10] ASoC: cs42l73: Use maple tree register cache

[Mark Brown]

The cs42l73 can only support single register read and write operations
so does not benefit from block writes. This means it gets no benefit from
using the rbtree register cache over the maple tree register cache so
convert it to use maple trees instead, it is more modern.

Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/codecs/cs42l73.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/cs42l73.c b/sound/soc/codecs/cs42l73.c
index 0a146319755a..d08ccd4627d9 100644
--- a/sound/soc/codecs/cs42l73.c
+++ b/sound/soc/codecs/cs42l73.c
@@ -1267,7 +1267,7 @@ static const struct regmap_config cs42l73_regmap = {
 	.num_reg_defaults = ARRAY_SIZE(cs42l73_reg_defaults),
 	.volatile_reg = cs42l73_volatile_register,
 	.readable_reg = cs42l73_readable_register,
-	.cache_type = REGCACHE_RBTREE,
+	.cache_type = REGCACHE_MAPLE,
 
 	.use_single_read = true,
 	.use_single_write = true,

-- 
2.30.2

[PATCH 08/10] ASoC: cs42l83: Use maple tree register cache

[Mark Brown]

The cs42l83 can only support single register read and write operations
so does not benefit from block writes. This means it gets no benefit from
using the rbtree register cache over the maple tree register cache so
convert it to use maple trees instead, it is more modern.

Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/codecs/cs42l83-i2c.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/cs42l83-i2c.c b/sound/soc/codecs/cs42l83-i2c.c
index 37629ebd90e0..3185b486e276 100644
--- a/sound/soc/codecs/cs42l83-i2c.c
+++ b/sound/soc/codecs/cs42l83-i2c.c
@@ -158,7 +158,7 @@ static const struct regmap_config cs42l83_regmap = {
 	.max_register = CS42L42_MAX_REGISTER,
 	.reg_defaults = cs42l83_reg_defaults,
 	.num_reg_defaults = ARRAY_SIZE(cs42l83_reg_defaults),
-	.cache_type = REGCACHE_RBTREE,
+	.cache_type = REGCACHE_MAPLE,
 
 	.use_single_read = true,
 	.use_single_write = true,

-- 
2.30.2

[PATCH 09/10] ASoC: cs43130: Use maple tree register cache

[Mark Brown]

The cs43130 can only support single register read and write operations
so does not benefit from block writes. This means it gets no benefit from
using the rbtree register cache over the maple tree register cache so
convert it to use maple trees instead, it is more modern.

Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/codecs/cs43130.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/cs43130.c b/sound/soc/codecs/cs43130.c
index db39abb2a31b..b81257f0ab34 100644
--- a/sound/soc/codecs/cs43130.c
+++ b/sound/soc/codecs/cs43130.c
@@ -2357,7 +2357,7 @@ static const struct regmap_config cs43130_regmap = {
 	.readable_reg		= cs43130_readable_register,
 	.precious_reg		= cs43130_precious_register,
 	.volatile_reg		= cs43130_volatile_register,
-	.cache_type		= REGCACHE_RBTREE,
+	.cache_type		= REGCACHE_MAPLE,
 	/* needed for regcache_sync */
 	.use_single_read	= true,
 	.use_single_write	= true,

-- 
2.30.2

[PATCH 10/10] ASoC: cs35l30: Use maple tree register cache

[Mark Brown]

The cs35l30 can only support single register read and write operations
so does not benefit from block writes. This means it gets no benefit from
using the rbtree register cache over the maple tree register cache so
convert it to use maple trees instead, it is more modern.

Signed-off-by: Mark Brown <broonie@kernel.org>
---
 sound/soc/codecs/cs53l30.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/cs53l30.c b/sound/soc/codecs/cs53l30.c
index 69db0013d243..dfbd5b2b2bd9 100644
--- a/sound/soc/codecs/cs53l30.c
+++ b/sound/soc/codecs/cs53l30.c
@@ -911,7 +911,7 @@ static struct regmap_config cs53l30_regmap = {
 	.volatile_reg = cs53l30_volatile_register,
 	.writeable_reg = cs53l30_writeable_register,
 	.readable_reg = cs53l30_readable_register,
-	.cache_type = REGCACHE_RBTREE,
+	.cache_type = REGCACHE_MAPLE,
 
 	.use_single_read = true,
 	.use_single_write = true,

-- 
2.30.2

Re: [PATCH 00/10] ASoC: Use maple tree for Cirrus Logic devices

[Rhodes, David]

On 6/10/23 8:56 AM, Mark Brown wrote:
> A lot of the Cirrus Logic devices only support single register
> read/write operations so they get no benefit from using the rbtree cache
> over the more modern maple tree cache, convert them to use maple tree.
> 

Acked-by: David Rhodes <david.rhodes@cirrus.com>

Thanks,
David

Re: [PATCH 00/10] ASoC: Use maple tree for Cirrus Logic devices

[Mark Brown]

On Sat, 10 Jun 2023 14:56:20 +0100, Mark Brown wrote:
> A lot of the Cirrus Logic devices only support single register
> read/write operations so they get no benefit from using the rbtree cache
> over the more modern maple tree cache, convert them to use maple tree.
> 
> 

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[01/10] ASoC: cs35l32: Use maple tree register cache
        commit: 176bb179f190682d496220be469ea20527bb5f43
[02/10] ASoC: cs35l33: Use maple tree register cache
        commit: 7a230512d335793fdc43bb85a7d5cff9dd171c26
[03/10] ASoC: cs35l34: Use maple tree register cache
        commit: e7795f2d29e08e15fbc5ad88b94cf1899915a7c3
[04/10] ASoC: cs35l35: Use maple tree register cache
        commit: 28f851babc484c86bc8e1919ad0bbe11f4fd9210
[05/10] ASoC: cs4234: Use maple tree register cache
        commit: bb1bd25ad79cf21b8fa4c0eae474307b2d24b268
[06/10] ASoC: cs42l42: Use maple tree register cache
        commit: 6b7fed83c9455f64a1509a9e1d512a92edaaf44e
[07/10] ASoC: cs42l73: Use maple tree register cache
        commit: 7e39a71876244639774c71144e4b5dee7799e1cf
[08/10] ASoC: cs42l83: Use maple tree register cache
        commit: 62145b0a537410d7ce237945c339635f9a86a895
[09/10] ASoC: cs43130: Use maple tree register cache
        commit: ce598b2f83608f3818f8a4079662d3844679b16f
[10/10] ASoC: cs35l30: Use maple tree register cache
        commit: 0eff26b13da4eb5a71a59280c3483273ccb5b9cb

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

Re: [PATCH 04/10] ASoC: cs35l35: Use maple tree register cache

[Charles Keepax]

On Sat, Jun 10, 2023 at 02:56:24PM +0100, Mark Brown wrote:
> The cs35l35 can only support single register read and write operations
> so does not benefit from block writes. This means it gets no benefit from
> using the rbtree register cache over the maple tree register cache so
> convert it to use maple trees instead, it is more modern.
> 
> Signed-off-by: Mark Brown <broonie@kernel.org>
> ---

A little late to the party, but for what it is worth on this
cs35l35 one:

Tested-by: Charles Keepax <ckeepax@opensource.cirrus.com>

Thanks,
Charles