Re: [PATCH -next] iio: temperature: fix non static symbol warnings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Reviewed-By: Matt Ranostay <mranostay@xxxxxxxxx>

> On Aug 26, 2016, at 07:33, Wei Yongjun <weiyj.lk@xxxxxxxxx> wrote:
> 
> From: Wei Yongjun <weiyongjun1@xxxxxxxxxx>
> 
> Fixes the following sparse warnings:
> 
> drivers/iio/temperature/maxim_thermocouple.c:35:28: warning:
> symbol 'max6675_channels' was not declared. Should it be static?
> drivers/iio/temperature/maxim_thermocouple.c:52:28: warning:
> symbol 'max31855_channels' was not declared. Should it be static?
> drivers/iio/temperature/maxim_thermocouple.c:98:38: warning:
> symbol 'maxim_thermocouple_chips' was not declared. Should it be static?
> 
> Signed-off-by: Wei Yongjun <weiyongjun1@xxxxxxxxxx>
> ---
> drivers/iio/temperature/maxim_thermocouple.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/iio/temperature/maxim_thermocouple.c b/drivers/iio/temperature/maxim_thermocouple.c
> index 030827e..39dd202 100644
> --- a/drivers/iio/temperature/maxim_thermocouple.c
> +++ b/drivers/iio/temperature/maxim_thermocouple.c
> @@ -32,7 +32,7 @@ enum {
>    MAX31855,
> };
> 
> -const struct iio_chan_spec max6675_channels[] = {
> +static const struct iio_chan_spec max6675_channels[] = {
>    {    /* thermocouple temperature */
>        .type = IIO_TEMP,
>        .info_mask_separate =
> @@ -49,7 +49,7 @@ const struct iio_chan_spec max6675_channels[] = {
>    IIO_CHAN_SOFT_TIMESTAMP(1),
> };
> 
> -const struct iio_chan_spec max31855_channels[] = {
> +static const struct iio_chan_spec max31855_channels[] = {
>    {    /* thermocouple temperature */
>        .type = IIO_TEMP,
>        .address = 2,
> @@ -95,7 +95,7 @@ struct maxim_thermocouple_chip {
>    u32 status_bit;
> };
> 
> -const struct maxim_thermocouple_chip maxim_thermocouple_chips[] = {
> +static const struct maxim_thermocouple_chip maxim_thermocouple_chips[] = {
>    [MAX6675] = {
>            .channels = max6675_channels,
>            .num_channels = ARRAY_SIZE(max6675_channels),
> 
> 
> 
--
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




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux