* [PATCH] Staging: greybus: audio_codec.h: Change uint32_t to u32
@ 2018-01-03 9:37 Sumit Pundir
2018-01-03 10:12 ` Johan Hovold
0 siblings, 1 reply; 2+ messages in thread
From: Sumit Pundir @ 2018-01-03 9:37 UTC (permalink / raw)
To: vaibhav.sr; +Cc: mgreer, johan, elder, gregkh, devel, linux-kernel
This patch fixes the following checkpatch.pl issues:
CHECK: Prefer kernel type 'u32' over 'uint32_t'
+ uint32_t format, rate;
CHECK: Prefer kernel type 'u32' over 'uint32_t'
+ uint32_t *rate, u8 *channels,
CHECK: Prefer kernel type 'u32' over 'uint32_t'
+ uint32_t rate, u8 channels,
Signed-off-by: Sumit Pundir <pundirsumit11@gmail.com>
---
drivers/staging/greybus/audio_codec.h | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/greybus/audio_codec.h b/drivers/staging/greybus/audio_codec.h
index 161b37c..fd7b893 100644
--- a/drivers/staging/greybus/audio_codec.h
+++ b/drivers/staging/greybus/audio_codec.h
@@ -53,7 +53,7 @@ enum gbaudio_codec_state {
struct gbaudio_stream_params {
int state;
u8 sig_bits, channels;
- uint32_t format, rate;
+ u32 format, rate;
};
struct gbaudio_codec_dai {
@@ -183,12 +183,12 @@ extern int gb_audio_gb_enable_widget(struct gb_connection *connection,
extern int gb_audio_gb_disable_widget(struct gb_connection *connection,
u8 widget_id);
extern int gb_audio_gb_get_pcm(struct gb_connection *connection,
- u16 data_cport, uint32_t *format,
- uint32_t *rate, u8 *channels,
+ u16 data_cport, u32 *format,
+ u32 *rate, u8 *channels,
u8 *sig_bits);
extern int gb_audio_gb_set_pcm(struct gb_connection *connection,
- u16 data_cport, uint32_t format,
- uint32_t rate, u8 channels,
+ u16 data_cport, u32 format,
+ u32 rate, u8 channels,
u8 sig_bits);
extern int gb_audio_gb_set_tx_data_size(struct gb_connection *connection,
u16 data_cport, u16 size);
--
2.7.4
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] Staging: greybus: audio_codec.h: Change uint32_t to u32
2018-01-03 9:37 [PATCH] Staging: greybus: audio_codec.h: Change uint32_t to u32 Sumit Pundir
@ 2018-01-03 10:12 ` Johan Hovold
0 siblings, 0 replies; 2+ messages in thread
From: Johan Hovold @ 2018-01-03 10:12 UTC (permalink / raw)
To: Sumit Pundir
Cc: vaibhav.sr, mgreer, johan, elder, gregkh, devel, linux-kernel
On Wed, Jan 03, 2018 at 03:07:33PM +0530, Sumit Pundir wrote:
> This patch fixes the following checkpatch.pl issues:
>
> CHECK: Prefer kernel type 'u32' over 'uint32_t'
> + uint32_t format, rate;
>
> CHECK: Prefer kernel type 'u32' over 'uint32_t'
> + uint32_t *rate, u8 *channels,
>
> CHECK: Prefer kernel type 'u32' over 'uint32_t'
> + uint32_t rate, u8 channels,
>
> Signed-off-by: Sumit Pundir <pundirsumit11@gmail.com>
> ---
> drivers/staging/greybus/audio_codec.h | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
Please fold this in with the changes to audio_codec.c and submit a
single patch for it all.
Thanks,
Johan
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-01-03 10:12 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-01-03 9:37 [PATCH] Staging: greybus: audio_codec.h: Change uint32_t to u32 Sumit Pundir
2018-01-03 10:12 ` Johan Hovold
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox