diff options
author | Charles Keepax <ckeepax@opensource.cirrus.com> | 2019-02-05 12:18:12 +0100 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2019-02-06 16:50:50 +0100 |
commit | 52cadf1fdbe87a3a3eee11d9cc4873796903c934 (patch) | |
tree | 4830d17993296c4df81a861133d44a43afbea03b /arch/x86/include/asm/spinlock.h | |
parent | ASoC: wcd9335: remove some unnecessary NULL checks (diff) | |
download | linux-52cadf1fdbe87a3a3eee11d9cc4873796903c934.tar.xz linux-52cadf1fdbe87a3a3eee11d9cc4873796903c934.zip |
ASoC: compress: Clarify the intent of current compressed ops handling
For callbacks configuring the state of the components (trigger,
set_params, ack and set_metadata) simplify the code a little and make
intention clearer by aborting as soon as an error is encountered. The
operation has already failed and there is nothing to be gained from
processing the callbacks on additional components. The operations
currently abort after the callbacks, so this simply shortens the
error path.
For callbacks returning information from the driver (copy,
get_metadata, pointer, get_codec_caps, get_caps and get_params)
only look for the first callback provided, currently the code will
call every callback only returning the information provided by the
last. Since we can only return one set of data, it makes no sense to
request the data from every component. Again this just makes the
currently supported feature set a little more clear.
Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'arch/x86/include/asm/spinlock.h')
0 files changed, 0 insertions, 0 deletions