All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Niklas Söderlund" <niklas.soderlund+renesas@ragnatech.se>
To: Kieran Bingham <kieran.bingham@ideasonboard.com>,
	Hans Verkuil <hverkuil-cisco@xs4all.nl>,
	linux-media@vger.kernel.org
Cc: linux-renesas-soc@vger.kernel.org,
	"Niklas Söderlund" <niklas.soderlund+renesas@ragnatech.se>
Subject: [PATCH 0/3] media: i2c: adv748x: Enable pattern generator without external HDMI source
Date: Wed, 22 Feb 2023 23:12:46 +0100	[thread overview]
Message-ID: <20230222221249.1324102-1-niklas.soderlund+renesas@ragnatech.se> (raw)

Hello,

This small series enables the pattern generator for the HDMI block to 
function correctly without an external HDMI source connected.

Without this series there are a few small errors in the timings code 
which prevents the user from setting the timings, instead they need to 
be queried from an external device.

Patch 1/3 fix the lookup and matching of requested timings to the 
timings the device supports, this lookup was incorrectly implemented.  

Patch 2/3 fixes a corner case where the initial timing selected by the 
driver to have something to report to user-space can't be used correctly 
if it is the first set of timings the user wish to use.

Finally patch 3/3 removes the need for an external HDMI source to 
operate the pattern generator. Instead of failing when querying DV 
timings and their is no HDMI signal but the pattern generator is 
enabled, report the timings programmed by the user. These user 
programmed timings are then picked up by the pattern generator to 
produce a test image using those timings.

Tested on a few different R-Car Gen3 boards (H3, M3-N) with no 
regressions.

I can finally run all my automated tests without turning on my Xbox! :-)

Niklas Söderlund (3):
  media: i2c: adv748x: Fix lookup of DV timings
  media: i2c: adv748x: Write initial DV timings to device
  media: i2c: adv748x: Report correct DV timings for pattern generator

 drivers/media/i2c/adv748x/adv748x-hdmi.c | 21 +++++++++++++++------
 1 file changed, 15 insertions(+), 6 deletions(-)

-- 
2.39.2


             reply	other threads:[~2023-02-22 22:14 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-22 22:12 Niklas Söderlund [this message]
2023-02-22 22:12 ` [PATCH 1/3] media: i2c: adv748x: Fix lookup of DV timings Niklas Söderlund
2023-02-23 10:05   ` Geert Uytterhoeven
2023-02-22 22:12 ` [PATCH 2/3] media: i2c: adv748x: Write initial DV timings to device Niklas Söderlund
2023-02-22 22:12 ` [PATCH 3/3] media: i2c: adv748x: Report correct DV timings for pattern generator Niklas Söderlund

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=20230222221249.1324102-1-niklas.soderlund+renesas@ragnatech.se \
    --to=niklas.soderlund+renesas@ragnatech.se \
    --cc=hverkuil-cisco@xs4all.nl \
    --cc=kieran.bingham@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.