Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <1552325436-8907-1-git-send-email-roy.pledge@nxp.com>
In-Reply-To: <1552325436-8907-1-git-send-email-roy.pledge@nxp.com>
From:   Li Yang <leoyang.li@nxp.com>
Date:   Wed, 1 May 2019 14:00:32 -0500
Message-ID: <CADRPPNRgDN9O2GGd2XyVdByQ8_00MF83ZJ+P8tf13to=-FuN7g@mail.gmail.com>
Subject: Re: [PATCH] soc: fsl: dpio: Increase timeout for QBMan Management Commands
To:     Roy Pledge <roy.pledge@nxp.com>
Cc:     "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        Youri Querry <youri.querry_1@nxp.com>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Mon, Mar 11, 2019 at 12:32 PM Roy Pledge <roy.pledge@nxp.com> wrote:
>
> From: Youri Querry <youri.querry_1@nxp.com>
>
> The timeout for QBMan Management Commands can falsely trigger on a
> busy system. This patch doubles the timeout to avoid the
> false error reports
>
> Signed-off-by: Youri Querry <youri.querry_1@nxp.com>
> Signed-off-by: Roy Pledge <roy.pledge@nxp.com>

Applied for next.  Thanks.

> ---
>  drivers/soc/fsl/dpio/qbman-portal.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/soc/fsl/dpio/qbman-portal.h b/drivers/soc/fsl/dpio/qbman-portal.h
> index 78c700e..f3ec5d2 100644
> --- a/drivers/soc/fsl/dpio/qbman-portal.h
> +++ b/drivers/soc/fsl/dpio/qbman-portal.h
> @@ -1,7 +1,7 @@
>  /* SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause) */
>  /*
>   * Copyright (C) 2014-2016 Freescale Semiconductor, Inc.
> - * Copyright 2016 NXP
> + * Copyright 2016-2019 NXP
>   *
>   */
>  #ifndef __FSL_QBMAN_PORTAL_H
> @@ -433,7 +433,7 @@ static inline int qbman_swp_CDAN_set_context_enable(struct qbman_swp *s,
>  static inline void *qbman_swp_mc_complete(struct qbman_swp *swp, void *cmd,
>                                           u8 cmd_verb)
>  {
> -       int loopvar = 1000;
> +       int loopvar = 2000;
>
>         qbman_swp_mc_submit(swp, cmd, cmd_verb);
>
> --
> 2.7.4
>