From nobody Fri Jun 12 20:21:14 2026 Received: from mail-wr1-f47.google.com (mail-wr1-f47.google.com [209.85.221.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 09B1A3D0BE7 for ; Tue, 12 May 2026 16:31:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.47 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778603480; cv=none; b=c3fOpdsJx3J9DN5zKhsySBUYdH26qrOBM0vHPIgaMOF0woCVLqFrEoHh5AwY7/+xhhAw7SBU180cWSaHV+exjGzG7MVbpBnmUcE1MTGzH/WrpY0L8cVWl9548PIAdff8QHXYHsW9DD+pBcNvZ2KZ4UP9cR64aiIH7Xja5+ec/Kw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778603480; c=relaxed/simple; bh=1J6EFe9turiLVjiZ1yIAw0MF+G7F+RRsz0gWd3gm1E0=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=My73+X+idrW1TNjaBb7xL+XtgrQFLIy8jPfaT+KS+KkscHu1WKO/hK8eFIW2SYQ1nwqYJG1ufltJsXsPrmWT5vx3pvG4HZN1xN5RU+FxpVPhzRRWwHNx1RVhNJv7aEOSHcSjiWDn8x0CZF2x8RpJOQEaBPC7ZE96NqargJFgq2I= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nimblex.net; spf=pass smtp.mailfrom=nimblex.org; dkim=pass (1024-bit key) header.d=nimblex.net header.i=@nimblex.net header.b=bbsTl/0I; arc=none smtp.client-ip=209.85.221.47 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nimblex.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nimblex.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=nimblex.net header.i=@nimblex.net header.b="bbsTl/0I" Received: by mail-wr1-f47.google.com with SMTP id ffacd0b85a97d-44e5624c053so3229503f8f.2 for ; Tue, 12 May 2026 09:31:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nimblex.net; s=google; t=1778603477; x=1779208277; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=WDpsU1Uf1Jvu40Vzfn0+jmnHCHZtJ2XXTAc7BhkcuAQ=; b=bbsTl/0Iane3p0XHocFbCIjE0l7Bkv0fit8JpF9cE0fSnMwq3pkDo19GSzrMh3hBmm c3m1uWzNg7C+GsgWqDa0AzaVjKuKjSec7LS6H7Ug18YOpmwm/NZ1OEUD9S8pVwXFmlvk BLCxO/WQX1z7m7Z1CC0jl7gFbxrsnFPzmAHbA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778603477; x=1779208277; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=WDpsU1Uf1Jvu40Vzfn0+jmnHCHZtJ2XXTAc7BhkcuAQ=; b=HEkb4HWICAtOeRBaRkZ+lo3LS98nUx3JPa7uMU4xwaOsmzUMxCgtxxmEMME5Na5zeY CYLwotuvYzIT+e70mXafUhON3ItkMRAnqP4+TtgtRW1jPsw76Omi33aJaG+zHcJapheo Edy6RiMHBdRIJNARi6kz+Hr9nkRDgo3okjFwxSf6K2C+5CVjOxDfBtP1uS7AREQTbARs +4SjXAI2GXhunMd9Evs2wqXKRnexMjptM5+eCfOeQVi/PC/rG5YMx3W4irhnomNgVYlz D/IM9BMnufZC+ROi3Fr+FZH6x+Qp0F7vIv+GXvLV4xX3tcXosDlvU+XZEC0aFbTg5snP oGJg== X-Forwarded-Encrypted: i=1; AFNElJ8KPGUqIvDBQlu/Xne9mie293s+JqQbblGdfwVc6K3wukSftZ/uPmkS8UxavcrVvfXm4UtDocEGrzVhXvY=@vger.kernel.org X-Gm-Message-State: AOJu0YziJVBjesEptcuT2PbPIyjAVNWnM/9N48QzOYZVPiQ8xjHjJw+P VSlmHW/qDPf7cAWNrbonQdSIWq8xLwzwMUtbjM6K3ELRnz/88vxyJSd1xQf163+llig= X-Gm-Gg: Acq92OFzqvEwH1MyFl22mcDKsQzAjTX6HXBWNM+bcD84b4Cj73dgPeXPg1yoiwwmgHJ zf5tLfkZiPJSxBQQ0wkTxBbv+fcbz4ESvI3gyglBCk1+mSx+sfYs47JbmRuKiba1IcfxPEo6HBP lWh8Xl6KSs0mpfds36IgEdFHelSBivXJqlNt+LXE1wACtV58Ffty7z7XWGHUx7ybjSqW/Fd3xoP OL6PtwLNcw5YqRM68QT2mMRsBLZxLuYHJkbzdN0hH0aJ8lJcGoDQiRsjSZqEQ5cAMqQ7D/dR+dd JyMa4/yhFCdyEktC1uoUqfLI/PHZ47dVoJ1xK5t6dvvOxEx5A1gLwP02NkkdQxqi8ToXpzX3njP s5wO7DjD7VFtaBV6z7Vnb1lfabula4KqZ6S2mEvFVXDrwK+1NsA8MSXeyBNoGq+sueMuEvHuE61 wCuD30yiBaCZxvDc1rTFk= X-Received: by 2002:a5d:5f53:0:b0:43e:a69b:d810 with SMTP id ffacd0b85a97d-4515d9a0839mr48268663f8f.38.1778603477309; Tue, 12 May 2026 09:31:17 -0700 (PDT) Received: from nimblex ([2a02:2f0c:590f:dd00:8c23:d8de:dbaa:cf0c]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-45492271510sm40437592f8f.37.2026.05.12.09.31.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 May 2026 09:31:16 -0700 (PDT) From: Bogdan Radulescu To: linux-media@vger.kernel.org Cc: Sakari Ailus , Jimmy Su , Mauro Carvalho Chehab , Hans de Goede , linux-kernel@vger.kernel.org, Bogdan Radulescu Subject: [PATCH] media: i2c: ov08x40: implement .get_selection Date: Tue, 12 May 2026 19:30:10 +0300 Message-ID: <20260512163010.2893978-1-bogdan@nimblex.net> X-Mailer: git-send-email 2.54.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" libcamera and other V4L2 subdev clients use VIDIOC_SUBDEV_G_SELECTION to query the sensor's native size and crop bounds, as required by Documentation/driver-api/media/camera-sensor.rst. ov08x40 doesn't implement the pad op, so libcamera falls back to a defaulted geometry and complains at every probe: ov08x40 N-0010: Failed to retrieve the sensor crop rectangle ov08x40 N-0010: The sensor kernel driver needs to be fixed The sensor has no optical-black margin and no analogue/digital crop in any of its supported modes; native size, crop bounds and crop default all coincide with the full 3856x2416 pixel array. Report that explicitly. While at it, add OV08X40_NATIVE_WIDTH / OV08X40_NATIVE_HEIGHT for the dimensions and drop the now-misleading 'No crop or compose' comment from ov08x40_open(). Tested on a Lenovo ThinkPad X1 Carbon Gen 13 (Lunar Lake, IPU7): the warnings above are gone and libcamera's simple pipeline reads back the correct rectangles. Signed-off-by: Bogdan Radulescu --- drivers/media/i2c/ov08x40.c | 28 +++++++++++++++++++++++++++- 1 file changed, 27 insertions(+), 1 deletion(-) diff --git a/drivers/media/i2c/ov08x40.c b/drivers/media/i2c/ov08x40.c index 5eaf454f4763..31062eedbd71 100644 --- a/drivers/media/i2c/ov08x40.c +++ b/drivers/media/i2c/ov08x40.c @@ -38,6 +38,10 @@ #define OV08X40_REG_CHIP_ID 0x300a #define OV08X40_CHIP_ID 0x560858 =20 +/* Pixel array */ +#define OV08X40_NATIVE_WIDTH 3856U +#define OV08X40_NATIVE_HEIGHT 2416U + /* V_TIMING internal */ #define OV08X40_REG_VTS 0x380e #define OV08X40_VTS_30FPS 0x09c4 /* the VTS need to be half in normal mod= e */ @@ -1556,12 +1560,33 @@ static int ov08x40_open(struct v4l2_subdev *sd, str= uct v4l2_subdev_fh *fh) try_fmt->code =3D MEDIA_BUS_FMT_SGRBG10_1X10; try_fmt->field =3D V4L2_FIELD_NONE; =20 - /* No crop or compose */ mutex_unlock(&ov08x->mutex); =20 return 0; } =20 +static int ov08x40_get_selection(struct v4l2_subdev *sd, + struct v4l2_subdev_state *state, + struct v4l2_subdev_selection *sel) +{ + if (sel->which !=3D V4L2_SUBDEV_FORMAT_ACTIVE) + return -EINVAL; + + switch (sel->target) { + case V4L2_SEL_TGT_NATIVE_SIZE: + case V4L2_SEL_TGT_CROP_BOUNDS: + case V4L2_SEL_TGT_CROP_DEFAULT: + case V4L2_SEL_TGT_CROP: + sel->r.top =3D 0; + sel->r.left =3D 0; + sel->r.width =3D OV08X40_NATIVE_WIDTH; + sel->r.height =3D OV08X40_NATIVE_HEIGHT; + return 0; + } + + return -EINVAL; +} + static int ov08x40_update_digital_gain(struct ov08x40 *ov08x, u32 d_gain) { int ret; @@ -2059,6 +2084,7 @@ static const struct v4l2_subdev_pad_ops ov08x40_pad_o= ps =3D { .enum_mbus_code =3D ov08x40_enum_mbus_code, .get_fmt =3D ov08x40_get_pad_format, .set_fmt =3D ov08x40_set_pad_format, + .get_selection =3D ov08x40_get_selection, .enum_frame_size =3D ov08x40_enum_frame_size, }; =20 --=20 2.54.0