From mboxrd@z Thu Jan 1 00:00:00 1970 From: Archit Taneja Subject: Re: [PATCH v3 0/7] drm/i2c: adv7511: ADV7533 support Date: Tue, 3 May 2016 12:23:57 +0530 Message-ID: <57284B05.1060400@codeaurora.org> References: <1441625800-1858-1-git-send-email-architt@codeaurora.org> <1457521038-5906-1-git-send-email-architt@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:32947 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751016AbcECGyK (ORCPT ); Tue, 3 May 2016 02:54:10 -0400 In-Reply-To: Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Xinliang Liu Cc: dri-devel , David Airlie , Daniel Vetter , treding@nvidia.com, linux-arm-msm@vger.kernel.org, Srinivas Kandagatla , Andy Green , laurent.pinchart@ideasonboard.com, lars@metafoo.de, Amit Kucheria , Rob Clark On 05/03/2016 07:22 AM, Xinliang Liu wrote: > On 9 March 2016 at 18:57, 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. >> >> [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(-) >> >> -- >> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, >> hosted by The Linux Foundation >> > > This patch set is Tested-by: Xinliang Liu Thanks for testing! Archit > > Thanks > -xinliang > -- > To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation