RE: [PATCH V2 1/5] remoteproc: fix the wrong default value of is_iomem

Subject: [PATCH V2 1/5] remoteproc: fix the wrong default value of is_iomem

Currently the is_iomem is a random value in the stack which may be default
to true even on those platforms that not use iomem to store firmware.

Cc: Bjorn Andersson <redacted>
Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: Peng Fan <peng.fan@nxp.com>
Fixes: 40df0a91b2a5 ("remoteproc: add is_iomem to da_to_va")
Signed-off-by: Dong Aisheng <aisheng.dong@nxp.com>
---
v1->v2:
 * update rproc_copy_segment as well
---
 drivers/remoteproc/remoteproc_coredump.c   | 2 +-
 drivers/remoteproc/remoteproc_elf_loader.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/remoteproc/remoteproc_coredump.c

b/drivers/remoteproc/remoteproc_coredump.c
index aee657cc08c6..c892f433a323 100644

--- a/drivers/remoteproc/remoteproc_coredump.c
+++ b/drivers/remoteproc/remoteproc_coredump.c

@@ -152,8 +152,8 @@ static void rproc_copy_segment(struct rproc *rproc,

void *dest,
 			       struct rproc_dump_segment *segment,
 			       size_t offset, size_t size)
 {
+	bool is_iomem = false;
 	void *ptr;
-	bool is_iomem;

 	if (segment->dump) {
 		segment->dump(rproc, segment, dest, offset, size); diff --git
a/drivers/remoteproc/remoteproc_elf_loader.c
b/drivers/remoteproc/remoteproc_elf_loader.c
index e8078efb3dec..3cd1798f17a3 100644

--- a/drivers/remoteproc/remoteproc_elf_loader.c
+++ b/drivers/remoteproc/remoteproc_elf_loader.c

@@ -178,8 +178,8 @@ int rproc_elf_load_segments(struct rproc *rproc,

const struct firmware *fw)
 		u64 filesz = elf_phdr_get_p_filesz(class, phdr);
 		u64 offset = elf_phdr_get_p_offset(class, phdr);
 		u32 type = elf_phdr_get_p_type(class, phdr);
+		bool is_iomem = false;
 		void *ptr;
-		bool is_iomem;

 		if (type != PT_LOAD)
 			continue;
--