Hi Christoph,
On 14/10/19 14:27, Christoph Niedermaier wrote:
A return type of u08 cuts the return value always to 0, because IMX6_SRC_GPR10_BMODE = bit 28 is above u08. Changing the return type to u32 gives the function the expected operation.
Fixes: cba586b400 ("imx6: Add imx6_src_get_boot_mode") Signed-off-by: Christoph Niedermaier cniedermaier@dh-electronics.com
arch/arm/include/asm/mach-imx/sys_proto.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/include/asm/mach-imx/sys_proto.h b/arch/arm/include/asm/mach-imx/sys_proto.h index aa66fdc88f..5222f6957d 100644 --- a/arch/arm/include/asm/mach-imx/sys_proto.h +++ b/arch/arm/include/asm/mach-imx/sys_proto.h @@ -99,7 +99,7 @@ enum imx6_bmode { IMX6_BMODE_NAND_MAX = 0xf, };
-static inline u8 imx6_is_bmode_from_gpr9(void) +static inline u32 imx6_is_bmode_from_gpr9(void) { return readl(&src_base->gpr10) & IMX6_SRC_GPR10_BMODE; }
This is correct but it rather conflicts with another patch I already applied. Could you rebase and repost ? Thanks !
Best regards, Stefano Babic