From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f47.google.com (mail-lf1-f47.google.com [209.85.167.47]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 935613B4EBC for ; Wed, 12 Aug 2026 20:55:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.47 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786568139; cv=none; b=OGnoDUXffwi3bW6lxy/p+0DnuwE3uLCCY8JG89S7vOcyz5eDJT5i7wLXbG+nZCpHj4GzV5ooQoeRKJcE8VivtZXk+HUDLtfzYm520S+VmJbqmGTR5G4P/nXc5cPvDHv7uSw2L9AkRp6DLTLQsiBWK7ufW35njdy39PRNW/wDaHo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786568139; c=relaxed/simple; bh=N7YavMIxn189yOCIt8BdETPEDeHw4RDxuO0Znx6qmho=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=ev3aZuf+JM9RVFXQYT2a6QTg+hGitMtHGQutOl9pkWfgEkt1eM67D+rN7gEbCD0UkKPcQB/ZD4LRy44IBZhFlYUm5LN+tazMoqRJlE6lLesAWGXD7vGlPsV3DK+A8BRNRHF4AqINkVvx0qjHThZnOzXcn3kBX/PfCJSaZ9NqKdw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=ZDJZ5eOG; arc=none smtp.client-ip=209.85.167.47 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="ZDJZ5eOG" Received: by mail-lf1-f47.google.com with SMTP id 2adb3069b0e04-5b0f19bea2fso1802692e87.1 for ; Wed, 12 Aug 2026 13:55:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786568126; x=1787172926; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=id+z5c34tokzsRgaGzBKaJJ/gVoWUF0wZyeg82CjYJg=; b=ZDJZ5eOG0YGhZAy9L3nzUtLrcINZKIvprA+DlcvTOu3vNA/izhBXmTotxaAAL5Weyv FA4MIlLIRkL8W/KtWlugW05WNSlyviHjzjp+R8aJq76KINvk4m+MYWirD71liPrHhYjX VTm9ET8zQuqO4R3OG9oa0xG8Dx7dVMaAG6raJTCXWIoVjeA+vVGKUIIcwEQHg1JZOKxT HiZL7h1RIjWhA7e0PNieMmvxxMjASTOJR2oZre2+4gypCWdieToEpQDxXVggQMZ98+IW p1079ryIHKwHrt1wywBkUAcgH6oU7zS8DCrSRVG9G5DVTpoX5bL8LisLATC9WejvThrS HuCw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786568126; x=1787172926; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=id+z5c34tokzsRgaGzBKaJJ/gVoWUF0wZyeg82CjYJg=; b=RWLM1S6KZkjJVNPjtWvls0FBqHogrSQRcWpgxMNyYwHE184cKNPsI2zBw//GM91UvH PFqZnvEmtIYvxABoTJyIam23ZQUzXHV6okLVIzaD4uKDpNiAsh1gZImp/okj3vPn8GUX fZn45KzPQWXHbata1FRfZxsa1c7ahuWWwB/DFILSDuDX4SQw7JRy+kUVoHc9nxWx9jR1 2D6A5PAb6364vNc4asBOvRsN7WPlNgrRrVV3ReL3eP9izvA0OrE4SGJZVyVpduIql7IN gx9mHNk55V+TehcHsxoJoM6oU0UPhLcUtWJs0LmwYR9EdtOzzuLSnMZeOnryvX4mq18Y wu1A== X-Forwarded-Encrypted: i=1; AHgh+Rr4PG98OJCmbxyMyWDP5gOuRqxGLq+KcCw0gcWZUzsY2eQwwZJo1dibGhElesan3f5tPylYUoBFxDpADuI=@vger.kernel.org X-Gm-Message-State: AOJu0YwpgzgUV7hTB0F9786key7kAh6jat9qE8uoCO2q0aAjZt5ZGO4I xpN+S/IsnbJ0fO8kgGxqh8YWM31XoG3zjSfqL4bh6VtFqiDfFg2ofbW+ X-Gm-Gg: AR+sD128OrK13S6BAbIJdQuNbMtppbK6WSvdlfk8bUnHX3ekIMWIMdSAk32bpbUmMgi W4R3qMNyBfoK7Rg+Pb86x7WQOnOCMM7zaDJ0Na9ktnaJkHOzEiYxjp4Kj4UkFeyurOniwf3ka/Y XxyC8HnGV1OnFXywjzcz4Dj/Y+dlQ5hZf6jTg3h3DVuTZ88fpPKnk52ik1IWf0mCcXYUU0bahiu SxLm0iztBIO6S1FHWkzVWts9ZvxO4bRhB2l82oI25/Ry1Z5OSO/1Ksg1i7t7TH4dGlD2BrcOokc 5VtM1S2uuIK5w0/mSwZMjDP/Vgf6defpQ02fxImBn3wx4+quMgeMpvzUmLQWQTNslONOOqNDHdd zDfAV/Sw6GigOadv3IpwIKdbwMsGD+sMiUo/yf+Fz4Bsjj2592mTlNQJy1vaQxEwySy8kF3T7db CQTzU8/eciqtOe6cJ7GXtZ7gYGTsAnvgX+hLjYbht+NWNTdGpW9FdBQiL8ymY= X-Received: by 2002:a05:6512:805b:b0:5ae:b86b:71f0 with SMTP id 2adb3069b0e04-5b453f830e2mr30498e87.37.1786568125682; Wed, 12 Aug 2026 13:55:25 -0700 (PDT) Received: from archlinux ([94.229.19.199]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5b4539e9605sm60999e87.22.2026.08.12.13.55.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 12 Aug 2026 13:55:25 -0700 (PDT) From: Nikolay Kulikov To: Greg Kroah-Hartman , Hans de Goede , Mauro Carvalho Chehab , Sakari Ailus , Andy Shevchenko , Andy Shevchenko Cc: linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Nikolay Kulikov Subject: [PATCH v3 3/5] staging: media: atomisp: inline the check_bo_status_*() macros Date: Wed, 12 Aug 2026 23:51:36 +0300 Message-ID: <20260812205429.39181-4-nikolayof23@gmail.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260812205429.39181-1-nikolayof23@gmail.com> References: <20260812205429.39181-1-nikolayof23@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit These macros check a bit in the bo->status field. Replace them with conditional expressions to make the code clearer. Signed-off-by: Nikolay Kulikov --- .../media/atomisp/include/hmm/hmm_bo.h | 12 -------- .../media/atomisp/include/hmm/hmm_common.h | 18 ----------- .../staging/media/atomisp/pci/hmm/hmm_bo.c | 30 ++++++++++++------- 3 files changed, 20 insertions(+), 40 deletions(-) diff --git a/drivers/staging/media/atomisp/include/hmm/hmm_bo.h b/drivers/staging/media/atomisp/include/hmm/hmm_bo.h index e974ab5ca2fc..69cf490bd88c 100644 --- a/drivers/staging/media/atomisp/include/hmm/hmm_bo.h +++ b/drivers/staging/media/atomisp/include/hmm/hmm_bo.h @@ -19,18 +19,6 @@ #include "hmm/hmm_common.h" #include "ia_css_types.h" -#define check_bo_status_yes_goto(bo, _status, label) \ - var_not_equal_goto((bo->status & (_status)), (_status), \ - label, \ - "HMM buffer status not contain %s.\n", \ - #_status) - -#define check_bo_status_no_goto(bo, _status, label) \ - var_equal_goto((bo->status & (_status)), (_status), \ - label, \ - "HMM buffer status contains %s.\n", \ - #_status) - #define rbtree_node_to_hmm_bo(root_node) \ container_of((root_node), struct hmm_buffer_object, node) diff --git a/drivers/staging/media/atomisp/include/hmm/hmm_common.h b/drivers/staging/media/atomisp/include/hmm/hmm_common.h index f215130e5e17..c406fe8b1345 100644 --- a/drivers/staging/media/atomisp/include/hmm/hmm_common.h +++ b/drivers/staging/media/atomisp/include/hmm/hmm_common.h @@ -33,22 +33,4 @@ } \ } while (0) -#define var_equal_goto(var1, var2, label, fmt, arg ...) \ - do { \ - if ((var1) == (var2)) { \ - dev_err(atomisp_dev, \ - fmt, ## arg); \ - goto label;\ - } \ - } while (0) - -#define var_not_equal_goto(var1, var2, label, fmt, arg ...) \ - do { \ - if ((var1) != (var2)) { \ - dev_err(atomisp_dev, \ - fmt, ## arg); \ - goto label;\ - } \ - } while (0) - #endif diff --git a/drivers/staging/media/atomisp/pci/hmm/hmm_bo.c b/drivers/staging/media/atomisp/pci/hmm/hmm_bo.c index ecd920e49488..dbda4394a239 100644 --- a/drivers/staging/media/atomisp/pci/hmm/hmm_bo.c +++ b/drivers/staging/media/atomisp/pci/hmm/hmm_bo.c @@ -687,7 +687,10 @@ int hmm_bo_alloc_pages(struct hmm_buffer_object *bo, } mutex_lock(&bo->mutex); - check_bo_status_no_goto(bo, HMM_BO_PAGE_ALLOCED, status_err); + if (bo->status & HMM_BO_PAGE_ALLOCED) { + dev_err(atomisp_dev, "HMM buffer status contains HMM_BO_PAGE_ALLOCED.\n"); + goto status_err; + } bo->pages = kzalloc_objs(struct page *, bo->pgnr); if (unlikely(!bo->pages)) { @@ -738,7 +741,10 @@ void hmm_bo_free_pages(struct hmm_buffer_object *bo) mutex_lock(&bo->mutex); - check_bo_status_yes_goto(bo, HMM_BO_PAGE_ALLOCED, status_err2); + if (!(bo->status & HMM_BO_PAGE_ALLOCED)) { + dev_err(atomisp_dev, "HMM buffer status not contain HMM_BO_PAGE_ALLOCED.\n"); + goto status_err2; + } /* clear the flag anyway. */ bo->status &= (~HMM_BO_PAGE_ALLOCED); @@ -788,11 +794,15 @@ int hmm_bo_bind(struct hmm_buffer_object *bo) mutex_lock(&bo->mutex); - check_bo_status_yes_goto(bo, - HMM_BO_PAGE_ALLOCED | HMM_BO_ALLOCED, - status_err1); + if (!(bo->status & (HMM_BO_PAGE_ALLOCED | HMM_BO_ALLOCED))) { + dev_err(atomisp_dev, "HMM buffer status not contain HMM_BO_PAGE_ALLOCED | HMM_BO_ALLOCED.\n"); + goto status_err1; + } - check_bo_status_no_goto(bo, HMM_BO_BINDED, status_err2); + if (bo->status & HMM_BO_BINDED) { + dev_err(atomisp_dev, "HMM buffer status contains HMM_BO_BINDED.\n"); + goto status_err2; + } bdev = bo->bdev; @@ -868,10 +878,10 @@ void hmm_bo_unbind(struct hmm_buffer_object *bo) mutex_lock(&bo->mutex); - check_bo_status_yes_goto(bo, - HMM_BO_PAGE_ALLOCED | - HMM_BO_ALLOCED | - HMM_BO_BINDED, status_err); + if (!(bo->status & (HMM_BO_PAGE_ALLOCED | HMM_BO_ALLOCED | HMM_BO_BINDED))) { + dev_err(atomisp_dev, "HMM buffer status not contain HMM_BO_PAGE_ALLOCED | HMM_BO_ALLOCED | HMM_BO_BINDED.\n"); + goto status_err; + } bdev = bo->bdev; -- 2.55.0