The dm_root_f field seems to be entirely write-only and hence
redundant, unless 'git grep' fails to find some access generated via
preprocessor token concatenation or similar.

Signed-off-by: Rasmus Villemoes <rasmus.villem...@prevas.dk>
---
 common/board_r.c                  | 3 +--
 include/asm-generic/global_data.h | 4 ----
 test/dm/core.c                    | 1 -
 3 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/common/board_r.c b/common/board_r.c
index da0b80f24ff..fd263f4a0a8 100644
--- a/common/board_r.c
+++ b/common/board_r.c
@@ -230,8 +230,7 @@ static int initr_dm(void)
 
        oftree_reset();
 
-       /* Save the pre-reloc driver model and start a new one */
-       gd->dm_root_f = gd->dm_root;
+       /* Drop the pre-reloc driver model and start a new one */
        gd->dm_root = NULL;
 #ifdef CONFIG_TIMER
        gd->timer = NULL;
diff --git a/include/asm-generic/global_data.h 
b/include/asm-generic/global_data.h
index fcc3c6e14ca..aa336d63e3a 100644
--- a/include/asm-generic/global_data.h
+++ b/include/asm-generic/global_data.h
@@ -195,10 +195,6 @@ struct global_data {
         * @dm_root: root instance for Driver Model
         */
        struct udevice *dm_root;
-       /**
-        * @dm_root_f: pre-relocation root instance
-        */
-       struct udevice *dm_root_f;
        /**
         * @uclass_root_s:
         * head of core tree when uclasses are not in read-only memory.
diff --git a/test/dm/core.c b/test/dm/core.c
index 7f3f8d183bc..34e5c163b98 100644
--- a/test/dm/core.c
+++ b/test/dm/core.c
@@ -1007,7 +1007,6 @@ static int dm_test_uclass_before_ready(struct 
unit_test_state *uts)
        ut_assertok(uclass_get(UCLASS_TEST, &uc));
 
        gd->dm_root = NULL;
-       gd->dm_root_f = NULL;
        memset(&gd->uclass_root, '\0', sizeof(gd->uclass_root));
 
        ut_asserteq_ptr(NULL, uclass_find(UCLASS_TEST));
-- 
2.40.1.1.g1c60b9335d

Reply via email to