mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Pierre Pinon <pierre@pinon1.fr>
To: okerixx@gmail.com
Cc: hansg@kernel.org, jason.z.chen@intel.com, jimmy.su@intel.com,
	linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
	mchehab@kernel.org, pierre@pinon1.fr,
	sakari.ailus@linux.intel.com
Subject: Re: [PATCH v2] media: i2c: ov08x40: Implement get_selection
Date: Mon,  5 Oct 2026 10:15:29 +0200	[thread overview]
Message-ID: <20261005081529.45697-1-pierre@pinon1.fr> (raw)
In-Reply-To: <179096167726.2508.10599420708324219706@gmail.com>

Hi Oleg,

On Fri, Oct 02, 2026 at 07:24:20PM +0200, Oleg Keri wrote:
> v2 works here on the Lenovo Yoga Slim 7x Gen 11 (2-lane, libcamera 0.7.2
> simple pipeline).
>
> Tested-by: Oleg Keri <okerixx@gmail.com> # Lenovo Yoga Slim 7x Gen 11

Thanks for testing!

> Yes, please do. It makes no sense to default to a mode a 2-lane board
> can't use.

Sent here:
https://lore.kernel.org/linux-media/20261005065535.16137-1-pierre@pinon1.fr/

For the record, on the Dell (IPU7 behind an Intel CVS, 2 lanes),
libcamera now picks the binned 1928x1088 mode for 1080p, and that mode
still does not stream: the IPU7 firmware rejects every frame with
INSYS_MSG_ERR_CAPTURE_HW_ERR_BAD_FRAME_DIM.

This is the issue discussed under v1 2/2, "media: i2c: ov08x40: Do not
expose the broken 1928x1088 binned mode":
https://lore.kernel.org/linux-media/20260904134604.595685-1-pierre@pinon1.fr/

In short:
- Jimmy relayed that the CVS firmware decodes and re-packs the sensor's
  MIPI stream, and only supports 3856x2176 linear and 1928x1088 sHDR,
  while the driver programs 1928x1088 as linear.
- Sakari suggested having the CVS driver reject the other sizes.

With your Yoga feeding the sensor into CAMSS, I assume there is no such
vision chip in between, which would explain why the mode works there.
Is that right?

I'll follow up in that thread.

Pierre

  reply	other threads:[~2026-10-05  8:15 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-04 13:45 [PATCH 1/2] media: i2c: ov08x40: Implement the selection API Pierre Pinon
2026-09-14 14:00 ` Oleg Keri
2026-10-02  9:49   ` [PATCH v2] media: i2c: ov08x40: Implement get_selection Pierre Pinon
2026-10-02 17:24     ` Oleg Keri
2026-10-05  8:15       ` Pierre Pinon [this message]
2026-10-05  8:53         ` Oleg Keri

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20261005081529.45697-1-pierre@pinon1.fr \
    --to=pierre@pinon1.fr \
    --cc=hansg@kernel.org \
    --cc=jason.z.chen@intel.com \
    --cc=jimmy.su@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@kernel.org \
    --cc=okerixx@gmail.com \
    --cc=sakari.ailus@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®