Hi Archit, On Thursday 14 Apr 2016 20:26:11 Archit Taneja wrote: > 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. Basic testing didn't show any regression. For the whole series, Tested-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > > [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.htm > > l > > [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(-) -- Regards, Laurent Pinchart -- 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
