[U-Boot] [PATCH V2 6/6] arm: mx5: Add support for DENX M53EVK
Marek Vasut
marex at denx.de
Thu Apr 25 22:12:40 CEST 2013
Dear Fabio Estevam,
[..]
> >> Shouldn't this function be placed into a common mx5 location? mx53ard
> >> uses the same.
> >
> > The WEIM and M4IF configuration is board-specific.
>
> Right, understood. It seems like a duplication of code from mx53ard,
> but anyway, what about:
>
> clrbits_le32(WEIM_BASE_ADDR + i, WEIM_GCR2_MUX16_BYP_GRANT) ?
>
> Accessing registers via offsets is not the best practice in U-boot.
>
> What about the weim struct defined at
> arch/arm/include/asm/arch-mx5/imx-regs.h
>
> for acessing such registers?
Yes, I agree, but you won't be able to do that in a loop as above if accessed
via struct-based access so I'm keeping this.
Best regards,
Marek Vasut
More information about the U-Boot
mailing list