[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Fri, 11 Nov 2016 14:23:26 +0300
From: Dan Carpenter <dan.carpenter@...cle.com>
To: Shiva Kerdel <shiva@...ev.nl>
Cc: stuart.yoder@....com, devel@...verdev.osuosl.org,
German.Rivera@...escale.com, gregkh@...uxfoundation.org,
nipun.gupta@....com, linux-kernel@...r.kernel.org,
german.rivera@....com, treding@...dia.com, itai.katz@....com
Subject: Re: [PATCH 1/2] Staging: fsl-mc: include: mc: Kernel type 's16'
preferred over 'int16_t'
On Fri, Nov 11, 2016 at 12:07:39PM +0100, Shiva Kerdel wrote:
> Follow the kernel type preferrences of using 's16' over 'int16_t'.
>
> Signed-off-by: Shiva Kerdel <shiva@...ev.nl>
> Acked-by: Stuart Yoder <stuart.yoder@....com>
> ---
> Changes for v2:
> - corrected an error in the log message, wrote 's32' instead of 's16'.
> Changes for v3:
> - added the missing annotates.
> Changes for v4:
> - corrected patch subject to version 4.
>
> drivers/staging/fsl-mc/include/mc-bus.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/fsl-mc/include/mc-bus.h b/drivers/staging/fsl-mc/include/mc-bus.h
> index e915574..c7cad87 100644
> --- a/drivers/staging/fsl-mc/include/mc-bus.h
> +++ b/drivers/staging/fsl-mc/include/mc-bus.h
> @@ -42,8 +42,8 @@ struct msi_domain_info;
> */
> struct fsl_mc_resource_pool {
> enum fsl_mc_pool_type type;
> - int16_t max_count;
> - int16_t free_count;
> + s16 max_count;
My understanding is that this has to be signed because the design of
this driver is that we keep adding devices until the the counter
overflows. After that there are a couple tests for
"if (WARN_ON(res_pool->max_count < 0)) " which prevent the driver from
working again.
This all seems pretty horrible.
regards,
dan carpenter
Powered by blists - more mailing lists