Hi Simon,
On Thu, 13 Aug 2015, William Towle wrote:
> (Obsoletes corresponding parts of "HDMI and Composite capture on
> Lager...", published previously)
> To follow:
> [PATCH 1/3] ARM: shmobile: lager dts: Add entries for VIN HDMI input
> [PATCH 2/3] media: adv7604: automatic "default-input" selection
> [PATCH 3/3] ARM: shmobile: lager dts: specify default-input for
I am wondering about the status of patch 2 of the series,
is it queued-up anywhere?
All of these are effectively new, although the first and third
debuted in another thread. The patchwork link for the latter (at
https://patchwork.linuxtv.org/patch/30707/) contains the discussion
that led to the above being separated out.
I am also wondering about the relationship between patch 2 and 3.
Does 3 work without 2? Does 2 make 3 unnecessary?
The device tree change in patch 3 is from Ian Molton's original
submissions, and works regardless of whether patch 2 is alongside it
or not.
As far as the automatic port selection goes, patch 2 is related to
the argument made in commit 7111cdd ("[media] media: adv7604: reduce
support to first (digital) input") that since cable detect doesn't
work for port B, the .max_port property for boards using an ADV7612
should be ADV76XX_PAD_HDMI_PORT_A and we can use this to configure
port A as the default where there is not also an entry to specify it
in the device tree.
If support for port B were available [in future], no action would
be taken where an ADV7612 is present and one would need to arrange
for applications in userland to actively make a choice where the
device tree does not have an appropriate value. In our particular case
we don't differentiate between hardware with ADV7611/7612 in the test
suite, and this necessitates having port selection available in the
device tree.
Laurent may wish to comment further; for earlier discussion, see
https://patchwork.linuxtv.org/patch/30707/
Cheers,
Wills.
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
