Re: [PATCH] staging: vme_user: Remove unnecessary parentheses

From: Greg Kroah-Hartman

Date: Tue Jul 28 2026 - 03:24:45 EST


On Thu, Jul 23, 2026 at 08:24:17PM +0500, Faisal Mukhtar wrote:
> Fix style problems found by checkpatch.pl,
> which are the unneeded parentheses inside if
> statements in vme.c.
>
> Signed-off-by: Faisal Mukhtar <mukhtarfaisal03@xxxxxxxxx>
> ---
> drivers/staging/vme_user/vme.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/staging/vme_user/vme.c b/drivers/staging/vme_user/vme.c
> index b5c66b66ce32..68b5c4613724 100644
> --- a/drivers/staging/vme_user/vme.c
> +++ b/drivers/staging/vme_user/vme.c
> @@ -1210,9 +1210,9 @@ void vme_bus_error_handler(struct vme_bridge *bridge,
> u32 aspace = vme_get_aspace(am);
>
> list_for_each_entry(handler, &bridge->vme_error_handlers, list) {
> - if ((aspace == handler->aspace) &&
> - (address >= handler->start) &&
> - (address < handler->end)) {
> + if (aspace == handler->aspace &&
> + address >= handler->start &&
> + address < handler->end) {

Please see the archives for why this patch just makes things harder to
understand. checkpatch should have not warned you about this, always be
sure to use the latest version.

thanks,

greg k-h