Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Sasha Levin <Alexander.Levin@microsoft.com>
To:     "stable@vger.kernel.org" <stable@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
CC:     Todor Tomov <todor.tomov@linaro.org>,
        Sakari Ailus <sakari.ailus@linux.intel.com>,
        Mauro Carvalho Chehab <mchehab+samsung@kernel.org>,
        Sasha Levin <Alexander.Levin@microsoft.com>
Subject: [PATCH AUTOSEL 4.18 32/88] media: ov5645: Supported external clock is
 24MHz
Thread-Topic: [PATCH AUTOSEL 4.18 32/88] media: ov5645: Supported external
 clock is 24MHz
Thread-Index: AQHURkLIvYlZYLQ7M0W/N1KAaslVcA==
Date:   Fri, 7 Sep 2018 00:36:15 +0000
Message-ID: <20180907003547.57567-32-alexander.levin@microsoft.com>
References: <20180907003547.57567-1-alexander.levin@microsoft.com>
In-Reply-To: <20180907003547.57567-1-alexander.levin@microsoft.com>
Accept-Language: en-US
Content-Language: en-US
received-spf: None (protection.outlook.com: microsoft.com does not designate
 permitted sender hosts)
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 80bfc6a1-83d7-46a1-6827-08d61459fd1a
X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Sep 2018 00:36:15.1176
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR21MB0469
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

From: Todor Tomov <todor.tomov@linaro.org>

[ Upstream commit 4adb0a0432f489c5eb802b33dae7737f69e6fd7a ]

The external clock frequency was set to 23.88MHz by mistake
because of a platform which cannot get closer to 24MHz.
The supported by the driver external clock is 24MHz so
set it correctly and also fix the values of the pixel
clock and link clock.
However allow 1% tolerance to the external clock as this
difference is small enough to be insignificant.

Signed-off-by: Todor Tomov <todor.tomov@linaro.org>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
---
 drivers/media/i2c/ov5645.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/drivers/media/i2c/ov5645.c b/drivers/media/i2c/ov5645.c
index b3f762578f7f..1722cdab0daf 100644
--- a/drivers/media/i2c/ov5645.c
+++ b/drivers/media/i2c/ov5645.c
@@ -510,8 +510,8 @@ static const struct reg_value ov5645_setting_full[] =3D=
 {
 };
=20
 static const s64 link_freq[] =3D {
-	222880000,
-	334320000
+	224000000,
+	336000000
 };
=20
 static const struct ov5645_mode_info ov5645_mode_info_data[] =3D {
@@ -520,7 +520,7 @@ static const struct ov5645_mode_info ov5645_mode_info_d=
ata[] =3D {
 		.height =3D 960,
 		.data =3D ov5645_setting_sxga,
 		.data_size =3D ARRAY_SIZE(ov5645_setting_sxga),
-		.pixel_clock =3D 111440000,
+		.pixel_clock =3D 112000000,
 		.link_freq =3D 0 /* an index in link_freq[] */
 	},
 	{
@@ -528,7 +528,7 @@ static const struct ov5645_mode_info ov5645_mode_info_d=
ata[] =3D {
 		.height =3D 1080,
 		.data =3D ov5645_setting_1080p,
 		.data_size =3D ARRAY_SIZE(ov5645_setting_1080p),
-		.pixel_clock =3D 167160000,
+		.pixel_clock =3D 168000000,
 		.link_freq =3D 1 /* an index in link_freq[] */
 	},
 	{
@@ -536,7 +536,7 @@ static const struct ov5645_mode_info ov5645_mode_info_d=
ata[] =3D {
 		.height =3D 1944,
 		.data =3D ov5645_setting_full,
 		.data_size =3D ARRAY_SIZE(ov5645_setting_full),
-		.pixel_clock =3D 167160000,
+		.pixel_clock =3D 168000000,
 		.link_freq =3D 1 /* an index in link_freq[] */
 	},
 };
@@ -1145,7 +1145,8 @@ static int ov5645_probe(struct i2c_client *client,
 		return ret;
 	}
=20
-	if (xclk_freq !=3D 23880000) {
+	/* external clock must be 24MHz, allow 1% tolerance */
+	if (xclk_freq < 23760000 || xclk_freq > 24240000) {
 		dev_err(dev, "external clock frequency %u is not supported\n",
 			xclk_freq);
 		return -EINVAL;
--=20
2.17.1