[PATCH v3 25/27] ASoC: codecs: rt712: Use guard() for mutex locks
Bui Duc Phuc
phucduc.bui at gmail.com
Fri Jul 10 15:35:40 PDT 2026
Hi Mark,
Thanks for the review.
> This function does also have some goto based unwinding, there's a
> similar thing in the rt721 changes.
It seems this goto block was added recently, and I missed it after rebasing.
Sorry for the inconvenience.
However, looking at the goto path, it seems we might need to unwind the
states more completely. What do you think about rewriting it like this?
------------------
regcache_cache_only(rt712->regmap, false);
ret = regcache_sync(rt712->regmap);
if (ret) {
regcache_cache_only(rt712->regmap, true);
regcache_mark_dirty(rt712->regmap);
return ret;
}
regcache_cache_only(rt712->mbq_regmap, false);
ret = regcache_sync(rt712->mbq_regmap);
if (ret) {
regcache_cache_only(rt712->regmap, true);
regcache_cache_only(rt712->mbq_regmap, true);
regcache_mark_dirty(rt712->mbq_regmap);
return ret;
}
return 0;
--------------------
Best regards,
Phuc
More information about the Linux-mediatek
mailing list