Message ID | 20220228153343.2263412-1-cezary.rojewski@intel.com (mailing list archive) |
---|---|
Headers | show |
Series | ASoC: Intel: AVS - Audio DSP for cAVS | expand |
> Changes v2 -> v1: > Almost all updates here are thanks to feedback from Pierre. Can I ask for a bit of mercy on reviewers and specifically my time? I sent my comments on Friday, I haven't had time to look at the answers and there's a new version already? > - several comments and few kernel-docs have been added in areas which > felt more or less unclear > - avs_ipc_wait_busy_completion() now spins up to 'repeats_left' number > of times before giving up > - 'adsp_ba' field of struct avs_dev has been renamed to 'dsp_ba' > - 'dops' field of struct avs_spec has been renamed to 'dsp_ops' > - IPC abstraction has been simplified: SKL_ADSP_REG_HIP* regs are used > directly > - fixed allnoconfig with AVS enabled compilation issues > - fixed code loading error paths: previously requested firmware is now > released before function return to the caller > - code and function arguments tied to D0IX support have been removed > from this patchset and will be part of followup series adding that > feature instead > - enriched dev_err() messages in avs_dsp_get_core() and > avs_dsp_put_core() > - numerous wording fixes used in power/reset/stall DSP operations
On 2022-02-28 6:23 PM, Pierre-Louis Bossart wrote: >> Changes v2 -> v1: >> Almost all updates here are thanks to feedback from Pierre. > > Can I ask for a bit of mercy on reviewers and specifically my time? I > sent my comments on Friday, I haven't had time to look at the answers > and there's a new version already? Hello, With all due respect, the v1 version of the series has been sent on 7th of Feb with most of its patches remaining rather unchanged since the RFC available on the list since 8th of Dec last year. Prior to that, there was also a period of time during which an internal review has been conducted. As majority of recent feedback was related to explanations and comments, I see no reason for not sending v2. Please note that the book is not finished with this chapter as the series does not provide a complete driver. There are several changes yet to come and code can, and probably will change in time. The team is not fleeing the list once these patches get merged either - we have been maintaining and supporting Intel solutions found in ASoC tree for a while now, and plan to continue to do so for years to come. Kind regards, Czarek
Sparse seems to have difficulties with your {0} initializations? export ARCH=x86_64 CF="-Wsparse-error -Wsparse-all -Wno-bitwise-pointer -Wno-pointer-arith -Wno-typesign -Wnoshadow" make sound/soc/intel/avs/ C=2 -k GEN Makefile CHECK /data/pbossart/ktest/broonie-next/scripts/mod/empty.c CALL /data/pbossart/ktest/broonie-next/scripts/checksyscalls.sh CALL /data/pbossart/ktest/broonie-next/scripts/atomic/check-atomics.sh DESCEND objtool CHECK /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/dsp.c CHECK /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/ipc.c CC [M] sound/soc/intel/avs/messages.o CHECK /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/messages.c /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/messages.c:18:39: error: missing braces around initializer /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/messages.c:72:39: error: missing braces around initializer /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/messages.c:90:39: error: missing braces around initializer /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/messages.c:110:39: error: missing braces around initializer /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/messages.c:127:39: error: missing braces around initializer /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/messages.c:145:39: error: missing braces around initializer /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/messages.c:146:37: error: missing braces around initializer /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/messages.c:225:39: error: missing braces around initializer /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/messages.c:255:39: error: missing braces around initializer /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/messages.c:289:39: error: missing braces around initializer /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/messages.c:381:37: error: missing braces around initializer /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/messages.c:452:39: error: missing braces around initializer make[4]: *** [/data/pbossart/ktest/broonie-next/scripts/Makefile.build:289: sound/soc/intel/avs/messages.o] Error 1 make[4]: *** Deleting file 'sound/soc/intel/avs/messages.o' CC [M] sound/soc/intel/avs/utils.o CHECK /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/utils.c CC [M] sound/soc/intel/avs/core.o CHECK /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/core.c CC [M] sound/soc/intel/avs/loader.o CHECK /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/loader.c /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/loader.c:339:47: error: Using plain integer as NULL pointer /data/pbossart/ktest/broonie-next/sound/soc/intel/avs/loader.c:405:47: error: Using plain integer as NULL pointer