Hi Laurent, On 3/9/2016 4:27 PM, Archit Taneja wrote:
ADV7533 is a DSI to HDMI encoder chip. It's like ADV7511, but with an additional DSI RX block that takes in DSI video mode output. Trying to get this driver merged has had some challenges: - ADV7533 has an I2C control bus, but acts as a DSI peripheral too. After discussions, it was concluded that we'd want to provide an API to create MIPI DSI devices, rather than expose two different interfaces on DT. The first version [1] tried the former approach the second version [2] showed how the driver would look like if exposed 2 DT nodes. This lateset patchset relies on the MIPI DSI device creation API provided by [3], this has been accepted and should be merged for 4.6. - The driver was designed as an I2C slave encoder. When ADV7533 patches were posted [1], it was modelled as a bridge, but ADV7511 and others were still left as I2C slave encoders. This wasn't accepted. After discussions, it was decided that ADV7511 too would be converted into a bridge driver, and all the users of ADV7511 should assume it is a bridge. This bridge conversion was done in [4]. There is still some debate over whether the bridge driver be involved in the connector creation, or the KMS driver that has the whole view of the display pipeline. This discussion shouldn't affect this patch set, though. This patch set enables ADV7533 support with the above two issues now resolved. It also incorporates ADV7533 specific features and fixes that we've discovered since the first version of this patch was posted. Tested on ADV7533 chips on DB410c. It should work on the Hikey board too. I'd appreaciate if someone could test it on a ADV7511 platform since I don't have one.
Could you try this patch set on the rcar-du platform? It would help to verify if this doesn't break ADV7511. Thanks, Archit
[4] https://lists.freedesktop.org/archives/dri-devel/2016-January/098287.html [3] https://lkml.org/lkml/2016/2/12/67 [2] https://lists.freedesktop.org/archives/dri-devel/2015-September/089884.html [1]: https://lists.freedesktop.org/archives/dri-devel/2015-July/087088.html Archit Taneja (7): drm/i2c: adv7511: Convert to drm_bridge drm/i2c: adv7511: Fix mutex deadlock when interrupts are disabled drm/i2c: adv7511: Initial support for ADV7533 drm/i2c: adv7511: Create a MIPI DSI device drm/i2c: adv7511: Use internal timing generator drm/i2c: adv7511: Change number of DSI lanes dynamically dt-bindings: drm/bridge: Update bindings for ADV7533 .../bindings/display/bridge/adi,adv7511.txt | 25 +- drivers/gpu/drm/i2c/adv7511.c | 539 +++++++++++++++++---- 2 files changed, 476 insertions(+), 88 deletions(-)
-- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
