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

Groups > linux.kernel > #1602926 > unrolled thread

[PATCH] iio: accel: Prefer unsigned int to bare use of unsigned

Back to article view | Back to linux.kernel

Contents

  [PATCH] iio: accel: Prefer unsigned int to bare use of unsigned Miguel Robles <miguel.robles@farole.net> - 2017-03-17 01:10 +0100
    Re: [PATCH] iio: accel: Prefer unsigned int to bare use of unsigned Jonathan Cameron <jic23@kernel.org> - 2017-03-19 11:50 +0100

#1602926 — [PATCH] iio: accel: Prefer unsigned int to bare use of unsigned

Fix checkpatch warnings:
WARNING: Prefer 'unsigned int' to bare use of 'unsigned'

Signed-off-by: Miguel Robles <miguel.robles@farole.net>
---
 drivers/iio/accel/bma180.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/iio/accel/bma180.c b/drivers/iio/accel/bma180.c
index 0890934..dd84e87 100644
--- a/drivers/iio/accel/bma180.c
+++ b/drivers/iio/accel/bma180.c
@@ -41,11 +41,11 @@ struct bma180_data;
 
 struct bma180_part_info {
 	const struct iio_chan_spec *channels;
-	unsigned num_channels;
+	unsigned int num_channels;
 	const int *scale_table;
-	unsigned num_scales;
+	unsigned int num_scales;
 	const int *bw_table;
-	unsigned num_bw;
+	unsigned int num_bw;
 
 	u8 int_reset_reg, int_reset_mask;
 	u8 sleep_reg, sleep_mask;
@@ -408,7 +408,7 @@ static void bma250_chip_disable(struct bma180_data *data)
 	dev_err(&data->client->dev, "failed to disable the chip\n");
 }
 
-static ssize_t bma180_show_avail(char *buf, const int *vals, unsigned n,
+static ssize_t bma180_show_avail(char *buf, const int *vals, unsigned int n,
 				 bool micros)
 {
 	size_t len = 0;
-- 
2.7.4

[toc] | [next] | [standalone]

#1603939

On 16/03/17 23:35, Miguel Robles wrote:
> Fix checkpatch warnings:
> WARNING: Prefer 'unsigned int' to bare use of 'unsigned'
> 
> Signed-off-by: Miguel Robles <miguel.robles@farole.net>
Applied to the togreg branch of iio.git and pushed out as testing
for the autobuilders to play with it.

Thanks,

Jonathan
> ---
>  drivers/iio/accel/bma180.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/iio/accel/bma180.c b/drivers/iio/accel/bma180.c
> index 0890934..dd84e87 100644
> --- a/drivers/iio/accel/bma180.c
> +++ b/drivers/iio/accel/bma180.c
> @@ -41,11 +41,11 @@ struct bma180_data;
>  
>  struct bma180_part_info {
>  	const struct iio_chan_spec *channels;
> -	unsigned num_channels;
> +	unsigned int num_channels;
>  	const int *scale_table;
> -	unsigned num_scales;
> +	unsigned int num_scales;
>  	const int *bw_table;
> -	unsigned num_bw;
> +	unsigned int num_bw;
>  
>  	u8 int_reset_reg, int_reset_mask;
>  	u8 sleep_reg, sleep_mask;
> @@ -408,7 +408,7 @@ static void bma250_chip_disable(struct bma180_data *data)
>  	dev_err(&data->client->dev, "failed to disable the chip\n");
>  }
>  
> -static ssize_t bma180_show_avail(char *buf, const int *vals, unsigned n,
> +static ssize_t bma180_show_avail(char *buf, const int *vals, unsigned int n,
>  				 bool micros)
>  {
>  	size_t len = 0;
> 

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web