[U-Boot] [PATCH] drivers: net: fsl_mc: Compare pointer value qbman_swp_mc_start
Prabhakar Kushwaha
prabhakar.kushwaha at nxp.com
Fri Jan 15 08:05:40 CET 2016
> -----Original Message-----
> From: Pratiyush Srivastava
> Sent: Tuesday, December 22, 2015 4:50 PM
> To: u-boot at lists.denx.de
> Cc: Yusong Sun <yorksun at freescale.com>; Prabhakar Kushwaha
> <prabhakar at freescale.com>; Pratiyush Srivastava
> <pratiyush.srivastava at freescale.com>
> Subject: [PATCH] drivers: net: fsl_mc: Compare pointer value
> qbman_swp_mc_start
>
> Current code compares the return pointer of function
> qbman_cena_write_start with NULL. Instead the value of the return pointer
> should be compared.
> Signed-off-by: Pratiyush Mohan Srivastava
> <pratiyush.srivastava at freescale.com>
> ---
> drivers/net/fsl-mc/dpio/qbman_portal.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/fsl-mc/dpio/qbman_portal.c b/drivers/net/fsl-
> mc/dpio/qbman_portal.c
> index 449ff8a..4b64c8a 100644
> --- a/drivers/net/fsl-mc/dpio/qbman_portal.c
> +++ b/drivers/net/fsl-mc/dpio/qbman_portal.c
> @@ -102,12 +102,14 @@ struct qbman_swp *qbman_swp_init(const struct
> qbman_swp_desc *d) void *qbman_swp_mc_start(struct qbman_swp *p)
> {
> void *ret;
> + int *return_val;
> #ifdef QBMAN_CHECKING
> BUG_ON(p->mc.check != swp_mc_can_start); #endif
> ret = qbman_cena_write_start(&p->sys, QBMAN_CENA_SWP_CR);
> #ifdef QBMAN_CHECKING
> - if (!ret)
> + return_val = (int *)ret;
> + if (!(*return_val))
> p->mc.check = swp_mc_can_submit;
> #endif
> return ret;
> --
Acked-by: Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>
--prabhakar
More information about the U-Boot
mailing list