[U-Boot] [PATCH V2] i.MX6: mx6q_sabrelite: add SATA bindings

Dirk Behme dirk.behme at de.bosch.com
Wed Mar 14 15:53:12 CET 2012

On 13.03.2012 17:59, Eric Nelson wrote:
> --- a/board/freescale/mx6qsabrelite/mx6qsabrelite.c
> +++ b/board/freescale/mx6qsabrelite/mx6qsabrelite.c
> +	/* Enable sata clock */
> +	reg = readl(&imx_ccm->CCGR5); /* CCGR5 */
> +	reg |= MXC_CCM_CCGR5_CG2_MASK;
> +	writel(reg, &imx_ccm->CCGR5);

We touch the CCGR5 already in the imximage.cfg. So we could drop this 
code completely and just add the MXC_CCM_CCGR5_CG2_MASK to the imximage.cfg.

What are the advantages/disadvantages of this?


Less code, touch the register only once in imximage.cfg.


Less readability, doing it in setup_sata() instead of imximage.cfg is 
easier to understand and disable (by removing CONFIG_CMD_SATA).


It sounds like this results in the basic question: Which registers 
should be touched in imximage.cfg, and which by explicit code in 
drivers/board files?

Best regards


More information about the U-Boot mailing list