On Thu, Jun 18, 2015 at 8:55 AM, Russell King - ARM Linux
Yakir: sounds like you now have some feedback on your patch now.Perhaps you can try <https://patchwork.kernel.org/patch/5906771/>Something like that needs to be done, but let's get rid of the mdvi
thing in struct hdmi_vmode - it doesn't belong there, it isn't part
of the currently set video mode, but becomes a property of the
I'd also prefer it to be called "is_dvi_sink", especially as its
function is changing from "is it a CEA mode" to "is the attached
device a DVI sink".
Even better would be to call it "is_hdmi_sink" to maintain positive
logic with single-negation where required, rather than double-
negation in places.
Perhaps you can spin it with Russell's feedback?
When you send it next, please make sure you include Russell in the
"To" line. Based at looking at who committed things to dw_hdmi in the
past, I've been sending my patches "To":
...so perhaps that would be good for you to do, too?
I will rebase on russell series and re-send my patch today.Ah, OK. Have you seen any that specifically confuse the DVI vs. HDMI bits?I am nowhere near an HDMI expert. If you have a better suggestionNeither am I, but I have had the ability to do some testing with AV
then I'm more than happy for you to post it and drop my patch. In my
non-expert opinion, it would seem awfully strange for an AV receiver
to modify the EDID though unless it was actively interpreting the
signal and generating a whole new signal on the other end. In any
case, perhaps you can find such a device and that will give insight to
how we should deal with it. Until such a device is found, it seems
fruitless to speculate.
receivers in the path of a HDMI device, and I've seen how they behave.
(I made copious notes on this, which I intend to publish when I have
a round tuit.) Unfortunately, I have no DVI devices to test with,
and DVI devices are a dying breed - most monitors today come with
HDMI sockets instead.
I still have copious DVI devices around, personally. I could have
sworn that supporting "older hardware" was actually pretty important
in Linux. ...and I can still buy plenty of DVI devices out there.
That's a shame... I'm not inclined to Ack it as-is - and I'd also likePerhaps when Yakir spins his series he can include a patch like mine
to see Yakir's patch reworked as I mentioned above.
in it. It doesn't make sense for me to re-spin it until his is