From mboxrd@z Thu Jan 1 00:00:00 1970 From: Archit Taneja Subject: Re: [PATCH 0/4] drm/bridge/adv7511: add CEC support Date: Wed, 9 Aug 2017 18:07:37 +0530 Message-ID: References: <20170730130743.19681-1-hverkuil@xs4all.nl> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20170730130743.19681-1-hverkuil@xs4all.nl> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Hans Verkuil , linux-media@vger.kernel.org Cc: linux-renesas-soc@vger.kernel.org, linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org CgpPbiAwNy8zMC8yMDE3IDA2OjM3IFBNLCBIYW5zIFZlcmt1aWwgd3JvdGU6Cj4gRnJvbTogSGFu cyBWZXJrdWlsIDxoYW5zLnZlcmt1aWxAY2lzY28uY29tPgo+IAo+IFRoaXMgcGF0Y2ggc2VyaWVz IGFkZHMgQ0VDIHN1cHBvcnQgdG8gdGhlIGRybSBhZHY3NTExL2Fkdjc1MzMgZHJpdmVycy4KPiAK PiBJIGhhdmUgdGVzdGVkIHRoaXMgd2l0aCB0aGUgUXVhbGNvbW0gRHJhZ29uYm9hcmQgQzQxMCAo YWR2NzUzMyBiYXNlZCkKPiBhbmQgdGhlIFJlbmVzYXMgUi1DYXIgS29lbHNjaCBib2FyZCAoYWR2 NzUxMSBiYXNlZCkuCj4gCj4gTm90ZTogdGhlIERyYWdvbmJvYXJkIG5lZWRzIHRoaXMgcGF0Y2g6 Cj4gCj4gaHR0cHM6Ly9wYXRjaHdvcmsua2VybmVsLm9yZy9wYXRjaC85ODI0NzczLwo+IAo+IEFy Y2hpdCwgY2FuIHlvdSBjb25maXJtIHRoYXQgdGhpcyBwYXRjaCB3aWxsIGdvIHRvIGtlcm5lbCA0 LjE0PwoKWWVzLCBpdCdzIGJlZW4gcXVldWVkIHRvIGNsay1uZXh0LgoKVGhhbmtzLApBcmNoaXQK Cj4gCj4gQW5kIHRoZSBLb2Vsc2NoIGJvYXJkIG5lZWRzIHRoaXMgNC4xMyBmaXg6Cj4gCj4gaHR0 cHM6Ly9wYXRjaHdvcmsua2VybmVsLm9yZy9wYXRjaC85ODM2ODY1Lwo+IAo+IEkgb25seSBoYXZl IHRoZSBLb2Vsc2NoIGJvYXJkIHRvIHRlc3Qgd2l0aCwgYnV0IGl0IGxvb2tzIGxpa2Ugb3RoZXIK PiBSLUNhciBib2FyZHMgdXNlIHRoZSBzYW1lIGFkdjc1MTEuIEl0IHdvdWxkIGJlIG5pY2UgaWYg c29tZW9uZSBjYW4KPiBhZGQgQ0VDIHN1cHBvcnQgdG8gdGhlIG90aGVyIFItQ2FyIGJvYXJkcyBh cyB3ZWxsLiBUaGUgbWFpbiB0aGluZwo+IHRvIGNoZWNrIGlzIGlmIHRoZXkgYWxsIHVzZSB0aGUg c2FtZSAxMiBNSHogZml4ZWQgQ0VDIGNsb2NrIHNvdXJjZS4KPiAKPiBBbnlvbmUgd2hvIHdhbnRz IHRvIHRlc3QgdGhpcyB3aWxsIG5lZWQgdGhlIENFQyB1dGlsaXRpZXMgdGhhdAo+IGFyZSBwYXJ0 IG9mIHRoZSB2NGwtdXRpbHMgZ2l0IHJlcG9zaXRvcnk6Cj4gCj4gZ2l0IGNsb25lIGdpdDovL2xp bnV4dHYub3JnL3Y0bC11dGlscy5naXQKPiBjZCB2NGwtdXRpbHMKPiAuL2Jvb3RzdHJhcC5zaAo+ IC4vY29uZmlndXJlCj4gbWFrZQo+IHN1ZG8gbWFrZSBpbnN0YWxsCj4gCj4gTm93IGNvbmZpZ3Vy ZSB0aGUgQ0VDIGFkYXB0ZXIgYXMgYSBQbGF5YmFjayBkZXZpY2U6Cj4gCj4gY2VjLWN0bCAtLXBs YXliYWNrCj4gCj4gRGlzY292ZXIgb3RoZXIgQ0VDIGRldmljZXM6Cj4gCj4gY2VjLWN0bCAtUwo+ IAo+IFJlZ2FyZHMsCj4gCj4gCUhhbnMKPiAKPiBIYW5zIFZlcmt1aWwgKDQpOgo+ICAgIGR0LWJp bmRpbmdzOiBhZGksYWR2NzUxMS50eHQ6IGRvY3VtZW50IGNlYyBjbG9jawo+ICAgIGFybTogZHRz OiBxY29tOiBhZGQgY2VjIGNsb2NrIGZvciBhcHE4MDE2IGJvYXJkCj4gICAgYXJtOiBkdHM6IHJl bmVzYXM6IGFkZCBjZWMgY2xvY2sgZm9yIEtvZWxzY2ggYm9hcmQKPiAgICBkcm06IGFkdjc1MTEv MzM6IGFkZCBIRE1JIENFQyBzdXBwb3J0Cj4gCj4gICAuLi4vYmluZGluZ3MvZGlzcGxheS9icmlk Z2UvYWRpLGFkdjc1MTEudHh0ICAgICAgICB8ICAgNCArCj4gICBhcmNoL2FybS9ib290L2R0cy9y OGE3NzkxLWtvZWxzY2guZHRzICAgICAgICAgICAgICB8ICAgOCArCj4gICBhcmNoL2FybTY0L2Jv b3QvZHRzL3Fjb20vYXBxODAxNi1zYmMuZHRzaSAgICAgICAgICB8ICAgMiArCj4gICBkcml2ZXJz L2dwdS9kcm0vYnJpZGdlL2Fkdjc1MTEvS2NvbmZpZyAgICAgICAgICAgICB8ICAgOCArCj4gICBk cml2ZXJzL2dwdS9kcm0vYnJpZGdlL2Fkdjc1MTEvTWFrZWZpbGUgICAgICAgICAgICB8ICAgMSAr Cj4gICBkcml2ZXJzL2dwdS9kcm0vYnJpZGdlL2Fkdjc1MTEvYWR2NzUxMS5oICAgICAgICAgICB8 ICA0NSArKy0KPiAgIGRyaXZlcnMvZ3B1L2RybS9icmlkZ2UvYWR2NzUxMS9hZHY3NTExX2NlYy5j ICAgICAgIHwgMzE0ICsrKysrKysrKysrKysrKysrKysrKwo+ICAgZHJpdmVycy9ncHUvZHJtL2Jy aWRnZS9hZHY3NTExL2Fkdjc1MTFfZHJ2LmMgICAgICAgfCAxNTIgKysrKysrKysrLQo+ICAgZHJp dmVycy9ncHUvZHJtL2JyaWRnZS9hZHY3NTExL2Fkdjc1MzMuYyAgICAgICAgICAgfCAgMzAgKy0K PiAgIDkgZmlsZXMgY2hhbmdlZCwgNTE0IGluc2VydGlvbnMoKyksIDUwIGRlbGV0aW9ucygtKQo+ ICAgY3JlYXRlIG1vZGUgMTAwNjQ0IGRyaXZlcnMvZ3B1L2RybS9icmlkZ2UvYWR2NzUxMS9hZHY3 NTExX2NlYy5jCj4gCgotLSAKUXVhbGNvbW0gSW5ub3ZhdGlvbiBDZW50ZXIsIEluYy4gaXMgYSBt ZW1iZXIgb2YgQ29kZSBBdXJvcmEgRm9ydW0sCmEgTGludXggRm91bmRhdGlvbiBDb2xsYWJvcmF0 aXZlIFByb2plY3QKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3Jn Cmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVs Cg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.codeaurora.org ([198.145.29.96]:39096 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751864AbdHIMhm (ORCPT ); Wed, 9 Aug 2017 08:37:42 -0400 From: Archit Taneja Subject: Re: [PATCH 0/4] drm/bridge/adv7511: add CEC support To: Hans Verkuil , linux-media@vger.kernel.org Cc: dri-devel@lists.freedesktop.org, linux-arm-msm@vger.kernel.org, linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org, Lars-Peter Clausen References: <20170730130743.19681-1-hverkuil@xs4all.nl> Message-ID: Date: Wed, 9 Aug 2017 18:07:37 +0530 MIME-Version: 1.0 In-Reply-To: <20170730130743.19681-1-hverkuil@xs4all.nl> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: On 07/30/2017 06:37 PM, Hans Verkuil wrote: > From: Hans Verkuil > > This patch series adds CEC support to the drm adv7511/adv7533 drivers. > > I have tested this with the Qualcomm Dragonboard C410 (adv7533 based) > and the Renesas R-Car Koelsch board (adv7511 based). > > Note: the Dragonboard needs this patch: > > https://patchwork.kernel.org/patch/9824773/ > > Archit, can you confirm that this patch will go to kernel 4.14? Yes, it's been queued to clk-next. Thanks, Archit > > And the Koelsch board needs this 4.13 fix: > > https://patchwork.kernel.org/patch/9836865/ > > I only have the Koelsch board to test with, but it looks like other > R-Car boards use the same adv7511. It would be nice if someone can > add CEC support to the other R-Car boards as well. The main thing > to check is if they all use the same 12 MHz fixed CEC clock source. > > Anyone who wants to test this will need the CEC utilities that > are part of the v4l-utils git repository: > > git clone git://linuxtv.org/v4l-utils.git > cd v4l-utils > ./bootstrap.sh > ./configure > make > sudo make install > > Now configure the CEC adapter as a Playback device: > > cec-ctl --playback > > Discover other CEC devices: > > cec-ctl -S > > Regards, > > Hans > > Hans Verkuil (4): > dt-bindings: adi,adv7511.txt: document cec clock > arm: dts: qcom: add cec clock for apq8016 board > arm: dts: renesas: add cec clock for Koelsch board > drm: adv7511/33: add HDMI CEC support > > .../bindings/display/bridge/adi,adv7511.txt | 4 + > arch/arm/boot/dts/r8a7791-koelsch.dts | 8 + > arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi | 2 + > drivers/gpu/drm/bridge/adv7511/Kconfig | 8 + > drivers/gpu/drm/bridge/adv7511/Makefile | 1 + > drivers/gpu/drm/bridge/adv7511/adv7511.h | 45 ++- > drivers/gpu/drm/bridge/adv7511/adv7511_cec.c | 314 +++++++++++++++++++++ > drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 152 +++++++++- > drivers/gpu/drm/bridge/adv7511/adv7533.c | 30 +- > 9 files changed, 514 insertions(+), 50 deletions(-) > create mode 100644 drivers/gpu/drm/bridge/adv7511/adv7511_cec.c > -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project