[alsa-devel] [PATCH v2 1/3] ASoC: Intel: sst: Simplify is_byt_cr()

Pierre-Louis Bossart pierre-louis.bossart at linux.intel.com
Wed Jan 2 21:44:33 CET 2019


On 1/2/19 1:39 PM, Stephan Gerhold wrote:
> is_byt_cr() and its usage can be simplified by returning the bool
> directly, instead of through a pointer. This works because the
> return value is just treated as bytcr = false and is not used
> otherwise.
>
> This patch also removes the extra check of
> IS_ENABLED(CONFIG_IOSF_MBI) in favor of checking
> iosf_mbi_available() directly. The header already takes care
> of returning false if the config option is not enabled.
>
> No functional change.
>
> Signed-off-by: Stephan Gerhold <stephan at gerhold.net>

Nice cleanup!

All 3 patches

Acked-by: Pierre-Louis Bossart <pierre-louis.bossart at linux.intel.com>

The next step will be to move this helper to a common library at some 
point, we'll need it for SOF as well.




More information about the Alsa-devel mailing list