str_id was unsigned, so check for (str_id <= 0) made no sense. Made it signed. Signed-off-by: Vasiliy Kulikov <segooon@xxxxxxxxx> --- Compile tested. drivers/staging/intel_sst/intel_sst_stream.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/staging/intel_sst/intel_sst_stream.c b/drivers/staging/intel_sst/intel_sst_stream.c index ff46d5c..1ce3a9c 100644 --- a/drivers/staging/intel_sst/intel_sst_stream.c +++ b/drivers/staging/intel_sst/intel_sst_stream.c @@ -117,7 +117,8 @@ int sst_alloc_stream(char *params, unsigned int stream_ops, { struct ipc_post *msg = NULL; struct snd_sst_alloc_params alloc_param; - unsigned int pcm_slot = 0, num_ch, str_id; + unsigned int pcm_slot = 0, num_ch; + int str_id; struct snd_sst_stream_params *sparams; struct stream_info *str_info; -- 1.7.0.4 -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html