Received: by 2002:ab2:78c:0:b0:1ec:b906:25e5 with SMTP id h12csp253054lqe; Fri, 23 Feb 2024 03:30:57 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCUaauWX2oC5nn5C5nl/IAl5ESpX82d0qgthNh4DmfoZKS5kmAwERx7ZkTxJB+YnUOsyTiVLom7+N2ciqmGL9ZcHdek7pZDMlq3lC4/VgA== X-Google-Smtp-Source: AGHT+IFEDIMhcmPA59YpdeqhjMj2GBDez/345Jx/2mJECd9Q8LJB6xP28EqLsvHHENTRsJMg3R+h X-Received: by 2002:a17:907:765b:b0:a3f:9de3:59a8 with SMTP id kj27-20020a170907765b00b00a3f9de359a8mr1007329ejc.1.1708687386698; Fri, 23 Feb 2024 03:23:06 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1708687386; cv=pass; d=google.com; s=arc-20160816; b=l8Hf2EPSg/rs0n9iR8eHccH/5gIfakzitrsSqsuzXygKIhZRt3dEE+/f/PgmXs8a8I zrbXhPJEfzOzesjl+C6YzGPB5ktLufQn/Hw58z6I2O6PU08XaPxCjNCij8O3hHrbibjh vSftjZx6VKFJhvYP6KxXcv0mhrnvRNvHyY29DLd9lrKbzi1fUlW3mrPd8LmEJtMcWEsq /D3xaPHMmsYsniGbXRgaQEd2/cx4oU/1XIVdGkFTuPwoqf2UQjURuLsmbls+eVxEsu2P c58Xu2sIOn11eC23xI/fa/7r1/k0QBrBbUUyqdF7eWzTxBS/tmOzVU8eq0gXn55Udxy4 gC1Q== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date:dkim-signature; bh=cJDpxjWdQStrWCNLt6W8vdX2H4z0Oh79zU3jYgZ0ne0=; fh=QT+iGvNcZeaNbQRUR4DJ7wLtNyZTa5ZJCEL+R9E9tc8=; b=dKYalSKB7pEzzHAd2uwu4ICtmA4vMYhxs6DtpPLrsGRguNmA6lEmF3iwA9f28qYyF5 y4cGOaXu6gUlk63cvGeAMmS51MYnIUeqXFjpkb2bbDccuxY+mLsChyVYMEdcdkCTl+gi c2/TgZkyNBJUgS4ThkpsEbEsX/jiZAZSCzW6KS41OD+U0x9o2heUf9W+JzMc0vs9D57I TV8GOGW7od8HzJITgWhNdZrOVJCeHBzJ+xav2wLcjAc3K5q7qR9BGpJooROFkKHVIgc0 B2xK03No+0P9dkUUrlhiJXBJ51tOaRJmuRYgHEFQLHxddRzPQdUfQ0XvXhlvbLj2rvR7 p1vw==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass (test mode) header.i=@ideasonboard.com header.s=mail header.b=V30E3W+e; arc=pass (i=1 spf=pass spfdomain=ideasonboard.com dkim=pass dkdomain=ideasonboard.com); spf=pass (google.com: domain of linux-kernel+bounces-78209-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-78209-linux.lists.archive=gmail.com@vger.kernel.org" Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id z14-20020a170906714e00b00a3e3a38e70esi5321610ejj.901.2024.02.23.03.23.06 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 23 Feb 2024 03:23:06 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-78209-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; dkim=pass (test mode) header.i=@ideasonboard.com header.s=mail header.b=V30E3W+e; arc=pass (i=1 spf=pass spfdomain=ideasonboard.com dkim=pass dkdomain=ideasonboard.com); spf=pass (google.com: domain of linux-kernel+bounces-78209-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-78209-linux.lists.archive=gmail.com@vger.kernel.org" Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id 6A4741F23111 for ; Fri, 23 Feb 2024 11:23:06 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 558057AE49; Fri, 23 Feb 2024 11:22:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="V30E3W+e" Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5A88E78B6B; Fri, 23 Feb 2024 11:22:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=213.167.242.64 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708687365; cv=none; b=L3lskYPmfteIK0/LDgTcxy7gOls2U01SL8FZeZMXO9kg+DmsicsAL4w3FERMhhlpCw1TI+IHn6aa0kuYEEod+2LkSM9adaUB17D/Mqpoc7Ims46hEC9lO/YWskyuq/pnLEqGnl4QgfP49MG2ZHYR/SYH0yB9ta1+czlB8+r455c= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708687365; c=relaxed/simple; bh=JlCjvXv1vhGyP/skKGYCXmNRqNPrHJWqMxafmlRLjxM=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=n94JL50eveR+U213CscG4jS/8Ki9FmtCQDfneK6t5e3XsbY2+a8yjo84ggqIw7KMhL5njeh+7LXLsemoV4llBEJBVG2u1pdp+KKiEvuNE+d+T2NXgEIFxp1vwkQJCqL3uZL4AJQxQUju9181X9L6DjvxRLfJj2mpi1XBI/Gc2g4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ideasonboard.com; spf=pass smtp.mailfrom=ideasonboard.com; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b=V30E3W+e; arc=none smtp.client-ip=213.167.242.64 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ideasonboard.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ideasonboard.com Received: from pendragon.ideasonboard.com (89-27-53-110.bb.dnainternet.fi [89.27.53.110]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id F3E2C2E7; Fri, 23 Feb 2024 12:22:31 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1708687352; bh=JlCjvXv1vhGyP/skKGYCXmNRqNPrHJWqMxafmlRLjxM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=V30E3W+eSPwI9/KBM4hzd8fBqQnH0AqPd5LpHGyLV+YxZ7u5njQ5dmCnZbXRDemcK 9ev1+S4RK88br/e0LlOwMoVSYtywLd5VA5SnEVO2FFKiBoYDH//rCnZFuDv6HWaER0 T5otOiUhnRP85wxfLcZS2CBSrjSxcxs8bte46Ecc= Date: Fri, 23 Feb 2024 13:22:44 +0200 From: Laurent Pinchart To: Mikhail Rudenko Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, Sakari Ailus , Jacopo Mondi , Tommaso Merciai , Christophe JAILLET , Dave Stevenson , Mauro Carvalho Chehab Subject: Re: [PATCH v2 02/20] media: i2c: ov4689: Sort register definitions by address Message-ID: <20240223112244.GM31348@pendragon.ideasonboard.com> References: <20231218174042.794012-1-mike.rudenko@gmail.com> <20231218174042.794012-3-mike.rudenko@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20231218174042.794012-3-mike.rudenko@gmail.com> Hi Mikhail, Thank you for the patch. On Mon, Dec 18, 2023 at 08:40:23PM +0300, Mikhail Rudenko wrote: > Put register defininitions in the order of increasing register > address. > > Signed-off-by: Mikhail Rudenko Reviewed-by: Laurent Pinchart > --- > drivers/media/i2c/ov4689.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/media/i2c/ov4689.c b/drivers/media/i2c/ov4689.c > index ff5213862974..1ae6d9b9c9b3 100644 > --- a/drivers/media/i2c/ov4689.c > +++ b/drivers/media/i2c/ov4689.c > @@ -19,15 +19,15 @@ > #include > #include > > -#define CHIP_ID 0x004688 > -#define OV4689_REG_CHIP_ID 0x300a > - > #define OV4689_XVCLK_FREQ 24000000 > > #define OV4689_REG_CTRL_MODE 0x0100 > #define OV4689_MODE_SW_STANDBY 0x0 > #define OV4689_MODE_STREAMING BIT(0) > > +#define OV4689_REG_CHIP_ID 0x300a > +#define CHIP_ID 0x004688 > + > #define OV4689_REG_EXPOSURE 0x3500 > #define OV4689_EXPOSURE_MIN 4 > #define OV4689_EXPOSURE_STEP 1 > @@ -41,12 +41,12 @@ > #define OV4689_GAIN_STEP 1 > #define OV4689_GAIN_DEFAULT 0x80 > > +#define OV4689_REG_VTS 0x380e > + > #define OV4689_REG_TEST_PATTERN 0x5040 > #define OV4689_TEST_PATTERN_ENABLE 0x80 > #define OV4689_TEST_PATTERN_DISABLE 0x0 > > -#define OV4689_REG_VTS 0x380e > - > #define REG_NULL 0xFFFF > > #define OV4689_REG_VALUE_08BIT 1 -- Regards, Laurent Pinchart