From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 99678C35272 for ; Mon, 7 Feb 2022 13:13:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1448561AbiBGNL6 (ORCPT ); Mon, 7 Feb 2022 08:11:58 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47512 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1445240AbiBGMlq (ORCPT ); Mon, 7 Feb 2022 07:41:46 -0500 Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 52BDBE031016 for ; Mon, 7 Feb 2022 04:33:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1644237197; x=1675773197; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=vhBTmSn8UkwEjJUl4ynGb5kIz8nOXSvEwd6JGQ6YNy8=; b=ABFF0jbt9bD8QBR9MYcd7MFWyf5rwaJVjImMDNTHcMLkYTQI1qa1+AQI rpx0pCFIG5Sd5BhE/l2FqAZLBaQtibUz36FnsWqMF37AdyD/9ahJqT5Al oxz5GWWh6HraxHYNAO5zebXZ+3rAPqAHgUZ88zHpeVM82lC/uqfjiAo3A d7YzbKIHC7b8+HOIZFAah7MaqAOw9GIphfPZi3ZoiJ5r8jJzsEteonG7P qz5cV9GBTob6siimsajLIEKYbubAZLHd7WVq89ID94uAR30FcKCqtmy7k jmG6lQnsLgnQAlznypBX9RiYoZnluWunWMyya6VlD93+gm/0IZCyn/UlZ Q==; X-IronPort-AV: E=McAfee;i="6200,9189,10250"; a="228674150" X-IronPort-AV: E=Sophos;i="5.88,349,1635231600"; d="scan'208";a="228674150" Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Feb 2022 04:32:13 -0800 X-IronPort-AV: E=Sophos;i="5.88,349,1635231600"; d="scan'208";a="621535538" Received: from smile.fi.intel.com ([10.237.72.61]) by fmsmga003-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Feb 2022 04:32:11 -0800 Received: from andy by smile.fi.intel.com with local (Exim 4.95) (envelope-from ) id 1nH3Al-001tq2-Md; Mon, 07 Feb 2022 14:31:11 +0200 Date: Mon, 7 Feb 2022 14:31:11 +0200 From: Andy Shevchenko To: Moses Christopher Bollavarapu Cc: gregkh@linuxfoundation.org, joe@perches.com, kitakar@gmail.com, laurent.pinchart@ideasonboard.com, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-staging@lists.linux.dev, mchehab@kernel.org, sakari.ailus@linux.intel.com, tomi.valkeinen@ideasonboard.com, dan.carpenter@oracle.com Subject: Re: [PATCH] staging: media: atomisp: Use GENMASK instead of manual masking Message-ID: References: <20220206185830.39360-1-mosescb.dev@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220206185830.39360-1-mosescb.dev@gmail.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Feb 06, 2022 at 07:58:30PM +0100, Moses Christopher Bollavarapu wrote: > There is a GENMASK macro available in linux/bits.h, for masking. > > Example: GENMASK(3, 0) = 0b00001111 (same as (1 << 4) - 1) Reviewed-by: Andy Shevchenko > Signed-off-by: Moses Christopher Bollavarapu > --- > drivers/staging/media/atomisp/pci/atomisp_compat_css20.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/drivers/staging/media/atomisp/pci/atomisp_compat_css20.c b/drivers/staging/media/atomisp/pci/atomisp_compat_css20.c > index 1173be0e72b0..a345fa1d7de9 100644 > --- a/drivers/staging/media/atomisp/pci/atomisp_compat_css20.c > +++ b/drivers/staging/media/atomisp/pci/atomisp_compat_css20.c > @@ -40,6 +40,7 @@ > > #include > #include > +#include Probably this can be still ordered. > /* Assume max number of ACC stages */ > #define MAX_ACC_STAGES 20 > @@ -1913,11 +1914,11 @@ void atomisp_css_input_set_mode(struct atomisp_sub_device *asd, > &asd->stream_env[ATOMISP_INPUT_STREAM_GENERAL].stream_config; > s_config->mode = IA_CSS_INPUT_MODE_TPG; > s_config->source.tpg.mode = IA_CSS_TPG_MODE_CHECKERBOARD; > - s_config->source.tpg.x_mask = (1 << 4) - 1; > + s_config->source.tpg.x_mask = GENMASK(3, 0); > s_config->source.tpg.x_delta = -2; > - s_config->source.tpg.y_mask = (1 << 4) - 1; > + s_config->source.tpg.y_mask = GENMASK(3, 0); > s_config->source.tpg.y_delta = 3; > - s_config->source.tpg.xy_mask = (1 << 8) - 1; > + s_config->source.tpg.xy_mask = GENMASK(7, 0); > return; > } > > -- > 2.30.2 > -- With Best Regards, Andy Shevchenko