[U-Boot] [PATCH 3/5] imx: mx6qp: Adjust AQos settings for peripherals

[Stefano Babic]

On 08/08/2017 10:21, Peng Fan wrote:
> To resolve USB camera bandwidth issue, the patch sets recommended AQoS
> setting from IC team value for peripheral and only on imx6qp.
> 
> Signed-off-by: Peng Fan <peng.fan at nxp.com>
> Cc: Stefano Babic <sbabic at denx.de>
> Cc: Fabio Estevam <fabio.estevam at nxp.com>
> ---
>  arch/arm/mach-imx/mx6/soc.c | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/arch/arm/mach-imx/mx6/soc.c b/arch/arm/mach-imx/mx6/soc.c
> index 9c3c8dd..e2ecc3a 100644
> --- a/arch/arm/mach-imx/mx6/soc.c
> +++ b/arch/arm/mach-imx/mx6/soc.c
> @@ -362,6 +362,9 @@ static void set_preclk_from_osc(void)
>  
>  int arch_cpu_init(void)
>  {
> +	struct sched_ctrl_regs *regs =
> +		(struct sched_ctrl_regs *)MF2_SCHED_BASE_ADDR;
> +
>  	init_aips();
>  
>  	/* Need to clear MMDC_CHx_MASK to make warm reset work. */
> @@ -431,6 +434,14 @@ int arch_cpu_init(void)
>  
>  	imx_set_wdog_powerdown(false); /* Disable PDE bit of WMCR register */
>  
> +	/*
> +	 * To resolve USB camera bandwidth issue, the patch sets
> +	 * recommended AQoS setting from IC team value for peripheral
> +	 * and only on imx6qp.

I swear, I cannot understand the comment - just that someone has
suggested this. Can you extend the comment explaining why you have to
set the register in this way (or at least, what you set if we do not
know why..)

> +	 */
> +	if (is_mx6dqp())
> +		writel(0x80000201, &regs->aqos[AQOS_FAST1FARIC_ID].priority);
> +
>  	init_src();
>  
>  	return 0;
> 

Best regards,
Stefano Babic

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