Re: [PATCH 2/2] ASoC: codecs: lpass-macro: Use enum for handling codec version
From: Dmitry Baryshkov
Date: Wed Jun 26 2024 - 00:52:37 EST
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@xxxxxxxxxx>
> ---
> 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@xxxxxxxxxx>
--
With best wishes
Dmitry