* [PATCH] staging: ccree: fix __dump_byte_array() declaration mismatch
@ 2017-12-20 18:16 Corentin Labbe
2017-12-21 17:12 ` Greg KH
0 siblings, 1 reply; 2+ messages in thread
From: Corentin Labbe @ 2017-12-20 18:16 UTC (permalink / raw)
To: gilad, gregkh
Cc: linux-crypto, driverdev-devel, devel, linux-kernel,
Corentin Labbe
This patch corrects the type of the size argument in __dump_byte_array()
from unsigned long to size_t as done only in drivers/staging/ccree/ssi_driver.c
This fix also a build error:
drivers/staging/ccree/ssi_driver.c:82:6: error: conflicting types for '__dump_byte_array'
Fixes: 3f268f5d6669 ("staging: ccree: turn compile time debug log to params")
Signed-off-by: Corentin Labbe <clabbe@baylibre.com>
---
drivers/staging/ccree/ssi_driver.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/ccree/ssi_driver.h b/drivers/staging/ccree/ssi_driver.h
index 5a56f7a76b71..0f57c9a8b8a6 100644
--- a/drivers/staging/ccree/ssi_driver.h
+++ b/drivers/staging/ccree/ssi_driver.h
@@ -174,8 +174,7 @@ static inline struct device *drvdata_to_dev(struct cc_drvdata *drvdata)
return &drvdata->plat_dev->dev;
}
-void __dump_byte_array(const char *name, const u8 *the_array,
- unsigned long size);
+void __dump_byte_array(const char *name, const u8 *buf, size_t size);
static inline void dump_byte_array(const char *name, const u8 *the_array,
unsigned long size)
{
--
2.13.6
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] staging: ccree: fix __dump_byte_array() declaration mismatch
2017-12-20 18:16 [PATCH] staging: ccree: fix __dump_byte_array() declaration mismatch Corentin Labbe
@ 2017-12-21 17:12 ` Greg KH
0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2017-12-21 17:12 UTC (permalink / raw)
To: Corentin Labbe; +Cc: gilad, linux-crypto, driverdev-devel, devel, linux-kernel
On Wed, Dec 20, 2017 at 06:16:08PM +0000, Corentin Labbe wrote:
> This patch corrects the type of the size argument in __dump_byte_array()
> from unsigned long to size_t as done only in drivers/staging/ccree/ssi_driver.c
>
> This fix also a build error:
> drivers/staging/ccree/ssi_driver.c:82:6: error: conflicting types for '__dump_byte_array'
>
> Fixes: 3f268f5d6669 ("staging: ccree: turn compile time debug log to params")
> Signed-off-by: Corentin Labbe <clabbe@baylibre.com>
> ---
> drivers/staging/ccree/ssi_driver.h | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
There's another warning here that Arnd's patches fixes, yet yours
didn't, so I'm going to take his patch, sorry.
greg k-h
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-12-21 17:13 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-12-20 18:16 [PATCH] staging: ccree: fix __dump_byte_array() declaration mismatch Corentin Labbe
2017-12-21 17:12 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox