* [PATCH] ASoC: fsl_micfil: Fix format and unused assignment
@ 2020-05-18 7:44 Tang Bin
2020-05-18 10:25 ` Mark Brown
0 siblings, 1 reply; 7+ messages in thread
From: Tang Bin @ 2020-05-18 7:44 UTC (permalink / raw)
To: broonie, timur, nicoleotsuka, Xiubo.Lee, perex
Cc: alsa-devel, linuxppc-dev, linux-kernel, Tang Bin
In the function fsl_micfil_startup(), the two lines of dev_err()
can be shortened to one line. And delete unused initialized value
of 'ret', because it will be assigned by the function
fsl_micfil_set_mclk_rate().
Signed-off-by: Tang Bin <tangbin@cmss.chinamobile.com>
---
sound/soc/fsl/fsl_micfil.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/sound/soc/fsl/fsl_micfil.c b/sound/soc/fsl/fsl_micfil.c
index f7f2d29f1..a7a6118be 100644
--- a/sound/soc/fsl/fsl_micfil.c
+++ b/sound/soc/fsl/fsl_micfil.c
@@ -217,8 +217,7 @@ static int fsl_micfil_startup(struct snd_pcm_substream *substream,
struct fsl_micfil *micfil = snd_soc_dai_get_drvdata(dai);
if (!micfil) {
- dev_err(dai->dev,
- "micfil dai priv_data not set\n");
+ dev_err(dai->dev, "micfil dai priv_data not set\n");
return -EINVAL;
}
@@ -296,7 +295,7 @@ static int fsl_set_clock_params(struct device *dev, unsigned int rate)
{
struct fsl_micfil *micfil = dev_get_drvdata(dev);
int clk_div;
- int ret = 0;
+ int ret;
ret = fsl_micfil_set_mclk_rate(micfil, rate);
if (ret < 0)
--
2.20.1.windows.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] ASoC: fsl_micfil: Fix format and unused assignment
2020-05-18 7:44 [PATCH] ASoC: fsl_micfil: Fix format and unused assignment Tang Bin
2020-05-18 10:25 ` Mark Brown
@ 2020-05-18 10:25 ` Mark Brown
0 siblings, 0 replies; 7+ messages in thread
From: Mark Brown @ 2020-05-18 10:25 UTC (permalink / raw)
To: Tang Bin
Cc: timur, nicoleotsuka, Xiubo.Lee, perex, alsa-devel, linuxppc-dev,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 377 bytes --]
On Mon, May 18, 2020 at 03:44:05PM +0800, Tang Bin wrote:
> In the function fsl_micfil_startup(), the two lines of dev_err()
> can be shortened to one line. And delete unused initialized value
> of 'ret', because it will be assigned by the function
> fsl_micfil_set_mclk_rate().
This is two separate changes with no overlap so would have been better
sent as separate patches.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ASoC: fsl_micfil: Fix format and unused assignment
@ 2020-05-18 10:25 ` Mark Brown
0 siblings, 0 replies; 7+ messages in thread
From: Mark Brown @ 2020-05-18 10:25 UTC (permalink / raw)
To: Tang Bin
Cc: alsa-devel, timur, Xiubo.Lee, linux-kernel, nicoleotsuka, linuxppc-dev
[-- Attachment #1: Type: text/plain, Size: 377 bytes --]
On Mon, May 18, 2020 at 03:44:05PM +0800, Tang Bin wrote:
> In the function fsl_micfil_startup(), the two lines of dev_err()
> can be shortened to one line. And delete unused initialized value
> of 'ret', because it will be assigned by the function
> fsl_micfil_set_mclk_rate().
This is two separate changes with no overlap so would have been better
sent as separate patches.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ASoC: fsl_micfil: Fix format and unused assignment
@ 2020-05-18 10:25 ` Mark Brown
0 siblings, 0 replies; 7+ messages in thread
From: Mark Brown @ 2020-05-18 10:25 UTC (permalink / raw)
To: Tang Bin
Cc: alsa-devel, timur, Xiubo.Lee, linux-kernel, perex, nicoleotsuka,
linuxppc-dev
[-- Attachment #1: Type: text/plain, Size: 377 bytes --]
On Mon, May 18, 2020 at 03:44:05PM +0800, Tang Bin wrote:
> In the function fsl_micfil_startup(), the two lines of dev_err()
> can be shortened to one line. And delete unused initialized value
> of 'ret', because it will be assigned by the function
> fsl_micfil_set_mclk_rate().
This is two separate changes with no overlap so would have been better
sent as separate patches.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ASoC: fsl_micfil: Fix format and unused assignment
2020-05-18 10:25 ` Mark Brown
(?)
@ 2020-05-18 10:47 ` Tang Bin
-1 siblings, 0 replies; 7+ messages in thread
From: Tang Bin @ 2020-05-18 10:47 UTC (permalink / raw)
To: Mark Brown
Cc: timur, nicoleotsuka, Xiubo.Lee, perex, alsa-devel, linuxppc-dev,
linux-kernel
On 2020/5/18 18:25, Mark Brown wrote:
> On Mon, May 18, 2020 at 03:44:05PM +0800, Tang Bin wrote:
>> In the function fsl_micfil_startup(), the two lines of dev_err()
>> can be shortened to one line. And delete unused initialized value
>> of 'ret', because it will be assigned by the function
>> fsl_micfil_set_mclk_rate().
> This is two separate changes with no overlap so would have been better
> sent as separate patches.
Got it, Thanks
Tang Bin
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ASoC: fsl_micfil: Fix format and unused assignment
@ 2020-05-18 10:47 ` Tang Bin
0 siblings, 0 replies; 7+ messages in thread
From: Tang Bin @ 2020-05-18 10:47 UTC (permalink / raw)
To: Mark Brown
Cc: alsa-devel, timur, Xiubo.Lee, linux-kernel, nicoleotsuka, linuxppc-dev
On 2020/5/18 18:25, Mark Brown wrote:
> On Mon, May 18, 2020 at 03:44:05PM +0800, Tang Bin wrote:
>> In the function fsl_micfil_startup(), the two lines of dev_err()
>> can be shortened to one line. And delete unused initialized value
>> of 'ret', because it will be assigned by the function
>> fsl_micfil_set_mclk_rate().
> This is two separate changes with no overlap so would have been better
> sent as separate patches.
Got it, Thanks
Tang Bin
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ASoC: fsl_micfil: Fix format and unused assignment
@ 2020-05-18 10:47 ` Tang Bin
0 siblings, 0 replies; 7+ messages in thread
From: Tang Bin @ 2020-05-18 10:47 UTC (permalink / raw)
To: Mark Brown
Cc: alsa-devel, timur, Xiubo.Lee, linux-kernel, perex, nicoleotsuka,
linuxppc-dev
On 2020/5/18 18:25, Mark Brown wrote:
> On Mon, May 18, 2020 at 03:44:05PM +0800, Tang Bin wrote:
>> In the function fsl_micfil_startup(), the two lines of dev_err()
>> can be shortened to one line. And delete unused initialized value
>> of 'ret', because it will be assigned by the function
>> fsl_micfil_set_mclk_rate().
> This is two separate changes with no overlap so would have been better
> sent as separate patches.
Got it, Thanks
Tang Bin
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-05-18 10:48 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-18 7:44 [PATCH] ASoC: fsl_micfil: Fix format and unused assignment Tang Bin
2020-05-18 10:25 ` Mark Brown
2020-05-18 10:25 ` Mark Brown
2020-05-18 10:25 ` Mark Brown
2020-05-18 10:47 ` Tang Bin
2020-05-18 10:47 ` Tang Bin
2020-05-18 10:47 ` Tang Bin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.