Message ID | 20240611095239.16026-2-patrice.chotard@foss.st.com |
---|---|
State | Accepted |
Commit | 12dfaa101b852b026ae5a7d14faeec08c73a279b |
Delegated to: | Patrice Chotard |
Headers | show |
Series | [1/2] stm32mp1: spl: Fix compilation warnings | expand |
Hi Patrice On 6/11/24 11:52, Patrice Chotard wrote: > In case node "/reserved-memory/optee" is not found, return -ENOENT > instead of 0. > > Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com> > --- > > arch/arm/mach-stm32mp/stm32mp1/spl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/mach-stm32mp/stm32mp1/spl.c b/arch/arm/mach-stm32mp/stm32mp1/spl.c > index 10abbed87f0..beda69f3359 100644 > --- a/arch/arm/mach-stm32mp/stm32mp1/spl.c > +++ b/arch/arm/mach-stm32mp/stm32mp1/spl.c > @@ -118,7 +118,7 @@ static int optee_get_reserved_memory(uint32_t *start, uint32_t *size) > > node = ofnode_path("/reserved-memory/optee"); > if (!ofnode_valid(node)) > - return 0; > + return -ENOENT; > > fdt_start = ofnode_get_addr_size(node, "reg", &fdt_mem_size); > *start = fdt_start; Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com> Thanks Patrick
On 6/14/24 09:59, Patrick DELAUNAY wrote: > Hi Patrice > > On 6/11/24 11:52, Patrice Chotard wrote: >> In case node "/reserved-memory/optee" is not found, return -ENOENT >> instead of 0. >> >> Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com> >> --- >> >> arch/arm/mach-stm32mp/stm32mp1/spl.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/arm/mach-stm32mp/stm32mp1/spl.c b/arch/arm/mach-stm32mp/stm32mp1/spl.c >> index 10abbed87f0..beda69f3359 100644 >> --- a/arch/arm/mach-stm32mp/stm32mp1/spl.c >> +++ b/arch/arm/mach-stm32mp/stm32mp1/spl.c >> @@ -118,7 +118,7 @@ static int optee_get_reserved_memory(uint32_t *start, uint32_t *size) >> node = ofnode_path("/reserved-memory/optee"); >> if (!ofnode_valid(node)) >> - return 0; >> + return -ENOENT; >> fdt_start = ofnode_get_addr_size(node, "reg", &fdt_mem_size); >> *start = fdt_start; > > > > Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com> > > Thanks > Patrick > > > Applied to u-boot-stm32/master Thanks Patrice
diff --git a/arch/arm/mach-stm32mp/stm32mp1/spl.c b/arch/arm/mach-stm32mp/stm32mp1/spl.c index 10abbed87f0..beda69f3359 100644 --- a/arch/arm/mach-stm32mp/stm32mp1/spl.c +++ b/arch/arm/mach-stm32mp/stm32mp1/spl.c @@ -118,7 +118,7 @@ static int optee_get_reserved_memory(uint32_t *start, uint32_t *size) node = ofnode_path("/reserved-memory/optee"); if (!ofnode_valid(node)) - return 0; + return -ENOENT; fdt_start = ofnode_get_addr_size(node, "reg", &fdt_mem_size); *start = fdt_start;
In case node "/reserved-memory/optee" is not found, return -ENOENT instead of 0. Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com> --- arch/arm/mach-stm32mp/stm32mp1/spl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)