[PATCH 1/2] remoteproc: remoteproc_core: Use 'gnu_printf' format notation

From: Lee Jones
Date: Wed Jul 15 2020 - 08:35:58 EST


Fixes the following W=1 kernel build warning(s):

drivers/remoteproc/remoteproc_core.c: In function ârproc_find_carveout_by_nameâ:
drivers/remoteproc/remoteproc_core.c:257:2: warning: function ârproc_find_carveout_by_nameâ might be a candidate for âgnu_printfâ format attribute [-Wsuggest-attribute=format]
257 | vsnprintf(_name, sizeof(_name), name, args);
| ^~~~~~~~~
drivers/remoteproc/remoteproc_core.c: In function ârproc_mem_entry_initâ:
drivers/remoteproc/remoteproc_core.c:993:2: warning: function ârproc_mem_entry_initâ might be a candidate for âgnu_printfâ format attribute [-Wsuggest-attribute=format]
993 | vsnprintf(mem->name, sizeof(mem->name), name, args);
| ^~~~~~~~~
drivers/remoteproc/remoteproc_core.c: In function ârproc_of_resm_mem_entry_initâ:
drivers/remoteproc/remoteproc_core.c:1029:2: warning: function ârproc_of_resm_mem_entry_initâ might be a candidate for âgnu_printfâ format attribute [-Wsuggest-attribute=format]
1029 | vsnprintf(mem->name, sizeof(mem->name), name, args);
| ^~~~~~~~~

Cc: Brian Swetland <swetland@xxxxxxxxxx>
Cc: Mark Grosen <mgrosen@xxxxxx>
Cc: Guzman Lugo <fernando.lugo@xxxxxx>
Cc: Suman Anna <s-anna@xxxxxx>
Cc: Robert Tivy <rtivy@xxxxxx>
Cc: De Leon <x0095078@xxxxxx>
Signed-off-by: Lee Jones <lee.jones@xxxxxxxxxx>
---
drivers/remoteproc/remoteproc_core.c | 3 +++
1 file changed, 3 insertions(+)

diff --git a/drivers/remoteproc/remoteproc_core.c b/drivers/remoteproc/remoteproc_core.c
index 0f95e025ba030..e00dff425c96e 100644
--- a/drivers/remoteproc/remoteproc_core.c
+++ b/drivers/remoteproc/remoteproc_core.c
@@ -243,6 +243,7 @@ EXPORT_SYMBOL(rproc_da_to_va);
*
* Return: a valid pointer on carveout entry on success or NULL on failure.
*/
+__printf(2, 3)
struct rproc_mem_entry *
rproc_find_carveout_by_name(struct rproc *rproc, const char *name, ...)
{
@@ -966,6 +967,7 @@ EXPORT_SYMBOL(rproc_add_carveout);
* This function allocates a rproc_mem_entry struct and fill it with parameters
* provided by client.
*/
+__printf(8, 9)
struct rproc_mem_entry *
rproc_mem_entry_init(struct device *dev,
void *va, dma_addr_t dma, size_t len, u32 da,
@@ -1009,6 +1011,7 @@ EXPORT_SYMBOL(rproc_mem_entry_init);
* This function allocates a rproc_mem_entry struct and fill it with parameters
* provided by client.
*/
+__printf(5, 6)
struct rproc_mem_entry *
rproc_of_resm_mem_entry_init(struct device *dev, u32 of_resm_idx, size_t len,
u32 da, const char *name, ...)
--
2.25.1