On 11/11/16 17:27, Russell King - ARM Linux wrote: > On Fri, Nov 11, 2016 at 05:10:09PM +0200, Jyri Sarha wrote: >> On 11/08/16 14:24, Russell King - ARM Linux wrote: >>> As no one responded to the previous round, I'm not spending soo much >>> time writing up a description of these changes again. It's also been >>> quite a long time, so I've forgotten all the details of the changes, >>> so I'll do my best. >>> >>> Changes from the previous series include: >>> - reorder the initial three patches >>> - change the (now third patch)... I think to increase the size of the >>> locked region. >>> - fix edid parsing for infoframe generation - as was pointed out for >>> dw-hdmi, parsing the EDID in get_modes() is incorrect, as that method >>> will not be called when an override-edid is in effect. We need to >>> parse the override-edid. Moreover, infoframe generation should not >>> be keyed to whether the monitor is HDMI or not, CEA-861B allows non- >>> HDMI to send infoframes. >>> - only send audio if audio and infoframes are supported. >>> >>> Otherwise, these are very much like the previous posting of the series, >>> except rebased upon the mali/hdlcd/tda998x change to remove the >>> drm_connector_register() call. >>> >>> https://www.spinics.net/lists/dri-devel/msg121495.html >>> >>> It'd be nice to have other tda998x users ack and test these patches, >>> I've tried to test on Juno, but the Juno situation seems to be a huge >>> fail. (HBI0282B completely fails with latest firmware - (a) FPGA image >>> incompatibilities io_b118 causes all FPGA AMBA devices to vanish, (b) >>> seems no way to get SCPI support on it - adding the BL0 executable >>> start address in the SCC registers seems to be incompatible with the >>> devchip, causing the PLLs to fail. In discussion with Sudeep over >>> these issues, but no idea where things are with it at the moment, other >>> than Sudeep needs to investigate. All Linaro firmware releases are >>> broken on HBI0282B.) >>> >>> drivers/gpu/drm/i2c/tda998x_drv.c | 826 ++++++++++++++++++++------------------ >>> 1 file changed, 429 insertions(+), 397 deletions(-) >>> >> >> Reviewed-by: Jyri Sarha <jsarha@xxxxxx> >> >> For the whole series. I am also happy to test these patches if I can >> fetch them from some git repo. > > git://git.armlinux.org.uk/~rmk/linux-arm.git drm-tda998x-devel > > The commit IDs are unstable, because I'll have to recommit them to add > your r-by and any other tags you later give me. :) > I tested the branch with my Beaglebone-black and couple of TVs. I played audio on different sample-rates while plugging and unplugging the cable to the TVs and changing video modes. Everything worked as it should. I let you decide whether or not this adhoc testing qualifies: Tested-by: Jyri Sarha <jsarha@xxxxxx> Best regards, Jyri _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
