Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1469561 > unrolled thread

[PATCH v2 1/2] ASoC: simple-card-utils: add __printf attribute

Started byNicolas Iooss <nicolas.iooss_linux@m4x.org>
First post2016-08-24 18:20 +0200
Last post2016-08-24 18:20 +0200
Articles 2 — 1 participant

Back to article view | Back to linux.kernel


Contents

  [PATCH v2 1/2] ASoC: simple-card-utils: add __printf attribute Nicolas Iooss <nicolas.iooss_linux@m4x.org> - 2016-08-24 18:20 +0200
    [PATCH v2 2/2] MAINTAINERS: document ASoC header files Nicolas Iooss <nicolas.iooss_linux@m4x.org> - 2016-08-24 18:20 +0200

#1469561 — [PATCH v2 1/2] ASoC: simple-card-utils: add __printf attribute

FromNicolas Iooss <nicolas.iooss_linux@m4x.org>
Date2016-08-24 18:20 +0200
Subject[PATCH v2 1/2] ASoC: simple-card-utils: add __printf attribute
Message-ID<s9J4d-4GN-15@gated-at.bofh.it>
asoc_simple_card_set_dailink_name() uses devm_kvasprintf() to format
some of its arguments.  Adding a __printf attribute to this function
makes it possible to detect at compile-time errors related to format
strings.

Signed-off-by: Nicolas Iooss <nicolas.iooss_linux@m4x.org>
---
 include/sound/simple_card_utils.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/include/sound/simple_card_utils.h b/include/sound/simple_card_utils.h
index 86088aed9002..3207b1a70d38 100644
--- a/include/sound/simple_card_utils.h
+++ b/include/sound/simple_card_utils.h
@@ -27,6 +27,7 @@ int asoc_simple_card_parse_daifmt(struct device *dev,
 				  struct device_node *codec,
 				  char *prefix,
 				  unsigned int *retfmt);
+__printf(3, 4)
 int asoc_simple_card_set_dailink_name(struct device *dev,
 				      struct snd_soc_dai_link *dai_link,
 				      const char *fmt, ...);
-- 
2.9.3

[toc] | [next] | [standalone]


#1469565 — [PATCH v2 2/2] MAINTAINERS: document ASoC header files

FromNicolas Iooss <nicolas.iooss_linux@m4x.org>
Date2016-08-24 18:20 +0200
Subject[PATCH v2 2/2] MAINTAINERS: document ASoC header files
Message-ID<s9J4e-4GN-27@gated-at.bofh.it>
In reply to#1469561
include/sound/simple_card_utils.h is handled by ASoC maintainers, as
stated in https://lkml.org/lkml/2016/8/22/307, and
include/sound/simple_card.h seems to be an ASoC file too. In the future
there will be more files named like these ones so introduce a pattern to
match them.

Signed-off-by: Nicolas Iooss <nicolas.iooss_linux@m4x.org>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 0bbe4b105c34..3f4e8bf042b9 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11012,6 +11012,7 @@ S:	Supported
 F:	Documentation/devicetree/bindings/sound/
 F:	Documentation/sound/alsa/soc/
 F:	sound/soc/
+F:	include/sound/simple*card*
 F:	include/sound/soc*
 
 SOUND - DMAENGINE HELPERS
-- 
2.9.3

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web