From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752171AbdLJQXJ (ORCPT ); Sun, 10 Dec 2017 11:23:09 -0500 Received: from mail.kernel.org ([198.145.29.99]:56738 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751746AbdLJQXG (ORCPT ); Sun, 10 Dec 2017 11:23:06 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B109521882 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=jic23@kernel.org Date: Sun, 10 Dec 2017 16:23:02 +0000 From: Jonathan Cameron To: Colin King Cc: Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , Lorenzo Bianconi , linux-iio@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH][next] iio: light: make a couple of config structures static Message-ID: <20171210162302.13236b42@archlinux> In-Reply-To: <20171204120406.4364-1-colin.king@canonical.com> References: <20171204120406.4364-1-colin.king@canonical.com> X-Mailer: Claws Mail 3.15.1-dirty (GTK+ 2.24.31; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 4 Dec 2017 12:04:06 +0000 Colin King wrote: > From: Colin Ian King > > Structures st_uvis25_i2c_regmap_config and st_uvis25_spi_regmap_config are > local to the source and do not need to be in global scope, so make them > both static. > > Cleans up sparse warnings: > warning: symbol 'st_uvis25_i2c_regmap_config' was not declared. Should > it be static? > warning: symbol 'st_uvis25_spi_regmap_config' was not declared. Should > it be static? > > Signed-off-by: Colin Ian King Thanks Colin, Not sure why I missed these when running build tests and they got past 0-day. Ah well.. Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with them. Thanks, Jonathan > --- > drivers/iio/light/st_uvis25_i2c.c | 2 +- > drivers/iio/light/st_uvis25_spi.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/iio/light/st_uvis25_i2c.c b/drivers/iio/light/st_uvis25_i2c.c > index c939c0b0ff10..afd6eb01a202 100644 > --- a/drivers/iio/light/st_uvis25_i2c.c > +++ b/drivers/iio/light/st_uvis25_i2c.c > @@ -19,7 +19,7 @@ > > #define UVIS25_I2C_AUTO_INCREMENT BIT(7) > > -const struct regmap_config st_uvis25_i2c_regmap_config = { > +static const struct regmap_config st_uvis25_i2c_regmap_config = { > .reg_bits = 8, > .val_bits = 8, > .write_flag_mask = UVIS25_I2C_AUTO_INCREMENT, > diff --git a/drivers/iio/light/st_uvis25_spi.c b/drivers/iio/light/st_uvis25_spi.c > index e697e14e7952..cdfee5e84d5e 100644 > --- a/drivers/iio/light/st_uvis25_spi.c > +++ b/drivers/iio/light/st_uvis25_spi.c > @@ -19,7 +19,7 @@ > #define UVIS25_SENSORS_SPI_READ BIT(7) > #define UVIS25_SPI_AUTO_INCREMENT BIT(6) > > -const struct regmap_config st_uvis25_spi_regmap_config = { > +static const struct regmap_config st_uvis25_spi_regmap_config = { > .reg_bits = 8, > .val_bits = 8, > .read_flag_mask = UVIS25_SENSORS_SPI_READ | UVIS25_SPI_AUTO_INCREMENT,