linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH RESEND] ASoC: pxa: remove unused variable
@ 2016-02-25 22:02 Arnd Bergmann
  2016-02-26  1:18 ` Mark Brown
  0 siblings, 1 reply; 3+ messages in thread
From: Arnd Bergmann @ 2016-02-25 22:02 UTC (permalink / raw)
  To: Mark Brown
  Cc: linux-arm-kernel, Arnd Bergmann, Daniel Mack, Haojian Zhuang,
	Robert Jarzmik, Liam Girdwood, Jaroslav Kysela, Takashi Iwai,
	alsa-devel, linux-kernel

As pointed out by Zhangfei Gao, the sspa_div variable in
brownstone_wm8994_hw_params() is completely unused, so as a cleanup
following a prior patch, this removes both the variable and the division.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
Submitted originally on Nov 23 2015, this apparently got lost,
resending it now.

 sound/soc/pxa/brownstone.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/sound/soc/pxa/brownstone.c b/sound/soc/pxa/brownstone.c
index 416ea646c3b1..ec522e94b0e2 100644
--- a/sound/soc/pxa/brownstone.c
+++ b/sound/soc/pxa/brownstone.c
@@ -52,7 +52,6 @@ static int brownstone_wm8994_hw_params(struct snd_pcm_substream *substream,
 	struct snd_soc_dai *codec_dai = rtd->codec_dai;
 	struct snd_soc_dai *cpu_dai = rtd->cpu_dai;
 	int freq_out, sspa_mclk, sysclk;
-	int sspa_div;
 
 	if (params_rate(params) > 11025) {
 		freq_out  = params_rate(params) * 512;
@@ -63,7 +62,6 @@ static int brownstone_wm8994_hw_params(struct snd_pcm_substream *substream,
 		sysclk    = params_rate(params) * 512;
 		sspa_mclk = params_rate(params) * 64;
 	}
-	sspa_div = freq_out / sspa_mclk;
 
 	snd_soc_dai_set_sysclk(cpu_dai, MMP_SSPA_CLK_AUDIO, freq_out, 0);
 	snd_soc_dai_set_pll(cpu_dai, MMP_SYSCLK, 0, freq_out, sysclk);
-- 
2.7.0

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH RESEND] ASoC: pxa: remove unused variable
  2016-02-25 22:02 [PATCH RESEND] ASoC: pxa: remove unused variable Arnd Bergmann
@ 2016-02-26  1:18 ` Mark Brown
  2016-02-26 13:46   ` Arnd Bergmann
  0 siblings, 1 reply; 3+ messages in thread
From: Mark Brown @ 2016-02-26  1:18 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: linux-arm-kernel, Daniel Mack, Haojian Zhuang, Robert Jarzmik,
	Liam Girdwood, Jaroslav Kysela, Takashi Iwai, alsa-devel,
	linux-kernel

[-- Attachment #1: Type: text/plain, Size: 695 bytes --]

On Thu, Feb 25, 2016 at 11:02:04PM +0100, Arnd Bergmann wrote:

> Submitted originally on Nov 23 2015, this apparently got lost,
> resending it now.

You didn't really submit it, you pasted it into the middle of a reply to
a message on an existing thread for a patch that was already applied:

https://lkml.org/lkml/2015/11/23/88

That's not really submitting a patch as far as I'm concerned, something
that won't just apply with git am without special faffing isn't helping
and things in the middle of threads aren't good either (it gets
difficult to follow and there's every chance they'll never even get seen
if they're buried in the middle of a reply where you don't expect to see
a patch).

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH RESEND] ASoC: pxa: remove unused variable
  2016-02-26  1:18 ` Mark Brown
@ 2016-02-26 13:46   ` Arnd Bergmann
  0 siblings, 0 replies; 3+ messages in thread
From: Arnd Bergmann @ 2016-02-26 13:46 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: Mark Brown, alsa-devel, linux-kernel, Takashi Iwai,
	Liam Girdwood, Haojian Zhuang, Jaroslav Kysela, Robert Jarzmik,
	Daniel Mack

On Friday 26 February 2016 10:18:27 Mark Brown wrote:
> On Thu, Feb 25, 2016 at 11:02:04PM +0100, Arnd Bergmann wrote:
> 
> > Submitted originally on Nov 23 2015, this apparently got lost,
> > resending it now.
> 
> You didn't really submit it, you pasted it into the middle of a reply to
> a message on an existing thread for a patch that was already applied:
> 
> https://lkml.org/lkml/2015/11/23/88
> 
> That's not really submitting a patch as far as I'm concerned, something
> that won't just apply with git am without special faffing isn't helping
> and things in the middle of threads aren't good either (it gets
> difficult to follow and there's every chance they'll never even get seen
> if they're buried in the middle of a reply where you don't expect to see
> a patch).

I see your point about the patch being in the middle of the thread.

Regarding the 'git am' compatibility, I thought I'd done it right,
but see now that I used incorrect scissors, apparently 'git mailinfo'
expects at least six '-' characters as the separator, while I used
just three.

	Arnd

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-02-26 13:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-25 22:02 [PATCH RESEND] ASoC: pxa: remove unused variable Arnd Bergmann
2016-02-26  1:18 ` Mark Brown
2016-02-26 13:46   ` Arnd Bergmann

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).