Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1471294 > unrolled thread
| Started by | Baoyou Xie <baoyou.xie@linaro.org> |
|---|---|
| First post | 2016-08-28 09:10 +0200 |
| Last post | 2016-08-29 21:20 +0200 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH v2] fix:iio:common:st_sensors:st_sensors_trigger:mark symbols static where possible Baoyou Xie <baoyou.xie@linaro.org> - 2016-08-28 09:10 +0200
Re: [PATCH v2] fix:iio:common:st_sensors:st_sensors_trigger:mark symbols static where possible Jonathan Cameron <jic23@kernel.org> - 2016-08-29 21:20 +0200
| From | Baoyou Xie <baoyou.xie@linaro.org> |
|---|---|
| Date | 2016-08-28 09:10 +0200 |
| Subject | [PATCH v2] fix:iio:common:st_sensors:st_sensors_trigger:mark symbols static where possible |
| Message-ID | <sb2o9-71z-5@gated-at.bofh.it> |
We get 2 warnings when biuld kernel with W=1:
drivers/iio/common/st_sensors/st_sensors_trigger.c:69:13: warning: no previous prototype
for 'st_sensors_irq_handler' [-Wmissing-prototypes]
drivers/iio/common/st_sensors/st_sensors_trigger.c:85:13: warning: no previous prototype
for 'st_sensors_irq_thread' [-Wmissing-prototypes]
In fact, these functions are only used in the file in which they are
declared and don't need a declaration, but can be made static.
so this patch marks these functions with 'static'.
Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
---
drivers/iio/common/st_sensors/st_sensors_trigger.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/iio/common/st_sensors/st_sensors_trigger.c b/drivers/iio/common/st_sensors/st_sensors_trigger.c
index e66f12e..fa73e67 100644
--- a/drivers/iio/common/st_sensors/st_sensors_trigger.c
+++ b/drivers/iio/common/st_sensors/st_sensors_trigger.c
@@ -66,7 +66,7 @@ static int st_sensors_new_samples_available(struct iio_dev *indio_dev,
* @irq: irq number
* @p: private handler data
*/
-irqreturn_t st_sensors_irq_handler(int irq, void *p)
+static irqreturn_t st_sensors_irq_handler(int irq, void *p)
{
struct iio_trigger *trig = p;
struct iio_dev *indio_dev = iio_trigger_get_drvdata(trig);
@@ -82,7 +82,7 @@ irqreturn_t st_sensors_irq_handler(int irq, void *p)
* @irq: irq number
* @p: private handler data
*/
-irqreturn_t st_sensors_irq_thread(int irq, void *p)
+static irqreturn_t st_sensors_irq_thread(int irq, void *p)
{
struct iio_trigger *trig = p;
struct iio_dev *indio_dev = iio_trigger_get_drvdata(trig);
--
2.7.4
[toc] | [next] | [standalone]
| From | Jonathan Cameron <jic23@kernel.org> |
|---|---|
| Date | 2016-08-29 21:20 +0200 |
| Subject | Re: [PATCH v2] fix:iio:common:st_sensors:st_sensors_trigger:mark symbols static where possible |
| Message-ID | <sbAga-2S5-21@gated-at.bofh.it> |
| In reply to | #1471294 |
On 28/08/16 08:04, Baoyou Xie wrote:
> We get 2 warnings when biuld kernel with W=1:
> drivers/iio/common/st_sensors/st_sensors_trigger.c:69:13: warning: no previous prototype
> for 'st_sensors_irq_handler' [-Wmissing-prototypes]
> drivers/iio/common/st_sensors/st_sensors_trigger.c:85:13: warning: no previous prototype
> for 'st_sensors_irq_thread' [-Wmissing-prototypes]
>
> In fact, these functions are only used in the file in which they are
> declared and don't need a declaration, but can be made static.
> so this patch marks these functions with 'static'.
>
> Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Hi.
Please don't resend just to add Reviewed-by / acked-by tags.
The maintainer will pick those up directly from the mailing list.
If there is anything else different here and I'm missing it you
should have highlighted what it was below the --- and
I'm really being unobservant!
Jonathan
> ---
> drivers/iio/common/st_sensors/st_sensors_trigger.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/iio/common/st_sensors/st_sensors_trigger.c b/drivers/iio/common/st_sensors/st_sensors_trigger.c
> index e66f12e..fa73e67 100644
> --- a/drivers/iio/common/st_sensors/st_sensors_trigger.c
> +++ b/drivers/iio/common/st_sensors/st_sensors_trigger.c
> @@ -66,7 +66,7 @@ static int st_sensors_new_samples_available(struct iio_dev *indio_dev,
> * @irq: irq number
> * @p: private handler data
> */
> -irqreturn_t st_sensors_irq_handler(int irq, void *p)
> +static irqreturn_t st_sensors_irq_handler(int irq, void *p)
> {
> struct iio_trigger *trig = p;
> struct iio_dev *indio_dev = iio_trigger_get_drvdata(trig);
> @@ -82,7 +82,7 @@ irqreturn_t st_sensors_irq_handler(int irq, void *p)
> * @irq: irq number
> * @p: private handler data
> */
> -irqreturn_t st_sensors_irq_thread(int irq, void *p)
> +static irqreturn_t st_sensors_irq_thread(int irq, void *p)
> {
> struct iio_trigger *trig = p;
> struct iio_dev *indio_dev = iio_trigger_get_drvdata(trig);
>
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web