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


Groups > linux.kernel > #1564481 > unrolled thread

[PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to u32

Started byMarcos Paulo de Souza <marcos.souza.org@gmail.com>
First post2017-01-22 19:00 +0100
Last post2017-01-23 07:20 +0100
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to u32 Marcos Paulo de Souza <marcos.souza.org@gmail.com> - 2017-01-22 19:00 +0100
    Re: [PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to  u32 Mark Greer <mgreer@animalcreek.com> - 2017-01-22 20:20 +0100
    Re: [PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to u32 Vaibhav Agarwal <vaibhav.sr@gmail.com> - 2017-01-23 07:20 +0100

#1564481 — [PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to u32

FromMarcos Paulo de Souza <marcos.souza.org@gmail.com>
Date2017-01-22 19:00 +0100
Subject[PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to u32
Message-ID<t2uAO-3K7-29@gated-at.bofh.it>
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

[toc] | [next] | [standalone]


#1564552 — Re: [PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to u32

FromMark Greer <mgreer@animalcreek.com>
Date2017-01-22 20:20 +0100
SubjectRe: [PATCH -next] staging: greybus: audio_gb.c: Change uint32_t to u32
Message-ID<t2vQd-4GL-1@gated-at.bofh.it>
In reply to#1564481
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>

[toc] | [prev] | [next] | [standalone]


#1564699

FromVaibhav Agarwal <vaibhav.sr@gmail.com>
Date2017-01-23 07:20 +0100
Message-ID<t2G8V-2T1-7@gated-at.bofh.it>
In reply to#1564481
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>

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web