Re: [PATCH iio] iio: imu: kmx61: kmx61_set_wake_up_odr() can be static

From: Daniel Baluta
Date: Fri Dec 12 2014 - 11:41:22 EST


On Fri, Dec 12, 2014 at 6:28 PM, kbuild test robot
<fengguang.wu@xxxxxxxxx> wrote:
> drivers/iio/imu/kmx61.c:422:5: sparse: symbol 'kmx61_set_wake_up_odr' was not declared. Should it be static?
>
> Signed-off-by: Fengguang Wu <fengguang.wu@xxxxxxxxx>

Reviewed-by: Daniel Baluta <daniel.baluta@xxxxxxxxx>

> ---
> kmx61.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/iio/imu/kmx61.c b/drivers/iio/imu/kmx61.c
> index bda9c40..9b32f01 100644
> --- a/drivers/iio/imu/kmx61.c
> +++ b/drivers/iio/imu/kmx61.c
> @@ -419,7 +419,7 @@ static int kmx61_get_mode(struct kmx61_data *data, u8 *mode, u8 device)
> return 0;
> }
>
> -int kmx61_set_wake_up_odr(struct kmx61_data *data, int val, int val2)
> +static int kmx61_set_wake_up_odr(struct kmx61_data *data, int val, int val2)
> {
> int ret, odr_bits;
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/