Chrome platform driver development
 help / color / mirror / Atom feed
* [PATCH] platform/chrome: cros_ec: fix compilation warning
@ 2023-10-03  8:04 Tzung-Bi Shih
  2023-10-05  2:13 ` Guenter Roeck
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Tzung-Bi Shih @ 2023-10-03  8:04 UTC (permalink / raw)
  To: bleung, groeck; +Cc: tzungbi, chrome-platform

When including cros_ec.h solely, the compiler emits the following
warning:
> 'struct cros_ec_device' declared inside parameter list
  will not be visible outside of this definition or declaration

Fix it by forward declaration.

Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
 drivers/platform/chrome/cros_ec.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/platform/chrome/cros_ec.h b/drivers/platform/chrome/cros_ec.h
index bbca0096868a..2b2574236030 100644
--- a/drivers/platform/chrome/cros_ec.h
+++ b/drivers/platform/chrome/cros_ec.h
@@ -10,6 +10,8 @@
 
 #include <linux/interrupt.h>
 
+struct cros_ec_device;
+
 int cros_ec_register(struct cros_ec_device *ec_dev);
 void cros_ec_unregister(struct cros_ec_device *ec_dev);
 
-- 
2.42.0.582.g8ccd20d70d-goog


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] platform/chrome: cros_ec: fix compilation warning
  2023-10-03  8:04 [PATCH] platform/chrome: cros_ec: fix compilation warning Tzung-Bi Shih
@ 2023-10-05  2:13 ` Guenter Roeck
  2023-10-05 10:20 ` patchwork-bot+chrome-platform
  2023-10-11  2:20 ` patchwork-bot+chrome-platform
  2 siblings, 0 replies; 4+ messages in thread
From: Guenter Roeck @ 2023-10-05  2:13 UTC (permalink / raw)
  To: Tzung-Bi Shih; +Cc: bleung, groeck, chrome-platform

On Tue, Oct 3, 2023 at 1:05 AM Tzung-Bi Shih <tzungbi@kernel.org> wrote:
>
> When including cros_ec.h solely, the compiler emits the following
> warning:
> > 'struct cros_ec_device' declared inside parameter list
>   will not be visible outside of this definition or declaration
>
> Fix it by forward declaration.
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>

Reviewed-by: Guenter Roeck <groeck@chromium.org>

> ---
>  drivers/platform/chrome/cros_ec.h | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/platform/chrome/cros_ec.h b/drivers/platform/chrome/cros_ec.h
> index bbca0096868a..2b2574236030 100644
> --- a/drivers/platform/chrome/cros_ec.h
> +++ b/drivers/platform/chrome/cros_ec.h
> @@ -10,6 +10,8 @@
>
>  #include <linux/interrupt.h>
>
> +struct cros_ec_device;
> +
>  int cros_ec_register(struct cros_ec_device *ec_dev);
>  void cros_ec_unregister(struct cros_ec_device *ec_dev);
>
> --
> 2.42.0.582.g8ccd20d70d-goog
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] platform/chrome: cros_ec: fix compilation warning
  2023-10-03  8:04 [PATCH] platform/chrome: cros_ec: fix compilation warning Tzung-Bi Shih
  2023-10-05  2:13 ` Guenter Roeck
@ 2023-10-05 10:20 ` patchwork-bot+chrome-platform
  2023-10-11  2:20 ` patchwork-bot+chrome-platform
  2 siblings, 0 replies; 4+ messages in thread
From: patchwork-bot+chrome-platform @ 2023-10-05 10:20 UTC (permalink / raw)
  To: Tzung-Bi Shih; +Cc: bleung, groeck, chrome-platform

Hello:

This patch was applied to chrome-platform/linux.git (for-kernelci)
by Tzung-Bi Shih <tzungbi@kernel.org>:

On Tue,  3 Oct 2023 08:04:53 +0000 you wrote:
> When including cros_ec.h solely, the compiler emits the following
> warning:
> > 'struct cros_ec_device' declared inside parameter list
>   will not be visible outside of this definition or declaration
> 
> Fix it by forward declaration.
> 
> [...]

Here is the summary with links:
  - platform/chrome: cros_ec: fix compilation warning
    https://git.kernel.org/chrome-platform/c/a7bad167bdc3

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] platform/chrome: cros_ec: fix compilation warning
  2023-10-03  8:04 [PATCH] platform/chrome: cros_ec: fix compilation warning Tzung-Bi Shih
  2023-10-05  2:13 ` Guenter Roeck
  2023-10-05 10:20 ` patchwork-bot+chrome-platform
@ 2023-10-11  2:20 ` patchwork-bot+chrome-platform
  2 siblings, 0 replies; 4+ messages in thread
From: patchwork-bot+chrome-platform @ 2023-10-11  2:20 UTC (permalink / raw)
  To: Tzung-Bi Shih; +Cc: bleung, groeck, chrome-platform

Hello:

This patch was applied to chrome-platform/linux.git (for-next)
by Tzung-Bi Shih <tzungbi@kernel.org>:

On Tue,  3 Oct 2023 08:04:53 +0000 you wrote:
> When including cros_ec.h solely, the compiler emits the following
> warning:
> > 'struct cros_ec_device' declared inside parameter list
>   will not be visible outside of this definition or declaration
> 
> Fix it by forward declaration.
> 
> [...]

Here is the summary with links:
  - platform/chrome: cros_ec: fix compilation warning
    https://git.kernel.org/chrome-platform/c/a7bad167bdc3

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-10-11  2:20 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-10-03  8:04 [PATCH] platform/chrome: cros_ec: fix compilation warning Tzung-Bi Shih
2023-10-05  2:13 ` Guenter Roeck
2023-10-05 10:20 ` patchwork-bot+chrome-platform
2023-10-11  2:20 ` patchwork-bot+chrome-platform

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox