[PATCH] arm: mx6: module_fuse: fix build failure due to wrong argument name

[PATCH] arm: mx6: module_fuse: fix build failure due to wrong argument name

[Giulio Benetti]

nodeoff variable should be variable off returned by fdt_path_offset() so
let's rename it to off.

Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
 arch/arm/mach-imx/mx6/module_fuse.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-imx/mx6/module_fuse.c b/arch/arm/mach-imx/mx6/module_fuse.c
index 0f4565e311..b58f11c1e5 100644
--- a/arch/arm/mach-imx/mx6/module_fuse.c
+++ b/arch/arm/mach-imx/mx6/module_fuse.c
@@ -206,7 +206,7 @@ int ft_system_setup(void *blob, struct bd_info *bd)
 			if (off < 0)
 				continue; /* Not found, skip it */
 add_status:
-			rc = fdt_setprop(blob, nodeoff, "status", status,
+			rc = fdt_setprop(blob, off, "status", status,
 					 strlen(status) + 1);
 			if (rc) {
 				if (rc == -FDT_ERR_NOSPACE) {
-- 
2.34.1

Re: [PATCH] arm: mx6: module_fuse: fix build failure due to wrong argument name

[Peng Fan]

On 4/7/2023 12:17 AM, Giulio Benetti wrote:
> nodeoff variable should be variable off returned by fdt_path_offset() so
> let's rename it to off.
> 
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>

Reviewed-by: Peng Fan <peng.fan@nxp.com>

Re: [PATCH] arm: mx6: module_fuse: fix build failure due to wrong argument name

[Giulio Benetti]

Hello All,

On 07/04/23 03:52, Peng Fan wrote:
> 
> 
> On 4/7/2023 12:17 AM, Giulio Benetti wrote:
>> nodeoff variable should be variable off returned by fdt_path_offset() so
>> let's rename it to off.
>>
>> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
> 
> Reviewed-by: Peng Fan <peng.fan@nxp.com>

I kindly ping

Best regards
-- 
CEO/CTO@Benetti Engineering sas

Re: [PATCH] arm: mx6: module_fuse: fix build failure due to wrong argument name

[Fabio Estevam]

On Thu, Apr 6, 2023 at 1:17 PM Giulio Benetti
<giulio.benetti@benettiengineering.com> wrote:
>
> nodeoff variable should be variable off returned by fdt_path_offset() so
> let's rename it to off.
>
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>

Reviewed-by: Fabio Estevam <festevam@denx.de>

[PATCH] arm: mx6: module_fuse: fix build failure due to wrong argument name

[sbabic]

> nodeoff variable should be variable off returned by fdt_path_offset() so
> let's rename it to off.
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
> Reviewed-by: Peng Fan <peng.fan@nxp.com>
> Reviewed-by: Fabio Estevam <festevam@denx.de>
Applied to u-boot-imx, master, thanks !

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,        Managing Director: Erika Unter  
HRB 165235 Munich,   Office: Kirchenstr.5, 82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de
=====================================================================