Re: [PATCH] ASoC: SOF: mediatek: mt8195: remove a redundant comparison of sram

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: YC Hung <yc.hung@xxxxxxxxxxxx>
Subject: Re: [PATCH] ASoC: SOF: mediatek: mt8195: remove a redundant comparison of sram
From: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx>
Date: Thu, 15 Dec 2022 11:31:10 +0100
Cc: Daniel Baluta <daniel.baluta@xxxxxxx>, alsa-devel@xxxxxxxxxxxxxxxx, Chunxu Li <chunxu.li@xxxxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, Allen-KH Cheng <Allen-KH.Cheng@xxxxxxxxxxxx>, Kai Vehmanen <kai.vehmanen@xxxxxxxxxxxxxxx>, Liam Girdwood <lgirdwood@xxxxxxxxx>, Peter Ujfalusi <peter.ujfalusi@xxxxxxxxxxxxxxx>, Takashi Iwai <tiwai@xxxxxxxx>, Ranjani Sridharan <ranjani.sridharan@xxxxxxxxxxxxxxx>, Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx>, Mark Brown <broonie@xxxxxxxxxx>, linux-mediatek@xxxxxxxxxxxxxxxxxxx, Matthias Brugger <matthias.bgg@xxxxxxxxx>, Curtis Malainey <cujomalainey@xxxxxxxxxxxx>, Bard Liao <yung-chuan.liao@xxxxxxxxxxxxxxx>, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, sound-open-firmware@xxxxxxxxxxxxxxxx
In-reply-to: <20221215061046.16934-1-yc.hung@mediatek.com>
References: <20221215061046.16934-1-yc.hung@mediatek.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.6.0

Il 15/12/22 07:10, YC Hung ha scritto:

DSP SRAM is not used for audio shared buffer between host and DSP so
TOTAL_SIZE_SHARED_SRAM_FROM_TAIL is zero. Remove the definition and
redundant comparison to fix coverity "unsigned compared against 0".

Signed-off-by: YC Hung <yc.hung@xxxxxxxxxxxx>


Reviewed-by: AngeloGioacchino Del Regno <angeloigoacchino.delregno@xxxxxxxxxxxxx>

References:
- [PATCH] ASoC: SOF: mediatek: mt8195: remove a redundant comparison of sram
  - From: YC Hung

Prev by Date: Re: [PATCH v2 2/3] ASoC: wm8940: Rewrite code to set proper clocks
Next by Date: Re: [PATCH 06/11] ASoC: dt-bindings: ti, pcm3168a: Convert to json-schema
Previous by thread: [PATCH] ASoC: SOF: mediatek: mt8195: remove a redundant comparison of sram
Next by thread: Re: [PATCH] ASoC: SOF: mediatek: mt8195: remove a redundant comparison of sram
Index(es):
- Date
- Thread

[Index of Archives] [ALSA User] [Linux Audio Users] [Pulse Audio] [Kernel Archive] [Asterisk PBX] [Photo Sharing] [Linux Sound] [Video 4 Linux] [Gimp] [Yosemite News]