From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Subject: [PATCH] ASoC: ssm2518: Use a signed return type for ssm2518_lookup_mcs() Date: Sun, 20 Dec 2015 10:40:24 +0100 Message-ID: <56767788.4020709@users.sourceforge.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mout.web.de (mout.web.de [212.227.15.4]) by alsa0.perex.cz (Postfix) with ESMTP id BE0F92606C9 for ; Sun, 20 Dec 2015 10:40:40 +0100 (CET) List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: alsa-devel@alsa-project.org, Jaroslav Kysela , Lars-Peter Clausen , Liam Girdwood , Mark Brown , Takashi Iwai Cc: Julia Lawall , kernel-janitors@vger.kernel.org, LKML List-Id: alsa-devel@alsa-project.org From: Markus Elfring Date: Sun, 20 Dec 2015 10:34:25 +0100 The return type "unsigned int" was used by the ssm2518_lookup_mcs() function even though it will eventually return a negative error code. Improve this implementation detail by deletion of the type modifier then. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- sound/soc/codecs/ssm2518.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/ssm2518.c b/sound/soc/codecs/ssm2518.c index 86b81a6..e2e0bfa 100644 --- a/sound/soc/codecs/ssm2518.c +++ b/sound/soc/codecs/ssm2518.c @@ -309,7 +309,7 @@ static const struct snd_pcm_hw_constraint_list ssm2518_constraints_12288000 = { .count = ARRAY_SIZE(ssm2518_rates_12288000), }; -static unsigned int ssm2518_lookup_mcs(struct ssm2518 *ssm2518, +static int ssm2518_lookup_mcs(struct ssm2518 *ssm2518, unsigned int rate) { const unsigned int *sysclks = NULL; -- 2.6.3 From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Date: Sun, 20 Dec 2015 09:40:24 +0000 Subject: [PATCH] ASoC: ssm2518: Use a signed return type for ssm2518_lookup_mcs() Message-Id: <56767788.4020709@users.sourceforge.net> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: alsa-devel@alsa-project.org, Jaroslav Kysela , Lars-Peter Clausen , Liam Girdwood , Mark Brown , Takashi Iwai Cc: Julia Lawall , kernel-janitors@vger.kernel.org, LKML From: Markus Elfring Date: Sun, 20 Dec 2015 10:34:25 +0100 The return type "unsigned int" was used by the ssm2518_lookup_mcs() function even though it will eventually return a negative error code. Improve this implementation detail by deletion of the type modifier then. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- sound/soc/codecs/ssm2518.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/ssm2518.c b/sound/soc/codecs/ssm2518.c index 86b81a6..e2e0bfa 100644 --- a/sound/soc/codecs/ssm2518.c +++ b/sound/soc/codecs/ssm2518.c @@ -309,7 +309,7 @@ static const struct snd_pcm_hw_constraint_list ssm2518_constraints_12288000 = { .count = ARRAY_SIZE(ssm2518_rates_12288000), }; -static unsigned int ssm2518_lookup_mcs(struct ssm2518 *ssm2518, +static int ssm2518_lookup_mcs(struct ssm2518 *ssm2518, unsigned int rate) { const unsigned int *sysclks = NULL; -- 2.6.3 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754299AbbLTJkw (ORCPT ); Sun, 20 Dec 2015 04:40:52 -0500 Received: from mout.web.de ([212.227.15.4]:54096 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754186AbbLTJkq (ORCPT ); Sun, 20 Dec 2015 04:40:46 -0500 To: alsa-devel@alsa-project.org, Jaroslav Kysela , Lars-Peter Clausen , Liam Girdwood , Mark Brown , Takashi Iwai Cc: LKML , kernel-janitors@vger.kernel.org, Julia Lawall From: SF Markus Elfring Subject: [PATCH] ASoC: ssm2518: Use a signed return type for ssm2518_lookup_mcs() Message-ID: <56767788.4020709@users.sourceforge.net> Date: Sun, 20 Dec 2015 10:40:24 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:mXaV/SgHZkQn1e4wVsdAU6Exdad4diXvHkhD6/umd4siIP1Cap8 VbfxNzuflIJwp7omaAZEIKSWozkd11uiup8WkahGnHgK6Gv5ADmAsQ+pgq00Afh8EbMjPYZ rWTs6O9Wq2/n84NaYAigu9AhnF6pycoKK5ZiD1asUHroXy7p4ZPKbBb4rDAv7URhs6Fdzxh pjiqLT8OWz+Sfln21h0ig== X-UI-Out-Filterresults: notjunk:1;V01:K0:HlKbUleWP5A=:6frTLTMH0Ig3+3Q5sifLpc gVLd7Pq9msaxmnKCke50Z00PuDimBWl5QFQ0Yzk3AWsZYf+9H2TPeDj6YAESrzl00v8OqZTO3 TK1Khk6iBS8Os3aQnplR6N6vyLMxz17A+2JoYgx/Wwc38u9fsIKmWwcoKimvZZSHPkSvqNNP1 wZ7IH4++k2pvh0fnfqpvn6S4X7ZDT7DbkEA1heoYWZS18ijy/0w9IRPIacQDuBtrjqDtCC6ZJ YdXKWWXo+PRa3k6i/UjKg+KmHKHtl0Rhf+Y63cVZ6gQXoh/98euvzBDbSGjO/8cAl6+RDC1LP 8o1bqgkhlFYzx73bOnucpm0Lb/JzTTYuLBafXgqMyS2yUnhDQYl9BF2N1mFQ1EYlB7sg7iJOY ke/JY+TaHpiMaP4MMxcVj0UCCa/cdmJDuti3Hxzr4RdiQz17wo+d43M7ayn902ktzwx8cnPhD qp9Y9Ulv1p71aGieCjLCYHL1DmEkSQpH9OEnh62tGQaqvtmDT/b1wwgc6/m5AIpmxPiGqVvqa l+xMRFZa9FtViBXWKIv+iLfLpuSAfxm+IItZhRQZS1ejuzEzq9c6vmtKuSPPXiSChutP7v8tj VTdfmc+hwH2OJlZuKgkNJwZ6oYrvYUXfsjSyN/fwyo7KfJjQtuRiDrCGe/G8X2tFILgjARdeo HZBCQOe07CULYeN6h6OVzkgtbU3oZrdM4EmyjG1prN8/mgwgmATM38Z3FQc83bbI68GKVm1bY F4hHxw6y9dxUhEl5cOagawsISyM3tU5uX1tHf1/tJMz2zFtskxVrmFp5NQp1vD0m7oghjYQdi Kwtre/t Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Sun, 20 Dec 2015 10:34:25 +0100 The return type "unsigned int" was used by the ssm2518_lookup_mcs() function even though it will eventually return a negative error code. Improve this implementation detail by deletion of the type modifier then. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- sound/soc/codecs/ssm2518.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sound/soc/codecs/ssm2518.c b/sound/soc/codecs/ssm2518.c index 86b81a6..e2e0bfa 100644 --- a/sound/soc/codecs/ssm2518.c +++ b/sound/soc/codecs/ssm2518.c @@ -309,7 +309,7 @@ static const struct snd_pcm_hw_constraint_list ssm2518_constraints_12288000 = { .count = ARRAY_SIZE(ssm2518_rates_12288000), }; -static unsigned int ssm2518_lookup_mcs(struct ssm2518 *ssm2518, +static int ssm2518_lookup_mcs(struct ssm2518 *ssm2518, unsigned int rate) { const unsigned int *sysclks = NULL; -- 2.6.3