From patchwork Tue Sep 17 07:14:42 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Robin van der Gracht X-Patchwork-Id: 58790 X-Patchwork-Delegate: hverkuil@xs4all.nl Received: from vger.kernel.org ([209.132.180.67]) by www.linuxtv.org with esmtp (Exim 4.84_2) (envelope-from ) id 1iA7q5-0000I3-28; Tue, 17 Sep 2019 07:23:53 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727635AbfIQHXv (ORCPT + 1 other); Tue, 17 Sep 2019 03:23:51 -0400 Received: from protonic.xs4all.nl ([83.163.252.89]:37996 "EHLO protonic.nl" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1725985AbfIQHXv (ORCPT ); Tue, 17 Sep 2019 03:23:51 -0400 X-Greylist: delayed 564 seconds by postgrey-1.27 at vger.kernel.org; Tue, 17 Sep 2019 03:23:50 EDT Received: from erd987.prtnl (erd987.prtnl [192.168.237.3]) by sparta (Postfix) with ESMTP id 63AC744A009E; Tue, 17 Sep 2019 09:16:27 +0200 (CEST) From: Robin van der Gracht To: Mauro Carvalho Chehab Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, Marco Felsch , Philipp Zabel , Andreas Pretzsch , Robin van der Gracht Subject: [PATCH] media: i2c: tvp5150: Fix horizontal crop stop boundry Date: Tue, 17 Sep 2019 09:14:42 +0200 Message-Id: <20190917071442.24986-1-robin@protonic.nl> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org The value for AVID stop is relative to the width of the active video area, not the maximum register value. Zero means equal and a negative value means we're cropping on the right side. Signed-off-by: Robin van der Gracht --- drivers/media/i2c/tvp5150.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/i2c/tvp5150.c b/drivers/media/i2c/tvp5150.c index f47cb9a023fb..6bc65ab5e8ab 100644 --- a/drivers/media/i2c/tvp5150.c +++ b/drivers/media/i2c/tvp5150.c @@ -1231,10 +1231,10 @@ __tvp5150_set_selection(struct v4l2_subdev *sd, struct v4l2_rect rect) regmap_write(decoder->regmap, TVP5150_ACT_VD_CROP_ST_LSB, rect.left | (1 << TVP5150_CROP_SHIFT)); regmap_write(decoder->regmap, TVP5150_ACT_VD_CROP_STP_MSB, - (rect.left + rect.width - TVP5150_MAX_CROP_LEFT) >> + (rect.left + rect.width - TVP5150_H_MAX) >> TVP5150_CROP_SHIFT); regmap_write(decoder->regmap, TVP5150_ACT_VD_CROP_STP_LSB, - rect.left + rect.width - TVP5150_MAX_CROP_LEFT); + rect.left + rect.width - TVP5150_H_MAX); } static int tvp5150_set_selection(struct v4l2_subdev *sd,