[{"id":1772632,"web_url":"http://patchwork.ozlabs.org/comment/1772632/","msgid":"<5172c185-c6b5-49c3-cf0a-c3e073459346@synopsys.com>","list_archive_url":null,"date":"2017-09-21T09:39:42","subject":"Re: [RESEND RFC PATCH 0/7] sun8i H3 HDMI glue driver for DW HDMI","submitter":{"id":68387,"url":"http://patchwork.ozlabs.org/api/people/68387/","name":"Jose Abreu","email":"Jose.Abreu@synopsys.com"},"content":"Hi Jernej,\n\nOn 20-09-2017 21:01, Jernej Skrabec wrote:\n> [added media mailing list due to CEC question]\n>\n> This patch series adds a HDMI glue driver for Allwinner H3 SoC. For now, only\n> video and CEC functionality is supported. Audio needs more tweaks.\n>\n> Series is based on the H3 DE2 patch series available on mailing list:\n> https://urldefense.proofpoint.com/v2/url?u=http-3A__lists.infradead.org_pipermail_linux-2Darm-2Dkernel_2017-2DAugust_522697.html&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=WHDsc6kcWAl4i96Vm5hJ_19IJiuxx_p_Rzo2g-uHDKw&m=coyfcQKSr2asrHcaCeWFmAP_9nkFkRK8s7Uw5bmVei4&s=JCFaMXK1MmZ3jE745_YcqZhZkaqtc6UapGfSSapcz_s&e= \n> (ignore patches marked with [NOT FOR REVIEW NOW] tag)\n>\n> Patch 1 adds support for polling plug detection since custom PHY used here\n> doesn't support HPD interrupt.\n>\n> Patch 2 enables overflow workaround for v1.32a. This HDMI controller exhibits\n> same issues as HDMI controller used in iMX6 SoCs.\n>\n> Patch 3 adds CLK_SET_RATE_PARENT to hdmi clock.\n>\n> Patch 4 adds dt bindings documentation.\n>\n> Patch 5 adds actual H3 HDMI glue driver.\n>\n> Patch 6 and 7 add HDMI node to DT and enable it where needed.\n>\n> Allwinner used DW HDMI controller in a non standard way:\n> - register offsets obfuscation layer, which can fortunately be turned off\n> - register read lock, which has to be disabled by magic number\n> - custom PHY, which have to be initialized before DW HDMI controller\n> - non standard clocks\n> - no HPD interrupt\n>\n> Because of that, I have two questions:\n> - Since HPD have to be polled, is it enough just to enable poll mode? I'm\n>   mainly concerned about invalidating CEC address here.\n\nYou mean you get no interrupt when HPD status changes? Hans can\nanswer this better but then you will need to invalidate the cec\nphysical address yourself because right now its invalidated in\nthe dw-hdmi irq handler (see dw_hdmi_irq()).\n\n> - PHY has to be initialized before DW HDMI controller to disable offset\n>   obfuscation and read lock among other things. This means that all clocks have\n>   to be enabled in glue driver. This poses a problem, since when using\n>   component model, dw-hdmi bridge uses drvdata for it's own private data and\n>   prevents glue layer to pass a pointer to unbind function, where clocks should\n>   be disabled. I noticed same issue in meson DW HDMI glue driver, where clocks\n>   are also not disabled when unbind callback is called. I noticed that when H3\n>   SoC is shutdown, HDMI output is still enabled and lastest image is shown on\n>   monitor until it is unplugged from power supply. Is there any simple solution\n>   to this?\n\nI don't know if you can use an empty platform device created with\nplatform_device_alloc(). Perhaps it would be better fix this in\nthe dw-hdmi driver. I see two solutions:\n\n    - Either you return the dw-hdmi private structure in the bind\ncallback, store it and pass it in the unbind\n    - Or, you pass your own private data to the dw-hdmi bind, the\ndw-hdmi stores it and you just create a public function in the\ndw-hdmi driver called like dw_hdmi_get_auxdata(struct device\n*dev) which returns your private data.\n\nI think first option is nice, maybe anyone else can suggest\nsomething better?\n\nBest regards,\nJose Miguel Abreu\n\n>\n> Chen-Yu,\n> TL Lim was unable to obtain any answer from Allwinner about HDMI clocks. I think\n> it is safe to assume that divider in HDMI clock doesn't have any effect.\n>\n> Branch based on linux-next from 1. September with integrated patches is\n> available here:\n> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_jernejsk_linux-2D1_tree_h3-5Fhdmi-5Frfc&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=WHDsc6kcWAl4i96Vm5hJ_19IJiuxx_p_Rzo2g-uHDKw&m=coyfcQKSr2asrHcaCeWFmAP_9nkFkRK8s7Uw5bmVei4&s=lDAnd3egsc2sxqVM-Ya_Me9ozWXKWvxxvsdV3Jn3vpA&e= \n>\n> Some additonal info about H3 HDMI:\n> https://urldefense.proofpoint.com/v2/url?u=https-3A__linux-2Dsunxi.org_DWC-5FHDMI-5FController&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=WHDsc6kcWAl4i96Vm5hJ_19IJiuxx_p_Rzo2g-uHDKw&m=coyfcQKSr2asrHcaCeWFmAP_9nkFkRK8s7Uw5bmVei4&s=d9iEgk23RCLJL4oXJ4kkt6NyYK90_vFy0mCD3WauJDk&e= \n>\n> Thanks to Jens Kuske, who figured out that it is actually DW HDMI controller\n> and mapped scrambled register offsets to original ones.\n>\n> Icenowy Zheng (1):\n>   ARM: sun8i: h3: Add DesignWare HDMI controller node\n>\n> Jernej Skrabec (6):\n>   drm: bridge: Enable polling hpd event in dw_hdmi\n>   drm: bridge: Enable workaround in dw_hdmi for v1.32a\n>   clk: sunxi: Add CLK_SET_RATE_PARENT flag for H3 HDMI clock\n>   dt-bindings: Document Allwinner DWC HDMI TX node\n>   drm: sun4i: Add a glue for the DesignWare HDMI controller in H3\n>   ARM: sun8i: h3: Enable HDMI output on H3 boards\n>\n>  .../bindings/display/sunxi/sun4i-drm.txt           | 158 ++++++-\n>  arch/arm/boot/dts/sun8i-h3-bananapi-m2-plus.dts    |  33 ++\n>  arch/arm/boot/dts/sun8i-h3-beelink-x2.dts          |  33 ++\n>  arch/arm/boot/dts/sun8i-h3-nanopi-m1.dts           |  33 ++\n>  arch/arm/boot/dts/sun8i-h3-orangepi-2.dts          |  33 ++\n>  arch/arm/boot/dts/sun8i-h3-orangepi-lite.dts       |  33 ++\n>  arch/arm/boot/dts/sun8i-h3-orangepi-one.dts        |  33 ++\n>  arch/arm/boot/dts/sun8i-h3-orangepi-pc.dts         |  33 ++\n>  arch/arm/boot/dts/sun8i-h3.dtsi                    |   5 +\n>  arch/arm/boot/dts/sunxi-h3-h5.dtsi                 |  36 ++\n>  drivers/clk/sunxi-ng/ccu-sun8i-h3.c                |   2 +-\n>  drivers/gpu/drm/bridge/synopsys/dw-hdmi.c          |  14 +-\n>  drivers/gpu/drm/sun4i/Kconfig                      |   9 +\n>  drivers/gpu/drm/sun4i/Makefile                     |   1 +\n>  drivers/gpu/drm/sun4i/sun8i_dw_hdmi.c              | 500 +++++++++++++++++++++\n>  15 files changed, 950 insertions(+), 6 deletions(-)\n>  create mode 100644 drivers/gpu/drm/sun4i/sun8i_dw_hdmi.c\n>","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"SR2nu2cY\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xyX6S1qCHz9t49\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tThu, 21 Sep 2017 19:55:28 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1duyCc-0006MG-OW; Thu, 21 Sep 2017 09:55:26 +0000","from smtprelay.synopsys.com ([198.182.47.9])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1duxyB-00054i-Pz for linux-arm-kernel@lists.infradead.org;\n\tThu, 21 Sep 2017 09:40:38 +0000","from mailhost.synopsys.com (mailhost3.synopsys.com [10.12.238.238])\n\tby smtprelay.synopsys.com (Postfix) with ESMTP id E65E324E12C0;\n\tThu, 21 Sep 2017 02:40:06 -0700 (PDT)","from mailhost.synopsys.com (localhost [127.0.0.1])\n\tby mailhost.synopsys.com (Postfix) with ESMTP id 72FF6E00;\n\tThu, 21 Sep 2017 02:40:06 -0700 (PDT)","from US01WEHTC2.internal.synopsys.com\n\t(us01wehtc2-vip.internal.synopsys.com [10.12.239.238])\n\tby mailhost.synopsys.com (Postfix) with ESMTP id 0A14FDBE;\n\tThu, 21 Sep 2017 02:40:05 -0700 (PDT)","from DE02WEHTCB.internal.synopsys.com (10.225.19.94) by\n\tUS01WEHTC2.internal.synopsys.com (10.12.239.237) with Microsoft SMTP\n\tServer (TLS) id 14.3.266.1; Thu, 21 Sep 2017 02:40:03 -0700","from DE02WEHTCA.internal.synopsys.com (10.225.19.92) by\n\tDE02WEHTCB.internal.synopsys.com (10.225.19.94) with Microsoft SMTP\n\tServer (TLS) id 14.3.266.1; Thu, 21 Sep 2017 11:40:01 +0200","from [10.0.2.15] (10.107.19.118) by\n\tDE02WEHTCA.internal.synopsys.com\n\t(10.225.19.80) with Microsoft SMTP Server (TLS) id 14.3.266.1;\n\tThu, 21 Sep 2017 11:40:00 +0200"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=oqTTKaBZdhfx8AllZv9VXULVohwOAxDdgTDjYqgoqGg=;\n\tb=SR2nu2cYTUV9+X\n\t4u7DFqt5R438zdbPx/fbYHQAgLfu++qmMP47Zj0Is/GV8YIcdJVYhCHoVW/8xG6xArma+4s9T33E0\n\tbZ1rq9TQgGQxZs/GOwpofgJxS7IOZ9dxuMfVQB6ItRDUqNhRwajHMW/AmqoztTlklGNidC+GOqUp2\n\tNR34J+oBhZnytWCrf1E+rqn9Gc64DacgfspjdLtSv2YzQOTY6ZaBPxXczl98PNcYMFQCrZ/TFgqx3\n\tFGDAK4MwKOVIfwt6c5vdv1qS/eAjO5yIOOqqh3orUQJj47ST/jL22/w+jquZuYTuw0SbV3O6etOme\n\tOQobXz91ahups0GxILng==;","Subject":"Re: [RESEND RFC PATCH 0/7] sun8i H3 HDMI glue driver for DW HDMI","To":"Jernej Skrabec <jernej.skrabec@siol.net>,\n\t<maxime.ripard@free-electrons.com>, <wens@csie.org>","References":"<20170920200124.20457-1-jernej.skrabec@siol.net>","From":"Jose Abreu <Jose.Abreu@synopsys.com>","Message-ID":"<5172c185-c6b5-49c3-cf0a-c3e073459346@synopsys.com>","Date":"Thu, 21 Sep 2017 10:39:42 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101\n\tThunderbird/45.4.0","MIME-Version":"1.0","In-Reply-To":"<20170920200124.20457-1-jernej.skrabec@siol.net>","X-Originating-IP":"[10.107.19.118]","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170921_024032_065705_C7099CDC ","X-CRM114-Status":"GOOD (  22.76  )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-1.9 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [198.182.47.9 listed in list.dnswl.org]\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"devicetree@vger.kernel.org, narmstrong@baylibre.com,\n\tlinux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,\n\tlinux-sunxi@googlegroups.com, hans.verkuil@cisco.com,\n\tLaurent.pinchart@ideasonboard.com, linux-media@vger.kernel.org,\n\tlinux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org,\n\ticenowy@aosc.io","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1772639,"web_url":"http://patchwork.ozlabs.org/comment/1772639/","msgid":"<68fa56f8-9878-12d0-1696-4393abe45db5@cisco.com>","list_archive_url":null,"date":"2017-09-21T09:59:27","subject":"Re: [RESEND RFC PATCH 0/7] sun8i H3 HDMI glue driver for DW HDMI","submitter":{"id":26081,"url":"http://patchwork.ozlabs.org/api/people/26081/","name":"Hans Verkuil (hansverk)","email":"hansverk@cisco.com"},"content":"On 09/21/17 11:39, Jose Abreu wrote:\n> Hi Jernej,\n> \n> On 20-09-2017 21:01, Jernej Skrabec wrote:\n>> [added media mailing list due to CEC question]\n>>\n>> This patch series adds a HDMI glue driver for Allwinner H3 SoC. For now, only\n>> video and CEC functionality is supported. Audio needs more tweaks.\n>>\n>> Series is based on the H3 DE2 patch series available on mailing list:\n>> https://urldefense.proofpoint.com/v2/url?u=http-3A__lists.infradead.org_pipermail_linux-2Darm-2Dkernel_2017-2DAugust_522697.html&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=WHDsc6kcWAl4i96Vm5hJ_19IJiuxx_p_Rzo2g-uHDKw&m=coyfcQKSr2asrHcaCeWFmAP_9nkFkRK8s7Uw5bmVei4&s=JCFaMXK1MmZ3jE745_YcqZhZkaqtc6UapGfSSapcz_s&e= \n>> (ignore patches marked with [NOT FOR REVIEW NOW] tag)\n>>\n>> Patch 1 adds support for polling plug detection since custom PHY used here\n>> doesn't support HPD interrupt.\n>>\n>> Patch 2 enables overflow workaround for v1.32a. This HDMI controller exhibits\n>> same issues as HDMI controller used in iMX6 SoCs.\n>>\n>> Patch 3 adds CLK_SET_RATE_PARENT to hdmi clock.\n>>\n>> Patch 4 adds dt bindings documentation.\n>>\n>> Patch 5 adds actual H3 HDMI glue driver.\n>>\n>> Patch 6 and 7 add HDMI node to DT and enable it where needed.\n>>\n>> Allwinner used DW HDMI controller in a non standard way:\n>> - register offsets obfuscation layer, which can fortunately be turned off\n>> - register read lock, which has to be disabled by magic number\n>> - custom PHY, which have to be initialized before DW HDMI controller\n>> - non standard clocks\n>> - no HPD interrupt\n>>\n>> Because of that, I have two questions:\n>> - Since HPD have to be polled, is it enough just to enable poll mode? I'm\n>>   mainly concerned about invalidating CEC address here.\n> \n> You mean you get no interrupt when HPD status changes? Hans can\n> answer this better but then you will need to invalidate the cec\n> physical address yourself because right now its invalidated in\n> the dw-hdmi irq handler (see dw_hdmi_irq()).\n\nThat's correct. When the HPD goes low you need to call cec_notifier_phys_addr_invalidate()\nto invalidate the physical address. This is not terribly time sensitive, i.e.\nchecking this once a second would be quick enough.\n\nRegards,\n\n\tHans","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"Wbwmc/bx\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=cisco.com header.i=@cisco.com\n\theader.b=\"cbfdwJVu\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xyXCh3Ln7z9t49\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tThu, 21 Sep 2017 20:00:00 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1duyGy-0007Sw-UL; Thu, 21 Sep 2017 09:59:57 +0000","from aer-iport-2.cisco.com ([173.38.203.52])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1duyGt-0007Np-Ei for linux-arm-kernel@lists.infradead.org;\n\tThu, 21 Sep 2017 09:59:54 +0000","from aer-iport-nat.cisco.com (HELO aer-core-1.cisco.com)\n\t([173.38.203.22])\n\tby aer-iport-2.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;\n\t21 Sep 2017 09:59:28 +0000","from [10.47.79.81] ([10.47.79.81]) (authenticated bits=0)\n\tby aer-core-1.cisco.com (8.14.5/8.14.5) with ESMTP id v8L9xRiV001663\n\t(version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NO);\n\tThu, 21 Sep 2017 09:59:28 GMT"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=kpbX0sf/HdYH8bOESUhGpOB/RqXAZh1tseEdYwGh76Y=;\n\tb=Wbwmc/bx2kD3Be\n\tH5U2FsO0RI563e0qRK5RYYAR+uHYlxHyCxcdgcu1bEcNg2e2BLK5MoPDHGuGY/rN65ngxzcXM9i9f\n\tXyeqoY8Cd5oRdZYaconQlBHTdCtSAOJXH5weNKRHKXtcmZ4+PhyPiA0VPH2GX0wGs3XCWW9qwSChO\n\tiEwYtwHrtnqSYmmo9cFOMBkffzL0GEjYkhQVKt3KBkgmMrOpbRvdT/6P0XzQy3b/jSSgRUZcHcORq\n\tz4xkme+d01lKqY+xlI4jxRtHBwUQD0rkR/shnIdVueRLJTjeiXmUcA/PWymw83j3QCUSt/C/0BFOO\n\thPzAYQVEKoBB8pFD9TnA==;","v=1; a=rsa-sha256; c=relaxed/simple;\n\td=cisco.com; i=@cisco.com; l=2251; q=dns/txt; s=iport;\n\tt=1505987991; x=1507197591;\n\th=subject:to:cc:references:from:message-id:date:\n\tmime-version:in-reply-to:content-transfer-encoding;\n\tbh=vkbQUAr2NmKJhQfmZYQiPjoyQlSQO1VTowsrqauktwc=;\n\tb=cbfdwJVuzOkrHH+n0ISeoOqCK+Bi89RtaGRI4CMujjcGep3g217rzMAm\n\tOFRrYm6dV67/8vE9qdyaajdSnKFZo7sBovsGgvGoXqmOMbY8INHrsgr1V\n\t/iOD9y3eCtUtdLRu7Xx41evoC87uUUjMGQEU+jchXg5cOELKEZLYgDUpc A=;"],"X-IronPort-AV":"E=Sophos;i=\"5.42,424,1500940800\"; d=\"scan'208\";a=\"654785411\"","Subject":"Re: [RESEND RFC PATCH 0/7] sun8i H3 HDMI glue driver for DW HDMI","To":"Jose Abreu <Jose.Abreu@synopsys.com>,\n\tJernej Skrabec <jernej.skrabec@siol.net>,\n\tmaxime.ripard@free-electrons.com, wens@csie.org","References":"<20170920200124.20457-1-jernej.skrabec@siol.net>\n\t<5172c185-c6b5-49c3-cf0a-c3e073459346@synopsys.com>","From":"Hans Verkuil <hansverk@cisco.com>","Message-ID":"<68fa56f8-9878-12d0-1696-4393abe45db5@cisco.com>","Date":"Thu, 21 Sep 2017 11:59:27 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<5172c185-c6b5-49c3-cf0a-c3e073459346@synopsys.com>","Content-Language":"en-US","X-Authenticated-User":"hansverk","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170921_025951_863375_F178FE1B ","X-CRM114-Status":"GOOD (  11.08  )","X-Spam-Score":"-14.5 (--------------)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-14.5 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/,\n\thigh trust [173.38.203.52 listed in list.dnswl.org]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-7.5 USER_IN_DEF_DKIM_WL From: address is in the default DKIM\n\twhite-list\n\t-0.0 SPF_HELO_PASS          SPF: HELO matches SPF record\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"devicetree@vger.kernel.org, narmstrong@baylibre.com,\n\tlinux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,\n\tlinux-sunxi@googlegroups.com, hans.verkuil@cisco.com,\n\tLaurent.pinchart@ideasonboard.com, linux-media@vger.kernel.org,\n\tlinux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org,\n\ticenowy@aosc.io","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1776649,"web_url":"http://patchwork.ozlabs.org/comment/1776649/","msgid":"<20170927215638.tjbllqq2twdvuivj@rob-hp-laptop>","list_archive_url":null,"date":"2017-09-27T21:56:38","subject":"Re: [RESEND RFC PATCH 4/7] dt-bindings: Document Allwinner DWC HDMI\n\tTX node","submitter":{"id":62529,"url":"http://patchwork.ozlabs.org/api/people/62529/","name":"Rob Herring (Arm)","email":"robh@kernel.org"},"content":"On Wed, Sep 20, 2017 at 10:01:21PM +0200, Jernej Skrabec wrote:\n> Add documentation about Allwinner DWC HDMI TX node, found in H3 SoC.\n> \n> Signed-off-by: Jernej Skrabec <jernej.skrabec@siol.net>\n> ---\n>  .../bindings/display/sunxi/sun4i-drm.txt           | 158 ++++++++++++++++++++-\n>  1 file changed, 157 insertions(+), 1 deletion(-)\n> \n> diff --git a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt\n> index 92512953943e..cb6aee5c486f 100644\n> --- a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt\n> +++ b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt\n> @@ -60,6 +60,40 @@ Required properties:\n>      first port should be the input endpoint. The second should be the\n>      output, usually to an HDMI connector.\n>  \n> +DWC HDMI TX Encoder\n> +-----------------------------\n> +\n> +The HDMI transmitter is a Synopsys DesignWare HDMI 1.4 TX controller IP\n> +with Allwinner's own PHY IP. It supports audio and video outputs and CEC.\n> +\n> +These DT bindings follow the Synopsys DWC HDMI TX bindings defined in\n> +Documentation/devicetree/bindings/display/bridge/dw_hdmi.txt with the\n> +following device-specific properties.\n> +\n> +Required properties:\n> +\n> +  - compatible: value must be one of:\n> +    * \"allwinner,sun8i-h3-dw-hdmi\"\n> +  - reg: two pairs of base address and size of memory-mapped region, first\n> +    for controller and second for PHY\n> +    registers.\n> +  - reg-io-width: See dw_hdmi.txt. Shall be 1.\n> +  - interrupts: HDMI interrupt number\n> +  - clocks: phandles to the clocks feeding the HDMI encoder\n> +    * iahb: the HDMI interface clock\n> +    * isfr: the HDMI module clock\n> +    * ddc: the HDMI ddc clock\n> +  - clock-names: the clock names mentioned above\n> +  - resets: phandles to the reset controllers driving the encoder\n> +    * hdmi: the reset line for the HDMI\n> +    * ddc: the reset line for the DDC\n> +  - reset-names: the reset names mentioned above\n> +\n> +  - ports: A ports node with endpoint definitions as defined in\n> +    Documentation/devicetree/bindings/media/video-interfaces.txt. The\n> +    first port should be the input endpoint. The second should be the\n> +    output, usually to an HDMI connector.\n> +\n>  TV Encoder\n>  ----------\n>  \n> @@ -255,7 +289,7 @@ Required properties:\n>    - allwinner,pipelines: list of phandle to the display engine\n>      frontends (DE 1.0) or mixers (DE 2.0) available.\n>  \n> -Example:\n> +Example 1:\n>  \n>  panel: panel {\n>  \tcompatible = \"olimex,lcd-olinuxino-43-ts\";\n> @@ -455,3 +489,125 @@ display-engine {\n>  \tcompatible = \"allwinner,sun5i-a13-display-engine\";\n>  \tallwinner,pipelines = <&fe0>;\n>  };\n> +\n> +Example 2:\n> +\n> +connector {\n> +\tcompatible = \"hdmi-connector\";\n> +\ttype = \"a\";\n> +\n> +\tport {\n> +\t\thdmi_con_in: endpoint {\n> +\t\t\tremote-endpoint = <&hdmi_out_con>;\n> +\t\t};\n> +\t};\n> +};\n> +\n> +de: display-engine {\n> +\tcompatible = \"allwinner,sun8i-h3-display-engine\";\n> +\tallwinner,pipelines = <&mixer0>;\n> +};\n> +\n> +hdmi: hdmi@1ee0000 {\n> +\tcompatible = \"allwinner,h3-dw-hdmi\";\n> +\treg = <0x01ee0000 0x10000>,\n> +\t      <0x01ef0000 0x10000>;\n> +\treg-io-width = <1>;\n> +\tinterrupts = <GIC_SPI 88 IRQ_TYPE_LEVEL_HIGH>;\n> +\tclocks = <&ccu CLK_BUS_HDMI>, <&ccu CLK_HDMI>,\n> +\t\t <&ccu CLK_HDMI_DDC>;\n> +\tclock-names = \"iahb\", \"isfr\", \"ddc\";\n> +\tresets = <&ccu RST_BUS_HDMI0>, <&ccu RST_BUS_HDMI1>;\n> +\treset-names = \"hdmi\", \"ddc\";\n> +\n> +\tports {\n> +\t\t#address-cells = <1>;\n> +\t\t#size-cells = <0>;\n> +\n> +\t\thdmi_in: port@0 {\n> +\t\t\t#address-cells = <1>;\n> +\t\t\t#size-cells = <0>;\n> +\t\t\treg = <0>;\n> +\n> +\t\t\thdmi_in_tcon0: endpoint@0 {\n> +\t\t\t\treg = <0>;\n\nYou don't need reg when there's only one.\n\nOtherwise,\n\nAcked-by: Rob Herring <robh@kernel.org>","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"lTlbxgNi\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2WrQ4rSKz9t6B\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tThu, 28 Sep 2017 07:57:10 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxKKH-0001WK-IK; Wed, 27 Sep 2017 21:57:05 +0000","from mail-pg0-f67.google.com ([74.125.83.67])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxKKD-0001P1-8I for linux-arm-kernel@lists.infradead.org;\n\tWed, 27 Sep 2017 21:57:03 +0000","by mail-pg0-f67.google.com with SMTP id m30so10480121pgn.5\n\tfor <linux-arm-kernel@lists.infradead.org>;\n\tWed, 27 Sep 2017 14:56:40 -0700 (PDT)","from localhost ([70.35.39.2]) by smtp.gmail.com with ESMTPSA id\n\tu73sm21833715pfi.58.2017.09.27.14.56.39\n\t(version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256);\n\tWed, 27 Sep 2017 14:56:39 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=9vAQnY1RsxcQYzeYB6bOnfUA1RQvOB/uGbOJkm6dTbk=;\n\tb=lTlbxgNi9sXsmM\n\taK3tu9b8IZdYfU7zY3LeMMRNQFk2hG0g+skvQJO9mnfh7L6qC7Et6vEVTyG5MArLAVwwBgJa/MYe6\n\tVNRVBwBWjTvgAS1LFhbwWfV/6Q9YIle5ijYtmfLThCGnmbkH8GEhMb/0u0t6Psmz88LYzsMyGURKM\n\tu1mAr+fq16f8AuJBmSPkV3PjWSp5CNuGGEv6odFb7kUYcM3rCQrGDL4p2ARqKiejLFWygLBu6lNFK\n\tK8Kb9QwqisngvRvOndyfvlRJ5YhoIDz3cksM++3F2WH2G7yGxwLtXJ4ZhBX30El6JkIsDlMXIxXcC\n\tuuvokMEGHIXsahEGltlg==;","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=nEGIR8jF0rUAHgGKmGvh9ttTyjfjYBmfWHgFoK3LKvY=;\n\tb=CEGi4f2rfSsmPpaMQhs8CmhHmCyi9tMdSkX3zyXwLDXuBIJ/BO/0guIdNDDakDPcft\n\tEJO4FwHC/jxMjDtMzXtwmSFL+wWNAgmgilj3EnE5qW7rTEf8tkLf6eA7TW0g4wALEx/Q\n\tcihNg3fYcat6qHevtBjgxQqmSJJN29+k12/C2ZpYVMOe3zuy5Z+4lfG5/NiP6BnkDdtO\n\t55ByX0U9jBw+nYcWyWtnhiq4tvEB8wmyktvc/LcqK5tm5cQafcw0aEIWYoQnYSFcQUA6\n\tOi0bYWE3GXasixhKlJC84QoHbVkEQzggqgYdXtTIFI31r/YL4njalURnxEBMziIw8mW3\n\tCChg==","X-Gm-Message-State":"AHPjjUiOFWKRZX0KSpJPFdBvoonWToSKG3oh4XamkN1iiRxM9mAyX695\n\tFK+Tem+IEeFiwkzw/9GEPOtGZzE=","X-Google-Smtp-Source":"AOwi7QByo9s4fTIR8BoJWuNg94Yxu7oSVWGT1fbb9gFnaEF3LumlkRvYmHP8wezztT1awFRTLiUcUg==","X-Received":"by 10.159.247.140 with SMTP id e12mr2328374pls.340.1506549400392;\n\tWed, 27 Sep 2017 14:56:40 -0700 (PDT)","Date":"Wed, 27 Sep 2017 16:56:38 -0500","From":"Rob Herring <robh@kernel.org>","To":"Jernej Skrabec <jernej.skrabec@siol.net>","Subject":"Re: [RESEND RFC PATCH 4/7] dt-bindings: Document Allwinner DWC HDMI\n\tTX node","Message-ID":"<20170927215638.tjbllqq2twdvuivj@rob-hp-laptop>","References":"<20170920200124.20457-1-jernej.skrabec@siol.net>\n\t<20170920200124.20457-5-jernej.skrabec@siol.net>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170920200124.20457-5-jernej.skrabec@siol.net>","User-Agent":"NeoMutt/20170113 (1.7.2)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170927_145701_347922_A199600E ","X-CRM114-Status":"GOOD (  15.86  )","X-Spam-Score":"-1.0 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-1.0 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t0.5 RCVD_IN_SORBS_SPAM     RBL: SORBS: sender is a spam source\n\t[74.125.83.67 listed in dnsbl.sorbs.net]\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [74.125.83.67 listed in list.dnswl.org]\n\t-0.0 RCVD_IN_MSPIKE_H3      RBL: Good reputation (+3)\n\t[74.125.83.67 listed in wl.mailspike.net]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t0.2 FREEMAIL_ENVFROM_END_DIGIT Envelope-from freemail username ends\n\tin digit (robherring2[at]gmail.com)\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (robherring2[at]gmail.com)\n\t0.0 HEADER_FROM_DIFFERENT_DOMAINS From and EnvelopeFrom 2nd level\n\tmail domains are different\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.0 RCVD_IN_MSPIKE_WL      Mailspike good senders\n\t0.2 FREEMAIL_FORGED_FROMDOMAIN 2nd level domains in From and\n\tEnvelopeFrom freemail headers are different","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"devicetree@vger.kernel.org, narmstrong@baylibre.com,\n\tlinux-sunxi@googlegroups.com, linux-kernel@vger.kernel.org,\n\tdri-devel@lists.freedesktop.org, wens@csie.org, hans.verkuil@cisco.com,\n\tLaurent.pinchart@ideasonboard.com, maxime.ripard@free-electrons.com, \n\tlinux-media@vger.kernel.org, linux-clk@vger.kernel.org,\n\tlinux-arm-kernel@lists.infradead.org, icenowy@aosc.io","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1777906,"web_url":"http://patchwork.ozlabs.org/comment/1777906/","msgid":"<51c50157-6794-852b-f89d-647b9cf06ef2@ozlabs.ru>","list_archive_url":null,"date":"2017-09-30T11:58:03","subject":"Re: [RESEND RFC PATCH 0/7] sun8i H3 HDMI glue driver for DW HDMI","submitter":{"id":7621,"url":"http://patchwork.ozlabs.org/api/people/7621/","name":"Alexey Kardashevskiy","email":"aik@ozlabs.ru"},"content":"On 21/09/17 06:01, Jernej Skrabec wrote:\n> [added media mailing list due to CEC question]\n> \n> This patch series adds a HDMI glue driver for Allwinner H3 SoC. For now, only\n> video and CEC functionality is supported. Audio needs more tweaks.\n> \n> Series is based on the H3 DE2 patch series available on mailing list:\n> http://lists.infradead.org/pipermail/linux-arm-kernel/2017-August/522697.html\n> (ignore patches marked with [NOT FOR REVIEW NOW] tag)\n> \n> Patch 1 adds support for polling plug detection since custom PHY used here\n> doesn't support HPD interrupt.\n> \n> Patch 2 enables overflow workaround for v1.32a. This HDMI controller exhibits\n> same issues as HDMI controller used in iMX6 SoCs.\n> \n> Patch 3 adds CLK_SET_RATE_PARENT to hdmi clock.\n> \n> Patch 4 adds dt bindings documentation.\n> \n> Patch 5 adds actual H3 HDMI glue driver.\n> \n> Patch 6 and 7 add HDMI node to DT and enable it where needed.\n> \n> Allwinner used DW HDMI controller in a non standard way:\n> - register offsets obfuscation layer, which can fortunately be turned off\n> - register read lock, which has to be disabled by magic number\n> - custom PHY, which have to be initialized before DW HDMI controller\n> - non standard clocks\n> - no HPD interrupt\n> \n> Because of that, I have two questions:\n> - Since HPD have to be polled, is it enough just to enable poll mode? I'm\n>   mainly concerned about invalidating CEC address here.\n> - PHY has to be initialized before DW HDMI controller to disable offset\n>   obfuscation and read lock among other things. This means that all clocks have\n>   to be enabled in glue driver. This poses a problem, since when using\n>   component model, dw-hdmi bridge uses drvdata for it's own private data and\n>   prevents glue layer to pass a pointer to unbind function, where clocks should\n>   be disabled. I noticed same issue in meson DW HDMI glue driver, where clocks\n>   are also not disabled when unbind callback is called. I noticed that when H3\n>   SoC is shutdown, HDMI output is still enabled and lastest image is shown on\n>   monitor until it is unplugged from power supply. Is there any simple solution\n>   to this?\n> \n> Chen-Yu,\n> TL Lim was unable to obtain any answer from Allwinner about HDMI clocks. I think\n> it is safe to assume that divider in HDMI clock doesn't have any effect.\n> \n> Branch based on linux-next from 1. September with integrated patches is\n> available here:\n> https://github.com/jernejsk/linux-1/tree/h3_hdmi_rfc\n\n\nOut of curiosity I tried this one and got:\n\n\n\n[    0.071711] sun4i-usb-phy 1c19400.phy: Couldn't request ID GPIO\n[    0.074809] sun8i-h3-pinctrl 1c20800.pinctrl: initialized sunXi PIO driver\n[    0.076167] sun8i-h3-r-pinctrl 1f02c00.pinctrl: initialized sunXi PIO driver\n[    0.148009] ------------[ cut here ]------------\n[    0.148035] WARNING: CPU: 0 PID: 1 at\ndrivers/clk/sunxi-ng/ccu_common.c:41 ccu_nm_set_rate+0x1d0/0x274\n[    0.148046] CPU: 0 PID: 1 Comm: swapper/0 Not tainted\n4.13.0-rc6-next-20170825-aik-aik #24\n[    0.148051] Hardware name: Allwinner sun8i Family\n[    0.148082] [<c010de6c>] (unwind_backtrace) from [<c010b260>]\n(show_stack+0x10/0x14)\n[    0.148101] [<c010b260>] (show_stack) from [<c077a464>]\n(dump_stack+0x84/0x98)\n[    0.148117] [<c077a464>] (dump_stack) from [<c011abe0>] (__warn+0xe0/0xfc)\n[    0.148132] [<c011abe0>] (__warn) from [<c011acac>]\n(warn_slowpath_null+0x20/0x28)\n[    0.148145] [<c011acac>] (warn_slowpath_null) from [<c03d1888>]\n(ccu_nm_set_rate+0x1d0/0x274)\n[    0.148161] [<c03d1888>] (ccu_nm_set_rate) from [<c03c78b4>]\n(clk_change_rate+0x19c/0x250)\n[    0.148175] [<c03c78b4>] (clk_change_rate) from [<c03c7b7c>]\n(clk_core_set_rate_nolock+0x68/0xb0)\n[    0.148187] [<c03c7b7c>] (clk_core_set_rate_nolock) from [<c03c8134>]\n(clk_set_rate+0x20/0x30)\n[    0.148202] [<c03c8134>] (clk_set_rate) from [<c03cc560>]\n(of_clk_set_defaults+0x200/0x364)\n[    0.148219] [<c03cc560>] (of_clk_set_defaults) from [<c045427c>]\n(platform_drv_probe+0x18/0xb0)\n[    0.148233] [<c045427c>] (platform_drv_probe) from [<c0452efc>]\n(driver_probe_device+0x234/0x2e8)\n[    0.148246] [<c0452efc>] (driver_probe_device) from [<c0453068>]\n(__driver_attach+0xb8/0xbc)\n[    0.148258] [<c0453068>] (__driver_attach) from [<c0451414[    1.336154]\nUnable to handle kernel NULL pointer dereference at virtual address 00000008\n\nand a bit later:\n\n[    1.995572] Rebooting in 10 seconds..\n\nOrange PI PC, script.bin.OPI-PC_1080p60_hdmi.\n\nWhat do I miss? Thanks.","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"lV8uJ4ku\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=ozlabs-ru.20150623.gappssmtp.com\n\theader.i=@ozlabs-ru.20150623.gappssmtp.com header.b=\"r9V6fAjb\"; \n\tdkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y46QY4fDRz9s06\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tSat, 30 Sep 2017 21:58:45 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dyGPm-0007Z6-KP; Sat, 30 Sep 2017 11:58:38 +0000","from mail-pf0-x241.google.com ([2607:f8b0:400e:c00::241])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dyGPh-0007Vh-Tp for linux-arm-kernel@lists.infradead.org;\n\tSat, 30 Sep 2017 11:58:36 +0000","by mail-pf0-x241.google.com with SMTP id e69so1422786pfg.4\n\tfor <linux-arm-kernel@lists.infradead.org>;\n\tSat, 30 Sep 2017 04:58:12 -0700 (PDT)","from [192.168.10.22] (124-171-134-202.dyn.iinet.net.au.\n\t[124.171.134.202]) by smtp.googlemail.com with ESMTPSA id\n\tc7sm12973689pfg.85.2017.09.30.04.58.06\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tSat, 30 Sep 2017 04:58:11 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=rP0u09y6uqIpM154N+h535lJGN4WghdXd6PUO1VSeMM=;\n\tb=lV8uJ4kuddu+bC\n\t+Fe4ZFysOZV/MhsCpH4AGvSk8ucc/PGgvLEGzk3BBJoIEL58axxpgzWWjESR7sMgZ4EoZVzhocwqt\n\trRMJL46ryUsGUBnJRMoRHlpIOQ8GhnY1bpDeq+q7rVN3Gq4qmLf6G2DUwV1SoHbhsZ/TCABAJyyqt\n\tXJmlxBsAZR13ad8JwAJdSMQj7O9WuaeCY9mHd3jwiKUE8woJbl3wnCpTsqqW17aHv3fUCj3JSfLKf\n\t4GM5hHwSW0OPFDHowVLJ2urRdT8JLupaDPD6ATuyIvvAuFeQzXHX8FiDW1iYMvOIWAuuSJLTheEO7\n\tEflMWKQg0pjPCbvv/hyg==;","v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=ozlabs-ru.20150623.gappssmtp.com; s=20150623;\n\th=subject:to:cc:references:from:message-id:date:user-agent\n\t:mime-version:in-reply-to:content-language:content-transfer-encoding; \n\tbh=MBqt/irApZzTV33VOp3NwcuagSLXqO3JDRTGgcybyHM=;\n\tb=r9V6fAjbRda1YyEtweBXlJR6BoRVTGHyC0kXsgR4SvW33VNLV6dTJga89cVsjUQM2O\n\tV0H4pVHuKMrssmuzePmKX6yD9IjA1oikbKo8NBSY2sXDmPOEEiXZo+1kIAOwLEZkmO9Y\n\tya8kRYz/PGZjH9YAq6D3nZCtp5n9St+8uMxkLqj25GiV4OcisBkWxad0wej6rrsBjqvg\n\tmvxr18MDnKVDCETDatzOyrwiyc4CeKywTiWlqdP7p/G4V4awUQamMUxf71cMzygjMAU0\n\tzr9kDcGgaN95uhvDmaoM3A5AJa26QbXemvyc0IaeZSu9icnHbjQCQtV7FW2YwOciEkwU\n\tZhFQ=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:subject:to:cc:references:from:message-id:date\n\t:user-agent:mime-version:in-reply-to:content-language\n\t:content-transfer-encoding;\n\tbh=MBqt/irApZzTV33VOp3NwcuagSLXqO3JDRTGgcybyHM=;\n\tb=q2r/PzEEByJSsMRSxMCTsiBQseORqwL8HRI/SXewTQvnPOk/FUbJ4tQk6seqnr6v6Z\n\tkqV7y7sW3AKh6r5h6zmoYJ1DbOjq3fLgi0a0wmF4n1rLtUNwQRAESxMwNAlHyDG0Bgsw\n\t2bqltN9krBwtEq7BlV59Ky/5Dqd/p6pG5aUkNRNGWy6Hm4u4jQEXOzj/DfXtf8gl8KzF\n\tsXiwR8bobneXNbSnL6sgxpTGsn5KNW4Obbwz/PBAHIgVdb15B4uqmWBGSbclgZqF08fM\n\t/tjRyDJQDMAlckWB82r/GOg0mUGqnS+1UIRYlbyeMOIbomxH9+pdCFd1R5DLtNA9jiIi\n\t8P7A==","X-Gm-Message-State":"AMCzsaXhMeglZHiNYHE+wZTDu1W56lqzMIyUBRU320j7KHfKYukN0Kyq\n\tXWjCKtE7Qjma2ZmEMqU5pTK3CQ==","X-Google-Smtp-Source":"AOwi7QCyAVsawnbxUbPN/oK62WWAu6y/dHhN38+lPAqTq+4JjZigcBNlZ2KKYFo2HAV/q6ZmAbNLmw==","X-Received":"by 10.99.95.207 with SMTP id t198mr1201462pgb.145.1506772692034; \n\tSat, 30 Sep 2017 04:58:12 -0700 (PDT)","Subject":"Re: [RESEND RFC PATCH 0/7] sun8i H3 HDMI glue driver for DW HDMI","To":"Jernej Skrabec <jernej.skrabec@siol.net>,\n\tmaxime.ripard@free-electrons.com, wens@csie.org","References":"<20170920200124.20457-1-jernej.skrabec@siol.net>","From":"Alexey Kardashevskiy <aik@ozlabs.ru>","Message-ID":"<51c50157-6794-852b-f89d-647b9cf06ef2@ozlabs.ru>","Date":"Sat, 30 Sep 2017 21:58:03 +1000","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<20170920200124.20457-1-jernej.skrabec@siol.net>","Content-Language":"en-AU","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170930_045833_998729_0D2F4222 ","X-CRM114-Status":"GOOD (  20.68  )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-1.9 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2607:f8b0:400e:c00:0:0:0:241 listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"devicetree@vger.kernel.org, narmstrong@baylibre.com,\n\tlinux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,\n\tlinux-sunxi@googlegroups.com, hans.verkuil@cisco.com,\n\tLaurent.pinchart@ideasonboard.com, linux-media@vger.kernel.org,\n\tlinux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org,\n\ticenowy@aosc.io","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1777934,"web_url":"http://patchwork.ozlabs.org/comment/1777934/","msgid":"<34246001.lKHNgTVc0h@jernej-laptop>","list_archive_url":null,"date":"2017-09-30T17:56:19","subject":"Re: [RESEND RFC PATCH 0/7] sun8i H3 HDMI glue driver for DW HDMI","submitter":{"id":70601,"url":"http://patchwork.ozlabs.org/api/people/70601/","name":"Jernej Škrabec","email":"jernej.skrabec@siol.net"},"content":"Hi,\n\nDne sobota, 30. september 2017 ob 13:58:03 CEST je Alexey Kardashevskiy \nnapisal(a):\n> On 21/09/17 06:01, Jernej Skrabec wrote:\n> > [added media mailing list due to CEC question]\n> > \n> > This patch series adds a HDMI glue driver for Allwinner H3 SoC. For now,\n> > only video and CEC functionality is supported. Audio needs more tweaks.\n> > \n> > Series is based on the H3 DE2 patch series available on mailing list:\n> > http://lists.infradead.org/pipermail/linux-arm-kernel/2017-August/522697.h\n> > tml (ignore patches marked with [NOT FOR REVIEW NOW] tag)\n> > \n> > Patch 1 adds support for polling plug detection since custom PHY used here\n> > doesn't support HPD interrupt.\n> > \n> > Patch 2 enables overflow workaround for v1.32a. This HDMI controller\n> > exhibits same issues as HDMI controller used in iMX6 SoCs.\n> > \n> > Patch 3 adds CLK_SET_RATE_PARENT to hdmi clock.\n> > \n> > Patch 4 adds dt bindings documentation.\n> > \n> > Patch 5 adds actual H3 HDMI glue driver.\n> > \n> > Patch 6 and 7 add HDMI node to DT and enable it where needed.\n> > \n> > Allwinner used DW HDMI controller in a non standard way:\n> > - register offsets obfuscation layer, which can fortunately be turned off\n> > - register read lock, which has to be disabled by magic number\n> > - custom PHY, which have to be initialized before DW HDMI controller\n> > - non standard clocks\n> > - no HPD interrupt\n> > \n> > Because of that, I have two questions:\n> > - Since HPD have to be polled, is it enough just to enable poll mode? I'm\n> > \n> >   mainly concerned about invalidating CEC address here.\n> > \n> > - PHY has to be initialized before DW HDMI controller to disable offset\n> > \n> >   obfuscation and read lock among other things. This means that all clocks\n> >   have to be enabled in glue driver. This poses a problem, since when\n> >   using component model, dw-hdmi bridge uses drvdata for it's own private\n> >   data and prevents glue layer to pass a pointer to unbind function,\n> >   where clocks should be disabled. I noticed same issue in meson DW HDMI\n> >   glue driver, where clocks are also not disabled when unbind callback is\n> >   called. I noticed that when H3 SoC is shutdown, HDMI output is still\n> >   enabled and lastest image is shown on monitor until it is unplugged\n> >   from power supply. Is there any simple solution to this?\n> > \n> > Chen-Yu,\n> > TL Lim was unable to obtain any answer from Allwinner about HDMI clocks. I\n> > think it is safe to assume that divider in HDMI clock doesn't have any\n> > effect.\n> > \n> > Branch based on linux-next from 1. September with integrated patches is\n> > available here:\n> > https://github.com/jernejsk/linux-1/tree/h3_hdmi_rfc\n> \n> Out of curiosity I tried this one and got:\n> \n> \n> \n> [    0.071711] sun4i-usb-phy 1c19400.phy: Couldn't request ID GPIO\n> [    0.074809] sun8i-h3-pinctrl 1c20800.pinctrl: initialized sunXi PIO\n> driver [    0.076167] sun8i-h3-r-pinctrl 1f02c00.pinctrl: initialized sunXi\n> PIO driver [    0.148009] ------------[ cut here ]------------\n> [    0.148035] WARNING: CPU: 0 PID: 1 at\n> drivers/clk/sunxi-ng/ccu_common.c:41 ccu_nm_set_rate+0x1d0/0x274\n> [    0.148046] CPU: 0 PID: 1 Comm: swapper/0 Not tainted\n> 4.13.0-rc6-next-20170825-aik-aik #24\n> [    0.148051] Hardware name: Allwinner sun8i Family\n> [    0.148082] [<c010de6c>] (unwind_backtrace) from [<c010b260>]\n> (show_stack+0x10/0x14)\n> [    0.148101] [<c010b260>] (show_stack) from [<c077a464>]\n> (dump_stack+0x84/0x98)\n> [    0.148117] [<c077a464>] (dump_stack) from [<c011abe0>]\n> (__warn+0xe0/0xfc) [    0.148132] [<c011abe0>] (__warn) from [<c011acac>]\n> (warn_slowpath_null+0x20/0x28)\n> [    0.148145] [<c011acac>] (warn_slowpath_null) from [<c03d1888>]\n> (ccu_nm_set_rate+0x1d0/0x274)\n> [    0.148161] [<c03d1888>] (ccu_nm_set_rate) from [<c03c78b4>]\n> (clk_change_rate+0x19c/0x250)\n> [    0.148175] [<c03c78b4>] (clk_change_rate) from [<c03c7b7c>]\n> (clk_core_set_rate_nolock+0x68/0xb0)\n> [    0.148187] [<c03c7b7c>] (clk_core_set_rate_nolock) from [<c03c8134>]\n> (clk_set_rate+0x20/0x30)\n> [    0.148202] [<c03c8134>] (clk_set_rate) from [<c03cc560>]\n> (of_clk_set_defaults+0x200/0x364)\n> [    0.148219] [<c03cc560>] (of_clk_set_defaults) from [<c045427c>]\n> (platform_drv_probe+0x18/0xb0)\n> [    0.148233] [<c045427c>] (platform_drv_probe) from [<c0452efc>]\n> (driver_probe_device+0x234/0x2e8)\n> [    0.148246] [<c0452efc>] (driver_probe_device) from [<c0453068>]\n> (__driver_attach+0xb8/0xbc)\n> [    0.148258] [<c0453068>] (__driver_attach) from [<c0451414[    1.336154]\n> Unable to handle kernel NULL pointer dereference at virtual address 00000008\n> \n\nPatch for that is already merged upstream and can be found here:\nhttps://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/\ndrivers/clk/sunxi-ng?id=62d212bdb022deeb875f92f6e376c799e3f35eca\n\n> and a bit later:\n> \n> [    1.995572] Rebooting in 10 seconds..\n\nI'm not sure about that one. Kernel config issue?\n\nRegards,\nJernej\n\n> \n> Orange PI PC, script.bin.OPI-PC_1080p60_hdmi.\n> \n> What do I miss? Thanks.\n> \n> \n> \n> --\n> Alexey","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"FYZtJM2I\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y4GN125c8z9sDB\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tSun,  1 Oct 2017 04:57:05 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dyM0Z-0002YJ-9g; Sat, 30 Sep 2017 17:56:59 +0000","from mailoutvs2.siol.net ([213.250.19.135] helo=mail.siol.net)\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dyM0U-0002WH-04 for linux-arm-kernel@lists.infradead.org;\n\tSat, 30 Sep 2017 17:56:57 +0000","from localhost (localhost [127.0.0.1])\n\tby mail.siol.net (Zimbra) with ESMTP id 388E852301E;\n\tSat, 30 Sep 2017 19:56:23 +0200 (CEST)","from mail.siol.net ([127.0.0.1])\n\tby localhost (psrvmta12.zcs-production.pri [127.0.0.1]) (amavisd-new, \n\tport 10032)\n\twith ESMTP id CHDuQ8TCwvew; Sat, 30 Sep 2017 19:56:22 +0200 (CEST)","from mail.siol.net (localhost [127.0.0.1])\n\tby mail.siol.net (Zimbra) with ESMTPS id 5C3FE5251C2;\n\tSat, 30 Sep 2017 19:56:22 +0200 (CEST)","from jernej-laptop.localnet (cpe-86-58-68-135.ftth.triera.net\n\t[86.58.68.135]) (Authenticated sender: 031275009)\n\tby mail.siol.net (Zimbra) with ESMTPA id 62B4452301E;\n\tSat, 30 Sep 2017 19:56:20 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:\n\tMessage-ID:Date:Subject:To:From:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=6+JOOaSNalikkYQiki5APUSms4Kp0lbZb2YYFeML5E4=;\n\tb=FYZtJM2ImcOsTO\n\tlJRC0ACKBNwmrMOc4XrRM65EhD/bFT6RjOTd+JVyuDmI1FjYwwnPStQtv2wR5HuJ6d/pYnmRIV0A5\n\t30DWGda2KasnGg289UbmMwdfUrH/HW/zsTYAptuV+ihwk0EpVpqkoHLPnQ1LL0RsLeXfiQVHWzwdM\n\tA1FwjVrilH982sGH1V8fL/zkgTDPk/h2cALQL8VBcDVTJWHohYlUL1P1OtjyP1Z2v5DPRCIgEuUqg\n\tA7Ro9xF550n+JFY7a21VUjJT0SV+Z2EwIsOB/aw2Z9aE4ycye4fjqIslHDCElrlcj8bYUh7Cj2OFK\n\to1sQPyZLkYaeVOUscZew==;","X-Virus-Scanned":"amavisd-new at psrvmta12.zcs-production.pri","From":"Jernej =?utf-8?q?=C5=A0krabec?= <jernej.skrabec@siol.net>","To":"Alexey Kardashevskiy <aik@ozlabs.ru>","Subject":"Re: [RESEND RFC PATCH 0/7] sun8i H3 HDMI glue driver for DW HDMI","Date":"Sat, 30 Sep 2017 19:56:19 +0200","Message-ID":"<34246001.lKHNgTVc0h@jernej-laptop>","In-Reply-To":"<51c50157-6794-852b-f89d-647b9cf06ef2@ozlabs.ru>","References":"<20170920200124.20457-1-jernej.skrabec@siol.net>\n\t<51c50157-6794-852b-f89d-647b9cf06ef2@ozlabs.ru>","MIME-Version":"1.0","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170930_105654_247661_A381D1B5 ","X-CRM114-Status":"GOOD (  22.34  )","X-Spam-Score":"-2.6 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-2.6 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at http://www.dnswl.org/,\n\tlow trust [213.250.19.135 listed in list.dnswl.org]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"devicetree@vger.kernel.org, narmstrong@baylibre.com,\n\tlinux-sunxi@googlegroups.com, linux-kernel@vger.kernel.org,\n\tdri-devel@lists.freedesktop.org, wens@csie.org, hans.verkuil@cisco.com,\n\tLaurent.pinchart@ideasonboard.com, maxime.ripard@free-electrons.com, \n\tlinux-media@vger.kernel.org, linux-clk@vger.kernel.org,\n\tlinux-arm-kernel@lists.infradead.org, icenowy@aosc.io","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1777970,"web_url":"http://patchwork.ozlabs.org/comment/1777970/","msgid":"<ccaebb62-ad28-4905-5ebf-bcdc1e7ebf54@ozlabs.ru>","list_archive_url":null,"date":"2017-10-01T01:46:38","subject":"Re: [RESEND RFC PATCH 0/7] sun8i H3 HDMI glue driver for DW HDMI","submitter":{"id":7621,"url":"http://patchwork.ozlabs.org/api/people/7621/","name":"Alexey Kardashevskiy","email":"aik@ozlabs.ru"},"content":"On 01/10/17 04:56, Jernej Škrabec wrote:\n> Hi,\n> \n> Dne sobota, 30. september 2017 ob 13:58:03 CEST je Alexey Kardashevskiy \n> napisal(a):\n>> On 21/09/17 06:01, Jernej Skrabec wrote:\n>>> [added media mailing list due to CEC question]\n>>>\n>>> This patch series adds a HDMI glue driver for Allwinner H3 SoC. For now,\n>>> only video and CEC functionality is supported. Audio needs more tweaks.\n>>>\n>>> Series is based on the H3 DE2 patch series available on mailing list:\n>>> http://lists.infradead.org/pipermail/linux-arm-kernel/2017-August/522697.h\n>>> tml (ignore patches marked with [NOT FOR REVIEW NOW] tag)\n>>>\n>>> Patch 1 adds support for polling plug detection since custom PHY used here\n>>> doesn't support HPD interrupt.\n>>>\n>>> Patch 2 enables overflow workaround for v1.32a. This HDMI controller\n>>> exhibits same issues as HDMI controller used in iMX6 SoCs.\n>>>\n>>> Patch 3 adds CLK_SET_RATE_PARENT to hdmi clock.\n>>>\n>>> Patch 4 adds dt bindings documentation.\n>>>\n>>> Patch 5 adds actual H3 HDMI glue driver.\n>>>\n>>> Patch 6 and 7 add HDMI node to DT and enable it where needed.\n>>>\n>>> Allwinner used DW HDMI controller in a non standard way:\n>>> - register offsets obfuscation layer, which can fortunately be turned off\n>>> - register read lock, which has to be disabled by magic number\n>>> - custom PHY, which have to be initialized before DW HDMI controller\n>>> - non standard clocks\n>>> - no HPD interrupt\n>>>\n>>> Because of that, I have two questions:\n>>> - Since HPD have to be polled, is it enough just to enable poll mode? I'm\n>>>\n>>>   mainly concerned about invalidating CEC address here.\n>>>\n>>> - PHY has to be initialized before DW HDMI controller to disable offset\n>>>\n>>>   obfuscation and read lock among other things. This means that all clocks\n>>>   have to be enabled in glue driver. This poses a problem, since when\n>>>   using component model, dw-hdmi bridge uses drvdata for it's own private\n>>>   data and prevents glue layer to pass a pointer to unbind function,\n>>>   where clocks should be disabled. I noticed same issue in meson DW HDMI\n>>>   glue driver, where clocks are also not disabled when unbind callback is\n>>>   called. I noticed that when H3 SoC is shutdown, HDMI output is still\n>>>   enabled and lastest image is shown on monitor until it is unplugged\n>>>   from power supply. Is there any simple solution to this?\n>>>\n>>> Chen-Yu,\n>>> TL Lim was unable to obtain any answer from Allwinner about HDMI clocks. I\n>>> think it is safe to assume that divider in HDMI clock doesn't have any\n>>> effect.\n>>>\n>>> Branch based on linux-next from 1. September with integrated patches is\n>>> available here:\n>>> https://github.com/jernejsk/linux-1/tree/h3_hdmi_rfc\n>>\n>> Out of curiosity I tried this one and got:\n>>\n>>\n>>\n>> [    0.071711] sun4i-usb-phy 1c19400.phy: Couldn't request ID GPIO\n>> [    0.074809] sun8i-h3-pinctrl 1c20800.pinctrl: initialized sunXi PIO\n>> driver [    0.076167] sun8i-h3-r-pinctrl 1f02c00.pinctrl: initialized sunXi\n>> PIO driver [    0.148009] ------------[ cut here ]------------\n>> [    0.148035] WARNING: CPU: 0 PID: 1 at\n>> drivers/clk/sunxi-ng/ccu_common.c:41 ccu_nm_set_rate+0x1d0/0x274\n>> [    0.148046] CPU: 0 PID: 1 Comm: swapper/0 Not tainted\n>> 4.13.0-rc6-next-20170825-aik-aik #24\n>> [    0.148051] Hardware name: Allwinner sun8i Family\n>> [    0.148082] [<c010de6c>] (unwind_backtrace) from [<c010b260>]\n>> (show_stack+0x10/0x14)\n>> [    0.148101] [<c010b260>] (show_stack) from [<c077a464>]\n>> (dump_stack+0x84/0x98)\n>> [    0.148117] [<c077a464>] (dump_stack) from [<c011abe0>]\n>> (__warn+0xe0/0xfc) [    0.148132] [<c011abe0>] (__warn) from [<c011acac>]\n>> (warn_slowpath_null+0x20/0x28)\n>> [    0.148145] [<c011acac>] (warn_slowpath_null) from [<c03d1888>]\n>> (ccu_nm_set_rate+0x1d0/0x274)\n>> [    0.148161] [<c03d1888>] (ccu_nm_set_rate) from [<c03c78b4>]\n>> (clk_change_rate+0x19c/0x250)\n>> [    0.148175] [<c03c78b4>] (clk_change_rate) from [<c03c7b7c>]\n>> (clk_core_set_rate_nolock+0x68/0xb0)\n>> [    0.148187] [<c03c7b7c>] (clk_core_set_rate_nolock) from [<c03c8134>]\n>> (clk_set_rate+0x20/0x30)\n>> [    0.148202] [<c03c8134>] (clk_set_rate) from [<c03cc560>]\n>> (of_clk_set_defaults+0x200/0x364)\n>> [    0.148219] [<c03cc560>] (of_clk_set_defaults) from [<c045427c>]\n>> (platform_drv_probe+0x18/0xb0)\n>> [    0.148233] [<c045427c>] (platform_drv_probe) from [<c0452efc>]\n>> (driver_probe_device+0x234/0x2e8)\n>> [    0.148246] [<c0452efc>] (driver_probe_device) from [<c0453068>]\n>> (__driver_attach+0xb8/0xbc)\n>> [    0.148258] [<c0453068>] (__driver_attach) from [<c0451414[    1.336154]\n>> Unable to handle kernel NULL pointer dereference at virtual address 00000008\n>>\n> \n> Patch for that is already merged upstream and can be found here:\n> https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/\n> drivers/clk/sunxi-ng?id=62d212bdb022deeb875f92f6e376c799e3f35eca\n\nLovely, it works, thanks!\n\n\n>> and a bit later:\n>>\n>> [    1.995572] Rebooting in 10 seconds..\n> \n> I'm not sure about that one. Kernel config issue?\n\n\n\nYup, I did not have CMA enabled.","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"t508JXv9\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=ozlabs-ru.20150623.gappssmtp.com\n\theader.i=@ozlabs-ru.20150623.gappssmtp.com header.b=\"VYLAxFnS\"; \n\tdkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y4Sq22tWPz9sPk\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tSun,  1 Oct 2017 12:47:38 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dyTLx-0001ab-DD; Sun, 01 Oct 2017 01:47:33 +0000","from mail-pg0-x242.google.com ([2607:f8b0:400e:c05::242])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dyTLZ-0001N2-Bo for linux-arm-kernel@lists.infradead.org;\n\tSun, 01 Oct 2017 01:47:31 +0000","by mail-pg0-x242.google.com with SMTP id u27so2132544pgn.1\n\tfor <linux-arm-kernel@lists.infradead.org>;\n\tSat, 30 Sep 2017 18:46:46 -0700 (PDT)","from [192.168.10.22] (124-171-134-202.dyn.iinet.net.au.\n\t[124.171.134.202]) by smtp.googlemail.com with ESMTPSA id\n\tz69sm12330947pfl.86.2017.09.30.18.46.40\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tSat, 30 Sep 2017 18:46:45 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=uOGs6qwSiXXvcKq01ZspaHYAnxvN27T0UM93PP3xGp0=;\n\tb=t508JXv9ZUF7Ol\n\twCePhKmtuxY7GhMuMuVxNhCAu8WQ9w9GJJIkO3iD4Oa6/gCPf2MYj5LYMjPkSEcv0CCR0GEOo+5Uj\n\tuQEOoEdfpsxYu/qW/YaB6gJUptfi0EB4XKB68Sg+HE9STjedfeT16CbrA2HyxXK0tEEJ036/btmfk\n\tdUu2JH3068kKcZ1tlsVsmPQGQzmPw5YiRrl6LXFD3uCfv/onjIS+fqaPhC9EHhLTk4QyRVa3NRVVH\n\tsjKix7ZuAHeW9o22CMuB7AZovqZR1gdkGXhV16kFNbtjvqETs9rysdWgEUVEXBdynxZ88Q7ur2bvq\n\tFJSB4jw+UIpVKCmBGY6A==;","v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=ozlabs-ru.20150623.gappssmtp.com; s=20150623;\n\th=subject:to:cc:references:from:message-id:date:user-agent\n\t:mime-version:in-reply-to:content-language:content-transfer-encoding; \n\tbh=9c2P1QxN0vNreRzBpW7sJLSu3/oJLkgE9SC7MQcJImU=;\n\tb=VYLAxFnSndUhRj0iedZ+6yKSnJIPKPOIJNpmDKwrN56yfY26C5lkLaQec1Q2n4SYAa\n\tMT5sQYH9G0dtS4u3Ul50IIdflBBR9JAzTZVweILroJ2xS6eCVYFq8/V/KXtrO+/i2EBs\n\tZsMQdj5WYqMbBnRnVsOB87bKr00iaBA7agbGo+z92bMo8SC/grepDfRyiMBSfNy1ItBG\n\tHo+hPi/npYNtg6P8ect9eXG9x9E1zIZ+u+cqvrzQPQ4pL5G5ynvR3DPEGLTklXEr8uzM\n\t+EEW5nja5qVsyrd4ekxVdSifkV4KMaS3DIChPAVICsu3QZiTwDYf29CwGilz2AIN+7Ab\n\t/ohw=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:subject:to:cc:references:from:message-id:date\n\t:user-agent:mime-version:in-reply-to:content-language\n\t:content-transfer-encoding;\n\tbh=9c2P1QxN0vNreRzBpW7sJLSu3/oJLkgE9SC7MQcJImU=;\n\tb=Lc4kqpgggot+Etj3I2VKD+VxPkvtlXPnVaJz8I0BZ1iuQOgkuWcP328i9Gv9LZRQ69\n\tQq1nYf/ueQZAfDatBKgd9BrynkZGLoTN8xvq0n/ycqPW/ynKMTLQacwSHLlkNYSuB5iY\n\tPaUaS7KN7cbQmFDJtFTV2SWrggJ2HOtEK8yzslQxEd3zTLt2Aa3OyVcz9nuFn7OAZyn8\n\tp3JRi4tsxygrPYXZbeNqafvjxce6jM5JaweF5QPmuHL541nb1VOhTVkfaxmdk2ZzJfQ/\n\tyk+k6wmCAsZsmz8ns0NIrzgvYIroasenEONQHfGdcV+GUZTdNS7snSmfpLAqOt9m3e9G\n\tMGQA==","X-Gm-Message-State":"AHPjjUiXRvMWyHRN7LTr2WMCvDLgWmvrXqeBI5btBLjDyhEfdZ6nE5LG\n\thUkWm61KGIMksygSS9fRBx+vAg==","X-Google-Smtp-Source":"AOwi7QDQgzWq105VvGL+wl++hEissYSLVO54hVtinM2jAq8GIZK21c11mdwETMBhj7Vka2+3opwv+Q==","X-Received":"by 10.159.252.66 with SMTP id t2mr10736317plz.31.1506822406054; \n\tSat, 30 Sep 2017 18:46:46 -0700 (PDT)","Subject":"Re: [RESEND RFC PATCH 0/7] sun8i H3 HDMI glue driver for DW HDMI","To":"=?utf-8?q?Jernej_=C5=A0krabec?= <jernej.skrabec@siol.net>","References":"<20170920200124.20457-1-jernej.skrabec@siol.net>\n\t<51c50157-6794-852b-f89d-647b9cf06ef2@ozlabs.ru>\n\t<34246001.lKHNgTVc0h@jernej-laptop>","From":"Alexey Kardashevskiy <aik@ozlabs.ru>","Message-ID":"<ccaebb62-ad28-4905-5ebf-bcdc1e7ebf54@ozlabs.ru>","Date":"Sun, 1 Oct 2017 12:46:38 +1100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.2.1","MIME-Version":"1.0","In-Reply-To":"<34246001.lKHNgTVc0h@jernej-laptop>","Content-Language":"en-AU","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170930_184725_284305_1A054A3A ","X-CRM114-Status":"GOOD (  18.05  )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-1.9 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2607:f8b0:400e:c05:0:0:0:242 listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"devicetree@vger.kernel.org, narmstrong@baylibre.com,\n\tlinux-sunxi@googlegroups.com, linux-kernel@vger.kernel.org,\n\tdri-devel@lists.freedesktop.org, wens@csie.org, hans.verkuil@cisco.com,\n\tLaurent.pinchart@ideasonboard.com, maxime.ripard@free-electrons.com, \n\tlinux-media@vger.kernel.org, linux-clk@vger.kernel.org,\n\tlinux-arm-kernel@lists.infradead.org, icenowy@aosc.io","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1797675,"web_url":"http://patchwork.ozlabs.org/comment/1797675/","msgid":"<20171102071942.GN30645@codeaurora.org>","list_archive_url":null,"date":"2017-11-02T07:19:42","subject":"Re: [RESEND RFC PATCH 3/7] clk: sunxi: Add CLK_SET_RATE_PARENT flag\n\tfor H3 HDMI clock","submitter":{"id":6071,"url":"http://patchwork.ozlabs.org/api/people/6071/","name":"Stephen Boyd","email":"sboyd@codeaurora.org"},"content":"On 09/20, Jernej Skrabec wrote:\n> When setting the HDMI clock of H3, the PLL_VIDEO clock needs to be set.\n> \n> Add CLK_SET_RATE_PARENT flag for H3 HDMI clock.\n> \n> Signed-off-by: Jernej Skrabec <jernej.skrabec@siol.net>\n> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>\n> ---\n\nAcked-by: Stephen Boyd <sboyd@codeaurora.org>","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"KgAbjL9W\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=codeaurora.org header.i=@codeaurora.org\n\theader.b=\"bL8xrLHj\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key)\n\theader.d=codeaurora.org header.i=@codeaurora.org header.b=\"GM1//mqw\"; \n\tdkim-atps=neutral","pdx-caf-mail.web.codeaurora.org;\n\tdmarc=none (p=none dis=none) header.from=codeaurora.org","pdx-caf-mail.web.codeaurora.org;\n\tspf=none smtp.mailfrom=sboyd@codeaurora.org"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3ySGh00Kdgz9t2V\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tThu,  2 Nov 2017 18:20:16 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1eA9nS-0008Jt-Iz; Thu, 02 Nov 2017 07:20:14 +0000","from smtp.codeaurora.org ([198.145.29.96])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1eA9nL-0007Xq-0X for linux-arm-kernel@lists.infradead.org;\n\tThu, 02 Nov 2017 07:20:12 +0000","by smtp.codeaurora.org (Postfix, from userid 1000)\n\tid AAA3160867; Thu,  2 Nov 2017 07:19:46 +0000 (UTC)","from localhost (i-global254.qualcomm.com [199.106.103.254])\n\t(using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits))\n\t(No client certificate requested)\n\t(Authenticated sender: sboyd@smtp.codeaurora.org)\n\tby smtp.codeaurora.org (Postfix) with ESMTPSA id BBBEA6085C;\n\tThu,  2 Nov 2017 07:19:43 +0000 (UTC)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=bnJ23t08UoUsOUzvUbPTV9BN01DSiQ7p6f8unzo0Byg=;\n\tb=KgAbjL9W2AbI/4\n\tMVu1jW/+DroTclKvxVZgdYr2fzJML57ioTw1JmktDfbGIELsmZm7KE43UrPDZOdBj5Rw289tvIxUn\n\tuKEKPIoDFO2r0G49c+B+3WXMGHrD9ea3OvKkGDss8rjZ+bW6Z4kJrD8waPjozBfbft/CQ0UW0QvGq\n\tqRkaq9GMHSmvCDUaqZC4IuPlLZif9mrIaTSP247NAleZfgAwV2xoZPA5eWbzeMKHiDyEEN9BrsDHo\n\tkI+3AcNgYhtzoxrSurfnhPQ2aBhFRWbQAdjg3pJyNsFYu7uR4aEk2gCSw8aOPlMG4WINGdE/v/WWq\n\tqzIRzWjTaWhYjXN3R0Yw==;","v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org;\n\ts=default; t=1509607186;\n\tbh=OEnFs+E46nwCMp/pRNP9OygHW6bGIieUhL4x4PpdOcc=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=bL8xrLHjf5nZ4Glw8iq9lT5ZkxREjOKrka47sh7kA36o2NBifveAisB6eHfk4Fdml\n\tlTIJ6pMsdHyl/UQKK4OaxapYNX2nIWSK3hraIraVY+KH4LUDDx/9sZr9O+3Ga9zsuk\n\tRZHeui4kxkxV2LF0UElgfvqDLld4pF2tHjNQFcn4=","v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org;\n\ts=default; t=1509607183;\n\tbh=OEnFs+E46nwCMp/pRNP9OygHW6bGIieUhL4x4PpdOcc=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=GM1//mqwomvbJoNoma0Igcq7K8gwyRcYKCNV2NuiS7RchHz5cCAnZ0WsKpwa47f5R\n\tNyvdh/31Zf7B/sZsc9DpBM+xkyRzfgj2azDMdCkVXIIvPq/7SZRbu1nUiyvBcKiwXH\n\t6VjY6Ts9cCNlr43yTRE2nUikdD/TKkgJVb+s/eSQ="],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on\n\tpdx-caf-mail.web.codeaurora.org","X-Spam-Level":"","X-Spam-Status":"No, score=-2.8 required=2.0 tests=ALL_TRUSTED,BAYES_00,\n\tDKIM_SIGNED,\n\tT_DKIM_INVALID autolearn=no autolearn_force=no version=3.4.0","DMARC-Filter":"OpenDMARC Filter v1.3.2 smtp.codeaurora.org BBBEA6085C","Date":"Thu, 2 Nov 2017 00:19:42 -0700","From":"Stephen Boyd <sboyd@codeaurora.org>","To":"Jernej Skrabec <jernej.skrabec@siol.net>","Subject":"Re: [RESEND RFC PATCH 3/7] clk: sunxi: Add CLK_SET_RATE_PARENT flag\n\tfor H3 HDMI clock","Message-ID":"<20171102071942.GN30645@codeaurora.org>","References":"<20170920200124.20457-1-jernej.skrabec@siol.net>\n\t<20170920200124.20457-4-jernej.skrabec@siol.net>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170920200124.20457-4-jernej.skrabec@siol.net>","User-Agent":"Mutt/1.5.21 (2010-09-15)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171102_002007_165563_FE6E843B ","X-CRM114-Status":"UNSURE (   9.34  )","X-CRM114-Notice":"Please train this message.","X-Spam-Score":"-4.3 (----)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-4.3 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at http://www.dnswl.org/,\n\tmedium trust [198.145.29.96 listed in list.dnswl.org]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"devicetree@vger.kernel.org, narmstrong@baylibre.com,\n\tlinux-sunxi@googlegroups.com, linux-kernel@vger.kernel.org,\n\tdri-devel@lists.freedesktop.org, wens@csie.org, hans.verkuil@cisco.com,\n\tLaurent.pinchart@ideasonboard.com, maxime.ripard@free-electrons.com, \n\tlinux-media@vger.kernel.org, linux-clk@vger.kernel.org,\n\tlinux-arm-kernel@lists.infradead.org, icenowy@aosc.io","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}}]