Re: [PATCH 2/2] ASoC: codecs: lpass-macro: Use enum for handling codec version

On Tue, Jun 25, 2024 at 06:57:36PM GMT, Krzysztof Kozlowski wrote:

Replace 'int' with proper 'enum lpass_codec_version' in every place which handles the parsed codec version (not raw register values!) to be explicit about contents of the variable. This makes code easier to read and compilers could check missing switch cases.

Signed-off-by: Krzysztof Kozlowski krzysztof.kozlowski@linaro.org

sound/soc/codecs/lpass-macro-common.c | 8 ++++---- sound/soc/codecs/lpass-macro-common.h | 4 ++-- sound/soc/codecs/lpass-rx-macro.c | 2 +- 3 files changed, 7 insertions(+), 7 deletions(-)

Reviewed-by: Dmitry Baryshkov dmitry.baryshkov@linaro.org