| Message ID | 1619345553-29781-1-git-send-email-yang.lee@linux.alibaba.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | ASoC: q6dsp: q6afe: remove unneeded dead-store initialization | expand |
On Sun, Apr 25, 2021 at 3:12 AM Yang Li <yang.lee@linux.alibaba.com> wrote: > > Variables 'wait' and 'port_id' are being initialized, however the > values are never read and updated later on, hence the redundant > initializations can be removed. > > Cleans up clang warnings: > sound/soc/qcom/qdsp6/q6afe.c:933:21: warning: Value stored to 'wait' > during its initialization is never read > sound/soc/qcom/qdsp6/q6afe.c:1186:6: warning: Value stored to 'port_id' > during its initialization is never read > > Reported-by: Abaci Robot <abaci@linux.alibaba.com> > Signed-off-by: Yang Li <yang.lee@linux.alibaba.com> Thanks for the patch, but it looks like this has already been fixed in: https://lore.kernel.org/alsa-devel/20210218222916.89809-5-pierre-louis.bossart@linux.intel.com/ > --- > sound/soc/qcom/qdsp6/q6afe.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/sound/soc/qcom/qdsp6/q6afe.c b/sound/soc/qcom/qdsp6/q6afe.c > index cad1cd1..442bf27 100644 > --- a/sound/soc/qcom/qdsp6/q6afe.c > +++ b/sound/soc/qcom/qdsp6/q6afe.c > @@ -930,7 +930,7 @@ int q6afe_get_port_id(int index) > static int afe_apr_send_pkt(struct q6afe *afe, struct apr_pkt *pkt, > struct q6afe_port *port, uint32_t rsp_opcode) > { > - wait_queue_head_t *wait = &port->wait; > + wait_queue_head_t *wait; > struct aprv2_ibasic_rsp_result_t *result; > int ret; > > @@ -1183,7 +1183,7 @@ int q6afe_port_stop(struct q6afe_port *port) > struct afe_port_cmd_device_stop *stop; > struct q6afe *afe = port->afe; > struct apr_pkt *pkt; > - int port_id = port->id; > + int port_id; > int ret = 0; > int index, pkt_size; > void *p; > -- > 1.8.3.1 >
On Sun, 25 Apr 2021 18:12:33 +0800, Yang Li wrote: > Variables 'wait' and 'port_id' are being initialized, however the > values are never read and updated later on, hence the redundant > initializations can be removed. > > Cleans up clang warnings: > sound/soc/qcom/qdsp6/q6afe.c:933:21: warning: Value stored to 'wait' > during its initialization is never read > sound/soc/qcom/qdsp6/q6afe.c:1186:6: warning: Value stored to 'port_id' > during its initialization is never read Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next Thanks! [1/1] ASoC: q6dsp: q6afe: remove unneeded dead-store initialization commit: 5f1b95d08de712327e452d082a50fded435ec884 All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark
diff --git a/sound/soc/qcom/qdsp6/q6afe.c b/sound/soc/qcom/qdsp6/q6afe.c index cad1cd1..442bf27 100644 --- a/sound/soc/qcom/qdsp6/q6afe.c +++ b/sound/soc/qcom/qdsp6/q6afe.c @@ -930,7 +930,7 @@ int q6afe_get_port_id(int index) static int afe_apr_send_pkt(struct q6afe *afe, struct apr_pkt *pkt, struct q6afe_port *port, uint32_t rsp_opcode) { - wait_queue_head_t *wait = &port->wait; + wait_queue_head_t *wait; struct aprv2_ibasic_rsp_result_t *result; int ret; @@ -1183,7 +1183,7 @@ int q6afe_port_stop(struct q6afe_port *port) struct afe_port_cmd_device_stop *stop; struct q6afe *afe = port->afe; struct apr_pkt *pkt; - int port_id = port->id; + int port_id; int ret = 0; int index, pkt_size; void *p;
Variables 'wait' and 'port_id' are being initialized, however the values are never read and updated later on, hence the redundant initializations can be removed. Cleans up clang warnings: sound/soc/qcom/qdsp6/q6afe.c:933:21: warning: Value stored to 'wait' during its initialization is never read sound/soc/qcom/qdsp6/q6afe.c:1186:6: warning: Value stored to 'port_id' during its initialization is never read Reported-by: Abaci Robot <abaci@linux.alibaba.com> Signed-off-by: Yang Li <yang.lee@linux.alibaba.com> --- sound/soc/qcom/qdsp6/q6afe.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)