[PATCH 03/12] trivial: ASoC: pxa: pxa-ssp.c: indent with TABs, not spaces

[PATCH 03/12] trivial: ASoC: pxa: pxa-ssp.c: indent with TABs, not spaces

[Antonio Ospite]

Signed-off-by: Antonio Ospite <ao2@ao2.it>
Cc: Daniel Mack <daniel@zonque.org>
Cc: Mark Brown <broonie@kernel.com>
Cc: alsa-devel@alsa-project.org
---
 sound/soc/pxa/pxa-ssp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/pxa/pxa-ssp.c b/sound/soc/pxa/pxa-ssp.c
index fbe2e93..eb181ab 100644
--- a/sound/soc/pxa/pxa-ssp.c
+++ b/sound/soc/pxa/pxa-ssp.c
@@ -586,7 +586,7 @@ static int pxa_ssp_hw_params(struct snd_pcm_substream *substream,
 
 	switch (priv->dai_fmt & SND_SOC_DAIFMT_FORMAT_MASK) {
 	case SND_SOC_DAIFMT_I2S:
-	       sspsp = pxa_ssp_read_reg(ssp, SSPSP);
+		sspsp = pxa_ssp_read_reg(ssp, SSPSP);
 
 		if ((pxa_ssp_get_scr(ssp) == 4) && (width == 16)) {
 			/* This is a special case where the bitclk is 64fs
-- 
2.1.4

[PATCH 05/12] trivial: ASoC: adau1977: fix typo s/Substraction/Subtraction/

[Antonio Ospite]

Signed-off-by: Antonio Ospite <ao2@ao2.it>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Mark Brown <broonie@kernel.org>
Cc: alsa-devel@alsa-project.org
---
 sound/soc/codecs/adau1977.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/adau1977.c b/sound/soc/codecs/adau1977.c
index 7ad8e15..dc8ad08 100644
--- a/sound/soc/codecs/adau1977.c
+++ b/sound/soc/codecs/adau1977.c
@@ -202,7 +202,7 @@ static const struct snd_soc_dapm_route adau1977_dapm_routes[] = {
 		ADAU1977_REG_DC_HPF_CAL, (x) - 1, 1, 0)
 
 #define ADAU1977_DC_SUB_SWITCH(x) \
-	SOC_SINGLE("ADC" #x " DC Substraction Capture Switch", \
+	SOC_SINGLE("ADC" #x " DC Subtraction Capture Switch", \
 		ADAU1977_REG_DC_HPF_CAL, (x) + 3, 1, 0)
 
 static const struct snd_kcontrol_new adau1977_snd_controls[] = {
-- 
2.1.4

Re: [PATCH 05/12] trivial: ASoC: adau1977: fix typo s/Substraction/Subtraction/

[Lars-Peter Clausen]

On 04/28/2015 01:11 PM, Antonio Ospite wrote:
> Signed-off-by: Antonio Ospite <ao2@ao2.it>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: alsa-devel@alsa-project.org

Acked-by: Lars-Peter Clausen <lars@metafoo.de>

Thanks.

Re: [PATCH 05/12] trivial: ASoC: adau1977: fix typo s/Substraction/Subtraction/

[Mark Brown]

[-- Attachment #1.1: Type: text/plain, Size: 236 bytes --]

On Tue, Apr 28, 2015 at 01:11:24PM +0200, Antonio Ospite wrote:
> Signed-off-by: Antonio Ospite <ao2@ao2.it>

Applied, but please use subject lines reflecting the style for the
subsystem and don't send only a single patch for a series.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]

Re: [PATCH 05/12] trivial: ASoC: adau1977: fix typo s/Substraction/Subtraction/

[Antonio Ospite]

On Tue, 28 Apr 2015 17:00:24 +0100
Mark Brown <broonie@kernel.org> wrote:

> On Tue, Apr 28, 2015 at 01:11:24PM +0200, Antonio Ospite wrote:
> > Signed-off-by: Antonio Ospite <ao2@ao2.it>
> 
> Applied, but please use subject lines reflecting the style for the
> subsystem and don't send only a single patch for a series.

Hi Mark,

I tried to follow the subject style for the subsystem, but then I
prefixed it with "trivial" to highlight that I was sending an aggregate
set of unrelated patches to trivial@kernel.org, that's also why I didn't
want to CC everyone on every patch: to reduce noise.

Maybe involving trivial@kernel.org for changes which apply to a
_specific_ subsystem is not necessary at all, next time I will just
aggregate by subsystem.

As a rule of thumb I will use trivial@kernel.org only for trivial
changes which span different subsystems and still make sense to have in
a single patch.

Thanks for making me notice that.

Ciao,
   Antonio

-- 
Antonio Ospite
http://ao2.it

A: Because it messes up the order in which people normally read text.
   See http://en.wikipedia.org/wiki/Posting_style
Q: Why is top-posting such a bad thing?

Re: [PATCH 05/12] trivial: ASoC: adau1977: fix typo s/Substraction/Subtraction/

[Mark Brown]

[-- Attachment #1.1: Type: text/plain, Size: 839 bytes --]

On Tue, Apr 28, 2015 at 11:27:58PM +0200, Antonio Ospite wrote:

> I tried to follow the subject style for the subsystem, but then I
> prefixed it with "trivial" to highlight that I was sending an aggregate
> set of unrelated patches to trivial@kernel.org, that's also why I didn't
> want to CC everyone on every patch: to reduce noise.

That's not really helpful - it means that the patch is more likely to be
missed and that the patch sticks out in the changelog.

> Maybe involving trivial@kernel.org for changes which apply to a
> _specific_ subsystem is not necessary at all, next time I will just
> aggregate by subsystem.

That's generally better, the expectation should be that maintainers are
going to apply even trivial patches and it does make life easier if
things go that way (less chance of conflicting changes for example).

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]