linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to u32
@ 2017-01-22 17:49 Marcos Paulo de Souza
  2017-01-22 19:14 ` Mark Greer
  2017-01-23  6:13 ` Vaibhav Agarwal
  0 siblings, 2 replies; 3+ messages in thread
From: Marcos Paulo de Souza @ 2017-01-22 17:49 UTC (permalink / raw)
  Cc: Marcos Paulo de Souza, Vaibhav Agarwal, Mark Greer, Johan Hovold,
	Alex Elder, Greg Kroah-Hartman, greybus-dev, devel, linux-kernel

Change uint32_t to u32, solved the issue reported by checkpatch.pl:

CHECK: Prefer kernel type 'u32' over 'uint32_t'
+			uint32_t *format, uint32_t *rate, u8 *channels,

CHECK: Prefer kernel type 'u32' over 'uint32_t'
+			uint32_t format, uint32_t rate, u8 channels,

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
---
 drivers/staging/greybus/audio_gb.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/greybus/audio_gb.c b/drivers/staging/greybus/audio_gb.c
index 42f287d..7884d84 100644
--- a/drivers/staging/greybus/audio_gb.c
+++ b/drivers/staging/greybus/audio_gb.c
@@ -108,7 +108,7 @@ int gb_audio_gb_disable_widget(struct gb_connection *connection,
 EXPORT_SYMBOL_GPL(gb_audio_gb_disable_widget);
 
 int gb_audio_gb_get_pcm(struct gb_connection *connection, u16 data_cport,
-			uint32_t *format, uint32_t *rate, u8 *channels,
+			u32 *format, u32 *rate, u8 *channels,
 			u8 *sig_bits)
 {
 	struct gb_audio_get_pcm_request req;
@@ -132,7 +132,7 @@ int gb_audio_gb_get_pcm(struct gb_connection *connection, u16 data_cport,
 EXPORT_SYMBOL_GPL(gb_audio_gb_get_pcm);
 
 int gb_audio_gb_set_pcm(struct gb_connection *connection, u16 data_cport,
-			uint32_t format, uint32_t rate, u8 channels,
+			u32 format, u32 rate, u8 channels,
 			u8 sig_bits)
 {
 	struct gb_audio_set_pcm_request req;
-- 
2.9.3

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to u32
  2017-01-22 17:49 [PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to u32 Marcos Paulo de Souza
@ 2017-01-22 19:14 ` Mark Greer
  2017-01-23  6:13 ` Vaibhav Agarwal
  1 sibling, 0 replies; 3+ messages in thread
From: Mark Greer @ 2017-01-22 19:14 UTC (permalink / raw)
  To: Marcos Paulo de Souza
  Cc: Vaibhav Agarwal, Mark Greer, Johan Hovold, Alex Elder,
	Greg Kroah-Hartman, greybus-dev, devel, linux-kernel

On Sun, Jan 22, 2017 at 03:49:21PM -0200, Marcos Paulo de Souza wrote:
> Change uint32_t to u32, solved the issue reported by checkpatch.pl:
> 
> CHECK: Prefer kernel type 'u32' over 'uint32_t'
> +			uint32_t *format, uint32_t *rate, u8 *channels,
> 
> CHECK: Prefer kernel type 'u32' over 'uint32_t'
> +			uint32_t format, uint32_t rate, u8 channels,
> 
> Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
> ---

Good catch, thanks.

Acked-by: Mark Greer <mgreer@animalcreek.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to u32
  2017-01-22 17:49 [PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to u32 Marcos Paulo de Souza
  2017-01-22 19:14 ` Mark Greer
@ 2017-01-23  6:13 ` Vaibhav Agarwal
  1 sibling, 0 replies; 3+ messages in thread
From: Vaibhav Agarwal @ 2017-01-23  6:13 UTC (permalink / raw)
  To: Marcos Paulo de Souza
  Cc: Mark Greer, Johan Hovold, Alex Elder, Greg Kroah-Hartman,
	greybus-dev, devel, linux-kernel

On Sun, Jan 22, 2017 at 11:19 PM, Marcos Paulo de Souza
<marcos.souza.org@gmail.com> wrote:
> Change uint32_t to u32, solved the issue reported by checkpatch.pl:
>
> CHECK: Prefer kernel type 'u32' over 'uint32_t'
> +                       uint32_t *format, uint32_t *rate, u8 *channels,
>
> CHECK: Prefer kernel type 'u32' over 'uint32_t'
> +                       uint32_t format, uint32_t rate, u8 channels,
>
> Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
> ---

Acked-by: Vaibhav Agarwal <vaibhav.sr@gmail.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-01-23  6:13 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-22 17:49 [PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to u32 Marcos Paulo de Souza
2017-01-22 19:14 ` Mark Greer
2017-01-23  6:13 ` Vaibhav Agarwal

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).