* [iio:testing 44/44] drivers/iio/imu/kmx61.c:422:5: sparse: symbol 'kmx61_set_wake_up_odr' was not declared. Should it be static?
@ 2014-12-12 16:28 kbuild test robot
2014-12-12 16:28 ` [PATCH iio] iio: imu: kmx61: kmx61_set_wake_up_odr() can be static kbuild test robot
0 siblings, 1 reply; 4+ messages in thread
From: kbuild test robot @ 2014-12-12 16:28 UTC (permalink / raw)
To: Daniel Baluta
Cc: kbuild-all, Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen,
Peter Meerwald, linux-iio, linux-kernel
tree: git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git testing
head: fd3ae7a9f21c2a51a1d220bc7a7c3b45ab5e6ad1
commit: fd3ae7a9f21c2a51a1d220bc7a7c3b45ab5e6ad1 [44/44] iio: imu: kmx61: Add support for any motion trigger
reproduce:
# apt-get install sparse
git checkout fd3ae7a9f21c2a51a1d220bc7a7c3b45ab5e6ad1
make ARCH=x86_64 allmodconfig
make C=1 CF=-D__CHECK_ENDIAN__
sparse warnings: (new ones prefixed by >>)
>> drivers/iio/imu/kmx61.c:422:5: sparse: symbol 'kmx61_set_wake_up_odr' was not declared. Should it be static?
Please review and possibly fold the followup patch.
---
0-DAY kernel test infrastructure Open Source Technology Center
http://lists.01.org/mailman/listinfo/kbuild Intel Corporation
^ permalink raw reply [flat|nested] 4+ messages in thread* [PATCH iio] iio: imu: kmx61: kmx61_set_wake_up_odr() can be static 2014-12-12 16:28 [iio:testing 44/44] drivers/iio/imu/kmx61.c:422:5: sparse: symbol 'kmx61_set_wake_up_odr' was not declared. Should it be static? kbuild test robot @ 2014-12-12 16:28 ` kbuild test robot 2014-12-12 16:41 ` Daniel Baluta 0 siblings, 1 reply; 4+ messages in thread From: kbuild test robot @ 2014-12-12 16:28 UTC (permalink / raw) To: Daniel Baluta Cc: kbuild-all, Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald, linux-iio, linux-kernel 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@intel.com> --- 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; ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH iio] iio: imu: kmx61: kmx61_set_wake_up_odr() can be static 2014-12-12 16:28 ` [PATCH iio] iio: imu: kmx61: kmx61_set_wake_up_odr() can be static kbuild test robot @ 2014-12-12 16:41 ` Daniel Baluta 2014-12-12 18:15 ` Jonathan Cameron 0 siblings, 1 reply; 4+ messages in thread From: Daniel Baluta @ 2014-12-12 16:41 UTC (permalink / raw) To: kbuild test robot Cc: Daniel Baluta, kbuild-all, Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald, linux-iio@vger.kernel.org, Linux Kernel Mailing List On Fri, Dec 12, 2014 at 6:28 PM, kbuild test robot <fengguang.wu@intel.com> 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@intel.com> Reviewed-by: Daniel Baluta <daniel.baluta@intel.com> > --- > 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@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH iio] iio: imu: kmx61: kmx61_set_wake_up_odr() can be static 2014-12-12 16:41 ` Daniel Baluta @ 2014-12-12 18:15 ` Jonathan Cameron 0 siblings, 0 replies; 4+ messages in thread From: Jonathan Cameron @ 2014-12-12 18:15 UTC (permalink / raw) To: Daniel Baluta, kbuild test robot Cc: kbuild-all, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald, linux-iio@vger.kernel.org, Linux Kernel Mailing List On 12/12/14 16:41, Daniel Baluta wrote: > On Fri, Dec 12, 2014 at 6:28 PM, kbuild test robot > <fengguang.wu@intel.com> 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@intel.com> > > Reviewed-by: Daniel Baluta <daniel.baluta@intel.com> Applied to the togreg branch of iio.git. Thanks! Jonathan > >> --- >> 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@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-12-12 18:15 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-12-12 16:28 [iio:testing 44/44] drivers/iio/imu/kmx61.c:422:5: sparse: symbol 'kmx61_set_wake_up_odr' was not declared. Should it be static? kbuild test robot 2014-12-12 16:28 ` [PATCH iio] iio: imu: kmx61: kmx61_set_wake_up_odr() can be static kbuild test robot 2014-12-12 16:41 ` Daniel Baluta 2014-12-12 18:15 ` Jonathan Cameron
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.