[PATCH] ASoC: tlv320aic31xx: Staticise non-exported symbols

[PATCH] ASoC: tlv320aic31xx: Staticise non-exported symbols

[Mark Brown]

From: Mark Brown <broonie@linaro.org>

Signed-off-by: Mark Brown <broonie@linaro.org>
---
 sound/soc/codecs/tlv320aic31xx.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/sound/soc/codecs/tlv320aic31xx.c b/sound/soc/codecs/tlv320aic31xx.c
index dcdc5751048f..d3517a919776 100644
--- a/sound/soc/codecs/tlv320aic31xx.c
+++ b/sound/soc/codecs/tlv320aic31xx.c
@@ -129,7 +129,7 @@ static const struct regmap_range_cfg aic31xx_ranges[] = {
 	},
 };
 
-struct regmap_config aic31xx_i2c_regmap = {
+static const struct regmap_config aic31xx_i2c_regmap = {
 	.reg_bits = 8,
 	.val_bits = 8,
 	.writeable_reg = aic31xx_writeable,
@@ -321,9 +321,9 @@ static const struct snd_kcontrol_new ldac_in_control =
 static const struct snd_kcontrol_new rdac_in_control =
 	SOC_DAPM_ENUM("DAC Right Input", rdac_in_enum);
 
-int aic31xx_wait_bits(struct aic31xx_priv *aic31xx, unsigned int reg,
-		      unsigned int mask, unsigned int wbits, int sleep,
-		      int count)
+static int aic31xx_wait_bits(struct aic31xx_priv *aic31xx, unsigned int reg,
+			     unsigned int mask, unsigned int wbits, int sleep,
+			     int count)
 {
 	unsigned int bits;
 	int counter = count;
@@ -1177,7 +1177,7 @@ static void aic31xx_pdata_from_of(struct aic31xx_priv *aic31xx)
 }
 #endif /* CONFIG_OF */
 
-void aic31xx_device_init(struct aic31xx_priv *aic31xx)
+static void aic31xx_device_init(struct aic31xx_priv *aic31xx)
 {
 	int ret, i;
 
-- 
1.9.0

Re: [PATCH] ASoC: tlv320aic31xx: Staticise non-exported symbols

[Jyri Sarha]

On 03/13/2014 07:48 PM, Mark Brown wrote:
> From: Mark Brown <broonie@linaro.org>
>
> Signed-off-by: Mark Brown <broonie@linaro.org>


Acked-by: Jyri Sarha <jsarha@it.com>

/me has added one more thing to his patch sending check list.

Best reggards,
Jyri

Re: [PATCH] ASoC: tlv320aic31xx: Staticise non-exported symbols

[Lars-Peter Clausen]

On 03/14/2014 09:09 AM, Jyri Sarha wrote:
> On 03/13/2014 07:48 PM, Mark Brown wrote:
>> From: Mark Brown <broonie@linaro.org>
>>
>> Signed-off-by: Mark Brown <broonie@linaro.org>
>
>
> Acked-by: Jyri Sarha <jsarha@it.com>
>
> /me has added one more thing to his patch sending check list.

What I typically try to do before submitting a patch is run

make C=2 .../driver.o
make C=2 CHECK=scripts/coccicheck .../driver.o
make C=2 CHECK="smatch --full-path -p=kernel" .../driver.o

that will typically catch these little mistakes. Also of course 
scripts/checkpatch.pl on the patch itself.

- Lars

Re: [PATCH] ASoC: tlv320aic31xx: Staticise non-exported symbols

[Mark Brown]

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

On Fri, Mar 14, 2014 at 09:31:44AM +0100, Lars-Peter Clausen wrote:
> On 03/14/2014 09:09 AM, Jyri Sarha wrote:

> >/me has added one more thing to his patch sending check list.

> What I typically try to do before submitting a patch is run

> make C=2 .../driver.o
> make C=2 CHECK=scripts/coccicheck .../driver.o
> make C=2 CHECK="smatch --full-path -p=kernel" .../driver.o

> that will typically catch these little mistakes. Also of course
> scripts/checkpatch.pl on the patch itself.

Make sure you're running against the latest development code when you do
this too.

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

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