2018-01-03 09:40:51

by Sumit Pundir

[permalink] [raw]
Subject: [PATCH] Staging: greybus: audio_codec.c: Change uint32_t to u32

This patch fixes the following checkpatch.pl issue at multiple lines:

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

Signed-off-by: Sumit Pundir <[email protected]>
---
drivers/staging/greybus/audio_codec.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/greybus/audio_codec.c b/drivers/staging/greybus/audio_codec.c
index fdb9e83..d81cdc9 100644
--- a/drivers/staging/greybus/audio_codec.c
+++ b/drivers/staging/greybus/audio_codec.c
@@ -47,7 +47,7 @@ static int gbaudio_module_enable_tx(struct gbaudio_codec_info *codec,
int module_state, ret = 0;
u16 data_cport, i2s_port, cportid;
u8 sig_bits, channels;
- uint32_t format, rate;
+ u32 format, rate;
struct gbaudio_data_connection *data;
struct gbaudio_stream_params *params;

@@ -182,7 +182,7 @@ static int gbaudio_module_enable_rx(struct gbaudio_codec_info *codec,
int module_state, ret = 0;
u16 data_cport, i2s_port, cportid;
u8 sig_bits, channels;
- uint32_t format, rate;
+ u32 format, rate;
struct gbaudio_data_connection *data;
struct gbaudio_stream_params *params;

@@ -412,7 +412,7 @@ static int gbcodec_hw_params(struct snd_pcm_substream *substream,
{
int ret;
u8 sig_bits, channels;
- uint32_t format, rate;
+ u32 format, rate;
struct gbaudio_module_info *module;
struct gbaudio_data_connection *data;
struct gb_bundle *bundle;
--
2.7.4


2018-01-03 10:20:39

by Vaibhav Agarwal

[permalink] [raw]
Subject: Re: [PATCH] Staging: greybus: audio_codec.c: Change uint32_t to u32

On Wed, Jan 3, 2018 at 3:09 PM, Sumit Pundir <[email protected]> wrote:
> This patch fixes the following checkpatch.pl issue at multiple lines:
>
> CHECK: Prefer kernel type 'u32' over 'uint32_t'
> + uint32_t format, rate;
>
> Signed-off-by: Sumit Pundir <[email protected]>
> ---

Hi Sumit,

Similar patches are already submitted by Kamal. Kindly go thru the
driverdev mailing list to get more details and share another series if
you find anything missing.


--
thanks,
./va