diff mbox series

[v3,05/17] dfu: export dfu_list

Message ID 20200710012537.6264-6-takahiro.akashi@linaro.org
State Superseded
Headers show
Series efi_loader: add capsule update support | expand

Commit Message

AKASHI Takahiro July 10, 2020, 1:25 a.m. UTC
This variable will be utilized to enumerate all dfu entities
for UEFI capsule firmware update in a later commit.

Signed-off-by: AKASHI Takahiro <takahiro.akashi at linaro.org>
---
 drivers/dfu/dfu.c | 2 +-
 include/dfu.h     | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

Comments

Heinrich Schuchardt July 10, 2020, 4:25 p.m. UTC | #1
On 10.07.20 03:25, AKASHI Takahiro wrote:
> This variable will be utilized to enumerate all dfu entities
> for UEFI capsule firmware update in a later commit.
>
> Signed-off-by: AKASHI Takahiro <takahiro.akashi at linaro.org>
> ---
>  drivers/dfu/dfu.c | 2 +-
>  include/dfu.h     | 3 +++
>  2 files changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/dfu/dfu.c b/drivers/dfu/dfu.c
> index a298c2c43999..501a60b34400 100644
> --- a/drivers/dfu/dfu.c
> +++ b/drivers/dfu/dfu.c
> @@ -18,7 +18,7 @@
>  #include <linux/list.h>
>  #include <linux/compiler.h>
>
> -static LIST_HEAD(dfu_list);
> +LIST_HEAD(dfu_list);
>  static int dfu_alt_num;
>  static int alt_num_cnt;
>  static struct hash_algo *dfu_hash_algo;
> diff --git a/include/dfu.h b/include/dfu.h
> index 94b0a9e68317..e9af9503d685 100644
> --- a/include/dfu.h
> +++ b/include/dfu.h
> @@ -158,6 +158,9 @@ struct dfu_entity {
>  	unsigned int inited:1;
>  };
>
> +struct list_head;
> +extern struct list_head dfu_list;
> +
>  #ifdef CONFIG_SET_DFU_ALT_INFO
>  /**
>   * set_dfu_alt_info() - set dfu_alt_info environment variable
>

Why would you need to export this list if the current implementation of
the dfu command does not need to export it?

Please, use the existing DFU API functions instead of accessing a
private member of the implementation.

Best regards

Heinrich
diff mbox series

Patch

diff --git a/drivers/dfu/dfu.c b/drivers/dfu/dfu.c
index a298c2c43999..501a60b34400 100644
--- a/drivers/dfu/dfu.c
+++ b/drivers/dfu/dfu.c
@@ -18,7 +18,7 @@ 
 #include <linux/list.h>
 #include <linux/compiler.h>
 
-static LIST_HEAD(dfu_list);
+LIST_HEAD(dfu_list);
 static int dfu_alt_num;
 static int alt_num_cnt;
 static struct hash_algo *dfu_hash_algo;
diff --git a/include/dfu.h b/include/dfu.h
index 94b0a9e68317..e9af9503d685 100644
--- a/include/dfu.h
+++ b/include/dfu.h
@@ -158,6 +158,9 @@  struct dfu_entity {
 	unsigned int inited:1;
 };
 
+struct list_head;
+extern struct list_head dfu_list;
+
 #ifdef CONFIG_SET_DFU_ALT_INFO
 /**
  * set_dfu_alt_info() - set dfu_alt_info environment variable